Proxmox/pve-manager
5
0
Fork 0
Code Pull Requests Releases Activity

ui: realm sync: let users set 'remove vanished options' independently

Browse Source 
and label it correctly. The 'Properties' option was mislabeled,
it means 'remove vanished fields from synced users', not 'remove
fields from vanished users'.

So don't couple that with the 'Entries' option, and update the label
to convey the right meaning.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
This commit is contained in:
Dominik Csapak 2022-05-02 15:15:20 +02:00 committed by Thomas Lamprecht
parent d9455193f1
commit 350f4dcb0e
2 changed files with 2 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
www/manager6/dc/AuthEditLDAP.js
View File

@ -99,22 +99,6 @@ Ext.define('PVE.panel.LDAPSyncInputPanel', {
extend: 'Proxmox.panel.InputPanel',
xtype: 'pveAuthLDAPSyncPanel',
controller: {
xclass: 'Ext.app.ViewController',
control: {
'proxmoxcheckbox[name=remove-vanished-entry]': {
change: function(_f, removeEntries) {
let propertyField = this.getView()
.down('proxmoxcheckbox[name=remove-vanished-properties]');
propertyField.setDisabled(removeEntries);
if (removeEntries) {
propertyField.setValue(true);
}
},
},
},
},
editableAttributes: ['email'],
editableDefaults: ['scope', 'enable-new'],
default_opts: {},
@ -312,7 +296,7 @@ Ext.define('PVE.panel.LDAPSyncInputPanel', {
xtype: 'proxmoxcheckbox',
fieldLabel: gettext('Properties'),
name: 'remove-vanished-properties',
boxLabel: gettext('Remove properties from vanished users.'),
boxLabel: gettext('Remove vanished properties from synced users.'),
},
],
},

2
www/manager6/dc/SyncWindow.js
View File

@ -138,7 +138,7 @@ Ext.define('PVE.dc.SyncWindow', {
xtype: 'proxmoxcheckbox',
fieldLabel: gettext('Properties'),
name: 'remove-vanished-properties',
boxLabel: gettext('Remove properties from vanished users.'),
boxLabel: gettext('Remove vanished properties from synced users.'),
},
],
},