closes #2586

src/favourites.js

@@ -172,6 +172,10 @@ var async = require('async'),
 				return callback(err);
 			}
 
+			if (!voteStatus || (!voteStatus.upvoted && !voteStatus.downvoted)) {
+				return callback();
+			}
+
 			var hook,
 				current = voteStatus.upvoted ? 'upvote' : 'downvote';
 
@@ -184,17 +188,17 @@ var async = require('async'),
 				current = 'unvote';
 			}
 
-			plugins.fireHook('action:post.' + hook, {
+			vote(voteStatus.upvoted ? 'downvote' : 'upvote', true, pid, uid, function(err, data) {
-				pid: pid,
+				if (err) {
-				uid: uid,
+					return callback(err);
-				current: current
+				}
+				plugins.fireHook('action:post.' + hook, {
+					pid: pid,
+					uid: uid,
+					current: current
+				});
+				callback(null, data);
 			});
-
-			if (!voteStatus || (!voteStatus.upvoted && !voteStatus.downvoted)) {
-				return callback();
-			}
-
-			vote(voteStatus.upvoted ? 'downvote' : 'upvote', true, pid, uid, callback);
 		});
 	}