You are correct. Actually, someone reported this bug already a few days ago:
http://issues.apache.org/bugzilla/show_bug.cgi?id=32894 For those [chain] committers who are listening, I submitted a patch for that and its still waiting ;-) sean On Fri, 7 Jan 2005 13:11:22 -0500, James Mitchell <[EMAIL PROTECTED]> wrote: > Just poking around in the source (for my own learning) and came across this > bit of code. > > I won't even pretend to know what this is supposed to do, but unless I'm way > off, shouldn't this > doing " value = context.get(getFromKey()); " on line 5? > > 1 public boolean execute(Context context) throws Exception { > 2 > 3 Object value = this.value; > 4 if (value == null) { > 5 context.get(getFromKey()); > 6 } > 7 if (value != null) { > 8 context.put(getToKey(), value); > 9 } else { > 10 context.remove(getToKey()); > 11 } > 12 return (false); > 13 > 14 } > 15 > > -- > James Mitchell > Software Engineer / Open Source Evangelist > EdgeTech, Inc. > 678.910.8017 > AIM: jmitchtx > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
