You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@oodt.apache.org by "Ross Laidlaw (JIRA)" <ji...@apache.org> on 2013/03/08 00:06:13 UTC

[jira] [Created] (OODT-571) setWorkflowInst method needs 'this' keyword

Ross Laidlaw created OODT-571:
---------------------------------

             Summary: setWorkflowInst method needs 'this' keyword
                 Key: OODT-571
                 URL: https://issues.apache.org/jira/browse/OODT-571
             Project: OODT
          Issue Type: Bug
          Components: workflow manager
    Affects Versions: 0.5
            Reporter: Ross Laidlaw
            Assignee: Ross Laidlaw
            Priority: Minor
             Fix For: 0.6


I found a minor bug in a setter method in the org.apache.oodt.cas.workflow.engine.IterativeWorkflowProcessorThread class:

{code:title=IterativeWorkflowProcessorThread.java|borderStyle=solid}
public void setWorkflowInst(WorkflowInstance workflowInst)
{
  workflowInst = workflowInst;
}
{code}

It should be changed to the following so that the value from the parameter is assigned to the member variable:

{code:title=IterativeWorkflowProcessorThread.java|borderStyle=solid}
public void setWorkflowInst(WorkflowInstance workflowInst)
{
  this.workflowInst = workflowInst;
}
{code}

I've attached a patch for reference.  Although this should be a trivial change I wanted to post this issue and patch to make sure that the change would be ok.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira