Bug 720 - flac w/ embedded cuesheets are blank in "Browse Music Folder"
: flac w/ embedded cuesheets are blank in "Browse Music Folder"
Status: RESOLVED DUPLICATE of bug 1360
Product: Logitech Media Server
Classification: Unclassified
Component: Formats
: unspecified
: PC All
: P2 normal (vote)
: Future
Assigned To: michael
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2004-12-09 13:53 UTC by michael
Modified: 2005-06-10 17:04 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description michael 2004-12-09 13:53:46 UTC
Using "Browse Music Folder" and navigating to a flac with an embedded cuesheet
and then clicking on that file to see file details results in a blank entry.
You'll see (in the default skin) a single line with the play and add song
buttons on the right, but no text and no song is referenced by the buttons.

In addition, if the file happens to be in a location other than in the root of
the music folder, you'll see an error message on the console about a file not
found, but the filename in the error message is incorrect. The file name
referenced in the error message is missing all path element between the music
folder and the file itself. For example, if my music folder is set to
/var/audio/ and I have a file at /var/audio/test/directory/file.flac then the
error message would say 
[/var/audio/file.flac] does not exist or cannot be read: No such file or
directory at /usr/src/slim/server/CPAN/Audio/FLAC.pm line 39.
(since this file doesn't exist, the file not found message is understandable.
but where is it losing it's path elements?)
If the file is placed in the root of the Music Folder, this error message won't
appear, but the browser behaves as described above either way.

This behaviour seems to have appeared between 5.2.0 and 5.3.0 (that is, you'll
see file details if you navigate to a file like this in 5.2.0, but anything
later will exibit the behaviour described here).
Comment 1 Blackketter Dean 2005-03-11 15:27:27 UTC
Michael: is this fixed in the 6.0beta? Please verify and reopen if it's still an issue.
Comment 2 michael 2005-03-11 17:22:10 UTC
still a problem in 6.0 (just tested with revision 2431).

choose Browse Music Folder then navigate to a FLAC file with an embedded
cuesheet.  Click into that file, and you'll see a page that mearly says "Empty".
Now go Navigate to somewhere you have a file with an external cuesheet. (The
cuesheet will be what you actually see in the listing.) Clicking into it will
give you a list of the songs it contains/references.
Comment 3 michael 2005-03-11 17:23:24 UTC
reassigning, since I seem to be the embedded cuesheet guy somehow.
Comment 4 Blackketter Dean 2005-03-11 20:48:32 UTC
You the man!
Comment 5 Blackketter Dean 2005-03-23 14:47:36 UTC
Michael:  Any progress on this?
Comment 6 michael 2005-03-23 15:22:02 UTC
no. This is an annoyance, but as for priority it really falls behind some of the
parsing related issues.  So I haven't even really attacked this one yet.
I'm going to push the target milestone to 6.0.1 if you don't mind.
Comment 7 Blackketter Dean 2005-04-04 17:38:07 UTC
Pushing to 6.0.2.  Michael, what's the prognosis on this one?
Comment 8 michael 2005-04-06 03:43:02 UTC
I'll dig into this next after I've fixed the utf-8 issues (bug 1288, etc)
Comment 9 michael 2005-04-10 18:03:26 UTC
partial fix in revision 2929

browse music folder on a flac with an embedded cuesheet will now show the songs 
inside it.

outstanding issue: if you wipe cache, you'll get a blank listing 
(and a message to set you music folder correctly) even after it's finished 
scanning.  This will return to the correct behaviour after a short time.
I'm guessing that a db update needs to be triggered somewhere...
Comment 10 michael 2005-04-19 13:33:46 UTC
Fixing bug 1373 seems to have reverted the progress made so far here.  Will need
to revisit this.
Comment 11 Blackketter Dean 2005-04-20 13:29:37 UTC
Michael: what's the status of this?  Is this ready for 6.0.2 or should we move it to 6.1?
Comment 12 michael 2005-04-20 14:27:41 UTC
I'll have to dig back into this to see why the fix for 1373 reverted this. I'd
say probably 6.1 at this point.
Comment 13 Blackketter Dean 2005-06-07 13:18:31 UTC
Michael: is this still an issue in the latest 6.1 nightly?
Comment 14 Blackketter Dean 2005-06-10 17:04:06 UTC
1360 supercedes this bug

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