There is a call to check_plain() in:

- core/modules/action/action.views_execution.inc

CommentFileSizeAuthor
#5 2196797-3.patch866 bytesdivesh.kumar
#3 2196797-2.patch899 bytesdivesh.kumar
#1 2196797-1.patch812 bytesdivesh.kumar

Comments

divesh.kumar’s picture

divesh.kumar commented
Assigned: Unassigned » divesh.kumar
Status: Active » Needs review
StatusFileSize
new 2196797-1.patch812 bytes

Patch Attached.

thedavidmeister’s picture

thedavidmeister commented
Status: Needs review » Needs work

please use namespace aliases, see the parent thread issue for more information.

divesh.kumar’s picture

divesh.kumar commented
Status: Needs work » Needs review
StatusFileSize
new 2196797-2.patch899 bytes
thedavidmeister’s picture

thedavidmeister commented
Status: Needs review » Needs work

Views Drupal\Component\Utility\String::checkPlain()s the column label, so we need to match that.

We could probably make this a little more eloquent while we're here, and we can just say String::checkPlain() in the comments.

divesh.kumar’s picture

divesh.kumar commented
Status: Needs work » Needs review
StatusFileSize
new 2196797-3.patch866 bytes
thedavidmeister’s picture

thedavidmeister commented
Status: Needs review » Reviewed & tested by the community
JiriK’s picture

JiriK commented
Assigned: divesh.kumar » Unassigned
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

Status: Fixed » Closed (fixed)

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