Assertion message generators in at least at assertRaw() use both the @raw placeholder (which in itself escapes the data) and the check_plain() function on the $raw value, thus double escaping the string. Probably applies to methods beyond assertRaw().

Found while writing tests for l10n_server (#560132: Write tests).

CommentFileSizeAuthor
#1 572564-double-escape.patch1.6 KBboombatower
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

boombatower’s picture

Project: SimpleTest » Drupal core
Version: 6.x-2.x-dev » 7.x-dev
Component: Code » simpletest.module
Assigned: Unassigned » boombatower
Status: Active » Needs review
FileSize
1.6 KB

Good catch.

Dries’s picture

Status: Needs review » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

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