dev@javaserverfaces.java.net

Re: Seeking Review: JAVASERVERFACES-3404-fileUpload

From: manfred riem <manfred.riem_at_oracle.com>
Date: Mon, 15 Sep 2014 11:30:39 -0500

Hi there,

As an HTML input control is not supposed to be saving state between
request the JSF component backing this behavior should not either.
That's the reason why the setTransient(true) call is in there.

So the setTransient(true) needs to stay.

Manfred

On 9/15/14, 11:14 AM, Edward Burns wrote:
> Hello Team,
>
> http://java.net/jira/browse/JAVASERVERFACES-3404 multiple file uploads.
>
> SECTION: Modified Files
> ----------------------------
> M jsf-ri/src/main/java/com/sun/faces/renderkit/html_basic/FileRenderer.java
>
> - In decode() remove the component.setTransient(true). I'm not sure why
> this was in there in the first place, but it's been there since the
> initial revision of the file. I re-ran all the automated tests and
> they all passed, however, the file upload tests are, in fact, not
> running due to some difficulties with HtmlUnit that we are counting on
> Manfred fixing.
>
> I executed this test manually and found that it worked.
>
> M test/web-profile/renderKit/basic/src/test/java/com/sun/faces/test/webprofile/renderKit/basic/Spec802IT.java
> A test/web-profile/renderKit/basic/src/main/webapp/uploadMultipleTimes.xhtml
>
> - Manual test only. Note that the other methods in this file need to
> have @Test.
>
>