From ef2ff0a88458edfd51820d57a57783d3d4554348 Mon Sep 17 00:00:00 2001
From: Simon Kornblith <simon@simonster.com>
Date: Sun, 22 Jun 2008 17:16:43 +0000
Subject: [PATCH] =?UTF-8?q?closes=20#1043,=20store=20EndNote=C2=AE=20style?=
 =?UTF-8?q?s=20as=20binary=20blobs=20closes=20#1046,=20Provide=20some=20in?=
 =?UTF-8?q?dicator=20of=20whether=20style=20is=20originally=20from=20EndNo?=
 =?UTF-8?q?te=20will=20require=20that=20you=20re-import=20previously=20imp?=
 =?UTF-8?q?orted=20styles?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../content/zotero/preferences/preferences.js |  25 +++++-----
 .../zotero/preferences/preferences.xul        |   1 +
 chrome/content/zotero/xpcom/cite.js           |  25 ++++++++--
 chrome/content/zotero/xpcom/enstyle.js        |  46 ++++++++++--------
 chrome/locale/en-US/zotero/preferences.dtd    |   1 +
 chrome/skin/default/zotero/tick.png           | Bin 0 -> 537 bytes
 6 files changed, 61 insertions(+), 37 deletions(-)
 create mode 100755 chrome/skin/default/zotero/tick.png

diff --git a/chrome/content/zotero/preferences/preferences.js b/chrome/content/zotero/preferences/preferences.js
index a9650bf5b..4cc05895b 100644
--- a/chrome/content/zotero/preferences/preferences.js
+++ b/chrome/content/zotero/preferences/preferences.js
@@ -938,15 +938,23 @@ function refreshStylesList(cslID) {
 		var treerow = document.createElement('treerow');
 		var titleCell = document.createElement('treecell');
 		var updatedCell = document.createElement('treecell');
+		var cslCell = document.createElement('treecell');
 		
 		var updatedDate = Zotero.Date.formatDate(Zotero.Date.strToDate(styleData[i].updated), true);
 		
 		treeitem.setAttribute('id', 'zotero-csl-'+styleData[i].cslID);
 		titleCell.setAttribute('label', styleData[i].title);
 		updatedCell.setAttribute('label', updatedDate);
+		// if not EN
+		if(styleData[i].cslID.length < Zotero.ENConverter.uriPrefix.length ||
+				styleData[i].cslID.substr(0, Zotero.ENConverter.uriPrefix.length) != Zotero.ENConverter.uriPrefix) {
+			cslCell.setAttribute('src', 'chrome://zotero/skin/tick.png');
+			Zotero.debug("ISCSL");
+		}
 		
 		treerow.appendChild(titleCell);
 		treerow.appendChild(updatedCell);
+		treerow.appendChild(cslCell);
 		treeitem.appendChild(treerow);
 		treechildren.appendChild(treeitem);
 		
@@ -988,21 +996,11 @@ function addStyle() {
 			// read the rest of the bytes in the file
 			read += bStream.readBytes(file.fileSize-6);
 			
-			// get fallback name and modification date
-			var fallbackName = file.leafName;
-			fallbackName = fallbackName.replace(/\.ens$/i, "");
+			// get name and modification date
+			var name = file.leafName.replace(/\.ens$/i, "");
 			var date = new Date(file.lastModifiedTime);
-			Zotero.debug(file.lastModifiedTime);
-			Zotero.debug(date);
 			
-			try {
-				var enConverter = new Zotero.ENConverter(read, date, fallbackName);
-				var xml = enConverter.parse();
-			} catch(e) {
-				styleImportError();
-				throw e;
-			}
-			var cslID = Zotero.Cite.installStyle(xml.toXMLString());
+			var cslID = Zotero.Cite.installStyle(read, false, date, name);
 		} else {
 			// This _should_ get the right charset for us automatically
 			var fileURI = Components.classes["@mozilla.org/network/protocol;1?name=file"]
@@ -1018,6 +1016,7 @@ function addStyle() {
 				styleImportError();
 				throw e;
 			}
+			
 			var cslID = Zotero.Cite.installStyle(req.responseText);
 		}
 	}
