Uploaded image for project: 'Click'
  1. Click
  2. CLK-118

Table Column's don't escape their output

    Details

    • Type: New Feature
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: core
    • Labels:
      None

      Description

      Columns don't escape by default This is ognl I have to use to make a table column escape it's value:

      table.addColumn(new Column("@org.apache.commons.lang.StringEscapeUtils@escapeHtml(title)","Title"));

      I propose escaping should be the default and that users should use a call like column.setEscaping(false) to turn it off it not required. This might cause some minor backwards compatibility issues but escaping by default is much more desirable than the small inconvenience this might cause. I might see if I can come up with a patch in the next few days.

        Attachments

          Activity

            People

            • Assignee:
              medgar Malcolm Edgar
              Reporter:
              gstamp Glen Stampoultzis
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: