sshyran / genxdm

Automatically exported from code.google.com/p/genxdm
0 stars 0 forks source link

demonstrate xquery update facility through api and bridge #34

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
/branches/xquf

Trimmed processors and other stuff out.

Original issue reported on code.google.com by aale...@gmail.com on 17 Nov 2010 at 3:34

GoogleCodeExporter commented 8 years ago
Pretty much complete at r70.  Take a look.

Original comment by aale...@gmail.com on 17 Nov 2010 at 8:14

GoogleCodeExporter commented 8 years ago
See this thread:

http://groups.google.com/group/genxdm/browse_thread/thread/39c70b5ca8650673

Summarizing: the initial implementation introduced a new parameter, F, to 
handle DOM stupidity.  It almost certainly shouldn't have, and Eric pushed 
back.  The discussion suggested that DOM handle its own conflation of 
"document" and "factory" internally, and that all APIs have "getNodeFactory" on 
the context and "getFactoryForContext" on model and cursor.

So I'm marking this "open" again, although I'm going to mark it "fixed" again 
in a minute.  :-)

Original comment by aale...@gmail.com on 29 Nov 2010 at 4:46

GoogleCodeExporter commented 8 years ago
Changes committed at r71, on xquf branch.  Marking "fixed" again, and we'll see 
if this one makes it.  :-)

Summary of fixes:

1) MutableContext lost the "F" parameter defined for the method call 
getNodeFactory, and the argument to the method.

2) NodeFactory lost its "F" parameter, and two methods associated with 
"getting" and "setting" the API factory.

3) MutableModel and MutableCursor added a method, getFactoryForContext (with N 
argument for model, of course).

4) the bridgekit (MutableCursorOnMutableModel) was modified (hmmm, could 
actually cache the factory there) to add the new method (delegating to model).

5) the DOM bridge was modified to implement the new methods and keep track of 
which Document was associated with this NodeFactory.

Sending mail to the thread mentioned above in a moment, too.

Original comment by aale...@gmail.com on 29 Nov 2010 at 4:51

GoogleCodeExporter commented 8 years ago

Original comment by eric%tib...@gtempaccount.com on 29 Apr 2011 at 11:15