www

commit c1035958d44cb32e1ce505b4868d451645e87de3
parent 4b602ed26c127c7fc182d26a3bad81470f6b2c3c
Author: Dan Stillman <dstillman@zotero.org>
Date:   Mon, 19 Dec 2016 04:19:18 -0500

Fix for child attachments being saved if prefs off (from 8c2c097c6)

Diffstat:
M
chrome/content/zotero/xpcom/translation/translate_item.js
 | 
5
+++--

1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/chrome/content/zotero/xpcom/translation/translate_item.js b/chrome/content/zotero/xpcom/translation/translate_item.js
@@ -137,9 +137,10 @@ Zotero.Translate.ItemSaver.prototype = {
 					// handle attachments
 					if (specialFields.attachments) {
 						for (let attachment of specialFields.attachments) {
-							if (this._canSaveAttachment(attachment)) {
-								attachmentCallback(attachment, 0);
+							if (!this._canSaveAttachment(attachment)) {
+								continue;
 							}
+							attachmentCallback(attachment, 0);
 							childAttachments.push([attachment, myID]);
 						}
 						// Restore the attachments field, since we use it later in