AdguardTeam / AdguardTeam/Scriptlets
Improve 'trusted-replace-argument' — add new constant values
- Dominant language
- JavaScript
- Stars
- 195
- Forks
- 33
- PR merge metrics
- No merged PRs in 30d
Description
Related to https://github.com/AdguardTeam/AdguardFilters/issues/232621
Steps to reproduce:
1. Add to user rules:
```adblock
example.org#%#//scriptlet('trusted-replace-argument', 'Promise', '0', 'noopPromiseResolve', 'isDetected')
```
2. Navigate to https://example.org/
3. In browser console run:
```js
(async () => {
const checkAdblock = () => {
console.log('Checking for adblock...');
return new Promise((resolve) => {
// Set to true by default for demonstration purposes
const isDetected = true;
resolve(isDetected);
});
};
const init = async () => {
const [result] = await Promise.all([
checkAdblock(),
]);
if (result) {
console.log('Adblock detected');
return;
}
console.log('Load content');
}
await init();
console.log('Script loaded');
})();
```
With current solution only `Checking for adblock...` is printed in console (the same for`noopCallbackFunc`, `noopFunc`, `falseFunc`).
Expected is:
```
Checking for adblock...
Load content
Script loaded
```
So, I think we could add functions which return callback with `false`/`true`, something like:
```js
const falseCallbackFunc = (callback) => { callback(false); };
const trueCallbackFunc = (callback) => { callback(true); };
```
And/Or we could add support for `valueWrapper` similar like in https://github.com/AdguardTeam/Scriptlets/blob/master/wiki/about-scriptlets.md#set-constant
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.