WIP: ScriptInfo: allow default values in scripts

[This is a WIP because it doesn't actually work in my tests.
Haven't had time to debug why yet, though.]

So now you can write:

  @int(value = 5) sigma

And the sigma variable will have a default value of 5 the first time
the script is run.

Nicer might be:

  @int sigma = 5

But doing that would be more involved, and so should probably wait
till script parameter parsing is truly fixed as described in #156.

Author: ctrueden

src/main/java/org/scijava/module/AbstractModuleItem.java

@@ -215,6 +215,11 @@ public String getWidgetStyle() {
 		return null;
 	}
 
+	@Override
+	public T getDefaultValue() {
+		return null;
+	}
+
 	@Override
 	public T getMinimumValue() {
 		return null;

src/main/java/org/scijava/module/DefaultModuleService.java

@@ -302,6 +302,8 @@ public <T> T load(final ModuleItem<T> item) {
 
 	@Override
 	public <T> T getDefaultValue(final ModuleItem<T> item) {
+		final T defaultValue = item.getDefaultValue();
+		if (defaultValue != null) return defaultValue;
 		final T min = item.getMinimumValue();
 		if (min != null) return min;
 		final T softMin = item.getSoftMinimum();

src/main/java/org/scijava/module/DefaultMutableModuleItem.java

@@ -59,6 +59,7 @@ public class DefaultMutableModuleItem<T> extends AbstractModuleItem<T>
 	private String initializer;
 	private String callback;
 	private String widgetStyle;
+	private T defaultValue;
 	private T minimumValue;
 	private T maximumValue;
 	private T softMinimum;
@@ -171,6 +172,11 @@ public void setWidgetStyle(final String widgetStyle) {
 		this.widgetStyle = widgetStyle;
 	}
 
+	@Override
+	public void setDefaultValue(final T defaultValue) {
+		this.defaultValue = defaultValue;
+	}
+
 	@Override
 	public void setMinimumValue(final T minimumValue) {
 		this.minimumValue = minimumValue;
@@ -259,6 +265,11 @@ public String getWidgetStyle() {
 		return widgetStyle;
 	}
 
+	@Override
+	public T getDefaultValue() {
+		return defaultValue;
+	}
+
 	@Override
 	public T getMinimumValue() {
 		return minimumValue;

src/main/java/org/scijava/module/ModuleItem.java

@@ -146,6 +146,9 @@ public interface ModuleItem<T> extends BasicDetails {
 	 */
 	String getWidgetStyle();
 
+	/** Gets the default value. */
+	T getDefaultValue();
+
 	/** Gets the minimum allowed value (if applicable). */
 	T getMinimumValue();
 

src/main/java/org/scijava/module/MutableModuleItem.java

@@ -60,6 +60,8 @@ public interface MutableModuleItem<T> extends ModuleItem<T> {
 
 	void setWidgetStyle(String widgetStyle);
 
+	void setDefaultValue(T defaultValue);
+
 	void setMinimumValue(T minimumValue);
 
 	void setMaximumValue(T maximumValue);

src/main/java/org/scijava/script/ScriptInfo.java

@@ -461,7 +461,7 @@ else if ("visibility".equalsIgnoreCase(key)) {
 			item.setVisibility(convertService.convert(value, ItemVisibility.class));
 		}
 		else if ("value".equalsIgnoreCase(key)) {
-			item.setWidgetStyle(value);
+			item.setDefaultValue(convertService.convert(value, item.getType()));
 		}
 		else {
 			throw new ScriptException("Invalid attribute name: " + key);