kdf Wrote:
> On 15-Oct-05, at 3:26 PM, gerph wrote:
>
> >
> > Whilst looking at the source for 6.2 I notice that there isn't any
> > option to display the artist name with the album name. That is, the
> > current Browse by year might look like this :
>
> This option does actually exist in 6.2. In server
> settings->formatting. The preference is called "Show Artist with
> Albums".
> >
> > "1970" -> "The Best Of (Disc 1) by Ralph McTell"
> >
> This is what I see right now. latest from svn.
>
> -kdf
Ok, I must have misunderstood both what that option means(1) and the
code that implements it, because it looked to me like browsedbItemName
only special cased that for level=0, exactly the same as the 6.1.1
source :
Code:
--------------------
} elsif (($levels[$level] eq 'album') && $level == 0) {
...
if (Slim::Utils::Prefs::get('showArtist')) {
... do the expansion bits ...
--------------------
If you say it works, I'll accept that; it certainly didn't look like it
did from that code and the $fieldInfo{'album'}->{'resultToName'} doesn't
do anything special as the fallback case.
At some point I'll install 6.2, but for now I'll just accept that
something else magical is going on that makes it work for you but which
I can't see in the code.
(1) My understanding of 'show artist with album' is that it affects
only the albums at the top level, not all albums everywhere. If it
affected all albums everywhere it would just be annoying.
--
gerph
_______________________________________________
beta mailing list
[email protected]
http://lists.slimdevices.com/lists/listinfo/beta
