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

IMDBTV Episdoe thumbnail wont scrape on Custom TV Scraper if no image #1725

Closed
3 tasks
kolbdog323 opened this issue Mar 3, 2024 · 4 comments · Fixed by #1736
Closed
3 tasks

IMDBTV Episdoe thumbnail wont scrape on Custom TV Scraper if no image #1725

kolbdog323 opened this issue Mar 3, 2024 · 4 comments · Fixed by #1736
Assignees
Milestone

Comments

@kolbdog323
Copy link

Scrapers that don't work:

  • Custom TV Scraper/IMDBTV Episode Thumbnail

Details which are wrong / not loaded:
When you scrape a episode using the Custom Tv Scraper and have iIMDBtv selected for epiosde thumbnail the image wont scrape if there is no image to replace if there is one on TMDB as a subsitute. But if there is an image it will work. The imdbtv scraper will grabe the episode thumbnail but on custom if no image on tmdb it wont grab it.

MediaElch Version:

  • 2.10.6 (stable)

Operating System:

  • Windows

Additional context:
Add any other context about the problem here.

@kolbdog323
Copy link
Author

An example is if episode 6 doesnt have a thumbnail for the episdoe on tmdb the custom scraper wont scrape an image from imdb for the same episdoe you have to use the imdbtv scraper first then custom scraper to keep the image. Its very annoying.

@bugwelle
Copy link
Collaborator

bugwelle commented Mar 7, 2024

Thanks for reporting! I will have a look soon.

@kolbdog323
Copy link
Author

Any luck on this as well. Also I will try the nightly build tomorrow to try out the missing overview and see if it's fixed.

@bugwelle
Copy link
Collaborator

bugwelle commented Apr 7, 2024

Thank you very much for reporting this issue. I've found few other bugs (one crash) that I've now also fixed.

Your bug was a copy&paste error. I looked at the TV show settings when loading episodes. The fix is trivial:

--- a/src/scrapers/tv_show/custom/CustomEpisodeScrapeJob.cpp
+++ b/src/scrapers/tv_show/custom/CustomEpisodeScrapeJob.cpp
@@ -46,7 +46,7 @@ void CustomEpisodeScrapeJob::onTmdbLoaded(EpisodeScrapeJob* job)
     copyDetailsToEpisode(episode(), job->episode(), job->config().details);
     job->deleteLater();
 
-    const QStringList scrapersToUse = m_customConfig.scraperForShowDetails.values();
+    const QStringList scrapersToUse = m_customConfig.scraperForEpisodeDetails.values();
     const bool loadImdb = episode().imdbId().isValid() && scrapersToUse.contains(ImdbTv::ID);
 
     m_loadCounter = 1;

It will be fixed shortly. Note that there are currently issues with our Nightly-build pipeline for Windows. It may take another day for a Nightly to appear.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants