jsf.js_next_gen 1.0.0-beta-7 → 1.0.0-beta-10

package/dist/window/jsf-development.js

@@ -611,7 +611,7 @@ var Implementation;
      *
      *
      *  handles the errors, in case of an onError exists within the context the onError is called as local error handler
-     *  the registered error handlers in the queue receiv an error message to be dealt with
+     *  the registered error handlers in the queue received an error message to be dealt with
      *  and if the projectStage is at development an alert box is displayed
      *
      *  note: we have additional functionality here, via the global config myfaces.config.defaultErrorOutput a function can be provided
@@ -1859,7 +1859,7 @@ var ExtLang;
             templateParams[_i - 2] = arguments[_i];
         }
         installedLocale = installedLocale !== null && installedLocale !== void 0 ? installedLocale : new Messages_1.Messages();
-        var msg = (_b = (_a = installedLocale[key]) !== null && _a !== void 0 ? _a : defaultMessage) !== null && _b !== void 0 ? _b : key + " - undefined message";
+        var msg = (_b = (_a = installedLocale[key]) !== null && _a !== void 0 ? _a : defaultMessage) !== null && _b !== void 0 ? _b : key;
         mona_dish_1.Stream.of.apply(mona_dish_1.Stream, templateParams).each(function (param, cnt) {
             msg = msg.replace(new RegExp(["\\{", cnt, "\\}"].join(Const_1.EMPTY_STR), "g"), param);
         });
@@ -2679,14 +2679,22 @@ var ResponseProcessor = /** @class */ (function () {
          */
         var mergedErrorData = new mona_dish_1.Config({});
         mergedErrorData.assign(Const_1.SOURCE).value = this.externalContext.getIf(Const_1.P_PARTIAL_SOURCE).get(0).value;
-        mergedErrorData.assign(Const_1.ERROR_NAME).value = node.getIf(Const_1.ERROR_NAME).textContent(Const_1.EMPTY_STR);
-        mergedErrorData.assign(Const_1.ERROR_MESSAGE).value = node.getIf(Const_1.ERROR_MESSAGE).cDATAAsString;
+        mergedErrorData.assign(Const_1.ERROR_NAME).value = node.querySelectorAll(Const_1.ERROR_NAME).textContent(Const_1.EMPTY_STR);
+        mergedErrorData.assign(Const_1.ERROR_MESSAGE).value = node.querySelectorAll(Const_1.ERROR_MESSAGE).cDATAAsString;
         var hasResponseXML = this.internalContext.get(Const_1.RESPONSE_XML).isPresent();
+        //we now store the response xml also in the error data for further details
         mergedErrorData.assignIf(hasResponseXML, Const_1.RESPONSE_XML).value = this.internalContext.getIf(Const_1.RESPONSE_XML).value.get(0).value;
+        // error post processing and enrichment (standard messages from keys)
         var errorData = ErrorData_1.ErrorData.fromServerError(mergedErrorData);
-        this.externalContext.getIf(Const_1.ON_ERROR).orElse(this.internalContext.getIf(Const_1.ON_ERROR).value).orElse(Const_1.EMPTY_FUNC).value(errorData);
+        // we now trigger an internally stored onError function which might be a attached to the context
+        // either we haven an internal on error, or an on error has been bassed via params from the outside
+        // in both cases they are attached to our contexts
+        this.triggerOnError(errorData);
         AjaxImpl_1.Implementation.sendError(errorData);
     };
+    ResponseProcessor.prototype.triggerOnError = function (errorData) {
+        this.externalContext.getIf(Const_1.ON_ERROR).orElse(this.internalContext.getIf(Const_1.ON_ERROR).value).orElse(Const_1.EMPTY_FUNC).value(errorData);
+    };
     /**
      * process the redirect operation
      *
@@ -3342,6 +3350,9 @@ var XhrRequest = /** @class */ (function () {
             xhrObject.open(this.ajaxType, (0, RequestDataResolver_1.resolveFinalUrl)(this.sourceForm, formData, this.ajaxType), true);
             //adding timeout
             this.timeout ? xhrObject.timeout = this.timeout : null;
+            xhrObject.onerror = function (data) {
+                _this.handleError(new Error("XHR Error: ".concat(JSON.stringify(data))));
+            };
             //a bug in the xhr stub library prevents the setRequestHeader to be properly executed on fake xhr objects
             //normal browsers should resolve this
             //tests can quietly fail on this one