Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 311735 - uiCustom editor doesn't reflect file changes
Summary: uiCustom editor doesn't reflect file changes
Status: CLOSED FIXED
Alias: None
Product: MoDisco
Classification: Modeling
Component: Infrastructure (show other bugs)
Version: 0.8.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 0.8.0   Edit
Assignee: Nicolas Bros CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-05 10:33 EDT by Nicolas Bros CLA
Modified: 2010-10-01 13:37 EDT (History)
3 users (show)

See Also:
nicolas.bros: helios+
hugo.bruneliere: review+

Attachments
patch (17.73 KB, patch)
2010-05-12 07:03 EDT, Nicolas Bros CLA 		no flags Details | Diff
View All 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 Nicolas Bros CLA 2010-05-05 10:33:42 EDT 
When a uiCustom file is changed or deleted from outside the editor, the uiCustom editor stays open on the old version of the file. 

When the file is deleted, the editor should close automatically.
And when the file is modified, the user should be notified, and asked whether to replace the editor's contents with those from the file on disk.
Comment 1 Gregoire Dupe CLA 2010-05-10 05:58:40 EDT 
Hi,

To many changes (including UI modifications), this bug will then be handle for 0.9.

Regards,
Grégoire
Comment 2 Nicolas Bros CLA 2010-05-12 07:03:32 EDT 
Created attachment 168118 [details]
patch

This patch handles resource changes and deletions. It doesn't change the UI (apart from a yes/no prompt to discard changes), so it can be integrated for RC1.
Comment 3 Hugo Bruneliere CLA 2010-05-12 08:06:11 EDT 
Hi Nicolas,

You can go ahead with the patch.

Regards,

Hugo
Comment 4 Nicolas Bros CLA 2010-05-12 08:59:22 EDT 
Thanks. I have applied the patch to branch 0_8_0.
Comment 5 Hugo Bruneliere CLA 2010-10-01 13:37:46 EDT 
Patch successfully committed.