From 5e9854454185fee9490b0d9aa996d36f65b7e2fd Mon Sep 17 00:00:00 2001
From: Ghostkeeper <rubend@tutanota.com>
Date: Fri, 21 Dec 2018 16:33:16 +0100
Subject: [PATCH] Don't change the model depending on isValidMaterial

That would cause a circular dependency in a way.

Contributes to issue CURA-6033.
---
 .../qml/Menus/ConfigurationMenu/ConfigurationItem.qml  | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/resources/qml/Menus/ConfigurationMenu/ConfigurationItem.qml b/resources/qml/Menus/ConfigurationMenu/ConfigurationItem.qml
index 31569c58d2..6a17353459 100644
--- a/resources/qml/Menus/ConfigurationMenu/ConfigurationItem.qml
+++ b/resources/qml/Menus/ConfigurationMenu/ConfigurationItem.qml
@@ -62,19 +62,13 @@ Button
             Repeater
             {
                 id: repeater
-                model:
-                {
-                    if (configurationItem.isValidMaterial)
-                    {
-                        return configuration.extruderConfigurations
-                    }
-                    return []
-                }
+                model: configuration.extruderConfigurations
 
                 delegate: PrintCoreConfiguration
                 {
                     width: Math.round(parent.width / 2)
                     printCoreConfiguration: modelData
+                    visible: configurationItem.isValidMaterial
                 }
             }