From 9f43a740a6a3146d32ef1684a69fa34a3c5a0f87 Mon Sep 17 00:00:00 2001
From: Ghostkeeper <rubend@tutanota.com>
Date: Fri, 28 Oct 2016 13:33:26 +0200
Subject: [PATCH] Also validate machine name in rename dialogue

Otherwise you could still circumvent the length limitation.

Contributes to issue CURA-2692.
---
 resources/qml/Preferences/MachinesPage.qml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/resources/qml/Preferences/MachinesPage.qml b/resources/qml/Preferences/MachinesPage.qml
index 78553bb37f..e6ddef7979 100644
--- a/resources/qml/Preferences/MachinesPage.qml
+++ b/resources/qml/Preferences/MachinesPage.qml
@@ -251,6 +251,8 @@ UM.ManagementPage
         {
             id: renameDialog;
             object: base.currentItem && base.currentItem.name ? base.currentItem.name : "";
+            property var machine_name_validator: Cura.MachineNameValidator { }
+            validName: renameDialog.newName.match(renameDialog.machine_name_validator.machineNameRegex) != null;
             onAccepted:
             {
                 Cura.MachineManager.renameMachine(base.currentItem.id, newName.trim());