Return-Path: Delivered-To: apmail-cocoon-dev-archive@www.apache.org Received: (qmail 73430 invoked from network); 20 Dec 2003 14:50:34 -0000 Received: from daedalus.apache.org (HELO mail.apache.org) (208.185.179.12) by minotaur-2.apache.org with SMTP; 20 Dec 2003 14:50:34 -0000 Received: (qmail 78003 invoked by uid 500); 20 Dec 2003 14:50:27 -0000 Delivered-To: apmail-cocoon-dev-archive@cocoon.apache.org Received: (qmail 77811 invoked by uid 500); 20 Dec 2003 14:50:26 -0000 Mailing-List: contact dev-help@cocoon.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: list-post: Reply-To: dev@cocoon.apache.org Delivered-To: mailing list dev@cocoon.apache.org Received: (qmail 77798 invoked from network); 20 Dec 2003 14:50:26 -0000 Received: from unknown (HELO smtp1.xs4all.be) (195.144.64.135) by daedalus.apache.org with SMTP; 20 Dec 2003 14:50:26 -0000 Received: from outerthought.org (195-144-088-003.dyn.adsl.xs4all.be [195.144.88.3]) (authenticated bits=0) by smtp1.xs4all.be (8.12.9/8.12.9) with ESMTP id hBKEoQOi001554 for ; Sat, 20 Dec 2003 15:50:27 +0100 Message-ID: <3FE461B1.7020808@outerthought.org> Date: Sat, 20 Dec 2003 15:50:25 +0100 From: Marc Portier Organization: Outerthought User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.5) Gecko/20031013 Thunderbird/0.3 X-Accept-Language: en-us, en MIME-Version: 1.0 To: dev@cocoon.apache.org Subject: Re: [cforms] add lenient setting to context binding References: <20031219145118.70676.qmail@web41905.mail.yahoo.com> In-Reply-To: <20031219145118.70676.qmail@web41905.mail.yahoo.com> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N X-Spam-Rating: minotaur-2.apache.org 1.6.2 0/1000/N Timothy Larson wrote: > In JXPathBindingBase.java, was this line removed on purpose: > jxpc.setFactory(new DOMFactory()); > nope, that's me being a bit clumsy :-( > Perhaps this is the cause of this users's continuing problems: > http://marc.theaimsgroup.com/?l=xml-cocoon-users&m=107176523405519&w=2 > I think so too, thx for pointing out fixing now... -marc= > Should it be added to makeJXPathContext() like this: > private JXPathContext makeJXPathContext(Object objModel) { > JXPathContext jxpc; > if (!(objModel instanceof JXPathContext)) { > jxpc = JXPathContext.newContext(objModel); > jxpc.setLenient(true); > //-- Add the following two lines --- > if (objModel instanceof Node) > jxpc.setFactory(new DOMFactory()); > //---------------------------------- > } else { > jxpc = (JXPathContext) objModel; > } > return jxpc; > } > > For reference, this snippet in revision 1.5: > JXPathContext jxpc; > if (!(objModel instanceof JXPathContext)) { > jxpc = JXPathContext.newContext(objModel); > > if (objModel instanceof Node) { > jxpc.setFactory(new DOMFactory()); > } > } else { > jxpc = (JXPathContext) objModel; > } > > was changed to this in revision 1.6: > JXPathContext jxpc = makeJXPathContext(objModel); > > WDTY? > > --Tim Larson > > > __________________________________ > Do you Yahoo!? > New Yahoo! Photos - easier uploading and sharing. > http://photos.yahoo.com/ > -- Marc Portier http://outerthought.org/ Outerthought - Open Source, Java & XML Competence Support Center Read my weblog at http://blogs.cocoondev.org/mpo/ mpo@outerthought.org mpo@apache.org