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