Bug 2125 - Fishbone playlist not showing track (by) artist (from) album
: Fishbone playlist not showing track (by) artist (from) album
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Skins
: 6.2.0
: PC RedHat Linux
: P2 normal (vote)
: ---
Assigned To: Blackketter Dean
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-09-14 13:41 UTC by Robin Bowes
Modified: 2008-09-15 14:37 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments
expose the new version of artist and includeartist to the playlist (583 bytes, patch)
2005-09-14 14:22 UTC, KDF
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Robin Bowes 2005-09-14 13:41:29 UTC
Robin Bowes said:
>>>Go on, remind me how to get fishbone to display the playlist in the
>>>> >> format:
>>>> >>
>>>> >>     title (by) artist (from) album
>>>> >>
>>>> >> Where the words in brackets are in a different colour/font.
>>>> >>
>>>> >> At the moment I only see track (from) album.
>>>> >>
>>

KDF said:
>>> > To have them in the two-tone (ie, with component links) just use TITLE
>>> > as your titleformat setting, and the rest should be automatic, as long
>>> > as there is an artist tag. However, there was a HUGE set of changes
>>> > yesterday to handle Various Artists. I'm seeing it fine in firefox on my
>>> > setup, but i'll double-check that everything on my system is properly
>>> > merged to svn.
>>> >
>>> > keep me posted if you dig up any more details.
>

Robin Bowes said:
>> Ok, I've set titleformat to "TITLE" (selection 0).
>>
>> I'm still seeing just "title (from) album" in the playlist.
>>
>> It looks fine in the LHS, i.e. if I click on an album from the playlist,
>> the album contents are displayed correctly on the LHS as "title (by)
>> artist (from) album"

Finally, KDF said:
This looks like it might be a server issue.  I'm not finding any artist object
being delivered to any current playlist.

feel like filing a bug report?

So I did :)
Comment 1 KDF 2005-09-14 13:55:12 UTC
and it also looks like it affects all skins.

using:
[% IF includeArtist && artist; PROCESS artistsAsHTML.html; END %]

should be the right thing, since it is the same code used for the browse.
HOwever, both includeArtist and artist seem to have no value.

using:
[% PROCESS artistsAsHTML.html%]

works, but would then cause odd results when there really is no artist.  looks
like the includeArtist param has been lost somewhere.
Comment 2 KDF 2005-09-14 14:22:44 UTC
Created attachment 818 [details]
expose the new version of artist and includeartist to the playlist

This will allow the use of:
[% IF includeArtist && artist; PROCESS artistsAsHTML.html; END %]

in the status_list for skins. all of the skins would have to be patched to use
this.  assign to me if this patch is ok, and I'll take care of all of the skins
before merging.
Comment 3 KDF 2005-09-14 20:23:50 UTC
should be fixed at change 4288, for Sept 15 build. reopen if issues remain.
Comment 4 Chris Owens 2006-06-16 14:41:29 UTC
There are 536 bugs in the database with targets of '---' that were fixed prior to new year 2006.  I am setting them to targets of 6.2.1 to keep them from showing up in my queries.