Bug 3206 - Search music page has invalid link for album under "Songs matching x"
: Search music page has invalid link for album under "Songs matching x"
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 6.5b1
: PC Windows XP
: P2 normal (vote)
: ---
Assigned To: Dan Sully
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-03-26 10:55 UTC by Tom Schmidt
Modified: 2006-03-31 09:02 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments
Source from IE frame (4.04 KB, text/plain)
2006-03-30 19:06 UTC, Tom Schmidt
Details
Screen shot of the problem (2.28 MB, image/bmp)
2006-03-30 19:13 UTC, Tom Schmidt
Details
Screen shot of problem (101.23 KB, image/jpeg)
2006-03-30 19:17 UTC, Tom Schmidt
Details
bring back full album url and playerURI (935 bytes, patch)
2006-03-30 23:39 UTC, KDF
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tom Schmidt 2006-03-26 10:55:12 UTC
From the home page select "Search Music"
Enter a search term which will match a song name
Under the "Song titles matching "search term":"
However over the link for the album the song came from, notice the link is in the form http://localhost:9000/album,track&level=1&album=164
This appears not to be a valid URL and fails when it is selected
Comment 1 KDF 2006-03-26 11:57:00 UTC
which skin?
Comment 2 KDF 2006-03-26 12:42:47 UTC
as a guess, I tried Fishbone, Default and Nokia770. I have been unable to reproduce using live and manual search results.
Comment 3 Tom Schmidt 2006-03-28 21:19:11 UTC
Default skin
I figured out something that I didn't realize in the initial report.  If you enter a search term and press the "search music" button everything works fine.  If you enter a search term and just wait for it to dynamically start matching the search as you type it creates the broken links.
Comment 4 KDF 2006-03-28 22:56:09 UTC
waiting for the dynamic results is referred to as live search.  
This is the url I get:
http://serverIP:9000/browsedb.html?hierarchy=artist,album,track&level=1&artist=78

the missing bit in your case is an odd thing.  The missing section isn't exactly matching the divisions of the bits that make up the final url.

have you recently upgraded from 6.2.1? 6.2.2?  What browser do you use? In this example you've given, is the album, or any part of the search result containing any accented characters?

and one other leap, to see if there is anything else to be seen, perhaps attach a copy of the page source (right click, view source or view frame source if firefox)

 
Comment 5 Tom Schmidt 2006-03-30 19:06:55 UTC
Created attachment 1172 [details]
Source from IE frame
Comment 6 Tom Schmidt 2006-03-30 19:10:53 UTC
Yes, I have recently upgraded from 6.2.1

I've tried both IE 6.0 and FireFox 1.5

None of the albums in the search contain special characters, it actually happens on every album in the library.

I've attached the source for the page and a screen shot, notice the url in the lower left.  I get the correct URL when I hover over a song name or artist.
Comment 7 Tom Schmidt 2006-03-30 19:13:13 UTC
Created attachment 1173 [details]
Screen shot of the problem
Comment 8 Tom Schmidt 2006-03-30 19:17:20 UTC
Created attachment 1174 [details]
Screen shot of problem
Comment 9 KDF 2006-03-30 23:39:50 UTC
Created attachment 1175 [details]
bring back full album url and playerURI

ah, I see it now.  back at change 4518, the url got cut off somehow.  
This brings back the full url and also adds the playerURI.

I'm attaching here becuase this problem also exists in 6.2.2. I'll commit to trunk shortly for 6.5, and await review for 6.2.2
Comment 10 KDF 2006-03-31 00:02:41 UTC
fixed in trunk at change 6774 for 6.5 March 31 build.
should be applied to 6.2.2 if the patch looks ok, or the 6.5 fix is confirmed.
Comment 11 Dan Sully 2006-03-31 08:58:15 UTC
Please commit for 6.2.2 as well.

Thanks.
Comment 12 KDF 2006-03-31 09:02:23 UTC
done.  change 6779 for April 1 build of 6.2.2