https://issues.apache.org/bugzilla/show_bug.cgi?id=49707
Thomas Deweese <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO --- Comment #5 from Thomas Deweese <[email protected]> 2010-08-22 11:14:05 EDT --- (In reply to comment #4) > The bug I encountered that is a consequence of the mistake in > - You say your code create a new attribute for me. Yes, but so I have as a > result two attributes with the same name? No the new value replaces the old value. > - In my code, the bug arise when the attribute is an Id attribute like > (xml:id). > The consequence of your error was that the map 'elementsById' in your class > "AbstractDocument" was not well updated. Finally, my call to "getElementById" > on the document gave bad results. Have to tried your test with the current SVN version of Batik? I can't reproduce your bug as described below with it. I ask because the code in the 'wrong' branch could just as easily be done by users (similar calls are available in the public DOM) in which case this would fail for them as well, so this bug is more serious than the performance issue. > Anyway, I suppose you will fix the agreed error in "setAttributeNS"? Yes, but I want to make sure that you agree that the xml:id bug is fixe in current SVN first, otherwise this simply hides the real bug. > > Otherwise I would make a test like that: > - create a document D with an element E with attribute xml:id="id1" > - then I would call: E.setAttributeNS( ..., "xml:id", "id2"); > - and finally I would test the result of a call to D.getElementById("id1") > Guess it is non null and return elt E? This doesn't happen for me with current SVN Batik. -- Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
