Merge branch 'master' into playlist

src/components/ChannelPage.vue

@@ -49,7 +49,7 @@ export default {
     data() {
         return {
             channel: null,
-            subscribed: this.authenticated ? false : this.isSubscribedLocally(this.channelId),
+            subscribed: false,
         };
     },
     mounted() {
@@ -68,7 +68,11 @@ export default {
     },
     methods: {
         async fetchSubscribedStatus() {
-            if (!this.channelId || !this.authenticated) return;
+            if (!this.channel.id) return;
+            if (!this.authenticated) {
+                this.subscribed = this.isSubscribedLocally(this.channel.id);
+                return;
+            }
 
             this.fetchJson(
                 this.authApiUrl() + "/subscribed",
@@ -94,7 +98,7 @@ export default {
                 .then(() => {
                     if (!this.channel.error) {
                         document.title = this.channel.name + " - Piped";
-                        if (this.authenticated) this.fetchSubscribedStatus();
+                        this.fetchSubscribedStatus();
                         this.updateWatched(this.channel.relatedStreams);
                     }
                 });

src/components/ImportPage.vue

@@ -155,7 +155,7 @@ export default {
         importSubscriptionsLocally(newChannels) {
             const subscriptions = this.override
                 ? [...new Set(newChannels)]
-                : [...new Set(this.getLocalSubscriptions().concat(newChannels))];
+                : [...new Set((this.getLocalSubscriptions() ?? []).concat(newChannels))];
             // Sort for better cache hits
             subscriptions.sort();
             localStorage.setItem("localSubscriptions", JSON.stringify(subscriptions));

src/components/VideoPlayer.vue

@@ -85,7 +85,7 @@ export default {
         this.hotkeysPromise.then(() => {
             var self = this;
             this.$hotkeys(
-                "f,m,j,k,l,c,space,up,down,left,right,0,1,2,3,4,5,6,7,8,9,shift+,,shift+.",
+                "f,m,j,k,l,c,space,up,down,left,right,0,1,2,3,4,5,6,7,8,9,shift+n,shift+,,shift+.",
                 function (e, handler) {
                     const videoEl = self.$refs.videoEl;
                     switch (handler.key) {
@@ -171,6 +171,10 @@ export default {
                             videoEl.currentTime = videoEl.duration * 0.9;
                             e.preventDefault();
                             break;
+                        case "shift+n":
+                            self.navigateNext();
+                            e.preventDefault();
+                            break;
                         case "shift+,":
                             self.$player.trickPlay(Math.max(videoEl.playbackRate - 0.25, 0.25));
                             break;
@@ -225,8 +229,11 @@ export default {
                 var request = store.get(this.video.id);
                 request.onsuccess = function (event) {
                     var video = event.target.result;
-                    if (video?.currentTime) {
+                    const currentTime = video?.currentTime;
-                        videoEl.currentTime = video.currentTime;
+                    if (currentTime) {
+                        if (currentTime < component.video.duration * 0.9) {
+                            videoEl.currentTime = currentTime;
+                        }
                     }
                 };
 
@@ -372,30 +379,12 @@ export default {
                 });
 
                 videoEl.addEventListener("ended", () => {
-                    if (!this.selectedAutoLoop && this.selectedAutoPlay && this.video.relatedStreams.length > 0) {
+                    if (
-                        const params = this.$route.query;
+                        !this.selectedAutoLoop &&
-                        let url = this.playlist?.relatedStreams?.[this.index]?.url ?? this.video.relatedStreams[0].url;
+                        this.selectedAutoPlay &&
-                        const searchParams = new URLSearchParams();
+                        (this.playlist?.relatedStreams?.length > 0 || this.video.relatedStreams.length > 0)
-                        for (var param in params)
+                    ) {
-                            switch (param) {
+                        this.navigateNext();
-                                case "v":
-                                case "t":
-                                    break;
-                                case "index":
-                                    if (this.index < this.playlist.relatedStreams.length)
-                                        searchParams.set("index", this.index + 1);
-                                    break;
-                                case "list":
-                                    if (this.index < this.playlist.relatedStreams.length)
-                                        searchParams.set("list", params.list);
-                                    break;
-                                default:
-                                    searchParams.set(param, params[param]);
-                                    break;
-                            }
-                        const paramStr = searchParams.toString();
-                        if (paramStr.length > 0) url += "&" + paramStr;
-                        this.$router.push(url);
                     }
                 });
             }
@@ -558,6 +547,29 @@ export default {
                 this.$refs.videoEl.currentTime = time;
             }
         },
+        navigateNext() {
+            const params = this.$route.query;
+            let url = this.playlist?.relatedStreams?.[this.index]?.url ?? this.video.relatedStreams[0].url;
+            const searchParams = new URLSearchParams();
+            for (var param in params)
+                switch (param) {
+                    case "v":
+                    case "t":
+                        break;
+                    case "index":
+                        if (this.index < this.playlist.relatedStreams.length) searchParams.set("index", this.index + 1);
+                        break;
+                    case "list":
+                        if (this.index < this.playlist.relatedStreams.length) searchParams.set("list", params.list);
+                        break;
+                    default:
+                        searchParams.set(param, params[param]);
+                        break;
+                }
+            const paramStr = searchParams.toString();
+            if (paramStr.length > 0) url += "&" + paramStr;
+            this.$router.push(url);
+        },
         updateMarkers() {
             const markers = this.$refs.container.querySelector(".shaka-ad-markers");
             const array = ["to right"];

src/components/WatchVideo.vue

@@ -427,7 +427,10 @@ export default {
         },
         async fetchSubscribedStatus() {
             if (!this.channelId) return;
-            if (!this.authenticated) this.subscribed = this.isSubscribedLocally(this.channelId);
+            if (!this.authenticated) {
+                this.subscribed = this.isSubscribedLocally(this.channelId);
+                return;
+            }
 
             this.fetchJson(
                 this.authApiUrl() + "/subscribed",

src/main.js

@@ -217,8 +217,8 @@ const mixin = {
             localStorage.setItem("localSubscriptions", JSON.stringify(localSubscriptions));
         },
         getUnauthenticatedChannels() {
-            const localSubscriptions = this.getLocalSubscriptions();
+            const localSubscriptions = this.getLocalSubscriptions() ?? [];
-            return localSubscriptions != null ? localSubscriptions.join(",") : "";
+            return localSubscriptions.join(",");
         },
         /* generate a temporary file and ask the user to download it */
         download(text, filename, type) {