--

CommentFileSizeAuthor
#1 724788-user.test.patch10.52 KBboombatower
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

boombatower’s picture

boombatower CreditAttribution: boombatower commented
Assigned: Unassigned » boombatower
Status: Active » Needs review
FileSize
724788-user.test.patch10.52 KB

This was discovered whilst attempting #500866: [META] remove t() from assert message.

Patch should be self explainitory, conditions should be merged...much less if () with no brace and another if block is straight up poor style.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Needs review » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.