“##+js(rmnt, #text, [...])” rule with non-ASCII UTF-8 characters is not converted

[DimitrisKalaboukas]

Issue Details

AdGuard can’t apply rules with “##+js(rmnt, #text, [...])” syntax.

Proposed solution

Add support for “##+js(rmnt, #text, [...])” rule syntax, as uBlock Origin already does.

Alternative solution

Convert such rules to “#%#//scriptlet(‘rmnt’, ‘#text’, ‘[...]’)” syntax and support it.

[Alex-302]

@DimitrisKalaboukas our syntax - https://github.com/AdguardTeam/Scriptlets/blob/master/wiki/about-scriptlets.md#remove-node-text

[sfionov]

@DimitrisKalaboukas This rule should be converted to #%#//scriptlet('ubo-rmnt', '#text', '[...]'), which is already supported. Can you please give an example of rule that does not convert?

[DimitrisKalaboukas]

@sfionov The conversion is done automatically or I have to make it manually?

[sfionov]

@DimitrisKalaboukas It is done automatically. And if corresponding alias exists, it will be applied. In case of uBO's rmnt
it exists.

[DimitrisKalaboukas]

@sfionov However, the rule:

gizchina.gr##+js(rmnt, #text, /^(\s*-----------\s*|Μπορεί να λάβουμε ένα μικρό ποσοστό εάν κάνετε κλικ σε έναν σύνδεσμο και αγοράσετε κάποιο προϊόν. Για περισσότερες λεπτομέρειες, | μπορείτε να μάθετε πώς χρησιμοποιούμε τους συνδέσμους συνεργατών. |Σας ευχαριστούμε για την υποστήριξη.|Ακολουθήστε το | για νέα και ειδήσεις στον χώρο της τεχνολογίας. Αν ψάχνετε HOT προσφορές και κουπόνια για κινητά και gadgets, κάντε εγγραφή στο | στο )$/)

isn’t applied by AdGuard for Android. I wonder if it is due to the rule’s length.

[sfionov]

@DimitrisKalaboukas Thanks for the example, it seems that rules with non-ascii symbols are not converted by some reason. I'll move this task into the CoreLibs repo.