Kodi Community Forum

Full Version: skin helper service
You're currently viewing a stripped down version of our content. View the full version with proper formatting.
Hi Marcel,

I noticed that rotten tomatoes ratings were no longer working with the latest skin helper code so looked into your changes. Basically the change you made prevents it from working when your browsing your movie library.

This code here:

Code:
if not imdbnumber:
            dbId = xbmc.getInfoLabel("ListItem.IMDBNumber")

Needs to be changed to

Code:
if not imdbnumber:
            imdbnumber = xbmc.getInfoLabel("ListItem.IMDBNumber")
(2015-12-01, 22:08)im85288 Wrote: [ -> ]I noticed that rotten tomatoes ratings were no longer working with the latest skin helper code so looked into your changes. Basically the change you made prevents it from working when your browsing your movie library.

Oops, thanks for the catch. Now fixed on Git.
(2015-12-01, 20:33)marcelveldt Wrote: [ -> ]
(2015-11-30, 22:14)Angelinas Wrote: [ -> ]Script working and showing Images but have error in log.Can you fixed this error

Should now be fixed on Git, please test.
My kodi.log is now without error...I already missed that info Smile
just kiding, thanks....good job
Code:
18:01:30 T:3324   ERROR: CCurlFile::Open failed with code 404 for http://localhost:52307/getthumb&title=Beck/
18:01:30 T:3324   ERROR: XFILE::CHTTPDirectory::GetDirectory - Unable to get http directory (http://localhost:52307/getthumb&title=Beck/)
18:01:30 T:3324   ERROR: XFILE::CDirectory::GetDirectory - Error getting http://localhost:52307/getthumb&title=Beck/

You or kodi adding '/' to end of string supplied. I dont supply the '/' ? I suppply Listitem.Artist which definitely doesn't have a '/'.
Unless needed, but then I get no images?

Thiis with old version of helper too [6 days old] same error, is it because at songs level? Is with old beta 2 kodi too. Odd

Code:
22:28:14 T:6440   ERROR: CCurlFile::Open failed with code 404 for http://localhost:52307/getthumb&title=James Curd/
22:28:14 T:6440   ERROR: XFILE::CHTTPDirectory::GetDirectory - Unable to get http directory (http://localhost:52307/getthumb&title=James Curd/)
22:28:14 T:6440   ERROR: XFILE::CDirectory::GetDirectory - Error getting http://localhost:52307/getthumb&title=James Curd/
22:28:15 T:7684   ERROR: CCurlFile::Open failed with code 404 for http://localhost:52307/getthumb&title=John Legend/
22:28:15 T:7684   ERROR: XFILE::CHTTPDirectory::GetDirectory - Unable to get http directory (http://localhost:52307/getthumb&title=John Legend/)
22:28:15 T:7684   ERROR: XFILE::CDirectory::GetDirectory - Error getting http://localhost:52307/getthumb&title=John Legend/
(2015-11-25, 23:22)Dumyat Wrote: [ -> ]Hi Marcel,
I was curious if it might be possible to add the classification rating for movies shown on live Tv, based on the movie classification given on the IMDB site? Also, since IMDB shows this data in text format, would be possible to show the classification as an image like in the local movie library?
Cheers

Hi Marcel,
Not sure if you missed this one or it's not doable, or no interest in adding?
(2015-12-01, 20:51)marcelveldt Wrote: [ -> ]
(2015-11-29, 21:35)cicero22 Wrote: [ -> ]I observed that the script always wakes up my homeserver and tries to load a banner for an episode although I do not use those banners anywhere. This happens during the execution of the "Next Episodes" script which runs very slow on my system (Windows 8.1, Kodi 16 Beta 2, 61 TV Shows with ca. 2000 episodes, MySQL on Rasperry Pi). It takes up to 10 seconds before the widget Displays the next episodes.
Is there anything I can do? I do not want the homeserver to wake up every time I open the widget for the next episodes...

I looked at the code and this script is not doing that but the skin is.
A good widget implementation should have all widgets loaded in the background (with visibility) conditions.
This will prevent the widget from reloading every time and it will display instantly. Only on startup you'll have that 10 seconds loading time.
I think that will also fix your homeserver waking up all the time.
Please contact the skin developer because he/she can be at more help with this.

OK, will do. Thanks a lot!
(2015-12-02, 21:25)Dumyat Wrote: [ -> ]
(2015-11-25, 23:22)Dumyat Wrote: [ -> ]Hi Marcel,
I was curious if it might be possible to add the classification rating for movies shown on live Tv, based on the movie classification given on the IMDB site? Also, since IMDB shows this data in text format, would be possible to show the classification as an image like in the local movie library?
Cheers

Hi Marcel,
Not sure if you missed this one or it's not doable, or no interest in adding?
Sounds like a good idea when needed. I would say that it makes then much more sens to scrape "all" found datas like any scraper (since it already gets the arts). Could be a feature request ? Wink ... but maybe after bugs tracking and a lil break Wink
(2015-12-02, 21:35)Jayz2K Wrote: [ -> ]
(2015-12-02, 21:25)Dumyat Wrote: [ -> ]
(2015-11-25, 23:22)Dumyat Wrote: [ -> ]Hi Marcel,
I was curious if it might be possible to add the classification rating for movies shown on live Tv, based on the movie classification given on the IMDB site? Also, since IMDB shows this data in text format, would be possible to show the classification as an image like in the local movie library?
Cheers

Hi Marcel,
Not sure if you missed this one or it's not doable, or no interest in adding?
Sounds like a good idea when needed. I would say that it makes then much more sens to scrape "all" found datas like any scraper (since it already gets the arts). Could be a feature request ? Wink ... but maybe after bugs tracking and a lil break Wink
Well yeah, I must admit I'm a bit of sucker for wanting to see the classification of the movies I watch Smile But I guess I was also thinking from the parental guidance aspect too, if we able to catch this data for movies shown on live tv.....
Not really to interested in classifications for TV shows, mainly because they don't seem to exist in the UK, but I could be wrong so don't quote me on that Smile
Yer I mentioned it earlier too, to add mpaa data. He let me know that I can add any other info like mppa in the pvrdetails.xml. so for a few shows I have added the mpaa rating in pvrdetails and it now shows the flags in recorded tv.

If you are going ahead with this feature now marcel please add the prefix option so users can set there country's prefix in front of the classification eg. Australia:MA

Here is the post from marcel reply

Skin.Helper.Service additional info in pvrdetails
(2015-12-03, 02:05)Rjsachse Wrote: [ -> ]Yer I mentioned it earlier too, to add mpaa data. He let me know that I can add any other info like mppa in the pvrdetails.xml. so for a few shows I have added the mpaa rating in pvrdetails and it now shows the flags in recorded tv.

If you are going ahead with this feature now marcel please add the prefix option so users can set there country's prefix in front of the classification eg. AustraliaTongueG

Here is the post from marcel reply

Skin.Helper.Service additional info in pvrdetails
Thanks for the info and link Smile
I just tried on a couple of sample movies broadcasting currently by manually entering the following classifications into the pvrdetails.xml for both movie titles:-

<mpaa>UK-15</mpaa> and also trying <mpaa>UK:15</mpaa>

and then using this code in MyPVRChannels.xml:-

PHP Code:
<control type="image"><left>280</left><top>775</top>    <width>70</width><height>70</height><texture>$INFO[Window(Home).Property(SkinHelper.PVR.mpaa)]</texture></control

Unfortunately, I didn't get any artwork returned, so maybe this is not implemented yet or maybe I'm doing something incorrectly
Yer you are doing something wrong sorry to say.
The texture has no image assigned to it.
Skin helper just gives the info eg UK-15
If you put .png after mpaa)] so looks like "mpaa)].png" and put the image in the root of the skin named "UK-15.png" it should work. But a very crude way of doing it.

Better using variables.

If you change it to a label you will see UK-15

What skin are you using?
(2015-12-03, 03:51)Rjsachse Wrote: [ -> ]Yer you are doing something wrong sorry to say.
The texture has no image assigned to it.
Skin helper just gives the info eg UK-15
If you put .png after mpaa)] so looks like "mpaa)].png" and put the image in the root of the skin named "UK-15.png" it should work. But a very crude way of doing it.

