In the list of actions on admin/config/system/actions/manage, the action label is displayed without check_plain(). I don't think action labels should be able to contain rich text.

The attached screen shot shows the action label on admin/config/system/actions/manage and on admin/structure/trigger/node — note the difference.

CommentFileSizeAuthor
actions-check-plain-1.patch1.12 KBc960657
action.png52.26 KBc960657

Comments

sivaji_ganesh_jojodae’s picture

sivaji_ganesh_jojodae commented

The patch looks good for me.

brianV requested that failed test be re-tested.

brianV’s picture

brianV commented
Priority: Minor » Normal
Status: Needs review » Reviewed & tested by the community

I was able to confirm the bug, and that it is fixed by the attached patch. Plus, a rerun on the tests came back all green.

RTBC.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed! Thanks.

Status: Fixed » Closed (fixed)
Issue tags: -quickfix

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

Issue tags: +

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.