Release Artist Slideshow addon (with skin and addon integration)
(2014-10-29, 02:30)pkscout Wrote:
(2014-10-29, 01:18)kyungrak Wrote: Hello,

thank you for maintaining this nice script.
I've stumbled upon a bug, might be helix related since I don't recall having this issue with gotham, so maybe you won't be able to look at it until next beta release. But anyway here's the issue:

the script works fine for songs with single artist, but with multiple artists the script stops, throwing a unicode exception. It seems that happens when it tries to copy pictures from the second artist into the merge directory.

Here's the log:

http://xbmclogs.com/show.php?id=333139

Thanks in advance.

Can anyone with a Helix confirm this? I tested in Gotham, and I definitely don't see that problem. Nor do I even understand why you'd be getting that error in Helix, especially given that none of the path in your case has any extended characters. I did check the code, and as far as I can tell everything is being properly encoded before being passed to xbmcvfs.copy (which then calls posixpath.py, which is what actually croaks in this case). So even if you had extended characters in the path, it should be working.

God I hate unicode. I hate it with the undying passion of 1000 burning suns.


I 've found the culprit. I'm using custom photos fetched on internet, and some of them had extended characters in their file name. No more issues after removing them.

FYI these characters don't cause any trouble in single mode, but in multiple artists mode the script crash with posixpath.py as you mentioned before.

PS: now I realize why I thought it was a general issue with multiple artists, I have extended characters in some picture file names for almost every artist in my database.
Reply


Messages In This Thread
New Test Version Available - by pkscout - 2012-03-08, 16:26
[No subject] - by mardup - 2012-03-10, 12:26
[No subject] - by pkscout - 2012-03-11, 01:35
RE: - by mardup - 2012-03-12, 17:50
RE: - by pkscout - 2012-03-13, 03:22
New Beta Version for Testing - by pkscout - 2012-07-13, 17:54
New Beta Version for Testing - by pkscout - 2012-07-14, 22:50
Pull Request for v1.4 Submitted - by pkscout - 2012-07-20, 23:01
New Beta Version for Testing - by pkscout - 2012-08-10, 03:18
New Frodo BETA Support Files - by pkscout - 2012-11-21, 06:20
RE: [RELEASE] Artist Slideshow 1.3.0 addon (with skin and addon integration) - by kyungrak - 2014-10-31, 01:08
New 2.0.0 Beta Coming Soon - by pkscout - 2016-05-12, 02:54
Writing tips - by Parkerbup - 2017-02-03, 14:56
Add-on Artist Slideshow - by jo26 - 2014-08-08, 22:58
Logout Mark Read Team Forum Stats Members Help
Artist Slideshow addon (with skin and addon integration)5