Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
With more complex installation profiles coming up, it would be really useful to have a consistent way of passing parameters to installation profiles forms.
Comment | File | Size | Author |
---|---|---|---|
#3 | 1198956-site-install-additional-parameters.patch | 3.57 KB | Damien Tournoud |
#1 | 1198956-site-install-additional-parameters.patch | 2.16 KB | Damien Tournoud |
Comments
Comment #1
Damien Tournoud CreditAttribution: Damien Tournoud commentedHere is probably one of the simplest way to do that:
Comment #2
moshe weitzman CreditAttribution: moshe weitzman commentedLooks pretty clean to me. Can anyone suggest a profile to use for testing this? Do we need to change anything to support D6 also?
Comment #3
Damien Tournoud CreditAttribution: Damien Tournoud commentedDrupal 6 support would be hairy, because there is not consistent form support during installation in there. I suggest supporting this only for Drupal 7 onwards.
The Git version of Commerce Kickstart (
git clone --branch 7.x-1.x damz@git.drupal.org:project/commerce_kickstart.git
) has a very simple configuration form that let you select if you want some demo content to be created or not (currently, the dummy content is exactly one product):Also, the standard core configuration form has some parameters that have no explicit options in
drush site-install
:I updated the
site-install --help
documentation in this new version of the same patch.Comment #4
moshe weitzman CreditAttribution: moshe weitzman commentedCommitted to master and 4.x. Thanks.
Comment #5
msonnabaum CreditAttribution: msonnabaum commentedThis was causing an error when drush_core_site_install() would get called without any addition args, which left $form_options as NULL instead of an array. THis was causing tests to fail.
I committed 2fcba50c1324413ef844922eee2d673838980736 which fixes it for me.
Comment #7
girishmuraly CreditAttribution: girishmuraly commentedUnchecking checkbox form elements through the above solution does not work properly. See #1247338: Site Install extra options checkboxes are difficult