Merge pull request #64 from nipreps/fix/loading-plugins-bug

FIX: Plugin inclusion via main bootstrap file did not work
nipreps · Mar 25, 2023 · 2269b86 · 2269b86
2 parents 7424386 + 0fc5d62
commit 2269b86
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/nireports/assembler/data/default.yml b/nireports/assembler/data/default.yml
@@ -253,6 +253,5 @@ sections:
 
 # Example pluging specification in bootstrap file:
 # plugins:
-# - rating-widget:
-#     module: nireports.assembler
-#     path: data/rating-widget/bootstrap.yml
+# - module: nireports.assembler
+#   path: data/rating-widget/bootstrap.yml
diff --git a/nireports/assembler/report.py b/nireports/assembler/report.py
@@ -294,7 +294,7 @@ def __init__(
         self.index(settings)
 
         # Override plugins specified in the bootstrap with arg
-        if plugins is not None:
+        if plugins is not None or (plugins := settings.get("plugins", [])):
             settings["plugins"] = [
                 load(Path(pkgrf(plugin["module"], plugin["path"])).read_text())
                 for plugin in plugins