move getSetting from background.js -> helpers.js

it will have use in `detailsInterface.js`, not just `background.js`
(next patch).
This commit is contained in:
2022-11-05 21:30:49 -07:00
parent 0fd15fddf8
commit e342264adf
2 changed files with 22 additions and 21 deletions

View File

@@ -450,7 +450,7 @@ async function fillFields(settings, login, fields) {
// build focus or submit request // build focus or submit request
let focusOrSubmitRequest = { let focusOrSubmitRequest = {
origin: new BrowserpassURL(settings.tab.url).origin, origin: new BrowserpassURL(settings.tab.url).origin,
autoSubmit: getSetting("autoSubmit", login, settings), autoSubmit: helpers.getSetting("autoSubmit", login, settings),
filledFields: filledFields, filledFields: filledFields,
}; };
@@ -561,25 +561,6 @@ async function getFullSettings() {
return settings; return settings;
} }
/**
* Get most relevant setting value
*
* @param string key Setting key
* @param object login Login object
* @param object settings Settings object
* @return object Setting value
*/
function getSetting(key, login, settings) {
if (typeof login.settings[key] !== "undefined") {
return login.settings[key];
}
if (typeof settings.stores[login.store.id].settings[key] !== "undefined") {
return settings.stores[login.store.id].settings[key];
}
return settings[key];
}
/** /**
* Deep copy an object * Deep copy an object
* *
@@ -968,7 +949,7 @@ async function parseFields(settings, login) {
if (key === "secret" && lines.length) { if (key === "secret" && lines.length) {
login.fields.secret = lines[0]; login.fields.secret = lines[0];
} else if (key === "login") { } else if (key === "login") {
const defaultUsername = getSetting("username", login, settings); const defaultUsername = helpers.getSetting("username", login, settings);
login.fields[key] = defaultUsername || login.login.match(/([^\/]+)$/)[1]; login.fields[key] = defaultUsername || login.login.match(/([^\/]+)$/)[1];
} else { } else {
delete login.fields[key]; delete login.fields[key];

View File

@@ -11,12 +11,32 @@ const BrowserpassURL = require("@browserpass/url");
module.exports = { module.exports = {
prepareLogins, prepareLogins,
filterSortLogins, filterSortLogins,
getSetting,
ignoreFiles, ignoreFiles,
makeTOTP, makeTOTP,
}; };
//----------------------------------- Function definitions ----------------------------------// //----------------------------------- Function definitions ----------------------------------//
/**
* Get most relevant setting value
*
* @param string key Setting key
* @param object login Login object
* @param object settings Settings object
* @return object Setting value
*/
function getSetting(key, login, settings) {
if (typeof login.settings[key] !== "undefined") {
return login.settings[key];
}
if (typeof settings.stores[login.store.id].settings[key] !== "undefined") {
return settings.stores[login.store.id].settings[key];
}
return settings[key];
}
/** /**
* Get the deepest available domain component of a path * Get the deepest available domain component of a path
* *