[pve-devel] applied: [PATCH widget-toolkit 1/1] combogrid: initialze value with [] by default
Thomas Lamprecht
t.lamprecht at proxmox.com
Tue Nov 14 09:04:43 CET 2023
Am 19/07/2023 um 14:11 schrieb Dominik Csapak:
> we have to initialize the value of a combogrid to something (else extjs
> does not initialize everything in the object *sometimes* for yet unknown
> reasons), but the empty string is wrong.
>
> we already have at least two places where we set the default value to []
> (namely NodeSelector and ha GroupSelector) with the comment:
>
> // set default value to empty array, else it inits it with
> // null and after the store load it is an empty array,
> // triggering dirtychange
>
> so it makes sense to always set it to that by default. This only ever is
> relevant when the combogrid has `allowBlank: true`, since if it does not
> it's either invalid (and thus "dirty") or it has a selected value anyway
>
> this should make the manual setting of
>
> value: [],
>
> unnecessary in the child classes. We can even remove it direcly in the
> NetworkSelector.
>
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
> src/form/ComboGrid.js | 2 +-
> src/form/NetworkSelector.js | 4 ----
> 2 files changed, 1 insertion(+), 5 deletions(-)
>
>
applied, thanks!
More information about the pve-devel
mailing list