[fixed] IMDB Info loads posters from previous file (1 Viewer)

ShIvADeSt

Portal Pro
May 30, 2009
71
2
MediaPortal Version: 1.2.0
MediaPortal Skin: DefaultWide

Steps to reproduce:
1. Show IMDB info for a film (Crank for ex.), MP starts loading posters for it for a choice.
2. When no posters are loaded (if many posters for a film or slow connection) exit from IMDB info.
3. Show IMDB info for other film (Crank 2 for ex). MP continue loading posters and at the end of loading we can switch posters... for Crank, not Crank 2. But after bit of time MP downloads posters for Crank 2 too.

I think it happens coz MP didnt close stream for loading posters and they continue downloading.

If need I'll try to attach logs.
 

ShIvADeSt

Portal Pro
May 30, 2009
71
2
Logs are attached as I promised, and I try to describe what I did.
1. I select video 127 hours and press F3 to show IMDB info.
2. Wait 1-2 seconds and close this info.
3. Open Harry Potter and deathly hollows.
4. MP donwload posters for 127 hours insted of harry potter.
Very interesting - in log exists string
2011-05-03 21:17:28.539873 [Info.][PoolThread12(12)]: Error retreiving WebPage: http://api.themoviedb.org/2.1/Movie.getImages/en/xml/2ed40b5d82aa804a2b1fcedb5ca8d97a/tt1542344/ Encoding:utf-8 err:Удаленный сервер возвратил ошибку: (404) Не найден. stack: в System.Net.HttpWebRequest.GetResponse()
в MediaPortal.Util.TMDBCoverSearch.GetPage(String strURL, String strEncode)
this is posters for 127 HOURS NOT Harry potter
After it I try again, but I show IMDB info for harry potter, thn after 1 second I close it and show IMDB info for 127 hours. And what I get - look title and see poster for it.
attachment.php
 

Attachments

  • 127hours.jpg
    127hours.jpg
    223.5 KB

Deda

Lead Dev MP1 Videos
  • Premium Supporter
  • March 18, 2009
    2,423
    2,385
    Zagreb
    Home Country
    Croatia Croatia
    Can you replicate that with disabled option "Enable file existance cache" ->Configuration-GUI.
     

    ShIvADeSt

    Portal Pro
    May 30, 2009
    71
    2
    when I disable this option what is happend.
    I know that MP downloads 29 pictures for Harry Potter and 13 for 127 hours.
    1. I press IMDB info for Harry Potter and close it immediatly.
    2. I press IMDB info for 127 Hours. On info screen appeared caption 29 picturies (like Harry Potter), then after some seconds it changed to 13 picturies (Like 127 Hours).
    But with disabled option poster don't change immediatly to Harry Potter but still remains 127 Hours. When option is enabled poster replaced on wrong.
     

    Deda

    Lead Dev MP1 Videos
  • Premium Supporter
  • March 18, 2009
    2,423
    2,385
    Zagreb
    Home Country
    Croatia Croatia
    Didn't have a time ti investigate this yesterday, probably it's something within MP cache or cover download thread in thread pool is not managed correctly on closing window. Could be something else but this will take a little time.
     

    Seidelin

    Retired Team Member
  • Premium Supporter
  • August 14, 2006
    1,755
    652
    Kgs. Lyngby
    Home Country
    Denmark Denmark
    Didn't have a time ti investigate this yesterday, probably it's something within MP cache or cover download thread in thread pool is not managed correctly on closing window. Could be something else but this will take a little time.

    So this is a confirmed bug? Then please add to mantis, and mark this thread as bug. :D
     

    Deda

    Lead Dev MP1 Videos
  • Premium Supporter
  • March 18, 2009
    2,423
    2,385
    Zagreb
    Home Country
    Croatia Croatia
    Can't say that it is a bug because I didn't manage to replicate that, probably it is but I can't do a cover switch for now or my dev laptop is too slow in GUI so it can finish all background processes while changing movie infos.
     

    ShIvADeSt

    Portal Pro
    May 30, 2009
    71
    2
    may be your Internet too fast, that posters are downloaded before info switch ^^
    Try to get info for film with huge numbers of potsters - 20 and more and switch to other film.
     

    Deda

    Lead Dev MP1 Videos
  • Premium Supporter
  • March 18, 2009
    2,423
    2,385
    Zagreb
    Home Country
    Croatia Croatia
    Can you try (on both cache cases) with this file (just backup original and replace it in Plugins\Windows)
     

    ShIvADeSt

    Portal Pro
    May 30, 2009
    71
    2
    Now with updated plugin it works perfect in both cases. No posters from other video. When I roll back to old plugin - again wrong posters. So with new plugin is no problem.
    Thanks for a patch. May be mark topic as a bug?
     

    Users who are viewing this thread

    Top Bottom