The following issue has been updated:
Updater: Jerome Lacoste (mailto:[EMAIL PROTECTED])
Date: Fri, 4 Jun 2004 10:10 AM
Comment:
This new version of patch also improves the documentation It removes the setId() code
documentation and puts it in the setId javadoc.
It also adds 2 notes:
- the base class id is never set
- it is possible that getId method won't return the value passed as argument to the
setter method.
Changes:
Attachment changed to maven.1.0.final.dependency.cleanup_v2.txt
---------------------------------------------------------------------
For a full history of the issue, see:
http://jira.codehaus.org/browse/MAVEN-1305?page=history
---------------------------------------------------------------------
View the issue:
http://jira.codehaus.org/browse/MAVEN-1305
Here is an overview of the issue:
---------------------------------------------------------------------
Key: MAVEN-1305
Summary: dependency id is never set. Clean up getId() method
Type: Improvement
Status: Unassigned
Priority: Trivial
Original Estimate: 5 minutes
Time Spent: Unknown
Remaining: 5 minutes
Project: maven
Components:
core
Versions:
1.0-rc3
Assignee:
Reporter: Jerome Lacoste
Created: Thu, 3 Jun 2004 11:06 PM
Updated: Fri, 4 Jun 2004 10:10 AM
Description:
setId() in the BaseObject class is overriden by setId() in the Dependency class.
So I don't see how id can be set to a non null value.
Clean up o.a.m.project.Dependency#getId() can be made.
---------------------------------------------------------------------
JIRA INFORMATION:
This message is automatically generated by JIRA.
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
If you want more information on JIRA, or have a bug to report see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
