loadAttachments: Handle the zero -> some attachments case

FREEBIE
signalapp · Jun 1, 2017 · eb1a4b3 · eb1a4b3
1 parent 0e1b534
commit eb1a4b3
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/js/views/message_view.js b/js/views/message_view.js
@@ -261,10 +261,13 @@
             this.$('.avatar').replaceWith(avatarView.render().$('.avatar'));
         },
         loadAttachments: function() {
-            if (this.loadedAttachments) {
+            this.loadedAttachments = this.loadedAttachments || [];
+
+            // Messages can go from no attachments to some attachments (via key approval),
+            //   but they can't go from some attachments to a different set of them.
+            if (this.loadedAttachments.length) {
                 return;
             }
-            this.loadedAttachments = [];
 
             this.model.get('attachments').forEach(function(attachment) {
                 var view = new Whisper.AttachmentView({