Re: [PATCH manager v2] fix #7896: ui: qemu: show effective machine version for Windows guests

["Jakob Klocker" <j.klocker@proxmox.com>]

On Mon Aug 31, 2026 at 10:03 AM CEST, Dominik Csapak wrote:
> ok i misunderstood what you meant from the comment, i get it now and
> could reproduce the misbehavior with the lines deleted.
> (i simply looked for a different thing)
>
> there is a much easier method instead of manipulating the value itself 
> and notifying the viewmodel:
>
> since both the effectiveVersion field and the hint are only modified
> once, we don't need to use the viewmodel here at all?
>
> just have the field value set via the normal 'value' object
> but set 'submitValue: false' so it does not get submitted on
> clicking ok. (the setValue parent handler then calls resetOriginal value 
> anyway)
>
> same for the hint, we can show/hide it in the setValue method, but
> don't have to use the viewmodel for it.
>
> In general the viewmodel is only really helpful if we change their
> values dynamically while the user has the editor open
> which isn't the case for the 'setValue' method (this is only
> called once on opening)

Thanks for the detailed explanation, makes a lot more sense
implementing it that way! As you mentioned, viewmodel isn't necessary
here -- not sure why I opted for it. I'll send a revision using the
'value' approach.