www

commit 0b552c5fdbf2e56238966a55500234a479912796
parent 85815dad4936a9dba244b71b5a148f9f16e22d1c
Author: David Norton <david@nortoncrew.com>
Date:   Fri,  4 Aug 2006 14:36:04 +0000

Closes #153, When deleting an item, don't give option to erase attached notes and files when there are none
Also, the default option (when applicable) is to delete attached notes and files.

Diffstat:
M
chrome/chromeFiles/content/scholar/overlay.js
 | 
15
+++++++++++++--

1 file changed, 13 insertions(+), 2 deletions(-)
diff --git a/chrome/chromeFiles/content/scholar/overlay.js b/chrome/chromeFiles/content/scholar/overlay.js
@@ -240,11 +240,22 @@ var ScholarPane = new function()
 	{
 		if(itemsView && itemsView.selection.count > 0)
 		{
-			var eraseChildren = {};
+			var eraseChildren = {value: true};
 			var promptService = Components.classes["@mozilla.org/embedcomp/prompt-service;1"]
                     						.getService(Components.interfaces.nsIPromptService);
+			var hasChildren;
+			
+			var start = new Object();
+			var end = new Object();
+			for (var i=0, len=itemsView.selection.getRangeCount(); i<len && !hasChildren; i++)
+			{
+				itemsView.selection.getRangeAt(i,start,end);
+				for (var j=start.value; j<=end.value && !hasChildren; j++)
+					if(itemsView._getItemAtRow(j).numNotes() || itemsView._getItemAtRow(j).numFiles())
+						hasChildren = true;
+			}
 
-			if(promptService.confirmCheck(window, Scholar.getString('pane.items.delete.title'), Scholar.getString('pane.items.delete'), Scholar.getString('pane.items.delete.attached'), eraseChildren))
+			if(promptService.confirmCheck(window, Scholar.getString('pane.items.delete.title'), Scholar.getString('pane.items.delete'), ( hasChildren ? Scholar.getString('pane.items.delete.attached') : ''), eraseChildren))
 				itemsView.deleteSelection(eraseChildren.value);
 		}
 	}