Removing containers now returns false if it was unable to remove containers

CURA-2196
This commit is contained in:
Jaime van Kessel 2016-08-24 13:34:40 +02:00
parent e05bb54047
commit b1ae25fb92

View file

@ -438,13 +438,17 @@ class ContainerManager(QObject):
# \return \type{bool} True if successful, False if not. # \return \type{bool} True if successful, False if not.
@pyqtSlot(str, result = bool) @pyqtSlot(str, result = bool)
def removeQualityChanges(self, quality_name): def removeQualityChanges(self, quality_name):
UM.Logger.log("d", "Attempting to remove the container %s", quality_name)
containers_found = False
if not quality_name: 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"): for container in self._getFilteredContainers(name = quality_name, type = "quality_changes"):
containers_found = True
UM.Settings.ContainerRegistry.getInstance().removeContainer(container.getId()) UM.Settings.ContainerRegistry.getInstance().removeContainer(container.getId())
return True return containers_found
## Rename a set of quality changes containers. ## Rename a set of quality changes containers.
# #