bttv-android / bttv

A mod of the Twitch Android Mobile App adding BetterTTV, FrankerFaceZ and 7TV emotes
MIT License
644 stars 21 forks source link

Can't blacklist twitch emotes #495

Open moonMan8 opened 2 years ago

moonMan8 commented 2 years ago

Existing Issues

Unmodded Twitch

What happened?

When I input a word with capital letters it tranforms the capital letters to small so the word as example: Kappa becomes kappa in the list which makes the emote not being blacklisted I hope you can fix it and thank you for your hard work

bttv-android version

30.1

android version

No response

device

No response

Anything else?

No response

FoseFx commented 2 years ago

Thank you for filing this issue!

The blacklist (and the highlight list) is in fact case insensitive (so "WOW", "WoW" and "wow" are all the same), but it's not the reason why "Kappa" can't be blacklisted.

The tokenizer (which also identifies blacklisted words in messages) only operates on the bits of a message that are identified as normal text by the twitch tokenizer. As Kappa already is a Twitch Emote it will not be tested for.

I'll keep this open, but implementing this might be a bit ugly and won't have a high priority right now.