Alon Bar-Lev has posted comments on this change. Change subject: aaa: extension-manager: support javascript extensions ......................................................................
Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/27281/1/backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionsManager.java File backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionsManager.java: Line 105: props.getProperty(Base.ConfigKeys.BINDINGS_SCRIPTENGINE_SCRIPT)); Line 106: Reader reader = new InputStreamReader(in); Line 107: ) { Line 108: ScriptEngine engine = new ScriptEngineManager().getEngineByName( Line 109: props.getProperty(Base.ConfigKeys.BINDINGS_SCRIPTENGINE_ENGINE)); > see my previous comment n the namespace, for example - I do not like multiple namespaces, it becomes more complex. also please notice that nobody will every actually use these... as they are hardcoded within the property files. Line 110: if (engine == null) { Line 111: throw new ConfigurationException(String.format("Script engine '%1$s' cannot be loaded.", Line 112: props.getProperty(Base.ConfigKeys.BINDINGS_SCRIPTENGINE_ENGINE))); Line 113: } -- To view, visit http://gerrit.ovirt.org/27281 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6d9dee411f5b0bf5019a35810f5f43b53ca3f3ce Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
