DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=890>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=890

transf.getOutputProperty("{bogus-name") returns null; should throw exception

[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From [EMAIL PROTECTED]  2003-10-16 20:16 -------
Patch was committed to Apache CVS head branch by Brian Minchau.
Patch was not committed to xslt20-compiled branch because this is a Xalan-J 
interpretive only fix.

The patch makes sure a non standard property has a key with only one '{' and 
only one '}' and that the '{' is first thing in the key.

One could be stricter about the valid keys which are clearly defined in the
serializers OutputPropertiesFactory class with constants:
OutputPropertiesFactory.S_KEY_INDENT_AMOUNT
OutputPropertiesFactory.S_KEY_CONTENT_HANDLER
OutputPropertiesFactory.S_KEY_ENTITIES
OutputPropertiesFactory.S_USE_URL_ESCAPING
OutputPropertiesFactory.S_OMIT_META_TAG

This would also require 5 other "OLD" constants for the same keys with the old
URL in front (http://xml.apache.org/xslt) rather than the current one
(http://xml.apache.org/xalan) and one could check against all of them, but the
testcase in TransformerAPITest seems pretty clear that it expects null to come
back rather than an exception thown, so I leave the fix as is.

Reply via email to