Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
Message-ID: <196696756.392091266599908117.JavaMail.jira@brutus.apache.org>
Date: Fri, 19 Feb 2010 17:18:28 +0000 (UTC)
From: "Razan Abbass (JIRA)" <jira@apache.org>
To: issues@commons.apache.org
Subject: [jira] Commented: (CHAIN-6) [chain] CopyCommand does not work
 unless setValue is called
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/CHAIN-6?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12835833#action_12835833 ] 

Razan Abbass commented on CHAIN-6:
----------------------------------

[buy cheap tramadol on|http://thoughtmesh.net/meshes.php?group=33]
[buy tramadol online|http://thoughtmesh.net/meshes.php?group=34]
[buy tramadol cod|http://thoughtmesh.net/meshes.php?group=35]
[cheap tramadol|http://thoughtmesh.net/meshes.php?group=36]
[cheapest tramadol on line|http://thoughtmesh.net/meshes.php?group=37]
[cod accepted orders for tramadol|http://thoughtmesh.net/meshes.php?group=38]
[tramadol prescription from discount pharmacy|http://thoughtmesh.net/meshes.php?group=39]
[best price on tramadol 50 and 100 mg|http://thoughtmesh.net/meshes.php?group=40]
[ingredients in tramadol|http://thoughtmesh.net/meshes.php?group=41]
[online prescriptions tramadoltramadol online|http://thoughtmesh.net/meshes.php?group=42]
[tramadol price|http://thoughtmesh.net/meshes.php?group=43]
[side effects tramadol|http://thoughtmesh.net/meshes.php?group=44]
[tramadol hcl 50 mg|http://thoughtmesh.net/meshes.php?group=45]
[tramadol hydrochloride capsules 50mg|http://thoughtmesh.net/meshes.php?group=46]
[tramadol 180|http://thoughtmesh.net/meshes.php?group=47]
[addiction to tramadol|http://thoughtmesh.net/meshes.php?group=48]
[is tramadol a narcotic|http://thoughtmesh.net/meshes.php?group=49]
[canine tramadol dosage|http://thoughtmesh.net/meshes.php?group=50]
[cheap tramadol fedex overnight|http://thoughtmesh.net/meshes.php?group=51]
[tramadol prescription drug|http://thoughtmesh.net/meshes.php?group=52]
[side effects of tramadol hydrochloride|http://thoughtmesh.net/meshes.php?group=53]
[tramadol sales|http://thoughtmesh.net/meshes.php?group=54]
[is tramadol hcl a narcotic|http://thoughtmesh.net/meshes.php?group=55]
[tramadol hydrochloride 100mg|http://thoughtmesh.net/meshes.php?group=56]
[tramadol sale us no prescription required|http://thoughtmesh.net/meshes.php?group=57]
[tramadol medication|http://thoughtmesh.net/meshes.php?group=58]
[what is tramadol|http://thoughtmesh.net/meshes.php?group=59]
[tramadol 100 mg|http://thoughtmesh.net/meshes.php?group=60]
[tramadol no prescription fedex|http://thoughtmesh.net/meshes.php?group=61]
[buy cheap tramadol on|http://thoughtmesh.net/meshes.php?group=62]
[cheapest tramadol available online|http://thoughtmesh.net/meshes.php?group=63]
[order tramadol cod|http://thoughtmesh.net/meshes.php?group=64]
[buy tramadol cheap medication inurl|http://thoughtmesh.net/meshes.php?group=65]
[overnight tramadol|http://thoughtmesh.net/meshes.php?group=66]
[can i take tramadol pain pills if theyre a year old|http://thoughtmesh.net/meshes.php?group=67]
[tramadol pharmacy|http://thoughtmesh.net/meshes.php?group=68]
[tramadol pill|http://thoughtmesh.net/meshes.php?group=69]
[tramadol pills|http://thoughtmesh.net/meshes.php?group=70]
[tramadol prescription|http://thoughtmesh.net/meshes.php?group=71]
[tramadol hydrochloride|http://thoughtmesh.net/meshes.php?group=72]
[best price tramadol|http://thoughtmesh.net/meshes.php?group=73]
[tramadol sale us no prescription required|http://thoughtmesh.net/meshes.php?group=74]
[tramadol side effects|http://thoughtmesh.net/meshes.php?group=75]
[tramadol buy online|http://thoughtmesh.net/meshes.php?group=76]
[buy tultram online cheap|http://thoughtmesh.net/meshes.php?group=77]
[what are the withdrawal symptoms of tramadol|http://thoughtmesh.net/meshes.php?group=78]
[tramadol hcl|http://thoughtmesh.net/meshes.php?group=79]
[ingredients in tramadol|http://thoughtmesh.net/meshes.php?group=80]
[tramadol drug|http://thoughtmesh.net/meshes.php?group=81]


> [chain] CopyCommand does not work unless setValue is called
> -----------------------------------------------------------
>
>                 Key: CHAIN-6
>                 URL: https://issues.apache.org/jira/browse/CHAIN-6
>             Project: Commons Chain
>          Issue Type: Bug
>         Environment: Operating System: All
> Platform: All
>            Reporter: William Slough
>             Fix For: 1.1
>
>         Attachments: CopyCommand.patch
>
>
> CopyCommand appears to only work if the setValue method is run.  The
> documentation says "... a context attribute stored under the fromKey (if any),
> to the toKey".  If only the fromKey and toKey are set, the toKey and its stored
> value are deleted from the Context regardless of fromKey's value.  This should
> only happen if the fromKey value is null.
> In the execute method of CopyCommand, it appears that
> if (value == null) {
>             context.get(getFromKey());
>         }
> should read
> if (value == null) {
>             value = context.get(getFromKey());
>         }
> then the rest of the code should function as expected:
> if (value != null) {
>         context.put(getToKey(), value);
>     } else {
>         context.remove(getToKey());
>     }
>     return (false);

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.