Re: [PATCH manager 6/7] ui: qga: call parameter me instead of this

[Fiona Ebner <f.ebner@proxmox.com>]

Am 25.02.26 um 4:46 PM schrieb Maximiliano Sandoval:
> This is how it is generally done, we will use `me` in the next commit.
> 
> Signed-off-by: Maximiliano Sandoval <m.sandoval@proxmox.com>

There is also a lot of code using this.something directly and we don't
have something written about this in the JavaScript style guide, so I
wouldn't mind either way:

Reviewed-by: Fiona Ebner <f.ebner@proxmox.com>

> ---
>  www/manager6/form/AgentFeatureSelector.js | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/www/manager6/form/AgentFeatureSelector.js b/www/manager6/form/AgentFeatureSelector.js
> index a3dda630..1a87182d 100644
> --- a/www/manager6/form/AgentFeatureSelector.js
> +++ b/www/manager6/form/AgentFeatureSelector.js
> @@ -104,6 +104,8 @@ Ext.define('PVE.form.AgentFeatureSelector', {
>      },
>  
>      setValues: function (values) {
> +        let me = this;
> +
>          let res = PVE.Parser.parsePropertyString(values.agent, 'enabled');
>          if (!Ext.isDefined(res['freeze-fs-on-backup'])) {
>              res['freeze-fs-on-backup'] = 1;
> @@ -112,6 +114,6 @@ Ext.define('PVE.form.AgentFeatureSelector', {
>              res['guest-fsfreeze'] = 1;
>          }
>  
> -        this.callParent([res]);
> +        me.callParent([res]);
>      },
>  });