NOTICE! This is a static HTML version of a legacy ImageJ Trac ticket.

The ImageJ project now uses GitHub Issues for issue tracking.

Please file all new issues there.

Ticket #547 (closed task: fixed)

Opened 2011-05-27T08:58:54-05:00

Last modified 2013-03-21T13:10:31-05:00

Make pure IJ2 Revert plugin

Reported by: bdezonia Owned by: bdezonia
Priority: major Milestone: imagej2-b2-bugfixes
Component: Core Version:
Severity: serious Keywords:
Cc: Blocked By: #1045
Blocking: #1171

Description (last modified by bdezonia)

Due to limitations laid out in bug #531 and bug #528 we need a pure IJ2 implementation of File::Revert. There is a skeleton implementation still in IJ2 as of 90fcdf5996c97a17ed65a096a31cebb4a50f095d. It is a core plugin called RevertImage. Flesh this skeleton out.

Change History

comment:1 Changed 2011-12-01T09:56:11-06:00 by bdezonia

  • Milestone changed from imagej-2.0-beta1 to imagej-2.0-beta2

comment:2 Changed 2011-12-01T10:09:03-06:00 by bdezonia

We should store image source as a string if it's a file name or URL name. Revert plugin can find original source and if its available then pass it to ImgOpener. ImgOpener would need to handle URLs too.

Note that some images cannot be reverted. For instance an image created by running a plugin.

comment:3 Changed 2012-02-26T22:36:47-06:00 by curtis

  • Blocking 1019 added

comment:4 Changed 2012-03-05T10:34:43-06:00 by curtis

  • Blocked By 1045 added

comment:5 Changed 2012-04-24T09:37:55-05:00 by curtis

  • Status changed from new to closed
  • Resolution set to fixed

Done in 879e1d002ba5b3227bec751b6c5e17e0e3ce1821, though it opens a new window at the moment.

comment:6 Changed 2012-05-14T12:02:30-05:00 by bdezonia

  • Blocking 1171 added; 1019 removed

comment:7 Changed 2012-05-18T12:25:15-05:00 by bdezonia

  • Milestone changed from imagej-2.0.0-beta3 to imagej-2.0.0-beta2

comment:8 Changed 2013-03-21T13:10:31-05:00 by bdezonia

  • Description modified

No longer seems to open a new window on revert.

Note that there are still problems. Since revert only reverts images that come from file (or URLSs???) it can't do things like revert a sample image to its original state.

I think what we really need here is for revert to do an undo all (at least if the image does not have a file/url source). This will need to wait until the undo code is in place.