[netinstall] isCritical is const after construction

src/modules/netinstall/PackageModel.cpp

@@ -234,6 +234,10 @@ PackageModel::setupModelData( const YAML::Node& data, PackageTreeItem* parent )
         {
             itemData.postScript = CalamaresUtils::yamlToVariant( itemDefinition[ "post-install" ] ).toString();
         }
+        if ( itemDefinition[ "critical" ] )
+        {
+            itemData.isCritical = CalamaresUtils::yamlToVariant( itemDefinition[ "critical" ] ).toBool();
+        }
         PackageTreeItem* item = new PackageTreeItem( itemData, parent );
 
         if ( itemDefinition[ "selected" ] )
@@ -249,10 +253,6 @@ PackageModel::setupModelData( const YAML::Node& data, PackageTreeItem* parent )
             item->setHidden( CalamaresUtils::yamlToVariant( itemDefinition[ "hidden" ] ).toBool() );
         }
 
-        if ( itemDefinition[ "critical" ] )
-        {
-            item->setCritical( CalamaresUtils::yamlToVariant( itemDefinition[ "critical" ] ).toBool() );
-        }
 
         if ( itemDefinition[ "packages" ] )
             for ( YAML::const_iterator packageIt = itemDefinition[ "packages" ].begin();

src/modules/netinstall/PackageTreeItem.cpp

@@ -217,12 +217,6 @@ PackageTreeItem::isCritical() const
     return m_data.isCritical;
 }
 
-void
-PackageTreeItem::setCritical( bool isCritical )
-{
-    m_data.isCritical = isCritical;
-}
-
 Qt::CheckState
 PackageTreeItem::isSelected() const
 {

src/modules/netinstall/PackageTreeItem.h

@@ -79,7 +79,6 @@ public:
     bool hiddenSelected() const;
 
     bool isCritical() const;
-    void setCritical( bool isCritical );
 
     Qt::CheckState isSelected() const;
     void setSelected( Qt::CheckState isSelected );