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

Created new Metadata provider for TheMovieDB: MovieDbEpisodeProvider #1515

Merged
merged 2 commits into from Mar 3, 2016

Conversation

softworkz
Copy link
Contributor

Most of the required logic already existed in MovieDbEpisodeImageProvider.
In order to avoid duplicated code, I have moved all common code to a new base class "MovieDbProviderBase" from which MovieDbEpisodeProvider and
MovieDbEpisodeImageProvider are inheriting now.

Most of the required logic already existed in
MovieDbEpisodeImageProvider.
In order to avoid duplicated code, I have moved all common code to a new
base class "MovieDbProviderBase" from which MovieDbEpisodeProvider and
MovieDbEpisodeImageProvider are inheriting now.
@softworkz
Copy link
Contributor Author

I used the unspecific name "MovieDbProviderBase" for the base class, because it could also serve as a base for MovieDbSeasonProvider, MovieDbSeriesProvider and MovieDbSeriesImageProvider in the future. There is still a lot of duplicate code.

Ensures that the new provider is disabled by default
@softworkz
Copy link
Contributor Author

I just added the same kind of migration task as I did for the OmdbEpisodeProvider.

Regarding query performance I would like to note that the new provider is less critical than the OmdbEpisodeProvider because it caches its results to disk (same like TvdbEpisodeProvider).

LukePulverenti added a commit that referenced this pull request Mar 3, 2016
Created new Metadata provider for TheMovieDB: MovieDbEpisodeProvider
@LukePulverenti LukePulverenti merged commit 5a0d821 into MediaBrowser:dev Mar 3, 2016
@softworkz softworkz deleted the MovieDbEpisodeProvider branch March 3, 2016 18:57
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

2 participants