Merge pull request #19 from intersystems-ru/added-request-error-callback

moved to responseHandler
objectscript · Nov 12, 2020 · e20fcb9 · e20fcb9
2 parents 1fc6df0 + 1927269
commit e20fcb9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/readme.md b/readme.md
@@ -59,7 +59,6 @@ var setup = { // Object that contain settings. Properties in brackets can be mis
             , rowClick: function (row, rowData) { console.log(row, rowData); }
             , contentRendered: function () {}
             // triggers when request status is not 200 
-            , requestError: function (message, xhr) {}
             , cellSelected: function ({ x: Number, y: Number, leftHeaderColumnsNumber: Number, topHeaderRowsNumber: Number }) {
                 return false; // return false to block default click action
             }

diff --git a/source/js/DataSource.js b/source/js/DataSource.js
@@ -51,7 +51,8 @@ DataSource.prototype._post = function (url, data, callback) {
             if (typeof handler === "function") {
                 handler.call(self.LPT, {
                     url: url,
-                    status: xhr.status
+                    status: xhr.status,
+                    xhr: xhr
                 });
             }
         }
@@ -73,11 +74,6 @@ DataSource.prototype._post = function (url, data, callback) {
                 }
             })());
         } else if (xhr.readyState === 4 && xhr.status !== 200) {
-            handler = self.LPT.CONFIG.triggers["requestError"];
-            if (typeof handler === "function") {
-                handler.call(this, xhr.responseText || pivotLocale.get(3) + "<br/>" +
-                xhr.status + ": " + xhr.statusText, xhr);
-            }
             callback({
                 error: xhr.responseText || pivotLocale.get(3) + "<br/>" +
                        xhr.status + ": " + xhr.statusText