|
From: William G. T. Jr. <wg...@ru...> - 2004-12-13 00:10:46
|
Nick Lothian wrote: > Someone (I think it was Rainer?) sent in a patch for > SimplePortletFormController to fix the problem he discusses below. I don't > think this has been applied, but it looks like a good change to me. I don't recall this patch going by...do you still have it? > (BTW, my name is spelt Lothian - not Lothiam. Could > http://opensource.atlassian.com/confluence/spring/display/JSR168/Home be > corrected?) fixed. sorry about that. later. Bill > > > Nick > > >>-----Original Message----- >>From: William G. Thompson, Jr. [mailto:wg...@ru...] >>Sent: Friday, 10 December 2004 11:31 PM >>To: spr...@li... >>Subject: Re: [Springframework-developer] Portlet support (again) >>Importance: Low >> >> >>Folks, >> >>I have updated the PortletMVC[1] project page to reflect the current >>status. At this point we need to indentify what is left to do to get >>out of the Sandbox...here is what is on my mind: >> >>Are we happy with the ViewServlet solution/work-around? >> >>Are we Feature Complete? >> >>Is the Code Quality up to snuff for Spring? >> >>later. >>Bill >> >>[1]http://opensource.atlassian.com/confluence/spring/display/J >>SR168/Home >> >> >>Rainer Schmitz wrote: >> >>>Nick Lothian wrote: >>> >>> >>>>Portlet support (in the sandbox) has been fairly stable >> >>for the past >> >>>>couple >>>>of months. I've been finding it works well and I think a >> >>few other people >> >>>>have used it successfully, too. >>> >>> >>>Is it stable or is just no-one working at it? >>> >>>Anyway, I'm using it and made quite a few changes mainly regarding >>>forms. Also I added a PortletControllerMapping to map by request >>>parameters. >>> >>>One general problem: >>>In the current CVS version form handling takes place totally in the >>>render phase, request parameters being forwarded by the >> >>action phase. >> >>>This approach does not work in all cases. E.g. if you want >> >>to redirect >> >>>to another URL the response.sendRedirect(URL) call has to >> >>be made on the >> >>>ActionResponse. >>> >>>The code is working for me, but not production ready and >> >>certainly needs >> >>>some clenup. >>> >>>Cheers, >>>Rainer >>> |