fix #1740

public/js/controllers/groupsCtrl.js

@@ -80,14 +80,16 @@ habitrpg.controller("GroupsCtrl", ['$scope', '$rootScope', 'Groups', '$http', 'A
 
   .controller('ChatCtrl', ['$scope', 'Groups', 'User', function($scope, Groups, User){
     $scope._chatMessage = '';
+    $scope._sending = false;
 
     $scope.postChat = function(group, message){
-      if (_.isEmpty(message)) return
-      $('.chat-btn').addClass('disabled');
+      console.log($scope._sending)
+      if (_.isEmpty(message) || $scope._sending) return;
+      $scope._sending = true;
       group.$postChat({message:message}, function(data){
-        $scope._chatMessage = '';
         group.chat = data.chat;
-        $('.chat-btn').removeClass('disabled');
+        $scope._chatMessage = '';
+        $scope._sending = false;
       });
     }
 

src/controllers/groups.js

@@ -210,9 +210,6 @@ api.postChat = function(req, res, next) {
     timestamp: +(new Date)
   };
 
-  // workaround for preventing duplicate chat messages. Real solution is to prevent <ENTER> between sends on the client
-  if (group.chat[0].uuid == message.uuid && group.chat[0].text == message.text) return res.json(group);
-
   group.chat.unshift(message);
   group.chat.splice(200);
 

views/options/social/chat-box.jade

@@ -6,7 +6,7 @@ form(ng-submit='postChat(group,_chatMessage)')
     table.pull-right
       tr
         td
-          input.btn.chat-btn(type='submit', value='Send Chat')
+          input.btn.chat-btn(type='submit', value='Send Chat', ng-class='{disabled: _sending == true}')
         td
           button.btn(type="button", ng-click='sync(group)', tooltip='Fetch Recent Messages')
             i(class='pull-right icon-refresh')