Skip to content

Commit 6af2242

Browse files
authored
Merge pull request TeamNewPipe#9521 from pratyaksh1610/branch-9518
Fixes TeamNewPipe#9518: Crash fix when click "Add to playlist" while the current list is still loading
2 parents 27f6c3b + d21fac6 commit 6af2242

1 file changed

Lines changed: 13 additions & 13 deletions

File tree

app/src/main/java/org/schabi/newpipe/fragments/list/playlist/PlaylistFragment.java

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -230,24 +230,24 @@ public boolean onOptionsItemSelected(final MenuItem item) {
230230
ShareUtils.openUrlInBrowser(requireContext(), url);
231231
break;
232232
case R.id.menu_item_share:
233-
if (currentInfo != null) {
234-
ShareUtils.shareText(requireContext(), name, url,
235-
currentInfo.getThumbnailUrl());
236-
}
233+
ShareUtils.shareText(requireContext(), name, url,
234+
currentInfo == null ? null : currentInfo.getThumbnailUrl());
237235
break;
238236
case R.id.menu_item_bookmark:
239237
onBookmarkClicked();
240238
break;
241239
case R.id.menu_item_append_playlist:
242-
disposables.add(PlaylistDialog.createCorrespondingDialog(
243-
getContext(),
244-
getPlayQueue()
245-
.getStreams()
246-
.stream()
247-
.map(StreamEntity::new)
248-
.collect(Collectors.toList()),
249-
dialog -> dialog.show(getFM(), TAG)
250-
));
240+
if (currentInfo != null) {
241+
disposables.add(PlaylistDialog.createCorrespondingDialog(
242+
getContext(),
243+
getPlayQueue()
244+
.getStreams()
245+
.stream()
246+
.map(StreamEntity::new)
247+
.collect(Collectors.toList()),
248+
dialog -> dialog.show(getFM(), TAG)
249+
));
250+
}
251251
break;
252252
default:
253253
return super.onOptionsItemSelected(item);

0 commit comments

Comments
 (0)