]> git.proxmox.com Git - pve-manager.git/commitdiff
ui: dc/TokenEdit: fix editing existing tokens
authorDominik Csapak <d.csapak@proxmox.com>
Fri, 12 Feb 2021 15:01:15 +0000 (16:01 +0100)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Fri, 19 Feb 2021 15:37:51 +0000 (16:37 +0100)
we always have to calculate the url correctly, not only on creation
otherwise we try to edit the token by doing a 'PUT' request on
/access/users
and not on
/access/users/USERID/token/TOKENID

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
www/manager6/dc/TokenEdit.js

index 3edf18cc9034e4501c3bb4d878a0695933b8d3ae..7039249cfaaacd36832703e89b905f8420e9ad2b 100644 (file)
@@ -21,14 +21,12 @@ Ext.define('PVE.dc.TokenEdit', {
            let me = this;
            let win = me.up('pveDcTokenEdit');
            win.url = '/api2/extjs/access/users/';
-           if (win.isCreate) {
-               let uid = encodeURIComponent(values.userid);
-               let tid = encodeURIComponent(values.tokenid);
-               delete values.userid;
-               delete values.tokenid;
+           let uid = encodeURIComponent(values.userid);
+           let tid = encodeURIComponent(values.tokenid);
+           delete values.userid;
+           delete values.tokenid;
 
-               win.url += `${uid}/token/${tid}`;
-           }
+           win.url += `${uid}/token/${tid}`;
            return values;
        },
        column1: [
@@ -36,8 +34,8 @@ Ext.define('PVE.dc.TokenEdit', {
                xtype: 'pmxDisplayEditField',
                cbind: {
                    editable: (get) => get('isCreate') && !get('fixedUser'),
-                   submitValue: (get) => get('isCreate') || get('fixedUser'),
                },
+               submitValue: true,
                editConfig: {
                    xtype: 'pveUserSelector',
                    allowBlank: false,
@@ -54,6 +52,7 @@ Ext.define('PVE.dc.TokenEdit', {
                },
                name: 'tokenid',
                fieldLabel: gettext('Token ID'),
+               submitValue: true,
                minLength: 2,
                allowBlank: false,
            },