You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@velocity.apache.org by nb...@apache.org on 2008/05/02 06:00:59 UTC

svn commit: r652755 - /velocity/engine/trunk/src/java/org/apache/velocity/app/FieldMethodizer.java

Author: nbubna
Date: Thu May  1 21:00:58 2008
New Revision: 652755

URL: http://svn.apache.org/viewvc?rev=652755&view=rev
Log:
VELOCITY-593 FieldMethodizer doesn't need to keep a classHash, thx to Benjamin Bentmann

Modified:
    velocity/engine/trunk/src/java/org/apache/velocity/app/FieldMethodizer.java

Modified: velocity/engine/trunk/src/java/org/apache/velocity/app/FieldMethodizer.java
URL: http://svn.apache.org/viewvc/velocity/engine/trunk/src/java/org/apache/velocity/app/FieldMethodizer.java?rev=652755&r1=652754&r2=652755&view=diff
==============================================================================
--- velocity/engine/trunk/src/java/org/apache/velocity/app/FieldMethodizer.java (original)
+++ velocity/engine/trunk/src/java/org/apache/velocity/app/FieldMethodizer.java Thu May  1 21:00:58 2008
@@ -64,9 +64,6 @@
     /** Hold the field objects by field name */
     private HashMap fieldHash = new HashMap();
 
-    /** Hold the class objects by field name */
-    private HashMap classHash = new HashMap();
-
     /**
      * Allow object to be initialized without any data. You would use
      * addObject() to add data later.
@@ -154,7 +151,7 @@
             Field f = (Field) fieldHash.get( fieldName );
             if (f != null)
             {
-                value = f.get(classHash.get(fieldName) );
+                value = f.get(null);
             }
         }
         catch( IllegalAccessException e )
@@ -181,7 +178,6 @@
             if ( Modifier.isStatic(mod) && Modifier.isPublic(mod) )
             {
                 fieldHash.put(fields[i].getName(), fields[i]);
-                classHash.put(fields[i].getName(), clas);
             }
         }
     }