[gatein-issues] [JBoss JIRA] (GTNPC-108) BaseURLImpl.FORMATS shouldn't use null to indicate false for escapeXML value

BaseURLImpl.FORMATS shouldn't use null to indicate false for escapeXML value ---------------------------------------------------------------------------- Key: GTNPC-108 URL: https://issues.jboss.org/browse/GTNPC-108 Project: GateIn Portlet Container Issue Type: Feature Request Affects Versions: 2.4.1.Final Reporter: Chris Laprun Assignee: Chris Laprun Fix For: 2.4.2.Final Context: JSR-286 specifies that portlet URLs are now XML encoded by default. URLFormat allows user to control how a Portlet URL is rendered, in particular with respect to XML escaping. However, BaseURLImpl uses an internal array (FORMATS) that uses null instead of false to that is passed along when portlets want to *not* XML-escape URLs. Since URLFormat uses a Boolean instead of a boolean and that the default value for escapeXML needs to be true per the spec, using null instead of Boolean.FALSE is confusing because null indicates absence of value which would indicate using whatever the default one is (in this case, true).