diff -urp /home/jmoy/oldxmlsitemap/xmlsitemap/xmlsitemap_node/xmlsitemap_node.module ./xmlsitemap_node/xmlsitemap_node.module --- /home/jmoy/oldxmlsitemap/xmlsitemap/xmlsitemap_node/xmlsitemap_node.module 2008-05-28 20:15:39.000000000 +0530 +++ ./xmlsitemap_node/xmlsitemap_node.module 2008-10-16 13:32:15.000000000 +0530 @@ -163,34 +163,34 @@ function xmlsitemap_node_perm() { * Implementation of hook_form_alter(). */ function xmlsitemap_node_form_alter(&$form, &$form_state, $form_id) { + if (isset($form['type']) && $form_id==$form['type']['#value'] .'_node_form') { + $node = $form['#node']; + if (user_access('override node priority')) { + $form['xmlsitemap_node'] = array( + '#type' => 'fieldset', + '#title' => t('Site map settings'), + '#collapsible' => TRUE, + '#collapsed' => TRUE, + '#weight' => 30, + ); + $options = xmlsitemap_priority_options('both'); + $default = variable_get("xmlsitemap_node_type_priority_$node->type", '0.5'); + $form['xmlsitemap_node']['priority_override'] = array( + '#type' => 'select', + '#title' => t('Site map priority'), + '#default_value' => $node->priority_override, + '#options' => $options, + '#description' => t('The default priority is %priority.', array('%priority' => $options[$default])), + ); + } + else { + $form['priority_override'] = array('#type' => 'value', '#value' => $node->priority_override); + } + $form['xmlsitemap_node_status'] = array('#type' => 'value', '#value' => $node->status); + return; + } + switch ($form_id) { - case $form['type']['#value'] .'_node_form': - if (isset($form['type'])) { - $node = $form['#node']; - if (user_access('override node priority')) { - $form['xmlsitemap_node'] = array( - '#type' => 'fieldset', - '#title' => t('Site map settings'), - '#collapsible' => TRUE, - '#collapsed' => TRUE, - '#weight' => 30, - ); - $options = xmlsitemap_priority_options('both'); - $default = variable_get("xmlsitemap_node_type_priority_$node->type", '0.5'); - $form['xmlsitemap_node']['priority_override'] = array( - '#type' => 'select', - '#title' => t('Site map priority'), - '#default_value' => $node->priority_override, - '#options' => $options, - '#description' => t('The default priority is %priority.', array('%priority' => $options[$default])), - ); - } - else { - $form['priority_override'] = array('#type' => 'value', '#value' => $node->priority_override); - } - $form['xmlsitemap_node_status'] = array('#type' => 'value', '#value' => $node->status); - } - break; case 'node_type_form': if (isset($form['identity']['type'])) { $form['xmlsitemap_node'] = array( diff -urp /home/jmoy/oldxmlsitemap/xmlsitemap/xmlsitemap_term/xmlsitemap_term.module ./xmlsitemap_term/xmlsitemap_term.module --- /home/jmoy/oldxmlsitemap/xmlsitemap/xmlsitemap_term/xmlsitemap_term.module 2008-05-20 23:18:47.000000000 +0530 +++ ./xmlsitemap_term/xmlsitemap_term.module 2008-10-16 12:47:20.000000000 +0530 @@ -99,7 +99,7 @@ function _xmlsitemap_term_frontpage() { if (count($frontpage) == 3 && $frontpage[0] == 'taxonomy' && $frontpage[1] == 'term' && is_numeric($frontpage[2])) { $tid = $frontpage[2]; } - elseif (count($frontpage == 2) && $frontpage[0] == 'forum' && is_numeric($frontpage[1])) { + elseif (count($frontpage) == 2 && $frontpage[0] == 'forum' && is_numeric($frontpage[1])) { $tid = $frontpage[1]; } } diff -urp /home/jmoy/oldxmlsitemap/xmlsitemap/xmlsitemap_user/xmlsitemap_user.module ./xmlsitemap_user/xmlsitemap_user.module --- /home/jmoy/oldxmlsitemap/xmlsitemap/xmlsitemap_user/xmlsitemap_user.module 2008-05-20 23:18:47.000000000 +0530 +++ ./xmlsitemap_user/xmlsitemap_user.module 2008-10-16 12:48:02.000000000 +0530 @@ -65,7 +65,7 @@ function _xmlsitemap_user_frontpage() { if (!isset($uid)) { $uid = 0; $frontpage = explode('/', drupal_get_normal_path(variable_get('site_frontpage', 'node'))); - if (count($frontpage == 2) && $frontpage[0] == 'user' && is_numeric($frontpage[1])) { + if (count($frontpage) == 2 && $frontpage[0] == 'user' && is_numeric($frontpage[1])) { $uid = $frontpage[1]; } }