[pve-devel] [PATCH widget-toolkit/manager] improve combogrid default value handling

Fabian Grünbichler f.gruenbichler at proxmox.com
Wed Jul 19 14:22:54 CEST 2023


somewhat plays into #4840 , in that we now no longer do a bogus GET call
for the 'null' storage and display a misleading permission error as a
result, but there still could be a nice (big?) error message about the
general lack of (accessible) storages ;)

https://bugzilla.proxmox.com/show_bug.cgi?id=4840

On July 19, 2023 2:11 pm, Dominik Csapak wrote:
> the default value for combogrids is '', but we often need to set it to
> [], to avoid issues with dirty tracking. Fix this by setting it to [] by
> default, making it unnecessary to carry the workaround + comment around
> in child classes.
> 
> the first patch of pve-manager is a bit unrelated but popped up during
> development (can be applied independently)
> 
> the second patch of pve-manager is also a bit independent, but it fixes
> wrong dirty tracking in the firewall rule edit window iff the widget
> toolkit patch is also present
> 
> the third manager patch depends on the widget toolkit patch
> 
> proxmox-widget-toolkit:
> 
> Dominik Csapak (1):
>   combogrid: initialze value with [] by default
> 
>  src/form/ComboGrid.js       | 2 +-
>  src/form/NetworkSelector.js | 4 ----
>  2 files changed, 1 insertion(+), 5 deletions(-)
> 
> pve-manager:
> 
> Dominik Csapak (3):
>   ui: ipset: make ip/cidr required
>   ui: don't set the default value of combogrids to ''
>   ui: don't set the default value of combogrids to []
> 
>  www/manager6/form/NodeSelector.js  | 5 +----
>  www/manager6/grid/FirewallRules.js | 2 --
>  www/manager6/ha/GroupSelector.js   | 1 -
>  www/manager6/panel/IPSet.js        | 2 +-
>  4 files changed, 2 insertions(+), 8 deletions(-)
> 
> -- 
> 2.30.2
> 
> 
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel at lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> 
> 
> 





More information about the pve-devel mailing list