From 425aa751651411dcf0f453f11da34d6b4c558409 Mon Sep 17 00:00:00 2001
From: calzoneman <calzoneman@gmail.com>
Date: Thu, 8 Aug 2013 10:39:58 -0400
Subject: [PATCH 01/11] Show popup with breakdown of usercount on hover

---
 www/assets/js/ui.js   | 32 ++++++++++++++++++++++++++++++++
 www/assets/js/util.js | 29 +++++++++++++++++++++++++++++
 2 files changed, 61 insertions(+)

diff --git a/www/assets/js/ui.js b/www/assets/js/ui.js
index 21d79933..614aad51 100644
--- a/www/assets/js/ui.js
+++ b/www/assets/js/ui.js
@@ -68,6 +68,38 @@ $("#logout").click(function() {
 });
 
 /* chatbox */
+
+$("#usercountwrap").mouseenter(function (ev) {
+    var breakdown = calcUserBreakdown();
+    // re-using profile-box class for convenience
+    var popup = $("<div/>")
+        .addClass("profile-box")
+        .css("top", (ev.pageY + 5) + "px")
+        .css("left", (ev.pageX) + "px")
+        .appendTo($("#usercountwrap"));
+
+    var contents = "";
+    for(var key in breakdown) {
+        contents += "<strong>" + key + ":&nbsp;</strong>" + breakdown[key];
+        contents += "<br>"
+    }
+
+    popup.html(contents);
+});
+
+$("#usercountwrap").mousemove(function (ev) {
+    var popup = $("#usercountwrap").find(".profile-box");
+    if(popup.length == 0)
+        return;
+
+    popup.css("top", (ev.pageY + 5) + "px");
+    popup.css("left", (ev.pageX) + "px");
+});
+
+$("#usercountwrap").mouseleave(function () {
+    $("#usercountwrap").find(".profile-box").remove();
+});
+
 $("#messagebuffer").mouseenter(function() { SCROLLCHAT = false; });
 $("#messagebuffer").mouseleave(function() { SCROLLCHAT = true; });
 
diff --git a/www/assets/js/util.js b/www/assets/js/util.js
index cc4f91a5..0fc77531 100644
--- a/www/assets/js/util.js
+++ b/www/assets/js/util.js
@@ -243,6 +243,35 @@ function addUserDropdown(entry, data) {
     });
 }
 
