drupal_set_title does a check_plain call by default, so check_plain calls inside the drupal_set_title function call are not necessary.

CommentFileSizeAuthor
authorize_remove_check_plain.patch1.29 KBbblake

Comments

bblake’s picture

bblake commented
Status: Active » Needs review
febbraro’s picture

febbraro commented

subscribe

Status: Needs review » Needs work

The last submitted patch, authorize_remove_check_plain.patch, failed testing.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Needs work » Needs review

Patch is clean, totally unrelated to that test fail. I blame testbot.

Queue for retest.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented

authorize_remove_check_plain.patch queued for re-testing.

droplet’s picture

droplet commented
Version: 7.0-rc1 » 7.x-dev
Status: Needs review » Reviewed & tested by the community
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks!

Status: Fixed » Closed (fixed)

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