#2 Fix for the temporaryXUpdateTree being left around.

open
nobody
xupdate (2)
6
2005-07-28
2004-08-30
byrne
No

Very simple fix for
/org/xmldb/xupdate/lexus/commands/InsertCommand.java
that removes the source line that was inserting a
tempory node to the end of the DOM tree.

Unfortantely this means patching four files.
/org/xmldb/xupdate/lexus/commands/InsertCommand.java
and its three children: AppendCommand.java,
InsertAfterCommand.java, and InsertBeforeCommand.java.

Attached is the first patch.

Discussion

  • byrne

    byrne - 2004-08-30

    Improved patch for just org/xmldb/xupdate/lexus/commands/InsertCommand.java

     
  • byrne

    byrne - 2004-08-30

    Logged In: YES
    user_id=1112156

    My simple fix proved to introduced problems with
    InsertVariable not working correctly.

    The issue was that if a node couldn't be selected then a
    exception was being thrown in selectNodes causing the
    various InsertCommands to exit before cleaning themselves up.

    This patch just overides selectNodes and catches any
    exceptions being thrown by super.selectNodes and does some
    quick clean up on the document.

     
  • byrne

    byrne - 2004-11-22

    Logged In: YES
    user_id=1112156

    Use this patch on the latest CVS head. Haven't tried or
    tested with anything else.

     
  • byrne

    byrne - 2005-07-28

    Logged In: YES
    user_id=1112156

    Any chance of getting this looked at?

     
  • byrne

    byrne - 2005-07-28
    • priority: 5 --> 6
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks