[Card DB] Properly pass along set priority controller to parsers

Took 16 minutes

Took 35 seconds

libcockatrice_card/libcockatrice/card/database/card_database.cpp

@@ -21,7 +21,7 @@ CardDatabase::CardDatabase(QObject *parent,
     qRegisterMetaType<CardInfoPtr>("CardSetPtr");
 
     // create loader and wire it up
-    loader = new CardDatabaseLoader(this, this, pathProvider, prefs);
+    loader = new CardDatabaseLoader(this, this, pathProvider, prefs, setPriorityController);
     // re-emit loader signals (so other code doesn't need to know about internals)
     connect(loader, &CardDatabaseLoader::loadingFinished, this, &CardDatabase::cardDatabaseLoadingFinished);
     connect(loader, &CardDatabaseLoader::loadingFailed, this, &CardDatabase::cardDatabaseLoadingFailed);

libcockatrice_card/libcockatrice/card/database/card_database_loader.cpp

@@ -12,12 +12,13 @@
 CardDatabaseLoader::CardDatabaseLoader(QObject *parent,
                                        CardDatabase *db,
                                        ICardDatabasePathProvider *_pathProvider,
-                                       ICardPreferenceProvider *_preferenceProvider)
+                                       ICardPreferenceProvider *_preferenceProvider,
+                                       ICardSetPriorityController *_priorityController)
     : QObject(parent), database(db), pathProvider(_pathProvider)
 {
     // instantiate available parsers here and connect them to the database
-    availableParsers << new CockatriceXml4Parser(_preferenceProvider);
+    availableParsers << new CockatriceXml4Parser(_preferenceProvider, _priorityController);
-    availableParsers << new CockatriceXml3Parser;
+    availableParsers << new CockatriceXml3Parser(_priorityController);
 
     for (auto *p : availableParsers) {
         // connect parser outputs to the database adders

libcockatrice_card/libcockatrice/card/database/card_database_loader.h

@@ -6,6 +6,7 @@
 #include <QLoggingCategory>
 #include <libcockatrice/interfaces/interface_card_database_path_provider.h>
 #include <libcockatrice/interfaces/interface_card_preference_provider.h>
+#include <libcockatrice/interfaces/interface_card_set_priority_controller.h>
 
 inline Q_LOGGING_CATEGORY(CardDatabaseLoadingLog, "card_database.loading");
 inline Q_LOGGING_CATEGORY(CardDatabaseLoadingSuccessOrFailureLog, "card_database.loading.success_or_failure");
@@ -52,7 +53,8 @@ public:
     explicit CardDatabaseLoader(QObject *parent,
                                 CardDatabase *db,
                                 ICardDatabasePathProvider *pathProvider,
-                                ICardPreferenceProvider *preferenceProvider);
+                                ICardPreferenceProvider *preferenceProvider,
+                                ICardSetPriorityController *_priorityController);
 
     /** @brief Destructor cleans up allocated parsers. */
     ~CardDatabaseLoader() override;

libcockatrice_card/libcockatrice/card/database/parser/card_database_parser.cpp

@@ -4,6 +4,10 @@
 
 SetNameMap ICardDatabaseParser::sets;
 
+ICardDatabaseParser::ICardDatabaseParser(ICardSetPriorityController *_cardSetPriorityController)
+    : cardSetPriorityController(_cardSetPriorityController)
+{
+}
 void ICardDatabaseParser::clearSetlist()
 {
     sets.clear();
@@ -19,7 +23,7 @@ CardSetPtr ICardDatabaseParser::internalAddSet(const QString &setName,
         return sets.value(setName);
     }
 
-    CardSetPtr newSet = CardSet::newInstance(new NoopCardSetPriorityController(), setName);
+    CardSetPtr newSet = CardSet::newInstance(cardSetPriorityController, setName);
     newSet->setLongName(longName);
     newSet->setSetType(setType);
     newSet->setReleaseDate(releaseDate);

libcockatrice_card/libcockatrice/card/database/parser/card_database_parser.h

@@ -20,6 +20,7 @@ class ICardDatabaseParser : public QObject
 {
     Q_OBJECT
 public:
+    ICardDatabaseParser(ICardSetPriorityController *cardSetPriorityController);
     ~ICardDatabaseParser() override = default;
 
     /**
@@ -59,6 +60,7 @@ public:
 protected:
     /** @brief Cached global list of sets shared between all parsers. */
     static SetNameMap sets;
+    ICardSetPriorityController *cardSetPriorityController;
 
     /**
      * @brief Internal helper to add a set to the global set cache.

libcockatrice_card/libcockatrice/card/database/parser/cockatrice_xml_3.cpp

@@ -14,6 +14,11 @@
 #define COCKATRICE_XML3_SCHEMALOCATION                                                                                 \
     "https://raw.githubusercontent.com/Cockatrice/Cockatrice/master/doc/carddatabase_v3/cards.xsd"
 
+CockatriceXml3Parser::CockatriceXml3Parser(ICardSetPriorityController *_cardSetPriorityController)
+    : ICardDatabaseParser(_cardSetPriorityController)
+{
+}
+
 bool CockatriceXml3Parser::getCanParseFile(const QString &fileName, QIODevice &device)
 {
     qCInfo(CockatriceXml3Log) << "Trying to parse: " << fileName;

libcockatrice_card/libcockatrice/card/database/parser/cockatrice_xml_3.h

@@ -26,7 +26,7 @@ class CockatriceXml3Parser : public ICardDatabaseParser
 {
     Q_OBJECT
 public:
-    CockatriceXml3Parser() = default;
+    CockatriceXml3Parser(ICardSetPriorityController *cardSetPriorityController);
     ~CockatriceXml3Parser() override = default;
 
     /**

libcockatrice_card/libcockatrice/card/database/parser/cockatrice_xml_4.cpp

@@ -14,8 +14,9 @@
 #define COCKATRICE_XML4_SCHEMALOCATION                                                                                 \
     "https://raw.githubusercontent.com/Cockatrice/Cockatrice/master/doc/carddatabase_v4/cards.xsd"
 
-CockatriceXml4Parser::CockatriceXml4Parser(ICardPreferenceProvider *_cardPreferenceProvider)
+CockatriceXml4Parser::CockatriceXml4Parser(ICardPreferenceProvider *_cardPreferenceProvider,
-    : cardPreferenceProvider(_cardPreferenceProvider)
+                                           ICardSetPriorityController *_cardSetPriorityController)
+    : ICardDatabaseParser(_cardSetPriorityController), cardPreferenceProvider(_cardPreferenceProvider)
 {
 }
 

libcockatrice_card/libcockatrice/card/database/parser/cockatrice_xml_4.h

@@ -29,7 +29,8 @@ class CockatriceXml4Parser : public ICardDatabaseParser
 {
     Q_OBJECT
 public:
-    explicit CockatriceXml4Parser(ICardPreferenceProvider *cardPreferenceProvider);
+    explicit CockatriceXml4Parser(ICardPreferenceProvider *cardPreferenceProvider,
+                                  ICardSetPriorityController *cardSetPriorityController);
     ~CockatriceXml4Parser() override = default;
 
     /**