Add more calls to watchdog()

This has been changed in CVS.

That is not true; the table could not exist for many other reasons, including the fact the table has been accidentally dropped. A fresh install must have the table installed, otherwise an error occurred during the installation of the module.

The message can be shorted to The xmlsitemap table has not been found; in this way, the message would not implicate anything wrong happened with the installation of the modules.

There are many other functions that could report a message in the Drupal log; I am changing the report title to reflect the extended task.

This is the patch that changes the previously added call to watchdog(); it also simplify the code by assigning a default to the static variable, which allows the IF-statement to be changed from

  if (empty($link_count)) {
    // ...
  }

to

  if (!$link_count) {
    // ...
  }

.

This is the complete patch for xmlsitemap.module.

xmlsitemap_092397.patch has been applied, and the changes committed in CVS. The final code is a little different from that because it has been slightly optimized.

xmlsitemap_delete_cache_files() is now defined as

function xmlsitemap_delete_cache_files() {
  file_scan_directory(
    variable_get('xmlsitemap_cache_directory', file_directory_path() .'/xmlsitemap'),
    'xsm-.*\.xml', array('.', '..', 'CVS'), 'unlink', FALSE
  );
  xmlsitemap_flag_sitemap();
  watchdog('xmlsitemap', 'The sitemap cache files have been deleted.');
}