[pve-devel] applied-series: [PATCH manager 1/4] ui: api token: remove record context from TokenView add button handler
Thomas Lamprecht
t.lamprecht at proxmox.com
Mon Nov 11 18:50:18 CET 2024
Am 31.10.24 um 14:46 schrieb Daniel Kral:
> Removes the record ("rec") variable from the TokenView, as it is always
> undefined, because the "Add" button is a ExtJS Button and therefore the
> button handler doesn't pass a third parameter as ProxmoxButton does.
>
> Signed-off-by: Daniel Kral <d.kral at proxmox.com>
> ---
> Please correct me if I'm missing something here, but I tested this
> through Firefox's JavaScript debugger and assumed that tbar items are
> "ExtJS.button.Button" by default, except when specifing another xtype.
>
> This will make the fix for #5722 clearer to implement.
>
> www/manager6/dc/TokenView.js | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
>
applied series with Fabian's T-b, thanks!
More information about the pve-devel
mailing list