Ticket #1836 (closed defect: moved)
Opened 2013-04-23T17:17:59-05:00
LegacyService dispose code is fragile
| Reported by: | bdezonia | Owned by: | bdezonia |
|---|---|---|---|
| Priority: | major | Milestone: | imagej2-b8-analysis |
| Component: | Legacy Compatibility | Version: | |
| Severity: | serious | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | #1459 |
Description
The DefaultLegacyService dispose() method closes all windows before shutting down IJ1. However the code that closes all windows can fail. You can see this by inspecting IJ1's source code. The closeAllWindows() command relies on the idea that Editor windows with changes will not close without saving first. Right now if a legacy plugin opens a Editor window and changes its text then the closeAllWindows() should fail. We might need some CodeHacker or reflection code for setting Editor's changed flag to false.
Change History
comment:1 Changed 2014-05-02T21:13:09-05:00 by curtis
- Status changed from new to closed
- Resolution set to moved
Migrated to https://github.com/imagej/imagej-legacy/issues/1