search

mecachisenros / aws

CiviCRM AWS
Other
7 stars 2 forks source link

Verify New Identity sets "Email Feedback Forwarding" enabled when unchecked #7

Open mattwire opened 4 years ago

mattwire commented 4 years ago

Email Feedback Forwarding is always set to enabled when a new identity is created via CiviCRM

mecachisenros commented 4 years ago

@mattwire I've added a workaround for this via e454d3a243e0b79365ad5b8e80a5eb5c7f221cb5

I'm slightly confused by the behaviour though, none of the checkbox fields save if unchecked.

$form->exportValues() ignores all unchecked boxes, meaning the fields are not present in the submitted data, I would have expected a falsy value there, this happens on core forms as well, as an example try editing a Message template, at the very bottom, uncheck the Enabled? setting, it doesn't disable the template.

Thoughts?

mattwire commented 4 years ago

@mecachisenros I think setting the html type to advcheckbox should make that work