You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by mg...@apache.org on 2014/08/04 11:40:33 UTC

[1/2] git commit: Java 7 diamonds.

Repository: wicket
Updated Branches:
  refs/heads/master ebd01006f -> 38d6b4a9c


Java 7 diamonds.

No functional changes


Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/34b46ca6
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/34b46ca6
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/34b46ca6

Branch: refs/heads/master
Commit: 34b46ca6214e5682e2ebbfefac3023970fd52c0d
Parents: ebd0100
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Mon Aug 4 11:36:24 2014 +0200
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Mon Aug 4 11:36:24 2014 +0200

----------------------------------------------------------------------
 .../wicket/markup/html/form/upload/MultiFileUploadField.java   | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/34b46ca6/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/MultiFileUploadField.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/MultiFileUploadField.java b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/MultiFileUploadField.java
index 8e907fd..42191ba 100644
--- a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/MultiFileUploadField.java
+++ b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/MultiFileUploadField.java
@@ -297,7 +297,7 @@ public class MultiFileUploadField extends FormComponentPanel<Collection<FileUplo
 		{
 			final IMultipartWebRequest request = (IMultipartWebRequest)getRequest();
 
-			uploads = new ArrayList<FileUpload>(filenames.length);
+			uploads = new ArrayList<>(filenames.length);
 
 			for (String filename : filenames)
 			{
@@ -374,9 +374,9 @@ public class MultiFileUploadField extends FormComponentPanel<Collection<FileUplo
 			}
 			else
 			{
-				HashMap<String, Object> vars = new HashMap<String, Object>(1);
+				HashMap<String, Object> vars = new HashMap<>(1);
 				vars.put("max", max);
-				return getString(RESOURCE_LIMITED, new Model<HashMap<String, Object>>(vars));
+				return getString(RESOURCE_LIMITED, new Model<>(vars));
 			}
 		}
 


[2/2] git commit: WICKET-5659 Add a setting to MultiFileUploadField to not close the file uploads' streams

Posted by mg...@apache.org.
WICKET-5659 Add a setting to MultiFileUploadField to not close the file uploads' streams


Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/38d6b4a9
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/38d6b4a9
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/38d6b4a9

Branch: refs/heads/master
Commit: 38d6b4a9c1bcf2a6ebcddeefcdf372839ca80e5c
Parents: 34b46ca
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Mon Aug 4 11:40:11 2014 +0200
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Mon Aug 4 11:40:11 2014 +0200

----------------------------------------------------------------------
 .../html/form/upload/MultiFileUploadField.java  | 41 +++++++++++++-------
 1 file changed, 28 insertions(+), 13 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/38d6b4a9/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/MultiFileUploadField.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/MultiFileUploadField.java b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/MultiFileUploadField.java
index 42191ba..e6d4c2b 100644
--- a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/MultiFileUploadField.java
+++ b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/upload/MultiFileUploadField.java
@@ -329,30 +329,45 @@ public class MultiFileUploadField extends FormComponentPanel<Collection<FileUplo
 	@Override
 	protected void onDetach()
 	{
-		// cleanup any opened filestreams
-		Collection<FileUpload> uploads = getConvertedInput();
-		if (uploads != null)
+		if (forceCloseStreamsOnDetach())
 		{
-			for (FileUpload upload : uploads)
+			// cleanup any opened filestreams
+			Collection<FileUpload> uploads = getConvertedInput();
+			if (uploads != null)
 			{
-				upload.closeStreams();
+				for (FileUpload upload : uploads)
+				{
+					upload.closeStreams();
+				}
 			}
-		}
 
-		// cleanup any caches
-		inputArrayCache = null;
+			// cleanup any caches
+			inputArrayCache = null;
 
-		// clean up the model because we don't want FileUpload objects in session
-		Collection<FileUpload> modelObject = getModelObject();
-		if (modelObject != null)
-		{
-			modelObject.clear();
+			// clean up the model because we don't want FileUpload objects in session
+			Collection<FileUpload> modelObject = getModelObject();
+			if (modelObject != null)
+			{
+				modelObject.clear();
+			}
 		}
 
 		super.onDetach();
 	}
 
 	/**
+	 * The FileUploadField will close any input streams you have opened in its FileUpload by
+	 * default. If you wish to manage the stream yourself (e.g. you want to use it in another
+	 * thread) then you can override this method to prevent this behavior.
+	 *
+	 * @return <code>true</code> if stream should be closed at the end of request
+	 */
+	protected boolean forceCloseStreamsOnDetach()
+	{
+		return true;
+	}
+
+	/**
 	 * Model that will construct the caption string
 	 * 
 	 * @author ivaynberg