feat: support preRegistration hook, and extension configuration at ti…

…me of registration

* Remove default config for tools

* feat: support preRegistration hook, and extension configuration at time of registration

* Prefer built-in check for array

src/extensions/ExtensionManager.js

@@ -15,21 +15,31 @@ export default class ExtensionManager {
   }
 
   /**
+   * An array of extensions, or an array of arrays that contains extension
+   * configuration pairs.
    *
    * @param {Object[]} extensions - Array of extensions
    */
   registerExtensions(extensions) {
     extensions.forEach(extension => {
-      this.registerExtension(extension);
+      const hasConfiguration = Array.isArray(extension);
+
+      if (hasConfiguration) {
+        const [ohifExtension, configuration] = extensions;
+        this.registerExtension(ohifExtension, configuration);
+      } else {
+        this.registerExtension(extension);
+      }
     });
   }
 
   /**
    *
    * TODO: Id Management: SopClassHandlers currently refer to viewport module by id; setting the extension id as viewport module id is a workaround for now
    * @param {Object} extension
+   * @param {Object} configuration
    */
-  registerExtension(extension) {
+  registerExtension(extension, configuration = {}) {
     if (!extension) {
       log.warn(
         'Attempting to register a null/undefined extension. Exiting early.'
@@ -54,6 +64,11 @@ export default class ExtensionManager {
       return;
     }
 
+    // preRegistrationHook
+    if (extension.preRegistration) {
+      extension.preRegistration(configuration);
+    }
+
     // Register Modules
     this.moduleTypeNames.forEach(moduleType => {
       const extensionModule = this._getExtensionModule(