]> git.proxmox.com Git - proxmox-widget-toolkit.git/commitdiff
window: ldap auth edit: set view-model form data explicitly on edit
authorThomas Lamprecht <t.lamprecht@proxmox.com>
Tue, 28 Mar 2023 15:47:34 +0000 (17:47 +0200)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Tue, 28 Mar 2023 15:49:17 +0000 (17:49 +0200)
Fix a race that shows up in chrome/chromium by setting the data in
the view-model that is binded to form fields explicitly on edit, this
avoids a race where the default declared in the viewModel data got
applied after the form data was set, thus having the state out of
sync and so marking the field potentially as dirty even if it wasn't.

Reported-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
src/window/AuthEditLDAP.js

index 43d2234d3ab4163d2fc9a2fe8c0d26443cd87027..cae7c80f547eae7749295c73674782c587d22ba0 100644 (file)
@@ -57,7 +57,9 @@ Ext.define('Proxmox.panel.LDAPInputPanel', {
     },
 
     onSetValues: function(values) {
+       let me = this;
        values.anonymous_search = values["bind-dn"] ? 0 : 1;
+       me.getViewModel().set('anonymous_search', values.anonymous_search);
 
        return values;
     },