Skip to content

Commit 9244202

Browse files
committed
Share currently selected stream instead of default
1 parent 48a1215 commit 9244202

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
@@ -92,7 +92,6 @@
9292
import org.schabi.newpipe.views.LargeTextMovementMethod;
9393

9494
import java.io.Serializable;
95-
import java.util.ArrayList;
9695
import java.util.Collection;
9796
import java.util.LinkedList;
9897
import java.util.List;
@@ -696,24 +695,13 @@ public boolean onOptionsItemSelected(final MenuItem item) {
696695
}
697696
return true;
698697
case R.id.menu_item_share_stream:
699-
if (currentInfo == null) {
700-
return true;
701-
}
702-
final Context context = requireContext();
703-
ArrayList<VideoStream> videoStreamsList = new ArrayList<>(
704-
ListHelper.getSortedStreamVideosList(context, currentInfo.getVideoStreams(),
705-
null, false));
706-
int index = ListHelper.getDefaultResolutionIndex(context, videoStreamsList);
707-
708-
if (index == -1) {
709-
Toast.makeText(context, R.string.video_streams_empty, Toast.LENGTH_SHORT)
710-
.show();
711-
return true;
698+
if (currentInfo != null) {
699+
VideoStream selectedVideoStream = getSelectedVideoStream();
700+
if (selectedVideoStream != null) {
701+
ShareUtils.shareUrl(requireContext(), currentInfo.getName(),
702+
selectedVideoStream.getUrl());
703+
}
712704
}
713-
714-
VideoStream videoStream = videoStreamsList.get(index);
715-
716-
ShareUtils.shareUrl(context, currentInfo.getName(), videoStream.getUrl());
717705
return true;
718706
case R.id.menu_item_openInBrowser:
719707
if (currentInfo != null) {

0 commit comments

Comments
 (0)