Remove advanced setting ignoreScriptInjectFilters

gorhill · gorhill · commit f5b453fae3c7 · 2021-04-13T08:49:10.000-04:00
Ever since the `redirect` code was refactored: 157cef6 This advanced setting is no longer needed, as the same can be accomplished with a plain network filter: @@*$redirect-rule
diff --git a/src/js/background.js b/src/js/background.js
@@ -63,7 +63,6 @@ const µBlock = (( ) => { // jshint ignore:line
         extensionUpdateForceReload: false,
         filterAuthorMode: false,
         filterOnHeaders: false,
-        ignoreRedirectFilters: false,
         ignoreScriptInjectFilters: false,
         loggerPopupType: 'popup',
         manualUpdateAssetFetchPeriod: 500,
diff --git a/src/js/pagestore.js b/src/js/pagestore.js
@@ -821,7 +821,6 @@ const PageStore = class {
     }
 
     redirectBlockedRequest(fctxt) {
-        if ( µb.hiddenSettings.ignoreRedirectFilters === true ) { return; }
         const directives = µb.staticNetFilteringEngine.redirectRequest(fctxt);
         if ( directives === undefined ) { return; }
         if ( µb.logger.enabled !== true ) { return; }

Original file line number	Diff line number	Diff line change
`@@ -821,7 +821,6 @@ const PageStore = class {`
`821`	`821`	`}`
`822`	`822`
`823`	`823`	`redirectBlockedRequest(fctxt) {`
`824`		`- if ( µb.hiddenSettings.ignoreRedirectFilters === true ) { return; }`
`825`	`824`	`const directives = µb.staticNetFilteringEngine.redirectRequest(fctxt);`
`826`	`825`	`if ( directives === undefined ) { return; }`
`827`	`826`	`if ( µb.logger.enabled !== true ) { return; }`