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

tvdb may return multiple series #96

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

tvdb may return multiple series #96

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Mar 19, 2020

In cases where TheTVDB.com returns multiple entries and the series we
are looking for is not the first one, the series would not be found,
because only the first one would be checked.

In my case, I am looking for 'The Good Place' and TheTVDB.com returns
the following structure (in XML equivalent).

- Series
  - SeriesName: The Good Place: The Podcast
- Series
  - SeriesName: The Good Place

I am looking for the second one, but the loop stops at the first entry
when raising the ShowNotFoundException.

Solve this problem by raising the exception only when all series are
checked.

In cases where TheTVDB.com returns multiple entries *and* the series we
are looking for is not the first one, the series would not be found,
because only the first one would be checked.

In my case, I am looking for 'The Good Place' and TheTVDB.com returns
the following structure (in XML equivalent).

    - Series
      - SeriesName: The Good Place: The Podcast
    - Series
      - SeriesName: The Good Place

I am looking for the second one, but the loop stops at the first entry
when raising the ShowNotFoundException.

Solve this problem by raising the exception only when all series are
checked.
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.

0 participants