From ae43a299dfdbc372bb798e4d3b7989f807045195 Mon Sep 17 00:00:00 2001
From: Julian Lam <julian@designcreateplay.com>
Date: Tue, 7 May 2013 13:57:30 -0400
Subject: [PATCH 1/7] hiding edit and delete buttons from indiv. posts if
 thread is locked

---
 public/templates/topic.tpl | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/public/templates/topic.tpl b/public/templates/topic.tpl
index 2166b2ffe5..47d859186b 100644
--- a/public/templates/topic.tpl
+++ b/public/templates/topic.tpl
@@ -223,6 +223,8 @@
 			var	threadReplyBtn = document.getElementById('post_reply'),
 				postReplyBtns = document.querySelectorAll('#post-container .post_reply'),
 				quoteBtns = document.querySelectorAll('#post-container .quote'),
+				editBtns = document.querySelectorAll('#post-container .edit'),
+				deleteBtns = document.querySelectorAll('#post-container .delete'),
 				numReplyBtns = postReplyBtns.length,
 				lockThreadEl = document.getElementById('lock_thread'),
 				x;
@@ -233,6 +235,8 @@
 				for(x=0;x<numReplyBtns;x++) {
 					postReplyBtns[x].innerHTML = 'Locked <i class="icon-lock"></i>';
 					quoteBtns[x].style.display = 'none';
+					editBtns[x].style.display = 'none';
+					deleteBtns[x].style.display = 'none';
 				}
 
 				thread_state.locked = '1';
@@ -243,6 +247,8 @@
 				for(x=0;x<numReplyBtns;x++) {
 					postReplyBtns[x].innerHTML = 'Reply <i class="icon-reply"></i>';
 					quoteBtns[x].style.display = 'inline-block';
+					editBtns[x].style.display = 'inline-block';
+					deleteBtns[x].style.display = 'inline-block';
 				}
 
 				thread_state.locked = '0';

From ae2be687b80a0a6c1decfee3bbe6cdafbd2a119c Mon Sep 17 00:00:00 2001
From: psychobunny <rodrigues.andrew@gmail.com>
Date: Tue, 7 May 2013 18:21:00 +0000
Subject: [PATCH 2/7] small refactor plus breadcrumbs for all ur pages

---
 public/templates/category.tpl |   8 +++
 public/templates/topic.tpl    |   3 +-
 src/categories.js             | 106 ++++++++++++++++++----------------
 src/posts.js                  |  14 +++--
 src/topics.js                 | 106 ++++++++++++++++++++++------------
 5 files changed, 142 insertions(+), 95 deletions(-)

diff --git a/public/templates/category.tpl b/public/templates/category.tpl
index 513070c198..5abee5a796 100644
--- a/public/templates/category.tpl
+++ b/public/templates/category.tpl
@@ -1,3 +1,11 @@
+<div class="container">
+	<ul class="breadcrumb">
+		<li><a href="/">Home</a><span class="divider">/</span></li>
+		<li class="active">{category_name}</li>
+		<div id="category_active_users"></div>
+	</ul>
+</div>
+
 <button id="new_post" class="btn btn-primary btn-large {show_topic_button}">New Topic</button>
 <ul class="topic-container">
 <!-- BEGIN topics -->
diff --git a/public/templates/topic.tpl b/public/templates/topic.tpl
index 25c48d99ac..8408e8fb2a 100644
--- a/public/templates/topic.tpl
+++ b/public/templates/topic.tpl
@@ -1,6 +1,7 @@
 <div class="container">
 	<ul class="breadcrumb">
-		<li><a href="/">Home</a> <span class="divider">/</span></li>
+		<li><a href="/">Home</a><span class="divider">/</span></li>
+		<li><a href="/category/{category_slug}">{category_name}</a><span class="divider">/</span></li>
 		<li class="active">{topic_name}</li>
 		<div id="thread_active_users"></div>
 	</ul>
diff --git a/src/categories.js b/src/categories.js
index 545538d377..5173e64955 100644
--- a/src/categories.js
+++ b/src/categories.js
@@ -25,60 +25,66 @@ var	RDB = require('./redis.js'),
 		});
 	};
 
