Handle null body when deciding which attachments to display #804

Foundry376 · Jul 8, 2018 · 2623d8e · 2623d8e
1 parent 8df74d1
commit 2623d8e
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/app/internal_packages/message-list/lib/message-item.jsx b/app/internal_packages/message-list/lib/message-item.jsx
@@ -101,9 +101,11 @@ export default class MessageItem extends React.Component {
   }
 
   _renderAttachments() {
-    const { files = [], body = '', id } = this.props.message;
+    const { files = [], body, id } = this.props.message;
     const { filePreviewPaths, downloads } = this.state;
-    const attachedFiles = files.filter(f => !f.contentId || !body.includes(`cid:${f.contentId}`));
+    const attachedFiles = files.filter(
+      f => !f.contentId || !(body || '').includes(`cid:${f.contentId}`)
+    );
 
     return (
       <div>