fix request params

Izumi-kun · Aug 20, 2017 · ef9d910 · ef9d910
1 parent f9873f1
commit ef9d910
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 9 deletions.
diff --git a/dist/jquery.longpoll.min.js b/dist/jquery.longpoll.min.js
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "jquery-longpoll-client",
-  "version": "0.2.1",
+  "version": "0.3.0",
   "description": "Simple client-side longpoll implementation",
   "keywords": ["jquery", "ajax", "longpoll", "comet"],
   "main": "src/jquery.longpoll.js",

diff --git a/src/jquery.longpoll.js b/src/jquery.longpoll.js
@@ -91,17 +91,15 @@
         data: params,
         dataType: "json"
       }).done(function (response) {
-        var triggered = false;
         $.each(response.params, function (id, value) {
-          if (params[id] != value) {
-            triggered = true;
+          if (params[id] !== value) {
+            params = response.params;
+            if ($.isFunction(self.callback)) {
+              self.callback.call(self, response.data);
+            }
             return false;
           }
         });
-        params = response.params;
-        if (triggered && $.isFunction(self.callback)) {
-          self.callback.call(self, response.data);
-        }
         timeoutId = setTimeout(doLoop, settings.pollInterval);
       }).fail(function (e) {
         if (e.status) {