Move per-node-execution preprocessing into private method.

Signed-off-by: Squareys <[email protected]>

Author: Squareys

org.knime.scijava.commands/src/org/knime/scijava/commands/module/DefaultNodeModule.java

@@ -70,25 +70,32 @@ public DefaultNodeModule(final Context context, final ModuleInfo info,
         this.inputMapping = inputMapping;
         this.outputMapping = outputMapping;
         this.module = ms.createModule(info);
+        this.outputListener = new NodeModuleOutputChangedListener();
 
+        preProcess(params);
+    }
+
+    /*
+     * "Once per node execution" pre-processing. Called in constructor.
+     *
+     * @param params key-value pairs of pre resolved module inputs.
+     */
+    private void preProcess(final Map<String, Object> params) {
         // Setting parameters
         for (final Entry<String, Object> entry : params.entrySet()) {
             module.setInput(entry.getKey(), entry.getValue());
             module.resolveInput(entry.getKey());
         }
 
         // FIXME: do we need them all?
-        // FIXME add our own preprocessors, i.e. to call preprocess?
-        // FIXME or move this to "preprocess"
         final List<PreprocessorPlugin> pre = ps
                 .createInstancesOfType(PreprocessorPlugin.class);
 
         for (final ModulePreprocessor p : pre) {
             p.process(module);
         }
 
-        outputListener = new NodeModuleOutputChangedListener();
-        for (final ModuleItem<?> item : info.inputs()) {
+        for (final ModuleItem<?> item : this.module.getInfo().inputs()) {
             if (MultiOutputListener.class.isAssignableFrom(item.getType())) {
                 final String name = item.getName();