PDA

View Full Version : web interface and browsing



Dolf.Dijkstra
2004-04-23, 08:56
Slim 5.1.5, RH 7.2, Default skin.

Hi,

I believe that this has been reported before, but just in case.

I observed some anomalies when browsing for songs, albums, artists and
music folder.

I have 3 folders under the genre Rock
Herman Brood
Herman Brood & His Wild Romance
Hermann Brood & His Wild Romance

Each directory contains at least one album.

../Herman Brood/Hooks
../Herman Brood/Saturday Night Live
../Herman Brood & His Wild Romance/50 The Soundtrack
../Herman Brood & His Wild Romance/Fresh Poison
../Herman Brood & His Wild Romance/The best of
../Hermann Brood & His Wild Romance/Bühnensucht

The MP3 tags (both V1.1 and V2.3 are present) correspond to the Artist
and Album name (except for Bühnensucht, which does not have the umlaut
=> Buhnensucht, and for Hermann which has the MP3 tag Herman Brood & His
Wild Romance).

First issue: browse by folder.
When I browse the folder list I get all the 3 artist names, but I can
only browse the ones with the ampersand. The HTML itself does not seem
to be broken, but clicking the link the following message is written:
"You need to specify a path to your directory containing your MP3
files." and no albums are displayed.

Also when browsing for albums the song info page is broken:

Title: Herman Brood
File Format: Unknown
Location: /music/MP3/test/Rock/Herman%20Brood%20
Date Modified: Friday, April 23, 2004, 5:24:09 PM






Second issue:browse by artist.

When browsing for artist, also Herman Brood and Herman Brood & His Wild
Romance are listed, but under Herman Brood & His Wild Romance nothing is
listed.

I thought it might be the ampersand in the file name, so I renamed
Herman Brood & His Wild Romance to Herman Brood and His Wild Romance,
and rescanned the music lib.

The problem for browsing for artists still persists.


Looks to me that searching for artist with ampersands are both in the
MP3 database and on the file system broken.

In the above cases the HTML is not broken (I believe).



Third issue: broken html.
In the now playing list a song from Herman Brood & His Wild Romance is
playing.
The link for the artist name (in the right hand panel) is:

http://danmark:9000/browseid3.html?genre=*&artist=Herman%20Brood%20&%20H
is%20Wild%20Romance&player=00:0c:41:3a:90:29

This looks properly escaped to me.

Clicking on it displays on the left hand panel:
Squeezebox Home / Browse Artists / Herman Brood
0 albums with 0 songs by 0 artists

The HTML is cut off after the ampersand (in the URL) or before (in the
html text).
------------------
<td valign="middle"><a
href="home.html?player=00:0c:41:3a:90:29">Squeezebox Home</a> /
<a
href="browseid3.html?song=&artist=&album=&genre=*&player=00:0c:41:3a:90:
29">
Browse Artists
</a>

/
<a
href="browseid3.html?song=&artist=Herman%20Brood%20&album=&genre=*&playe
r=00:0c:41:3a:90:29">
Herman Brood
</a>

</td>
--------------------

Cheers

Dolf

dean
2004-04-23, 10:07
Hi Dolf,

On Apr 23, 2004, at 8:56 AM, Dolf.Dijkstra wrote:
> http://danmark:9000/browseid3.html?
> genre=*&artist=Herman%20Brood%20&%20His%20Wild%20Romance&player=00:0c:
> 41:3a:90:29
>
> This looks properly escaped to me.
I don't think it is escaped properly. It should probably be:

http://danmark:9000/browseid3.html?
genre=*&artist=Herman%20Brood%20%26%20His%20Wild%20Romance&player=00:
0c:41:3a:90:29

This may be the root of the problem.

I'll look into it.

-dean

kdf
2004-04-23, 10:16
What version of Perl is this?
I had noticed that 5.6.1 wasn't escaping URI's properly.
-kdf

Quoting dean blackketter <dean (AT) slimdevices (DOT) com>:

> Hi Dolf,
>
> On Apr 23, 2004, at 8:56 AM, Dolf.Dijkstra wrote:
> > http://danmark:9000/browseid3.html?
> > genre=*&artist=Herman%20Brood%20&%20His%20Wild%20Romance&player=00:0c:
> > 41:3a:90:29
> >
> > This looks properly escaped to me.
> I don't think it is escaped properly. It should probably be:
>
> http://danmark:9000/browseid3.html?
> genre=*&artist=Herman%20Brood%20%26%20His%20Wild%20Romance&player=00:
> 0c:41:3a:90:29
>
> This may be the root of the problem.
>
> I'll look into it.
>
> -dean
>
>