Opened 4 years ago

Closed 4 years ago

#18505 closed defect (invalid)

dojox.html.entites decode should support upper and lower case versions of the entity

Reported by: Jared Jurkiewicz Owned by:
Priority: undecided Milestone: tbd
Component: General Version: 1.10.4
Keywords: Cc:
Blocked By: Blocking:

Description

Looking at the code for the dojox/html/entities component, I noticed it was not using case insensitive matching. It really should be. Will provide a patch/pull request for this.

Change History (4)

comment:2 Changed 4 years ago by Jared Jurkiewicz

I need to do more testing on the pull, just wanted to get an initial out there.

comment:3 Changed 4 years ago by Jared Jurkiewicz

I just got reminded why I didn't use case insensitive matching before, because there are a chunk of entities where case absolutely matters:

http://www.w3.org/MarkUp/html3/latin1.html

Several latin ones.

Á is not the same as á

So, it's wrong to do case intensive matching in general. Closing this as invalid.

comment:4 Changed 4 years ago by Jared Jurkiewicz

Resolution: invalid
Status: newclosed
Note: See TracTickets for help on using tickets.