Make MaterialItemContainer::T_MATERIALS private

GUI/coregui/Models/JobModelFunctions.cpp

@@ -180,8 +180,8 @@ void JobModelFunctions::muteMagnetizationData(JobItem* jobItem)
 {
     MaterialItemContainer* container =
         static_cast<MaterialItemContainer*>(jobItem->getItem(JobItem::T_MATERIAL_CONTAINER));
-    for (SessionItem* item : container->getItems(MaterialItemContainer::T_MATERIALS))
-        static_cast<MaterialItem*>(item)->hideMagnetization();
+    for (MaterialItem* material : container->getMaterials())
+        material->hideMagnetization();
 
     MultiLayerItem* sample = static_cast<MultiLayerItem*>(jobItem->getItem(JobItem::T_SAMPLE));
     sample->getItem(MultiLayerItem::P_EXTERNAL_FIELD)->setVisible(false);

GUI/coregui/Models/MaterialItemContainer.cpp

@@ -49,3 +49,8 @@ const MaterialItem* MaterialItemContainer::findMaterialById(QString id) const
     }
     return nullptr;
 }
+
+QVector<MaterialItem*> MaterialItemContainer::getMaterials()
+{
+    return items<MaterialItem>(T_MATERIALS);
+}

GUI/coregui/Models/MaterialItemContainer.h

@@ -20,9 +20,10 @@
 class MaterialItem;
 
 class MaterialItemContainer : public SessionItem {
-public:
+private:
     static const QString T_MATERIALS;
 
+public:
     MaterialItemContainer();
 
     //! Copies MaterialItem, inserts it into the container
@@ -31,6 +32,8 @@ public:
 
     const MaterialItem* findMaterialById(QString id) const;
     MaterialItem* findMaterialById(QString id);
+
+    QVector<MaterialItem*> getMaterials();
 };
 
 #endif // BORNAGAIN_GUI_COREGUI_MODELS_MATERIALITEMCONTAINER_H

GUI/coregui/Models/SessionItem.h

@@ -65,6 +65,7 @@ public:
     SessionItem* getItem(const QString& tag = "", int row = 0) const;
     template <typename T> T* item(const QString& tag) const;
     QVector<SessionItem*> getItems(const QString& tag = "") const;
+    template <typename T> QVector<T*> items(const QString& tag = "") const;
     bool insertItem(int row, SessionItem* item, const QString& tag = "");
     SessionItem* takeItem(int row, const QString& tag);
 
@@ -155,6 +156,17 @@ template <typename T> T* SessionItem::item(const QString& tag) const
     return t;
 }
 
+template <typename T>
+QVector<T*> SessionItem::items(const QString& tag) const
+{
+    QVector<T*> result;
+
+    for ( SessionItem* item : getItems(tag) )
+        if (T* titem = dynamic_cast<T*>(item)) 
+            result.push_back(titem);
+    return result;
+}
+
 template <typename T> T* SessionItem::addProperty(const QString& tagname)
 {
     auto property = new T;