Skip to content
This repository has been archived by the owner on Dec 28, 2022. It is now read-only.

Group albums into artists regardless of artist name capitalisation #498

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -67,15 +67,15 @@ public static List<AlbumArtist> albumsToAlbumArtists(List<Album> albums) {
AlbumArtist albumArtist = album.getAlbumArtist();

//Check if there's already an equivalent album-artist in our albumArtistMap
AlbumArtist oldAlbumArtist = albumArtistMap.get(albumArtist.name);
AlbumArtist oldAlbumArtist = albumArtistMap.get(albumArtist.name.toLowerCase());
if (oldAlbumArtist != null) {

//Add this album to the album artist's albums
if (!oldAlbumArtist.albums.contains(album)) {
oldAlbumArtist.albums.add(album);
}
} else {
albumArtistMap.put(albumArtist.name, albumArtist);
albumArtistMap.put(albumArtist.name.toLowerCase(), albumArtist);
}
}

Expand Down
2 changes: 1 addition & 1 deletion buildSrc/src/main/kotlin/dependencies/Dependencies.kt
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ object Dependencies {
const val crashlytics = "2.9.9"
const val dashClockApi = "2.0.0"
const val fastScroll = "1.0.20"
const val glide = "3.8.0-SNAPSHOT"
const val glide = "3.8.0"
const val glideOkhttp = "1.4.0@aar"
const val materialDialogs = "0.9.6.0"
const val permiso = "0.3.0"
Expand Down