Bugzilla – Full Text Bug Listing
| Summary: | [implementation] AbstractTextEditor does not prompt when out of sync in MultiPageEditorPart | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Nick Sandonato <nsand.dev> | ||||
| Component: | Text | Assignee: | Nick Sandonato <nsand.dev> | ||||
| Status: | RESOLVED FIXED | QA Contact: | |||||
| Severity: | normal | ||||||
| Priority: | P3 | CC: | browe, daniel_megert, remy.suen | ||||
| Version: | 3.6.2 | ||||||
| Target Milestone: | 3.6.2+ | ||||||
| Hardware: | All | ||||||
| OS: | All | ||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
Thanks for the patch Nick. Next time please also add your name to the copyright section (see HEAD how to do it). Fixed in HEAD. Available in builds >= I201102022-0800. I also added a 'null' check for complete safety. The fix causes an NPE on exit. Fixed in HEAD. Rational Application Developer needs a fix in the 3.6.2 stream. Could this be backported please? (In reply to comment #4) > Rational Application Developer needs a fix in the 3.6.2 stream. Could this be > backported please? Done. |
Created attachment 189409 [details] patch When an AbstractTextEditor is embedded as a page in a MultiPageEditorPart, the AbstractTextEditor is currently not updated as to when it has been activated. This leads to never prompting to replace the editor contents in the case of the file changing on the filesystem. It may be useful to check if the active part can adapt to AbstractTextEditor in addition to the active part being the AbstractTextEditor.