You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by so...@apache.org on 2008/03/07 01:41:18 UTC
svn commit: r634482 - in
/myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/upload:
ActionUploadRequestWrapper.java FileUploadConfiguratorImpl.java
Author: sobryan
Date: Thu Mar 6 16:41:16 2008
New Revision: 634482
URL: http://svn.apache.org/viewvc?rev=634482&view=rev
Log:
TRINIDAD-995: FileUpload sets render parameters in portal
* Removed the call to setRenderParameter
* Added a Portlet Bridge annotation to the "applied flag"
that will not be preserved between requests. This will
be needed for ResourceRequest's..
Modified:
myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/upload/ActionUploadRequestWrapper.java
myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/upload/FileUploadConfiguratorImpl.java
Modified: myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/upload/ActionUploadRequestWrapper.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/upload/ActionUploadRequestWrapper.java?rev=634482&r1=634481&r2=634482&view=diff
==============================================================================
--- myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/upload/ActionUploadRequestWrapper.java (original)
+++ myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/upload/ActionUploadRequestWrapper.java Thu Mar 6 16:41:16 2008
@@ -51,9 +51,6 @@
_extractedParams.putAll(params);
_encoding = super.getCharacterEncoding();
-
- //add these params to the render request
- _response.setRenderParameters(_extractedParams);
}
/**
Modified: myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/upload/FileUploadConfiguratorImpl.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/upload/FileUploadConfiguratorImpl.java?rev=634482&r1=634481&r2=634482&view=diff
==============================================================================
--- myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/upload/FileUploadConfiguratorImpl.java (original)
+++ myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/upload/FileUploadConfiguratorImpl.java Thu Mar 6 16:41:16 2008
@@ -25,6 +25,9 @@
import java.util.Map;
import javax.faces.context.ExternalContext;
+
+import javax.portlet.faces.annotation.ExcludeFromManagedRequestScope;
+
import javax.servlet.http.HttpServletRequest;
import org.apache.myfaces.trinidad.config.Configurator;
@@ -74,7 +77,7 @@
@SuppressWarnings("unchecked")
static public void apply(ExternalContext context)
{
- context.getRequestMap().put(_APPLIED, Boolean.TRUE);
+ context.getRequestMap().put(_APPLIED, AppliedClass.APPLIED);
}
/* (non-Javadoc)
@@ -231,6 +234,13 @@
//return the origional external context
return externalContext;
}
+
+ //This will ensure the property is removed on the next request
+ @ExcludeFromManagedRequestScope
+ static private class AppliedClass
+ {
+ static public final AppliedClass APPLIED = new AppliedClass();
+ }
static private class TempUploadedFile implements UploadedFile
{
@@ -276,5 +286,6 @@
static private final String _APPLIED = FileUploadConfiguratorImpl.class.getName()+".APPLIED";
static private final TrinidadLogger _LOG = TrinidadLogger.createTrinidadLogger(FileUploadConfiguratorImpl.class);
static private final String _PARAMS = FileUploadConfiguratorImpl.class.getName()+".PARAMS";
+
private long _maxAllowedBytes = 1L << 27;
}