Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 99197 - For plugin editor, Revert action doesn't work
Summary: For plugin editor, Revert action doesn't work
Status: RESOLVED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows 2000
: P3 trivial (vote)
Target Milestone: 3.3 M6   Edit
Assignee: Mike Pawlowski CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 67747 135030 175333 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-06-09 12:30 EDT by Ian Graham CLA
Modified: 2007-02-23 13:20 EST (History)
4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ian Graham CLA 2005-06-09 12:30:59 EDT 
The Revert action does nothing in the plugin editor(RC1).  Admittedly there is 
some confusion about what to revert when more than one file is being edited by 
the editor, but I would assume all the associated files would revert(plugin.xml, 
MANIFEST.MF, build.properties).
Comment 1 Wassim Melhem CLA 2006-04-07 01:36:13 EDT 
*** Bug 135030 has been marked as a duplicate of this bug. ***
Comment 2 Mike Pawlowski CLA 2007-02-13 14:57:08 EST 
Fixed. Patch released to HEAD.

Target:  3.3 M6
Patch:   patch_99197_2007-02-13_13-57.txt

Performing the revert operation in the plug-in manifest editor source or
form pages will revert all three editor inputs:  plugin.xml, MANIFEST.MF, 
build.properties.

Work Done:

* Enabled revert operation for source pages

* Algorithm:
  -> Commit all active form page dirty field values to the model
  -> Flush any pending editor operations into the document
  -> Revert all three editor inputs to the contents of their last save to file
     - Reload workspace model
     - Fired events triggers a refresh of all form page fields and source page
       contents
  -> Undirty the editor
Comment 3 Mike Pawlowski CLA 2007-02-15 11:43:23 EST 
*** Bug 67747 has been marked as a duplicate of this bug. ***
Comment 4 Wassim Melhem CLA 2007-02-23 13:20:33 EST 
*** Bug 175333 has been marked as a duplicate of this bug. ***