Bugzilla – Bug 2626
option to show year with 'browse artists'
Last modified: 2008-09-15 14:37:04 UTC
I cannot reopen bug 739 so I am opening a new bug. The bug was not fixed; the request was to show the year when browsing artists. However, the year is only displayed when browsing by album, due to the code in server/Slim/Buttons/BrowseDB.pm being surrounded by the condition "$level == 0", i.e.: } elsif (($levels[$level] eq 'album') && $level == 0) { my @name = &{$levelInfo->{'resultToName'}}($item); if (Slim::Utils::Prefs::get('showYear')) { my $year = $item->year; push @name, " ($year)" if $year; } I believe this condition should be moved down into subsequent condition which checks the "showArtist" option, though even that may be questionable.
CC KDF since he made the fix for the original bug.
Created attachment 1051 [details] show artist/year in any album context and not previously selected. The bug was fixed, but not kept the same when converting to browsedb. The real solution here is to do the same as is done in the web interface (which is the same policy as before). This means, show artit and year if pref is set for any album context where the artist or year is not already known (ie browse artist -> album = don't show artist)
Ah, yeah, that sounds even better.. thanks KDF
committed to 6.5b1 builds at change 5305 awaiting word on whether this can go into 6.2.2 or not.
looking at this, it IS more a feature add than a bug fix, so leaving this for 6.5 (given the last quoted timeline of March-ish, it wont be a long wait anyway)
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.