Make JobItem::T_FIT_SUITE private

GUI/Models/JobItem.cpp

@@ -260,6 +260,11 @@ FitSuiteItem* JobItem::fitSuiteItem()
     return dynamic_cast<FitSuiteItem*>(getItem(JobItem::T_FIT_SUITE));
 }
 
+FitSuiteItem* JobItem::createFitSuiteItem()
+{
+    return model()->insertItem<FitSuiteItem>(this, -1, T_FIT_SUITE);
+}
+
 ParameterContainerItem* JobItem::parameterContainerItem()
 {
     return const_cast<ParameterContainerItem*>(

GUI/Models/JobItem.h

@@ -53,9 +53,9 @@ private:
     static const QString T_DATAVIEW;
     static const QString T_PARAMETER_TREE;
     static const QString T_SIMULATION_OPTIONS;
+    static const QString T_FIT_SUITE;
 
 public:
-    static const QString T_FIT_SUITE;
 
     JobItem();
 
@@ -107,6 +107,8 @@ public:
     void setResults(const ISimulation* simulation);
 
     FitSuiteItem* fitSuiteItem();
+    FitSuiteItem* createFitSuiteItem();
+
     ParameterContainerItem* parameterContainerItem();
     const ParameterContainerItem* parameterContainerItem() const;
     FitParameterContainerItem* fitParameterContainerItem();

GUI/Models/JobModelFunctions.cpp

@@ -253,13 +253,13 @@ void createFitContainers(JobItem* jobItem)
 {
     SessionModel* model = jobItem->model();
 
-    SessionItem* fitSuiteItem = jobItem->getItem(JobItem::T_FIT_SUITE);
+    SessionItem* fitSuiteItem = jobItem->fitSuiteItem();
     if (fitSuiteItem != nullptr) {
         throw Error("JobModel::createFitContainers() -> Error. Attempt to create "
                     "a second FitSuiteItem.");
     }
 
-    fitSuiteItem = model->insertItem<FitSuiteItem>(jobItem, -1, JobItem::T_FIT_SUITE);
+    fitSuiteItem = jobItem->createFitSuiteItem();
 
     SessionItem* parsContainerItem =
         fitSuiteItem->getItem(FitSuiteItem::T_FIT_PARAMETERS_CONTAINER);