adapt-authoring-adaptframework 2.0.3 → 2.0.4

package/lib/AdaptFrameworkImport.js

@@ -661,40 +661,41 @@ class AdaptFrameworkImport {
         throw App.instance.errors.FW_IMPORT_MISSING_PLUGINS
           .setData({ plugins: pluginsToInstall.join(', ') })
       }
-      const errors = []
-      await Promise.all([...pluginsToInstall, ...pluginsToUpdate].map(async p => {
+      const allPlugins = [...pluginsToInstall, ...pluginsToUpdate]
+      // pre-process: store update metadata and fix missing targetAttributes
+      for (const p of allPlugins) {
+        const isUpdate = pluginsToUpdate.includes(p)
+        if (isUpdate && this.installedPlugins[p]) {
+          this.updatedContentPlugins[p] = this.installedPlugins[p]
+        }
+        const pluginBowerPath = path.join(this.usedContentPlugins[p].path, 'bower.json')
+        const bowerJson = await readJson(pluginBowerPath)
+        if (!bowerJson.targetAttribute) {
+          bowerJson.targetAttribute = `_${bowerJson.component || bowerJson.extension || bowerJson.menu || bowerJson.theme}`
+          await writeJson(pluginBowerPath, bowerJson)
+        }
+        this.statusReport.info.push({ code: 'INSTALL_PLUGIN', data: { name: p, version: bowerJson.version } })
+      }
+      if (!this.settings.isDryRun) {
         try {
-          // Store original plugin metadata for updates before overwriting
-          const isUpdate = pluginsToUpdate.includes(p)
-          if (isUpdate && this.installedPlugins[p]) {
-            this.updatedContentPlugins[p] = this.installedPlugins[p]
-          }
-          // try and infer a targetAttribute if there isn't one
-          const pluginBowerPath = path.join(this.usedContentPlugins[p].path, 'bower.json')
-          const bowerJson = await readJson(pluginBowerPath)
-          if (!bowerJson.targetAttribute) {
-            bowerJson.targetAttribute = `_${bowerJson.component || bowerJson.extension || bowerJson.menu || bowerJson.theme}`
-            await writeJson(pluginBowerPath, bowerJson)
-          }
-          if (!this.settings.isDryRun) {
-            const [pluginData] = await this.contentplugin.installPlugins([[p, this.usedContentPlugins[p].path]], { strict: true })
-            if (!isUpdate) {
-              this.newContentPlugins[p] = pluginData
+          const installed = await this.contentplugin.installPlugins(
+            allPlugins.map(p => [p, this.usedContentPlugins[p].path]),
+            { strict: true }
+          )
+          installed.forEach(pluginData => {
+            if (!pluginsToUpdate.includes(pluginData.name)) {
+              this.newContentPlugins[pluginData.name] = pluginData
             }
-          }
-          this.statusReport.info.push({ code: 'INSTALL_PLUGIN', data: { name: p, version: bowerJson.version } })
+          })
         } catch (e) {
-          if (e.code === 'EEXIST') {
-            log('warn', 'PLUGIN_ALREADY_EXISTS', p)
-          } else {
-            log('error', 'PLUGIN_IMPORT_FAILED', p, e)
-            errors.push({ plugin: p, error: e.data?.errors?.[0] ?? e })
-          }
+          const errors = e.data?.errors?.map(err => ({
+            plugin: err.data?.name ?? 'unknown',
+            error: err.data?.errors?.[0] ?? err
+          })) ?? [{ plugin: 'unknown', error: e }]
+          errors.forEach(err => log('error', 'PLUGIN_IMPORT_FAILED', err.plugin, err.error))
+          throw App.instance.errors.FW_IMPORT_PLUGINS_FAILED
+            .setData({ errors: errors.map(e => App.instance.lang.translate(undefined, e)).join(', ') })
         }
-      }))
-      if (errors.length) {
-        throw App.instance.errors.FW_IMPORT_PLUGINS_FAILED
-          .setData({ errors: errors.map(e => App.instance.lang.translate(undefined, e)).join(', ') })
       }
     }
     this.componentNameMap = Object.values({ ...this.installedPlugins, ...this.newContentPlugins }).reduce((m, v) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "adapt-authoring-adaptframework",
-  "version": "2.0.3",
+  "version": "2.0.4",
   "description": "Adapt framework integration for the Adapt authoring tool",
   "homepage": "https://github.com/adapt-security/adapt-authoring-adaptframework",
   "license": "GPL-3.0",