Fix #646: Sort artist discography by full date

backend/mediaprovider/model.go

@@ -38,6 +38,26 @@ type ItemDate struct {
 	Day   *int
 }
 
+func (i ItemDate) After(other ItemDate) bool {
+	a := []*int{i.Year, i.Month, i.Day}
+	b := []*int{other.Year, other.Month, other.Day}
+	for i := range a {
+		if a[i] == nil {
+			return false
+		}
+		if b[i] == nil {
+			return true
+		}
+		if *a[i] < *b[i] {
+			return false
+		}
+		if *a[i] > *b[i] {
+			return true
+		}
+	}
+	return false
+}
+
 type Album struct {
 	ID           string
 	CoverArtID   string

ui/browsing/artistpage.go

@@ -195,12 +195,12 @@ func (a *ArtistPage) getGridViewAlbumsModel() []widgets.GridViewItemModel {
 		return nil
 	}
 	sortFunc := func(x, y int) bool {
-		return a.artistInfo.Albums[x].YearOrZero() < a.artistInfo.Albums[y].YearOrZero()
+		return a.artistInfo.Albums[y].Date.After(a.artistInfo.Albums[x].Date)
 	}
 	switch a.cfg.DiscographySort {
 	case discographySorts[1]: /*year descending*/
 		sortFunc = func(x, y int) bool {
-			return a.artistInfo.Albums[x].YearOrZero() > a.artistInfo.Albums[y].YearOrZero()
+			return a.artistInfo.Albums[x].Date.After(a.artistInfo.Albums[y].Date)
 		}
 	case discographySorts[2]: /*name*/
 		sortFunc = func(x, y int) bool {