[Release] - script.playlists
#16
pecinko Wrote:Thanks I'm adding it to requred by skin Smile

would it be possible to get a few users to test it.
i really want to make sure there aren't to many bugs before i add it to the repo.

Machine-Sanctum Wrote:I won't say anything Rolleyes

lol! :-)
Do not PM or e-mail Team-Kodi members directly asking for support.
Always read the Forum rules, Kodi online-manual, FAQ, Help and Search the forum before posting.
Reply
#17
ronie Wrote:would it be possible to get a few users to test it.
i really want to make sure there aren't to many bugs before i add it to the repo.

I was thinking you are kidding, but it turns out you don't trust my judgementSmile

On a more serious note. It's a fairly simple script, do you think it's really necessary? It pulls playlists well, the labels match. What did you have in mind by testing?
My skins:

Amber
Quartz

Reply
#18
pecinko Wrote:I was thinking you are kidding, but it turns out you don't trust my judgementSmile

On a more serious note. It's a fairly simple script, do you think it's really necessary? It pulls playlists well, the labels match. What did you have in mind by testing?

i don't even trust my own judgment :-)

reason i ask is, coding a script for you is one thing and pushing to the repo isn't a problem either...

but who's is gonna handle the support for this script?
who will everyone turn to when things don't work as expected?

right....me :-)


rule of thumb, you never know what to expect until a bunch of people have tested it.

most likely pittfalls:
- what happens if a user doesn't have any playlists at all
- does the script properly handle non-ascii characters in filenames / playlist names
Do not PM or e-mail Team-Kodi members directly asking for support.
Always read the Forum rules, Kodi online-manual, FAQ, Help and Search the forum before posting.
Reply
#19
ronie Wrote:most likely pittfalls:
- what happens if a user doesn't have any playlists at all
- does the script properly handle non-ascii characters in filenames / playlist names

It seams that it's not a problem if you do not have playlists but using non ascii is problematic and causes script to fail. Why do I have a feeling you already new this? :-)

My proposal is - if you are able to solve non ascii characters I would put it in a repo and solve possible problems once they get reported. Reason is simple, I do not believe that a user will download and test the script on it's own as this is an extension of already existing system feature. Opposite case is when you want extended weather info so you download weather+

If you disagree, I'll make the script is available check in the skin and run it conditionally but I would really like to avid it if possible.

Here's a log:

Code:
12:37:44 T:2958716928   DEBUG: Process - Entering source directory /Users/pecinko/Library/Application Support/XBMC/addons/script.playlists
12:37:44 T:2958716928   DEBUG: Instantiating addon using automatically obtained id of "script.playlists" dependent on version 2.0 of the xbmc.python api
12:37:44 T:2958716928   DEBUG: script.artistslideshow: script version 0.0.1 started
12:37:44 T:2958716928   DEBUG: script.artistslideshow: params: {'type': 'video'}
12:37:44 T:2958716928   DEBUG: script.artistslideshow: dirlist: ['ddd.xsp', "e\xcc\x8cs\xcc\x8cc\xcc\x8cr\xcc\x8cz\xcc\x8cy\xcc\x81a\xcc\x81i\xcc\x81e\xcc\x81='+u\xcc\x8a.xsp", 'Movies HD.xsp']
12:37:44 T:2958716928   DEBUG: script.artistslideshow: playlists: [('ddd', 'special://profile/playlists/video/ddd.xsp'), (u'Do\u0159e', "special://profile/playlists/video/e\xcc\x8cs\xcc\x8cc\xcc\x8cr\xcc\x8cz\xcc\x8cy\xcc\x81a\xcc\x81i\xcc\x81e\xcc\x81='+u\xcc\x8a.xsp"), ('Movies HD', 'special://profile/playlists/video/Movies HD.xsp')]
12:37:44 T:2958716928    INFO: -->Python script returned the following error<--
12:37:44 T:2958716928   ERROR: Error Type: <type 'exceptions.UnicodeEncodeError'>
12:37:44 T:2958716928   ERROR: Error Contents: 'ascii' codec can't encode character u'\u0159' in position 2: ordinal not in range(128)
12:37:44 T:2958716928   ERROR: Traceback (most recent call last):
                                              File "/Users/pecinko/Library/Application Support/XBMC/addons/script.playlists/default.py", line 70, in <module>
                                                Main()
                                              File "/Users/pecinko/Library/Application Support/XBMC/addons/script.playlists/default.py", line 19, in __init__
                                                self._set_properties()
                                              File "/Users/pecinko/Library/Application Support/XBMC/addons/script.playlists/default.py", line 65, in _set_properties
                                                self.WINDOW.setProperty( 'ScriptPlaylist.%d.Name' % (count + 1), str( item[0] ) )
                                            UnicodeEncodeError: 'ascii' codec can't encode character u'\u0159' in position 2: ordinal not in range(128)
