fixed #4623

[knighthat]

I put a condition to remove explicit prefix during sort to make it more accurate.

Question

Why are we using prefixes instead of having a boolean column for these? For example

From

songId	title	artistsText	durationText	thumbnailUrl	likedAt	totalPlayTimeMs
id1	artist	title goes here	2:59	https://example.com	null	0
id1	artist	e:title goes here	2:59	https://example.com	null	0
id1	artist	e:title goes here	2:59	https://example.com	null	0

To

songId	title	artistsText	durationText	thumbnailUrl	likedAt	totalPlayTimeMs	isExplicit
id1	artist	title goes here	2:59	https://example.com	null	0	false
id1	artist	title goes here	2:59	https://example.com	null	0	true
id1	artist	title goes here	2:59	https://example.com	null	0	true

[fast4x]

Because i don't want to modify db structure.

[fast4x]

Why have removed @SuppressWarnings(RoomWarnings.QUERY_MISMATCH)

[knighthat]

Is it throwing error or something?

It ready returns all columns from Song table as Song object and the other 2 are indicated in @Query?

Or am I mistaken here?

I think there's @Embeded to add to Song in SongEntity to remove the warning

[fast4x]

It remove warning for non all fields from SongEntity was returned

[fast4x]

Is it throwing error or something?

It ready returns all columns from Song table as Song object and the other 2 are indicated in @Query?

Or am I mistaken here?

I think there's @Embeded to add to Song in SongEntity to remove the warning

Song have just embedded song...

[knighthat]

yes, that's what I was talking about. The only column's missing form the query is contentLength, do you want me too make it null or fetch from Format?

[fast4x]

Fetch from format it's a good idea

[knighthat]

K, expect new commit(s) in a few hrs

[knighthat]

Done, contentLength from Format table is now a part of the result

[fast4x]

Nice thanks