GIT_REPOSITORIES/LPP/Users/mperrinel/SciQLop-fork Commit - r79:0afc39d45cc4

Adds type for a data source item...

Alexandre Leroux -

r79:0afc39d45cc4

parent child

Context file:

r79:0afc39d45cc4

core/include/DataSource/DataSourceItem.h +8 -1

             #include <QVector>
             /**
+             * Possible types of an item
+             */
+            enum class DataSourceItemType { NODE, PRODUCT };
+            /**
              * @brief The DataSourceItem class aims to represent a structure element of a data source.
              * A data source has a tree structure that is made up of a main DataSourceItem object (root)
              * containing other DataSourceItem objects (children).
              */
             class DataSourceItem {
             public:
-                explicit DataSourceItem(QVector<QVariant> data = {});
+                explicit DataSourceItem(DataSourceItemType type, QVector<QVariant> data = {});
                 /**
                  * Adds a child to the item. The item takes ownership of the child.
                  */
                 DataSourceItem *parentItem() const noexcept;
+                DataSourceItemType type() const noexcept;
             private:
                 class DataSourceItemPrivate;
                 spimpl::unique_impl_ptr<DataSourceItemPrivate> impl;

core/src/DataSource/DataSourceItem.cpp +10 -4

             #include <QVector>
             struct DataSourceItem::DataSourceItemPrivate {
-                explicit DataSourceItemPrivate(QVector<QVariant> data)
+                explicit DataSourceItemPrivate(DataSourceItemType type, QVector<QVariant> data)
-                        : m_Parent{nullptr}, m_Children{}, m_Data{std::move(data)}
+                        : m_Parent{nullptr}, m_Children{}, m_Type{type}, m_Data{std::move(data)}
                 {
                 }
                 DataSourceItem *m_Parent;
                 std::vector<std::unique_ptr<DataSourceItem> > m_Children;
+                DataSourceItemType m_Type;
                 QVector<QVariant> m_Data;
             };
-            DataSourceItem::DataSourceItem(QVector<QVariant> data)
+            DataSourceItem::DataSourceItem(DataSourceItemType type, QVector<QVariant> data)
-                    : impl{spimpl::make_unique_impl<DataSourceItemPrivate>(data)}
+                    : impl{spimpl::make_unique_impl<DataSourceItemPrivate>(type, std::move(data))}
             {
             }
             {
                 return impl->m_Parent;
             }
+            DataSourceItemType DataSourceItem::type() const noexcept
+            {
+                return impl->m_Type;
+            }

core/tests/DataSource/TestDataSourceController.cpp +4 -2

                 // Create a data source item
                 auto source1Name = QStringLiteral("Source1");
                 auto source1Values = QVector<QVariant>{source1Name};
-                auto source1Item = std::make_unique<DataSourceItem>(std::move(source1Values));
+                auto source1Item
+                    = std::make_unique<DataSourceItem>(DataSourceItemType::PRODUCT, std::move(source1Values));
                 // Add data source item to the controller and check that a signal has been emitted after setting
                 // data source item in the controller
                 // Try to a data source item with an unregistered uid and check that no signal has been emitted
                 auto unregisteredUid = QUuid::createUuid();
-                dataSourceController.setDataSourceItem(unregisteredUid, std::make_unique<DataSourceItem>());
+                dataSourceController.setDataSourceItem(
+                    unregisteredUid, std::make_unique<DataSourceItem>(DataSourceItemType::PRODUCT));
                 QCOMPARE(signalSpy.count(), 1);
             }

General Comments 0

You need to be logged in to leave comments. Login now

No TODOs yet

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages