Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Status: Active » Needs review
FileSize
776 bytes
Dave Reid’s picture

Incorrectly formatted the patch.

dww’s picture

Status: Needs review » Needs work
Issue tags: +Needs tests

Excellent, thanks! Tested locally, confirmed the bug, and verified the patch fixes it. Committed and pushed to master:

http://drupal.org/commitlog/commit/5296/ba49ec0b316a831057c2d3913aa0656e...

However, it'd be great to add simpletests for this (and the rest of this nasty module). ;) There's not even a comment_alter_taxonomy.test file at all yet, so I don't fault you for not adding any with your initial patch. But, if you wanted to help make d.o more stable, I'd love to see another patch in here to add tests.

Thanks!
-Derek