From 5411d4b67f2dcd6bc2764b23c36b0639cf9b7f6f Mon Sep 17 00:00:00 2001
From: David Norton <david@nortoncrew.com>
Date: Mon, 5 Jun 2006 15:49:11 +0000
Subject: [PATCH] Sorting on a colum: Much, much, much, better. (hope)fully
 implemented!

---
 .../content/scholar/itemTreeView.js           | 76 ++++++++++++++++---
 .../content/scholar/metadataPane.js           |  6 +-
 .../chromeFiles/content/scholar/overlay.xul   | 12 +--
 .../chromeFiles/content/scholar/test/test.xul |  2 +-
 4 files changed, 75 insertions(+), 21 deletions(-)

diff --git a/chrome/chromeFiles/content/scholar/itemTreeView.js b/chrome/chromeFiles/content/scholar/itemTreeView.js
index 65740662e..74c078dfb 100644
--- a/chrome/chromeFiles/content/scholar/itemTreeView.js
+++ b/chrome/chromeFiles/content/scholar/itemTreeView.js
@@ -31,6 +31,7 @@ Scholar.ItemTreeView.prototype.setTree = function(treebox)
 	if(this._treebox)
 		return;
 	this._treebox = treebox;
+	this.sort();
 }
 
 Scholar.ItemTreeView.prototype.getCellText = function(row, column)
@@ -57,7 +58,14 @@ Scholar.ItemTreeView.prototype._hideItem = function(row) 						{ this._dataItems
 
 Scholar.ItemTreeView.prototype._getItemAtRow = function(row)					{ return this._dataItems[row]; }
 
-Scholar.ItemTreeView.prototype.isSorted = function() 							{ return false; }
+Scholar.ItemTreeView.prototype.isSorted = function()
+{
+	for(var i=0, len=this._treebox.columns.count; i<len; i++)
+		if(this._treebox.columns.getColumnAt(i).element.getAttribute('sortActive'))
+			return true;
+	return false;
+}
+
 Scholar.ItemTreeView.prototype.isSeparator = function(row) 						{ return false; }
 Scholar.ItemTreeView.prototype.isContainer = function(row) 						{ return false; }
 Scholar.ItemTreeView.prototype.getLevel = function(row) 						{ return 0; }
@@ -68,8 +76,41 @@ Scholar.ItemTreeView.prototype.getImageSrc = function(row, col) 				{ }
 
 Scholar.ItemTreeView.prototype.cycleHeader = function(column)
 {
-	var order = 0;
-	if(order==0)
+	for(var i=0, len=this._treebox.columns.count; i<len; i++)
+	{
+		col = this._treebox.columns.getColumnAt(i);
+		if(column != col)
+		{
+			col.element.removeAttribute('sortActive');
+			col.element.removeAttribute('sortDirection');
+		}
+		else
+		{
+			col.element.setAttribute('sortActive',true);
+			col.element.setAttribute('sortDirection',col.element.getAttribute('sortDirection') == 'descending' ? 'ascending' : 'descending');
+		}
+	}
+	
+	this.sort();
+}
+
+Scholar.ItemTreeView.prototype.sort = function()
+{
+	this.selection.selectEventsSuppressed = true;
+	
+	var selectedIDs = new Array();
+	var start = new Object();
+	var end = new Object();
+	for (var i=0, len=this.selection.getRangeCount(); i<len; i++)
+	{
+		this.selection.getRangeAt(i,start,end);
+		for (var j=start.value; j<=end.value; j++)
+			selectedIDs.push(this._getItemAtRow(j).getID());
+	}
+	
+	var column = this._treebox.columns.getSortedColumn()
+	var order = column.element.getAttribute('sortDirection') == 'descending';
+	if(order)
 	{
 		function columnSort(a,b)
 		{
@@ -86,6 +127,13 @@ Scholar.ItemTreeView.prototype.cycleHeader = function(column)
 	
 	this._dataItems.sort(columnSort);
 	this._refreshHashMap();
+	
+	this.selection.clearSelection();
+	for(var i=0; i < selectedIDs.length; i++)
+	{
+		this.selection.toggleSelect(this._itemRowMap[selectedIDs[i]]);
+	}
+	this.selection.selectEventsSuppressed = false;
 	this._treebox.invalidate();
 }
 
@@ -187,7 +235,10 @@ Scholar.ItemTreeView.prototype.notify = function(action, type, ids)
 			var row = this._itemRowMap[ids[i]];
 			if(action == 'modify' && row != null) 	//must check for null because it could legitimately be 0
 			{
-				this._treebox.invalidateRow(row)
+				var item = Scholar.Items.get(ids[i]);
+				
+				this._treebox.invalidateRow(row);
+				madeChanges = true;
 			}
 			else if(action == 'add' && row == null)
 			{
@@ -200,19 +251,22 @@ Scholar.ItemTreeView.prototype.notify = function(action, type, ids)
 				}
 			
 				madeChanges = true;
-				
-				//TODO: sorted? figure it out later
 			}
 			
 		}
 	}
 	
 	if(madeChanges)
