Make FitParameterItem::T_LINK private

GUI/Models/FitParameterHelper.cpp

@@ -48,7 +48,7 @@ void FitParameterHelper::removeFromFitParameters(FitParameterContainerItem* cont
     FitParameterItem* fitParItem = getFitParameterItem(container, parameterItem);
 
     if (fitParItem) {
-        for (auto linkItem : fitParItem->items<FitParameterLinkItem>(FitParameterItem::T_LINK)) {
+        for (auto linkItem : fitParItem->linkItems()) {
             if (getParameterItemPath(parameterItem) == linkItem->link()) {
                 fitParItem->model()->removeRow(linkItem->index().row(), linkItem->index().parent());
                 break;

GUI/Models/FitParameterItems.cpp

@@ -199,6 +199,11 @@ SessionItem* FitParameterItem::maximumItem() const
     return getItem(P_MAX);
 }
 
+QVector<FitParameterLinkItem*> FitParameterItem::linkItems() const
+{
+    return items<FitParameterLinkItem>(T_LINK);
+}
+
 QString FitParameterItem::parameterType() const
 {
     ComboProperty partype = getItemValue(P_TYPE).value<ComboProperty>();
@@ -318,9 +323,8 @@ FitParameterContainerItem::FitParameterContainerItem() : SessionItem(M_TYPE)
 
 FitParameterItem* FitParameterContainerItem::fitParameterItem(const QString& link)
 {
-    for (auto item : getItems(T_FIT_PARAMETERS)) {
-        for (FitParameterLinkItem* linkItem : item->items<FitParameterLinkItem>
-                 (FitParameterItem::T_LINK)) {
+    for (FitParameterItem* item : items<FitParameterItem>(T_FIT_PARAMETERS)) {
+        for (FitParameterLinkItem* linkItem : item->linkItems()) {
             if (link == linkItem->link())
                 return dynamic_cast<FitParameterItem*>(item);
         }
@@ -348,11 +352,11 @@ void FitParameterContainerItem::setValuesInParameterContainer(
 {
     ASSERT(parameterContainer);
 
-    QVector<SessionItem*> fitPars = getItems(FitParameterContainerItem::T_FIT_PARAMETERS);
+    QVector<FitParameterItem*> fitPars = items<FitParameterItem>(T_FIT_PARAMETERS);
 
     int index(0);
     for (int i = 0; i < fitPars.size(); ++i) {
-        auto link_list = fitPars[i]->items<FitParameterLinkItem>(FitParameterItem::T_LINK);
+        auto link_list = fitPars[i]->linkItems();
         if (link_list.empty())
             continue;
         for (FitParameterLinkItem* linkItem : link_list) {

GUI/Models/FitParameterItems.h

@@ -18,6 +18,8 @@
 #include "Fit/Param/AttLimits.h"
 #include "GUI/Models/SessionItem.h"
 
+class FitParameterLinkItem;
+
 //! FitParameterItems is a collection of items to define fit parameters in GUI.
 
 //! The FitParameterItem class represents a fit parameter in GUI. Contains links to corresponding
@@ -29,10 +31,9 @@ private:
     static const QString P_START_VALUE;
     static const QString P_MIN;
     static const QString P_MAX;
-
-public:
     static const QString T_LINK;
 
+public:
     static const QString M_TYPE;
 
     FitParameterItem();
@@ -55,6 +56,8 @@ public:
     void setMaximum(double maximum);
     SessionItem* maximumItem() const;
 
+    QVector<FitParameterLinkItem*> linkItems() const;
+
 private:
     QString parameterType() const;
     void onTypeChange();

GUI/Views/FitWidgets/FitParameterWidget.cpp

@@ -368,7 +368,7 @@ QVector<FitParameterItem*> FitParameterWidget::emptyFitParameters()
 {
     QVector<FitParameterItem*> result;
     for (auto fitParItem : jobItem()->fitParameterContainerItem()->fitParameterItems())
-        if (fitParItem->getItems(FitParameterItem::T_LINK).empty())
+        if (fitParItem->linkItems().empty())
             result.push_back(fitParItem);
 
     return result;

GUI/Views/FitWidgets/FitSessionController.cpp

@@ -179,8 +179,9 @@ void FitSessionController::updateLog(const FitProgressInfo& info)
     FitParameterContainerItem* fitParContainer = m_jobItem->fitParameterContainerItem();
     int index(0);
     QVector<double> values = GUI::Helpers::fromStdVector(info.parValues());
-    for (auto item : fitParContainer->getItems(FitParameterContainerItem::T_FIT_PARAMETERS)) {
-        if (item->getItems(FitParameterItem::T_LINK).empty())
+    for (FitParameterItem* item : fitParContainer->items<FitParameterItem>
+             (FitParameterContainerItem::T_FIT_PARAMETERS)) {
+        if (item->linkItems().empty())
             continue;
         QString parinfo = QString("      %1 %2\n").arg(item->displayName()).arg(values[index++]);
         message.append(parinfo);