Created attachment 6443 [details]
MusicIP server log, Version 7.3

Server log when creating mix in 7.3

what are your MIP import settings?

(In reply to comment #2)
> what are your MIP import settings?

Under SB Settings -> Plugins -> MusicIP it says:

Use MusicIP:  Only read mixable status from MusicIP (faster)

It that what you mean?   Thanks,

I'm sorry to say, but this is working fine for me (7.5 on XP). I selected a track with an umlaut in it, and it mixed just fine.

The errors you have in the log aren't directly related to MusicIP, but they seem to arise when a playlist is to be written(?). Please enable logging for plugin.musicip and try again.

(In reply to comment #4)
> I'm sorry to say, but this is working fine for me (7.5 on XP). I selected a
> track with an umlaut in it, and it mixed just fine.
> 
> The errors you have in the log aren't directly related to MusicIP, but they
> seem to arise when a playlist is to be written(?). Please enable logging for
> plugin.musicip and try again.

Thanks for the feedback.  Sorry for the confusion.  Yes, a MusicIP can create a mix from a song with an umlaut, but you can not play them.  For example, if there is a song with an umlaut present in the playlist as song number 5, on the webinterface it will not be listed in the playlist.  If you play song 4, and skip to the next song, it will play song 6 (there is no song 5 in the playlist).  On the squeezebox the display will show totally black when you browse to song 5, and when you hit play, it will display: "Problem: can't open file for:".  On the Boom it will also show black.

If I just search for the trouble song and play it directly it plays fine.  

In the logs I see that the character encoding in 7.4 is very different from 7.3, so I suspect that to be the problem.  

Uploaded a new log, with MusicIP debug enabled.  Near time 10:44 you will not the errors in the log when I browse to a problem song (where the SB display is black) and try to play it.

Thanks.

Created attachment 6444 [details]
Server Log with MusicIP debugging enabled.  SqueezeServer 7.4.2

QA to get solid repro instructions

*** Bug 15701 has been marked as a duplicate of this bug. ***

Created attachment 6555 [details]
Server Log with MusicIP debugging enabled.  SqueezeServer 7.5.0 - r30155

Bug is also in Version 7.5.  Attached server log, trying to make a mix based on:
Location: E:\Maho Backup\Music\All Music\Music\' P Q R S T\Rokia Traoré\2003 - Bowmboï\03 - Kôté Don.mp3

In scanner log imported from MusicIP as (no errors):
[10-02-22 12:43:08.0334] Slim::Plugin::MusicMagic::Importer::setMixable (352) track: E:\Maho Backup\Music\All Music\Music\' P Q R S T\Rokia Traoré\2003 - Bowmboï\03 - Kôté Don.mp3 is mixable

Creating mix as (no errors):
[10-02-22 16:19:36.2499] Slim::Plugin::MusicMagic::Plugin::getMix (940) Creating mix for: song using: E:\Maho Backup\Music\All Music\Music\' P Q R S T\Rokia Traoré\2003 - Bowmboï\03 - Kôté Don.mp3 as seed.
[10-02-22 16:19:36.2513] Slim::Plugin::MusicMagic::Plugin::getMix (951) Request http://localhost:8082/api/mix?song=E%3A%5CMaho%20Backup%5CMusic%5CAll%20Music%5CMusic%5C'%20P%20Q%20R%20S%20T%5CRokia%20Traor%E9%5C2003%20-%20Bowmbo%EF%5C03%20-%20K%F4t%E9%20Don.mp3&variety=2&rejectsize=0&mixgenre=0&style=15&sizetype=tracks&size=50

Tries to play as (error):
[10-02-22 16:32:14.5006] Slim::Formats::readTags (157) File missing: E:\Maho Backup\Music\All Music\Music\' P Q R S T\Rokia Traoré\2003 - Bowmboï\03 - Kôté Don.mp3

Other songs in the mix with 'strange' characters fail to play as well.

I also have problems with creating MIP mixes for files with German umlauts in the file name. I am using SC 7.5.1 on Ubuntu.

For all files with German umlauts in the file name the "M" symbol is not shown when browsing through the web interface. On the Squeezebox, the menu item "Create MusicIP Mix" does not exist for such songs. For all other songs the "M" is shown in the web interface and the menu item exists on the Squeezebox.

I am sure that this worked in the past but unfortunately do not know when it has stopped to work.

Creating mixes for such songs directly in the MusicIP programm works without problems.

Can I provide any log to help solving this bug?

Created attachment 7090 [details]
Possible patch

It is not clear what the actual API to Music IP is, especially with respect to character-set encodings. It has been suggested that it is always UTF-8, even when the local file-system used a different character-set encoding.

This patch would help if the encoding is indeed UTF-8 but there are several other places in the code where character-set encoding is probably not handled correctly.

Resolution of these issues would require someone with access to all the necessary systems to go through these aspects of the code carefully. In particular, look at places where pathnames are encoded or decoded or where URI-escaping is used. Places where Slim::Utils::Misc::pathFromFileURL() and Slim::Utils::Misc::fileURLFromPath() would deserve inspection.

I am now on SC 7.6.1 - r33110 on Ubuntu Linux and it is still not possible to use a track as seed for a MusicIP mix if there is a German umlaut (or another "irregular" character) within the file path (track name or directory name).

For these tracks/albums the MusicIP "m" is neither shwon in the web interface nor in the player display. Further, on the player there is no item "Create MusicIP mix" in the context menu of a track (when presseing the RIGHT button on the remote) as it is for other tracks.

I love MusicIP and would really appreciate if this very old bug could be solved. The problem is only related to file/directory names not to tags with umlauts. I know that I could rename all files/directories but I cannot go throuh 15000 songs to look for file/directory names comprising umlauts. Further, when ripping CDs the file names and directory structure is automatically derived from the tags (which can comprise umlauts). So I had to manually correct all file/directory names after each ripping to avoid umlauts in file/directory names.

Is there a chance that this problem will ever be solved?

Dieter - would Alan's patch fix the issue for you?

Can I simply apply the patch or do I have to compile something? I remember that with the precompiled Windows version of squeezecenter it was not possible to use amended pm files. This may be different since I am using now the linux version.

(In reply to comment #14)
> Can I simply apply the patch or do I have to compile something?

Correct, most of the code on Windows is built in to the binary, but in this case you're lucky: plugins are still there as source code. You can edit them and the change should be picked up when you restart the server.

I will test it this evening and then report back.

I have applied the patch to /usr/share/perl5/Slim/Plugin/MusicMagic/Common.pm but unfortuntately it did not help. It is still not possible to create MusicIP mixes for files which have an umlaut in the file name or path. For those files the "context menu" item "Create MusicIP mix" is not shown and long pressing PLAY on the remote, which usually creates a mix based on the selected item, does also not work. 

I previously mentioned that the "m" (indicating that the selected item can be used as seed for a MusicIP mix) is not shown for these files neither in the web interface nor in the player display. I just recognized that in the player display the "m" is not shown for ANY items AT ALL as long as one use the standard browse menus. I use CustomBrowse and in the CustomBrowse Artist and Album browse menus the "m" is shown but only for items which have no umlaut in the file name or path.

I also checked the web interface (which I hardly ever use) and I found that the standard Artist and Album browse menus do also behave different from the CustomBrowse Artist and Album browse menus. While the CB menus in the web interace behave exactly like on the player (showing the "m" only for mixable items) the standard menus show the "m" for ALL items (that's the opposite to the player standard browse menus). However, although the "m" is shown in the standard browse menus, items with an umlaut in the file name or path are not mixable even through the web interface. So it seems that CB does a more correct job with showing the MusicIP indicator.

BTW, when I talk of my player I always mean a squeezebox 2 or 3, which may behave different from a squeezebox touch (which I don't have).

Is there anything else I can test. I would really appreciate to see this bug being fixed and can provide you with log files or anything else you need.

> I have applied the patch to /usr/share/perl5/Slim/Plugin/MusicMagic/Common.pm

I thought you were on Windows?

> but unfortuntately it did not help. It is still not possible to create MusicIP
> mixes for files which have an umlaut in the file name or path. For those files

Did you run a wipe & rescan of your collection?

(In reply to comment #18)
> I thought you were on Windows?
No, in the past (long time ago) I was on Windows, but now I am on Ubuntu Linux (as stated in comments 12 and 14 ;).
 
> Did you run a wipe & rescan of your collection?
No, should I have done that? I restarted the server (several times) and there was always an automatic scan after the restart (that's new, perhaps since I turned on the new automatic detection of new music feature?).

But I assume that this automatic scan (which takes only 1,5 minutes for about 13000 tracks) does not make a wipe. So should I repeat the test with a wipe?

I again applied Alans's patch, restarted the server and made a complete wipe and rescan afterwards but it did not change anything. Tracks with umlauts in the file/directory name can still not be used to make a MusicIP mix.

Anything else I can test?

Michael, would it help if I provided a log file? Or can I do anything else?

logs always are helpful, but I doubt we'll soon have time to further investigate this issue

I enclose my server.log and scanner.log files. I have made a wipe and rescan and you see from the scanner.log file that for each track with an umlaut in the filename/path during the MusicIP-import there is an error entry like 

Slim::Plugin::MusicMagic::Importer::setMixable (352) Couldn't get track for file:///media/Daten/Musik/Herbert%20Gr%F6nemeyer/4630%20Bochum/01-Herbert%20Gr%F6nemeyer%20-%20Bochum.ogg

So it seems that there is (at least) a problem in the MusicIP-import.

Can you point me to the source code which must be changed that the MusicIP-import is done correctly? If we are lucky, that's all what must be corrected.

Created attachment 7517 [details]
scanner.log file

Created attachment 7518 [details]
server.log file

Created attachment 7519 [details]
scanner.log file

Slim::Plugin::MusicMagic::Importer is the starting point. It's calling some methods in Slim::Plugin::MusicMagic::Common too, so make sure to follow the flow.