-
+	Categories.edit = function(data, callback) {
+		// just a reminder to self that name + slugs are stored into topics data as well.
+	};
 
 	Categories.get = function(callback) {
 		RDB.lrange('categories:cid', 0, -1, function(cids) {
-			var name = [],
-				description = [],
-				icon = [],
-				blockclass = [],
-				slug = [];
-
-			for (var i=0, ii=cids.length; i<ii; i++) {
-				name.push('cid:' + cids[i] + ':name');
-				description.push('cid:' + cids[i] + ':description');
-				icon.push('cid:' + cids[i] + ':icon');
-				blockclass.push('cid:' + cids[i] + ':blockclass');
-				slug.push('cid:' + cids[i] + ':slug');
-			}
-
-			if (cids.length > 0) {
-				RDB.multi()
-					.mget(name)
-					.mget(description)
-					.mget(icon)
-					.mget(blockclass)
-					.mget(slug)
-					.exec(function(err, replies) {
-						name = replies[0];
-						description = replies[1];
-						icon = replies[2];
-						blockclass = replies[3];
-						slug = replies[4];
-						
-						var categories = [];
-						for (var i=0, ii=cids.length; i<ii; i++) {
-							categories.push({
-								'name' : name[i],
-								'cid' : cids[i],
-								'slug' : slug[i],
-								'description' : description[i],
-								'blockclass' : blockclass[i],
-								'icon' : icon[i],
-								/*'topics' : [0,1], later
-								'latest_post' : {
-									'uid' : 1,
-									'pid' : 1,
-									timestamp and shit
-								}*/
-							});
-						}
-
-						callback({'categories': categories});
-					});
-			} else callback({'categories' : []});
+			Categories.get_category(cids, callback);
 		});
 	}
 
+	Categories.get_category = function(cids, callback) {
+		var name = [],
+			description = [],
+			icon = [],
+			blockclass = [],
+			slug = [];
+
+		for (var i=0, ii=cids.length; i<ii; i++) {
+			name.push('cid:' + cids[i] + ':name');
+			description.push('cid:' + cids[i] + ':description');
+			icon.push('cid:' + cids[i] + ':icon');
+			blockclass.push('cid:' + cids[i] + ':blockclass');
+			slug.push('cid:' + cids[i] + ':slug');
+		}
+
+		if (cids.length > 0) {
+			RDB.multi()
+				.mget(name)
+				.mget(description)
+				.mget(icon)
+				.mget(blockclass)
+				.mget(slug)
+				.exec(function(err, replies) {
+					name = replies[0];
+					description = replies[1];
+					icon = replies[2];
+					blockclass = replies[3];
+					slug = replies[4];
+					
+					var categories = [];
+					for (var i=0, ii=cids.length; i<ii; i++) {
+						categories.push({
+							'name' : name[i],
+							'cid' : cids[i],
+							'slug' : slug[i],
+							'description' : description[i],
+							'blockclass' : blockclass[i],
+							'icon' : icon[i],
+							/*'topics' : [0,1], later
+							'latest_post' : {
+								'uid' : 1,
+								'pid' : 1,
+								timestamp and shit
+							}*/
+						});
+					}
+
+					callback({'categories': categories});
+				});
+		} else callback({'categories' : []});
+	};
+
 }(exports));
\ No newline at end of file
diff --git a/src/posts.js b/src/posts.js
index c437f57bc1..7538cad7ca 100644
--- a/src/posts.js
+++ b/src/posts.js
@@ -33,12 +33,14 @@ var	RDB = require('./redis.js'),
 					'user_rep' : user_data[uid].reputation || 0,
 					'gravatar' : user_data[uid].picture,
 					'fav_star_class' : vote_data[pid] ? 'icon-star' : 'icon-star-empty',
