Profiles should be permantently loaded in memory and they should have access to all drupal hooks. From: Adrian Rossouw This patch adds the install profile to the results returned from system_get_module_data, and therefore adds the profiles to the system table. The existing default_profile_setup was no longer necessary, and is now default_install in the new default.install file. Patch for http://drupal.org/node/509398 Also inadvertantly resolves http://drupal.org/node/80272 --- includes/common.inc | 40 +++-- includes/install.inc | 21 +- includes/module.inc | 5 + install.php | 12 + modules/simpletest/drupal_web_test_case.php | 5 - modules/system/system.module | 12 + profiles/default/default.info | 1 profiles/default/default.install | 216 ++++++++++++++++++++++++++ profiles/default/default.profile | 228 --------------------------- profiles/expert/expert.info | 1 profiles/expert/expert.install | 71 ++++++++ profiles/expert/expert.profile | 81 ---------- 12 files changed, 359 insertions(+), 334 deletions(-) create mode 100644 profiles/default/default.install create mode 100644 profiles/expert/expert.install diff --git includes/common.inc includes/common.inc index ef8a93a..a646de9 100644 --- includes/common.inc +++ includes/common.inc @@ -120,6 +120,32 @@ function drupal_get_region_content($region = NULL, $delimiter = ' ') { } /** + * Get the name of the currently active install profile. + * + * When this function is called during Drupal's initial installation process, + * the name of the profile that's about to be installed is stored in the global + * installation state. At all other times, the standard Drupal systems variable + * table contains the name of the current profile, and we can call variable_get() + * to determine what one is active. + * + * @return $profile + * The name of the install profile. + */ +function drupal_get_profile() { + global $install_state; + + if (isset($install_state['parameters']['profile'])) { + $profile = $install_state['parameters']['profile']; + } + else { + $profile = variable_get('install_profile', 'default'); + } + + return $profile; +} + + +/** * Set the breadcrumb trail for the current page. * * @param $breadcrumb @@ -3724,20 +3750,10 @@ function drupal_cron_cleanup() { * An array of file objects of the specified type. */ function drupal_system_listing($mask, $directory, $key = 'name', $min_depth = 1) { - global $install_state; $config = conf_path(); - // When this function is called during Drupal's initial installation process, - // the name of the profile that's about to be installed is stored in the global - // installation state. At all other times, the standard Drupal systems variable - // table contains the name of the current profile, and we can call variable_get() - // to determine what one is active. - if (isset($install_state['parameters']['profile'])) { - $profile = $install_state['parameters']['profile']; - } - else { - $profile = variable_get('install_profile', 'default'); - } + $profile = drupal_get_profile(); + $searchdir = array($directory); $files = array(); diff --git includes/install.inc includes/install.inc index d4198c6..71e05d4 100644 --- includes/install.inc +++ includes/install.inc @@ -166,16 +166,12 @@ function drupal_set_installed_schema_version($module, $version) { */ function drupal_install_profile_name() { global $install_state; - $profile = $install_state['parameters']['profile']; - static $name = NULL; - if (!isset($name)) { - // Load profile details. - $function = $profile . '_profile_details'; - if (function_exists($function)) { - $details = $function(); - } - $name = isset($details['name']) ? $details['name'] : 'Drupal'; + if (isset($install_state['profile_info']['name'])) { + $name = $install_state['profile_info']['name']; + } + else { + $name = 'Drupal'; } return $name; @@ -522,6 +518,9 @@ function drupal_verify_profile($install_state) { $present_modules[] = $present_module->name; } + // We have already verified that the install profile is present. + $present_modules[] = drupal_get_profile(); + // Verify that all of the profile's required modules are present. $missing_modules = array_diff($info['dependencies'], $present_modules); @@ -1087,6 +1086,10 @@ function install_profile_info($profile, $locale = 'en') { $info['dependencies'], ($locale != 'en' && !empty($locale) ? array('locale') : array())) ); + + // drupal_required_modules includes the current profile as a dependency. + array_shift($info['dependencies']); + $cache[$profile] = $info; } return $cache[$profile]; diff --git includes/module.inc includes/module.inc index a4be65f..e24a4e2 100644 --- includes/module.inc +++ includes/module.inc @@ -502,11 +502,16 @@ function module_invoke_all() { function drupal_required_modules() { $files = drupal_system_listing('/\.info$/', 'modules', 'name', 0); $required = array(); + + // An install profile is required and one must always be loaded. + $required[] = drupal_get_profile(); + foreach ($files as $name => $file) { $info = drupal_parse_info_file($file->uri); if (!empty($info) && !empty($info['required']) && $info['required']) { $required[] = $name; } } + return $required; } diff --git install.php install.php index cee14f2..89ed991 100644 --- install.php +++ install.php @@ -733,6 +733,11 @@ function install_system_module(&$install_state) { // variable_set() can be used now that system.module is installed and // Drupal is bootstrapped. $modules = $install_state['profile_info']['dependencies']; + + // The install profile is also a module, which needs to be installed + // after all the dependencies have been. + $modules[] = drupal_get_profile(); + variable_set('install_profile_modules', array_diff($modules, array('system'))); $install_state['database_tables_exist'] = TRUE; } @@ -1424,7 +1429,12 @@ function install_finished(&$install_state) { _drupal_flush_css_js(); // Remember the profile which was used. - variable_set('install_profile', $install_state['parameters']['profile']); + variable_set('install_profile', drupal_get_profile()); + db_update('system') + ->fields(array('weight' => 1000)) + ->condition('type', 'module') + ->condition('name', drupal_get_profile()) + ->execute(); // Cache a fully-built schema. drupal_get_schema(NULL, TRUE); diff --git modules/simpletest/drupal_web_test_case.php modules/simpletest/drupal_web_test_case.php index 159c7e0..29c07b2 100644 --- modules/simpletest/drupal_web_test_case.php +++ modules/simpletest/drupal_web_test_case.php @@ -1062,12 +1062,13 @@ class DrupalWebTestCase extends DrupalTestCase { $this->preloadRegistry(); // Include the default profile + variable_set('install_profile', 'default'); require_once('./profiles/default/default.profile'); $profile_details = install_profile_info('default', 'en'); // Add the specified modules to the list of modules in the default profile. // Install the modules specified by the default profile. - drupal_install_modules($profile_details['dependencies'], TRUE); + drupal_install_modules(array_merge($profile_details['dependencies'], array('default')), TRUE); node_type_clear(); @@ -1086,7 +1087,6 @@ class DrupalWebTestCase extends DrupalTestCase { // Run default profile tasks. $install_state = array(); - default_profile_site_setup($install_state); // Rebuild caches. node_types_rebuild(); @@ -1101,7 +1101,6 @@ class DrupalWebTestCase extends DrupalTestCase { $user = user_load(1); // Restore necessary variables. - variable_set('install_profile', 'default'); variable_set('install_task', 'done'); variable_set('clean_url', $clean_url_original); variable_set('site_mail', 'simpletest@example.com'); diff --git modules/system/system.module modules/system/system.module index 697a876..f17129d 100644 --- modules/system/system.module +++ modules/system/system.module @@ -1744,6 +1744,15 @@ function _system_get_module_data() { // Find modules $modules = drupal_system_listing('/\.module$/', 'modules', 'name', 0); + // Include the install profile in modules that are loaded. + $profile = drupal_get_profile(); + $modules[$profile]->name = $profile; + $modules[$profile]->uri = 'profiles/' . $profile . '/' . $profile . '.profile'; + $modules[$profile]->filename = $profile . '.profile'; + + // Install profile hooks are always executed last. + $modules[$profile]->weight = 1000; + // Set defaults for module info. $defaults = array( 'dependencies' => array(), @@ -1774,6 +1783,9 @@ function _system_get_module_data() { drupal_alter('system_info', $modules[$key]->info, $modules[$key]); } + // The install profile is required. + $modules[$profile]->info['required'] = TRUE; + return $modules; } diff --git profiles/default/default.info profiles/default/default.info index 12a4f29..e87c652 100644 --- profiles/default/default.info +++ profiles/default/default.info @@ -15,3 +15,4 @@ dependencies[] = dblog dependencies[] = search dependencies[] = toolbar dependencies[] = field_ui +files[] = default.profile diff --git profiles/default/default.install profiles/default/default.install new file mode 100644 index 0000000..397f93d --- /dev/null +++ profiles/default/default.install @@ -0,0 +1,216 @@ + 'system', + 'delta' => 'main', + 'theme' => 'garland', + 'status' => 1, + 'weight' => 0, + 'region' => 'content', + 'pages' => '', + 'cache' => -1, + ), + array( + 'module' => 'user', + 'delta' => 'login', + 'theme' => 'garland', + 'status' => 1, + 'weight' => 0, + 'region' => 'sidebar_first', + 'pages' => '', + 'cache' => -1, + ), + array( + 'module' => 'system', + 'delta' => 'navigation', + 'theme' => 'garland', + 'status' => 1, + 'weight' => 0, + 'region' => 'sidebar_first', + 'pages' => '', + 'cache' => -1, + ), + array( + 'module' => 'system', + 'delta' => 'management', + 'theme' => 'garland', + 'status' => 1, + 'weight' => 1, + 'region' => 'sidebar_first', + 'pages' => '', + 'cache' => -1, + ), + array( + 'module' => 'system', + 'delta' => 'powered-by', + 'theme' => 'garland', + 'status' => 1, + 'weight' => 10, + 'region' => 'footer', + 'pages' => '', + 'cache' => -1, + ), + array( + 'module' => 'system', + 'delta' => 'help', + 'theme' => 'garland', + 'status' => 1, + 'weight' => 0, + 'region' => 'help', + 'pages' => '', + 'cache' => -1, + ), + array( + 'module' => 'system', + 'delta' => 'main', + 'theme' => 'seven', + 'status' => 1, + 'weight' => 0, + 'region' => 'content', + 'pages' => '', + 'cache' => -1, + ), + array( + 'module' => 'system', + 'delta' => 'help', + 'theme' => 'seven', + 'status' => 1, + 'weight' => 0, + 'region' => 'help', + 'pages' => '', + 'cache' => -1, + ), + array( + 'module' => 'user', + 'delta' => 'login', + 'theme' => 'seven', + 'status' => 1, + 'weight' => 10, + 'region' => 'content', + 'pages' => '', + 'cache' => -1, + ), + ); + $query = db_insert('block')->fields(array('module', 'delta', 'theme', 'status', 'weight', 'region', 'pages', 'cache')); + foreach ($values as $record) { + $query->values($record); + } + $query->execute(); + + // Insert default user-defined node types into the database. For a complete + // list of available node type attributes, refer to the node type API + // documentation at: http://api.drupal.org/api/HEAD/function/hook_node_info. + $types = array( + array( + 'type' => 'page', + 'name' => st('Page'), + 'base' => 'node_content', + 'description' => st("Use pages for your static content, such as an 'About us' page."), + 'custom' => 1, + 'modified' => 1, + 'locked' => 0, + ), + array( + 'type' => 'article', + 'name' => st('Article'), + 'base' => 'node_content', + 'description' => st('Use articles for time-specific content like news, press releases or blog posts.'), + 'custom' => 1, + 'modified' => 1, + 'locked' => 0, + ), + ); + + foreach ($types as $type) { + $type = node_type_set_defaults($type); + node_type_save($type); + } + + // Default page to not be promoted and have comments disabled. + variable_set('node_options_page', array('status')); + variable_set('comment_page', COMMENT_NODE_HIDDEN); + + // Don't display date and author information for page nodes by default. + variable_set('node_submitted_page', FALSE); + + // Create an image style. + $style = array('name' => 'thumbnail'); + $style = image_style_save($style); + $effect = array( + 'isid' => $style['isid'], + 'name' => 'image_scale_and_crop', + 'data' => array('width' => '85', 'height' => '85'), + ); + image_effect_save($effect); + + // Enable user picture support and set the default to a square thumbnail option. + variable_set('user_pictures', '1'); + variable_set('user_picture_dimensions', '1024x1024'); + variable_set('user_picture_file_size', '800'); + variable_set('user_picture_style', 'thumbnail'); + + $theme_settings = theme_get_settings(); + $theme_settings['toggle_node_user_picture'] = '1'; + $theme_settings['toggle_comment_user_picture'] = '1'; + variable_set('theme_settings', $theme_settings); + + // Create a default vocabulary named "Tags", enabled for the 'article' content type. + $description = st('Use tags to group articles on similar topics into categories.'); + $help = st('Enter a comma-separated list of words to describe your content.'); + + $vid = db_insert('taxonomy_vocabulary')->fields(array( + 'name' => 'Tags', + 'description' => $description, + 'machine_name' => 'tags', + 'help' => $help, + 'relations' => 0, + 'hierarchy' => 0, + 'multiple' => 0, + 'required' => 0, + 'tags' => 1, + 'module' => 'taxonomy', + 'weight' => 0, + ))->execute(); + db_insert('taxonomy_vocabulary_node_type')->fields(array('vid' => $vid, 'type' => 'article'))->execute(); + + // Create a default role for site administrators. + $rid = db_insert('role')->fields(array('name' => 'administrator'))->execute(); + + // Set this as the administrator role. + variable_set('user_admin_role', $rid); + + // Assign all available permissions to this role. + foreach (module_invoke_all('permission') as $key => $value) { + db_insert('role_permission') + ->fields(array( + 'rid' => $rid, + 'permission' => $key, + ))->execute(); + } + + // Update the menu router information. + menu_rebuild(); + + // Save some default links. + $link = array('link_path' => 'admin/structure/menu-customize/main-menu/add', 'link_title' => 'Add a main menu link', 'menu_name' => 'main-menu'); + menu_link_save($link); + + // Enable the admin theme. + db_update('system') + ->fields(array('status' => 1)) + ->condition('type', 'theme') + ->condition('name', 'seven') + ->execute(); + variable_set('admin_theme', 'seven'); + variable_set('node_admin_theme', '1'); +} + diff --git profiles/default/default.profile profiles/default/default.profile index 6854d6f..4f5f759 100644 --- profiles/default/default.profile +++ profiles/default/default.profile @@ -2,234 +2,6 @@ // $Id: default.profile,v 1.62 2009-08-03 03:04:34 webchick Exp $ /** - * Implement hook_profile_tasks(). - */ -function default_profile_tasks() { - $tasks = array( - 'default_profile_site_setup' => array(), - ); - return $tasks; -} - -/** - * Installation task; perform actions to set up the site for this profile. - * - * This task does not return any output, meaning that control will be passed - * along to the next task without ending the page request. - * - * @param $install_state - * An array of information about the current installation state. - */ -function default_profile_site_setup(&$install_state) { - - // Enable some standard blocks. - $values = array( - array( - 'module' => 'system', - 'delta' => 'main', - 'theme' => 'garland', - 'status' => 1, - 'weight' => 0, - 'region' => 'content', - 'pages' => '', - 'cache' => -1, - ), - array( - 'module' => 'user', - 'delta' => 'login', - 'theme' => 'garland', - 'status' => 1, - 'weight' => 0, - 'region' => 'sidebar_first', - 'pages' => '', - 'cache' => -1, - ), - array( - 'module' => 'system', - 'delta' => 'navigation', - 'theme' => 'garland', - 'status' => 1, - 'weight' => 0, - 'region' => 'sidebar_first', - 'pages' => '', - 'cache' => -1, - ), - array( - 'module' => 'system', - 'delta' => 'management', - 'theme' => 'garland', - 'status' => 1, - 'weight' => 1, - 'region' => 'sidebar_first', - 'pages' => '', - 'cache' => -1, - ), - array( - 'module' => 'system', - 'delta' => 'powered-by', - 'theme' => 'garland', - 'status' => 1, - 'weight' => 10, - 'region' => 'footer', - 'pages' => '', - 'cache' => -1, - ), - array( - 'module' => 'system', - 'delta' => 'help', - 'theme' => 'garland', - 'status' => 1, - 'weight' => 0, - 'region' => 'help', - 'pages' => '', - 'cache' => -1, - ), - array( - 'module' => 'system', - 'delta' => 'main', - 'theme' => 'seven', - 'status' => 1, - 'weight' => 0, - 'region' => 'content', - 'pages' => '', - 'cache' => -1, - ), - array( - 'module' => 'system', - 'delta' => 'help', - 'theme' => 'seven', - 'status' => 1, - 'weight' => 0, - 'region' => 'help', - 'pages' => '', - 'cache' => -1, - ), - array( - 'module' => 'user', - 'delta' => 'login', - 'theme' => 'seven', - 'status' => 1, - 'weight' => 10, - 'region' => 'content', - 'pages' => '', - 'cache' => -1, - ), - ); - $query = db_insert('block')->fields(array('module', 'delta', 'theme', 'status', 'weight', 'region', 'pages', 'cache')); - foreach ($values as $record) { - $query->values($record); - } - $query->execute(); - - // Insert default user-defined node types into the database. For a complete - // list of available node type attributes, refer to the node type API - // documentation at: http://api.drupal.org/api/HEAD/function/hook_node_info. - $types = array( - array( - 'type' => 'page', - 'name' => st('Page'), - 'base' => 'node_content', - 'description' => st("Use pages for your static content, such as an 'About us' page."), - 'custom' => 1, - 'modified' => 1, - 'locked' => 0, - ), - array( - 'type' => 'article', - 'name' => st('Article'), - 'base' => 'node_content', - 'description' => st('Use articles for time-specific content like news, press releases or blog posts.'), - 'custom' => 1, - 'modified' => 1, - 'locked' => 0, - ), - ); - - foreach ($types as $type) { - $type = node_type_set_defaults($type); - node_type_save($type); - } - - // Default page to not be promoted and have comments disabled. - variable_set('node_options_page', array('status')); - variable_set('comment_page', COMMENT_NODE_HIDDEN); - - // Don't display date and author information for page nodes by default. - variable_set('node_submitted_page', FALSE); - - // Create an image style. - $style = array('name' => 'thumbnail'); - $style = image_style_save($style); - $effect = array( - 'isid' => $style['isid'], - 'name' => 'image_scale_and_crop', - 'data' => array('width' => '85', 'height' => '85'), - ); - image_effect_save($effect); - - // Enable user picture support and set the default to a square thumbnail option. - variable_set('user_pictures', '1'); - variable_set('user_picture_dimensions', '1024x1024'); - variable_set('user_picture_file_size', '800'); - variable_set('user_picture_style', 'thumbnail'); - - $theme_settings = theme_get_settings(); - $theme_settings['toggle_node_user_picture'] = '1'; - $theme_settings['toggle_comment_user_picture'] = '1'; - variable_set('theme_settings', $theme_settings); - - // Create a default vocabulary named "Tags", enabled for the 'article' content type. - $description = st('Use tags to group articles on similar topics into categories.'); - $help = st('Enter a comma-separated list of words to describe your content.'); - - $vid = db_insert('taxonomy_vocabulary')->fields(array( - 'name' => 'Tags', - 'description' => $description, - 'machine_name' => 'tags', - 'help' => $help, - 'relations' => 0, - 'hierarchy' => 0, - 'multiple' => 0, - 'required' => 0, - 'tags' => 1, - 'module' => 'taxonomy', - 'weight' => 0, - ))->execute(); - db_insert('taxonomy_vocabulary_node_type')->fields(array('vid' => $vid, 'type' => 'article'))->execute(); - - // Create a default role for site administrators. - $rid = db_insert('role')->fields(array('name' => 'administrator'))->execute(); - - // Set this as the administrator role. - variable_set('user_admin_role', $rid); - - // Assign all available permissions to this role. - foreach (module_invoke_all('permission') as $key => $value) { - db_insert('role_permission') - ->fields(array( - 'rid' => $rid, - 'permission' => $key, - ))->execute(); - } - - // Update the menu router information. - menu_rebuild(); - - // Save some default links. - $link = array('link_path' => 'admin/structure/menu-customize/main-menu/add', 'link_title' => 'Add a main menu link', 'menu_name' => 'main-menu'); - menu_link_save($link); - - // Enable the admin theme. - db_update('system') - ->fields(array('status' => 1)) - ->condition('type', 'theme') - ->condition('name', 'seven') - ->execute(); - variable_set('admin_theme', 'seven'); - variable_set('node_admin_theme', '1'); -} - -/** * Implement hook_form_alter(). * * Allows the profile to alter the site-configuration form. This is diff --git profiles/expert/expert.info profiles/expert/expert.info index 70f0473..1b43914 100644 --- profiles/expert/expert.info +++ profiles/expert/expert.info @@ -5,3 +5,4 @@ version = VERSION core = 7.x dependencies[] = block dependencies[] = dblog +files[] = expert.profile diff --git profiles/expert/expert.install profiles/expert/expert.install new file mode 100644 index 0000000..013fa21 --- /dev/null +++ profiles/expert/expert.install @@ -0,0 +1,71 @@ + 'system', + 'delta' => 'main', + 'theme' => 'garland', + 'status' => 1, + 'weight' => 0, + 'region' => 'content', + 'pages' => '', + 'cache' => -1, + ), + array( + 'module' => 'user', + 'delta' => 'login', + 'theme' => 'garland', + 'status' => 1, + 'weight' => 0, + 'region' => 'sidebar_first', + 'pages' => '', + 'cache' => -1, + ), + array( + 'module' => 'system', + 'delta' => 'navigation', + 'theme' => 'garland', + 'status' => 1, + 'weight' => 0, + 'region' => 'sidebar_first', + 'pages' => '', + 'cache' => -1, + ), + array( + 'module' => 'system', + 'delta' => 'management', + 'theme' => 'garland', + 'status' => 1, + 'weight' => 1, + 'region' => 'sidebar_first', + 'pages' => '', + 'cache' => -1, + ), + array( + 'module' => 'system', + 'delta' => 'help', + 'theme' => 'garland', + 'status' => 1, + 'weight' => 0, + 'region' => 'help', + 'pages' => '', + 'cache' => -1, + ), + ); + $query = db_insert('block')->fields(array('module', 'delta', 'theme', 'status', 'weight', 'region', 'pages', 'cache')); + foreach ($values as $record) { + $query->values($record); + } + $query->execute(); +} + + diff --git profiles/expert/expert.profile profiles/expert/expert.profile index 3d44ef9..50e1b59 100644 --- profiles/expert/expert.profile +++ profiles/expert/expert.profile @@ -2,87 +2,6 @@ // $Id: expert.profile,v 1.12 2009-08-11 12:20:26 dries Exp $ /** - * Implement hook_profile_tasks(). - */ -function expert_profile_tasks() { - $tasks = array( - 'expert_profile_site_setup' => array(), - ); - return $tasks; -} - -/** - * Installation task; perform actions to set up the site for this profile. - * - * This task does not return any output, meaning that control will be passed - * along to the next task without ending the page request. - * - * @param $install_state - * An array of information about the current installation state. - */ -function expert_profile_site_setup(&$install_state) { - - // Enable some standard blocks. - $values = array( - array( - 'module' => 'system', - 'delta' => 'main', - 'theme' => 'garland', - 'status' => 1, - 'weight' => 0, - 'region' => 'content', - 'pages' => '', - 'cache' => -1, - ), - array( - 'module' => 'user', - 'delta' => 'login', - 'theme' => 'garland', - 'status' => 1, - 'weight' => 0, - 'region' => 'sidebar_first', - 'pages' => '', - 'cache' => -1, - ), - array( - 'module' => 'system', - 'delta' => 'navigation', - 'theme' => 'garland', - 'status' => 1, - 'weight' => 0, - 'region' => 'sidebar_first', - 'pages' => '', - 'cache' => -1, - ), - array( - 'module' => 'system', - 'delta' => 'management', - 'theme' => 'garland', - 'status' => 1, - 'weight' => 1, - 'region' => 'sidebar_first', - 'pages' => '', - 'cache' => -1, - ), - array( - 'module' => 'system', - 'delta' => 'help', - 'theme' => 'garland', - 'status' => 1, - 'weight' => 0, - 'region' => 'help', - 'pages' => '', - 'cache' => -1, - ), - ); - $query = db_insert('block')->fields(array('module', 'delta', 'theme', 'status', 'weight', 'region', 'pages', 'cache')); - foreach ($values as $record) { - $query->values($record); - } - $query->execute(); -} - -/** * Implement hook_form_alter(). * * Allows the profile to alter the site-configuration form. This is