Change managed option groups to cleanup:always, update:unmodified
Aside from a lot of whitespace noise (because civilint
), this changes the policy for updating managed records:
Before
Update: always
would override any user changes. That's frustrating when you want to alter the label of something.
Cleanup: unused
is unnecessarily guarded. If the extension gets uninstalled, then these option values should definitely be deleted.
After
Update: unmodified
respects user changes.
Cleanup: always
ensures the extension can be completely uninstalled.