[prev in list] [next in list] [prev in thread] [next in thread] 

List:       webservices-general
Subject:    [jira] [Updated] (AXIOM-359) OMProcessingInstructionImpl
From:       "Eric Johnson (JIRA)" <jira () apache ! org>
Date:       2011-03-29 23:03:06
Message-ID: 456016840.20139.1301439786020.JavaMail.tomcat () hel ! zones ! apache ! org
[Download RAW message or body]


     [ 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: dev-unsubscribe@ws.apache.org
For additional commands, e-mail: dev-help@ws.apache.org


[prev in list] [next in list] [prev in thread] [next in thread]