Rename canStartExtension to more descriptive isEnabledForSite

This commit is contained in:
Tamius Han 2022-09-28 00:39:20 +02:00
parent a2f0617d5a
commit d70783cc82

View File

@ -422,15 +422,15 @@ class Settings {
}
}
canStartExtension(site) {
// returns 'true' if extension can be started on a given site. Returns false if we shouldn't run.
/**
* Returns whether extension can start on a given site or not.
* @param site default value fof this argument is window.location.hostname
* @returns true if extension can run on this site, false otherwise
*/
isEnabledForSite(site = window.location.hostname) {
if (!site) {
site = window.location.hostname;
if (!site) {
this.logger?.log('info', 'settings', `[Settings::canStartExtension] window.location.hostname is null or undefined: ${window.location.hostname} \nactive settings:`, this.active);
return false;
}
this.logger?.log('info', 'settings', `[Settings::canStartExtension] window.location.hostname is null or undefined: ${window.location.hostname} \nactive settings:`, this.active);
return false;
}
// if (Debug.debug) {
@ -486,10 +486,6 @@ class Settings {
return this.active.sites['@global'].mode !== ExtensionMode.Disabled
}
extensionEnabledForSite(site) {
return this.canStartExtension(site);
}
canStartAutoAr(site?: string) {
// 'site' argument is only ever used when calling this function recursively for debugging
if (!site) {