Save channelUrl to Database if it doesn't exist

app/src/main/java/org/schabi/newpipe/database/stream/dao/StreamDAO.kt

@@ -6,6 +6,7 @@ import androidx.room.Insert
 import androidx.room.OnConflictStrategy
 import androidx.room.Query
 import androidx.room.Transaction
+import io.reactivex.rxjava3.core.Completable
 import io.reactivex.rxjava3.core.Flowable
 import org.schabi.newpipe.database.BasicDAO
 import org.schabi.newpipe.database.stream.model.StreamEntity
@@ -29,6 +30,9 @@ abstract class StreamDAO : BasicDAO<StreamEntity> {
     @Query("SELECT * FROM streams WHERE url = :url AND service_id = :serviceId")
     abstract fun getStream(serviceId: Long, url: String): Flowable<List<StreamEntity>>
 
+    @Query("UPDATE streams SET uploader_url = :uploaderUrl WHERE url = :url AND service_id = :serviceId")
+    abstract fun setUploaderUrl(serviceId: Long, url: String, uploaderUrl: String): Completable
+
     @Insert(onConflict = OnConflictStrategy.IGNORE)
     internal abstract fun silentInsertInternal(stream: StreamEntity): Long
 

app/src/main/java/org/schabi/newpipe/local/history/StatisticsPlaylistFragment.java

@@ -53,8 +53,6 @@ import io.reactivex.rxjava3.android.schedulers.AndroidSchedulers;
 import io.reactivex.rxjava3.disposables.CompositeDisposable;
 import io.reactivex.rxjava3.disposables.Disposable;
 
-import static org.schabi.newpipe.extractor.utils.Utils.isNullOrEmpty;
-
 public class StatisticsPlaylistFragment
         extends BaseLocalListFragment<List<StreamStatisticsEntry>, Void> {
     private final CompositeDisposable disposables = new CompositeDisposable();
@@ -363,10 +361,7 @@ public class StatisticsPlaylistFragment
         if (KoreUtils.shouldShowPlayWithKodi(context, infoItem.getServiceId())) {
             entries.add(StreamDialogEntry.play_with_kodi);
         }
-
+        entries.add(StreamDialogEntry.show_channel_details);
-        if (!isNullOrEmpty(infoItem.getUploaderUrl())) {
-            entries.add(StreamDialogEntry.show_channel_details);
-        }
 
         StreamDialogEntry.setEnabledEntries(entries);
 

app/src/main/java/org/schabi/newpipe/local/playlist/LocalPlaylistFragment.java

@@ -67,7 +67,6 @@ import io.reactivex.rxjava3.disposables.Disposable;
 import io.reactivex.rxjava3.schedulers.Schedulers;
 import io.reactivex.rxjava3.subjects.PublishSubject;
 
-import static org.schabi.newpipe.extractor.utils.Utils.isNullOrEmpty;
 import static org.schabi.newpipe.ktx.ViewUtils.animate;
 import static org.schabi.newpipe.util.ThemeHelper.shouldUseGridLayout;
 
@@ -778,10 +777,7 @@ public class LocalPlaylistFragment extends BaseLocalListFragment<List<PlaylistSt
         if (KoreUtils.shouldShowPlayWithKodi(context, infoItem.getServiceId())) {
             entries.add(StreamDialogEntry.play_with_kodi);
         }
-
+        entries.add(StreamDialogEntry.show_channel_details);
-        if (!isNullOrEmpty(infoItem.getUploaderUrl())) {
-            entries.add(StreamDialogEntry.show_channel_details);
-        }
 
         StreamDialogEntry.setEnabledEntries(entries);
 

app/src/main/java/org/schabi/newpipe/util/StreamDialogEntry.java

@@ -5,6 +5,7 @@ import android.net.Uri;
 
 import androidx.fragment.app.Fragment;
 
+import org.schabi.newpipe.NewPipeDatabase;
 import org.schabi.newpipe.R;
 import org.schabi.newpipe.extractor.stream.StreamInfoItem;
 import org.schabi.newpipe.local.dialog.PlaylistAppendDialog;
@@ -20,7 +21,9 @@ import java.util.Collections;
 import java.util.List;
 
 import io.reactivex.rxjava3.android.schedulers.AndroidSchedulers;
+import io.reactivex.rxjava3.schedulers.Schedulers;
 
+import static org.schabi.newpipe.extractor.utils.Utils.isNullOrEmpty;
 import static org.schabi.newpipe.player.MainPlayer.PlayerType.AUDIO;
 import static org.schabi.newpipe.player.MainPlayer.PlayerType.POPUP;
 
@@ -29,12 +32,29 @@ public enum StreamDialogEntry {
     // enum values with DEFAULT actions //
     //////////////////////////////////////
 
-    show_channel_details(R.string.show_channel_details, (fragment, item) ->
+    show_channel_details(R.string.show_channel_details, (fragment, item) -> {
-        // For some reason `getParentFragmentManager()` doesn't work, but this does.
+        if (isNullOrEmpty(item.getUploaderUrl())) {
-        NavigationHelper.openChannelFragment(
+            final int serviceId = item.getServiceId();
-                fragment.requireActivity().getSupportFragmentManager(),
+            final String url = item.getUrl();
-                item.getServiceId(), item.getUploaderUrl(), item.getUploaderName())
+            // TODO: Some visual loading indicator
-    ),
+            final String uploaderUrl = ExtractorHelper.getStreamInfo(serviceId, url, false)
+                    .subscribeOn(Schedulers.io())
+                    .blockingGet()
+                    .getUploaderUrl();
+            NewPipeDatabase.getInstance(fragment.getContext()).streamDAO()
+                    .setUploaderUrl(serviceId, url, uploaderUrl)
+                    .subscribeOn(Schedulers.io()).subscribe();
+            // For some reason `getParentFragmentManager()` doesn't work, but this does.
+            NavigationHelper.openChannelFragment(
+                    fragment.requireActivity().getSupportFragmentManager(),
+                    item.getServiceId(), uploaderUrl, item.getUploaderName());
+        } else {
+            // For some reason `getParentFragmentManager()` doesn't work, but this does.
+            NavigationHelper.openChannelFragment(
+                    fragment.requireActivity().getSupportFragmentManager(),
+                    item.getServiceId(), item.getUploaderUrl(), item.getUploaderName());
+        }
+    }),
 
     /**
      * Enqueues the stream automatically to the current PlayerType.<br>