Hi Pavitra, ok. Not sure I really got everything here...
Once I apply the "IllegalStateException.patch", and I run the "pprDemos.jspx" (there is no ajaxPPRDemos.jspx; not in the demo, nor in the patch), I get the following JS alert() Error, when I do some ajax stuff (e.g. clicking the "partialSubmit Button" button in the pprDemos.jspx page): (I had to modify the pprDemos.jspx page by adding <h:outputScript /> to trigger the JS rendering, for the Ajax library) => malformed XML: Top level node must be one of: changes, redirect, error, received: #text instead I am running on => Ubuntu 9.10 and Firefox is 3.5.5. A screenshot of the response is attached to this mail I also don't get any server-side error regarding the IllegalStateException. Are you talking about a specific page, which you forgot to include in your page ? Or am I just missing something here ? Let me know :-) PS: I only CC'd you as I want to make sure you receive the attached image... ;-) Greetings, Matthias On Wed, Dec 2, 2009 at 9:01 PM, Pavitra Subramaniam <[email protected]> wrote: > Hi, > > I have provided a patch which fixes issue 1649. The fix is to update the > Trinidad ResponseWriter classes to override the new APIs added to > javax.faces.context.ResponseWriter in JSF2.0. I am working on a project that > uses the JSF 2.0 Ajax channel over Trinidad's and without these changes, the > partial response never gets built properly. > > The changes made are: > ResponseWriterDecorator.java (now extends ResponseWriterWrapper. Ideally we > should remove ResponseWriterDecorator entirely and just use JSF2.0 > ResponseWriterWrapper) > HtmlResponseWriter.java (overrides 2 new methods added to > javax.faces.context.ResponseWriter but never really gives a compilation > error). > XmlResponseWriter.java (same as above) > XhtmlResponseWriter.java (-do-) > > If there are no questions regarding this change, I would like to submit my > patch for approval. > > Thanks > Pavitra -- Matthias Wessendorf blog: http://matthiaswessendorf.wordpress.com/ sessions: http://www.slideshare.net/mwessendorf twitter: http://twitter.com/mwessendorf
<<attachment: ajax.png>>
