Revision: 462
          http://svn.sourceforge.net/stripes/?rev=462&view=rev
Author:   tfenne
Date:     2006-10-29 05:14:52 -0800 (Sun, 29 Oct 2006)

Log Message:
-----------
Merge of fix for STS-293 on to the branch.

Modified Paths:
--------------
    
branches/1.4.x/stripes/src/net/sourceforge/stripes/config/RuntimeConfiguration.java
    
branches/1.4.x/stripes/src/net/sourceforge/stripes/controller/AnnotatedClassActionResolver.java
    
branches/1.4.x/stripes/src/net/sourceforge/stripes/exception/DelegatingExceptionHandler.java
    
branches/1.4.x/stripes/src/net/sourceforge/stripes/localization/DefaultLocalePicker.java

Added Paths:
-----------
    branches/1.4.x/stripes/src/net/sourceforge/stripes/util/StringUtil.java

Modified: 
branches/1.4.x/stripes/src/net/sourceforge/stripes/config/RuntimeConfiguration.java
===================================================================
--- 
branches/1.4.x/stripes/src/net/sourceforge/stripes/config/RuntimeConfiguration.java
 2006-10-29 13:08:11 UTC (rev 461)
+++ 
branches/1.4.x/stripes/src/net/sourceforge/stripes/config/RuntimeConfiguration.java
 2006-10-29 13:14:52 UTC (rev 462)
@@ -30,6 +30,7 @@
 import net.sourceforge.stripes.tag.TagErrorRendererFactory;
 import net.sourceforge.stripes.util.Log;
 import net.sourceforge.stripes.util.ReflectUtil;
+import net.sourceforge.stripes.util.StringUtil;
 import net.sourceforge.stripes.validation.TypeConverterFactory;
 
 import java.util.Collection;
@@ -163,7 +164,7 @@
             return null;
         }
         else {
-            String[] classNames = classList.split(",");
+            String[] classNames = StringUtil.standardSplit(classList);
             Map<LifecycleStage, Collection<Interceptor>> map =
                     new HashMap<LifecycleStage, Collection<Interceptor>>();
 

Modified: 
branches/1.4.x/stripes/src/net/sourceforge/stripes/controller/AnnotatedClassActionResolver.java
===================================================================
--- 
branches/1.4.x/stripes/src/net/sourceforge/stripes/controller/AnnotatedClassActionResolver.java
     2006-10-29 13:08:11 UTC (rev 461)
+++ 
branches/1.4.x/stripes/src/net/sourceforge/stripes/controller/AnnotatedClassActionResolver.java
     2006-10-29 13:14:52 UTC (rev 462)
@@ -25,6 +25,7 @@
 import net.sourceforge.stripes.exception.StripesServletException;
 import net.sourceforge.stripes.util.Log;
 import net.sourceforge.stripes.util.ResolverUtil;
+import net.sourceforge.stripes.util.StringUtil;
 
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.ServletContext;
@@ -515,7 +516,7 @@
 
         String temp = bootstrap.getProperty(URL_FILTERS);
         if (temp != null) {
-            urlFilters.addAll(Arrays.asList( temp.split(",")));
+            urlFilters.addAll(Arrays.asList( StringUtil.standardSplit(temp) ));
         }
 
         return urlFilters;
@@ -533,7 +534,7 @@
 
         String temp = bootstrap.getProperty(PACKAGE_FILTERS);
         if (temp != null) {
-            packageFilters.addAll(Arrays.asList( temp.split(",")));
+            packageFilters.addAll(Arrays.asList( 
StringUtil.standardSplit(temp) ));
         }
 
         return packageFilters;

Modified: 
branches/1.4.x/stripes/src/net/sourceforge/stripes/exception/DelegatingExceptionHandler.java
===================================================================
--- 
branches/1.4.x/stripes/src/net/sourceforge/stripes/exception/DelegatingExceptionHandler.java
        2006-10-29 13:08:11 UTC (rev 461)
+++ 
branches/1.4.x/stripes/src/net/sourceforge/stripes/exception/DelegatingExceptionHandler.java
        2006-10-29 13:14:52 UTC (rev 462)
@@ -19,6 +19,7 @@
 import net.sourceforge.stripes.config.Configuration;
 import net.sourceforge.stripes.util.Log;
 import net.sourceforge.stripes.util.ResolverUtil;
+import net.sourceforge.stripes.util.StringUtil;
 
 import javax.servlet.ServletContext;
 import javax.servlet.ServletException;
@@ -236,7 +237,7 @@
 
         String temp = bootstrap.getProperty(URL_FILTERS);
         if (temp != null) {
-            urlFilters.addAll(Arrays.asList( temp.split(",")));
+            urlFilters.addAll(Arrays.asList( StringUtil.standardSplit(temp) ));
         }
 
         return urlFilters;
@@ -254,7 +255,7 @@
 
         String temp = bootstrap.getProperty(PACAKGE_FILTERS);
         if (temp != null) {
-            packageFilters.addAll(Arrays.asList( temp.split(",")));
+            packageFilters.addAll(Arrays.asList( 
StringUtil.standardSplit(temp) ));
         }
 
         return packageFilters;

Modified: 
branches/1.4.x/stripes/src/net/sourceforge/stripes/localization/DefaultLocalePicker.java
===================================================================
--- 
branches/1.4.x/stripes/src/net/sourceforge/stripes/localization/DefaultLocalePicker.java
    2006-10-29 13:08:11 UTC (rev 461)
+++ 
branches/1.4.x/stripes/src/net/sourceforge/stripes/localization/DefaultLocalePicker.java
    2006-10-29 13:14:52 UTC (rev 462)
@@ -16,6 +16,7 @@
 
 import net.sourceforge.stripes.config.Configuration;
 import net.sourceforge.stripes.util.Log;
+import net.sourceforge.stripes.util.StringUtil;
 
 import javax.servlet.http.HttpServletRequest;
 import java.util.ArrayList;
@@ -79,7 +80,7 @@
         }
         else {
             // Split apart the Locales on commas, and then parse the local 
strings into their bits
-            String[] localeStrings = configuredLocales.split(",");
+            String[] localeStrings = 
StringUtil.standardSplit(configuredLocales);
             for (String localeString : localeStrings) {
                 // Each locale string can be made up of two parts, 
locale:encoding
                 // and the locale can be made up of up to three segment, e.g. 
en_US_PC

Copied: branches/1.4.x/stripes/src/net/sourceforge/stripes/util/StringUtil.java 
(from rev 461, trunk/stripes/src/net/sourceforge/stripes/util/StringUtil.java)
===================================================================
--- branches/1.4.x/stripes/src/net/sourceforge/stripes/util/StringUtil.java     
                        (rev 0)
+++ branches/1.4.x/stripes/src/net/sourceforge/stripes/util/StringUtil.java     
2006-10-29 13:14:52 UTC (rev 462)
@@ -0,0 +1,34 @@
+package net.sourceforge.stripes.util;
+
+import java.util.regex.Pattern;
+
+/**
+ * Provies utility methods for manipulating and parsing Strings.
+ *
+ * @author Tim Fennell
+ * @since Stripes 1.4.2
+ */
+public class StringUtil {
+    /**
+     * A regular expression for splitting apart a String where individual 
parts are
+     * separated by any whitespace (including new lines) and/or a comma.
+     */
+    private static final Pattern STANDARD_SPLIT = Pattern.compile("[\\s,]+");
+
+    /**
+     * Splits apart the input String on any whitespace and/or commas. Leading 
and trailing
+     * whitespace are ignored. If a null String is provided as input a zero 
length array
+     * will be returned.
+     *
+     * @param input the String to split apart
+     * @return an array of substrings of the input String based on the split
+     */
+    public static String[] standardSplit(String input) {
+        if (input == null) {
+            return new String[0];
+        }
+        else {
+            return STANDARD_SPLIT.split(input.trim());
+        }
+    }
+}


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development

Reply via email to