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 2011/04/29 09:46:32 UTC

svn commit: r1097714 - /wicket/trunk/wicket-core/src/main/java/org/apache/wicket/settings/IResourceSettings.java

Author: mgrigorov
Date: Fri Apr 29 07:46:32 2011
New Revision: 1097714

URL: http://svn.apache.org/viewvc?rev=1097714&view=rev
Log:
WICKET-3650: parentFolderPlaceholder has inconsistent javadoc and is not disabled when set to null

Update the javadoc of getParentFolderPlaceholder() - the default is null, not ''


Modified:
    wicket/trunk/wicket-core/src/main/java/org/apache/wicket/settings/IResourceSettings.java

Modified: wicket/trunk/wicket-core/src/main/java/org/apache/wicket/settings/IResourceSettings.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket-core/src/main/java/org/apache/wicket/settings/IResourceSettings.java?rev=1097714&r1=1097713&r2=1097714&view=diff
==============================================================================
--- wicket/trunk/wicket-core/src/main/java/org/apache/wicket/settings/IResourceSettings.java (original)
+++ wicket/trunk/wicket-core/src/main/java/org/apache/wicket/settings/IResourceSettings.java Fri Apr 29 07:46:32 2011
@@ -16,8 +16,6 @@
  */
 package org.apache.wicket.settings;
 
-import java.util.List;
-
 import org.apache.wicket.IResourceFactory;
 import org.apache.wicket.Localizer;
 import org.apache.wicket.javascript.IJavaScriptCompressor;
@@ -33,6 +31,8 @@ import org.apache.wicket.util.resource.l
 import org.apache.wicket.util.time.Duration;
 import org.apache.wicket.util.watch.IModificationWatcher;
 
+import java.util.List;
+
 
 /**
  * Interface for resource related settings
@@ -301,13 +301,13 @@ public interface IResourceSettings
 
 	/**
 	 * Placeholder string for '..' within resource urls (which will be crippled by the browser and
-	 * not work anymore). Note that by default the placeholder string is empty '' and thus will not
-	 * allow to access parent folders. That is by purpose and for security reasons (see
+	 * not work anymore). Note that by default the placeholder string is <code>null</code> and thus
+	 * will not allow to access parent folders. That is by purpose and for security reasons (see
 	 * Wicket-1992). In case you really need it, a good value for placeholder would e.g. be "$up$".
 	 * Resources additionally are protected by a
 	 * {@link org.apache.wicket.markup.html.IPackageResourceGuard IPackageResourceGuard}
-	 * implementation such as {@link org.apache.wicket.markup.html.PackageResourceGuard}
-	 * which you may use or extend based on your needs.
+	 * implementation such as {@link org.apache.wicket.markup.html.PackageResourceGuard} which you
+	 * may use or extend based on your needs.
 	 * 
 	 * @return placeholder
 	 */
@@ -332,17 +332,17 @@ public interface IResourceSettings
 
 	/**
 	 * gets the resource caching strategy
-	 *
+	 * 
 	 * @return strategy
 	 */
 	IResourceCachingStrategy getCachingStrategy();
 
 	/**
 	 * sets the resource caching strategy
-	 *
+	 * 
 	 * @param strategy
-	 *           instance of resource caching strategy
-	 *
+	 *            instance of resource caching strategy
+	 * 
 	 * @see IResourceCachingStrategy
 	 */
 	void setCachingStrategy(IResourceCachingStrategy strategy);