-					'display_moderator_tools' : uid === current_user ? 'show' : 'hide'
+					'display_moderator_tools' : uid == current_user ? 'show' : 'hide'
 				});
 			}
 
 			callback({
 				'topic_name':thread_data.topic_name,
+				'category_name':thread_data.category_name,
+				'category_slug':thread_data.category_slug,
 				'locked': parseInt(thread_data.locked) || 0,
 				'topic_id': tid,
 				'posts': posts
@@ -58,10 +60,6 @@ var	RDB = require('./redis.js'),
 				pid.push(pids[i]);
 			}
 
-			RDB.get('tid:' + tid + ':title', function(topic_name) {
-				thread_data = {topic_name: topic_name};
-				generateThread();
-			});
 
 			Posts.getFavouritesByPostIDs(pids, current_user, function(fav_data) {
 				vote_data = fav_data;
@@ -76,6 +74,8 @@ var	RDB = require('./redis.js'),
 				.mget(post_rep)
 				.get('tid:' + tid + ':title')
 				.get('tid:' + tid + ':locked')
+				.get('tid:' + tid + ':category_name')
+				.get('tid:' + tid + ':category_slug')
 				.exec(function(err, replies) {
 					post_data = {
 						pid: pids,
@@ -87,7 +87,9 @@ var	RDB = require('./redis.js'),
 
 					thread_data = {
 						topic_name: replies[4],
-						locked: replies[5]
+						locked: replies[5],
+						category_name: replies[6],
+						category_slug: replies[7]
 					};
 
 					user.getMultipleUserFields(post_data.uid, ['username','reputation','picture'], function(user_details){
diff --git a/src/topics.js b/src/topics.js
index be6bf06e32..e43b32e67d 100644
--- a/src/topics.js
+++ b/src/topics.js
@@ -1,7 +1,8 @@
 var	RDB = require('./redis.js'),
 	posts = require('./posts.js'),
 	utils = require('./utils.js'),
-	user = require('./user.js');
+	user = require('./user.js')
+	categories = require('./categories.js');
 
 (function(Topics) {
 
@@ -33,54 +34,75 @@ var	RDB = require('./redis.js'),
 				postcount.push('tid:' + tids[i] + ':postcount');
 			}
 
+			var multi = RDB.multi()
+				.get('cid:' + category_id + ':name');
+
 			if (tids.length > 0) {
-				RDB.multi()
+				multi
 					.mget(title)
 					.mget(uid)
 					.mget(timestamp)
 					.mget(slug)
 					.mget(postcount)
-					.exec(function(err, replies) {
-						
-						title = replies[0];
-						uid = replies[1];
-						timestamp = replies[2];
-						slug = replies[3];
-						postcount = replies[4];
-						
-						
-						
-						user.get_usernames_by_uids(uid, function(userNames) {
-							var topics = [];
-							
-							for (var i=0, ii=title.length; i<ii; i++) {
-								
-								topics.push({
-									'title' : title[i],
-									'uid' : uid[i],
-									'username': userNames[i],
-									'timestamp' : timestamp[i],
-									'relativeTime': utils.relativeTime(timestamp[i]),
-									'slug' : slug[i],
-									'post_count' : postcount[i]
-								});
-							}
-						
-							callback({
-								'show_topic_button' : category_id ? 'show' : 'hidden',
-								'category_id': category_id,
-								'topics': topics
-							});
-						});
+			}
+				
+			
+			multi.exec(function(err, replies) {
+				category_name = replies[0];
+				var topics = [];
 
+				if (tids.length > 0) {
+					title = replies[1];
+					uid = replies[2];
+					timestamp = replies[3];
+					slug = replies[4];
+					postcount = replies[5];
+					
+					
+					
+					
+					user.get_usernames_by_uids(uid, function(userNames) {
 						
-					}
-				);
-			} else callback({'category_id': category_id, 'topics': []});
+						for (var i=0, ii=title.length; i<ii; i++) {
+							
+							topics.push({
+								'title' : title[i],
+								'uid' : uid[i],
+								'username': userNames[i],
+								'timestamp' : timestamp[i],
+								'relativeTime': utils.relativeTime(timestamp[i]),
+								'slug' : slug[i],
+								'post_count' : postcount[i]
+							});
+						}
+
+						callback({
+							'category_name' : category_id ? category_name : 'Recent',
+							'show_topic_button' : category_id ? 'show' : 'hidden',
+							'category_id': category_id,
+							'topics': topics
+						});
+					
+					});	
+				}
+				else {
+					callback({
+						'category_name' : category_id ? category_name : 'Recent',
+						'show_topic_button' : category_id ? 'show' : 'hidden',
+						'category_id': category_id,
+						'topics': []
+					});
+				}
+				
+
+
+			});
+			//} else callback({'category_id': category_id, 'topics': []});
 		});
 	}
 
 	Topics.post = function(socket, uid, title, content, category_id) {
+		if (!category_id) throw new Error('Attempted to post without a category_id');
 		
 		if (uid === 0) {
 			socket.emit('event:alert', {
@@ -103,7 +125,7 @@ var	RDB = require('./redis.js'),
 				RDB.lpush('topics:tid', tid);	
 			} else {
 				// need to add some unique key sent by client so we can update this with the real uid later
-				RDB.lpush('topics:queued:tid', tid);		
+				RDB.lpush('topics:queued:tid', tid);
 			}
 			
 
@@ -138,6 +160,14 @@ var	RDB = require('./redis.js'),
 				type: 'notify',
 				timeout: 2000
 			});
+
+
+			// in future it may be possible to add topics to several categories, so leaving the door open here.
+			categories.get_category([category_id], function(data) {
+				RDB.set('tid:' + tid + ':category_name', data.categories[0].name);
+				RDB.set('tid:' + tid + ':category_slug', data.categories[0].slug);
+			});
+
 		});
 
 		

From bf7c602cd86a6ab882b276bc77db6f9c93abd19a Mon Sep 17 00:00:00 2001
From: Julian Lam <julian@designcreateplay.com>
Date: Tue, 7 May 2013 15:14:47 -0400
Subject: [PATCH 3/7] thread pinning

---
 public/templates/topic.tpl | 51 ++++++++++++++++++++++++++++++++------
 src/topics.js              | 32 ++++++++++++++++++++++++
 src/websockets.js          |  8 ++++++
 3 files changed, 83 insertions(+), 8 deletions(-)

diff --git a/public/templates/topic.tpl b/public/templates/topic.tpl
index 47d859186b..67bb5d0060 100644
--- a/public/templates/topic.tpl
+++ b/public/templates/topic.tpl
@@ -37,6 +37,7 @@
 <div class="btn-group pull-right" id="thread-tools" style="visibility: hidden;">
 	<button class="btn dropdown-toggle" data-toggle="dropdown">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>
 		<li class="divider"></li>
 		<li><a href="#" id="delete_thread"><span class="text-error"><i class="icon-trash"></i>  Delete Thread</span></a></li>
@@ -50,7 +51,8 @@
 			tid = '{topic_id}',
 			thread_state = {
 				locked: '{locked}',
-				deleted: '{deleted}'
+				deleted: '{deleted}',
+				pinned: '{pinned}'
 			};
 
 		jQuery('document').ready(function() {
@@ -62,10 +64,12 @@
 
 			if (thread_state.locked === '1') set_locked_state(true);
 			if (thread_state.deleted === '1') set_delete_state(true);
+			if (thread_state.pinned === '1') set_pinned_state(true);
 
 			if (expose_tools === '1') {
 				var deleteThreadEl = document.getElementById('delete_thread'),
-					lockThreadEl = document.getElementById('lock_thread');
+					lockThreadEl = document.getElementById('lock_thread'),
+					pinThreadEl = document.getElementById('pin_thread');
 
 				adminTools.style.visibility = 'inherit';
 
@@ -86,13 +90,18 @@
 				lockThreadEl.addEventListener('click', function(e) {
 					e.preventDefault();
 					if (thread_state.locked !== '1') {
-						if (confirm('really lock thread? (THIS DIALOG TO BE REPLACED WITH BOOTBOX)')) {
-							socket.emit('api:topic.lock', { tid: tid });
-						}
+						socket.emit('api:topic.lock', { tid: tid });
 					} else {
-						if (confirm('really unlock thread? (THIS DIALOG TO BE REPLACED WITH BOOTBOX)')) {
-							socket.emit('api:topic.unlock', { tid: tid });
-						}
+						socket.emit('api:topic.unlock', { tid: tid });
+					}
+				});
+
+				pinThreadEl.addEventListener('click', function(e) {
+					e.preventDefault();
+					if (thread_state.pinned !== '1') {
+						socket.emit('api:topic.pin', { tid: tid });
+					} else {
+						socket.emit('api:topic.unpin', { tid: tid });
 					}
 				});
 			}
@@ -163,6 +172,18 @@
 			}
 		});
 
+		socket.on('event:topic_pinned', function(data) {
+			if (data.tid === tid && data.status === 'ok') {
+				set_pinned_state(true);
+			}
+		});
+
+		socket.on('event:topic_unpinned', function(data) {
+			if (data.tid === tid && data.status === 'ok') {
+				set_pinned_state(false);
+			}
+		});
+
 		function adjust_rep(value, pid, uid) {
 			var post_rep = jQuery('.post_rep_' + pid),
 				user_rep = jQuery('.user_rep_' + uid);
@@ -280,5 +301,19 @@
 				thread_state.deleted = '0';
 			}
 		}
+
+		function set_pinned_state(pinned) {
+			var pinEl = document.getElementById('pin_thread');
+
+			if (pinned) {
+				pinEl.innerHTML = '<i class="icon-pushpin"></i> Unpin Thread';
+
+				thread_state.pinned = '1';
+			} else {
+				pinEl.innerHTML = '<i class="icon-pushpin"></i> Pin Thread';
+
+				thread_state.pinned = '0';
+			}
+		}
 	})();
 </script>
