at 6.x-1.x-dev 2009-Mar-10
I found that only type "node" is not output to xml file.
So I looked in function xmlsitemap_node_xmlsitemap_links
at

      if (!($posts_threshold = variable_get('xmlsitemap_node_posts_count_'. $node->uid, variable_get('xmlsitemap_node_posts_count', 100)))) {
        continue;
      }
      $posts = (integer) db_result(db_query("SELECT COUNT(nid) FROM {node} WHERE uid = %d"));
      if ($posts_threshold > $posts) {
        continue;
      }

about variable 'xmlsitemap_node_posts_count' it was undefined (I checked in {variable}).
and the other thing is
(integer) db_result(db_query("SELECT COUNT(nid) FROM {node} WHERE uid = %d"));
i think it should be
(integer) db_result(db_query("SELECT COUNT(nid) FROM {node} WHERE uid = %d",$node->uid));

I think my english is not good.^^'

Comments

ikae’s picture

ikae commented

addition:
in function xmlsitemap_node_node_operations()
and other module,too. 

    'xmlsitemap_change_nodes_priority' => array(
      'label' => t('Change the XML site map priority of the selected posts to default'),
      'callback' => '_xmlsitemap_node_priority_operations',
      'callback arguments' => array('piority' => -2.0),
    ),

at array('piority' => -2.0), should be correct to "array('priority' => -2.0),"

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Title: Errors in the code » about missing variable xmlsitemap_node_posts_count

The Drupal xmlsitemap_node_posts_count variable is not undefined (it does not exist such a concept for Drupal variables), but it has not been set to a value different from its default value.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Title: about missing variable xmlsitemap_node_posts_count » Errors in the code
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Title: about missing variable xmlsitemap_node_posts_count » Errors in the code
Status: Active » Fixed

This has been fixed in CVS.

Thanks for your report.

Status: Fixed » Closed (fixed)

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