It was added so that the value of a field could be validated before being used to select a case for a "union" widget.
Unfortunately I have not worked with those new/class/struct/union widgets, so I do not really understand what it is for.
We should change it to only validate the field widgets that are actually put to this use (selecting a case for a "union" widget)
But this is not determinable at the moment, is it?
Whether we should hide the messages in that case like the proposed patch does, I do not know.
If the validation is needed or at least good for further processing, what should be the sense of not showing an error?
When "union" get replaced by the redesigned "choice" widget we will still have this issue, but it will get triggered by the "expr" attributes' expressions referencing the values in various widgets.
Ok, just a proposal, don't know if you can live with it: If there is a refactoring still to be done and the validation is only "good" not necessary, what about removing this stuff (at least additional getValue()) for now. This will make all other fields behave normally again while it seems not to break the new widgets (which are furthermore not that common in use I guess).
WDYT?
Tim? Any response?
Joerg