\ No newline at end of file
diff --git a/src/topics.js b/src/topics.js
index a71103efcc..ac99f83819 100644
--- a/src/topics.js
+++ b/src/topics.js
@@ -215,4 +215,36 @@ var	RDB = require('./redis.js'),
 			}
 		});
 	}
+
+	Topics.pin = function(tid, uid, socket) {
+		user.getUserField(uid, 'reputation', function(rep) {
+			if (rep >= configs.privilege_thresholds.manage_thread) {
+				// Mark thread as deleted
+				RDB.set('tid:' + tid + ':pinned', 1);
+
+				if (socket) {
+					io.sockets.in('topic_' + tid).emit('event:topic_pinned', {
+						tid: tid,
+						status: 'ok'
+					});
+				}
+			}
+		});
+	}
+
+	Topics.unpin = function(tid, uid, socket) {
+		user.getUserField(uid, 'reputation', function(rep) {
+			if (rep >= configs.privilege_thresholds.manage_thread) {
+				// Mark thread as deleted
+				RDB.del('tid:' + tid + ':pinned');
+
+				if (socket) {
+					io.sockets.in('topic_' + tid).emit('event:topic_unpinned', {
+						tid: tid,
+						status: 'ok'
+					});
+				}
+			}
+		});
+	}
 }(exports));
