Skip to content

Commit 4eaa3e9

Browse files
committed
Avoid null album names
1 parent 2046683 commit 4eaa3e9

File tree

1 file changed

+12
-10
lines changed

1 file changed

+12
-10
lines changed

app/src/main/kotlin/com/simplemobiletools/musicplayer/helpers/SimpleMediaScanner.kt

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -211,25 +211,27 @@ class SimpleMediaScanner(private val context: Application) {
211211
val title = cursor.getStringValue(Audio.Media.TITLE)
212212
val duration = cursor.getIntValue(Audio.Media.DURATION) / 1000
213213
val trackId = cursor.getIntValue(Audio.Media.TRACK) % 1000
214-
val path = cursor.getStringValue(Audio.Media.DATA)
214+
val path = cursor.getStringValue(Audio.Media.DATA).orEmpty()
215215
val artist = cursor.getStringValue(Audio.Media.ARTIST) ?: MediaStore.UNKNOWN_STRING
216-
val album = cursor.getStringValue(Audio.Media.ALBUM)
217-
val albumId = cursor.getLongValue(Audio.Media.ALBUM_ID)
218-
val artistId = cursor.getLongValue(Audio.Media.ARTIST_ID)
219-
val year = cursor.getIntValue(Audio.Media.YEAR)
220-
val dateAdded = cursor.getIntValue(Audio.Media.DATE_ADDED)
221216
val folderName = if (isQPlus()) {
222217
cursor.getStringValue(Audio.Media.BUCKET_DISPLAY_NAME) ?: MediaStore.UNKNOWN_STRING
223218
} else {
224219
""
225220
}
226221

222+
val album = cursor.getStringValue(Audio.Media.ALBUM) ?: folderName
223+
val albumId = cursor.getLongValue(Audio.Media.ALBUM_ID)
224+
val artistId = cursor.getLongValue(Audio.Media.ARTIST_ID)
225+
val year = cursor.getIntValue(Audio.Media.YEAR)
226+
val dateAdded = cursor.getIntValue(Audio.Media.DATE_ADDED)
227227
val coverUri = ContentUris.withAppendedId(artworkUri, albumId)
228228
val coverArt = coverUri.toString()
229229

230-
val track = Track(0, id, title, artist, path, duration, album, coverArt, 0, trackId, folderName, albumId, artistId, year, dateAdded, 0)
231-
track.title = track.getProperTitle(showFilename)
232-
tracks.add(track)
230+
if (!title.isNullOrEmpty()) {
231+
val track = Track(0, id, title, artist, path, duration, album, coverArt, 0, trackId, folderName, albumId, artistId, year, dateAdded, 0)
232+
track.title = track.getProperTitle(showFilename)
233+
tracks.add(track)
234+
}
233235
}
234236

235237
return tracks
@@ -277,7 +279,7 @@ class SimpleMediaScanner(private val context: Application) {
277279
context.queryCursor(uri, projection.toTypedArray(), null, null, showErrors = true) { cursor ->
278280
val id = cursor.getLongValue(Audio.Albums._ID)
279281
val artistName = cursor.getStringValue(Audio.Albums.ARTIST) ?: MediaStore.UNKNOWN_STRING
280-
val title = cursor.getStringValue(Audio.Albums.ALBUM)
282+
val title = cursor.getStringValue(Audio.Albums.ALBUM) ?: MediaStore.UNKNOWN_STRING
281283
val coverArt = ContentUris.withAppendedId(artworkUri, id).toString()
282284
val year = cursor.getIntValue(Audio.Albums.FIRST_YEAR)
283285
val trackCnt = cursor.getIntValue(Audio.Albums.NUMBER_OF_SONGS)

0 commit comments

Comments
 (0)