www

commit 7aeae3817dd8c39f62de16a906c951d1d8f44b40
parent 916db3fd4878c1e72f530c21ecb92bd0159a70ba
Author: Simon Kornblith <simon@simonster.com>
Date:   Mon, 25 Jun 2012 00:35:29 -0400

Make htmlSpecialChars faster

Diffstat:
M
chrome/content/zotero/xpcom/utilities.js
 | 
23
++++++++---------------

1 file changed, 8 insertions(+), 15 deletions(-)
diff --git a/chrome/content/zotero/xpcom/utilities.js b/chrome/content/zotero/xpcom/utilities.js
@@ -304,24 +304,19 @@ Zotero.Utilities = {
 	 * @type String
 	 */
 	 "htmlSpecialChars":function(/**String*/ str) {
-		if (typeof str != 'string') {
-			throw "Argument '" + str + "' must be a string in Zotero.Utilities.htmlSpecialChars()";
-		}
+		if (typeof str != 'string') str = str.toString();
 		
 		if (!str) {
 			return '';
 		}
 		
-		var chars = ['&', '"',"'",'<','>'];
-		var entities = ['amp', 'quot', 'apos', 'lt', 'gt'];
-		
-		var newString = str;
-		for (var i = 0; i < chars.length; i++) {
-			var re = new RegExp(chars[i], 'g');
-			newString = newString.replace(re, '&' + entities[i] + ';');
-		}
-		
-		newString = newString.replace(/&lt;ZOTERO([^\/]+)\/&gt;/g, function (str, p1, offset, s) {
+		return str
+			.replace(/&/g, '&amp;')
+			.replace(/"/g, '&quot;')
+			.replace(/'/g, '&apos;')
+			.replace(/</g, '&lt;')
+			.replace(/>/g, '&gt;')
+			.replace(/&lt;ZOTERO([^\/]+)\/&gt;/g, function (str, p1, offset, s) {
 			switch (p1) {
 				case 'BREAK':
 					return '<br/>';
@@ -331,8 +326,6 @@ Zotero.Utilities = {
 					return p1;
 			}
 		});
-		
-		return newString;
 	},
 
 	/**