[pve-devel] [PATCH widget-toolkit] ui: remove extra parenthesis from check to avoid eslint error

Dominik Csapak d.csapak at proxmox.com
Wed Feb 15 16:03:16 CET 2023


On 2/15/23 15:46, Stefan Sterz wrote:
> with the additional parenthesis eslint throws an error due to the
> "no-extra-parens" rule that avoids unnecessary parenthesis. remove
> them to get rid of the error.
> 
> Signed-off-by: Stefan Sterz <s.sterz at proxmox.com>
> ---
> i am personally not too happy with this eslint requirement here, but
> the widget toolkit won't build otherwise.
> 
>   src/window/DiskSmart.js | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/window/DiskSmart.js b/src/window/DiskSmart.js
> index b538ea1..be52a4e 100644
> --- a/src/window/DiskSmart.js
> +++ b/src/window/DiskSmart.js
> @@ -159,7 +159,7 @@ Ext.define('Proxmox.window.DiskSmart', {
>   	    {
>   		name: 'real-value',
>   		// FIXME remove with next major release (PBS 3.0)
> -		calculate: data => (data.normalized ?? false) ? data.raw : data.value,
> +		calculate: data => data.normalized ?? false ? data.raw : data.value,
>   	    },
>   	    {
>   		name: 'real-normalized',


imho the '?? false' part is unnecessary since that expression will already be coerced to a
boolean and null and undefined are falsy...

so i'd be happy with
----
data.normalized ? data.raw : data.value,
----

any thoughts @thomas?






More information about the pve-devel mailing list