mirrors/zotero
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Remove Zotero.File.getCharsetFromFile()/addCharsetListener()

Browse source 
Unused in over a decade, and now replaced by the PageData actor
This commit is contained in:
Dan Stillman 2022-06-17 04:09:21 -04:00
parent 6a2949be8a
commit 1dd24f7082
1 changed files with 0 additions and 87 deletions
Download patch file Download diff file Expand all files Collapse all files

87
chrome/content/zotero/xpcom/file.js
View file

@ -37,8 +37,6 @@ Zotero.File = new function(){
this.putContents = putContents; this.putContents = putContents;
this.getValidFileName = getValidFileName; this.getValidFileName = getValidFileName;
this.truncateFileName = truncateFileName; this.truncateFileName = truncateFileName;
this.getCharsetFromFile = getCharsetFromFile;
this.addCharsetListener = addCharsetListener;
this.pathToFile = function (pathOrFile) { this.pathToFile = function (pathOrFile) {
@ -1306,91 +1304,6 @@ Zotero.File = new function(){
return name + ext; return name + ext;
} }
/*
* Not implemented, but it'd sure be great if it were
*/
function getCharsetFromByteArray(arr) {
}
/*
* An extraordinarily inelegant way of getting the character set of a
* text file using a hidden browser
*
* I'm quite sure there's a better way
*
* Note: This is for text files -- don't run on other files
*
* 'callback' is the function to pass the charset (and, if provided, 'args')
* to after detection is complete
*/
function getCharsetFromFile(file, mimeType, callback, args){
if (!file || !file.exists()){
callback(false, args);
return;
}
if (mimeType.substr(0, 5) != 'text/' ||
!Zotero.MIME.hasInternalHandler(mimeType, this.getExtension(file))) {
callback(false, args);
return;
}
var browser = Zotero.Browser.createHiddenBrowser();
var url = Components.classes["@mozilla.org/network/protocol;1?name=file"]
.getService(Components.interfaces.nsIFileProtocolHandler)
.getURLSpecFromFile(file);
this.addCharsetListener(browser, function (charset, args) {
callback(charset, args);
Zotero.Browser.deleteHiddenBrowser(browser);
}, args);
browser.loadURI(url);
}
/*
* Attach a load listener to a browser object to perform charset detection
*
* We make sure the universal character set detector is set to the
* universal_charset_detector (temporarily changing it if not--shhhh)
*
* 'callback' is the function to pass the charset (and, if provided, 'args')
* to after detection is complete
*/
function addCharsetListener(browser, callback, args){
var prefService = Components.classes["@mozilla.org/preferences-service;1"]
.getService(Components.interfaces.nsIPrefBranch);
var oldPref = prefService.getCharPref('intl.charset.detector');
var newPref = 'universal_charset_detector';
//Zotero.debug("Default character detector is " + (oldPref ? oldPref : '(none)'));
if (oldPref != newPref){
//Zotero.debug('Setting character detector to universal_charset_detector');
prefService.setCharPref('intl.charset.detector', 'universal_charset_detector');
}
var onpageshow = function(){
// ignore spurious about:blank loads
if(browser.contentDocument.location.href == "about:blank") return;
browser.removeEventListener("pageshow", onpageshow, false);
var charset = browser.contentDocument.characterSet;
Zotero.debug("Detected character set '" + charset + "'");
//Zotero.debug('Resetting character detector to ' + (oldPref ? oldPref : '(none)'));
prefService.setCharPref('intl.charset.detector', oldPref);
callback(charset, args);
};
browser.addEventListener("pageshow", onpageshow, false);
}
this.checkFileAccessError = function (e, file, operation) { this.checkFileAccessError = function (e, file, operation) {
file = this.pathToFile(file); file = this.pathToFile(file);