Index: /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/handlers/RequestCycleHandler.java
===================================================================
--- /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/handlers/RequestCycleHandler.java	(revision 3419)
+++ /FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/handlers/RequestCycleHandler.java	(revision 3420)
@@ -23,4 +23,5 @@
 import javax.servlet.http.HttpServletRequest;
 
+import net.fckeditor.connector.Connector;
 import net.fckeditor.requestcycle.UserAction;
 import net.fckeditor.requestcycle.UserPathBuilder;
@@ -31,6 +32,8 @@
 
 /**
- * Handler for classes which implement the interfaces from package
- * {@link net.fckeditor.requestcycle}.
+ * The <code>RequestCycleHandler</code> handles the {@link UserAction} and
+ * {@link UserPathBuilder} interfaces. <br />
+ * This class instantiates the the chosen implementations and acts as a proxy
+ * for their methods/access.
  * 
  * @version $Id$
@@ -78,10 +81,12 @@
 
 	/**
-	 * Just a wrapper to {@link UserAction#isEnabledForFileBrowsing()}.
+	 * Just a wrapper to
+	 * {@link UserAction#isEnabledForFileBrowsing(HttpServletRequest)}.
 	 * 
-	 * @return {@link UserAction#isEnabledForFileBrowsing()} or false if
-	 *         {@link UserAction} isn't set.
+	 * @return {@link UserAction#isEnabledForFileBrowsing(HttpServletRequest)}
+	 *         or false if {@link UserAction} isn't set.
 	 * @see #isGetResourcesEnabled(HttpServletRequest)
-	 * @deprecated Method will be removed in FCKeditor.Java 2.6.
+	 * @deprecated Method will be removed in FCKeditor.Java 2.6, user
+	 *             {@link #isGetResourcesEnabled(HttpServletRequest)}.
 	 */
 	@Deprecated
@@ -93,7 +98,11 @@
 	
 	/**
+	 * Returns <code>true</code> if user is allowed to list resources. The
+	 * behavior is specified by the current UserAction instance.
 	 * 
 	 * @param request
-	 * @return
+	 *            current request instance
+	 * @return true if user is allowed to list resources, false otherwise
+	 * @see UserAction#isEnabledForFileBrowsing(HttpServletRequest)
 	 */
 	public static boolean isGetResourcesEnabled(final HttpServletRequest request) {
@@ -102,10 +111,12 @@
 
 	/**
-	 * Just a wrapper to {@link UserAction#isEnabledForFileUpload()}.
+	 * Just a wrapper to
+	 * {@link UserAction#isEnabledForFileUpload(HttpServletRequest)}.
 	 * 
-	 * @return {@link UserAction#isEnabledForFileUpload()} or false if
-	 *         {@link UserAction} isn't set.
+	 * @return {@link UserAction#isEnabledForFileUpload(HttpServletRequest)} or
+	 *         false if {@link UserAction} isn't set.
 	 * @see #isFileUploadEnabled(HttpServletRequest)
-	 * @deprecated Method will be removed in FCKeditor.Java 2.6.
+	 * @deprecated Method will be removed in FCKeditor.Java 2.6,
+	 *             {@link #isFileUploadEnabled(HttpServletRequest)}.
 	 */
 	public static boolean isEnabledForFileUpload(
@@ -116,7 +127,11 @@
 	
 	/**
+	 * Returns <code>true</code> if user is allowed to upload files. The
+	 * behavior is specified by the current UserAction instance.
 	 * 
 	 * @param request
-	 * @return
+	 *            current request instance
+	 * @return true if user is allowed to upload files, false otherwise
+	 * @see UserAction#isEnabledForFileUpload(HttpServletRequest)
 	 */
 	public static boolean isFileUploadEnabled(HttpServletRequest request) {
@@ -125,8 +140,11 @@
 
 	/**
-	 * Just a wrapper to {@link UserAction#isEnabledForFolderCreation()}.
+	 * Returns <code>true</code> if user is allowed to create folders. The
+	 * behavior is specified by the current UserAction instance.
 	 * 
-	 * @return {@link UserAction#isEnabledForFolderCreation()} or false if
-	 *         {@link UserAction} isn't set.
+	 * @param request
+	 *            current request instance
+	 * @return true if user is allowed to create folders, false otherwise
+	 * @see UserAction#isEnabledForFileBrowsing(HttpServletRequest)
 	 */
 	public static boolean isCreateFolderEnabled(final HttpServletRequest request) {
@@ -135,25 +153,29 @@
 
 	/**
-	 * Getter for the <code>UserFilesPath</code>.<br />
-	 * Method is used by other handlers only!
+	 * Returns the current userfiles path. The path is specified by the current
+	 * UserPathBuilder instance.
 	 * 
 	 * @param request
-	 * @return {@link UserPathBuilder#getUserFilesPath()} or <code>null</code>
-	 *         if <code>userPathBuilder</code> is <code>null</code>.
+	 *            current request instance
+	 * @return current userfiles path
+	 * @see UserPathBuilder#getUserFilesPath(HttpServletRequest)
 	 */
 	public static String getUserFilesPath(final HttpServletRequest request) {
-		return (userPathBuilder != null) ? userPathBuilder
-				.getUserFilesPath(request) : null;
+		return userPathBuilder.getUserFilesPath(request);
 	}
 
 	/**
+	 * Returns the current absolute userfiles path. The path is specified by the
+	 * current UserPathBuilder instance. <br />
+	 * Note that the path is absolute to the underlying system of the current
+	 * {@link Connector} instance.
 	 * 
 	 * @param request
-	 * @return
+	 *            current request instance
+	 * @return current absolute userfiles path
+	 * @see UserPathBuilder#getUserFilesAbsolutePath(HttpServletRequest)
 	 */
-	public static String getUserFilesAbsolutePath(
-			final HttpServletRequest request) {
-		return (userPathBuilder != null) ? userPathBuilder
-				.getUserFilesAbsolutePath(request) : null;
+	public static String getUserFilesAbsolutePath(final HttpServletRequest request) {
+		return userPathBuilder.getUserFilesAbsolutePath(request);
 	}
 }