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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chx’s picture

I like it but dblog imo needs to be there.

catch’s picture

FileSize
3.94 KB

I 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.

macgirvin’s picture

subscribe

Dries’s picture

I'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.

catch’s picture

OK 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).

catch’s picture

catch’s picture

FileSize
3.94 KB
alpritt’s picture

The 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.

catch’s picture

Title: 'Advanced user' profile » 'Minimum' install profile
Status: Needs work » Needs review
FileSize
3.98 KB

Here 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.

Dries’s picture

It looks to me as it is too minimal. Wasn't the goal to create some dummy data and to setup a vocabulary?

Dries’s picture

Ah, 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.

Dries’s picture

I'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. ;)

catch’s picture

Assigned: Unassigned » catch
FileSize
752 bytes
3.97 KB

How 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.

Dries’s picture

Status: Needs review » Fixed

Committed 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.

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

Xano’s picture

Status: Closed (fixed) » Active
  • There are two profiles called 'Drupal' now, which may be confusing.
  • Beginning users that know how Drupal works, but don't consider themselves experts probably don't want dummy data, but perhaps also don't check out the expert profile since they think it's not meant for them.

I would change the profile names to the following:

Default
Enables some basic Drupal functionality and adds dummy data.
Custom
Enables only minimum functionality.
catch’s picture

I'd be fine with such a change. It also mirrors how desktop software does this.

Xano’s picture

Status: Active » Needs review
FileSize
5.22 KB

Here's the patch that applies the changes as mentioned in #16.

Title: 'Minimum' install profile » 'Minimum' install profile
Status: Needs review » Needs work

Patch failed to apply. More information can be found at http://testing.drupal.org/node/14338. If you need help with creating patches please look at http://drupal.org/patch/create

Xano’s picture

I 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.

Xano’s picture

Status: Needs work » Needs review
CorniI’s picture

Status: Needs review » Reviewed & tested by the community

Kaaskop wanted to know if this patch still applies cleanly to HEAD, and yes it does, so webchick here we go :)

webchick’s picture

Status: Reviewed & tested by the community » Needs work

If we're changing the name, we ought to change the function names to match.

Xano’s picture

Assigned: catch » Xano
Status: Needs work » Needs review
FileSize
5.08 KB
Xano’s picture

FileSize
7.99 KB

I 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.

Dave Reid’s picture

FileSize
9.51 KB

Here's the patch for HEAD that adds and deletes files.

keith.smith’s picture

Title: 'Minimum' install profile » 'Minimum' install profile
Anonymous’s picture

Status: Needs review » Needs work

The last submitted patch failed testing.

Xano’s picture

Could somebody update that last patch? It's my code, but I'm no good at creating patches that add or delete files.

Xano’s picture

Status: Needs work » Needs review

I 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.

Xano’s picture

Somehow 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?

keith.smith’s picture

Oops. 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.

Xano’s picture

Status: Needs review » Closed (duplicate)

Fixed in another issue already.

Xano’s picture

Assigned: Xano » Unassigned