Posts: 8,283
Joined: Jul 2014
(2015-09-12, 06:03)tomer953 Wrote: 1) Oh, Contextmenu button... - that was hidden... I think its better to move it to somewhere else from contextmenu in the "choose theme" dialog - its not the obvious place to export theme to zip...
so about that: no way to let user load from ".theme" file directly? ill just tell them pass the ".theme" files between them, or - make the "createcolortheme" let you choose if you want to export to zip
2) I made a changes in the skinshortcuts, refreshed, created backup, restore shortcuts to default, tried to restore my backup - and nothing.
right now I tried to do it again but in the restore I get error during some unicoded path file, I'm providing log in the next line....
3) mm this is the log http://pastebin.com/fq1U5UTc but I think this one will be more useful: http://pastebin.com/CWhykTQ4 - those are the reason that script failes, actually I don't use them anymore, In the past I set them from skinshosrtcuts dialog background picker to test and fix some unicode errors overthere, so they are causing the problem. If it hard to you to fix, I'll fix it in the next few days.
and just checking - are you pushing updates of the helper to the official repo? you changed the version to 0.0.4 and I auto-get it somehow, I wondered If you did it...
1) I tried to use as less additional dialogs as possible. So the dialog used for the skinthemes is just the default DialogSelect.xml in your skin.
The export button is a bit more advanced so I decoded to "hide" it under the contextmenu. What I can do is specify an additional button that you can include in your skin if you want.
2) This should be fixed whenever problem 3 is fixed.
3) OK, I think/hope I've finally nalied the unicode issues but I can't test (as I don't have any unicode paths/settings) so please test again with latest Git version.
If it still fails I will extend my testsetup with some fake paths with some unicode characters in it.
4) I am updating the script on my beta repo. The script isn't yet released to official repo yet. As soon as the current features are stable and the unicode issues are fixed I will submit it to the repo.
Posts: 8,283
Joined: Jul 2014
(2015-09-12, 21:04)Angelinas Wrote: Just try to open your Db in database/video96.db and find streamdetails like in my picture.
You will se what are I talking about,when Kodi Show resolution for movies its different then Marcel script do.
For pvr widget ,just tested its ok,little slow but working .I put limit 500 and widget counter show number 407 ?!?
Poster,thumb working perfect,
Can't link widget with
PHP Code:
<content>plugin://script.skin.helper.service/?action=getcast&movie=$INFO[Container(510).ListItem.Title]</content>
to show textbox
PHP Code:
<label>$INFO[SkinHelper.ListItemCast]</label>
but have some free time,I will figureouthow this working.
1) I have just adjusted the calculation of the resolution, you were right, it was just returning the height while Kodi has some logic in place to convert the width+height to a resolution that makes sense. I have implemented that exact same logic now in the helper so it should return the same values as native Kodi does for items in the library. Please test latest git of the script.
2) 500 items in a widget ? That's a bit insane don't you think ? For speed reasons keep it so a low number like 25 or something.
And about that count beiing 407, I think you just have 407 channels (with info)
3) You can't use that to show a textbox, that content pluginpath is to be used with a list container. If you just need the text to be displayed in a label/textbox, use the window prop SkinHelper.ListItemCast, note that you need need to add a (fake) list to trigger the plugin, it isn't set by the background service.
Posts: 8,283
Joined: Jul 2014
(2015-09-13, 18:00)tomer953 Wrote: I'm sorry but... -> http://pastebin.com/UBjxx0JE
and I had those before the latest git:
http://forum.kodi.tv/showthread.php?tid=...pid2103975
Edit:
you can take this:
Code:
<setting type="bool" name="skin.phenomenal.skinshortcuts-background-D:\תכשיטים\בתהליך\סיכות ראש\">false</setting>
for testing. spaces+unicode chars..
OK, I used you testdata and now fixed the unicode issue ;-)
Please try latest Git.
Posts: 8,283
Joined: Jul 2014
(2015-09-15, 21:40)im85288 Wrote: Hi Marcel, I'm experiencing a few errors when browsing live tv with what looks like unicode issues
Would you consider adding the Top250 rating scrapings to this addon? There's now a TV top 250 TV Top250 making the field more useful and would save on needing to use other scripts to do the same purpose.
1) Not sure I fixed it but please test latest Git for the unicode issue with pvr thumbs.You did notice that the background service now also fills some window properties for the pvr thumbs btw ?
2) Well, adding the top250, couchpotatoe etc. sounds doable to add to the addon. I must take a look how the api works. Will check/implement within a few days.
Posts: 8,283
Joined: Jul 2014
(2015-09-17, 18:26)tomer953 Wrote: Also I have script failed error on startup - http://pastebin.com/jafWmK3r
I redownloaded latest git for skinshortcuts and helper to verify, and still - error on boot.
OK, another attempt. Please check latest git if both issues are now fixed
Posts: 8,283
Joined: Jul 2014
(2015-09-17, 19:08)tomer953 Wrote: boot fixed, but the backup is failed right away: http://pastebin.com/Xi12Ctpm
Can you share your backupfile with me please ? So I can test with some actual unicode data which will be easier to fix the issue.
Posts: 808
Joined: Dec 2014
Reputation:
51
badaas
Skilled Skinner
Posts: 808
Can ''Window(Home).Property(SkinHelper.ListItemGenres)'' return music genres too please?