+function calcUserBreakdown() {
+    var breakdown = {
+        "Site Admins": 0,
+        "Channel Admins": 0,
+        "Moderators": 0,
+        "Regular Users": 0,
+        "Guests": 0,
+        "AFK": 0
+    };
+    $("#userlist .userlist_item").each(function (index, item) {
+        var data = $(item).data("dropdown-info");
+        if(data.rank >= 255)
+            breakdown["Site Admins"]++;
+        else if(data.rank >= 3)
+            breakdown["Channel Admins"]++;
+        else if(data.rank == 2)
+            breakdown["Moderators"]++;
+        else if(data.rank >= 1)
+            breakdown["Regular Users"]++;
+        else
+            breakdown["Guests"]++;
+
+        if($(item).find(".icon-time").length > 0)
+            breakdown["AFK"]++;
+    });
+    
+    return breakdown;
+}
+
 /* queue stuff */
 
 function scrollQueue() {

From c7501bfd50e54ba1c2ab52c539cc874613a216fc Mon Sep 17 00:00:00 2001
From: calzoneman <calzoneman@gmail.com>
Date: Thu, 8 Aug 2013 10:54:07 -0400
Subject: [PATCH 02/11] Add different userlist sort options

---
 www/assets/js/callbacks.js | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/www/assets/js/callbacks.js b/www/assets/js/callbacks.js
index d5d13504..75559384 100644
--- a/www/assets/js/callbacks.js
+++ b/www/assets/js/callbacks.js
@@ -626,15 +626,8 @@ Callbacks = {
         var nametag = $("<span/>").text(data.name).appendTo(div);
         formatUserlistItem(div, data);
         addUserDropdown(div, data);
-        var users = $("#userlist").children();
-        for(var i = 0; i < users.length; i++) {
-            var othername = users[i].children[1].innerHTML;
-            if(othername.toLowerCase() > data.name.toLowerCase()) {
-                div.insertBefore(users[i]);
-                return;
-            }
-        }
         div.appendTo($("#userlist"));
+        sortUserlist();
     },
 
     updateUser: function(data) {
@@ -668,6 +661,8 @@ Callbacks = {
         var user = $(".userlist-" + data.name);
         formatUserlistItem(user, data);
         addUserDropdown(user, data);
+        if(USEROPTS.sort_rank)
+            sortUserlist();
     },
 
     setAFK: function (data) {
@@ -679,6 +674,8 @@ Callbacks = {
                 .appendTo(user[0].children[0]);
             $(user[0].children[1]).css("font-style", "italic");
         }
+        if(USEROPTS.sort_afk)
+            sortUserlist();
     },
 
     userLeave: function(data) {

From 9256928d826c5b408935dc0f367eef94950be9c1 Mon Sep 17 00:00:00 2001
From: calzoneman <calzoneman@gmail.com>
Date: Thu, 8 Aug 2013 10:57:46 -0400
Subject: [PATCH 03/11] Handle user impersonation using bold filter differently

---
 channel.js            |  2 +-
 www/assets/js/util.js | 57 +++++++++++++++++++++++++++++++++++++++++--
 2 files changed, 56 insertions(+), 3 deletions(-)

diff --git a/channel.js b/channel.js
index 63a72c28..cb50d1c6 100644
--- a/channel.js
+++ b/channel.js
@@ -88,7 +88,7 @@ var Channel = function(name, Server) {
     };
     this.filters = [
         new Filter("monospace", "`([^`]+)`", "g", "<code>$1</code>"),
-        new Filter("bold", "(.)\\*([^\\*]+)\\*", "g", "$1<strong>$2</strong>"),
+        new Filter("bold", "(^|\\s)\\*([^\\*]+)\\*", "g", "$1<strong>$2</strong>"),
         new Filter("italic", "(^| )_([^_]+)_", "g", "$1<em>$2</em>"),
         new Filter("strikethrough", "~~([^~]+)~~", "g", "<s>$1</s>"),
         new Filter("inline spoiler", "\\[spoiler\\](.*)\\[\\/spoiler\\]", "ig", "<span class=\"spoiler\">$1</span>"),
diff --git a/www/assets/js/util.js b/www/assets/js/util.js
index 0fc77531..b2aae050 100644
--- a/www/assets/js/util.js
+++ b/www/assets/js/util.js
@@ -272,6 +272,42 @@ function calcUserBreakdown() {
     return breakdown;
 }
 
+function sortUserlist() {
+    var slice = Array.prototype.slice;
+    var list = slice.call($("#userlist .userlist_item"));
+    list.sort(function (a, b) {
+        var r1 = $(a).data("dropdown-info").rank;
+        var r2 = $(b).data("dropdown-info").rank;
+        var afk1 = $(a).find(".icon-time").length > 0;
+        var afk2 = $(b).find(".icon-time").length > 0;
+        var name1 = a.children[1].innerHTML.toLowerCase();
+        var name2 = b.children[1].innerHTML.toLowerCase();
+
+        if(USEROPTS.sort_afk) {
+            if(afk1 && !afk2)
+                return 1;
+            if(!afk1 && afk2)
+                return -1;
+        }
+
+        if(USEROPTS.sort_rank) {
+            if(r1 < r2)
+                return 1;
+            if(r1 > r2)
+                return -1;
+        }
+
+        return name1 === name2 ? 0 : (name1 < name2 ? -1 : 1);
+    });
+
+    list.forEach(function (item) {
+        $(item).detach();
+    });
+    list.forEach(function (item) {
+        $(item).appendTo($("#userlist"));
+    });
+}
+
 /* queue stuff */
 
 function scrollQueue() {
@@ -541,6 +577,18 @@ function showOptionsMenu() {
     showts.prop("checked", USEROPTS.show_timestamps);
     addOption("Show timestamps", tscontainer);
 
+    var srcontainer = $("<label/>").addClass("checkbox")
+        .text("Sort userlist by rank");
+    var sr = $("<input/>").attr("type", "checkbox").appendTo(srcontainer);
+    sr.prop("checked", USEROPTS.sort_rank);
+    addOption("Userlist sort", srcontainer);
+
+    var sacontainer = $("<label/>").addClass("checkbox")
+        .text("AFKers at bottom of userlist");
+    var sa = $("<input/>").attr("type", "checkbox").appendTo(sacontainer);
+    sa.prop("checked", USEROPTS.sort_afk);
+    addOption("Userlist sort", sacontainer);
+
     var blinkcontainer = $("<label/>").addClass("checkbox")
         .text("Flash title on every incoming message");
     var blink = $("<input/>").attr("type", "checkbox").appendTo(blinkcontainer);
@@ -596,6 +644,9 @@ function showOptionsMenu() {
         USEROPTS.qbtn_idontlikechange = oqbtn.prop("checked");
         USEROPTS.ignore_channelcss    = nocss.prop("checked");
         USEROPTS.ignore_channeljs     = nojs.prop("checked");
+        USEROPTS.sort_rank            = sr.prop("checked");
+        USEROPTS.sort_afk             = sa.prop("checked");
+        sortUserlist();
         if(CLIENT.rank >= Rank.Moderator) {
             USEROPTS.modhat = modhat.prop("checked");
             USEROPTS.joinmessage = join.prop("checked");
@@ -1237,9 +1288,11 @@ function formatChatMessage(data) {
     }
     if(data.superadminflair)
         skip = false;
-    if(data.msgclass == "server-whisper") {
+    if(data.msgclass == "server-whisper")
         skip = true;
-    }
+    // Prevent impersonation by abuse of the bold filter
+    if(data.msg.match(/^\s*<strong>\w+\s*:\s*<\/strong>\s*/))
+        skip = false;
     LASTCHATNAME = data.username;
     LASTCHATTIME = data.time;
     var div = $("<div/>");

From 07a3e3cc21efa243042b2b4cba618bd5857d8fef Mon Sep 17 00:00:00 2001
From: calzoneman <calzoneman@gmail.com>
Date: Thu, 8 Aug 2013 18:25:56 -0400
Subject: [PATCH 04/11] A few minor changes - Fix paginator always showing 5
 buttons even if there are less pages - Clear AFK timer when clicking voteskip
 - Change the modestbranding youtube param because people wanted the
 like/dislike buttons

---
 channel.js                 | 5 ++---
 www/assets/js/paginator.js | 2 +-
 www/assets/js/player.js    | 1 -
 www/assets/js/util.js      | 2 +-
 4 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/channel.js b/channel.js
index cb50d1c6..edbf163e 100644
--- a/channel.js
+++ b/channel.js
@@ -1516,9 +1516,8 @@ Channel.prototype.tryVoteskip = function(user) {
         return;
     }
     // Voteskip = auto-unafk
-    if(user.meta.afk) {
-        user.setAFK(false);
-    }
+    user.setAFK(false);
+    user.autoAFK();
     if(!this.voteskip) {
         this.voteskip = new Poll("voteskip", "voteskip", ["yes"]);
     }
diff --git a/www/assets/js/paginator.js b/www/assets/js/paginator.js
index efaa6ad6..e2fcac53 100644
--- a/www/assets/js/paginator.js
+++ b/www/assets/js/paginator.js
@@ -48,7 +48,7 @@
                     .appendTo(sep);
             }
         }
-        for(var i = s; i < s + this.opts.maxPages; i++) {
+        for(var i = s; i < s + this.opts.maxPages && i < s + pages; i++) {
             (function (i) {
             var li = $("<li/>").appendTo(ul);
             if(i == p)
diff --git a/www/assets/js/player.js b/www/assets/js/player.js
index 40661b5c..ca451fc7 100644
--- a/www/assets/js/player.js
+++ b/www/assets/js/player.js
@@ -37,7 +37,6 @@ var YouTubePlayer = function (data) {
                     autoplay: 1,        // Autoplay video
                     controls: 1,        // Show controls
                     iv_load_policy: 3,  // No annotations
-                    modestbranding: 1,  // No logo
                     rel: 0,             // No related videos
                     wmode: wmode
                 },
diff --git a/www/assets/js/util.js b/www/assets/js/util.js
index b2aae050..91a70f55 100644
--- a/www/assets/js/util.js
+++ b/www/assets/js/util.js
@@ -1499,12 +1499,12 @@ function genPermissionsEditor() {
     makeOption("Jump to video", "playlistjump", standard, CHANNEL.perms.playlistjump+"");
     makeOption("Queue playlist", "playlistaddlist", standard, CHANNEL.perms.playlistaddlist+"");
     makeOption("Queue livestream", "playlistaddlive", standard, CHANNEL.perms.playlistaddlive+"");
+    makeOption("Embed custom media", "playlistaddcustom", standard, CHANNEL.perms.playlistaddcustom + "");
     makeOption("Exceed maximum media length", "exceedmaxlength", standard, CHANNEL.perms.exceedmaxlength+"");
     makeOption("Add nontemporary media", "addnontemp", standard, CHANNEL.perms.addnontemp+"");
     makeOption("Temp/untemp playlist item", "settemp", standard, CHANNEL.perms.settemp+"");
     makeOption("Shuffle playlist", "playlistshuffle", standard, CHANNEL.perms.playlistshuffle+"");
     makeOption("Clear playlist", "playlistclear", standard, CHANNEL.perms.playlistclear+"");
-    makeOption("Embed custom media", "playlistaddcustom", standard, CHANNEL.perms.playlistaddcustom + "");
 
     addDivider("Polls");
     makeOption("Open/Close poll", "pollctl", modleader, CHANNEL.perms.pollctl+"");

From e53a0eee4c4bc5139fe3f48ad6732a90d598f04f Mon Sep 17 00:00:00 2001
From: calzoneman <calzoneman@gmail.com>
Date: Thu, 8 Aug 2013 18:28:24 -0400
Subject: [PATCH 05/11] Increment version number

---
 package.json | 2 +-
 server.js    | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/package.json b/package.json
index 42902bea..faa921eb 100644
--- a/package.json
+++ b/package.json
@@ -2,7 +2,7 @@
     "author": "Calvin Montgomery",
     "name": "CyTube",
     "description": "Online media synchronizer and chat",
-    "version": "2.3.1",
+    "version": "2.3.2",
     "repository": {
         "url": "http://github.com/calzoneman/sync"
     },
diff --git a/server.js b/server.js
index fa744a35..38a28dfe 100644
--- a/server.js
+++ b/server.js
@@ -6,7 +6,7 @@ var Logger = require("./logger");
 var Channel = require("./channel");
 var User = require("./user");
 
-const VERSION = "2.3.1";
+const VERSION = "2.3.2";
 
 function getIP(req) {
     var raw = req.connection.remoteAddress;

From 539d728df410c096a6807ba37619ae8171da99e4 Mon Sep 17 00:00:00 2001
From: calzoneman <calzoneman@gmail.com>
Date: Thu, 8 Aug 2013 20:42:46 -0400
Subject: [PATCH 06/11] Fix weird case of playlist skipping when removing
 leader

---
 playlist.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/playlist.js b/playlist.js
index d10a68ba..c91de926 100644
--- a/playlist.js
+++ b/playlist.js
@@ -458,6 +458,7 @@ Playlist.prototype.lead = function(lead) {
         this._leadInterval = false;
     }
     else if(this.leading && !this._leadInterval) {
+        this._lastUpdate = Date.now();
         this._leadInterval = setInterval(function() {
             pl._leadLoop();
         }, 1000);

From 8f190403f8a169b6343c608ce47fe68532530ffa Mon Sep 17 00:00:00 2001
From: Calvin Montgomery <calzoneman@gmail.com>
Date: Thu, 8 Aug 2013 23:23:19 -0400
Subject: [PATCH 07/11] Fix leader bug

---
 www/assets/js/callbacks.js | 11 +----------
 1 file changed, 1 insertion(+), 10 deletions(-)

diff --git a/www/assets/js/callbacks.js b/www/assets/js/callbacks.js
index b303fd59..04f9928c 100644
--- a/www/assets/js/callbacks.js
+++ b/www/assets/js/callbacks.js
@@ -640,16 +640,7 @@ Callbacks = {
                 // I'm a leader!  Set up sync function
                 if(LEADTMR)
                     clearInterval(LEADTMR);
-                LEADTMR = setInterval(function() {
-                    PLAYER.getTime(function(seconds) {
-                        socket.emit("mediaUpdate", {
-                            id: PLAYER.id,
-                            currentTime: seconds,
-                            paused: PLAYER.paused,
-                            type: PLAYER.type
-                        });
-                    });
-                }, 5000);
+                LEADTMR = setInterval(sendVideoUpdate, 5000);
             }
             // I'm not a leader.  Don't send syncs to the server
             else {

From 637eadea237f7e5a7037d6d0127608570b73f45d Mon Sep 17 00:00:00 2001
From: calzoneman <calzoneman@gmail.com>
Date: Fri, 9 Aug 2013 16:28:25 -0400
Subject: [PATCH 08/11] Fix #249.  I don't even want to calculate the odds of
 someone saying a message that contains /afk not at the beginning while being
 afk.

---
 user.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/user.js b/user.js
index dfec554e..6395c6c8 100644
--- a/user.js
+++ b/user.js
@@ -205,7 +205,7 @@ User.prototype.initCallbacks = function() {
 
     this.socket.on("chatMsg", function(data) {
         if(this.channel != null) {
-            if(data.msg.indexOf("/afk") == -1) {
+            if(data.msg.indexOf("/afk") != 0) {
                 this.setAFK(false);
                 this.autoAFK();
             }

From 293576a3af892c9f88aa14691f083f746894bbf8 Mon Sep 17 00:00:00 2001
From: calzoneman <calzoneman@gmail.com>
Date: Fri, 9 Aug 2013 16:32:05 -0400
Subject: [PATCH 09/11] Fix #244 again. I want to get off autotemp's wild ride

---
 channel.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/channel.js b/channel.js
index 11e69f95..3460e139 100644
--- a/channel.js
+++ b/channel.js
@@ -1144,6 +1144,7 @@ Channel.prototype.tryQueue = function(user, data) {
     }
 
     data.queueby = user ? user.name : "";
+    data.temp = !this.hasPermission(user, "addnontemp");
 
     if(data.list)
         this.addMediaList(data, user);
@@ -1160,7 +1161,7 @@ Channel.prototype.addMedia = function(data, user) {
         user.socket.emit("queueFail", "You don't have permission to add cusstom embeds");
         return;
     }
-    data.temp = isLive(data.type) || !this.hasPermission(user, "addnontemp");
+    data.temp = data.temp || isLive(data.type);
     data.queueby = user ? user.name : "";
     data.maxlength = this.hasPermission(user, "exceedmaxlength") ? 0 : this.opts.maxlength;
     var chan = this;

From dba93ca3b6f590d4b97457cd9b9e4b18eea4c12b Mon Sep 17 00:00:00 2001
From: calzoneman <calzoneman@gmail.com>
Date: Sun, 11 Aug 2013 17:39:10 -0400
Subject: [PATCH 10/11] Fix #254

---
 www/assets/js/data.js | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/www/assets/js/data.js b/www/assets/js/data.js
index dd324cc9..f7948d08 100644
--- a/www/assets/js/data.js
+++ b/www/assets/js/data.js
@@ -115,7 +115,9 @@ var USEROPTS = {
     qbtn_idontlikechange : getOrDefault("qbtn_idontlikechange", false),
     first_visit          : getOrDefault("first_visit", true),
     ignore_channelcss    : getOrDefault("ignore_channelcss", false),
-    ignore_channeljs     : getOrDefault("ignore_channeljs", false)
+    ignore_channeljs     : getOrDefault("ignore_channeljs", false),
+    sort_rank            : getOrDefault("sort_rank", false),
+    sort_afk             : getOrDefault("sort_afk", false)
 };
 
 var NO_WEBSOCKETS = USEROPTS.altsocket;

From 9c22fbc462b989b105ccab981eee5ac8c9d7556a Mon Sep 17 00:00:00 2001
From: calzoneman <calzoneman@gmail.com>
Date: Sun, 11 Aug 2013 17:44:48 -0400
Subject: [PATCH 11/11] Add try-catch to urlRetrieve to prevent uncaught HTTPS
 exceptions

---
 get-info.js | 24 ++++++++++++++----------
 1 file changed, 14 insertions(+), 10 deletions(-)

diff --git a/get-info.js b/get-info.js
index 24ef4071..c545926d 100644
--- a/get-info.js
+++ b/get-info.js
@@ -17,18 +17,22 @@ var CustomEmbedFilter = require("./customembed").filter;
 
 module.exports = function (Server) {
     function urlRetrieve(transport, options, callback) {
-        var req = transport.request(options, function (res) {
-            var buffer = "";
-            res.setEncoding("utf-8");
-            res.on("data", function (chunk) {
-                buffer += chunk;
+        try {
+            var req = transport.request(options, function (res) {
+                var buffer = "";
+                res.setEncoding("utf-8");
+                res.on("data", function (chunk) {
+                    buffer += chunk;
+                });
+                res.on("end", function () {
+                    callback(res.statusCode, buffer);
+                });
             });
-            res.on("end", function () {
-                callback(res.statusCode, buffer);
-            });
-        });
 
-        req.end();
+            req.end();
+        } catch(e) {
+            callback(503, "");
+        }
     }
 
     var Getters = {