Bug 7537 - MusicIP icon in invalid locations
: MusicIP icon in invalid locations
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 7.0.1
: PC All
: P4 minor (vote)
: 7.x
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-03-17 15:00 UTC by Mark Miksis
Modified: 2009-07-31 10:18 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 Mark Miksis 2008-03-17 15:00:15 UTC
With MusicIP running, if I browse SC by year or genre, the M icon appears on the year or genre.  AFAIK, neither of these are valid MusicIP seed types.  If I click the M for a year, an empty playlist is always generated.  If I click the M for a genre, a playlist is created that seems unrelated to the genre in question.  Both of these should be removed.
Comment 1 KDF 2008-03-17 22:59:33 UTC
our %validMixTypes = (
        'track'    => 'song',
        'album'    => 'album',
        'artist'   => 'artist',
        'genre'    => 'genre',
        'mood'     => 'mood',
        'playlist' => 'playlist',
        'year'     => 'filter=?year',
);

You can test using the musicip API, but it will vary depending on your variety, etc settings.

from api:
filter=name	Use the named filter when creating the playlist (default none)
If there is no such filter, the name may refer to a genre as an implicit filter.

so, valid types for genre should probably be "filter=$genre"
Comment 2 Michael Herger 2008-03-17 23:41:40 UTC
mix by year does not seem to be working for me neither
Comment 3 Michael Herger 2008-03-18 04:06:27 UTC
Kevin - according to http://www.musicip.com/mixer/httpprotocol.jsp I don't see a way to filter by year or genre. IMHO the "filter" keyword would support the genre trick, but not the year. Did I miss something?
Comment 4 Michael Herger 2008-03-18 04:22:25 UTC
Looking at the history I see that the year mix has been added/removed more than once in the past few years. What's the secret behind it? It definitely doesn't work with my MIP 1.8. And I don't see why it should :-).

change 17912 - fixing the mixing by genre
Comment 5 Michael Herger 2008-03-18 04:30:28 UTC
http://localhost:10002/api/mix?filter=?year=1999 - "MusicIP API error - invalid request or internal error."

Replacing the second question mark with an ampersand:
http://localhost:10002/api/mix?filter=&year=1999 

/Users/mh/Music/iTunes/iTunes Music/Peter Gabriel/Growing Up (Summer 2003) - 06.26.03 Milwaukee/1-03 Secret World.mp3
... (many more)

Believe me, that track wasn't recorded in 1999 nor tagged with 1999 :-)

Secret World    
Artist: 	Peter Gabriel
Album: 	Growing Up (Summer 2003) - 06.26.03 Milwaukee (Disc 1)
Genre: 	Rock/Pop
Year: 	2003
Disc: 	1
Track: 	3
File Format: 	MP3
Duration: 	9:04
Comment: 	Live
File Length: 	8,724,274 Bytes
Bitrate: 	128kbps CBR
Sample Rate: 	44.1 kHz
ID3 Tag Version: 	ID3v2.3.0
Location: 	/Users/mh/Music/iTunes/iTunes Music/Peter Gabriel/Growing Up (Summer 2003) - 06.26.03 Milwaukee/1-03 Secret World.mp3 (Download)
Date Modified: 	Monday, 8. August 2005, 10:51
Comment 6 KDF 2008-03-18 08:37:12 UTC
this is where it comes from:
http://www.musicip.com/mixer/powersearch.jsp

Comment 7 KDF 2008-03-18 08:40:54 UTC
oh, and http:/<myhostname>:10002/api/mix?filter=?year=1999 gives me a mix.
MusicIP Mixer. Version 1.8
Comment 8 Michael Herger 2008-03-18 09:34:18 UTC
"Power Search is an advanced feature of MusicIP Mixer, and requires an active registration key." - you seem to be more power user than us :-).

I guess we'll have to disable this feature for 7.0.1 and see whether we can check for its availability in something post 7.0.1
Comment 9 Mark Miksis 2008-03-18 11:38:46 UTC
(In reply to comment #8)
> "Power Search is an advanced feature of MusicIP Mixer, and requires an active
> registration key." - you seem to be more power user than us :-).

I'm using the basic unregistered version.  I should also add that I'm running the headless MusicMagicServer, but hopefully that has the same CLI.

> 
> I guess we'll have to disable this feature for 7.0.1 and see whether we can
> check for its availability in something post 7.0.1
> 

Rather then check for it, maybe just add a checkbox to the MusicIP settings page for the user to activate power search.
Comment 10 Michael Herger 2008-03-19 06:55:29 UTC
change 17928 - add a simple check to determine wheter MIP's Power Search is available or not
Comment 11 Mark Miksis 2008-03-22 09:39:17 UTC
I finally got around to testing this.  Change 17928 seems to remove the M links from all pages.  (Only tested with Power Search not available.)
Comment 12 Michael Herger 2008-03-24 01:59:50 UTC
change 17964 - continue MIP initilization even if http connection or check for advanced features fails. Thanks for the heads up!
Comment 13 James Richardson 2008-05-15 12:25:25 UTC
This bug has recently been fixed in the latest release of SqueezeCenter 7.0.1

Please try that version, if you still see the error, then reopen this bug.

To download this version, please navigate to: http://www.slimdevices.com/su_downloads.html
Comment 14 Chris Owens 2009-07-31 10:18:12 UTC
Reduce number of active targets for SC