Merge pull request #8127 from litetex/fix-SparseItemUtil

Fix `SparseItemUtil` so we don't enqueue twice
This commit is contained in:
Robin 2022-04-07 17:21:18 +02:00 committed by GitHub
commit a528cee5f4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 4 deletions

View File

@ -54,6 +54,7 @@ public final class SparseItemUtil {
// if the duration is >= 0 (provided that the item is not a livestream) and there is an // if the duration is >= 0 (provided that the item is not a livestream) and there is an
// uploader url, probably all info is already there, so there is no need to fetch it // uploader url, probably all info is already there, so there is no need to fetch it
callback.accept(new SinglePlayQueue(item)); callback.accept(new SinglePlayQueue(item));
return;
} }
// either the duration or the uploader url are not available, so fetch more info // either the duration or the uploader url are not available, so fetch more info
@ -80,12 +81,12 @@ public final class SparseItemUtil {
@NonNull final String url, @NonNull final String url,
@Nullable final String uploaderUrl, @Nullable final String uploaderUrl,
@NonNull final Consumer<String> callback) { @NonNull final Consumer<String> callback) {
if (isNullOrEmpty(uploaderUrl)) { if (!isNullOrEmpty(uploaderUrl)) {
fetchStreamInfoAndSaveToDatabase(context, serviceId, url,
streamInfo -> callback.accept(streamInfo.getUploaderUrl()));
} else {
callback.accept(uploaderUrl); callback.accept(uploaderUrl);
return;
} }
fetchStreamInfoAndSaveToDatabase(context, serviceId, url,
streamInfo -> callback.accept(streamInfo.getUploaderUrl()));
} }
/** /**