Bug 8751 - MusicIP mix not available from album page in default/fishbone skin
: MusicIP mix not available from album page in default/fishbone skin
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 7.1
: PC Other
: P3 minor (vote)
: 7.x
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-07-15 23:38 UTC by Erland Isaksson
Modified: 2009-07-31 10:24 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments
working for me (54.64 KB, image/jpeg)
2008-07-16 01:10 UTC, Michael Herger
Details
fishbone too (47.42 KB, image/jpeg)
2008-07-16 01:11 UTC, Michael Herger
Details
Patch to solve the problem (1.18 KB, patch)
2008-07-16 02:17 UTC, Erland Isaksson
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Erland Isaksson 2008-07-15 23:38:01 UTC
In the default skin mixer icons is supposed to be shown in the page header in default/fishbone skins when you have selected an album. So on the right side of the play/add buttons mixer icons should appear.

This doesn't work for MusicIP mixes, its mix icons never appear in this place. The mix icons are shown on the previous page when listing albums and it is shown in the album page for all tracks, but it isn't shown in the header of the album page.

Some detailed information from my own investigation:
=========================
The Slim::Plugin::MusicMagic::Plugin::mixerlink function is called correctly and sets the mixermagic_mixable value in the $form correctly. In the album page it is called once for each track in the album and at last also one time for the album object itself.

The problem is that the $form doesn't seem to be forwarded correctly to for the album to plugins/MusicMagic/mixerlink.html for the album object. The mixerlink.html is processed for the album, but it doesn't get the form as the "item" parameter and due to this it doesn't show the mixer buttons in the header.

I'm not completely sure but I think the problem might be when songinfo_header.html is called with PROCESS in HTML/Default/browsedb.html, it should probably needs to somehow pass the $form parameter besides the parameters it passes already now.
Comment 1 Michael Herger 2008-07-16 01:10:33 UTC
Created attachment 3591 [details]
working for me
Comment 2 Michael Herger 2008-07-16 01:11:05 UTC
Created attachment 3592 [details]
fishbone too
Comment 3 Michael Herger 2008-07-16 01:11:51 UTC
Please make sure you've got the latest code as I've changed this bit yesterday.
Comment 4 Erland Isaksson 2008-07-16 02:17:49 UTC
Created attachment 3595 [details]
Patch to solve the problem

The problem is if you have several mixers, so I think it works as long as you only have the MusicIP mixer. But if you for example install the TrackStat plugin it will no longer be displayed.

I had the problem with the latest svn version.

This patch should solve the problem.
Comment 5 Michael Herger 2008-07-17 01:31:13 UTC
Will have to test with an additional mixer.
Comment 6 Michael Herger 2008-07-17 02:53:27 UTC
Just to be sure: should there be a TS icon on the track page too?
Comment 7 Michael Herger 2008-07-17 05:17:29 UTC
change 21820 - thanks!
Comment 8 Chris Owens 2008-07-30 15:31:34 UTC
This bug has now been fixed in the 7.1 release version of SqueezeCenter!  Please download the new version from http://www.slimdevices.com if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.
Comment 9 Chris Owens 2009-07-31 10:24:52 UTC
Reduce number of active targets for SC