Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Comment | File | Size | Author |
---|---|---|---|
#6 | dblog-1711310-6.patch | 1.08 KB | kid_icarus |
#6 | interdiff.txt | 581 bytes | kid_icarus |
#4 | dblog-1711310-4.patch | 1.08 KB | xjm |
#4 | interdiff-dblog.txt | 1.02 KB | xjm |
#1 | drupal-1711310-2.patch | 964 bytes | kid_icarus |
Comments
Comment #1
kid_icarus CreditAttribution: kid_icarus commentedSeems simple enough :)
I followed the directions from the meta issue and looked at https://drupal.org/files/drupal-1711434-2.patch
Comment #2
tim.plunkettGreat, thanks.
Comment #3
xjmNeeds doxygen fix as discussed in IRC. :)
Comment #4
xjmIn addition to adding the docblock, I removed the docblock for the
setUp()
method. (It needed to be changed anyway to remove the mention of enabling modules, and the standards dictate that they have no docblock.)Comment #5
droplet CreditAttribution: droplet commentedcan it add a line between doc & code
Comment #6
kid_icarus CreditAttribution: kid_icarus commentedSure :)
Comment #7
xjmYep, that looks good to me. Thanks @droplet and @kid_icarus!
Comment #8
tim.plunkett