YAMJ / yamj-v2

Yet Another Movie Jukebox (YAMJ) v2
GNU General Public License v3.0
28 stars 11 forks source link

hashed paths for artwork for moview with a dot in the name is wrong #2644

Closed Omertron closed 9 years ago

Omertron commented 9 years ago

Original issue 2646 created by Omertron on 2013-04-21T09:32:23.000Z:

What steps will reproduce the problem? 1.Run YAMJ creating hashed directories with a movie title containing a dot 2.Check XML 3.

What is the expected output? What do you see instead? Artwork is missing for e.g. L.A. Confidential, A.I. Artificial Intelligence, G.I. Joe.

The XML points to a directory called e.g. l. a. or g.

<posterFile>l%5Cl.%5CL.A.%20Confidential%20%5B1997%5D%20720p%20BRRip%20H264%20AC3.jpg</posterFile>

What version/revision of YAMJ are you using? On what operating system? YAMJ 2.9-SNAPSHOT (r3684)

What skin are you using? evHell 4

What device are you using YAMJ on? Networked Media Tank, PlayonHD, Other? A-400

Please provide any additional information below.

Omertron commented 9 years ago

Comment #1 originally posted by Omertron on 2013-04-21T10:39:05.000Z:

Sorry should have written that the created directories corresponding to e.g. l. is only l

Don't know if it matters but I run YAMJ from windows.

Omertron commented 9 years ago

Comment #2 originally posted by Omertron on 2013-04-24T04:27:17.000Z:

Sorry should have written that the created directories corresponding to e.g. l. is only l

The actual artwork is downloaded by YAMJ, but is not shown due to the faulty path in the xml.

Don't know if it matters but I run YAMJ from windows.

Omertron commented 9 years ago

Comment #3 originally posted by Omertron on 2013-04-28T09:21:09.000Z:

I think this is the same as Issue 2278 comment # 41-42-43

I suggest to remove non alphanumeric chars in the hash algorithm or substitute problematic chars with placeholder like "_" or "-"

Omertron commented 9 years ago

Comment #4 originally posted by Omertron on 2013-04-29T14:36:23.000Z:

Probably a placeholder of "_" in case both characters are replaced.

Omertron commented 9 years ago

Comment #5 originally posted by Omertron on 2013-04-29T14:56:11.000Z:

This issue was closed by revision r3699.

Omertron commented 9 years ago

Comment #6 originally posted by Omertron on 2013-04-29T14:56:58.000Z:

Note: You may need to rebuild your jukebox folder after this change