new API response for like/dislike

isso/js/app/api.js

@@ -169,18 +169,18 @@ define(["app/lib/promise", "app/globals"], function(Q, globals) {
         return deferred.promise;
     };
 
-    var like = function(id) {
+    var vote = function(type) {
-        var deferred = Q.defer();
+        return function(id) {
-        curl("POST", endpoint + "/id/" + id + "/like", null,
+            var deferred = Q.defer();
-            function(rv) { deferred.resolve(JSON.parse(rv.body)); });
+            curl("POST", endpoint + "/id/" + id + "/" + type, null, function(rv) {
-        return deferred.promise;
+                if (rv.status === 200) {
-    };
+                    return deferred.resolve(null);
-
+                } else {
-    var dislike = function(id) {
+                    return deferred.reject("Unable to vote!");
-        var deferred = Q.defer();
+                }
-        curl("POST", endpoint + "/id/" + id + "/dislike", null,
+            });
-            function(rv) { deferred.resolve(JSON.parse(rv.body)); });
+            return deferred.promise;
-        return deferred.promise;
+        };
     };
 
     return {
@@ -193,7 +193,7 @@ define(["app/lib/promise", "app/globals"], function(Q, globals) {
         view: view,
         fetch: fetch,
         count: count,
-        like: like,
+        like: vote("like"),
-        dislike: dislike
+        dislike: vote("dislike")
     };
 });

isso/js/app/isso.js

@@ -144,29 +144,36 @@ define(["app/dom", "app/utils", "app/config", "app/api", "app/jade", "app/i18n",
             }
         );
 
-        // update vote counter, but hide if votes sum to 0
+        var vote = (function() {
-        var votes = function(value) {
+            var span = $("span.votes", footer),
-            var span = $("span.votes", footer);
+                value = span === null ? 0 : parseInt(span.textContent, 10);
-            if (span === null && value !== 0) {
+
-                footer.prepend($.new("span.votes", value));
+            return function(n) {
-            } else {
+                value += n;
+
                 if (value === 0) {
-                    span.remove();
+                    if (span !== null) {
+                        span.remove();
+                        span = null;
+                    }
                 } else {
+                    if (span === null) {
+                        span = footer.prepend($.new("span.votes"));
+                    }
                     span.textContent = value;
                 }
-            }
+            };
-        };
+        }());
 
         $("a.upvote", footer).on("click", function() {
-            api.like(comment.id).then(function(rv) {
+            api.like(comment.id).then(function() {
-                votes(rv.likes - rv.dislikes);
+                vote(+1);
             });
         });
 
         $("a.downvote", footer).on("click", function() {
-            api.dislike(comment.id).then(function(rv) {
+            api.dislike(comment.id).then(function() {
-                votes(rv.likes - rv.dislikes);
+                vote(-1);
             });
         });