[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