From 6758cdc82e4cb295123b0a84819c6a6ebd5ebad0 Mon Sep 17 00:00:00 2001
From: dec05eba <dec05eba@protonmail.com>
Date: Fri, 15 Jul 2022 12:48:38 +0200
Subject: Youtube: do not autoplay next if starting qm with youtube url,
 opening youtube url in info page or matrix

---
 src/QuickMedia.cpp       | 12 +++++++++---
 src/plugins/Info.cpp     |  2 +-
 src/plugins/Mangadex.cpp |  3 +++
 src/plugins/NyaaSi.cpp   |  2 +-
 src/plugins/Youtube.cpp  |  2 +-
 5 files changed, 15 insertions(+), 6 deletions(-)

(limited to 'src')

diff --git a/src/QuickMedia.cpp b/src/QuickMedia.cpp
index 147afd9..e710fb5 100644
--- a/src/QuickMedia.cpp
+++ b/src/QuickMedia.cpp
@@ -1258,7 +1258,7 @@ namespace QuickMedia {
                 tabs.push_back(Tab{std::move(history_body), std::move(history_page), create_search_bar("Search...", SEARCH_DELAY_FILTER)});
             } else {
                 current_page = PageType::VIDEO_CONTENT;
-                auto youtube_video_page = std::make_unique<YoutubeVideoPage>(this, youtube_url);
+                auto youtube_video_page = std::make_unique<YoutubeVideoPage>(this, youtube_url, false);
                 video_content_page(nullptr, youtube_video_page.get(), "", false, nullptr, 0);
             }
         } else if(strcmp(plugin_name, "peertube") == 0) {
@@ -3113,7 +3113,7 @@ namespace QuickMedia {
 
         BodyItems related_videos;
         bool move_in_parent = false;
-        if(parent_body && video_page->autoplay_next_item() && play_index + 1 >= 0 && play_index + 1 < (int)parent_body->get_num_items()) {
+        if(parent_body && video_page->autoplay_next_item() && video_page->should_autoplay() && play_index + 1 >= 0 && play_index + 1 < (int)parent_body->get_num_items()) {
             parent_body->copy_range(play_index + 1, (size_t)-1, related_videos);
             move_in_parent = true;
         }
@@ -3568,6 +3568,12 @@ namespace QuickMedia {
             } else if(update_err == VideoPlayer::Error::EXITED && video_player->exit_status == 0 && (!is_matrix || is_youtube)) {
                 std::string new_video_url;
 
+                if(!video_page->should_autoplay()) {
+                    current_page = previous_page;
+                    go_to_previous_page = true;
+                    break;
+                }
+
                 if(related_videos_task.valid()) {
                     TaskResult task_result = run_task_with_loading_screen([&]() {
                         while(!program_is_dead_in_current_thread()) {
@@ -6270,7 +6276,7 @@ namespace QuickMedia {
                 watched_videos.clear();
                 page_stack.push(PageType::CHAT);
                 current_page = PageType::VIDEO_CONTENT;
-                auto youtube_video_page = std::make_unique<YoutubeVideoPage>(this, std::move(url));
+                auto youtube_video_page = std::make_unique<YoutubeVideoPage>(this, std::move(url), false);
                 // TODO: Use real title
                 video_content_page(matrix_chat_page, youtube_video_page.get(), "", false, tabs[MESSAGES_TAB_INDEX].body.get(), tabs[MESSAGES_TAB_INDEX].body->get_selected_item());
                 redraw = true;
diff --git a/src/plugins/Info.cpp b/src/plugins/Info.cpp
index 90c8fe1..252cbea 100644
--- a/src/plugins/Info.cpp
+++ b/src/plugins/Info.cpp
@@ -42,7 +42,7 @@ namespace QuickMedia {
             const std::string search_url = "https://www.google.com/search?q=" + url_param_encode(search_term);
             return open_with_browser(search_url);
         } else if(is_youtube_url(args.url)) {
-            result_tabs.push_back(Tab{nullptr, std::make_unique<YoutubeVideoPage>(program, args.url), nullptr});
+            result_tabs.push_back(Tab{nullptr, std::make_unique<YoutubeVideoPage>(program, args.url, false), nullptr});
             return PluginResult::OK;
         } else {
             return open_with_browser(args.url);
diff --git a/src/plugins/Mangadex.cpp b/src/plugins/Mangadex.cpp
index 2efd2f1..c100b0b 100644
--- a/src/plugins/Mangadex.cpp
+++ b/src/plugins/Mangadex.cpp
@@ -107,8 +107,11 @@ namespace QuickMedia {
 
             std::string title;
             const Json::Value &title_en_json = title_json["en"];
+            const Json::Value &title_ja_json = title_json["ja"];
             if(title_en_json.isString())
                 title = title_en_json.asString();
+            else if(title_ja_json.isString())
+                title = title_ja_json.asString();
             else
                 title = "No title"; // TODO: Verify if this happens. If it happens, get the title in any available language
 
diff --git a/src/plugins/NyaaSi.cpp b/src/plugins/NyaaSi.cpp
index fc3e7ee..fd24e71 100644
--- a/src/plugins/NyaaSi.cpp
+++ b/src/plugins/NyaaSi.cpp
@@ -328,7 +328,7 @@ namespace QuickMedia {
         std::string title;
         BodyItems result_items;
 
-        auto torrent_item = BodyItem::create("💾 Download magnet");
+        auto torrent_item = BodyItem::create("💾 Download");
         std::string magnet_url;
         std::string description;
 
diff --git a/src/plugins/Youtube.cpp b/src/plugins/Youtube.cpp
index 1b46ed6..7552967 100644
--- a/src/plugins/Youtube.cpp
+++ b/src/plugins/Youtube.cpp
@@ -1883,7 +1883,7 @@ namespace QuickMedia {
         return;
     }
 
-    YoutubeVideoPage::YoutubeVideoPage(Program *program, std::string url) : VideoPage(program, "") {
+    YoutubeVideoPage::YoutubeVideoPage(Program *program, std::string url, bool autoplay) : VideoPage(program, "", autoplay) {
         set_url(std::move(url));
     }
 
-- 
cgit v1.2.3