Skip to content

Commit 8248bc1

Browse files
author
Jad Kik
committed
Share currently selected stream instead of default
1 parent 25f2ff5 commit 8248bc1

1 file changed

Lines changed: 6 additions & 18 deletions

File tree

app/src/main/java/org/schabi/newpipe/fragments/detail/VideoDetailFragment.java

Lines changed: 6 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,6 @@
8888
import org.schabi.newpipe.views.AnimatedProgressBar;
8989

9090
import java.io.Serializable;
91-
import java.util.ArrayList;
9291
import java.util.Collection;
9392
import java.util.LinkedList;
9493
import java.util.List;
@@ -645,24 +644,13 @@ public boolean onOptionsItemSelected(final MenuItem item) {
645644
}
646645
return true;
647646
case R.id.menu_item_share_stream:
648-
if (currentInfo == null) {
649-
return true;
650-
}
651-
final Context context = requireContext();
652-
ArrayList<VideoStream> videoStreamsList = new ArrayList<>(
653-
ListHelper.getSortedStreamVideosList(context, currentInfo.getVideoStreams(),
654-
null, false));
655-
int index = ListHelper.getDefaultResolutionIndex(context, videoStreamsList);
656-
657-
if (index == -1) {
658-
Toast.makeText(context, R.string.video_streams_empty, Toast.LENGTH_SHORT)
659-
.show();
660-
return true;
647+
if (currentInfo != null) {
648+
VideoStream selectedVideoStream = getSelectedVideoStream();
649+
if (selectedVideoStream != null) {
650+
ShareUtils.shareUrl(requireContext(), currentInfo.getName(),
651+
selectedVideoStream.getUrl());
652+
}
661653
}
662-
663-
VideoStream videoStream = videoStreamsList.get(index);
664-
665-
ShareUtils.shareUrl(context, currentInfo.getName(), videoStream.getUrl());
666654
return true;
667655
case R.id.menu_item_openInBrowser:
668656
if (currentInfo != null) {

0 commit comments

Comments
 (0)