ui: ldap: add 'Check connection' checkbox as advanced option

The checkbox is enabled by default, setting the new `check-connection`
parameter. See also [0] for the rationale.

[0] https://lists.proxmox.com/pipermail/pve-devel/2023-July/058559.html

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>
This commit is contained in:
Christoph Heiss 2023-08-10 14:37:08 +02:00 committed by Wolfgang Bumiller
parent e1d996dc55
commit b9d23c8787
2 changed files with 30 additions and 0 deletions

View File

@ -79,6 +79,21 @@ Ext.define('PVE.panel.ADInputPanel', {
},
];
me.advancedItems = [
{
xtype: 'proxmoxcheckbox',
fieldLabel: gettext('Check connection'),
name: 'check-connection',
uncheckedValue: 0,
checked: true,
autoEl: {
tag: 'div',
'data-qtip':
gettext('Verify connection parameters and bind credentials on save'),
},
},
];
me.callParent();
},
onGetValues: function(values) {

View File

@ -79,6 +79,21 @@ Ext.define('PVE.panel.LDAPInputPanel', {
},
];
me.advancedItems = [
{
xtype: 'proxmoxcheckbox',
fieldLabel: gettext('Check connection'),
name: 'check-connection',
uncheckedValue: 0,
checked: true,
autoEl: {
tag: 'div',
'data-qtip':
gettext('Verify connection parameters and bind credentials on save'),
},
},
];
me.callParent();
},
onGetValues: function(values) {