Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 197701 - [prov] Property change in the profile not being tracked
Summary: [prov] Property change in the profile not being tracked
Status: RESOLVED FIXED
Alias: None
Product: Equinox
Classification: Eclipse Project
Component: Incubator (show other bugs)
Version: 3.4   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.4 M4   Edit
Assignee: Simon Kaegi CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 208251
Blocks: 205223
  Show dependency tree
 
Reported: 2007-07-24 15:52 EDT by Pascal Rapicault CLA
Modified: 2007-12-08 22:33 EST (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Pascal Rapicault CLA 2007-07-24 15:52:43 EDT
When a property changes in a profile, there is no event associated with it making it impossible for the user level rollback support to keep track of the previous values.
It would be good to have a notification event when properties in a profile changes.
Comment 1 Susan McCourt CLA 2007-08-13 17:33:32 EDT
would also be good to have property change events for repo property changes
Comment 2 Pascal Rapicault CLA 2007-08-13 18:06:40 EDT
I opened bug #199806 to keep track of the requirement in comment #1.
Comment 3 Susan McCourt CLA 2007-10-25 17:54:38 EDT
I've just had to hack a ProfileEvent.CHANGE event.
My hack involved Profile.setInstallableUnitProfileProperty(...).
I did not attempt to address the other cases where this is needed.

See also bug #206077.
Comment 4 Susan McCourt CLA 2007-12-04 11:53:37 EST
Simon, this is also becoming important now for the UI for revert.  

When I look at a "saved profile" IU (See FormerState.profileToIU), I need to be able to know what the install roots on that profile were.  So those IU profile properties need to be retained somehow.
Comment 5 Simon Kaegi CLA 2007-12-07 00:43:38 EST
PropertyChange events are now being properly tracked in terms of commit/rollback with the code in HEAD. I'm going to add a dependency on bug 208251 as that's where I'm doing the bulk of the work and come back and close this once that's in good shape.
Comment 6 Simon Kaegi CLA 2007-12-08 22:33:30 EST
Marking FIXED.
Profile properties and ProfileIUProperties are now persisted along with all other changes to the Profile when a transaction in the Engine is committed.