Bug 3686 - Browse Music Folder Dirs that are Symbolic links have null names
: Browse Music Folder Dirs that are Symbolic links have null names
Status: RESOLVED DUPLICATE of bug 3681
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 6.3.0
: PC Linux (other)
: P2 major (vote)
: ---
Assigned To: Dan Sully
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-07-02 11:52 UTC by deegy
Modified: 2011-03-16 04:39 UTC (History)
0 users

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description deegy 2006-07-02 11:52:21 UTC
In Browse Music Folder:

I have a directory that contains a symbolic link to another directory on another disk
ie.:

Rock
   |
    - Led Zep (Symoblic link)
    - Slade 
    - Saxon

Three rows appear except the 1st has a blank name. If you select one of the icons on the right the files 
in the Led Zep directory are added to the play list. However as there is no Led Zep link it is not possible to drill down any further
buy turning on d_files

it can be seen that normal directories do not return a following / however symbolic links do:

extracted /Music/Rock/Led Zep/ from file url file://Music/Rock/Led Zep/
.....
extracted /Music/Rock/Slade from file url file://Music/Rock/Slade
....
extracted /Music/Rock/Saxon from file url file://Music/Rock/Saxon

this seems to be significant as I added 
   $file=~s!/$!!;

to pathFromFileURL in Slim/Utils/Misc.pm

and it fixes the problem but this may not be the best fix! (maybe better to fix the url rather than the file as per all of the other fixes)
This never happened in earlier versions.
Comment 1 KDF 2006-07-02 12:19:42 UTC
it is not the right fix, since forcing the / at the end was part of fixing a problem with folders that end with a space. (annying workaround to deal with a CPAN module problem. Useful, however, as it does isolate the problem.
Comment 2 Chris Owens 2006-07-02 17:12:50 UTC
Is this the same as bug 3681?
Comment 3 Dan Sully 2006-07-02 17:20:50 UTC
Yes.

*** This bug has been marked as a duplicate of 3681 ***