Some API cleanups, RSS Guard now allows you to add multiple "standard" accounts simultaneously.

src/librssguard/core/feedsmodel.cpp

@@ -302,24 +302,6 @@ QList<ServiceRoot*>FeedsModel::serviceRoots() const {
   return roots;
 }
 
-bool FeedsModel::containsServiceRootFromEntryPoint(const ServiceEntryPoint* point) const {
-  return boolinq::from(serviceRoots()).any([=](ServiceRoot* root) {
-    return root->code() == point->code();
-  });
-}
-
-StandardServiceRoot* FeedsModel::standardServiceRoot() const {
-  for (ServiceRoot* root : serviceRoots()) {
-    StandardServiceRoot* std_service_root;
-
-    if ((std_service_root = dynamic_cast<StandardServiceRoot*>(root)) != nullptr) {
-      return std_service_root;
-    }
-  }
-
-  return nullptr;
-}
-
 QList<Feed*>FeedsModel::feedsForScheduledUpdate(bool auto_update_now) {
   QList<Feed*>feeds_for_update;
 

src/librssguard/core/feedsmodel.h

@@ -47,12 +47,6 @@ class RSSGUARD_DLLSPEC FeedsModel : public QAbstractItemModel {
     // the model root item.
     QList<ServiceRoot*> serviceRoots() const;
 
-    // Determines if there is any account activated from given entry point.
-    bool containsServiceRootFromEntryPoint(const ServiceEntryPoint* point) const;
-
-    // Direct and the only global accessor to standard service root.
-    StandardServiceRoot* standardServiceRoot() const;
-
     // Returns the list of feeds which should be updated
     // according to auto-update schedule.
     // Variable "auto_update_now" is true, when global timeout

src/librssguard/gui/dialogs/formaddaccount.cpp

@@ -52,15 +52,8 @@ void FormAddAccount::loadEntryPoints() {
   for (const ServiceEntryPoint* entry_point : m_entryPoints) {
     QListWidgetItem* item = new QListWidgetItem(entry_point->icon(), entry_point->name(), m_ui->m_listEntryPoints);
 
-    if (entry_point->isSingleInstanceService() && m_model->containsServiceRootFromEntryPoint(entry_point)) {
-      // Oops, this item cannot be added, it is single instance and is already added.
-      item->setFlags(Qt::ItemFlag::NoItemFlags);
-      item->setToolTip(tr("This account can be added only once."));
-    }
-    else {
     item->setToolTip(entry_point->description());
   }
-  }
 
   m_ui->m_listEntryPoints->setCurrentRow(m_entryPoints.size() - 1);
 }

src/librssguard/miscellaneous/application.cpp

@@ -2,6 +2,7 @@
 
 #include "miscellaneous/application.h"
 
+#include "3rd-party/boolinq/boolinq.h"
 #include "dynamic-shortcuts/dynamicshortcuts.h"
 #include "exceptions/applicationexception.h"
 #include "gui/dialogs/formabout.h"
@@ -361,10 +362,12 @@ void Application::processExecutionMessage(const QString& message) {
       }
       else if (msg.startsWith(QL1S(URI_SCHEME_FEED_SHORT))) {
         // Application was running, and someone wants to add new feed.
-        StandardServiceRoot* root = qApp->feedReader()->feedsModel()->standardServiceRoot();
+        ServiceRoot* rt = boolinq::from(feedReader()->feedsModel()->serviceRoots()).firstOrDefault([](ServiceRoot* root) {
+          return root->supportsFeedAdding();
+        });
 
-        if (root != nullptr) {
+        if (rt != nullptr) {
-          root->checkArgumentForFeedAdding(msg);
+          rt->addNewFeed(nullptr, msg);
         }
         else {
           showGuiMessage(tr("Cannot add feed"),

src/librssguard/services/abstract/serviceentrypoint.h

@@ -28,11 +28,6 @@ class ServiceEntryPoint {
     // to the global feed model.
     virtual QList<ServiceRoot*> initializeSubtree() const = 0;
 
-    // Can this service account be added just once?
-    // NOTE: This is true particularly for "standard" service
-    // which operates with normal RSS/ATOM feeds.
-    virtual bool isSingleInstanceService() const = 0;
-
     // Human readable service name, for example "TT-RSS".
     virtual QString name() const = 0;
 

src/librssguard/services/gmail/gmailentrypoint.cpp

@@ -24,10 +24,6 @@ QList<ServiceRoot*> GmailEntryPoint::initializeSubtree() const {
   return DatabaseQueries::getGmailAccounts(database);
 }
 
-bool GmailEntryPoint::isSingleInstanceService() const {
-  return false;
-}
-
 QString GmailEntryPoint::name() const {
   return QSL("Gmail");
 }

src/librssguard/services/gmail/gmailentrypoint.h

@@ -7,15 +7,13 @@
 
 class GmailEntryPoint : public ServiceEntryPoint {
   public:
-    ServiceRoot* createNewRoot() const;
+    virtual ServiceRoot* createNewRoot() const;
-
+    virtual QList<ServiceRoot*> initializeSubtree() const;
-    QList<ServiceRoot*> initializeSubtree() const;
+    virtual QString name() const;
-    bool isSingleInstanceService() const;
+    virtual QString code() const;
-    QString name() const;
+    virtual QString description() const;
-    QString code() const;
+    virtual QString author() const;
-    QString description() const;
+    virtual QIcon icon() const;
-    QString author() const;
-    QIcon icon() const;
 };
 
 #endif // GMAILENTRYPOINT_H

src/librssguard/services/inoreader/inoreaderentrypoint.cpp

@@ -25,10 +25,6 @@ QList<ServiceRoot*> InoreaderEntryPoint::initializeSubtree() const {
   return DatabaseQueries::getInoreaderAccounts(database);
 }
 
-bool InoreaderEntryPoint::isSingleInstanceService() const {
-  return false;
-}
-
 QString InoreaderEntryPoint::name() const {
   return QSL("Inoreader");
 }

src/librssguard/services/inoreader/inoreaderentrypoint.h

@@ -7,15 +7,13 @@
 
 class InoreaderEntryPoint : public ServiceEntryPoint {
   public:
-    ServiceRoot* createNewRoot() const;
+    virtual ServiceRoot* createNewRoot() const;
-
+    virtual QList<ServiceRoot*> initializeSubtree() const;
-    QList<ServiceRoot*> initializeSubtree() const;
+    virtual QString name() const;
-    bool isSingleInstanceService() const;
+    virtual QString code() const;
-    QString name() const;
+    virtual QString description() const;
-    QString code() const;
+    virtual QString author() const;
-    QString description() const;
+    virtual QIcon icon() const;
-    QString author() const;
-    QIcon icon() const;
 };
 
 #endif // INOREADERENTRYPOINT_H

src/librssguard/services/owncloud/owncloudserviceentrypoint.cpp

@@ -22,10 +22,6 @@ QList<ServiceRoot*> OwnCloudServiceEntryPoint::initializeSubtree() const {
   return DatabaseQueries::getOwnCloudAccounts(database);
 }
 
-bool OwnCloudServiceEntryPoint::isSingleInstanceService() const {
-  return false;
-}
-
 QString OwnCloudServiceEntryPoint::name() const {
   return QSL("Nextcloud News");
 }

src/librssguard/services/owncloud/owncloudserviceentrypoint.h

@@ -7,15 +7,13 @@
 
 class OwnCloudServiceEntryPoint : public ServiceEntryPoint {
   public:
-    ServiceRoot* createNewRoot() const;
+    virtual ServiceRoot* createNewRoot() const;
-
+    virtual QList<ServiceRoot*> initializeSubtree() const;
-    QList<ServiceRoot*> initializeSubtree() const;
+    virtual QString name() const;
-    bool isSingleInstanceService() const;
+    virtual QString code() const;
-    QString name() const;
+    virtual QString description() const;
-    QString code() const;
+    virtual QString author() const;
-    QString description() const;
+    virtual QIcon icon() const;
-    QString author() const;
-    QIcon icon() const;
 };
 
 #endif // OWNCLOUDSERVICEENTRYPOINT_H

src/librssguard/services/standard/gui/formstandardfeeddetails.cpp

@@ -10,6 +10,7 @@
 #include "services/abstract/serviceroot.h"
 #include "services/standard/gui/standardfeeddetails.h"
 #include "services/standard/standardfeed.h"
+#include "services/standard/standardserviceroot.h"
 
 #include <QFileDialog>
 #include <QTextCodec>
@@ -32,7 +33,10 @@ int FormStandardFeedDetails::addEditFeed(StandardFeed* input_feed, RootItem* par
   if (input_feed == nullptr) {
     // User is adding new feed.
     setWindowTitle(tr("Add new feed"));
-    m_standardFeedDetails->prepareForNewFeed(parent_to_select, url);
+
+    auto processed_url = qobject_cast<StandardServiceRoot*>(m_serviceRoot)->processFeedUrl(url);
+
+    m_standardFeedDetails->prepareForNewFeed(parent_to_select, processed_url);
   }
   else {
     setEditableFeed(input_feed);

src/librssguard/services/standard/standardserviceentrypoint.cpp

@@ -7,10 +7,6 @@
 #include "miscellaneous/databasequeries.h"
 #include "services/standard/standardserviceroot.h"
 
-bool StandardServiceEntryPoint::isSingleInstanceService() const {
-  return true;
-}
-
 QString StandardServiceEntryPoint::name() const {
   return QObject::tr("Standard online feeds (RSS/ATOM/JSON)");
 }

src/librssguard/services/standard/standardserviceentrypoint.h

@@ -7,16 +7,13 @@
 
 class StandardServiceEntryPoint : public ServiceEntryPoint {
   public:
-    bool isSingleInstanceService() const;
+    virtual QString name() const;
-    QString name() const;
+    virtual QString description() const;
-    QString description() const;
+    virtual QString author() const;
-    QString author() const;
+    virtual QIcon icon() const;
-    QIcon icon() const;
+    virtual QString code() const;
-    QString code() const;
+    virtual ServiceRoot* createNewRoot() const;
-
+    virtual QList<ServiceRoot*> initializeSubtree() const;
-    ServiceRoot* createNewRoot() const;
-
-    QList<ServiceRoot*> initializeSubtree() const;
 };
 
 #endif // STANDARDSERVICEENTRYPOINT_H

src/librssguard/services/standard/standardserviceroot.cpp

@@ -146,7 +146,7 @@ void StandardServiceRoot::checkArgumentsForFeedAdding() {
 
 QString StandardServiceRoot::processFeedUrl(const QString& feed_url) {
   if (feed_url.startsWith(QL1S(URI_SCHEME_FEED_SHORT))) {
-    QString without_feed_prefix = feed_url.mid(5);
+    QString without_feed_prefix = feed_url.mid(QSL(URI_SCHEME_FEED_SHORT).size());
 
     if (without_feed_prefix.startsWith(QL1S("https:")) || without_feed_prefix.startsWith(QL1S("http:"))) {
       return without_feed_prefix;

src/librssguard/services/standard/standardserviceroot.h

@@ -46,6 +46,7 @@ class StandardServiceRoot : public ServiceRoot {
     // NOTE: This is used for import/export of the model.
     bool mergeImportExportModel(FeedsImportExportModel* model, RootItem* target_root_node, QString& output_message);
 
+    QString processFeedUrl(const QString& feed_url);
     void loadFromDatabase();
     void checkArgumentForFeedAdding(const QString& argument);
 
@@ -56,7 +57,6 @@ class StandardServiceRoot : public ServiceRoot {
     void exportFeeds();
 
   private:
-    QString processFeedUrl(const QString& feed_url);
     void checkArgumentsForFeedAdding();
 
     QPointer<StandardFeed> m_feedForMetadata = {};

src/librssguard/services/tt-rss/ttrssserviceentrypoint.cpp

@@ -9,10 +9,6 @@
 #include "services/tt-rss/gui/formeditttrssaccount.h"
 #include "services/tt-rss/ttrssserviceroot.h"
 
-bool TtRssServiceEntryPoint::isSingleInstanceService() const {
-  return false;
-}
-
 QString TtRssServiceEntryPoint::name() const {
   return QSL("Tiny Tiny RSS");
 }

src/librssguard/services/tt-rss/ttrssserviceentrypoint.h

@@ -7,16 +7,13 @@
 
 class TtRssServiceEntryPoint : public ServiceEntryPoint {
   public:
-    bool isSingleInstanceService() const;
+    virtual QString name() const;
-    QString name() const;
+    virtual QString description() const;
-    QString description() const;
+    virtual QString author() const;
-    QString author() const;
+    virtual QIcon icon() const;
-    QIcon icon() const;
+    virtual QString code() const;
-    QString code() const;
+    virtual ServiceRoot* createNewRoot() const;
-
+    virtual QList<ServiceRoot*> initializeSubtree() const;
-    ServiceRoot* createNewRoot() const;
-
-    QList<ServiceRoot*> initializeSubtree() const;
 };
 
 #endif // TTRSSSERVICEENTRYPOINT_H