/*
 * #%L
 * ImageJ software for multidimensional image processing and analysis.
 * %%
 * Copyright (C) 2009 - 2013 Board of Regents of the University of
 * Wisconsin-Madison, Broad Institute of MIT and Harvard, and Max Planck
 * Institute of Molecular Cell Biology and Genetics.
 * %%
 * Redistribution and use in source and binary forms, with or without
 * modification, are permitted provided that the following conditions are met:
 * 
 * 1. Redistributions of source code must retain the above copyright notice,
 *    this list of conditions and the following disclaimer.
 * 2. Redistributions in binary form must reproduce the above copyright notice,
 *    this list of conditions and the following disclaimer in the documentation
 *    and/or other materials provided with the distribution.
 * 
 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
 * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
 * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
 * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDERS OR CONTRIBUTORS BE
 * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
 * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
 * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
 * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
 * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
 * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
 * POSSIBILITY OF SUCH DAMAGE.
 * 
 * The views and conclusions contained in the software and documentation are
 * those of the authors and should not be interpreted as representing official
 * policies, either expressed or implied, of any organization.
 * #L%
 */

package imagej.core.commands.display.interactive;

import imagej.command.ContextCommand;
import imagej.data.display.ImageDisplay;
import imagej.data.overlay.ThresholdOverlay;
import imagej.data.overlay.ThresholdService;
import imagej.menu.MenuConstants;
import imagej.plugin.Menu;
import imagej.plugin.Parameter;
import imagej.plugin.Plugin;

/**
 * TODO
 * 
 * @author Barry DeZonia
 */
@Plugin(iconPath = "/icons/bricks.png", menu = {
	@Menu(label = MenuConstants.IMAGE_LABEL, weight = MenuConstants.IMAGE_WEIGHT,
		mnemonic = MenuConstants.IMAGE_MNEMONIC), @Menu(label = "Adjust"),
	@Menu(label = "Threshold...", accelerator = "control shift T") })
public class Threshold extends ContextCommand {

	// -- Parameters --

	@Parameter
	private ThresholdService service;

	@Parameter
	private ImageDisplay display;

	@Parameter
	private double min;

	@Parameter
	private double max;

	// -- accessors --

	public void setImageDisplay(final ImageDisplay disp) {
		display = disp;
	}

	public ImageDisplay getImageDisplay() {
		return display;
	}

	// -- Command methods --

	@Override
	public void run() {
		ThresholdOverlay overlay = service.getThreshold(display);
		overlay.setRange(min, max);
		overlay.update();
		// or display.update()?
		// or display be an ItemIO.BOTH?
	}

}

@@ -2073,16 +2073,45 @@
 stants;%0A
+import imagej.module.ItemIO;%0A
 import i
@@ -2129,16 +2129,16 @@
 n.Menu;%0A
-
 import i
@@ -2622,32 +2622,52 @@
 ce;%0A%0A%09@Parameter
+(type = ItemIO.BOTH)
 %0A%09private ImageD
@@ -2972,16 +2972,101 @@
 run() %7B%0A
+%09%09if (min %3E max) %7B%0A%09%09%09cancel(%22Invalid threshold setting: min %3E max%22);%0A%09%09%09return;%0A%09%09%7D%0A
 %09%09Thresh
@@ -3153,86 +3153,478 @@
 ;%0A%09%09
-overlay.update();%0A%09%09// or display.update()?%0A%09%09// or display be an ItemIO.BOTH?
+// TODO - determine correct strategy%0A%09%09//%0A%09%09// overlay.update(); // works but eventually overwritten with all red%0A%09%09// OR%0A%09%09// display.update(); // kills ability to see anything but red%0A%09%09// OR%0A%09%09// display be an ItemIO.BOTH? has same limitation as disp.update().%0A%09%09// Actually it draws thresh fig correct, then draws all red thresh & fig%0A%09%09// over it. Delete in ovr mgr and return to correct fig. No go away tho%0A%09%09// nothing in ovr mgr. Also editing fill has no effect.
 %0A%09%7D%0A

Change how plugin updates overlay/display