From b1ae25fb9275ed82a3f9bea5c4f99de0dc007c90 Mon Sep 17 00:00:00 2001
From: Jaime van Kessel <nallath@gmail.com>
Date: Wed, 24 Aug 2016 13:34:40 +0200
Subject: [PATCH] Removing containers now returns false if it was unable to
 remove containers

CURA-2196
---
 cura/Settings/ContainerManager.py | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/cura/Settings/ContainerManager.py b/cura/Settings/ContainerManager.py
index 740f2dbd06..6c4188934c 100644
--- a/cura/Settings/ContainerManager.py
+++ b/cura/Settings/ContainerManager.py
@@ -438,13 +438,17 @@ class ContainerManager(QObject):
     #   \return \type{bool} True if successful, False if not.
     @pyqtSlot(str, result = bool)
     def removeQualityChanges(self, quality_name):
+        UM.Logger.log("d", "Attempting to remove the container %s", quality_name)
+        containers_found = False
+
         if not quality_name:
-            return False
+            return containers_found  # Without a name we will never find a container to remove.
 
         for container in self._getFilteredContainers(name = quality_name, type = "quality_changes"):
+            containers_found = True
             UM.Settings.ContainerRegistry.getInstance().removeContainer(container.getId())
 
-        return True
+        return containers_found
 
     ##  Rename a set of quality changes containers.
     #