Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

problem with DomContentHandler [SWS-389] #541

Closed
gregturn opened this issue Jul 8, 2008 · 1 comment
Closed

problem with DomContentHandler [SWS-389] #541

gregturn opened this issue Jul 8, 2008 · 1 comment
Assignees
Labels
Milestone

Comments

@gregturn
Copy link
Member

@gregturn gregturn commented Jul 8, 2008

Alessandro Palumbo opened SWS-389 and commented

Everytime the function appendChild on a Node is used , the return value should be set to the original node (the one passed as argument). This must be done as Jboss create it's own element and use it as a wrapper around the one has been passed in the function.
So , for example, the code in the function startElement(..) should be changed from:

Node parent = getParent();
Element element = document.createElementNS(uri, qName);
...........(more code here)
parent.appendChild(element);
elements.add(element);

to

Node parent = getParent();
Element element = document.createElementNS(uri, qName);
...........(more code here)
element = parent.appendChild(element);
elements.add(element);

And so on everywhere appendChild is used.


Affects: 1.5.3

Referenced from: commits 7d50d57

@gregturn
Copy link
Member Author

@gregturn gregturn commented May 4, 2012

Arjen Poutsma commented

Closing old issues

@gregturn gregturn closed this May 4, 2012
@gregturn gregturn added the type: bug label Sep 22, 2020
@gregturn gregturn added this to the 1.5.4 milestone Sep 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.