• 1
  • 15
  • 16
  • 17(current)
  • 18
  • 19
  • 309
Release skin helper service
(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.
Reply
(2015-09-13, 17:13)marcelveldt Wrote: 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.

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..
Phenomenal™ Skin • ForumWebsiteGitHub
Reply
Hi marcel, great work and support! Can you add tomato rating from emby please? Big Grin
Reply
Hi Marcel, I'm experiencing a few errors when browsing live tv with what looks like unicode issues

Code:
20:34:50 T:79580   ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
                                             - NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
                                            Error Type: <type 'exceptions.AttributeError'>
                                            Error Contents: 'list' object has no attribute 'has_key'
                                            -->End of Python script error report<--
20:34:50 T:79580   ERROR: SystemError
20:34:50 T:79580   ERROR: :
20:34:50 T:79580   ERROR: bad format char passed to Py_BuildValue
20:34:50 T:79580   ERROR: CPythonInvoker(110, C:\Users\Ian\AppData\Roaming\Kodi\addons\script.skin.helper.service\plugin.py): failed to run the gc to clean up after running prior to shutting down the Interpreter
20:34:50 T:70368  NOTICE: plugin.video.emby stopped
20:34:50 T:76692   ERROR: XFILE::CDirectory::GetDirectory - Error getting plugin://script.skin.helper.service/?action=getthumb&path=Live Cubs @ Pirates BT Sport 1

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.
Reply
(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.
Reply
(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.
Reply
(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.
Reply
(2015-09-17, 17:28)marcelveldt Wrote:
(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.
10x.
but still
http://pastebin.com/6UsTN13T
(i'm not using titan skin)
I think its something with:
Code:
<setting type="bool" name="skin.phenomenal.IsTitanSkin">false</setting>
Phenomenal™ Skin • ForumWebsiteGitHub
Reply
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.
Phenomenal™ Skin • ForumWebsiteGitHub
Reply
(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
Reply
(2015-09-17, 17:30)marcelveldt Wrote:
(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.

Thanks Marcel, will test and report back if any errors. Yes I use the PVR.Thumb, PVR.Poster etc now in place of the auto thumb generator.
Reply
(2015-09-17, 18:46)marcelveldt Wrote:
(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

boot fixed, but the backup is failed right away: http://pastebin.com/Xi12Ctpm
Phenomenal™ Skin • ForumWebsiteGitHub
Reply
(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.
Reply
Actually I can't give it to you because the problem is made when I'm trying to backup,
But I guess the old ones, that I made before two or three fixes, when the problem was on the restore - will be ok:
https://drive.google.com/file/d/0B8R_P39...sp=sharing
Phenomenal™ Skin • ForumWebsiteGitHub
Reply
Can ''Window(Home).Property(SkinHelper.ListItemGenres)'' return music genres too please?
Reply
  • 1
  • 15
  • 16
  • 17(current)
  • 18
  • 19
  • 309

Logout Mark Read Team Forum Stats Members Help
skin helper service18