ActionResponseImpl does exactly what the Javadoc says with render parameter validation. The problem is the Javadoc is inconsistent, setRenderParameter(String, String) says neither the name or value can be null but setRenderParameters(String, String[]) and setRenderParameters(Map) just state that neither the key or values String[] can be null. With this documentation having a null value for a parameter is possible simply by calling setRenderParameters("foo", new String[] { null }). Is this valid and if so how should it be encoded in the URL? Should additional checks be added to the setRenderParameters methods to check for null values in the String[]?

-Eric

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature



Reply via email to