You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by iv...@apache.org on 2010/09/21 18:44:02 UTC

svn commit: r999494 - /wicket/trunk/wicket/src/main/java/org/apache/wicket/pageStore/DiskDataStore.java

Author: ivaynberg
Date: Tue Sep 21 16:44:02 2010
New Revision: 999494

URL: http://svn.apache.org/viewvc?rev=999494&view=rev
Log:

Issue: WICKET-3055

Modified:
    wicket/trunk/wicket/src/main/java/org/apache/wicket/pageStore/DiskDataStore.java

Modified: wicket/trunk/wicket/src/main/java/org/apache/wicket/pageStore/DiskDataStore.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/pageStore/DiskDataStore.java?rev=999494&r1=999493&r2=999494&view=diff
==============================================================================
--- wicket/trunk/wicket/src/main/java/org/apache/wicket/pageStore/DiskDataStore.java (original)
+++ wicket/trunk/wicket/src/main/java/org/apache/wicket/pageStore/DiskDataStore.java Tue Sep 21 16:44:02 2010
@@ -73,10 +73,23 @@ public class DiskDataStore implements ID
 		this.applicationName = applicationName;
 		this.fileStoreFolder = fileStoreFolder;
 		maxSizePerPageSession = maxSizePerSession;
-		fileChannelPool = new FileChannelPool(fileChannelPoolCapacity);
 
-		this.fileStoreFolder.mkdirs();
-		loadIndex();
+
+		try
+		{
+			fileChannelPool = new FileChannelPool(fileChannelPoolCapacity);
+
+			this.fileStoreFolder.mkdirs();
+			loadIndex();
+		}
+		catch (SecurityException e)
+		{
+			throw new WicketRuntimeException(
+				// TODO improve the message by explaining where in the API the disk store can be
+				// changed
+				"SecurityException occurred while creating DiskDataStore. Consider using a non-disk based IDataStore implementation.",
+				e);
+		}
 	}
 
 	/**