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

Bugfix :: Metatdata Season Thumbnail : Fixes bug that attempted retrieval of non-existent season 0 (Special) and skipped last season when a show lacks a season 0. #610

Closed

Conversation

Conjuro
Copy link
Contributor

@Conjuro Conjuro commented Feb 28, 2013

This bug was discovered by luxmoggy; see forum post.

Some show are missing thumbnails for the last season. This is because when retrieving the thumbnails a list of seasons is created using number of seasons and a range(0..n) (e.g. [0, 1, 2, 3, 4, 5]. When a show lacks a season 0, it will still create the list using the same range starting point; Thereby creating a list containing a non-existent season 0 and missing the last season ([0, 1, 2, 3, 4]).

This patch drops creating a new list in favor of using the already existing list.

@thezoggy
Copy link
Contributor

thezoggy commented Mar 5, 2013

this bug was already previously found, i do believe @itofzo already fixed it in his fork, may want to review his code:
https://github.com/itofzo/Sick-Beard/commits/all_fixes_dev/sickbeard/metadata/generic.py

@Conjuro
Copy link
Contributor Author

Conjuro commented Mar 5, 2013

Thanks @thezoggy. Reviewed his code and it does indeed fix the bug while also preventing excess downloads.

Also found that my patch duplicates pull request #386 by @hmflash. Though I believe mine to be a better solution.

In my opinion, pull #409 from @itofzo and close this and #386. Otherwise pull this one and close #386.

Leaving this pull open until #409 or other gets pulled.

…eval of non-existent season 0 (Special) and skipped last season when a show lacks a season 0.
@Conjuro
Copy link
Contributor Author

Conjuro commented Jun 7, 2013

Rebased for current development branch. Still recommend PR 409 from itofzo.

@thezoggy
Copy link
Contributor

if you recommend itofzo's pr.. why this one? have you compared the two ?

@Conjuro
Copy link
Contributor Author

Conjuro commented Jun 12, 2013

Yes, I reviewed his code when you first pointed out that his patch existed and I found it to be more comprehensive. I left this PR open in case #409 didn't get pulled for whatever reason. Now that it's been merged I'm closing this PR.

@Conjuro Conjuro closed this Jun 12, 2013
@Conjuro Conjuro deleted the Bugfix-Season_Thumbnails branch June 12, 2013 07:34
@thezoggy
Copy link
Contributor

ty

gborri pushed a commit to gborri/Sick-Beard that referenced this pull request Jul 16, 2014
…indPropers

Port:Fix omgwtfnzbs findPropers.
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.

2 participants