Commons Lang
  1. Commons Lang
  2. LANG-21

[lang] escapeXML() -> Not escaping low characters

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 2.1
    • Fix Version/s: 2.2
    • Component/s: None
    • Labels:
      None
    • Environment:

      Operating System: other
      Platform: Other

      Description

      Low characters such as #18 are not escaped, and therefore are mixed together
      with other characteres as if there were normal characteres such as 'a', '1' etc.

      Possibly solved in other versions?

        Activity

        Hide
        Henri Yandell added a comment -
            • This bug has been marked as a duplicate of 38210 ***
        Show
        Henri Yandell added a comment - This bug has been marked as a duplicate of 38210 ***
        Hide
        Henri Yandell added a comment -

        Oops, not a dupe. It's at the opposite end of the spectrum.

        Show
        Henri Yandell added a comment - Oops, not a dupe. It's at the opposite end of the spectrum.
        Hide
        Henri Yandell added a comment -

        There's been discussion on this on the user mailing list.

        Show
        Henri Yandell added a comment - There's been discussion on this on the user mailing list.
        Hide
        Henri Yandell added a comment -

        We have two reports currently, one that we don't escape low characters and should do; and one that we do escape high characters and shouldn't do.

        I'm marking this as WONTFIX based on Phil's thoughts on the commons-user mailing list. It's easier to add the code to escape than it would be to undo it.

        The escaping high characters issue will be fixed to not escape.

        Show
        Henri Yandell added a comment - We have two reports currently, one that we don't escape low characters and should do; and one that we do escape high characters and shouldn't do. I'm marking this as WONTFIX based on Phil's thoughts on the commons-user mailing list. It's easier to add the code to escape than it would be to undo it. The escaping high characters issue will be fixed to not escape.

          People

          • Assignee:
            Unassigned
            Reporter:
            David López
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development