Return-Path: Delivered-To: apmail-cocoon-dev-archive@cocoon.apache.org Received: (qmail 57798 invoked by uid 500); 19 Jul 2003 07:32:16 -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 57777 invoked from network); 19 Jul 2003 07:32:15 -0000 Received: from unknown (HELO pulse.betaversion.org) (217.158.110.65) by daedalus.apache.org with SMTP; 19 Jul 2003 07:32:15 -0000 Received: (qmail 13792 invoked from network); 19 Jul 2003 07:32:24 -0000 Received: from unknown (HELO apache.org) (stefano@127.0.0.1) by pulse.betaversion.org with SMTP; 19 Jul 2003 07:32:24 -0000 Date: Sat, 19 Jul 2003 02:34:29 -0500 Subject: Re: Improving (?) the RequestGenerator Content-Type: text/plain; charset=US-ASCII; format=flowed Mime-Version: 1.0 (Apple Message framework v552) From: Stefano Mazzocchi To: dev@cocoon.apache.org Content-Transfer-Encoding: 7bit In-Reply-To: <02a401c34d0b$338be110$49020a0a@hw0393> Message-Id: <6E5C1FC4-B9BB-11D7-AD03-000393D2CB02@apache.org> X-Mailer: Apple Mail (2.552) X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N On Friday, Jul 18, 2003, at 04:01 America/Guayaquil, Guido Casper wrote: > Hi all, > > I want to extend RequestGenerator so that request parameters which > names > start with "html:" are parsed (very much like it already parses request > parameters which names start with xml:" see: > http://marc.theaimsgroup.com/?l=xml-cocoon-dev&m=105174146229060&w=2 ) > but > with JTidy. > > This would allow easy integration of HTML editors like "htmlArea" > http://sourceforge.net/projects/itools-htmlarea/ > into the sitemap. > > I already submitted a patch to bugzilla. > I'm however unsure where this RequestGenerator should go since it has a > dependancy on JTidy. Maybe it should be renamed and go in the HTML > block > besides the HTMLGenerator. But it doesn't really make sense to have 2 > RequestGenerators. > > Hope this will be useful for others as well. I welcome the patch, but dislike the fact that RequestGenerator ends up being in a block. What about HTMLRequestGenerator that extends RequestGenerator and put that into the block? (the xml: parsing is general enough to remain there, I think) -- Stefano.