DisplayTag

TagConstants.AMPERSAND is unnecessarily escaped

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Won't Fix
  • Affects Version/s: 1.0 RC2
  • Fix Version/s: None
  • Component/s: Paging/Sorting
  • Labels:
    None
  • Application server:
    tomcat 5

Description

TagConstants.AMPERSAND = "&";
should be
TagConstants.AMPERSAND = "&";

the only place this constant is used is in Href.java where it is
used to seperate url request parameters. the "&" seperating request parameters need not be escaped.
it affects the urls built by displaytag in for example sortable="true" headings.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
fabrizio giustina added a comment -
ampersand MUST be escaped in links to produce valid html (from html 2.0 onwards):
http://www.w3.org/MarkUp/html-spec/html-spec_foot.html#FOOT26
Show
fabrizio giustina added a comment - ampersand MUST be escaped in links to produce valid html (from html 2.0 onwards): http://www.w3.org/MarkUp/html-spec/html-spec_foot.html#FOOT26
Hide
Permalink
Wouter Scheele added a comment -
thanks for setting me straight, more current info regarding xhtml:
http://www.w3.org/TR/xhtml1/#C_12
Show
Wouter Scheele added a comment - thanks for setting me straight, more current info regarding xhtml: http://www.w3.org/TR/xhtml1/#C_12

People

  • Reporter:
    Wouter Scheele
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: