Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
There are several tasks in the Commons installer where configuration is made without the various benefits of Features.
For example, drupal_commons_config_vars() sets variables, when these could easily exported with Strongarm + Features.
Let's look at these tasks and see which ones would be worth refactoring as Features exports.
Comments
Comment #1
mstef CreditAttribution: mstef commentedThese were originally done like this, because we wanted variables set by default as per our "suggestion". These are variables that we expect admins to change, so we didn't want them in Features, because changing them would result in Features being overridden, and reverting features would obviously revert these as well.
Hope that makes sense..
Comment #2
ezra-g CreditAttribution: ezra-g commentedCouldn't you just put them in a non-required feature?
Comment #3
mstef CreditAttribution: mstef commentedI suppose you could - but that seems weird (to me). I don't see any harm in setting a few initial defaults this way, but that's just my take.
Comment #4
lsolesen CreditAttribution: lsolesen commented