\ No newline at end of file
diff --git a/src/websockets.js b/src/websockets.js
index b8ddb7b39c..98b0b6471f 100644
--- a/src/websockets.js
+++ b/src/websockets.js
@@ -190,6 +190,14 @@ var	SocketIO = require('socket.io').listen(global.server,{log:false}),
 		socket.on('api:topic.unlock', function(data) {
 			modules.topics.unlock(data.tid, uid, socket);
 		});
+
+		socket.on('api:topic.pin', function(data) {
+			modules.topics.pin(data.tid, uid, socket);
+		});
+
+		socket.on('api:topic.unpin', function(data) {
+			modules.topics.unpin(data.tid, uid, socket);
+		});
 	});
 	
 }(SocketIO));

From 91d9775cc42e3503982619764e8f65228ced17e1 Mon Sep 17 00:00:00 2001
From: psychobunny <rodrigues.andrew@gmail.com>
Date: Tue, 7 May 2013 19:22:38 +0000
Subject: [PATCH 4/7] fixed slugs from disappearing if f5ing

---
 src/webserver.js | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/webserver.js b/src/webserver.js
index 25ff4ab05a..6a8e99703c 100644
--- a/src/webserver.js
+++ b/src/webserver.js
@@ -145,15 +145,17 @@ passport.deserializeUser(function(uid, done) {
 	
 	// Complex Routes
 	app.get('/topic/:topic_id/:slug?', function(req, res) {
-		res.send(templates['header'] + '<script>templates.ready(function(){ajaxify.go("' + 'topic/' + req.params.topic_id + '");});</script>' + templates['footer']);
+		var topic_url = req.params.topic_id + (req.params.slug ? '/' + req.params.slug : '');
+		res.send(templates['header'] + '<script>templates.ready(function(){ajaxify.go("topic/' + topic_url + '");});</script>' + templates['footer']);
 	});
 
 	app.get('/category/:category_id/:slug?', function(req, res) {
-		res.send(templates['header'] + '<script>templates.ready(function(){ajaxify.go("' + 'category/' + req.params.category_id + '");});</script>' + templates['footer']);
+		var category_url = req.params.category_id + (req.params.slug ? '/' + req.params.slug : '');
+		res.send(templates['header'] + '<script>templates.ready(function(){ajaxify.go("category/' + category_url + '");});</script>' + templates['footer']);
 	});
 
 	app.get('/confirm/:code', function(req, res) {
-		res.send(templates['header'] + '<script>templates.ready(function(){ajaxify.go("' + 'confirm/' + req.params.code + '");});</script>' + templates['footer']);
+		res.send(templates['header'] + '<script>templates.ready(function(){ajaxify.go("confirm/' + req.params.code + '");});</script>' + templates['footer']);
 	});
 
 	// These functions are called via ajax once the initial page is loaded to populate templates with data

From 96b9d416983d4ac50963ae708c055c2d905107b7 Mon Sep 17 00:00:00 2001
From: psychobunny <rodrigues.andrew@gmail.com>
Date: Tue, 7 May 2013 19:24:52 +0000
Subject: [PATCH 5/7] removed return to home link from topic, realized that
 breadcrumbs are right on top

---
 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 9a93699f64..6f31fec8bf 100644
--- a/public/templates/topic.tpl
+++ b/public/templates/topic.tpl
@@ -269,7 +269,7 @@
 				// Spawn a 'deleted' notice at the top of the page
 				deleteNotice.setAttribute('id', 'thread-deleted');
 				deleteNotice.className = 'alert';
-				deleteNotice.innerHTML = 'This thread has been deleted. Only users with thread management privileges can see it.<br /><br /><a href="/">Home</a>';
+				deleteNotice.innerHTML = 'This thread has been deleted. Only users with thread management privileges can see it.';
 				document.getElementById('content').insertBefore(deleteNotice, threadEl);
 
 				thread_state.deleted = '1';

From f0eeec893290a433ba79ac744c607c8901f93aec Mon Sep 17 00:00:00 2001
From: Julian Lam <julian@designcreateplay.com>
Date: Tue, 7 May 2013 15:36:32 -0400
Subject: [PATCH 6/7] fixing alert removal by alert_id and adding alerts when
 thread is locked or pinned

---
 public/src/app.js             |  4 ++-
 public/templates/category.tpl |  2 +-
 public/templates/topic.tpl    | 51 ++++++++++++++++++++++++++++++-----
 src/posts.js                  |  7 +++--
 src/topics.js                 | 15 +++++++----
 5 files changed, 64 insertions(+), 15 deletions(-)

diff --git a/public/src/app.js b/public/src/app.js
index 89be5c0a2c..bf5065d6c6 100644
--- a/public/src/app.js
+++ b/public/src/app.js
@@ -33,6 +33,8 @@ var socket,
 
 	// use unique alert_id to have multiple alerts visible at a time, use the same alert_id to fade out the current instance  
 	// type : error, success, info, warning/notify
+	// title = bolded title text
+	// message = alert message content
 	// timeout default = permanent
 	// location : notification_window (default) or content
 	app.alert = function(params) {
@@ -41,7 +43,7 @@ var socket,
 			strong = document.createElement('strong'),
 			p = document.createElement('p');
 
-		var alert_id = 'alert_button_' + ((alert_id) ? alert_id : new Date().getTime()); 
+		var alert_id = 'alert_button_' + ((params.alert_id) ? params.alert_id : new Date().getTime()); 
 
 		jQuery('#'+alert_id).fadeOut(500, function() {
 			this.remove();
diff --git a/public/templates/category.tpl b/public/templates/category.tpl
index 7f01999c9c..99d54f8140 100644
--- a/public/templates/category.tpl
+++ b/public/templates/category.tpl
@@ -10,7 +10,7 @@
 <ul class="topic-container">
 <!-- BEGIN topics -->
 <a href="../../topic/{topics.slug}"><li class="topic-row">
-	<h4><i class="{topics.icon}"></i> {topics.title}</h4>
+	<h4><i class="{topics.pin-icon}"></i><i class="{topics.lock-icon}"></i> {topics.title}</h4>
 	<p>Posted {topics.relativeTime} ago by <span class="username">{topics.username}</span>. {topics.post_count} posts.</p>
 </li></a>
 <!-- END topics -->
diff --git a/public/templates/topic.tpl b/public/templates/topic.tpl
index ca07094fd2..b5e007e58f 100644
--- a/public/templates/topic.tpl
+++ b/public/templates/topic.tpl
@@ -163,25 +163,25 @@
 
 		socket.on('event:topic_locked', function(data) {
 			if (data.tid === tid && data.status === 'ok') {
-				set_locked_state(true);
+				set_locked_state(true, 1);
 			}
 		});
 
 		socket.on('event:topic_unlocked', function(data) {
 			if (data.tid === tid && data.status === 'ok') {
-				set_locked_state(false);
+				set_locked_state(false, 1);
 			}
 		});
 
 		socket.on('event:topic_pinned', function(data) {
 			if (data.tid === tid && data.status === 'ok') {
-				set_pinned_state(true);
+				set_pinned_state(true, 1);
 			}
 		});
 
 		socket.on('event:topic_unpinned', function(data) {
 			if (data.tid === tid && data.status === 'ok') {
-				set_pinned_state(false);
+				set_pinned_state(false, 1);
 			}
 		});
 
@@ -241,7 +241,7 @@
 			});
 		}
 
-		function set_locked_state(locked) {
+		function set_locked_state(locked, alert) {
 			var	threadReplyBtn = document.getElementById('post_reply'),
 				postReplyBtns = document.querySelectorAll('#post-container .post_reply'),
 				quoteBtns = document.querySelectorAll('#post-container .quote'),
@@ -250,6 +250,7 @@
 				numReplyBtns = postReplyBtns.length,
 				lockThreadEl = document.getElementById('lock_thread'),
 				x;
+
 			if (locked === true) {
 				lockThreadEl.innerHTML = '<i class="icon-unlock"></i> Unlock Thread';
 				threadReplyBtn.disabled = true;
@@ -261,6 +262,16 @@
 					deleteBtns[x].style.display = 'none';
 				}
 
+				if (alert) {
+					app.alert({
+						'alert_id': 'thread_lock',
+						type: 'success',
+						title: 'Thread Locked',
+						message: 'Thread has been successfully locked',
+						timeout: 5000
+					});
+				}
+
 				thread_state.locked = '1';
 			} else {
 				lockThreadEl.innerHTML = '<i class="icon-lock"></i> Lock Thread';
@@ -273,6 +284,16 @@
 					deleteBtns[x].style.display = 'inline-block';
 				}
 
+				if (alert) {
+					app.alert({
+						'alert_id': 'thread_lock',
+						type: 'success',
+						title: 'Thread Unlocked',
+						message: 'Thread has been successfully unlocked',
+						timeout: 5000
+					});
+				}
+
 				thread_state.locked = '0';
 			}
 		}
@@ -303,15 +324,33 @@
 			}
 		}
 
