From acdca15e6a54516ced1299bf25a55588ad457131 Mon Sep 17 00:00:00 2001
From: Baris Soner Usakli <barisusakli@gmail.com>
Date: Fri, 21 Jun 2013 12:23:49 -0400
Subject: [PATCH 1/3] possible fix for issue #12

---
 src/posts.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/posts.js b/src/posts.js
index 30807e830f..99ec325a54 100644
--- a/src/posts.js
+++ b/src/posts.js
@@ -185,12 +185,12 @@ marked.setOptions({
 
 					Posts.get_cid_by_pid(pid, function(cid) {
 						RDB.del('cid:' + cid + ':read_by_uid');
+						
 						RDB.zadd('categories:recent_posts:cid:' + cid, Date.now(), pid);
+
+						topics.markAsRead(tid, uid);
 					});
 
-					// Re-add the poster, so he/she does not get an "unread" flag on this topic
-					topics.markAsRead(tid, uid);
-					// this will duplicate once we enter the thread, which is where we should be going
 
 					socket.emit('event:alert', {
 						title: 'Reply Successful',

From e49d2707ebacc5e75505aebb4e4d94a3c773bc83 Mon Sep 17 00:00:00 2001
From: Baris Soner Usakli <barisusakli@gmail.com>
Date: Fri, 21 Jun 2013 12:53:50 -0400
Subject: [PATCH 2/3] changed the follower/following keys

---
 src/posts.js      |  2 +-
 src/user.js       | 39 ++++++++++++++++-----------------------
 src/websockets.js |  2 --
 3 files changed, 17 insertions(+), 26 deletions(-)

diff --git a/src/posts.js b/src/posts.js
index 99ec325a54..f287928ae6 100644
--- a/src/posts.js
+++ b/src/posts.js
@@ -203,7 +203,7 @@ marked.setOptions({
 					threadTools.notify_followers(tid, uid);
 
 					user.getUserFields(uid, ['username','reputation','picture','signature'], function(data) {
-						
+
 						var timestamp = Date.now();
 						
 						io.sockets.in('topic_' + tid).emit('event:new_post', {
diff --git a/src/user.js b/src/user.js
index 887263cba9..cc08d41cf1 100644
--- a/src/user.js
+++ b/src/user.js
@@ -11,7 +11,7 @@ var utils = require('./../public/src/utils.js'),
 
 (function(User) {
 	User.getUserField = function(uid, field, callback) {
-		RDB.hget('user:'+uid, field, function(err, data){
+		RDB.hget('user:'+uid, field, function(err, data) {
 			if(err === null)
 				callback(data);
 			else
@@ -20,7 +20,7 @@ var utils = require('./../public/src/utils.js'),
 	}
 	
 	User.getUserFields = function(uid, fields, callback) {
-		RDB.hmget('user:'+uid, fields, function(err, data){
+		RDB.hmget('user:'+uid, fields, function(err, data) {
 			if(err === null) {
 				var returnData = {};
 				
@@ -50,7 +50,7 @@ var utils = require('./../public/src/utils.js'),
 
 		for (var i=0, ii=uuids.length; i<ii; i++) {
 			(function(user_id) {
-				User.getUserFields(user_id, fields, function(user_data){
+				User.getUserFields(user_id, fields, function(user_data) {
 					data[user_id] = user_data;
 					loaded ++;
 					if (loaded == uuids.length) callback(data);
@@ -61,9 +61,8 @@ var utils = require('./../public/src/utils.js'),
 
 	User.getUserData = function(uid, callback) {
 
-		RDB.hgetall('user:'+uid, function(err, data){
-			if(err === null)
-			{
+		RDB.hgetall('user:'+uid, function(err, data) {
+			if(err === null) {
 				if(data) {
 					if(data['password'])
 						delete data['password'];
@@ -81,8 +80,7 @@ var utils = require('./../public/src/utils.js'),
 		var fields = ['email', 'fullname', 'website', 'location', 'birthday', 'signature'];
 		var key = '';
 		
-		if(data['signature'] !== undefined && data['signature'].length > 150)
-		{
+		if(data['signature'] !== undefined && data['signature'].length > 150) {
 			callback({error:'Signature can\'t be longer than 150 characters!'});
 			return;
 		}
@@ -126,11 +124,6 @@ var utils = require('./../public/src/utils.js'),
 				userkeys.splice(anonUserIndex, 1);
 			}
 
-			// removes user:1:following and user:1:followers, - need to find a better way for this
-			userkeys = userkeys.filter(function(value, index, self){
-				return value.indexOf(':f') === -1;
-			});
-
 			for(var i=0,ii=userkeys.length; i<ii; ++i) {
 				var uid = userkeys[i].substr(5);
 				
@@ -391,9 +384,9 @@ var utils = require('./../public/src/utils.js'),
 	}
 
 	User.follow = function(uid, followid, callback) {
-		RDB.sadd('user:'+uid+':following', followid, function(err, data) {
+		RDB.sadd('following:'+uid, followid, function(err, data) {
 			if(err === null) {
-				RDB.sadd('user:'+followid+':followers', uid, function(err, data) {
+				RDB.sadd('followers:'+followid, uid, function(err, data) {
 					callback(data);	
 				});
 			}
@@ -403,9 +396,9 @@ var utils = require('./../public/src/utils.js'),
 	}
 
 	User.unfollow = function(uid, unfollowid, callback) {
-		RDB.srem('user:'+uid+':following', unfollowid, function(err, data){
+		RDB.srem('following:'+uid, unfollowid, function(err, data){
 			if(err === null) {
-				RDB.srem('user:'+unfollowid+':followers', uid, function(err, data){
+				RDB.srem('followers:'+unfollowid, uid, function(err, data){
 					callback(data);
 				});
 			}
@@ -415,7 +408,7 @@ var utils = require('./../public/src/utils.js'),
 	}
 
 	User.getFollowing = function(uid, callback) {
-		RDB.smembers('user:'+uid+':following', function(err, userIds) {
+		RDB.smembers('following:'+uid, function(err, userIds) {
 			if(err === null)
 				User.getDataForUsers(userIds, callback);
 			else
@@ -424,7 +417,7 @@ var utils = require('./../public/src/utils.js'),
 	}
 
 	User.getFollowers = function(uid, callback) {
-		RDB.smembers('user:'+uid+':followers', function(err, userIds) {
+		RDB.smembers('followers:'+uid, function(err, userIds) {
 			if(err === null)
 				User.getDataForUsers(userIds, callback);
 			else
@@ -433,7 +426,7 @@ var utils = require('./../public/src/utils.js'),
 	}
 	
 	User.getFollowingCount = function(uid, callback) {
-		RDB.smembers('user:'+uid+':following', function(err, userIds) {
+		RDB.smembers('following:'+uid, function(err, userIds) {
 			if(err === null)
 				callback(userIds.length);
 			else
@@ -442,7 +435,7 @@ var utils = require('./../public/src/utils.js'),
 	}
 	
 	User.getFollowerCount = function(uid, callback) {
-		RDB.smembers('user:'+uid+':followers', function(err, userIds) {
+		RDB.smembers('followers:'+uid, function(err, userIds) {
 			if(err === null)
 				callback(userIds.length);
 			else
@@ -471,7 +464,7 @@ var utils = require('./../public/src/utils.js'),
 	User.sendPostNotificationToFollowers = function(uid, tid, pid) {
 
 		User.getUserField(uid, 'username', function(username) {
-			RDB.smembers('user:'+uid+':followers', function(err, followers) {
+			RDB.smembers('followers:'+uid, function(err, followers) {
 				
 				topics.getSlug(tid, function(slug) {
 
@@ -487,7 +480,7 @@ var utils = require('./../public/src/utils.js'),
 	}
 
 	User.isFollowing = function(uid, theirid, callback) {
-		RDB.sismember('user:'+uid+':following', theirid, function(err, data) {
+		RDB.sismember('following:'+uid, theirid, function(err, data) {
 			if(err === null)
 				callback(data === 1);
 			else
diff --git a/src/websockets.js b/src/websockets.js
index 1579f8b50f..e02a4ad0f2 100644
--- a/src/websockets.js
+++ b/src/websockets.js
@@ -40,9 +40,7 @@ var SocketIO = require('socket.io').listen(global.server, { log:false }),
 		
 		user.get_uid_by_session(sessionID, function(userId) {
 			if (userId)
-			{
 				users[sessionID] = userId;
-			}			
 			else 
 				users[sessionID] = 0;
 

From c19f9fa54b137a8c5eba1b4e6a93c7e191f3983e Mon Sep 17 00:00:00 2001
From: Baris Soner Usakli <barisusakli@gmail.com>
Date: Fri, 21 Jun 2013 13:20:37 -0400
Subject: [PATCH 3/3] closes issue #18

---
 public/templates/topic.tpl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/public/templates/topic.tpl b/public/templates/topic.tpl
index 63f88a2e88..d608a532bf 100644
--- a/public/templates/topic.tpl
+++ b/public/templates/topic.tpl
@@ -94,7 +94,7 @@
 <hr />
 <button id="post_reply" class="btn btn-primary btn-large post_reply" type="button">Reply</button>
 <div class="btn-group pull-right" id="thread-tools" style="visibility: hidden;">
-	<button class="btn dropdown-toggle" data-toggle="dropdown" type="button">Thread Tools <span class="caret"></span></button>
+	<button class="btn btn-large dropdown-toggle" data-toggle="dropdown" type="button">Thread Tools <span class="caret"></span></button>
 	<ul class="dropdown-menu">
 		<li><a href="#" id="pin_thread"><i class="icon-pushpin"></i> Pin Thread</a></li>
 		<li><a href="#" id="lock_thread"><i class="icon-lock"></i> Lock Thread</a></li>