Re: [PATCH pve-manager] Web UI: VM creation Wizard: Pupulate EFI and TPM storage by default

[Jonas Theisen <j.theisen@proxmox.com>]

On 8/5/26 10:49, Elias Huhsovitz wrote:
> Nice quality of life change! Just one small bug:
>
> The "EFI Storage" and "TPM Storage" fields are only populated the first
> time the options are selected.
>
> Repeatedly selecting/unselecting the checkboxes "Addi EFI Disk" and "Add
> TPM", will not cause the respective storage fields to be populated, reverting
> to the old behaviour.
According to my very limited knowledge of Javascript and our implementation
this might be expected or working as intended?

Since the autoSelect property is mapped as a CBind in
[pve-manager.git] / www / manager6 / form / IsoSelector.js
> 76             cbind: {
> 77                 nodename: '{nodename}',
> 78                 autoSelect: '{insideWizard}',
> 79                 insideWizard: '{insideWizard}',
> 80                 disabled: '{disabled}',
> 81                 labelWidth: '{labelWidth}',
> 82                 labelAlign: '{labelAlign}',
> 83             }
and [proxmox-widget-toolkit.git] / src / mixin / CBind.js states
> 2  * The Proxmox CBind mixin is intended to supplement the 'bind' 
> mechanism
> 3  * of ExtJS. In contrast to the 'bind', 'cbind' only acts during the 
> creation
> 4  * of the component, not during its lifetime. It's only applied once 
> before
> 5  * the 'initComponent' method is executed, and thus you have only access
> 6  * to the basic initial configuration of it.
i assume the selector is not recreated after un- and reticking the checkbox.

For regular disks this situation never arose afaict since the entry is 
always
freshly created.

I currently don't know where to look for the culprit here or maybe am even
on the wrong track and am open for suggestions.