commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Keese" <bi...@tech.beacon-it.co.jp>
Subject [jelly] PATCH - threads / context not set
Date Fri, 06 Jun 2003 01:55:32 GMT
This fixes the bug where the context variable is not accessible from
threads.  Inside a <threads:thread> call you can't do things like
${context.getVariable(foo)}

init() correctly sets the "context" variable, but it is erased by the
setVariables() call.  So I think this (or something similar) is an
appropriate fix.

Bill

Index: JellyContext.java
===================================================================
RCS file:
/home/cvspublic/jakarta-commons/jelly/src/java/org/apache/commons/jelly/Jell
yContext.java,v
retrieving revision 1.42
diff -u -r1.42 JellyContext.java
--- JellyContext.java 7 Mar 2003 13:17:54 -0000 1.42
+++ JellyContext.java 6 Jun 2003 01:53:19 -0000
@@ -324,6 +324,7 @@

     public void setVariables(Map variables) {
         this.variables = variables;
+        this.variables.put("context", this);
     }

     /**

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message