Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

redbits/hd-spain VOSE titles mistmatch #11615

Closed
vinanrra opened this issue Apr 27, 2021 · 4 comments
Closed

redbits/hd-spain VOSE titles mistmatch #11615

vinanrra opened this issue Apr 27, 2021 · 4 comments

Comments

@vinanrra
Copy link
Contributor

Description

Titles with VOSE are mark as Spanish, when should be marked a English, because this tracker only have Spanish or English, same happens at hd-spain

Screenshots

imagen
imagen

Fix (Not working)

I have tried to add this at redbits.yml (didnt try anything at hd-spain.yml), but isn't working properly, check screenshot

imagen

    title:
      selector: a[href*="/torrents/"] a:contains("VOSE") # Select titles with VOSE
      optional: true
      filters:
        - name: re_replace
          args: ["\\[", " "]
        - name: re_replace
          args: ["\\]", " "]
        - name: re_replace
          args: ["(?i)bdfull", "BluRay"] # BDfull  -> BluRay
        - name: re_replace
          args: ["(?i)RedBits", ""] # Delete RedBits
        - name: append
          args: " English" # Add English to fix Sonarr/Radarr language
    title:
      selector: a[href*="/torrents/"] a:not(:contains("VOSE"))
      optional: true
@garfield69
Copy link
Contributor

try changing your selectors from a[href*="/torrents/"] a:contains("VOSE")
to
a[href*="/torrents/"]:contains("VOSE")
and
a[href*="/torrents/"] a:not(:contains("VOSE")) to
a[href*="/torrents/"]:not(:contains("VOSE"))

@vinanrra
Copy link
Contributor Author

try changing your selectors from a[href*="/torrents/"] a:contains("VOSE")
to
a[href*="/torrents/"]:contains("VOSE")
and
a[href*="/torrents/"] a:not(:contains("VOSE")) to
a[href*="/torrents/"]:not(:contains("VOSE"))

With that change i get this:
imagen

@garfield69
Copy link
Contributor

ok, then tomorrow I'll make time to properly explore and test a working PR.

@garfield69 garfield69 self-assigned this Apr 27, 2021
@garfield69 garfield69 added this to To do in Jackett Indexers Apr 27, 2021
garfield69 added a commit that referenced this issue Apr 27, 2021
@garfield69 garfield69 removed their assignment Apr 27, 2021
@garfield69 garfield69 moved this from To do to Done in Jackett Indexers Apr 27, 2021
@garfield69 garfield69 removed this from Done in Jackett Indexers Apr 27, 2021
@ilike2burnthing
Copy link
Contributor

v0.17.954

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants