Author: michiel
Date: 2009-11-19 17:10:34 +0100 (Thu, 19 Nov 2009)
New Revision: 39810

Modified:
   
mmbase/trunk/applications/taglib/src/main/java/org/mmbase/bridge/jsp/taglib/util/CollectorBacking.java
   
mmbase/trunk/applications/taglib/src/test/java/org/mmbase/bridge/jsp/taglib/StringListTagTest.java
Log:
fixes failing test-case

Modified: 
mmbase/trunk/applications/taglib/src/main/java/org/mmbase/bridge/jsp/taglib/util/CollectorBacking.java
===================================================================
--- 
mmbase/trunk/applications/taglib/src/main/java/org/mmbase/bridge/jsp/taglib/util/CollectorBacking.java
      2009-11-19 15:52:28 UTC (rev 39809)
+++ 
mmbase/trunk/applications/taglib/src/main/java/org/mmbase/bridge/jsp/taglib/util/CollectorBacking.java
      2009-11-19 16:10:34 UTC (rev 39810)
@@ -67,6 +67,15 @@
         return super.put(key, value, r);
     }
 
+    @Override
+    public void setJspVar(PageContext pc, String jspvar, int vartype, Object 
value) {
+        if (jspvar == null) return;
+        if (value == null) return;
+        myKeys.add(jspvar);
+        super.setJspVar(pc, jspvar, vartype, value);
+    }
 
 
+
+
 }

Modified: 
mmbase/trunk/applications/taglib/src/test/java/org/mmbase/bridge/jsp/taglib/StringListTagTest.java
===================================================================
--- 
mmbase/trunk/applications/taglib/src/test/java/org/mmbase/bridge/jsp/taglib/StringListTagTest.java
  2009-11-19 15:52:28 UTC (rev 39809)
+++ 
mmbase/trunk/applications/taglib/src/test/java/org/mmbase/bridge/jsp/taglib/StringListTagTest.java
  2009-11-19 16:10:34 UTC (rev 39810)
@@ -66,7 +66,7 @@
 
             fooValue = "bla" + (++index);
             Import.tag(pageContext, tag, "foo", fooValue, true);
-
+            Import.tag(pageContext, tag, "somethingelse", fooValue, false);
             Import.jspvar(pageContext, tag, "jspvarfoo", fooValue, "String", 
false);
 
             assertEquals("bla" + index, pageContext.getAttribute("foo"));

_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to