Hello,
The webrev:
http://cr.opensolaris.org/~migi/12881_changes_lost_modify_pubs/
Fixes the bug:
http://defect.opensolaris.org/bz/show_bug.cgi?id=12881
How it works?
Basically if we are clicking on the ok button in the modify publisher
dialog, then we are reusing the data model from the modify publisher
treeview and we are replacing two things which might change using modify
publisher dialog and were stored in our list model:
- alias of the publisher
- publisher object itself
The rest things in our list model needs to be preserved:
- pubs marked for removal
- priority changes for the publishers
- pubs marked for disabling/enabling
- name of the publisher (as it can not change)
The number of the publishers also could not changed as we are marking
the pubs for removal and not removing them, so we can safely use:
pub = pubs[j] to get information about the changed publisher and the
publisher object (line 450).
At the end, after setting model to the main treeview, we can restore
selections, we need to do this after attaching the model to the treeview.
--
best
Michal Pryc
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
