Uploaded image for project: 'Commons Lang'
  1. Commons Lang
  2. LANG-169

[lang][patch] Implementation of escape/unescapeHtml methods with Writer

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 2.2
    • None
    • None
    • Operating System: other
      Platform: Other

    • 35366

    Description

      I've implemented the escapeHtml and unescapeHtml methods that accept Writer
      instances in the StringEscapeUtils class. The patches are are in the three
      forthcoming attachments. One of the patch files is an update to the JUnit test.

      I ran the regression tests and they all worked.

      Note: The patch for the 'Entities.java' file seems to include some changes I
      didn't actually make, but I couldn't get SVN's diff to work as expected.

      Attachments

        Activity

          nbeyer Nathan Beyer added a comment -

          Created an attachment (id=15412)
          Patch for the 'Entities.java' file.

          nbeyer Nathan Beyer added a comment - Created an attachment (id=15412) Patch for the 'Entities.java' file.
          nbeyer Nathan Beyer added a comment -

          Created an attachment (id=15413)
          Patch for the 'StringEscapeUtils.java' file.

          nbeyer Nathan Beyer added a comment - Created an attachment (id=15413) Patch for the 'StringEscapeUtils.java' file.
          nbeyer Nathan Beyer added a comment -

          Created an attachment (id=15414)
          Patch for the 'StringEscapeUtilsTest.java' file.

          nbeyer Nathan Beyer added a comment - Created an attachment (id=15414) Patch for the 'StringEscapeUtilsTest.java' file.
          stevencaswell@apache.org Steven Caswell added a comment -

          Applied patches minus the unrelated diffs to Entities

          stevencaswell@apache.org Steven Caswell added a comment - Applied patches minus the unrelated diffs to Entities

          People

            Unassigned Unassigned
            nbeyer Nathan Beyer
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: