fix(imap-indexer): fix fileContentHash error in imap indexer, check for attachmentInfo beforehand ZMS-186 (#756)

NickOvt · web-flow · commit 7daa0e35d546 · 2024-11-07T22:48:11.000+02:00
diff --git a/imap-core/lib/indexer/indexer.js b/imap-core/lib/indexer/indexer.js
@@ -626,11 +626,12 @@ class Indexer {
                 mimeTree.attachmentMap[node.attachmentId] = id;
 
                 let attachmentInfo = maildata.attachments && maildata.attachments.find(a => a.id === node.attachmentId); // get reference to attachment info
+
                 if (attachmentInfo && node.body) {
                     attachmentInfo.size = node.body.length;
                 }
 
-                if (fileContentHash) {
+                if (attachmentInfo && fileContentHash) {
                     attachmentInfo.fileContentHash = fileContentHash;
                 }
 

Original file line number	Diff line number	Diff line change
`@@ -626,11 +626,12 @@ class Indexer {`
`626`	`626`	`mimeTree.attachmentMap[node.attachmentId] = id;`
`627`	`627`
`628`	`628`	`let attachmentInfo = maildata.attachments && maildata.attachments.find(a => a.id === node.attachmentId); // get reference to attachment info`
	`629`	`+`
`629`	`630`	`if (attachmentInfo && node.body) {`
`630`	`631`	`attachmentInfo.size = node.body.length;`
`631`	`632`	`}`
`632`	`633`
`633`		`- if (fileContentHash) {`
	`634`	`+ if (attachmentInfo && fileContentHash) {`
`634`	`635`	`attachmentInfo.fileContentHash = fileContentHash;`
`635`	`636`	`}`
`636`	`637`