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

GimmePeers Blah Query Value Update #338

Merged
merged 3 commits into from
May 16, 2023
Merged

Conversation

IncognitoCoding
Copy link
Contributor

Indexer/Tracker

https://github.com/Prowlarr/Indexers/blob/master/definitions/v7/gimmepeers.yml

Description

GP recently released an updated IMDb ID search option for queries. This change was added because queries against "blah=3" were turning up inconsistent results because not all "title" or "NFO" contained the IMDb. The default "blah" value in the gimmepeers.yaml file is 3, but it should now be changed to 4.

Issues Fixed or Closed by this PR

Revised "blah" value for better matches.

  • Fixes #XXXX

GP recently released an updated IMDB ID search option for queries. This change was added because queries against "blah=3" were turning up inconsistent results because not all "title" or "NFO" contained the IMDb. The default "blah" value in the gimmepeers.yaml file is 3, but it should now be changed to 4.
@@ -96,8 +96,8 @@ search:
inputs:
$raw: "{{ range .Categories }}c{{.}}=1&{{end}}"
search: "{{ if .Query.IMDBID }}{{ .Query.IMDBID }}{{ else }}{{ .Keywords }}{{ end }}{{ if .Query.Genre }} genre:{{ .Query.Genre }}{{ else }}{{ end }}"
# 0 title, 1 nfo, 2 filelist, 3 title+nfo
blah: "{{ if or .Query.IMDBID .Query.Genre }}3{{ else }}0{{ end }}"
# 0 title, 1 nfo, 2 filelist, 3 title+nfo, 4 IMDB (tt#)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this imdbid only or also for genre?
If it's not for both then the condition will need to be changed

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point! I overlooked the Genre. This change is only for the IMDb id. I haven't had to dig into these indexers before, but I requested GP to make this change for IMDb, so I figured I could take the extra step to request the indexer to be updated.

Would this line do the trick?

blah: "{{ if .Query.IMDBID }}4{{ else }}0{{ end }}{{ if .Query.Genre }}3{{ else }}0{{ end }}"

ilike2burnthing added a commit to Jackett/Jackett that referenced this pull request May 16, 2023
Revised the "blah" line to only set 4 when the IMDb id is queried, but continue to use "blah=3" for Genre.
@ilike2burnthing
Copy link
Contributor

Jackett/Jackett@927ffdf

Revised to match the Jackett update for the same "blah" query.

Jackett/Jackett@927ffdf
@IncognitoCoding
Copy link
Contributor Author

Jackett/Jackett@927ffdf

Good stuff! I updated my PR to match the one submitted with Jackett. I had also forked the Jackett repo and planned on updating that one after I verified this one was good. We both were trying to get this updated quickly.

@Qstick Qstick merged commit 2c72da8 into Prowlarr:master May 16, 2023
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

Successfully merging this pull request may close these issues.

None yet

4 participants