[Asterisk-java-cvs] CVS: asterisk-java/src/java/net/sf/asterisk/manager ActionBuilder.java,1.3,1.4

[Stefan R. <sr...@us...>]

Update of /cvsroot/asterisk-java/asterisk-java/src/java/net/sf/asterisk/manager
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv22641/src/java/net/sf/asterisk/manager

Modified Files:
	ActionBuilder.java 
Log Message:
Removed superflous check for a getter being public
Added check to exclude zero length properties

Index: ActionBuilder.java
===================================================================
RCS file: /cvsroot/asterisk-java/asterisk-java/src/java/net/sf/asterisk/manager/ActionBuilder.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -d -p -r1.3 -r1.4
--- ActionBuilder.java	23 Feb 2005 22:50:57 -0000	1.3
+++ ActionBuilder.java	25 Feb 2005 23:34:52 -0000	1.4
@@ -18,7 +18,6 @@ package net.sf.asterisk.manager;
 
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
-import java.lang.reflect.Modifier;
 import java.util.HashMap;
 import java.util.Iterator;
 import java.util.Map;
@@ -140,12 +139,6 @@ public class ActionBuilder
                 continue;
             }
 
-            // skip non public methods
-            if ((method.getModifiers() & Modifier.PUBLIC) == 0)
-            {
-                continue;
-            }
-
             // skip methods with != 0 parameters
             if (method.getParameterTypes().length != 0)
             {
@@ -154,6 +147,12 @@ public class ActionBuilder
 
             // ok seems to be an accessor
             name = methodName.substring("get".length()).toLowerCase();
+
+            if (name.length() == 0)
+            {
+                continue;
+            }
+
             accessors.put(name, method);
         }