ModuleService#getSingleInput: skip unfillable inputs

If an input is unfillable (i.e., the ModuleItem's isAutoFill method
returns false), then it is intended that preprocessors not fill in the
input with any sort of preset or "active" data value.

For example, the ActiveDisplayPreprocessor should not fill in a Display
input with the active display if that input is marked autoFill=false.

This fixes a longstanding bug with the Active*Preprocessor classes,
which made it impossible to exclude inputs from being handled by them.

Fixes http://trac.imagej.net/ticket/2022.

Author: ctrueden

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

@@ -374,6 +374,7 @@ private <T> ModuleItem<T> getSingleItem(final Module module,
 			final String name = item.getName();
 			final boolean resolved = module.isResolved(name);
 			if (resolved) continue; // skip resolved inputs
+			if (!item.isAutoFill()) continue; // skip unfillable inputs
 			if (!type.isAssignableFrom(item.getType())) continue;
 			if (result != null) return null; // multiple matching items
 			@SuppressWarnings("unchecked")

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

@@ -260,9 +260,9 @@ <M extends Module> Future<M> run(M module,
 	<M extends Module> M waitFor(Future<M> future);
 
 	/**
-	 * Checks the given module for a solitary unresolved input of the given type,
-	 * returning the relevant {@link ModuleItem} if found, or null if not exactly
-	 * one unresolved input of that type.
+	 * Checks the given module for a solitary unresolved fillable input of the
+	 * given type, returning the relevant {@link ModuleItem} if found, or null if
+	 * not exactly one unresolved fillable input of that type.
 	 */
 	<T> ModuleItem<T> getSingleInput(Module module, Class<T> type);