--

CommentFileSizeAuthor
#1 724788-user.test.patch10.52 KBboombatower

Comments

boombatower’s picture

boombatower commented
Assigned: Unassigned » boombatower
Status: Active » Needs review
StatusFileSize
new 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 commented
Status: Needs review » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

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