Radio as button group doesn't trigger CDD

[trob]

http://www.fabrikar.com/forums/index.php?threads/where-for-dbjoin-using-data-in-current-form.38195/
ok, if radio element is set to buttongroup=no

[trob]

I assume the same reason is here
http://www.fabrikar.com/forums/index.php?threads/ajax-calculation-watch-elements-does-not-work-on-radio-button-groups.38512/

This may be related
http://fabrikar.com/forums/index.php?threads/javascript-not-fired-when-using-button-group-for-radiogroup.36756/

[trob]

It seems button groups need "onclick" to trigger (e.g. in ajax calc, CDD,...)
http://www.fabrikar.com/forums/index.php?threads/show-or-hide-field-based-on-another-yes-no-field.39584/#post-199044

[cheesegrits]

Hmmm. the only solution I can think of for that is to add an element model method, like changeEvent(), which in the main element model just return 'change', but can be overridden in specific element plugins to return 'click' if that's what the element knows it needs. Then in the calc element's elementJavascript(), where it builds the $opts->observe array (a list of element full names used by the calc), loop through the that array, load up the element model by name, call $elementModel->changeEvent() for each one, build an $opts->observeEvents array, and in the calc's attachToForm() method in the JS, use that for attaching the event that triggers the calc.

Long way round the houses, but I don't see any other way round it.

[trob]

It's not only calc but also CDD

[cheesegrits]

Ignore all that. I can just add a getChangeEvent() in element.js, and override it in specific elements, like yesno.js, then call that in the calc's init.

[cheesegrits]

OK, should be fixed as of f648384

[Duke3D]

Has the fix been confirmed? Does something need to be added to the element setup for calc to be triggered by a button group? Adding elements in watchlist with ..._raw did not help.

[trob]

The fix was for yesno only, radiobutton is still missing.
#1531

I don't know if it can be removed from yesno then (extends radio)

[cheesegrits]

Should be fixed in 5267bd4