Skip to content

Commit c96036f

Browse files
authored
Merge pull request #331 from scijava/directive-script-processor-fixes
When processing #@script, register module with ModuleService
2 parents 560ad96 + 759e2ee commit c96036f

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

src/main/java/org/scijava/script/process/ScriptDirectiveScriptProcessor.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@
3838
import org.scijava.Priority;
3939
import org.scijava.log.LogService;
4040
import org.scijava.module.ModuleInfo;
41+
import org.scijava.module.ModuleService;
4142
import org.scijava.plugin.Parameter;
4243
import org.scijava.plugin.Plugin;
4344

@@ -95,7 +96,7 @@
9596
*
9697
* @author Curtis Rueden
9798
*/
98-
@Plugin(type = ScriptProcessor.class)
99+
@Plugin(type = ScriptProcessor.class, priority=Priority.HIGH)
99100
public class ScriptDirectiveScriptProcessor extends DirectiveScriptProcessor {
100101

101102
public ScriptDirectiveScriptProcessor() {
@@ -104,6 +105,9 @@ public ScriptDirectiveScriptProcessor() {
104105

105106
@Parameter
106107
private LogService log;
108+
109+
@Parameter
110+
private ModuleService moduleService;
107111

108112
// -- Internal DirectiveScriptProcessor methods --
109113

@@ -114,6 +118,7 @@ protected String process(final String directive,
114118
for (final String k : attrs.keySet()) {
115119
assignAttribute(k == null ? "name" : k, attrs.get(k));
116120
}
121+
moduleService.addModule(info()); // TODO how to handle duplicate names?
117122
return "";
118123
}
119124

0 commit comments

Comments
 (0)