[
https://issues.apache.org/jira/browse/OODT-41?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12971503#action_12971503
]
Brian Foster edited comment on OODT-41 at 12/14/10 8:01 PM:
------------------------------------------------------------
- however, the Product object has an argument which it doesn't use . . . this
is misleading . . . why is it okay for the Product constructor to just ignore
its transferStatus argument . . . we are talking about a one like fix, adding
the following to the constructor: this.transferStatus = transferStatus . . .
otherwise i would argue that constructor be deprecated and another one be
created without the 'String transferStatus' argument . . . this would be
problematic if someone were trying to clone a Product object and used this
constructor in question
(this is the last time i'm changing this comment . . . totally butchered it
with the whole structure instead of transferStatus . . . tired . . . time to go
home!!!)
was (Author: bfoster):
- however, the Product object has an argument which it doesn't use . . .
this is misleading . . . why is it okay for the Product constructor to just
ignore its transferStatus argument . . . we are talking about a one like fix,
adding the following to the constructor: this.transferStatus = transferStatus .
. . otherwise i would argue that constructor be deprecated and another one be
created without the 'String structure' argument . . . this would be problematic
if someone were trying to clone a Product object and used this constructor in
question
> Product Constructor does not set transfer status.
> -------------------------------------------------
>
> Key: OODT-41
> URL: https://issues.apache.org/jira/browse/OODT-41
> Project: OODT
> Issue Type: Bug
> Components: file manager
> Reporter: Paul Ramirez
> Priority: Minor
> Fix For: 0.2
>
>
> The construtor below takes in transferStatus but does nothing with it. Not
> sure if would break code to change this. Probably requires an update to a
> unit test too.
> public Product(String name, ProductType pType, String structure,
> String transferStatus, List<Reference> refs) {
> productName = name;
> productType = pType;
> productStructure = structure;
> references = refs;
> }
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
