Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.The site creation form is not a straight forward form.
When there are multiple platforms that can be selected for creation, it will display a set of radio buttons to select the correct platform.
Depending on the platform you select, there will be a selection of installation profiles for that platform.
Depending on the profile you select, there will be a selection of languages that install profile is available in.
Currently I am displaying all the options available (but have not yet added the validation on the form submission to stop you from making invalid choices yet).
This is a very unfriendly way of doing this functionality. We need to be able to select the platform (if selectable), which populates the profiles , which populates the languages.
I'm unsure of the best way to implement this in drupal 5. We might need to settle for single select boxes for these options.
I'd like to try and avoid doing multi-page forms too.
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | hosting_site_form.js_.txt | 1.54 KB | xiong |
| #13 | hosting_site_form.css_.txt | 131 bytes | xiong |
| #13 | loading.gif | 14.41 KB | xiong |
| #12 | hosting.x.oct_.22.patch | 6.53 KB | xiong |
| #11 | hosting.oct_.21.x.patch | 7.87 KB | xiong |
Comments
Comment #1
adrian commentedAfter speaking to Katherine from RCS, it seems we either need to add a dependency to http://drupal.org/project/ahah_forms to easily accomplish this, or we need to port hosting to D6 (which has this functionality included).
Need to look more deeply into what's needed for an upgrade to d6, and just add more validation in place of this.
Comment #2
xiong commentedI implemented with AHAH forms, it's functioning in the desired way. Still need to clean up the code.
Comment #3
xiong commentedFinished, please review and test.
Comment #4
xiong commentedforgot to set status as: patch (code needs review) in last post >_< and added dependencies of ahah_forms and dynamic_subform
Comment #5
adrian commentedThis patch is malformed, and doesn't apply.
Could you create it using 'cvs diff -u > patchfile.diff' inside the checkout.
Comment #6
adrian commentedComment #7
xiong commentedsorry for not using cvs diff..
here's the new patch with 'cvs diff -u > patchfile.diff'.
please review and test.
Comment #8
adrian commentedThis works beautifully on the user interface level, except it doesn't seem to be saving the profile or language correctly.
it always uses an empty profile int he node created, and always results in the language being empty.
But s far this is exactly what we need.
Comment #9
adrian commentedThe hostmaster.profile also needs the dependencies added.
Comment #10
adrian commentedAlso realized we need clean_urls enabled for this. which is ok, since we force them to use apache.
anyway, i tested it with clean urls, and still no luck.
Comment #11
xiong commentedthe new patch against new checkout form cvs HEAD.
two problems encountered:
* the AHAH is generating the subform twice in an wrong way;
* the AHAH is bypassing the main form validation.
at the same time, will try to just use jquery (not AHAH) to meet the requirement.
Comment #12
xiong commentednew patch with custom jquey implemented
Comment #13
xiong commentedthe jquery file, css file and a default gif image (display when ajax loading) of the new patch, they should go in "hosting/site/" folder
Comment #14
adrian commentedCommitted. This needs some clearing up of the interface
Ac, can you take a look at it ?
Comment #15
adrian commentedthis was working till i broke it
here's the resulting ticket. : http://drupal.org/node/325415
Comment #16
acIf you submit the form after selecting a platform (and a profile) but forget to put in the site URL (so there is a drupal error message), then the profiles that are shown for your selected platform revert to the profile for the first platform listed.
Comment #17
adrian commentedThis is fixed in HEAD now.
Comment #18
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.