You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@velocity.apache.org by cb...@apache.org on 2014/04/23 03:34:19 UTC

svn commit: r1589329 - /velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/ValueParser.java

Author: cbrisson
Date: Wed Apr 23 01:34:19 2014
New Revision: 1589329

URL: http://svn.apache.org/r1589329
Log:
bugfix for r/w ValueParser

Modified:
    velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/ValueParser.java

Modified: velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/ValueParser.java
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/ValueParser.java?rev=1589329&r1=1589328&r2=1589329&view=diff
==============================================================================
--- velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/ValueParser.java (original)
+++ velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/ValueParser.java Wed Apr 23 01:34:19 2014
@@ -581,7 +581,7 @@ public class ValueParser extends Convers
         {
             hasSubkeys = Boolean.TRUE;
         }
-        return source.put(key,value); // TODO this tool should be made thread-safe (the request-scoped ParameterTool doesn't need it, but other uses could...)
+        return getSource().put(key,value); // TODO this tool should be made thread-safe (the request-scoped ParameterTool doesn't need it, but other uses could...)
     }
 
     public Object remove(Object key)
@@ -594,7 +594,7 @@ public class ValueParser extends Convers
         {
             hasSubkeys = null;
         }
-        return source.remove(key);
+        return getSource().remove(key);
     }
 
     public void putAll(Map<? extends String,? extends Object> m) {
@@ -603,7 +603,7 @@ public class ValueParser extends Convers
             throw new UnsupportedOperationException("Cannot putAll("+m+"); "+getClass().getName()+" is read-only");
         }
         hasSubkeys = null;
-        source.putAll(m);
+        getSource().putAll(m);
     }
 
     public void clear() {
@@ -612,7 +612,7 @@ public class ValueParser extends Convers
             throw new UnsupportedOperationException("Cannot clear(); "+getClass().getName()+" is read-only");
         }
         hasSubkeys = Boolean.FALSE;
-        source.clear();
+        getSource().clear();
     }
 
     public Set<String> keySet() {