From 157e1219bbb92415fb0bb243f1a9b5868a552cab Mon Sep 17 00:00:00 2001 From: Bart De Vries Date: Sat, 2 Oct 2021 20:32:36 +0200 Subject: [PATCH] Fix 'Allow Once' feed update after it was broken by refactoring --- src/fetchfeedsjob.cpp | 2 ++ src/qml/ConnectionCheckAction.qml | 2 -- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/fetchfeedsjob.cpp b/src/fetchfeedsjob.cpp index 5d070101..4115a63e 100644 --- a/src/fetchfeedsjob.cpp +++ b/src/fetchfeedsjob.cpp @@ -35,12 +35,14 @@ void FetchFeedsJob::start() void FetchFeedsJob::fetch() { + /* We remove this because otherwise 'Allow Once' would not work ... if (Fetcher::instance().isMeteredConnection() && !SettingsManager::self()->allowMeteredFeedUpdates()) { setError(0); setErrorText(i18n("Podcast updates not allowed due to user setting")); emitResult(); return; } + */ setTotalAmount(KJob::Unit::Items, m_urls.count()); setProcessedAmount(KJob::Unit::Items, 0); diff --git a/src/qml/ConnectionCheckAction.qml b/src/qml/ConnectionCheckAction.qml index f268fabd..644ac11a 100644 --- a/src/qml/ConnectionCheckAction.qml +++ b/src/qml/ConnectionCheckAction.qml @@ -30,9 +30,7 @@ Kirigami.OverlaySheet { // This function will be executed when the "Allow once" action is chosen; can be overloaded function allowOnceAction() { - SettingsManager.allowMeteredFeedUpdates = true; action() - SettingsManager.allowMeteredFeedUpdates = false; } // This function will be executed when the "Always allow" action is chosed; can be overloaded