The following comment has been added to this issue:
Author: Jerome Lacoste
Created: Sun, 6 Jun 2004 6:00 PM
Body:
Maybe returning direct access to the internal properties field is not a good idea...
---------------------------------------------------------------------
View this comment:
http://jira.codehaus.org/browse/MAVEN-1066?page=comments#action_20419
---------------------------------------------------------------------
View the issue:
http://jira.codehaus.org/browse/MAVEN-1066
Here is an overview of the issue:
---------------------------------------------------------------------
Key: MAVEN-1066
Summary: No way to (re)initialize properties on a BaseObject subclass
Type: Bug
Status: Unassigned
Priority: Critical
Original Estimate: Unknown
Time Spent: Unknown
Remaining: Unknown
Project: maven
Components:
model
Fix Fors:
1.1
Versions:
1.0-rc1
Assignee:
Reporter: Jeffrey Bonevich
Created: Wed, 3 Dec 2003 12:48 AM
Updated: Sun, 6 Jun 2004 6:00 PM
Description:
The resolvedProperties() method gives me access to the 'properties' field, so I can
empty it if need be. getProperties(), setProperties() and addProperty() all access
combinedProperties w/o consequentially touching properties. But all other accessors
assume that 'properties' need only be initialized if it is null. What happens when
'properties' is emptied, but then new combinedProperties are added. In mevenide, I
want to be able to remove all combinedProperties and properties, and then add a new
set.
Here are some suggested fixes:
(1) make resolveProperties() public so it can be called to reinit the properties list
(a hack)
(2) check properties for emptiness as well as nullness (also a hack - just means that
property-less objects are biased against)
(3) invalidate properties whenever addProperty() or setProperties() gets called
(preferred)
---------------------------------------------------------------------
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]
