burtonator wrote:
>
> > I got something working but I needed to modify ECS and Turbine to make
> > it work.
>
> Post diffs so that we can see.
>
> > Since these changes are not backward compatible with the current Turbine
> > or ECS codebase, I don't know if the patches will be welcome by the projects
> > maintainer and contributors.
>
> Without diffs no one can tell :)
>
You want some diffs ? you got some diffs... :)
This is the RunData change :
RCS file:
/products/cvs/turbine/turbine/src/java/org/apache/turbine/util/RunData.java,v
retrieving revision 1.33
diff -r1.33 RunData.java
193c193,204
< this.page = new Document();
---
> this.page = new HTMLDocument();
> return this.page;
> }
>
> /**
> Sets the page
> */
> public Document setPage(Document doc)
> {
> pageSet = true;
> if ( doc != null )
> this.page = doc;
The ECS change is much harder to diff because it's mainly moving files around. What I
did is :
- rename org.apache.ecs.Document to org.apache.ecs.html.HTMLDocument
- implement a new org.apache.ecs.Document which stores all content in an
ElementContainer
- modify the HTMLDocument, XMLDocument and WMLDocument classes so that they
are subclasses of Document.
This has the following impact on Turbine code, I had to modify all the classes using
the RunData.getPage() to cast the returned Document to the expected document type,
eg
RCS file:
/products/cvs/turbine/turbine/src/java/org/apache/turbine/modules/layouts/DefaultLayout.java,v
retrieving revision 1.3
diff -r1.3 DefaultLayout.java
83c83
< data.getPage().getBody().addElement( topNav );
---
> ((HTMLDocument)data.getPage()).getBody().addElement( topNav );
--
Rapha�l Luta - [EMAIL PROTECTED]
--
--------------------------------------------------------------
Please read the FAQ! <http://java.apache.org/faq/>
To subscribe: [EMAIL PROTECTED]
To unsubscribe: [EMAIL PROTECTED]
Archives and Other: <http://java.apache.org/main/mail.html>
Problems?: [EMAIL PROTECTED]
