Skip to content

Comments

Fix Windows 8.3 file name handling#206

Merged
mherger merged 1 commit intoLMS-Community:public/7.9from
BorisMahone:public/7.9
Jun 15, 2018
Merged

Fix Windows 8.3 file name handling#206
mherger merged 1 commit intoLMS-Community:public/7.9from
BorisMahone:public/7.9

Conversation

@BorisMahone
Copy link
Contributor

The filename who has multi-byte characters that contain 0x7e is garbled.
(ex. {0x83, 0x7e} expresses "ミ" in Shift-JIS, and this regular expression will catch the character.)
This change fixes the issue that showing garbled file name in "Browse Music Folder".

@mherger
Copy link
Contributor

mherger commented Jun 14, 2018

Could you provide a sample file, or a filename I can use to test this with?

@BorisMahone
Copy link
Contributor Author

I attached the file with zip that is containing one mp3 file for testing.
And here are the screen shots of web control.
I think that the test with English environment may require installing language pack.

[Incorrect encoding (garbled)]
incorrect
[Correct encoding]
correct

music.zip

@mherger
Copy link
Contributor

mherger commented Jun 15, 2018

That file didn't even arrive on my computer:

bildschirmfoto 2018-06-15 um 13 29 11

Could you please upload the raw file (not compressed) to the following link:

https://www.dropbox.com/request/5nM0UBKfwnghGkyAZy9F

Thanks!

@mherger mherger merged commit 5aee266 into LMS-Community:public/7.9 Jun 15, 2018
@mherger
Copy link
Contributor

mherger commented Jun 15, 2018

Let's give this a try anyway. Shouldn't hurt. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants