When "Use front page meta tags" is checked, if the front page is a node, the "meta tags" fieldset should display and update the default values defined in "Default and specific meta tags" > "Front page."

Here is my use case:

  • Checked "Use front page meta tags" so that I could display site verification codes.
  • On "Default and specific meta tags" > "Front page," I filled in Description, Keywords and the site verification codes.
  • Everything works as expected and the front page is outputting the appropriate meta tags.
  • The client comes along to edit the "description" and gets confused when she pulls up the edit form (node/#/edit), finds the "Description" field blank and then when she enters the new description, it won't update in the HTML.

So, in order to make it easier on the client, the "meta tags" fieldset should be pre-populated with the default values (including the site verification codes which currently don't show up there) and submitting the node updates the default values.

Another option is to simply hide the "meta tags" fieldset from the front page edit node form, with a link or note for the user that she needs to visit "Default and specific meta tags" > "Front page" in order to edit the meta tags on the front page.

CommentFileSizeAuthor
#2 nodewords-665042-2.txt1.78 KBjbjaaz

Comments

jbjaaz’s picture

jbjaaz commented
Title: If front page is a node, meta tags fieldset should update default values » If front page is a node, meta tags fieldset should point the user to the front page settings page
Version: 6.x-1.8 » 6.x-1.11

If anyone is curious how to achieve this without hacking the module, hook_form_alter works well.

function mymodule_form_alter(&$form, $form_state, $form_id) {
  if ($form['#id'] == 'node-form') {
    $nid = $form['nid']['#value'];
    $frontpage_tags = variable_get('nodewords_use_frontpage_tags', FALSE);
    $frontpage_path = variable_get('site_frontpage', '');
    if ($frontpage_tags && (drupal_get_path_alias("node/$nid") == drupal_get_path_alias($frontpage_path))) {
      $new = array(
        '#type' => 'fieldset',
        '#title' => $form['nodewords']['#title'],
        '#collapsible' => $form['nodewords']['#collapsible'],
        '#collapsed' => $form['nodewords']['#collapsed'],
        '#weight' => $form['nodewords']['#weight'], 
        'user_message' => array(
          '#prefix' => '<p>',
          '#value' => t('You are editing your front page. Update the meta tags (including search engine verification codes) by editing the !page.', array(
            '!page' => l('Meta Tags Front Page settings', 'admin/content/nodewords/meta-tags/frontpage'))
          ),
          '#suffix' => '</p>',
        ),
      );
      $form['nodewords'] = $new;
    }    
  }
}
jbjaaz’s picture

jbjaaz commented
StatusFileSize
new nodewords-665042-2.txt1.78 KB

So, I gave modifying the module a shot and came up with something. I'm taking the approach of hiding the Meta Tags form from the front page edit node form and replacing it with a message pointing the user to the Front Page settings page. See the attached patch.

I've tested it on Drupal 6.15.

Anonymous’s picture

Anonymous (not verified) commented
Title: If front page is a node, meta tags fieldset should point the user to the front page settings page » If the front page is a node, meta tags fieldset should not be shown for the node
Category: support » feature
Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Fixed

The code has been changed, and the meta tags edit fields don't appear for the node that is used as front page; the help message for the front page meta tags edit page reminds of this.

Anonymous’s picture

Anonymous (not verified) commented
Version: 6.x-1.11 » 6.x-1.12-beta4

I am changing the referring version to the latest version available, which does not implement the requested feature.

vikingew’s picture

vikingew commented

After updating to 6.x-1.x-dev(2010-Apr-09) which has a CVS commit referensing this issue, I get a warning:

warning: Cannot modify header information - headers already sent by (output started at /var/www/druproot/sites/all/modules/nodewords/nodewords_custom_pages/nodewords_custom_pages.module:1) in /var/www/druproot/includes/common.inc on line 148.

running update.php appearing on 'Overview' and 'Select update' stages and when finally clicking update button just a blank page.

And I found the solution! :-) it appears as a space character has sneeked in infront of the opening <?php tag of the nodewords_custom_pages.module file.

Status: Fixed » Closed (fixed)

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