Context menu blocking does not work anymore

[keldrin2]

as title suggests, context menu blocking is broken once again.

[keldrin2]

There seem to be a lot of open issues on the same thing right now. Not a single one describes the problem correctly. Namely 490 491 489 484 and maybe some others.

[luxidoptera]

Mhm, corroborating. BlockTube itself still works fine, but context menu blocking is broken.

[sclient]

Yup, just installed the extension today. Tried to block all kinds of videos, all of them continue to show up.

[keldrin2]

at the moment you have to edit the block list manually.

[sclient]

That is more annoying than seeing some videos repeatedly, so.... Oh well. Guess Ill just wait for an update, surely it wont be too long.

[keldrin2]

It is.

If it helps you, I made a little python script to get the channel ID from a channel link.

import requests
import sys
from bs4 import BeautifulSoup as bs

def main():
    if len(sys.argv) == 2:
        response = requests.get(sys.argv[1])
        if response.status_code == 200:
            soup = bs(response.content, "lxml")
            for link in soup.find_all("link"):
                if link.get("href") and link.get("href").find("channel_id=") > 0:
                    print(link["href"].split("=")[1])
                    exit(0)
    else:
        print(f"Usage: {sys.argv[0]} URL")


if __name__ == "__main__":
    main()

[RubenKelevra]

Blocking itself works fine, just the addition of new channels with their channel-ids does not.

You can still add the "title" of the channel under channel names and they will be blocked.

The converter site in the readme also will just throw 400 codes as response:

https://vabs.github.io/youtube-channel-name-converter/

[josephcrowell]

YouTube changed the dom structure again and the event sink variable can't be populated again.

[zaXsa]

I think the issue is the same as with 0.4.4:
Adding new eventSink location should work @amitbl

if (!eventSink) {
   eventSink = getObjectByPath(this.parentElement.parentElement, 'polymerController.forwarder_.eventSink');
}

found it using

    for (const [key1, value1] of Object.entries(this.parentElement.parentElement)) {
        if (value1 !== null && typeof(value1) === 'object') {
          for (const [key2, value2] of Object.entries(value1)) {
            if (value2 !== null && typeof(value2) === 'object') {
              for (const [key3, value3] of Object.entries(value2)) {       
                if (key3.includes('eventSink')) {
                  console.log('Found: ' + `${key1}:${key2}:${key3}`);
                }
              }
            }
    
            if (key2.includes('eventSink')) {
              console.log('Found: ' + `${key1}:${key2}`);
            }
          }
        }

        if (key1.includes('eventSink')) {
          console.log('Found: ' + `${key1}`);
        }
      }

Note as you can see looking at my code i usually don't work with javascript or any webdev but this seems to work.

Edit: fixed code blocks

[keldrin2]

fixed in v0.4.5

[RubenKelevra]

Thanks @keldrin2!

[keldrin2]

Thanks @keldrin2!

Wasn't me who fixed it. I just noticed it's fixed in the newest release and closed my bug report.