ode-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Karthick Sankarachary (JIRA)" <j...@apache.org>
Subject [jira] Commented: (ODE-664) Namespace declarations not being copied in ASSIGN.replaceElement
Date Wed, 16 Sep 2009 21:11:57 GMT

    [ https://issues.apache.org/jira/browse/ODE-664?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12756227#action_12756227
] 

Karthick Sankarachary commented on ODE-664:
-------------------------------------------

We do adhere to the RE (replace-element-properties) rule that requires us to "replace the
element at the destination with a copy of the entire element at the source, including [children]
and [attribute] properties.", as you tell by looking at the calls to "importNode" that precede
that line. 

The line you are referring to ensures that  the namespaces defined in the to-spec carry over
to the copy of the target element that we create in ASSIGN.replaceElement.

> Namespace declarations not being copied in ASSIGN.replaceElement
> ----------------------------------------------------------------
>
>                 Key: ODE-664
>                 URL: https://issues.apache.org/jira/browse/ODE-664
>             Project: ODE
>          Issue Type: Bug
>          Components: BPEL Runtime
>    Affects Versions: 1.3.3, 1.3.4
>            Reporter: Mark Ford
>            Assignee: Karthick Sankarachary
>            Priority: Critical
>
> The following line in ASSIGN.replaceElement looks wrong:
>         DOMUtils.copyNSContext(ptr, replacement); 
> I think it should be:
>         DOMUtils.copyNSContext(src, replacement); 
> The rationale is that the RE strategy should copy the attributes from the source element
to the target element. Without this change, you may end up with undeclared namespace prefixes
in the output of your BPEL.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message