Noticed a couple of stray nbsp characters lurking in block module.

CommentFileSizeAuthor
#2 nbsp_2.patch1.22 KBtr
nbsp.patch1.22 KBjody lynn

Comments

tr’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Needs backport to D7

I verified the above patch applies and removes the non-breaking spaces (UTF-8 character C2A0) that appear in the code. Patch also applies to D7 HEAD and should be committed there too.

I also searched all the rest of the Drupal source code and found no other unintentional uses of non-breaking spaces. (There are two other instances where non-breaking space characters were *intentionally* used, in includes/unicode.entities.inc and in modules/simpletest/tests/common.test - these instances should not be and are not affected by the patch.)

Above patch is in -p0 format. I will re-roll it if the testbot complains.

tr’s picture

Status: Reviewed & tested by the community » Needs review
StatusFileSize
new1.22 KB

Reposting exact patch from the original issue to try to the the testbot to run.

tr’s picture

Status: Needs review » Reviewed & tested by the community
dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x and 8.x.

Status: Fixed » Closed (fixed)
Issue tags: -Needs backport to D7

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

grahamvalue’s picture

Issue summary: View changes

Deleting comment.