Merge pull request #291 from torhve/fix-empty

Don't send empty commands, use angular model instead of accessing element value directly. Fixes problem with pressing enter will send previous value in history.

js/glowingbear.js

@@ -1279,22 +1279,27 @@ weechat.directive('inputBar', function() {
                 var input = $scope.getInputNode();
                 var ab = models.getActiveBuffer();
 
-                // log to buffer history
-                ab.addToHistory(input.value);
+                // It's undefined early in the lifecycle of the program.
+                // Don't send empty commands
+                if($scope.command !== undefined && $scope.command !== '') {
 
-                // Split the command into multiple commands based on line breaks
-                _.each($scope.command.split(/\r?\n/), function(line) {
-                    connection.sendMessage(line);
-                });
+                    // log to buffer history
+                    ab.addToHistory($scope.command);
 
-                // Check for /clear command
-                if ($scope.command === '/buffer clear' || $scope.command === '/c') {
-                    $log.debug('Clearing lines');
-                    ab.clear();
-                }
+                    // Split the command into multiple commands based on line breaks
+                    _.each($scope.command.split(/\r?\n/), function(line) {
+                        connection.sendMessage(line);
+                    });
 
-                // Empty the input after it's sent
-                input.value = '';
+                    // Check for /clear command
+                    if ($scope.command === '/buffer clear' || $scope.command === '/c') {
+                        $log.debug('Clearing lines');
+                        ab.clear();
+                    }
+
+                    // Empty the input after it's sent
+                    $scope.command = '';
+                }
             };
 
             // Handle key presses in the input bar