Changeset 2864


Ignore:
Timestamp:
01/09/2009 09:03:33 PM (6 years ago)
Author:
mosipov
Message:
  • Types are case-sensitive again
  • Improved exception handling
  • fixed a bug in getDefaultType
  • More JUnit tests
  • Bumped taglist plugin to 2.3
  • Bumped assembly plugin to 2.2-beta-3
Location:
FCKeditor.Java/trunk/java-core
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • FCKeditor.Java/trunk/java-core/pom.xml

    r2607 r2864  
    138138                                <groupId>org.codehaus.mojo</groupId>
    139139                                <artifactId>taglist-maven-plugin</artifactId>
     140                                <version>2.3</version>
    140141                                <configuration>
    141142                                        <tags>
  • FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/handlers/ResourceType.java

    r2738 r2864  
    5151
    5252        /** Resource type <code>File</code> */
    53         public static final ResourceType FILE = new ResourceType("file",
     53        public static final ResourceType FILE = new ResourceType("File",
    5454                        PropertiesLoader.getFileResourceTypePath(), Utils
    5555                                        .getSet(PropertiesLoader
     
    5858                                                        .getFileResourceTypeDeniedExtensions()));
    5959        /** Resource type <code>Flash</code> */
    60         public static final ResourceType FLASH = new ResourceType("flash",
     60        public static final ResourceType FLASH = new ResourceType("Flash",
    6161                        PropertiesLoader.getFlashResourceTypePath(), Utils
    6262                                        .getSet(PropertiesLoader
     
    6565                                                        .getFlashResourceTypeDeniedExtensions()));
    6666        /** Resource type <code>Image</code> */
    67         public static final ResourceType IMAGE = new ResourceType("image",
     67        public static final ResourceType IMAGE = new ResourceType("Image",
    6868                        PropertiesLoader.getImageResourceTypePath(), Utils
    6969                                        .getSet(PropertiesLoader
     
    7272                                                        .getImageResourceTypeDeniedExtensions()));
    7373        /** Resource type <code>Media</code> */
    74         public static final ResourceType MEDIA = new ResourceType("media",
     74        public static final ResourceType MEDIA = new ResourceType("Media",
    7575                        PropertiesLoader.getMediaResourceTypePath(), Utils
    7676                                        .getSet(PropertiesLoader
     
    158158        /**
    159159         * Returns the {@link ResourceType} for a specified string. <br />
    160          * <em>The retrieval is case-insensitive!</em>
    161160         *
    162161         * @param name
     
    169168        public static ResourceType valueOf(final String name) {
    170169                if (Utils.isEmpty(name))
    171                         throw new IllegalArgumentException("Parameter name is empty");
    172 
    173                 ResourceType rt = types.get(name.toLowerCase());
     170                        throw new NullPointerException("Parameter name is null or empty");
     171
     172                ResourceType rt = types.get(name);
    174173                if (rt == null)
    175174                        throw new IllegalArgumentException(
     
    180179        /**
    181180         * Determines if a specified string represents a valid resource type.<br />
    182          * <em>The retrieval is case-insensitive!</em>
    183181         *
    184182         * @param name
     
    188186         */
    189187        public static boolean isValidType(final String name) {
    190                 return (Utils.isEmpty(name)) ? false : types.containsKey(name
    191                                 .toLowerCase());
     188                return (Utils.isEmpty(name)) ? false : types.containsKey(name);
    192189        }
    193190
     
    195192         * This method wraps {@link #valueOf(String)}. It returns <code>null</code>
    196193         * instead of throwing an IllegalArgumentException.<br />
    197          * <em>The retrieval is case-insensitive!</em>
    198194         *
    199195         * @param name
     
    204200        public static ResourceType getResourceType(final String name) {
    205201                try {
    206                         return ResourceType.valueOf(name.toLowerCase());
     202                        return ResourceType.valueOf(name);
    207203                } catch (Exception e) {
    208204                        return null;
     
    213209         * This method wraps {@link #getResourceType(String)}. It returns
    214210         * {@link #FILE} instead of returning <code>null</code>.<br />
    215          * <em>The retrieval is case-insensitive!</em>
    216211         *
    217212         * @param name
     
    221216         */
    222217        public static ResourceType getDefaultResourceType(final String name) {
    223                 if (Utils.isEmpty(name))
    224                         return null;
    225218                ResourceType rt = getResourceType(name);
    226219                return rt == null ? FILE : rt;
  • FCKeditor.Java/trunk/java-core/src/test/java/net/fckeditor/handlers/ResourceTypeTest.java

    r2729 r2864  
    2323import static org.junit.Assert.assertEquals;
    2424import static org.junit.Assert.assertFalse;
     25import static org.junit.Assert.assertNotSame;
    2526import static org.junit.Assert.assertNull;
    26 import static org.junit.Assert.assertTrue;
    2727
    2828import org.junit.Test;
     
    4747        @Test
    4848        public void getType03() throws Exception {
    49                 assertEquals(ResourceType.IMAGE, ResourceType.getResourceType("ImaGe"));
     49                assertNotSame(ResourceType.IMAGE, ResourceType.getResourceType("ImaGe"));
    5050        }
     51       
     52        @Test
     53        public void getType04() throws Exception {
     54                assertNull(ResourceType.getResourceType(null));
     55        }
     56       
     57        @Test
     58        public void getType05() throws Exception {
     59                assertNull(ResourceType.getResourceType(""));
     60        }
     61       
    5162        @Test
    5263        public void isValid01() throws Exception {
     
    5667        @Test
    5768        public void isValid02() throws Exception {
    58                 assertTrue(ResourceType.isValidType("fLash"));
     69                assertFalse(ResourceType.isValidType("fLash"));
    5970        }
    6071
    6172        @Test
    6273        public void isValid03() throws Exception {
    63                 assertTrue(ResourceType.isValidType("MeDiA"));
     74                assertFalse(ResourceType.isValidType("MeDiA"));
    6475        }
    6576
    6677        @Test
    67         public void getTypeDefault01() throws Exception {
     78        public void getDefaultType01() throws Exception {
    6879                assertEquals(ResourceType.FILE, ResourceType
    6980                                .getDefaultResourceType("wrong-type"));
     
    7182
    7283        @Test
    73         public void getTypeDefault02() throws Exception {
    74                 assertEquals(ResourceType.FLASH, ResourceType
     84        public void getDefaultType02() throws Exception {
     85                assertNotSame(ResourceType.FLASH, ResourceType
    7586                                .getDefaultResourceType("flAsh"));
    7687        }
     88       
     89        @Test
     90        public void getDefaultType03() throws Exception {
     91                assertEquals(ResourceType.FILE, ResourceType
     92                                .getDefaultResourceType(null));
     93        }
     94       
     95        @Test
     96        public void getDefaultType04() throws Exception {
     97                assertEquals(ResourceType.FILE, ResourceType
     98                                .getDefaultResourceType(""));
     99        }
     100       
     101        @Test
     102        public void getDefaultType05() throws Exception {
     103                assertNotSame(ResourceType.FILE, ResourceType
     104                                .getDefaultResourceType("Image"));
     105        }
    77106}
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy