I think we should add more dummy content to the default installation profile - participants at the UMN usability testing asked for this, and it's been mentioned elsewhere. I've opened another issue to enable a 'tags' vocabulary by default here: http://drupal.org/node/261869
The main issue with adding dummy content and settings to default.profile however, is that people installing Drupal other than for the first couple of times will have to sit there deleting it before they can get any work done. Attached is a 'Drupal Advanced" profile - which is just the default profile with nearly everything removed, no content types, no optional modules enabled.
Having both a 'default' and 'advanced' profile would be consistent with a lot of desktop software - where you click an advanced option to install a lighter set of options and set up your own preferences. We might want to change 'advanced' to 'minimum' or something.
Attached is a new advanced.profile which should go in profiles/advanced (you'll need to make the folder). I won't make a proper patch until there's been some feedback, but this at least gives an idea what it'd look like. Note that at the moment, this doesn't really remove all that much, but I'm hoping we can add more things to the default profile as time goes on.
Comment | File | Size | Author |
---|---|---|---|
#26 | profiles.patch | 9.51 KB | Dave Reid |
#25 | profiles.zip | 7.99 KB | Xano |
#24 | rename_install_profiles_3.patch | 5.08 KB | Xano |
#20 | rename_install_profiles_2.patch | 5.14 KB | Xano |
#18 | rename_install_profiles.patch | 5.22 KB | Xano |
Comments
Comment #1
chx CreditAttribution: chx commentedI like it but dblog imo needs to be there.
Comment #2
catchI considered leaving dblog but wasn't sure about leaving only one module, that's probably the main candidate for leaving enabled and would be fine with me.
By the way, that patch is completely errant here's the file I meant to upload.
Comment #3
macgirvin CreditAttribution: macgirvin commentedsubscribe
Comment #4
Dries CreditAttribution: Dries commentedI'd support this. It would be a great thing to do. Re-roll and commit? I'm sure we'll be refining this install profile over time.
Comment #5
catchOK here it is with 'dblog' included. Just noticed that locale gets enabled anyway though, so do we need to add a task at the end to disable it? Leaving at needs work (and still not a patch, not sure how to add folders in cvs).
Comment #6
catchComment #7
catchComment #8
alpritt CreditAttribution: alpritt commentedThe rename to something like 'minimum' is a good idea I think. It's not advanced in the same way that desktop applications have advanced modes (in fact it is frequently the opposite as it often means enabling more features). 'Minimum' describes what the install profile actually is.
Comment #9
catchHere it is as minimum.profile - and with locale disabled in minimum_profile_tasks()
Might need some more tweaking, but back to review to get some feedback.
Comment #10
Dries CreditAttribution: Dries commentedIt looks to me as it is too minimal. Wasn't the goal to create some dummy data and to setup a vocabulary?
Comment #11
Dries CreditAttribution: Dries commentedAh, I get it now. The idea is to add stuff to the default profile, and to keep the minimum profile the minimum profile. Makes sense.
Comment #12
Dries CreditAttribution: Dries commentedI'd suggest two changes though:
1. Let's rename 'minimum' to 'expert' as that better communicates the target audience and looks at it from an end-user point of view.
2. Let's add an entry to CHANGELOG.txt.
If you do a re-roll, I'll commit it the same day. ;)
Comment #13
catchHow can I resist an offer like that :)
Here's expert.profile as a .txt file (not sure how to roll a new folder into a patch), and a patch for CHANGELOG.txt
I'm still not happy with locale.module being enabled then immediately disabled again afterwards, but don't see a way 'round this the way profiles currently work.
Comment #14
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD.
This opens the doors to add more things to the default profile and/or to create additional install profiles for core. It also helps set an example for more install profiles in the contributed modules repository.
Comment #15
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #16
XanoI would change the profile names to the following:
Comment #17
catchI'd be fine with such a change. It also mirrors how desktop software does this.
Comment #18
XanoHere's the patch that applies the changes as mentioned in #16.
Comment #20
XanoI justed tried patching this version against the latest dev and it works. I couldn't figure out how to let the patch delete the 'Expert' profile though.
Comment #21
XanoComment #22
CorniI CreditAttribution: CorniI commentedKaaskop wanted to know if this patch still applies cleanly to HEAD, and yes it does, so webchick here we go :)
Comment #23
webchickIf we're changing the name, we ought to change the function names to match.
Comment #24
XanoComment #25
XanoI attached a zipped version of the modified
/profiles
directory. Since I cannot create a patch for removed and added files somebody else has got to do this.Comment #26
Dave ReidHere's the patch for HEAD that adds and deletes files.
Comment #27
keith.smith CreditAttribution: keith.smith commentedComment #28
Anonymous (not verified) CreditAttribution: Anonymous commentedThe last submitted patch failed testing.
Comment #29
XanoCould somebody update that last patch? It's my code, but I'm no good at creating patches that add or delete files.
Comment #30
XanoI can confirm this patch still works with the latest HEAD. I think the patch failed automatic testing due to known problems with HEAD a few days ago.
Comment #31
XanoSomehow people (Keith? :-P ) started working on a similar issue: #297181: Modify descriptions on initial installation form.. Are going through with this one or do we let it be like it is now?
Comment #32
keith.smith CreditAttribution: keith.smith commentedOops. I was indeed unaware of this issue, or would have tried to take it into account when changing those names and descriptions in the other issue. I haven't read through this issue yet, so I'm not sure how, of if, it fits in. I'll read through it though.
Comment #33
XanoFixed in another issue already.
Comment #34
Xano