12:37:44 T:2958716928    INFO: -->End of Python script error report<--
12:37:44 T:2958716928    INFO: Python script stopped
12:37:44 T:2958716928   DEBUG: Thread XBPyThread 2958716928 terminating
12:37:44 T:2896233152   DEBUG: waiting for python thread 13 to stop
12:37:44 T:2896233152   DEBUG: python thread 13 destructed
12:37:44 T:2896233152   DEBUG: ------ Window Init (DialogKaiToast.xml) ------
12:37:44 T:2896233152   DEBUG: Alloc resources: 0.01ms (0.00 ms skin load)
12:37:49 T:2896233152   DEBUG: ------ Window Deinit (DialogKaiToast.xml) ------
12:37:54 T:2896233152    INFO: Python, unloading python shared library because no scripts are running anymore
12:38:14 T:2957651968   DEBUG: Thread Jobworker 2957651968 terminating (autodelete)
12:38:14 T:2957119488   DEBUG: Thread Jobworker 2957119488 terminating (autodelete)
12:38:14 T:2956587008   DEBUG: Thread Jobworker 2956587008 terminating (autodelete)
12:38:14 T:2955980800   DEBUG: Thread Jobworker 2955980800 terminating (autodelete)
12:38:14 T:2896233152   DEBUG: SECTION:UnloadDelayed(DLL: special://xbmcbin/system/ImageLib-x86-osx.so)
12:38:14 T:2896233152   DEBUG: Unloading: ImageLib-x86-osx.so
My skins:

Amber
Quartz

Reply
#20
pecinko Wrote:It seams that it's not a problem if you do not have playlists but using non ascii is problematic and causes script to fail. Why do I have a feeling you already new this? :-)

experience ? :-)

i haven't tested the script at all, but know several other scripts
have had the same issue.

i'll see if i can come up with a fix for it.
Do not PM or e-mail Team-Kodi members directly asking for support.
Always read the Forum rules, Kodi online-manual, FAQ, Help and Search the forum before posting.
Reply
#21
Thanks Ronie, good script

with Two buttons in the Home
Image
Reply
#22
this one should fix possible unicode issues:
script.playlists-0.0.2.zip
Do not PM or e-mail Team-Kodi members directly asking for support.
Always read the Forum rules, Kodi online-manual, FAQ, Help and Search the forum before posting.
Reply
#23
ronie Wrote:this one should fix possible unicode issues:
script.playlists-0.0.2.zip

Thanks. Seams that problem with special characters is solved. However, there seams to be other problem now:

Code:
20:22:43 T:2962444288   DEBUG: script.playlists: script version 0.0.2 started
20:22:43 T:2962444288   DEBUG: script.playlists: params: {'type': 'music'}
20:22:43 T:2962444288   DEBUG: script.playlists: dirlist: ['Electronic.xsp', 'New test.m3u', 'Pametna.xsp', 's\xcc\x8cc\xcc\x8c test 02 e\xcc\x81i\xcc\x81.m3u', 'Test z\xcc\x8cy\xcc\x81a\xcc\x81i\xcc\x81.xsp']
20:22:43 T:2962444288   DEBUG: script.playlists: playlists: [('Electronic', 'special://profile/playlists/music/Electronic.xsp'), ('Pametna', 'special://profile/playlists/music/Pametna.xsp'), ('Test zyai', 'special://profile/playlists/music/Test z\xcc\x8cy\xcc\x81a\xcc\x81i\xcc\x81.xsp')]
20:22:43 T:2962444288   DEBUG: script.playlists: script stopped
20:22:43 T:2962444288    INFO: Scriptresult: Success
20:22:43 T:2962444288    INFO: Python script stopped


This gives me 3 playlist items, although 5 items were (correctly) pulled from XBMC.

EDIT:

Here's a screenshot:

Image
My skins:

Amber
Quartz

Reply
#24
pecinko Wrote:Thanks. Seams that problem with special characters is solved. However, there seams to be other problem now:

