www

commit f64d0879264f7b681c8d27063d35c0df36cdab47
parent 6dd94f0a3ae2be60b0c6ebd4c27559f679adaaad
Author: Aurimas Vinckevicius <aurimas.dev@gmail.com>
Date:   Sat,  2 Mar 2013 19:14:38 -0600

Don't set unknown mimeTypes to "application/octet-stream"

Diffstat:
M
chrome/content/zotero/xpcom/mime.js
 | 
2
--
M
chrome/content/zotero/xpcom/translation/translate_item.js
 | 
4
++--

2 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/chrome/content/zotero/xpcom/mime.js b/chrome/content/zotero/xpcom/mime.js
@@ -330,8 +330,6 @@ Zotero.MIME = new function(){
 				var mimeType = xmlhttp.channel.contentType;
 			}
 			
-			if(!mimeType) mimeType = 'application/octet-stream';	//unknown item type according to RFC 2046 section 4.5.1
-			
 			var nsIURL = Components.classes["@mozilla.org/network/standard-url;1"]
 						.createInstance(Components.interfaces.nsIURL);
 			nsIURL.spec = url;
diff --git a/chrome/content/zotero/xpcom/translation/translate_item.js b/chrome/content/zotero/xpcom/translation/translate_item.js
@@ -405,12 +405,12 @@ Zotero.Translate.ItemSaver.prototype = {
 				// Save attachment if snapshot pref enabled or not HTML
 				// (in which case downloadAssociatedFiles applies)
 				} else {
-					if(!attachment.mimeType && attachment.mimeType !== '') {	//in case '' indicates unknwon mime type at some point
+					if(!attachment.mimeType && attachment.mimeType !== '') {
 						Zotero.debug("Translate: No mimeType specified for a possible snapshot. Trying to determine mimeType.", 4);
 						var me = this;
 						try {
 							Zotero.MIME.getMIMETypeFromURL(attachment.url, function (mimeType, hasNativeHandler) {
-								attachment.mimeType = mimeType;
+								attachment.mimeType = mimeType || '';
 								me._saveAttachmentDownload(attachment, parentID, attachmentCallback);
 							}, this._cookieSandbox);
 						} catch(e) {