fix(email-notifications): 🐛 Fixed the issue where legacy requests were showing broken poster images #4452

pull/4470/head
tidusjar 3 years ago
parent 9fe1f8e988
commit 0ece2fd6e0

@ -43,7 +43,17 @@ namespace Ombi.Notifications
var img = req?.PosterPath ?? string.Empty; var img = req?.PosterPath ?? string.Empty;
if (img.HasValue()) if (img.HasValue())
{ {
PosterImage = $"https://image.tmdb.org/t/p/w300/{req?.PosterPath?.TrimStart('/') ?? string.Empty}"; if (img.StartsWith("http"))
{
// This means it's a legacy request from when we used TvMaze as a provider.
// The poster url is the fully qualified address, so just use it
PosterImage = img;
}
else
{
PosterImage =
$"https://image.tmdb.org/t/p/w300/{img?.TrimStart('/') ?? string.Empty}";
}
} }
CalculateRequestStatus(req); CalculateRequestStatus(req);
} }
@ -61,8 +71,17 @@ namespace Ombi.Notifications
var img = req?.ParentRequest?.PosterPath ?? string.Empty; var img = req?.ParentRequest?.PosterPath ?? string.Empty;
if (img.HasValue()) if (img.HasValue())
{ {
PosterImage = if (img.StartsWith("http"))
$"https://image.tmdb.org/t/p/w300/{req?.ParentRequest?.PosterPath?.TrimStart('/') ?? string.Empty}"; {
// This means it's a legacy request from when we used TvMaze as a provider.
// The poster url is the fully qualified address, so just use it
PosterImage = img;
}
else
{
PosterImage =
$"https://image.tmdb.org/t/p/w300/{img?.TrimStart('/') ?? string.Empty}";
}
} }
// Generate episode list. // Generate episode list.

Loading…
Cancel
Save