[
https://issues.apache.org/jira/browse/AXIOM-359?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Eric Johnson updated AXIOM-359:
-------------------------------
Description:
When using Axiom with the Santuario port to GenXDM
(https://code.google.com/a/apache-extras.org/p/santuario-genxdm/ ), we noticed
that certain C14N tests failed. These tests failed because the constructor to
OMProcessingInstructionImpl does:
this.value = (value == null) ? null : value.trim();
In fact, it should do this:
this.value = (value == null) ? null : value;
(That is, not call trim() ). Note that the "setValue()" method does not call
"trim()" so the behavior is *also* inconsistent. At least the "setValue()"
method appears to provide a work-around for this issue.
was:
When using Axiom with the Santuario port to GenXDM
(https://code.google.com/a/apache-extras.org/p/santuario-genxdm/), we noticed
that certain C14N tests failed. These tests failed because the constructor to
OMProcessingInstructionImpl does:
this.value = (value == null) ? null : value.trim();
In fact, it should do this:
this.value = (value == null) ? null : value;
(That is, not call trim() ). Note that the "setValue()" method does not call
"trim()" so the behavior is *also* inconsistent. At least the "setValue()"
method appears to provide a work-around for this issue.
Updated the description to fix the link to santuario-genxdm
> OMProcessingInstructionImpl incorrectly trims the value passed in the
> constructor
> ---------------------------------------------------------------------------------
>
> Key: AXIOM-359
> URL: https://issues.apache.org/jira/browse/AXIOM-359
> Project: Axiom
> Issue Type: Bug
> Components: LLOM
> Affects Versions: 1.2.11
> Reporter: Eric Johnson
> Priority: Minor
>
> When using Axiom with the Santuario port to GenXDM
> (https://code.google.com/a/apache-extras.org/p/santuario-genxdm/ ), we
> noticed that certain C14N tests failed. These tests failed because the
> constructor to OMProcessingInstructionImpl does:
> this.value = (value == null) ? null : value.trim();
> In fact, it should do this:
> this.value = (value == null) ? null : value;
> (That is, not call trim() ). Note that the "setValue()" method does not call
> "trim()" so the behavior is *also* inconsistent. At least the "setValue()"
> method appears to provide a work-around for this issue.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