Code:
20:22:43 T:2962444288   DEBUG: script.playlists: script version 0.0.2 started
20:22:43 T:2962444288   DEBUG: script.playlists: params: {'type': 'music'}
20:22:43 T:2962444288   DEBUG: script.playlists: dirlist: ['Electronic.xsp', 'New test.m3u', 'Pametna.xsp', 's\xcc\x8cc\xcc\x8c test 02 e\xcc\x81i\xcc\x81.m3u', 'Test z\xcc\x8cy\xcc\x81a\xcc\x81i\xcc\x81.xsp']
20:22:43 T:2962444288   DEBUG: script.playlists: playlists: [('Electronic', 'special://profile/playlists/music/Electronic.xsp'), ('Pametna', 'special://profile/playlists/music/Pametna.xsp'), ('Test zyai', 'special://profile/playlists/music/Test z\xcc\x8cy\xcc\x81a\xcc\x81i\xcc\x81.xsp')]
20:22:43 T:2962444288   DEBUG: script.playlists: script stopped
20:22:43 T:2962444288    INFO: Scriptresult: Success
20:22:43 T:2962444288    INFO: Python script stopped


This gives me 3 playlist items, although 5 items were (correctly) pulled from XBMC.

EDIT:

Here's a screenshot:

Image

ok, the script currrently only adds .xsp playlists,
will see if i can add .m3u as well :-)
Do not PM or e-mail Team-Kodi members directly asking for support.
Always read the Forum rules, Kodi online-manual, FAQ, Help and Search the forum before posting.
Reply
#25
ronie Wrote:ok, the script currrently only adds .xsp playlists,
will see if i can add .m3u as well :-)

Ohh, sorry - it's working great in that case!
My skins:

Amber
Quartz

Reply
#26
added .m3u support: script.playlists-0.0.3.zip
Do not PM or e-mail Team-Kodi members directly asking for support.
Always read the Forum rules, Kodi online-manual, FAQ, Help and Search the forum before posting.
Reply
#27
Nice, this one works even greater!

Testing with the same files from the screenshot I sent, you might want to consider adding supportSmile for spaces in .m3u names for the new version.


Code:
21:17:50 T:2966171648   DEBUG: script.playlists: script version 0.0.3 started
21:17:50 T:2966171648   DEBUG: script.playlists: params: {'type': 'music'}
21:17:50 T:2966171648   DEBUG: script.playlists: dirlist: ['Electronic.xsp', 'New test.m3u', 'Pametna.xsp', 's\xcc\x8cc\xcc\x8c test 02 e\xcc\x81i\xcc\x81.m3u', 'Test z\xcc\x8cy\xcc\x81a\xcc\x81i\xcc\x81.xsp']
21:17:50 T:2966171648   DEBUG: script.playlists: playlists: [('Electronic', 'special://profile/playlists/music/Electronic.xsp'), ('New ', 'special://profile/playlists/music/New test.m3u'), ('Pametna', 'special://profile/playlists/music/Pametna.xsp'), ('s\xcc\x8cc', 'special://profile/playlists/music/s\xcc\x8cc\xcc\x8c test 02 e\xcc\x81i\xcc\x81.m3u'), ('Test zyai', 'special://profile/playlists/music/Test z\xcc\x8cy\xcc\x81a\xcc\x81i\xcc\x81.xsp')]
21:17:50 T:2966171648   DEBUG: script.playlists: script stopped
21:17:50 T:2966171648    INFO: Scriptresult: Success
21:17:50 T:2966171648    INFO: Python script stopped
My skins:

Amber
Quartz

Reply
#28
pecinko Wrote:Nice, this one works even greater!

Testing with the same files from the screenshot I sent, you might want to consider adding supportSmile for spaces in .m3u names for the new version.

should be fixed now:
script.playlists-0.0.4.zip
Do not PM or e-mail Team-Kodi members directly asking for support.
Always read the Forum rules, Kodi online-manual, FAQ, Help and Search the forum before posting.
Reply
#29
This one is working perfectly at my side!
My skins:

Amber
Quartz

Reply
#30
pecinko Wrote:This one is working perfectly at my side!

cool, we're ready then ;-)

available in our repo as of now:
http://mirrors.xbmc.org/addons/eden-pre/...playlists/
Do not PM or e-mail Team-Kodi members directly asking for support.
Always read the Forum rules, Kodi online-manual, FAQ, Help and Search the forum before posting.
Reply

Logout Mark Read Team Forum Stats Members Help
[Release] - script.playlists0