-		function set_pinned_state(pinned) {
+		function set_pinned_state(pinned, alert) {
 			var pinEl = document.getElementById('pin_thread');
 
 			if (pinned) {
 				pinEl.innerHTML = '<i class="icon-pushpin"></i> Unpin Thread';
+				if (alert) {
+					app.alert({
+						'alert_id': 'thread_pin',
+						type: 'success',
+						title: 'Thread Pinned',
+						message: 'Thread has been successfully pinned',
+						timeout: 5000
+					});
+				}
 
 				thread_state.pinned = '1';
 			} else {
 				pinEl.innerHTML = '<i class="icon-pushpin"></i> Pin Thread';
+				if (alert) {
+					app.alert({
+						'alert_id': 'thread_pin',
+						type: 'success',
+						title: 'Thread Unpinned',
+						message: 'Thread has been successfully unpinned',
+						timeout: 5000
+					});
+				}
 
 				thread_state.pinned = '0';
 			}
diff --git a/src/posts.js b/src/posts.js
index c16b7cb770..ae74b0ed57 100644
--- a/src/posts.js
+++ b/src/posts.js
@@ -44,6 +44,7 @@ var	RDB = require('./redis.js'),
 				'category_slug':thread_data.category_slug,
 				'locked': parseInt(thread_data.locked) || 0,
 				'deleted': parseInt(thread_data.deleted) || 0,
+				'pinned': parseInt(thread_data.pinned) || 0,
 				'topic_id': tid,
 				'expose_tools': viewer_data.reputation >= config.privilege_thresholds.manage_thread ? 1 : 0,
 				'posts': posts
@@ -80,6 +81,7 @@ var	RDB = require('./redis.js'),
 				.get('tid:' + tid + ':category_name')
 				.get('tid:' + tid + ':category_slug')
 				.get('tid:' + tid + ':deleted')
+				.get('tid:' + tid + ':pinned')
 				.exec(function(err, replies) {
 					post_data = {
 						pid: pids,
@@ -91,10 +93,11 @@ var	RDB = require('./redis.js'),
 
 					thread_data = {
 						topic_name: replies[4],
-						locked: replies[5],
+						locked: replies[5] || 0,
 						category_name: replies[6],
 						category_slug: replies[7],
-						deleted: replies[8] || 0
+						deleted: replies[8] || 0,
+						pinned: replies[9] || 0
 					};
 
 					user.getMultipleUserFields(post_data.uid, ['username','reputation','picture'], function(user_details){
diff --git a/src/topics.js b/src/topics.js
index 1b2044a4d6..94b36a872c 100644
--- a/src/topics.js
+++ b/src/topics.js
@@ -27,16 +27,18 @@ var	RDB = require('./redis.js'),
 				slug = [],
 				postcount = [],
 				locked = [],
-				deleted = [];
+				deleted = [],
+				pinned = [];
 
 			for (var i=0, ii=tids.length; i<ii; i++) {
 				title.push('tid:' + tids[i] + ':title');
 				uid.push('tid:' + tids[i] + ':uid');
 				timestamp.push('tid:' + tids[i] + ':timestamp');
 				slug.push('tid:' + tids[i] + ':slug');
-				postcount.push('tid:' + tids[i] + ':postcount'),
-				locked.push('tid:' + tids[i] + ':locked'),
+				postcount.push('tid:' + tids[i] + ':postcount');
+				locked.push('tid:' + tids[i] + ':locked');
 				deleted.push('tid:' + tids[i] + ':deleted');
+				pinned.push('tid:' + tids[i] + ':pinned');
 			}
 
 			var multi = RDB.multi()
@@ -51,6 +53,7 @@ var	RDB = require('./redis.js'),
 					.mget(postcount)
 					.mget(locked)
 					.mget(deleted)
+					.mget(pinned)
 			}
 				
 			
@@ -66,6 +69,7 @@ var	RDB = require('./redis.js'),
 					postcount = replies[5];
 					locked = replies[6];
 					deleted = replies[7];
+					pinned = replies[8];
 					
 					user.get_usernames_by_uids(uid, function(userNames) {
 						
@@ -79,8 +83,9 @@ var	RDB = require('./redis.js'),
 								'relativeTime': utils.relativeTime(timestamp[i]),
 								'slug' : slug[i],
 								'post_count' : postcount[i],
-								'icon': locked[i] === '1' ? 'icon-lock' : 'hide',
-								'deleted': deleted[i]
+								'lock-icon': locked[i] === '1' ? 'icon-lock' : 'hide',
+								'deleted': deleted[i],
+								'pin-icon': pinned[i] === '1' ? 'icon-pushpin' : 'hide'
 							});
 						}
 

From b57109f6fd78242987b66e1ded6b06abe411c0e8 Mon Sep 17 00:00:00 2001
From: Julian Lam <julian@designcreateplay.com>
Date: Tue, 7 May 2013 15:43:51 -0400
Subject: [PATCH 7/7] floating pinned articles to the top of the topic list

---
 src/topics.js | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/topics.js b/src/topics.js
index 94b36a872c..9697b87b7c 100644
--- a/src/topics.js
+++ b/src/topics.js
@@ -85,10 +85,16 @@ var	RDB = require('./redis.js'),
 								'post_count' : postcount[i],
 								'lock-icon': locked[i] === '1' ? 'icon-lock' : 'hide',
 								'deleted': deleted[i],
+								'pinned': parseInt(pinned[i] || 0),	// For sorting purposes
 								'pin-icon': pinned[i] === '1' ? 'icon-pushpin' : 'hide'
 							});
 						}
 
+						// Float pinned topics to the top
+						topics = topics.sort(function(a, b) {
+							return b.pinned - a.pinned;
+						});
+
 						callback({
 							'category_name' : category_id ? category_name : 'Recent',
 							'show_topic_button' : category_id ? 'show' : 'hidden',