[jboss-cvs] JBossAS SVN: r71133 - in projects/jboss-reflect/trunk/src: tests/org/jboss/test/beaninfo/test and 1 other directory.

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

Author: alesj
Date: 2008-03-21 03:54:37 -0400 (Fri, 21 Mar 2008)
New Revision: 71133

Modified:
   projects/jboss-reflect/trunk/src/main/org/jboss/reflect/plugins/introspection/ReflectFieldInfoImpl.java
   projects/jboss-reflect/trunk/src/tests/org/jboss/test/beaninfo/test/AccessRestrictionTestCase.java
Log:
Null check.
Comment fix.

Modified: projects/jboss-reflect/trunk/src/main/org/jboss/reflect/plugins/introspection/ReflectFieldInfoImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/org/jboss/reflect/plugins/introspection/ReflectFieldInfoImpl.java	2008-03-21 06:42:26 UTC (rev 71132)
+++ projects/jboss-reflect/trunk/src/main/org/jboss/reflect/plugins/introspection/ReflectFieldInfoImpl.java	2008-03-21 07:54:37 UTC (rev 71133)
@@ -81,8 +81,11 @@
     */
    public void setField(Field field)
    {
-      accessCheck(Modifier.isPublic(field.getModifiers()));
+      if (field != null)
+         accessCheck(Modifier.isPublic(field.getModifiers()));
+
       this.field = field;
+
       if (isPublic() == false && field != null)
          setAccessible();
    }

Modified: projects/jboss-reflect/trunk/src/tests/org/jboss/test/beaninfo/test/AccessRestrictionTestCase.java
===================================================================
--- projects/jboss-reflect/trunk/src/tests/org/jboss/test/beaninfo/test/AccessRestrictionTestCase.java	2008-03-21 06:42:26 UTC (rev 71132)
+++ projects/jboss-reflect/trunk/src/tests/org/jboss/test/beaninfo/test/AccessRestrictionTestCase.java	2008-03-21 07:54:37 UTC (rev 71133)
@@ -84,7 +84,7 @@
       beanInfo.setProperty(test, "pubString", "public");
       assertEquals("public", test.pubString);
       
-      // But we should be able to set the private field
+      // But we shouldn't be able to set the private field
       try
       {
          beanInfo.setProperty(test, "privString", "private");