[office/skrooge] /: FEATURE: Better management of deprecated source of download of currencies

Stéphane MANKOWSKI <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit c1e694962b46e4d296c585dc32a405ce69c07357 by Stéphane MANKOWSKI.
Committed on 28/07/2026 at 20:16.
Pushed by smankowski into branch 'master'.

FEATURE: Better management of deprecated source of download of currencies

M  +1    -0    CHANGELOG
M  +7    -0    skgbankmodeler/skgunitobject.cpp
M  +2    -1    skgbankmodeler/sources/org.kde.skrooge-source-stooq.json

https://invent.kde.org/office/skrooge/-/commit/c1e694962b46e4d296c585dc32a405ce69c07357

diff --git a/CHANGELOG b/CHANGELOG
index ec74e02db..5b182b121 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -4,6 +4,7 @@ skrooge (26.x.x)
  *Correction: Sources' keys not displayed in settings
  *Feature: Keep focus on selection when the current filter is changed
  *Feature: New option to choose to currency format (currency or numerical)
+ *Feature: Better management of deprecated source of download of currencies
 
  -- Stephane MANKOWSKI <[email protected]>  xx
 
diff --git a/skgbankmodeler/skgunitobject.cpp b/skgbankmodeler/skgunitobject.cpp
index 8d67222a6..83b3a830c 100644
--- a/skgbankmodeler/skgunitobject.cpp
+++ b/skgbankmodeler/skgunitobject.cpp
@@ -2395,6 +2395,13 @@ SKGError SKGUnitObject::downloadUnitValue(UnitDownloadMode iMode, int iNbMaxValu
                 for (const auto &service : list2) {
                     auto name = service.pluginId().remove(QStringLiteral("org.kde.skrooge-source-")).toLower();
                     if (name == source.toLower()) {
+                        // Check if the source is deprecated
+                        QString deprecated = service.value(QStringLiteral("X-SKROOGE-deprecated"), QString());
+                        if (!deprecated.isEmpty()) {
+                            err = SKGError(ERR_FAIL,
+                                           i18nc("Error message", "Source of download %1 is deprecated. You should use %2 instead.", source, deprecated));
+                            return err;
+                        }
                         path = service.value(QStringLiteral("X-SKROOGE-url"), QString());
                         if (path.isEmpty()) {
                             path = service.value(QStringLiteral("X-SKROOGE-script"), QString());
diff --git a/skgbankmodeler/sources/org.kde.skrooge-source-stooq.json b/skgbankmodeler/sources/org.kde.skrooge-source-stooq.json
index 3632c1053..5ce6867e6 100644
--- a/skgbankmodeler/sources/org.kde.skrooge-source-stooq.json
+++ b/skgbankmodeler/sources/org.kde.skrooge-source-stooq.json
@@ -83,5 +83,6 @@
     "X-SKROOGE-keyAPI": "false",
     "X-SKROOGE-mode": "CSV",
     "X-SKROOGE-price": "%1,[^,]*,[^,]*,[^,]*,[^,]*,[^,]*,([^,]*),.*",
-    "X-SKROOGE-url": "https://stooq.pl/q/l/?s=%1&f=sd2t2ohlcv&h&e=csv"
+    "X-SKROOGE-url": "https://stooq.pl/q/l/?s=%1&f=sd2t2ohlcv&h&e=csv",
+    "X-SKROOGE-deprecated": "boursorama" 
 }
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.