Ticket #510 (closed defect: fixed)

Opened 9 years ago

Last modified 9 years ago

Menu entry File::Revert does not work

Reported by: bdezonia Owned by: bdezonia
Priority: major Milestone: biweekly-2011: May-09 to May-20
Component: Core Version:
Severity: serious Keywords:
Cc: Blocked By:
Blocking:

Description

The file revert menu command does not work. Create an IJ2 plugin to replace its functionality.

Change History

comment:1 Changed 9 years ago by bdezonia

  • Status changed from new to accepted

comment:2 Changed 9 years ago by bdezonia

Added a RevertImage plugin to the core plugins. It's skeleton is correct but it cannot work correctly until Imglib IO stores the name of the original data source (absolute path or URL) with the ImgPlus.

comment:3 Changed 9 years ago by bdezonia

  • Owner changed from bdezonia to curtis
  • Status changed from accepted to assigned

comment:4 Changed 9 years ago by bdezonia

  • Owner changed from curtis to bdezonia

comment:5 Changed 9 years ago by bdezonia

  • Status changed from assigned to closed
  • Resolution set to fixed
  • Milestone changed from imagej-2.0-alpha3 to biweekly-2011: May-09 to May-20

It turns out IJ1's revert was not working because the underlying plane reference was getting changed via setProcessor() and LegacyImageMap was not aware of it. Made LegacyImageMap always sync plane references between IJ1 & IJ2 data structures when the IJ2 type is planar. This catches numerous cases of plugins changing data out from under IJ2.

We could still write a pure IJ2 Revert plugin. I have left the RevertImage plugin in svn revision 2833 as menu entry "IJ2 Revert". We can adapt that if necessary. Closing ticket for now.

Note: See TracTickets for help on using tickets.