fix accidental double entry for sent messages

eklavya · Jul 6, 2018 · 091138c · 091138c
1 parent 48feb19
commit 091138c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 10 deletions.
diff --git a/gitterapi.js b/gitterapi.js
@@ -72,7 +72,7 @@ function markRead(token, userId, roomId, msgIds) {
     xhr.send({"chat": JSON.stringify(msgIds)});
 }
 
-function sendMessage(token, roomId, m, onSuccess) {
+function sendMessage(token, roomId, m) {
     print(token, roomId, m);
     var xhr = new XMLHttpRequest();
     xhr.open("POST", "https://api.gitter.im/v1/rooms/" + roomId + "/chatMessages", true);
@@ -81,10 +81,10 @@ function sendMessage(token, roomId, m, onSuccess) {
     xhr.onreadystatechange = function() {
         if(xhr.readyState === XMLHttpRequest.DONE) {
             if (xhr.status !== 200) print(xhr.statusText, xhr.responseText)
-            if (xhr.status === 200) {
-                var message = JSON.parse(xhr.responseText);
-                onSuccess(roomId, message);
-            }
+//            if (xhr.status === 200) {
+//                var message = JSON.parse(xhr.responseText);
+//                onSuccess(roomId, message);
+//            }
         }
     }
     var body = JSON.stringify({"text": m});

diff --git a/main.qml b/main.qml
@@ -519,11 +519,7 @@ Window {
     }
 
     function sendMessage() {
-        Gitter.sendMessage(settings.token, window.curRoom, curMsg.text,
-                           function (rmId, message) {
-                               msgs.model.append(message)
-                               saveMessage(rmId, message)
-                           })
+        Gitter.sendMessage(settings.token, window.curRoom, curMsg.text)
         curMsg.text = ""
     }