Fixes #2235

conversejs · Sep 18, 2020 · 0a7dff4 · 0a7dff4
1 parent 7330530
commit 0a7dff4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 19 deletions.
diff --git a/src/headless/converse-chat.js b/src/headless/converse-chat.js
@@ -179,8 +179,7 @@ converse.plugins.add('converse-chat', {
 
             getMessageText () {
                 if (this.get('is_encrypted')) {
-                    return this.get('plaintext') ||
-                           (api.settings.get('loglevel') === 'debug' ? __('Unencryptable OMEMO message') : null);
+                    return this.get('plaintext') || this.get('body') || __('Undecryptable OMEMO message');
                 }
                 return this.get('message');
             },
@@ -788,11 +787,16 @@ converse.plugins.add('converse-chat', {
 
             getMessageBodyQueryAttrs (attrs) {
                 if (attrs.message && attrs.msgid) {
-                    return {
-                        'message': attrs.message,
+                    const query = {
                         'from': attrs.from,
                         'msgid': attrs.msgid
                     }
+                    if (!attrs.is_encrypted) {
+                        // We can't match the message if it's a reflected
+                        // encrypted message (e.g. via MAM or in a MUC)
+                        query['message'] =  attrs.message;
+                    }
+                    return query;
                 }
             },
 

diff --git a/src/headless/converse-muc.js b/src/headless/converse-muc.js
@@ -1930,21 +1930,6 @@ converse.plugins.add('converse-muc', {
                 return false;
             },
 
-            getMessageBodyQueryAttrs (attrs) {
-                if (attrs.message && attrs.msgid) {
-                    const query = {
-                        'from': attrs.from,
-                        'msgid': attrs.msgid
-                    }
-                    if (!attrs.is_encrypted) {
-                        // We can't match the message if it's a reflected
-                        // encrypted MUC message
-                        query['message'] =  attrs.message;
-                    }
-                    return query;
-                }
-            },
-
             /**
              * Queue an incoming message stanza meant for this {@link _converse.Chatroom} for processing.
              * @async