From ee92e3603c7b9dceb22ee5ff6cbd4aab86cc9faf Mon Sep 17 00:00:00 2001
From: Dan Stillman <dstillman@zotero.org>
Date: Wed, 22 Aug 2007 15:25:30 +0000
Subject: [PATCH] Fixes #727, Dragging linked file out of Zotero copies entire
 parent directory

---
 chrome/content/zotero/xpcom/itemTreeView.js | 21 ++++++++++++---------
 1 file changed, 12 insertions(+), 9 deletions(-)

diff --git a/chrome/content/zotero/xpcom/itemTreeView.js b/chrome/content/zotero/xpcom/itemTreeView.js
index 295a55a07..3912940fc 100644
--- a/chrome/content/zotero/xpcom/itemTreeView.js
+++ b/chrome/content/zotero/xpcom/itemTreeView.js
@@ -1490,10 +1490,12 @@ Zotero.ItemTreeView.fileDragDataProvider.prototype = {
 			var existingFileNames = [];
 			
 			for (var i=0; i<items.length; i++) {
+				// TODO create URL?
 				if (!items[i].isAttachment() ||
 						items[i].getAttachmentLinkMode() == Zotero.Attachments.LINK_MODE_LINKED_URL) {
 					continue;
 				}
+				
 				var file = items[i].getFile();
 				if (!file) {
 					Components.utils.reportError("File not found for item " + items[i].getID());
@@ -1502,14 +1504,16 @@ Zotero.ItemTreeView.fileDragDataProvider.prototype = {
 				
 				// Determine if we need to copy multiple files for this item
 				// (web page snapshots)
-				var parentDir = file.parent;
-				var files = parentDir.directoryEntries;
-				var numFiles = 0;
-				while (files.hasMoreElements()) {
-					var f = files.getNext();
-					f.QueryInterface(Components.interfaces.nsILocalFile);
-					if (f.leafName.indexOf('.') != 0) {
-						numFiles++;
+				if (items[i].getAttachmentLinkMode() != Zotero.Attachments.LINK_MODE_LINKED_FILE) {
+					var parentDir = file.parent;
+					var files = parentDir.directoryEntries;
+					var numFiles = 0;
+					while (files.hasMoreElements()) {
+						var f = files.getNext();
+						f.QueryInterface(Components.interfaces.nsILocalFile);
+						if (f.leafName.indexOf('.') != 0) {
+							numFiles++;
+						}
 					}
 				}
 				
@@ -1545,7 +1549,6 @@ Zotero.ItemTreeView.fileDragDataProvider.prototype = {
 						}
 					}
 					catch (e) {
-						Zotero.debug(e);
 						if (e.name == 'NS_ERROR_FILE_ALREADY_EXISTS') {
 							// Keep track of items that already existed
 							existingItems.push(items[i].getID());