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 #997 (closed task: fixed)

Opened 2012-02-22T15:02:04-06:00

Last modified 2012-08-13T15:52:08-05:00

Convert JHotDrawOverlayAdapter annotation to a Plugin type

Reported by: curtis Owned by: curtis
Priority: minor Milestone: imagej2-b4-plugins
Component: Plugin Framework Version:
Severity: minor Keywords:
Cc: Blocked By:
Blocking: #993, #1052

Description

As part of the effort to standardize discovered elements to use ImageJ's Plugin whenever possible, the JHotDrawOverlayAdapter SezPoz annotation could use the Plugin annotation instead, as a new type of IPlugin. The main obstacle here is that some classes are annotated with both JHotDrawOverlayAdapter and Tool, and it is not possible to have multiple Plugin annotations on the same class. So we need to find a way around this.

Change History

comment:1 Changed 2012-06-08T12:35:03-05:00 by curtis

  • Component changed from ij-tools to ij-ext

comment:2 Changed 2012-06-08T12:35:29-05:00 by curtis

  • Blocking 1052 added

comment:3 Changed 2012-07-10T17:39:24-05:00 by curtis

  • Blocking 993 added

comment:4 Changed 2012-08-07T14:47:23-05:00 by curtis

The requirement for two separate annotations (JHotDrawOverlayAdapter and Tool) is easily solved by making the IJHotDrawOverlayAdapter class a subinterface of imagej.ext.tool.Tool. The former interface has no methods apart from a priority() which @Plugins already have anyway.

comment:5 Changed 2012-08-09T17:10:50-05:00 by curtis

I completed initial work on this ticket on the  jhotdraw-tools branch, with the most telling commit being 183062e0. We'll merge to master soon, pending code review.

comment:6 Changed 2012-08-09T17:13:08-05:00 by curtis

  • Status changed from new to accepted

comment:7 Changed 2012-08-13T15:52:08-05:00 by curtis

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

Merged to master in 75070c55.