Bug 14266 - Many browse menus don't work in the mysb.com web remote
: Many browse menus don't work in the mysb.com web remote
Status: CLOSED FIXED
Product: MySqueezebox.com
Classification: Unclassified
Component: Web UI
: MySB
: PC Windows XP
: P1 critical (vote)
: INXS
Assigned To: Andy Grundman
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-09-24 22:25 UTC by Michael Herger
Modified: 2010-02-08 09:56 UTC (History)
1 user (show)

See Also:
Category: Bug


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Herger 2009-09-24 22:25:29 UTC
The following (and probably more) don't work:
- last.fm Start Station (tags, artist)
- podcasts
- Internet Radio menu (picks etc.) - the same menus browsed through RadioTime are working fine though
Comment 1 SVN Bot 2009-09-25 02:02:09 UTC
 == Auto-comment from SVN commit #7426 to the network repo by michael ==
 == https://svn.slimdevices.com/network?view=revision&revision=7426 ==

Bug: 14266
Description: remove debug statement leading to failure in search menus in some browsers
Comment 2 Michael Herger 2009-09-25 02:13:32 UTC
Andy - there's a comment in Browse/OPML, around line 230:

# XXX: Index may be a search query and not numeric here

That's exactly what we're tripping over when searching from Internet Radio
Comment 3 Michael Herger 2009-09-25 02:14:46 UTC
some more details: searching for eg. "drs" results in the following query:

http://localhost:3000/browse/radio/4_drs?_dc=1253869721832

causing the following error message:

Argument "\x{34}\x{5f}..." isn't numeric in array element at /Users/mh/Documents/workspace/SN/script/../lib/SDI/SqueezeNetwork/Controller/Browse/OPML.pm line 406.

That hex expression is the leading "4_"
Comment 4 Michael Herger 2009-09-25 02:56:36 UTC
Ok, and about the failure to browse picks: at some point the valid external url http://www.slimdevices.com/picks/split/Best%20of%20Internet%20Radio.opml is reduced to the path without the external host name to fetch the playlist. It's considered a subreq, which are always considered local. This obviously doesn't work:

[debug] Path is "public/radio/picks"
[debug] Language set from Accept-Language: DE, DE, DE, EN, EN, EN
[debug] Language set from cookie/select box: EN, EN
[debug] Cache headers set: 3600 / Fri, 25 Sep 2009 10:52:20 GMT
[debug] OPML: Subfeed got 4 items
[debug] OPML: Getting subfeed data from http://www.slimdevices.com/picks/split/Best%20of%20Internet%20Radio.opml
[debug] "GET" request for "picks/split/Best%20of%20Internet%20Radio.opml" from "127.0.0.1"
[debug] Arguments are "picks/split/Best of Internet Radio.opml"
[debug] Language set from Accept-Language: DE, DE, DE, EN, EN, EN
[debug] Found user from cookie: slim@herger.net
[debug] Language set from user account: EN, EN
[debug] Language set from cookie/select box: EN, EN
[debug] User roles: -commercial, classical, fakeip, napster, queen, stats, tier1, tier2
[debug] Rendering template "home.tt"
Comment 5 SVN Bot 2009-09-25 05:22:03 UTC
 == Auto-comment from SVN commit #7427 to the network repo by andy ==
 == https://svn.slimdevices.com/network?view=revision&revision=7427 ==

Bug 14266, fix warning when index contains search query
Comment 6 SVN Bot 2009-09-25 05:40:44 UTC
 == Auto-comment from SVN commit #7428 to the network repo by andy ==
 == https://svn.slimdevices.com/network?view=revision&revision=7428 ==

Fixed bug 14266, change the way local vs remote is determined, so browsing remote under a local path works right
Comment 7 James Richardson 2009-10-05 16:42:27 UTC
This bug has been fixed in the latest release of MySqueezebox.com (formally known as SqueezeNetwork)!

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.
Comment 8 Chris Owens 2010-02-08 09:56:46 UTC
These are all bugs that have been marked 'fixed' or 'closed' but still have the bug_meeting keyword.  They were not showing up in James's bug meeting search.

Please let me know (chris_owens@logitech.com) if these need to be brought up at the next bug meeting.  In the meantime I will fix the search.