[chain] CopyCommand

2005-01-07 Thread James Mitchell
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

Re: [chain] CopyCommand

2005-01-07 Thread Sean Schofield
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

Re: [chain] CopyCommand

2005-01-07 Thread James Mitchell
. 678.910.8017 AIM: jmitchtx - Original Message - From: Sean Schofield [EMAIL PROTECTED] To: Jakarta Commons Developers List commons-dev@jakarta.apache.org Sent: Friday, January 07, 2005 1:47 PM Subject: Re: [chain] CopyCommand You are correct. Actually, someone reported this bug already a few days

Re: [chain] CopyCommand

2005-01-07 Thread Craig McClanahan
On Fri, 7 Jan 2005 13:55:17 -0500, James Mitchell [EMAIL PROTECTED] wrote: I am a committer on Struts and a few Sandbox items, if I had the karma, I could fix this add a few more tests that demonstrate/validate the fix. It looks like you already have karma on jakarta-commons. I don't recall

DO NOT REPLY [Bug 32894] - [chain] CopyCommand does not work unless setValue is called

2005-01-07 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://issues.apache.org/bugzilla/show_bug.cgi?id=32894. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.

Re: [chain] CopyCommand

2005-01-07 Thread Sean Schofield
+1 for having you work on commons-chain. Just add yourself to the STATUS file. Yes the more the merrier! Craig sean - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]

Re: [chain] CopyCommand

2005-01-07 Thread James Mitchell
Cc: Jakarta Commons Developers List commons-dev@jakarta.apache.org Sent: Friday, January 07, 2005 2:00 PM Subject: Re: [chain] CopyCommand On Fri, 7 Jan 2005 13:55:17 -0500, James Mitchell [EMAIL PROTECTED] wrote: I am a committer on Struts and a few Sandbox items, if I had the karma, I could fix

DO NOT REPLY [Bug 32894] - [chain] CopyCommand does not work unless setValue is called

2004-12-31 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://issues.apache.org/bugzilla/show_bug.cgi?id=32894. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.

DO NOT REPLY [Bug 32894] - [chain] CopyCommand does not work unless setValue is called

2004-12-30 Thread bugzilla
/show_bug.cgi?id=32894 [EMAIL PROTECTED] changed: What|Removed |Added Summary|CopyCommand does not work |[chain] CopyCommand does not |unless setValue is called

DO NOT REPLY [Bug 31844] New: - [chain] CopyCommand should use new CatalogFactory

2004-10-21 Thread bugzilla
/show_bug.cgi?id=31844 [chain] CopyCommand should use new CatalogFactory Summary: [chain] CopyCommand should use new CatalogFactory Product: Commons Version: Nightly Builds Platform: Other OS/Version: Other Status: NEW Severity

DO NOT REPLY [Bug 31844] - [chain] CopyCommand should use new CatalogFactory

2004-10-21 Thread bugzilla
/show_bug.cgi?id=31844 [chain] CopyCommand should use new CatalogFactory --- Additional Comments From [EMAIL PROTECTED] 2004-10-22 03:31 --- Created an attachment (id=13180) [chain] [patch] LookupCommand - To unsubscribe, e