Fix setting outBound_option readonly in UI #16774
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
Follow on from #16451
Fix setting the SMTP outbound option readonly.
Before
outBound_option readonly ignored in UI. It was actually working, just not shown as frozen in UI.
After
Elements frozen if you set in civicrm.settings.php:
$civicrm_setting['mailing']['mailing_backend']['outBound_option'] = 5;
Technical Details
readonly attribute was set after adding the element!
Comments
@yashodha you were interested in this? I think it actually already works with the SMTP authentication setting because of #16451