@kaspernj/api-maker 1.0.190 → 1.0.193

package/package.json

@@ -16,7 +16,7 @@
     ]
   },
   "name": "@kaspernj/api-maker",
-  "version": "1.0.190",
+  "version": "1.0.193",
   "description": "",
   "main": "index.js",
   "repository": {
@@ -72,6 +72,6 @@
     "eslint-plugin-react": "^7.23.2",
     "i18n-on-steroids": "^1.0.2",
     "jest": "^28.0.3",
-    "jsdom": "^19.0.0"
+    "jsdom": "^20.0.0"
   }
 }

package/src/base-model.cjs

@@ -14,6 +14,17 @@ const {ValidationErrors} = require("./validation-errors.cjs")
 
 const shared = {}
 
+const newCustomEvent = (validationErrors) => {
+  return new CustomEvent("validation-errors", {detail: validationErrors})
+}
+
+const sendValidationErrorsEvent = (validationErrors, options) => {
+  if (options && options.form) {
+    const event = newCustomEvent(validationErrors)
+    options.form.dispatchEvent(event)
+  }
+}
+
 class BaseModel {
   static modelClassData () {
     throw new Error("modelClassData should be overriden by child")
@@ -177,7 +188,7 @@ class BaseModel {
         {}
       )
     } catch (error) {
-      this.parseValidationErrors(error, options)
+      BaseModel.parseValidationErrors({error, model: this, options})
       throw error
     }
 
@@ -190,7 +201,7 @@ class BaseModel {
   }
 
   async createRaw (rawData, options = {}) {
-    const objectData = this._objectDataFromGivenRawData(rawData, options)
+    const objectData = BaseModel._objectDataFromGivenRawData(rawData, options)
 
     let response
 
@@ -208,7 +219,7 @@ class BaseModel {
         {}
       )
     } catch (error) {
-      this.parseValidationErrors(error, options)
+      BaseModel.parseValidationErrors({error, model: this, options})
       throw error
     }
 
@@ -284,7 +295,7 @@ class BaseModel {
   }
 
   handleResponseError (response) {
-    this.parseValidationErrors(response)
+    BaseModel.parseValidationErrors({model: this, response})
     throw new new CustomError("Response wasn't successful", {model: this, response})
   }
 
@@ -299,33 +310,22 @@ class BaseModel {
     return false
   }
 
-  parseValidationErrors (error, options) {
+  static parseValidationErrors ({error, model, options}) {
     if (!(error instanceof CustomError)) return
     if (!error.args.response.validation_errors) return
 
     const validationErrors = new ValidationErrors({
-      model: this,
+      model,
       validationErrors: digg(error, "args", "response", "validation_errors")
     })
 
-    this.sendValidationErrorsEvent(validationErrors, options)
+    sendValidationErrorsEvent(validationErrors, options)
 
     if (!options || options.throwValidationError != false) {
       throw new ValidationError(validationErrors, digg(error, "args"))
     }
   }
 
-  sendValidationErrorsEvent (validationErrors, options) {
-    if (options && options.form) {
-      const event = this.newCustomEvent(validationErrors)
-      options.form.dispatchEvent(event)
-    }
-  }
-
-  newCustomEvent (validationErrors) {
-    return new CustomEvent("validation-errors", {detail: validationErrors})
-  }
-
   static humanAttributeName (attributeName) {
     const keyName = digg(this.modelClassData(), "i18nKey")
 
@@ -517,7 +517,7 @@ class BaseModel {
         {}
       )
     } catch (error) {
-      this.parseValidationErrors(error, options)
+      BaseModel.parseValidationErrors({error, model: this, options})
       throw error
     }
 
@@ -549,7 +549,7 @@ class BaseModel {
     this.setNewModelData(newModel)
   }
 
-  _objectDataFromGivenRawData (rawData, options) {
+  static _objectDataFromGivenRawData (rawData, options) {
     if (rawData instanceof FormData || rawData.nodeName == "FORM") {
       const formData = FormDataObjectizer.formDataFromObject(rawData, options)
 
@@ -560,8 +560,7 @@ class BaseModel {
   }
 
   async updateRaw (rawData, options = {}) {
-    const objectData = this._objectDataFromGivenRawData(rawData, options)
-
+    const objectData = BaseModel._objectDataFromGivenRawData(rawData, options)
     let response
 
     try {
@@ -579,7 +578,7 @@ class BaseModel {
         {}
       )
     } catch (error) {
-      this.parseValidationErrors(error, options)
+      BaseModel.parseValidationErrors({error, model: this, options})
       throw error
     }
 
@@ -636,8 +635,18 @@ class BaseModel {
     return this.uniqueKeyValue
   }
 
-  static _callCollectionCommand (args, commandArgs) {
-    return CommandsPool.addCommand(args, commandArgs)
+  static async _callCollectionCommand (args, commandArgs) {
+    const formOrDataObject = args.args
+
+    try {
+      return await CommandsPool.addCommand(args, commandArgs)
+    } catch (error) {
+      if (formOrDataObject.nodeName == "FORM") {
+        BaseModel.parseValidationErrors({error, options: {form: formOrDataObject}})
+      }
+
+      throw error
+    }
   }
 
   _callMemberCommand (args, commandArgs) {

package/src/commands-pool.cjs

@@ -61,18 +61,17 @@ module.exports = class ApiMakerCommandsPool {
 
       this.pool[id] = {resolve, reject}
 
-      if (!this.poolData[commandType])
-        this.poolData[commandType] = {}
-
-      if (!this.poolData[commandType][collectionName])
-        this.poolData[commandType][collectionName] = {}
-
-      if (!this.poolData[commandType][collectionName][commandName])
-        this.poolData[commandType][collectionName][commandName] = {}
+      if (!this.poolData[commandType]) this.poolData[commandType] = {}
+      if (!this.poolData[commandType][collectionName]) this.poolData[commandType][collectionName] = {}
+      if (!this.poolData[commandType][collectionName][commandName]) this.poolData[commandType][collectionName][commandName] = {}
 
       let args
 
-      if (data.args instanceof FormData) {
+      if (data.args?.nodeName == "FORM") {
+        const formData = new FormData(data.args)
+
+        args = FormDataObjectizer.toObject(formData)
+      } else if (data.args instanceof FormData) {
         args = FormDataObjectizer.toObject(data.args)
       } else {
         args = Serializer.serialize(data.args)

package/src/source-maps-loader.cjs

@@ -30,7 +30,7 @@ module.exports = class SourceMapsLoader {
 
     if (error) sources = sources.concat(this.getSourcesFromError(error))
 
-    return uniqunize(sources)
+    return uniqunize(sources, (source) => source.originalUrl)
   }
 
   async loadSourceMaps (error) {
@@ -40,11 +40,11 @@ module.exports = class SourceMapsLoader {
       const promises = []
       const sources = this.getSources(error)
 
-      for(const src of sources) {
-        if (src && !this.srcLoaded[src]) {
-          this.srcLoaded[src] = true
+      for(const source of sources) {
+        if (source.originalUrl && !this.srcLoaded[source.originalUrl]) {
+          this.srcLoaded[source.originalUrl] = true
 
-          const promise = this.loadSourceMapForSource(src)
+          const promise = this.loadSourceMapForSource(source)
           promises.push(promise)
         }
       }
@@ -60,7 +60,13 @@ module.exports = class SourceMapsLoader {
     const sources = []
 
     for (const trace of stack) {
-      sources.push(trace.file)
+      const file = trace.file
+
+      if (file != "\u003Canonymous>") {
+        const sourceMapUrl = this.getMapURL({src: file})
+
+        if (sourceMapUrl) sources.push({originalUrl: file, sourceMapUrl})
+      }
     }
 
     return sources
@@ -71,40 +77,41 @@ module.exports = class SourceMapsLoader {
     const sources = []
 
     for (const script of scripts) {
-      const src = this.loadSourceMapsForScriptTagsCallback(script)
+      const sourceMapUrl = this.getMapURL({script, src: script.src})
 
-      if (src) {
-        this.srcLoaded[src] = true
-        sources.push(src)
-      }
+      if (sourceMapUrl) sources.push({originalUrl: script.src, sourceMapUrl})
     }
 
     return sources
   }
 
-  async loadSourceMapForSource (src) {
+  getMapURL({script, src}) {
     const url = this.loadUrl(src)
     const originalUrl = `${url.origin}${url.pathname}`
 
-    let mapUrl
-
     if (this.sourceMapForSourceCallback) {
       // Use custom callback to resolve which map-file to download
-      mapUrl = this.sourceMapForSourceCallback({src, url})
-    } else {
+      return this.sourceMapForSourceCallback({originalUrl, script, src, url})
+    } else if (this.includeMapURL(src)) {
       // Default to original URL with '.map' appended
-      mapUrl = `${originalUrl}.map`
+      return `${originalUrl}.map`
     }
+  }
+
+  includeMapURL(src) {
+    return src.includes("/packs/") || src.includes("/assets/")
+  }
 
+  async loadSourceMapForSource ({originalUrl, sourceMapUrl}) {
     const xhr = new XMLHttpRequest()
 
-    xhr.open("GET", mapUrl, true)
+    xhr.open("GET", sourceMapUrl, true)
 
     await this.loadXhr(xhr)
 
     const consumer = await new SourceMapConsumer(xhr.responseText)
 
-    this.sourceMaps.push({consumer, originalUrl, src})
+    this.sourceMaps.push({consumer, originalUrl})
   }
 
   loadUrl (url) {