github/Ombi
1
0
Fork 0
mirror of https://github.com/Ombi-app/Ombi.git synced 2025-08-19 04:49:33 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fixed a bug with the RefreshMetadata where we would never get TheMovieDBId's if it was missing it

Browse source
This commit is contained in:
Jamie 2018-04-13 22:38:38 +01:00
commit a681932d2e
1 changed files with 23 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

30
src/Ombi.Schedule/Jobs/Ombi/RefreshMetadata.cs
View file

@ -80,7 +80,7 @@ namespace Ombi.Schedule.Jobs.Ombi
if (!hasTheMovieDb) if (!hasTheMovieDb)
{ {
var id = await GetTheMovieDbId(hasTvDbId, hasImdb, show.TvDbId, show.ImdbId, show.Title); var id = await GetTheMovieDbId(hasTvDbId, hasImdb, show.TvDbId, show.ImdbId, show.Title, false);
show.TheMovieDbId = id; show.TheMovieDbId = id;
} }
@ -120,7 +120,7 @@ namespace Ombi.Schedule.Jobs.Ombi
if (!hasTheMovieDb) if (!hasTheMovieDb)
{ {
var id = await GetTheMovieDbId(hasTvDbId, hasImdb, show.TvDbId, show.ImdbId, show.Title); var id = await GetTheMovieDbId(hasTvDbId, hasImdb, show.TvDbId, show.ImdbId, show.Title, false);
show.TheMovieDbId = id; show.TheMovieDbId = id;
} }
@ -166,7 +166,7 @@ namespace Ombi.Schedule.Jobs.Ombi
} }
if (!hasTheMovieDb) if (!hasTheMovieDb)
{ {
var id = await GetTheMovieDbId(false, hasImdb, string.Empty, movie.ImdbId, movie.Title); var id = await GetTheMovieDbId(false, hasImdb, string.Empty, movie.ImdbId, movie.Title, true);
movie.TheMovieDbId = id; movie.TheMovieDbId = id;
_plexRepo.UpdateWithoutSave(movie); _plexRepo.UpdateWithoutSave(movie);
} }
@ -200,7 +200,7 @@ namespace Ombi.Schedule.Jobs.Ombi
} }
if (!hasTheMovieDb) if (!hasTheMovieDb)
{ {
var id = await GetTheMovieDbId(false, hasImdb, string.Empty, movie.ImdbId, movie.Title); var id = await GetTheMovieDbId(false, hasImdb, string.Empty, movie.ImdbId, movie.Title, true);
movie.TheMovieDbId = id; movie.TheMovieDbId = id;
_embyRepo.UpdateWithoutSave(movie); _embyRepo.UpdateWithoutSave(movie);
} }
@ -215,7 +215,7 @@ namespace Ombi.Schedule.Jobs.Ombi
await _embyRepo.SaveChangesAsync(); await _embyRepo.SaveChangesAsync();
} }
private async Task<string> GetTheMovieDbId(bool hasTvDbId, bool hasImdb, string tvdbID, string imdbId, string title) private async Task<string> GetTheMovieDbId(bool hasTvDbId, bool hasImdb, string tvdbID, string imdbId, string title, bool movie)
{ {
_log.LogInformation("The Media item {0} does not have a TheMovieDbId, searching for TheMovieDbId", title); _log.LogInformation("The Media item {0} does not have a TheMovieDbId, searching for TheMovieDbId", title);
FindResult result = null; FindResult result = null;
@ -230,13 +230,29 @@ namespace Ombi.Schedule.Jobs.Ombi
if (hasImdb && !hasResult) if (hasImdb && !hasResult)
{ {
result = await _movieApi.Find(imdbId, ExternalSource.imdb_id); result = await _movieApi.Find(imdbId, ExternalSource.imdb_id);
hasResult = result?.tv_results?.Length > 0; if (movie)
{
hasResult = result?.movie_results?.Length > 0;
}
else
{
hasResult = result?.tv_results?.Length > 0;
}
_log.LogInformation("Setting Show {0} because we have ImdbId, result: {1}", title, hasResult); _log.LogInformation("Setting Show {0} because we have ImdbId, result: {1}", title, hasResult);
} }
if (hasResult) if (hasResult)
{ {
return result.tv_results?[0]?.id.ToString() ?? string.Empty; if (movie)
{
return result.movie_results?[0]?.id.ToString() ?? string.Empty;
}
else
{
return result.tv_results?[0]?.id.ToString() ?? string.Empty;
}
} }
return string.Empty; return string.Empty;
} }