mrkipling / maraschino

A front-end for HTPCs
MIT License
424 stars 125 forks source link

trakt image load issues #383

Open mboeru opened 9 years ago

mboeru commented 9 years ago

Since the recent trakt updates the trakt widget seems to have an issue loading the images. Looking at the log files I see the following:

Jan-06 10:31:52 :: ERROR    :: Failed to download https://walter.trakt.us/images/shows/000/001/409/posters/original/c336773ac7-138.jpg?1420531398
Jan-06 10:31:52 :: ERROR    :: Failed to download https://walter.trakt.us/images/shows/000/060/291/posters/original/88149f56e7-138.jpg?1420531230
Jan-06 10:31:52 :: ERROR    :: Failed to download https://walter.trakt.us/images/shows/000/039/192/posters/original/5fe49a60b2-138.jpg?1420532751

Also when clicking on friends or calendar the following error occurs:

Jan-06 10:34:56 :: INFO     :: TRAKT :: Fetching friends list
Jan-06 10:35:00 :: DEBUG    :: TRAKT :: EXCEPTION -- HTTP Error 404: Not Found
Jan-06 10:35:02 :: INFO     :: TRAKT :: Fetching my shows calendar
Jan-06 10:35:02 :: DEBUG    :: TRAKT :: EXCEPTION -- HTTP Error 404: Not Found

I am using latest version:

[root@cubox maraschino]# git pull origin master
From https://github.com/mrkipling/maraschino
 * branch            master     -> FETCH_HEAD
Already up-to-date.

screen shot 2015-01-06 at 10 31 22

Cheers, mboeru

ebright commented 9 years ago

It looks like Trakt has upgraded their API calls. This is going to require a code change in order to support the new API

ebright commented 9 years ago

Trakt still supports the old API calls for now but change how they named their posters. You can fix this by removing the removing the small_poster function in the traktplus.py file https://github.com/ebright/maraschino/commit/de932d061b56952e19d3915bd5ccff05faafe3fb

mboeru commented 9 years ago

Thank You @ebright, commenting that line worked for me.

Guruleenyc commented 9 years ago

Can someone help me identify exactly which lines of code in the 'traktplus.py' file that need to be modified to fix this, or is their a alternative fix so images show?

mboeru commented 9 years ago

if you are using mrkipling's version then I think it line 43 https://github.com/mrkipling/maraschino/blob/master/modules/traktplus.py#L43