From 0e332ec19d6d68c6b4aef58eb3753aa711a89e3f Mon Sep 17 00:00:00 2001 From: Yash Srivastav Date: Sat, 19 Aug 2017 21:07:22 +0530 Subject: [PATCH] Merge redundant functions into one --- client/js/lounge.js | 29 ++++++++++------------------- 1 file changed, 10 insertions(+), 19 deletions(-) diff --git a/client/js/lounge.js b/client/js/lounge.js index 002f7dd7..9fcf8cb1 100644 --- a/client/js/lounge.js +++ b/client/js/lounge.js @@ -71,9 +71,10 @@ $(function() { search(term, callback) { term = term.slice(1); if (term[0] === "@") { - callback(completeNicks(term.slice(1)).map((val) => "@" + val)); + callback(completeNicks(term.slice(1), true) + .map((val) => ["@" + val[0], "@" + val[1]])); } else { - callback(completeNicks(term)); + callback(completeNicks(term, true)); } }, template([string, ]) { @@ -290,7 +291,7 @@ $(function() { chat.find(".chan.active .chat").trigger("msg.sticky"); // fix growing }) - .tab(tabCompleteNicks, {hint: false}) + .tab((word) => completeNicks(word, false), {hint: false}) .on("autocomplete:on", function() { enableAutocomplete(); }); @@ -943,8 +944,9 @@ $(function() { return results.map((el) => [el.string, el.original]); } - function tabCompleteNicks(word) { + function completeNicks(word, isFuzzy) { const users = chat.find(".active .users"); + word = word.toLowerCase(); // Lobbies and private chats do not have an user list if (!users.length) { @@ -952,26 +954,15 @@ $(function() { } const words = users.data("nicks"); - + if (isFuzzy) { + return fuzzyGrep(word, words); + } return $.grep( words, - (w) => !w.toLowerCase().indexOf(word.toLowerCase()) + (w) => !w.toLowerCase().indexOf(word) ); } - function completeNicks(word) { - const users = chat.find(".active .users"); - - // Lobbies and private chats do not have an user list - if (!users.length) { - return []; - } - - const words = users.data("nicks"); - - return fuzzyGrep(word, words); - } - function completeCommands(word) { const words = constants.commands.slice();