Better using variables.

If you change it to a label you will see UK-15

What skin are you using?

Okay, got it Smile
Aye, like you say, probably not the best solution. Hopefully something less labour intensive can be added.
Thanks for the suggestion though
Cheers
Oh ps, I'm currently using MadNox Smile
(2015-12-02, 21:25)Dumyat Wrote: [ -> ]Not sure if you missed this one or it's not doable, or no interest in adding?

Like mentioned before by Rjsachse you can put any info you like in the pvrdetails.xml file and it will result in a window property.
I will look into scraping the ratings and mpaa from TMDB/IMDB but this will take me a while to get right. It's on my TODO list.
(2015-12-02, 20:06)badaas Wrote: [ -> ]
Code:
18:01:30 T:3324   ERROR: CCurlFile::Open failed with code 404 for http://localhost:52307/getthumb&title=Beck/
18:01:30 T:3324   ERROR: XFILE::CHTTPDirectory::GetDirectory - Unable to get http directory (http://localhost:52307/getthumb&title=Beck/)
18:01:30 T:3324   ERROR: XFILE::CDirectory::GetDirectory - Error getting http://localhost:52307/getthumb&title=Beck/

You or kodi adding '/' to end of string supplied. I dont supply the '/' ? I suppply Listitem.Artist which definitely doesn't have a '/'.
Unless needed, but then I get no images?

Thiis with old version of helper too [6 days old] same error, is it because at songs level? Is with old beta 2 kodi too. Odd

Code:
22:28:14 T:6440   ERROR: CCurlFile::Open failed with code 404 for http://localhost:52307/getthumb&title=James Curd/
22:28:14 T:6440   ERROR: XFILE::CHTTPDirectory::GetDirectory - Unable to get http directory (http://localhost:52307/getthumb&title=James Curd/)
22:28:14 T:6440   ERROR: XFILE::CDirectory::GetDirectory - Error getting http://localhost:52307/getthumb&title=James Curd/
22:28:15 T:7684   ERROR: CCurlFile::Open failed with code 404 for http://localhost:52307/getthumb&title=John Legend/
22:28:15 T:7684   ERROR: XFILE::CHTTPDirectory::GetDirectory - Unable to get http directory (http://localhost:52307/getthumb&title=John Legend/)
22:28:15 T:7684   ERROR: XFILE::CDirectory::GetDirectory - Error getting http://localhost:52307/getthumb&title=John Legend/
Still broken, no new imaages for google thumbs at all,404 dir errors everywhere..
I was just coming on here to post about this Smile

The old Google image search api - long since deprecated - has now been discontinued, so all Google image searches by the script are failing. It's probably possible to switch it to the Google custom search api without too much difficulty, and indeed I'll give it a go myself over the weekend if I'm not beaten to it Wink

(With the possible difficulty in that it needs an api key, which - if I am the first to make the changes and PR - I won't include for obvious reasons!)

Edit:- Or not... requires a search engine ID and an api key for 100 queries a day, which obviously wouldn't be anywhere near enough for this script, and would likely be too much to ask each user who wants it to sign up for + enter details into the script (and probably against the TOS too).