Unsanitizied should be spelt Unsanitized

CommentFileSizeAuthor
#9 typo-in-comment-2205159-9.patch961 bytesdineshw
#2 drupal-typo-2205159-2.patch974 bytesmandar.harkare

Comments

mandar.harkare’s picture

mandar.harkare commented
Assigned: Unassigned » mandar.harkare
Issue tags: +Assigned
mandar.harkare’s picture

mandar.harkare commented
StatusFileSize
new drupal-typo-2205159-2.patch974 bytes
mandar.harkare’s picture

mandar.harkare commented
Assigned: mandar.harkare » Unassigned
Status: Active » Needs review

Added the patch.

rlmumford’s picture

rlmumford
Primary language English
Location Manchester
commented
Status: Needs review » Reviewed & tested by the community

Looks fine to me.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 2: drupal-typo-2205159-2.patch, failed testing.

The last submitted patch, 2: drupal-typo-2205159-2.patch, failed testing.

scor’s picture

scor commented

@mandar.harkare your patch format is wrong. Did you use Git to roll the patch? https://drupal.org/node/707484

mandar.harkare’s picture

mandar.harkare commented

No, I used PHPStrom.

dineshw’s picture

dineshw commented
StatusFileSize
new typo-in-comment-2205159-9.patch961 bytes

creating patch with using git diff on 8.x branch
$ git diff > [description]-[issue-number]-[comment-number].patch
$ git diff > typo-in-comment-2205159-9.patch

dineshw’s picture

dineshw commented
Status: Needs work » Needs review
scor’s picture

scor commented
Status: Needs review » Reviewed & tested by the community

@mandar.harkare looks like phpStorm doesn't generate legit patch and might need to be tweaked. I use phpStorm but always revert to the command line to roll patches. Most people use the command line or a tool like one of those: https://drupal.org/node/777182

In any case, @dineshw's patch looks good to me.

webchick’s picture

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

Good catch!

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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