From 3d9619abaf75e00a14027e34196d667ae6095d01 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Lorenz=20H=C3=BCbschle-Schneider?=
 <lorenz-dev@lgh-alumni.de>
Date: Wed, 7 May 2014 23:38:26 +0100
Subject: [PATCH] Fix nick clicking

- move to correct scope
- use Angular input model instead of accessing DOM directly
- nicklist has changed, adapt to that
---
 js/glowingbear.js | 78 +++++++++++++++++++++++++----------------------
 1 file changed, 41 insertions(+), 37 deletions(-)

diff --git a/js/glowingbear.js b/js/glowingbear.js
index 5316ce8..83b9850 100644
--- a/js/glowingbear.js
+++ b/js/glowingbear.js
@@ -870,43 +870,6 @@ weechat.controller('WeechatCtrl', ['$rootScope', '$scope', '$store', '$timeout',
         }
     };
 
-    $scope.highlightNick = function(prefix) {
-        // Extract nick from bufferline prefix
-        var nick = prefix[prefix.length - 1].text;
-
-        var input = document.getElementById('sendMessage');
-        var newValue = input.value;
-        var addColon = newValue.length === 0;
-        if (newValue.length > 0) {
-            // Try to determine if it's a sequence of nicks
-            var trimmedValue = newValue.trim();
-            if (trimmedValue.charAt(trimmedValue.length - 1) === ':') {
-                // get last word
-                var lastSpace = trimmedValue.lastIndexOf(' ') + 1;
-                var lastWord = trimmedValue.slice(lastSpace, trimmedValue.length - 1);
-                var nicklist = models.getActiveBuffer().getNicklistByTime();
-                // check against nicklist to see if it's a list of highlights
-                if (nicklist.indexOf(lastWord) !== -1) {
-                    // It's another highlight!
-                    newValue = newValue.slice(0, newValue.lastIndexOf(':')) + ' ';
-                    addColon = true;
-                }
-            }
-
-            // Add a space before the nick if there isn't one already
-            // Last char might have changed above, so re-check
-            if (newValue.charAt(newValue.length - 1) !== ' ') {
-                newValue += ' ';
-            }
-        }
-        // Add highlight to nicklist
-        newValue += nick;
-        if (addColon) {
-            newValue += ': ';
-        }
-        input.value = newValue;
-        input.focus();
-    };
 
     // Calculate number of lines to fetch
     $scope.calculateNumLines = function() {
@@ -1297,6 +1260,47 @@ weechat.directive('inputBar', function() {
                 }
             };
 
+            $rootScope.highlightNick = function(prefix) {
+                // Extract nick from bufferline prefix
+                var nick = prefix[prefix.length - 1].text;
+
+                var newValue = $scope.command || '';  // can be undefined, in that case, use the empty string
+                var addColon = newValue.length === 0;
+                if (newValue.length > 0) {
+                    // Try to determine if it's a sequence of nicks
+                    var trimmedValue = newValue.trim();
+                    if (trimmedValue.charAt(trimmedValue.length - 1) === ':') {
+                        // get last word
+                        var lastSpace = trimmedValue.lastIndexOf(' ') + 1;
+                        var lastWord = trimmedValue.slice(lastSpace, trimmedValue.length - 1);
+                        var nicklist = models.getActiveBuffer().getNicklistByTime();
+                        // check against nicklist to see if it's a list of highlights
+                        for (var index in nicklist) {
+                            if (nicklist[index].name === lastWord) {
+                                // It's another highlight!
+                                newValue = newValue.slice(0, newValue.lastIndexOf(':')) + ' ';
+                                addColon = true;
+                                break;
+                            }
+                        }
+                    }
+
+                    // Add a space before the nick if there isn't one already
+                    // Last char might have changed above, so re-check
+                    if (newValue.charAt(newValue.length - 1) !== ' ') {
+                        newValue += ' ';
+                    }
+                }
+                // Add highlight to nicklist
+                newValue += nick;
+                if (addColon) {
+                    newValue += ': ';
+                }
+                $scope.command = newValue;
+                $scope.getInputNode().focus();
+            };
+
+
             // Handle key presses in the input bar
             $rootScope.handleKeyPress = function($event) {
                 // don't do anything if not connected