[jboss-cvs] JBossAS SVN: r70194 - in projects/microcontainer/trunk/container/src: tests/org/jboss/test/beaninfo/support and 1 other directory.

[jboss-cvs-commits at lists.jboss.org]

Author: adrian at jboss.org
Date: 2008-02-27 17:00:43 -0500 (Wed, 27 Feb 2008)
New Revision: 70194

Modified:
   projects/microcontainer/trunk/container/src/main/org/jboss/beans/info/plugins/AbstractBeanInfoFactory.java
   projects/microcontainer/trunk/container/src/tests/org/jboss/test/beaninfo/support/BeanInfoMethods.java
Log:
[JBMICROCONT-229] - Don't try to get the uppercase property name until we know it is a getter/setter

Modified: projects/microcontainer/trunk/container/src/main/org/jboss/beans/info/plugins/AbstractBeanInfoFactory.java
===================================================================
--- projects/microcontainer/trunk/container/src/main/org/jboss/beans/info/plugins/AbstractBeanInfoFactory.java	2008-02-27 19:18:10 UTC (rev 70193)
+++ projects/microcontainer/trunk/container/src/main/org/jboss/beans/info/plugins/AbstractBeanInfoFactory.java	2008-02-27 22:00:43 UTC (rev 70194)
@@ -231,13 +231,14 @@
          for (MethodInfo methodInfo : methods)
          {
             String name = methodInfo.getName();
-            String upperName = getUpperPropertyName(name);
             if (isGetter(methodInfo))
             {
+               String upperName = getUpperPropertyName(name);
                getters.put(upperName, methodInfo);
             }
             else if (isSetter(methodInfo))
             {
+               String upperName = getUpperPropertyName(name);
                List<MethodInfo> list = setters.get(upperName);
                if (list == null)
                {

Modified: projects/microcontainer/trunk/container/src/tests/org/jboss/test/beaninfo/support/BeanInfoMethods.java
===================================================================
--- projects/microcontainer/trunk/container/src/tests/org/jboss/test/beaninfo/support/BeanInfoMethods.java	2008-02-27 19:18:10 UTC (rev 70193)
+++ projects/microcontainer/trunk/container/src/tests/org/jboss/test/beaninfo/support/BeanInfoMethods.java	2008-02-27 22:00:43 UTC (rev 70194)
@@ -57,4 +57,11 @@
    public void voidMethodintStringPublic(int p1, String p2) {}
    public long longMethodintStringPublic(int p1, String p2) { return 0; }
    public long longMethodintStringThrowsPublic(int p1, String p2) throws IllegalArgumentException, IllegalStateException { return 0; }
+   public long a() { return 0; };
+   public long ab() { return 0; };
+   public boolean is() { return false; };
+   public long get() { return 0; };
+   public void x(long x) {}
+   public void xy(long xy) {}
+   public void set(long xy) {}
 }