From ab9ae36135f0e9e34739fc99d6832341734f35a0 Mon Sep 17 00:00:00 2001
From: Dan Stillman <dstillman@zotero.org>
Date: Tue, 12 Apr 2011 14:47:44 +0000
Subject: [PATCH] Fix date-based sorting in reports

---
 components/zotero-protocol-handler.js | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/components/zotero-protocol-handler.js b/components/zotero-protocol-handler.js
index a842ddf4a..e5b3abedf 100644
--- a/components/zotero-protocol-handler.js
+++ b/components/zotero-protocol-handler.js
@@ -417,9 +417,16 @@ function ChromeExtensionHandler() {
 							valA = Zotero.Items.getSortTitle(valA);
 							valB = Zotero.Items.getSortTitle(valB);
 						}
+						// Use multipart date -- would be better just to get
+						// the unformatted date value directly, but toArray(),
+						// which we currently use above, doesn't do that
+						else if (sorts[index].field == 'date') {
+							valA = Zotero.Date.strToMultipart(a[sorts[index].field]);
+							valB = Zotero.Date.strToMultipart(b[sorts[index].field]);
+						}
 						else {
-							var valA = a[sorts[index].field];
-							var valB = b[sorts[index].field];
+							valA = a[sorts[index].field];
+							valB = b[sorts[index].field];
 						}
 						
 						// Put empty values last