diff options
Diffstat (limited to 'modules/node')
| -rw-r--r-- | modules/node/content_types.inc | 29 | ||||
| -rw-r--r-- | modules/node/node.admin.inc | 9 | ||||
| -rw-r--r-- | modules/node/node.install | 21 | ||||
| -rw-r--r-- | modules/node/node.module | 42 | ||||
| -rw-r--r-- | modules/node/node.pages.inc | 5 |
5 files changed, 84 insertions, 22 deletions
diff --git a/modules/node/content_types.inc b/modules/node/content_types.inc index c681ab9b9..c611f6753 100644 --- a/modules/node/content_types.inc +++ b/modules/node/content_types.inc @@ -136,6 +136,17 @@ function node_type_form(&$form_state, $type = NULL) { '#options' => drupal_map_assoc(array(0, 1, 10, 25, 50, 75, 100, 125, 150, 175, 200)), '#description' => t('The minimum number of words for the body field to be considered valid for this content type. This can be useful to rule out submissions that do not meet the site\'s standards, such as short test posts.') ); + $form['submission']['node_preview'] = array( + '#type' => 'radios', + '#title' => t('Preview post'), + '#default_value' => variable_get('node_preview_' . $type->type, DRUPAL_OPTIONAL), + '#options' => array( + DRUPAL_DISABLED => t('Disabled'), + DRUPAL_OPTIONAL => t('Optional'), + DRUPAL_REQUIRED => t('Required'), + ), + '#description' => t('Should users preview posts before submitting?'), + ); $form['submission']['help'] = array( '#type' => 'textarea', '#title' => t('Explanation or submission guidelines'), @@ -171,6 +182,13 @@ function node_type_form(&$form_state, $type = NULL) { '#default_value' => variable_get('node_submitted_' . $type->type, TRUE), '#description' => t('Enable the <em>submitted by Username on date</em> text.'), ); + $form['display']['teaser_length'] = array( + '#type' => 'select', + '#title' => t('Length of trimmed posts'), + '#default_value' => 600, + '#options' => drupal_map_assoc(array(0, 200, 400, 600, 800, 1000, 1200, 1400, 1600, 1800, 2000), '_node_characters'), + '#description' => t("The maximum number of characters used in the trimmed version of a post. Drupal will use this setting to determine at which offset long posts should be trimmed. The trimmed version of a post is typically used as a teaser when displaying the post on the main page, in XML feeds, etc. To disable teasers, set to 'Unlimited' . Note that this setting will only affect new or updated content and will not affect existing teasers.") + ); $form['old_type'] = array( '#type' => 'value', '#value' => $type->type, @@ -223,7 +241,14 @@ function node_type_form(&$form_state, $type = NULL) { } /** - * Implement hook_form_validate(). + * Helper function for teaser length choices. + */ +function _node_characters($length) { + return ($length == 0) ? t('Unlimited') : format_plural($length, '1 character', '@count characters'); +} + +/** + * Implementation of hook_form_validate(). */ function node_type_form_validate($form, &$form_state) { $type = new stdClass(); @@ -407,6 +432,8 @@ function node_type_delete_confirm(&$form_state, $type) { function node_type_delete_confirm_submit($form, &$form_state) { node_type_delete($form_state['values']['type']); + variable_del('teaser_length_' . $form_state['values']['type']); + variable_del('node_preview_' . $form_state['values']['type']); $t_args = array('%name' => $form_state['values']['name']); drupal_set_message(t('The content type %name has been deleted.', $t_args)); watchdog('menu', 'Deleted content type %name.', $t_args, WATCHDOG_NOTICE); diff --git a/modules/node/node.admin.inc b/modules/node/node.admin.inc index 3750e5891..dbddaea1e 100644 --- a/modules/node/node.admin.inc +++ b/modules/node/node.admin.inc @@ -48,13 +48,6 @@ function node_configure() { } /** - * Helper function for teaser length choices. - */ -function _node_characters($length) { - return ($length == 0) ? t('Unlimited') : format_plural($length, '1 character', '@count characters'); -} - -/** * Form button submit callback. */ function node_configure_access_submit($form, &$form_state) { @@ -74,7 +67,7 @@ function node_configure_rebuild_confirm() { */ function node_configure_rebuild_confirm_submit($form, &$form_state) { node_access_rebuild(TRUE); - $form_state['redirect'] = 'admin/content/node-settings'; + $form_state['redirect'] = 'admin/reports/status'; } /** diff --git a/modules/node/node.install b/modules/node/node.install index aa278adba..e2f8ec8d9 100644 --- a/modules/node/node.install +++ b/modules/node/node.install @@ -407,6 +407,27 @@ function node_update_7003() { } /** + * Extend the existing default preview and teaser settings to all node types. + */ +function node_update_7004() { + // Get original settings and all types. + $original_length = variable_get('teaser_length', 600); + $original_preview = variable_get('node_preview', 0); + + // Map old preview setting to new values order. + $original_preview ? $original_preview = 2 : $original_preview = 1; + $type_list = node_get_types('types'); + + // Apply original settings to all types. + foreach ($type_list as $type) { + variable_set('teaser_length_' . $type, $original_length); + variable_set('node_preview_' . $type, $original_preview); + } + // Delete old variable but leave 'teaser_length' for aggregator module upgrade. + variable_del('node_preview'); +} + +/** * @} End of "defgroup updates-6.x-to-7.x" * The next series of updates should start at 8000. */ diff --git a/modules/node/node.module b/modules/node/node.module index dee045a9e..af28dd192 100644 --- a/modules/node/node.module +++ b/modules/node/node.module @@ -1046,7 +1046,7 @@ function node_submit($node) { if (!isset($node->teaser)) { if (isset($node->body)) { $node->format = (!empty($node->body_format) ? $node->body_format : FILTER_FORMAT_DEFAULT); - $node->teaser = node_teaser($node->body, isset($node->format) ? $node->format : NULL); + $node->teaser = node_teaser($node->body, isset($node->format) ? $node->format : NULL, variable_get('teaser_length_' . $node->type, 600)); // Chop off the teaser from the body if needed. The teaser_include // property might not be set (eg. in Blog API postings), so only act on // it, if it was set with a given value. @@ -1870,15 +1870,9 @@ function node_menu() { 'weight' => -10, ); - $items['admin/content/node-settings'] = array( - 'title' => 'Post settings', - 'description' => 'Control posting behavior, such as teaser length, requiring previews before posting, and the number of posts on the front page.', - 'page callback' => 'drupal_get_form', - 'page arguments' => array('node_configure'), - 'access arguments' => array('administer nodes'), - ); - $items['admin/content/node-settings/rebuild'] = array( + $items['admin/reports/status/rebuild'] = array( 'title' => 'Rebuild permissions', + 'page callback' => 'drupal_get_form', 'page arguments' => array('node_configure_rebuild_confirm'), // Any user than can potentially trigger a node_access_needs_rebuild(TRUE) // has to be allowed access to the 'node access rebuild' confirm form. @@ -2191,7 +2185,7 @@ function node_page_default() { ->addTag('node_access'); $nids = $select->execute()->fetchCol(); - + if (!empty($nids)) { $nodes = node_load_multiple($nids); $build = node_build_multiple($nodes); @@ -2206,7 +2200,7 @@ function node_page_default() { } else { drupal_set_title(t('Welcome to @site-name', array('@site-name' => variable_get('site_name', 'Drupal')))); - + $default_message = '<p>' . t('No front page content has been created yet.') . '</p>'; if (user_access('access administration pages')) { @@ -3376,3 +3370,29 @@ function node_elements() { function theme_node_links($element) { return theme('links', $element['#value'], array('class' => 'links inline')); } + +/** + * Implement hook_requirements(). + */ +function node_requirements($phase) { + $requirements = array(); + // Ensure translations don't break at install time + $t = get_t(); + // Only show rebuild button if there are either 0, or 2 or more, rows + // in the {node_access} table, or if there are modules that + // implement hook_node_grants(). + $grant_count = db_result(db_query('SELECT COUNT(*) FROM {node_access}')); + if ($grant_count != 1 || count(module_implements('node_grants')) > 0) { + $value = format_plural($grant_count, 'One permission in use', '@count permissions in use', array('@count' => $grant_count)); + } else { + $value = $t('Disabled'); + } + $description = $t('If the site is experiencing problems with permissions to content, you may have to rebuild the permissions cache. Rebuilding will remove all privileges to posts, and replace them with permissions based on the current modules and settings. Rebuilding may take some time if there is a lot of content or complex permission settings. After rebuilding has completed posts will automatically use the new permissions.'); + + $requirements['node_access'] = array( + 'title' => $t('Node Access Permissions'), + 'value' => $value, + 'description' => $description . ' ' . l('Rebuild Permissions', 'admin/reports/status/rebuild'), + ); + return $requirements; +} diff --git a/modules/node/node.pages.inc b/modules/node/node.pages.inc index 2ded07985..729a991db 100644 --- a/modules/node/node.pages.inc +++ b/modules/node/node.pages.inc @@ -257,12 +257,13 @@ function node_form(&$form_state, $node) { $form['buttons']['#weight'] = 100; $form['buttons']['submit'] = array( '#type' => 'submit', - '#access' => !variable_get('node_preview', 0) || (!form_get_errors() && isset($form_state['node_preview'])), + '#access' => variable_get('node_preview_' . $node->type, 1) != DRUPAL_REQUIRED || (!form_get_errors() && isset($form_state['node_preview'])), '#value' => t('Save'), '#weight' => 5, '#submit' => array('node_form_submit'), ); $form['buttons']['preview'] = array( + '#access' => variable_get('node_preview_' . $node->type, 1) != DRUPAL_DISABLED, '#type' => 'submit', '#value' => t('Preview'), '#weight' => 10, @@ -395,7 +396,7 @@ function node_preview($node) { // Extract a teaser, if it hasn't been set (e.g. by a module-provided // 'teaser' form item). if (!isset($node->teaser)) { - $node->teaser = empty($node->body) ? '' : node_teaser($node->body, $node->format); + $node->teaser = empty($node->body) ? '' : node_teaser($node->body, $node->format, variable_get('teaser_length_' . $type, 600)); // Chop off the teaser from the body if needed. if (!$node->teaser_include && $node->teaser == substr($node->body, 0, strlen($node->teaser))) { $node->body = substr($node->body, strlen($node->teaser)); |