On 1/31/20 11:48 AM, Dominik Csapak wrote:
> since any change to them is currently silently ignored by the backend
> and should result in an error anyway
> 
> partially fixes #2575
> 
> Signed-off-by: Dominik Csapak <d.csa...@proxmox.com>
> ---
>  www/manager6/dc/RoleView.js | 10 +++-------
>  1 file changed, 3 insertions(+), 7 deletions(-)
> 
> diff --git a/www/manager6/dc/RoleView.js b/www/manager6/dc/RoleView.js
> index ee8571ba..e21e2920 100644
> --- a/www/manager6/dc/RoleView.js
> +++ b/www/manager6/dc/RoleView.js
> @@ -45,7 +45,7 @@ Ext.define('PVE.dc.RoleView', {
>               return;
>           }
>  
> -         if (rec.data.special === "1") {
> +         if (!!rec.data.special) {
>               return;
>           }
>  
> @@ -108,9 +108,7 @@ Ext.define('PVE.dc.RoleView', {
>                   disabled: true,
>                   selModel: sm,
>                   handler: run_editor,
> -                 enableFn: function(record) {
> -                     return record.data.special !== '1';
> -                 }
> +                 enableFn: (rec) => !rec.data.special,
>               },
>               {
>                   xtype: 'proxmoxStdRemoveButton',
> @@ -119,9 +117,7 @@ Ext.define('PVE.dc.RoleView', {
>                       reload();
>                   },
>                   baseurl: '/access/roles/',
> -                 enableFn: function(record) {
> -                     return record.data.special !== '1';
> -                 }
> +                 enableFn: (rec) => !rec.data.special,
>               }
>           ]
>       });
> 

applied, thanks!

_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to