load filters in adblock dialog even when disabled + other minor fix

src/librssguard/miscellaneous/application.cpp

@@ -86,6 +86,8 @@ Application::Application(const QString& id, int& argc, char** argv)
   });
 #endif
 
+  m_notifications->load(settings());
+
   qDebugNN << LOGSEC_CORE
            << "OpenSSL version:"
            << QUOTE_W_SPACE_DOT(QSslSocket::sslLibraryVersionString());

src/librssguard/miscellaneous/notificationfactory.cpp

@@ -22,6 +22,8 @@ Notification NotificationFactory::notificationForEvent(Notification::Event event
   }
 }
 
-void NotificationFactory::load(Settings* settings) {}
+void NotificationFactory::load(Settings* settings) {
+  //settings->allKeys(Notifications::ID)
+}
 
 void NotificationFactory::save(const QList<Notification> new_notifications, Settings* settings) {}

src/librssguard/miscellaneous/settings.cpp

@@ -312,6 +312,9 @@ DVALUE(char*) Database::ActiveDriverDef = APP_DB_SQLITE_DRIVER;
 // Keyboard.
 DKEY Keyboard::ID = "keyboard";
 
+// Notifications.
+DKEY Notifications::ID = "notifications";
+
 // Web browser.
 DKEY Browser::ID = "browser";
 
@@ -350,6 +353,19 @@ Settings::Settings(const QString& file_name, Format format, const SettingsProper
 
 Settings::~Settings() = default;
 
+QStringList Settings::allKeys(const QString& section) {
+  if (!section.isEmpty()) {
+    beginGroup(section);
+    auto keys = QSettings::allKeys();
+
+    endGroup();
+    return keys;
+  }
+  else {
+    return QSettings::allKeys();
+  }
+}
+
 QString Settings::pathName() const {
   return QFileInfo(fileName()).absolutePath();
 }

src/librssguard/miscellaneous/settings.h

@@ -360,6 +360,11 @@ namespace Keyboard {
   KEY ID;
 }
 
+// Notifications.
+namespace Notifications {
+  KEY ID;
+}
+
 // Web browser.
 namespace Browser {
   KEY ID;
@@ -420,6 +425,8 @@ class Settings : public QSettings {
     QVariant password(const QString& section, const QString& key, const QVariant& default_value = QVariant()) const;
     void setPassword(const QString& section, const QString& key, const QVariant& value);
 
+    QStringList allKeys(const QString& section);
+
     QVariant value(const QString& section, const QString& key, const QVariant& default_value = QVariant()) const;
     void setValue(const QString& section, const QString& key, const QVariant& value);
     void setValue(const QString& key, const QVariant& value);

src/librssguard/network-web/adblock/adblockdialog.cpp

@@ -96,7 +96,7 @@ void AdBlockDialog::testConfiguration() {
 }
 
 void AdBlockDialog::load() {
-  if (m_loaded || !m_ui.m_cbEnable->isChecked()) {
+  if (m_loaded) {
     return;
   }
 

src/librssguard/network-web/adblock/adblockmanager.cpp

@@ -316,7 +316,7 @@ QProcess* AdBlockManager::restartServer(int port) {
   proc->setProcessEnvironment(pe);
   proc->setProcessChannelMode(QProcess::ProcessChannelMode::ForwardedErrorChannel);
 
-  if (!proc->open()) {
+  if (!proc->open() || proc->state() == QProcess::ProcessState::NotRunning) {
     auto ers = proc->errorString();
     proc->deleteLater();
 

src/librssguard/network-web/cookiejar.cpp

@@ -47,11 +47,7 @@ QList<QNetworkCookie> CookieJar::extractCookiesFromUrl(const QString& url) {
 
 void CookieJar::loadCookies() {
   Settings* sett = qApp->settings();
-
+  auto keys = sett->allKeys(Cookies::ID);
-  sett->beginGroup(GROUP(Cookies));
-  auto keys = sett->allKeys();
-
-  sett->endGroup();
 
   for (const QString& cookie_key : qAsConst(keys)) {
     QByteArray encoded = sett->password(GROUP(Cookies), cookie_key, {}).toByteArray();