Merge pull request phoenixframework#1954 from korczis/master

Fix accesing onerror(status) instead of onerror(resp && resp.status)
boydm · Oct 5, 2016 · 0cefe23 · 0cefe23
2 parents cf67130 + 64aad60
commit 0cefe23
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/priv/static/phoenix.js b/priv/static/phoenix.js
@@ -993,7 +993,7 @@ var LongPoll = exports.LongPoll = function () {
 
       Ajax.request("POST", this.endpointURL(), "application/json", body, this.timeout, this.onerror.bind(this, "timeout"), function (resp) {
         if (!resp || resp.status !== 200) {
-          _this9.onerror(status);
+          _this9.onerror(resp && resp.status);
           _this9.closeAndRetry();
         }
       });

diff --git a/web/static/js/phoenix.js b/web/static/js/phoenix.js
@@ -703,7 +703,7 @@ export class LongPoll {
   send(body){
     Ajax.request("POST", this.endpointURL(), "application/json", body, this.timeout, this.onerror.bind(this, "timeout"), (resp) => {
       if(!resp || resp.status !== 200){
-        this.onerror(status)
+        this.onerror(resp && resp.status)
         this.closeAndRetry()
       }
     })