Currently, when creating an active-backup bond, it is not possible to leave the 
primary-bond field blank, even though this value is optional in the backend.
If no value is currently set in the frontend, it sends the value '' to the 
backend. There, it is interpreted as the name of an interface, which is 
incorrect, and therefore the error 'bond-primary: invalid format - invalid 
network interface name '' ' is displayed.
Changing the field type to 'proxmoxtextfield' will prevent the empty string 
from being sent due to the way we submit the data and make it therefore 
possible to not set any value in the frontend.
Addtionally, the deleteEmpty flag is set, so that devices, that were unset, 
will be deleted by the backend.

Signed-off-by: Lukas Sichert <[email protected]>
---
 src/node/NetworkEdit.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/node/NetworkEdit.js b/src/node/NetworkEdit.js
index 39c7e35..c945139 100644
--- a/src/node/NetworkEdit.js
+++ b/src/node/NetworkEdit.js
@@ -239,10 +239,11 @@ Ext.define('Proxmox.node.NetworkEdit', {
                 disabled: true,
             });
 
-            let primaryfield = Ext.createWidget('textfield', {
+            let primaryfield = Ext.createWidget('proxmoxtextfield', {
                 fieldLabel: 'bond-primary',
                 name: 'bond-primary',
                 value: '',
+                deleteEmpty: !me.isCreate,
                 disabled: true,
             });
 
-- 
2.47.3



Reply via email to