Changeset 2588


Ignore:
Timestamp:
10/19/08 00:21:27 (5 years ago)
Author:
mosipov
Message:

NEW - #2582: Rework *Path and fullUrl properties

Location:
FCKeditor.Java/trunk
Files:
10 edited

Legend:

Unmodified
Added
Removed
  • FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/connector/impl/SimpleFileSystemConnector.java

    r2584 r2588  
    7272                // create user's default dir 
    7373                SimpleFileSystemConnector.servletContext = servletContext; 
    74                 String realDefaultUserFilesPath = SimpleFileSystemConnector.servletContext.getRealPath(ConnectorHandler.getDefaultUserFilesPath()); 
     74                String realDefaultUserFilesPath = SimpleFileSystemConnector.servletContext.getRealPath(ConnectorHandler.getUserFilesPath()); 
    7575                File defaultUserFilesDir = new File(realDefaultUserFilesPath); 
    7676                UtilsFile.checkDirAndCreate(defaultUserFilesDir); 
  • FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/handlers/ConnectorHandler.java

    r2587 r2588  
    2121package net.fckeditor.handlers; 
    2222 
     23import net.fckeditor.connector.Connector; 
     24 
    2325import org.slf4j.Logger; 
    2426import org.slf4j.LoggerFactory; 
    25  
    26 import net.fckeditor.connector.Connector; 
    27 import net.fckeditor.requestcycle.UserPathBuilder; 
    2827 
    2928/** 
    3029 * Handler for Connector-related properties.<br /> 
    3130 * Wraps to the {@link PropertiesLoader}. 
     31 * TODO improve documentation of the property key mapping 
    3232 *  
    3333 * @version $Id$ 
     
    4141                 
    4242                // try to instantiate the Connector object 
     43                // TODO create a better property 
    4344                String fqcn = PropertiesLoader.getProperty("connector.implementation"); 
    4445                if (fqcn == null) 
     
    5758 
    5859        /** 
    59          * Getter for the <code>UserFilesPath</code>.<br> 
    60          * Should be use in the implementations of {@link Connector}. 
     60         * Getter for the <code>UserFilesPath</code>. 
    6161         *  
    62          * @return {@link UserPathBuilder#getUserFilesPath()} or 
    63          *         the <code>DefaultUserFilePath</code> if {@link UserPathBuilder} 
    64          *         isn't set. 
     62         * @return UserFilePath 
    6563         */ 
    6664        public static String getUserFilesPath() { 
    67         String userFilePath = RequestCycleHandler.getUserFilesPath(); 
    68         return (userFilePath != null) ? userFilePath : getDefaultUserFilesPath(); 
     65                return PropertiesLoader.getProperty("connector.userFilesPath"); 
     66    } 
     67         
     68        /** 
     69         * Getter for the <code>UserFilesAbsolutePath</code>. 
     70         *  
     71         * @return UserFilesAbsolutePath 
     72         */ 
     73        public static String getUserFilesAbsolutePath() { 
     74                return PropertiesLoader.getProperty("connector.userFilesAbsolutePath"); 
    6975    } 
    7076 
     
    7985        } 
    8086 
    81         /** 
    82          * Getter for the value to instruct the connector to return the full URL of 
    83          * a file/folder in the XML response rather than the absolute URL. 
    84          *  
    85          * @return <code>true</code> if the property <code>connector.fullUrl</code> is 
    86          *         set else <code>false</code>. 
    87          */ 
    88         public static boolean isFullUrl() { 
    89                 return Boolean.parseBoolean(PropertiesLoader.getProperty("connector.fullUrl")); 
    90         } 
    91          
    92         public static String getBaseUrl() { 
    93                 return PropertiesLoader.getProperty("connector.baseUrl"); 
    94         } 
    95  
    96         /** 
    97          * Getter for the default <code>UserFilesPath</code>. 
    98          *  
    99          * @return <code>DefaultUserFilesPath</code> (/userfiles) 
    100          */ 
    101         public static String getDefaultUserFilesPath() { 
    102                 return PropertiesLoader.getProperty("connector.userFilesPath"); 
    103         } 
    104          
    10587        /** 
    10688         * Getter for the value to instruct the Connector to check if the uploaded 
  • FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/handlers/LocalizedPropertiesLoader.java

    r2584 r2588  
    9393                                                .getClass()); 
    9494                        } catch (Exception e) { 
     95                                // TODO align message to other warnings 
    9596                                logger.warn( 
    9697                                                "Error while loading LocaleResolver implementation", e); 
  • FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/handlers/RequestCycleHandler.java

    r2584 r2588  
    6868                                logger.info("UserPathBuilder initialized to {}", userPathBuilder.getClass()); 
    6969                        } catch (Exception e) { 
     70                                // TODO align message with others 
    7071                                logger.error("Couldn't instantiate class [".concat(fqcn) 
    7172                                        .concat("], The 'DefaultUserFilesPath' will be used in the ConnectorServlet!"), e); 
  • FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/requestcycle/Context.java

    r2584 r2588  
    114114        StringBuffer sb = new StringBuffer(); 
    115115        HttpServletRequest request = ThreadLocalData.getServletRequest(); 
    116  
    117                 if (Utils.isNotEmpty(ConnectorHandler.getBaseUrl())) { 
    118                         sb.append(ConnectorHandler.getBaseUrl()); 
    119                 } else if (ConnectorHandler.isFullUrl()) { 
    120                 String address = request.getRequestURL().toString(); 
    121                 sb.append(address.substring(0, address.indexOf('/', 8))); 
    122         } 
    123116                 
    124117        sb.append(baseDir); 
  • FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/requestcycle/UserPathBuilder.java

    r2584 r2588  
    5858         * @return 
    5959         */ 
    60         public String getLocalUserFilesPath(); 
     60        public String getUserFilesAbsolutePath(); 
    6161} 
  • FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/requestcycle/impl/ContextPathBuilder.java

    r2584 r2588  
    3838         */ 
    3939        public String getUserFilesPath() { 
    40                 String path = ThreadLocalData.getServletRequest().getContextPath().concat(ConnectorHandler.getDefaultUserFilesPath()); 
     40                String path = ThreadLocalData.getServletRequest().getContextPath().concat(ConnectorHandler.getUserFilesPath()); 
    4141                return path; 
    4242        } 
     
    4545         * @see net.fckeditor.requestcycle.UserPathBuilder#getLocalUserFilesPath() 
    4646         */ 
    47         public String getLocalUserFilesPath() { 
    48                 return ConnectorHandler.getDefaultUserFilesPath(); 
     47        public String getUserFilesAbsolutePath() { 
     48                return ConnectorHandler.getUserFilesPath(); 
    4949        } 
    5050} 
  • FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/UtilsResponse.java

    r2584 r2588  
    4040        public static String constructResponseUrl(String filePath, String resourceTypePath, String urlPath) { 
    4141                StringBuffer sb = new StringBuffer(); 
    42                  
    43                 if (Utils.isNotEmpty(ConnectorHandler.getBaseUrl())) { 
    44                         sb.append(ConnectorHandler.getBaseUrl()); 
    45                 } 
    46                          
     42                         
    4743                sb.append(filePath); 
    4844                if (Utils.isNotEmpty(resourceTypePath)) 
  • FCKeditor.Java/trunk/java-core/src/main/resources/net/fckeditor/handlers/default.properties

    r2584 r2588  
    3939connector.forceSingleExtension = true 
    4040 
    41 # base directory for the user files relative to the context root 
     41# base URL path for the userfiles 
    4242connector.userFilesPath = /userfiles 
    4343 
    44 # Instructs the Connector to return the full URL of a file/folder in the XML 
    45 # response rather than the absolute URL 
    46 connector.fullUrl = false 
     44# base system path on the backend for the userfiles 
     45connector.userFilesAbsolutePath = /userfiles 
    4746 
    4847# Instructs the Connector to check, if the uploaded image is really one 
  • FCKeditor.Java/trunk/src/site/apt/properties.apt

    r2521 r2588  
    121121|                                                               | swf\|tif\|tiff\|wav\|wma\|wmv                         |                                                       |                                                                                                       | 
    122122*-------------------------------+-------------------------------------------+---------------------------+---------------------------------------------------+ 
    123 | connector.userFilesPath               | /userfiles                                                            | any path                                      | Indicates the path where files are stored.            | 
     123| connector.userFilesPath               | /userfiles                                                            | any path                                      | Indicates the URL path under which files can be       | 
     124|                                                               |                                                                                       |                                                       | retrieved from the server. Prepend the domain         | 
     125|                                                               |                                                                                       |                                                       | name if you want to have the full URL. For more       | 
     126|                                                               |                                                                                       |                                                       | details see {{{userpathsbuilder.html}here}}.          | 
     127*-------------------------------+-------------------------------------------+---------------------------+---------------------------------------------------+ 
     128| connector.userFilesAbsolutePath| /userfiles                                                           | any path                                      | Indicates the system path under which files can       | 
     129|                                                               |                                                                                       |                                                       | be stored on the backend. For more details see        | 
     130|                                                               |                                                                                       |                                                       | {{{userpathsbuilder.html}here}}.                                      | 
    124131*-------------------------------+-------------------------------------------+---------------------------+---------------------------------------------------+ 
    125132| connector.forceSingleExtension| true                                                                          | true, false (boolean)         | Indicates whether filenames with multiple             | 
    126133|                                                               |                                                                                       |                                                       | extensions are sanitized.                                                     | 
    127134*-------------------------------+-------------------------------------------+-------------------------- +---------------------------------------------------+ 
    128 | connector.fullUrl                     | false                                                                         | true, false (boolean)         | Indicates whether the domain name will be                     | 
    129 |                                                               |                                                                                       |                                                       | prepended. E.g. <<<http://www.mysite.com>>>           | 
    130 *-------------------------------+-------------------------------------------+---------------------------+---------------------------------------------------+ 
    131135| connector.secureImageUploads  |true                                                                           | true, false (boolean)         | Checks uploaded images for validity.                          | 
    132136|                                                               |                                                                                       |                                                       | I.e. the uploaded file is a real image and not any| 
    133 |                                                               |                                                                                       |                                                       | other renamed file type.                                                      | 
     137|                                                               |                                                                                       |                                                       | other renamed file type. (magic bytes check)          | 
    134138*-------------------------------+-------------------------------------------+---------------------------+---------------------------------------------------+ 
    135139| connector.userActionImpl              | {{{java-core/apidocs/net/fckeditor/requestcycle/impl/DisabledUserAction.html}<<<net.fckeditor.requestcycle. | any fully-qualified class       | Provides custom security functions for the File       | 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy