Author: bpontarelli
Date: Thu Mar  6 12:58:00 2008
New Revision: 634412

URL: http://svn.apache.org/viewvc?rev=634412&view=rev
Log:
Fixed issue with empty locator and action packages

Modified:
    
struts/sandbox/trunk/struts2-convention-plugin/src/main/java/org/apache/struts2/convention/PackageBasedActionConfigBuilder.java

Modified: 
struts/sandbox/trunk/struts2-convention-plugin/src/main/java/org/apache/struts2/convention/PackageBasedActionConfigBuilder.java
URL: 
http://svn.apache.org/viewvc/struts/sandbox/trunk/struts2-convention-plugin/src/main/java/org/apache/struts2/convention/PackageBasedActionConfigBuilder.java?rev=634412&r1=634411&r2=634412&view=diff
==============================================================================
--- 
struts/sandbox/trunk/struts2-convention-plugin/src/main/java/org/apache/struts2/convention/PackageBasedActionConfigBuilder.java
 (original)
+++ 
struts/sandbox/trunk/struts2-convention-plugin/src/main/java/org/apache/struts2/convention/PackageBasedActionConfigBuilder.java
 Thu Mar  6 12:58:00 2008
@@ -107,7 +107,9 @@
      */
     @Inject(value = "struts.convention.action.packages", required = false)
     public void setActionPackages(String actionPackages) {
-        this.actionPackages = actionPackages.split("\\s*[,]\\s*");
+        if (!StringTools.isTrimmedEmpty(actionPackages)) {
+            this.actionPackages = actionPackages.split("\\s*[,]\\s*");
+        }
     }
 
     /**
@@ -116,7 +118,9 @@
      */
     @Inject(value = "struts.convention.exclude.packages", required = false)
     public void setExcludePackages(String excludePackages) {
-        this.excludePackages = excludePackages.split("\\s*[,]\\s*");
+        if (!StringTools.isTrimmedEmpty(excludePackages)) {
+            this.excludePackages = excludePackages.split("\\s*[,]\\s*");
+        }
     }
 
     /**


Reply via email to