-		this._refreshHashMap();
-	
-	//Select last add
-	if(action == 'add' && item)
-		this.selection.select(this._itemRowMap[item.getID()]);
+	{
+		if(action == 'add')
+			this.selection.select(this._itemRowMap[item.getID()]);
+
+		if(this.isSorted())
+			this.sort();				//this also refreshes the hash map
+		else if(action != 'modify') //no need to update this if we just modified
+			this._refreshHashMap();
+			
+	}	
 }
 
 Scholar.ItemTreeView.prototype.canDrop = function(index, orient)
diff --git a/chrome/chromeFiles/content/scholar/metadataPane.js b/chrome/chromeFiles/content/scholar/metadataPane.js
index 3c2ec364f..55310632a 100644
--- a/chrome/chromeFiles/content/scholar/metadataPane.js
+++ b/chrome/chromeFiles/content/scholar/metadataPane.js
@@ -89,15 +89,15 @@ MetadataPane = new function()
 	
 	function toggleEdit(save)
 	{
-		if(_editButton.hidden && save)
-			saveItem();
-		
 		_cancelButton.hidden = _editButton.hidden;
 		_saveButton.hidden = _editButton.hidden;
 		_creatorsToolbar.hidden = _editButton.hidden;
 		
 		_editButton.hidden = !_editButton.hidden;
 		
+		if(!_editButton.hidden && save)
+			saveItem();
+		
 		reloadFields();
 	}
 	
diff --git a/chrome/chromeFiles/content/scholar/overlay.xul b/chrome/chromeFiles/content/scholar/overlay.xul
index 1e12189cd..29d45970d 100644
--- a/chrome/chromeFiles/content/scholar/overlay.xul
+++ b/chrome/chromeFiles/content/scholar/overlay.xul
@@ -86,32 +86,32 @@
 						<treecol
 							id="title"
 							label="&items.title_column;"
-							flex="4" persist="width ordinal hidden"/>
+							flex="4" persist="width ordinal hidden sortActive sortDirection"/>
 						<splitter class="tree-splitter"/>
 						<treecol
 							id="firstCreator"
 							label="&items.creator_column;"
-							flex="1" persist="width ordinal hidden"/>
+							flex="1" persist="width ordinal hidden sortActive sortDirection"/>
 						<splitter class="tree-splitter"/>
 						<treecol
 							id="source"
 							label="&items.source_column;"
-							flex="1" persist="width ordinal hidden"/>
+							flex="1" persist="width ordinal hidden sortActive sortDirection"/>
 						<splitter class="tree-splitter"/>
 						<treecol
 							id="rights" hidden="true"
 							label="&items.rights_column;"
-							flex="1" persist="width ordinal hidden"/>
+							flex="1" persist="width ordinal hidden sortActive sortDirection"/>
 						<splitter class="tree-splitter"/>
 						<treecol
 							id="dateAdded" hidden="true"
 							label="&items.dateAdded_column;"
-							flex="1" persist="width ordinal hidden"/>
+							flex="1" persist="width ordinal hidden sortActive sortDirection"/>
 						<splitter class="tree-splitter"/>
 						<treecol
 							id="dateModified" hidden="true"
 							label="&items.dateModified_column;"
-							flex="1" persist="width ordinal hidden"/>
+							flex="1" persist="width ordinal hidden sortActive sortDirection"/>
 					</treecols>
 			
 					<treechildren/>
diff --git a/chrome/chromeFiles/content/scholar/test/test.xul b/chrome/chromeFiles/content/scholar/test/test.xul
index 9ba04770f..4c3b9501f 100644
--- a/chrome/chromeFiles/content/scholar/test/test.xul
+++ b/chrome/chromeFiles/content/scholar/test/test.xul
@@ -12,5 +12,5 @@
 	<script src="../include.js"/>
 	<script src="test.js"/>
 	
-	<label></label>
+	<hbox><label>hello asf saf saf saf saf sa sf faj sj ak jgaslg jasg asglj</label></hbox>
 </window>