diff --git a/chrome/content/zotero/preferences/preferences.xul b/chrome/content/zotero/preferences/preferences.xul
index b6aeb8e5b..2ab5ac030 100644
--- a/chrome/content/zotero/preferences/preferences.xul
+++ b/chrome/content/zotero/preferences/preferences.xul
@@ -338,6 +338,7 @@ To add a new preference:
 				<treecols>
 					<treecol id="styleManager-title" label="&zotero.preferences.styles.styleManager.title;" flex="3"/>
 					<treecol id="styleManager-updated" label="&zotero.preferences.styles.styleManager.updated;" flex="1"/>
+					<treecol id="styleManager-csl" label="&zotero.preferences.styles.styleManager.csl;"/>
 				</treecols>
 				<treechildren id="styleManager-rows"/>
 			</tree>
diff --git a/chrome/content/zotero/xpcom/cite.js b/chrome/content/zotero/xpcom/cite.js
index 469d77ffa..b68200131 100644
--- a/chrome/content/zotero/xpcom/cite.js
+++ b/chrome/content/zotero/xpcom/cite.js
@@ -76,6 +76,11 @@ Zotero.Cite = new function() {
 			if(csl.indexOf("<defaults") != -1) {
 				_lastCSL = new Zotero.CSL.Compat(csl);
 			} else {
+				if(csl.substr(0, 6) == "\x00\x08\xFF\x00\x00\x00") {	
+					// EN style
+					var enConverter = new Zotero.ENConverter(csl);
+					csl = enConverter.parse();
+				}
 				_lastCSL = new Zotero.CSL(csl);
 			}
 			
@@ -96,12 +101,20 @@ Zotero.Cite = new function() {
 	/**
 	 * installs a style 
 	 **/
-	function installStyle(cslString, loadURI) {
+	function installStyle(cslString, loadURI, date, name) {
+		var error = false;
 		try {
-			var xml = new XML(Zotero.CSL.Global.cleanXML(cslString));
+			if(cslString.substr(0, 6) == "\x00\x08\xFF\x00\x00\x00") { 			
+				// EN style
+				var enConverter = new Zotero.ENConverter(cslString, date, name);
+				var xml = enConverter.parse();
+			} else {
+				// CSL
+				var xml = new XML(Zotero.CSL.Global.cleanXML(cslString));
+			}
 		}
 		catch (e) {
-			var error = true;
+			error = true;
 			Components.utils.reportError(e);
 		}
 		
@@ -309,7 +322,11 @@ Zotero.Cite.MIMEHandler.StreamListener.prototype.onStopRequest = function(channe
 Zotero.CSL = function(csl) {
 	default xml namespace = "http://purl.org/net/xbiblio/csl"; with ({});
 	
-	this._csl = new XML(Zotero.CSL.Global.cleanXML(csl));
+	if(typeof csl != "XML") {
+		this._csl = new XML(Zotero.CSL.Global.cleanXML(csl));
+	} else {
+		this._csl = CSL;
+	}
 	
 	// initialize CSL
 	Zotero.CSL.Global.init();
diff --git a/chrome/content/zotero/xpcom/enstyle.js b/chrome/content/zotero/xpcom/enstyle.js
index 971daf97a..ea7d9a4a0 100644
--- a/chrome/content/zotero/xpcom/enstyle.js
+++ b/chrome/content/zotero/xpcom/enstyle.js
@@ -35,6 +35,9 @@ Zotero.ENConverter = function(styleData, date, fileTitle) {
 	}
 }
 
+// The URI prefix for EN styles
+Zotero.ENConverter.uriPrefix = "http://zotero.org/styles/en-converted/";
+
 /**
  * Mappings for item types
  *
@@ -201,31 +204,35 @@ Zotero.ENConverter.seriesCodes = ["\x01", "\x09", "\x0A", "\x0B", "\x15", "\x1D"
  **/
 Zotero.ENConverter.prototype.parseInfo = function() {
 	default xml namespace = "http://purl.org/net/xbiblio/csl"; with({});
-	
-	var guid = "";
-	for(var i=0; i<16; i++) {
-		var bite = Math.floor(Math.random() * 255);
-		
-		if(i == 4 || i == 6 || i == 8 || i == 10) {
-			guid += "-";
-			
-			// version
-			if(i == 6) bite = bite & 0x0f | 0x40;
-			// variant
-			if(i == 8) bite = bite & 0x3f | 0x80;
-		}
-		var str = bite.toString(16);
-		guid += str.length == 1 ? '0' + str : str;
-	}
-	
-	this.xml.info.id = "urn:uuid:"+guid;
+
 	if(this.fileTitle) {
 		var title = this.fileTitle;
 	} else {
 		var title = this.convertFromUTF16(this.findField(this.fields, "\x10")[0].data.replace("\xFB", "", "g"));
-		if(!title) title = "Untitled Style";
 	}
+	if(!title) {
+		var title = "Untitled Style";	
+		var identifier = "";
+		for(var i=0; i<16; i++) {
+			var bite = Math.floor(Math.random() * 255);
+			
+			if(i == 4 || i == 6 || i == 8 || i == 10) {
+				identifier += "-";
+				
+				// version
+				if(i == 6) bite = bite & 0x0f | 0x40;
+				// variant
+				if(i == 8) bite = bite & 0x3f | 0x80;
+			}
+			var str = bite.toString(16);
+			identifier += str.length == 1 ? '0' + str : str;
+		}
+	} else {
+		var identifier = encodeURIComponent(title);
+	}
+	
 	this.xml.info.title = title;
+	this.xml.info.id = Zotero.ENConverter.uriPrefix+identifier;
 	
 	if(this.date) {
 		var date = this.date;
@@ -1369,6 +1376,5 @@ Zotero.ENConverter.prototype.parse = function() {
 	this.parseCitation();
 	this.parseBibliography();
 	
-	Zotero.debug(this.xml);
 	return this.xml;
 }
\ No newline at end of file
diff --git a/chrome/locale/en-US/zotero/preferences.dtd b/chrome/locale/en-US/zotero/preferences.dtd
index 217bda58e..12e51724b 100644
--- a/chrome/locale/en-US/zotero/preferences.dtd
+++ b/chrome/locale/en-US/zotero/preferences.dtd
@@ -64,6 +64,7 @@
 <!ENTITY zotero.preferences.styles.styleManager				"Style Manager">
 <!ENTITY zotero.preferences.styles.styleManager.title		"Title">
 <!ENTITY zotero.preferences.styles.styleManager.updated		"Updated">
+<!ENTITY zotero.preferences.styles.styleManager.csl		"CSL">
 <!ENTITY zotero.preferences.export.getAdditionalStyles			"Get additional styles...">
 
 <!ENTITY zotero.preferences.prefpane.keys				"Shortcut Keys">
diff --git a/chrome/skin/default/zotero/tick.png b/chrome/skin/default/zotero/tick.png
new file mode 100755
index 0000000000000000000000000000000000000000..a9925a06ab02db30c1e7ead9c701c15bc63145cb
GIT binary patch
literal 537
zcmV+!0_OdRP)<h;3K|Lk000e1NJLTq000mG000mO1^@s6AM^iV00004XF*Lt006JZ
zHwB960000PbVXQnQ*UN;cVTj606}DLVr3vnZDD6+Qe|Oed2z{QJOBUzt4TybR5;7M
zli4mqVHAaX=Q!_TZq*o5LaK&TC+g@aMJbINs;8}1t3yf*$E6r*j>Hs{AQG2a)rMyf
zFQK~pm1x3+7!nu%-M`k}``c>^00{o_1pjWJUTfl8mg=3qGEl8H@}^@w`VUx0_$uy4
z2FhRqKX}xI*?Tv<H&*Dj)Tr<j*}f%KtVns|p$^k=7pCG)bZ)nz(p!uYhmcBtGG4MZ
zhBKz;vsk_?<x?$B#+(Sm9B-A&-Qq;?SOJwD!Fa)z^`0_M+_eepu5##IOknYS`&tC<
z#^ArRgWXi2%vG2uvgcE2i2di<+TKw6qc6;1IyMf?mMlSF4apWIQ-(bTzNj7ETMiu?
z3Y3nE5@K5(^<j-YCz?$CVH?a>1DJd8z#F#0c%*~rM30HE1@2o5m~}ZyoWhqv>ql{V
z1ZGE0lgcoK^lx+eqc*rAX1Ky;Xx3U%u#zG!m-;eD1Qsn@kf3|F9qz~|95=&g3(7!X
zB}JAT>RU;a%vaNOGnJ%e1=K6eAh43c(QN8RQ6~GP%O}Jju$~Ld*%`mO1p<rCAIZx2
bmuq|lzM`o-QVIh|00000NkvXXu0mjfO@{7Q

literal 0
HcmV?d00001