How to filter out "page not Found" from Popular content

Why don't you just use views to build a block to what you require?

Views is perfect for that sort of task :)

EDIT:

Also your best bet is to look at what page not found errors you are getting and build some redirects for the highest ones in the .htaccess file (in the root of your site), that way you can redirect the links to what the user expects and keep people coming from that source.

The following code if put in a block will give you the All time popular content *excluding* any 403 or 404 pages that have been defined. This is largely cribbed from http://api.drupal.org/api/function/statistics_block/6 and http://api.drupal.org/api/function/statistics_title_list/6.

I agree that something like this should be built-in, probably in core.

// First define customised version of statistics_title_list()
function statistics_title_list_custom($dbfield, $dbrows) {
  // This version excludes nodes used for 403 and 404 pages
  $path = $nid = array(); // Arrays to hold path and nid of error pages
  foreach (array(403, 404) as $err_code) {
    $path[$err_code] = drupal_get_normal_path(variable_get("site_$err_code", ''));
    if (substr($path[$err_code], 0, 5) == 'node/') {
      $nid[$err_code] = substr($path[$err_code], 5);
    }
    // If the nid is not valid (i.e. (possibly empty) string, NULL) then set to 0
    $nid[$err_code] = is_numeric($nid[$err_code]) ? $nid[$err_code] : 0;
}
  if (in_array($dbfield, array('totalcount', 'daycount', 'timestamp'))) {
    return db_query_range(db_rewrite_sql("SELECT n.nid, n.title, u.uid, u.name FROM {node} n INNER JOIN {node_counter} s ON n.nid = s.nid INNER JOIN {users} u ON n.uid = u.uid WHERE s.$dbfield != 0 AND n.status = 1 AND n.nid != %d AND n.nid != %d ORDER BY s.$dbfield DESC"), $nid[403], $nid[404], 0, $dbrows);
  }
  return FALSE;
}

// Now get the all time popular content
$alltimetop = variable_get('statistics_block_top_all_num', 0);
if ($alltimetop && ($result = statistics_title_list_custom('totalcount', $alltimetop)) && ($node_title_list = node_title_list($result, t('All time:')))) {
  $content = $node_title_list;
}

print $content;

I'm also interested in a 5.7 version.

(I did try it in 5.7 but it did not work)

This was written on 5.x, should also work on 6.x. :)

There was a bug at Drupal 5.7 and panel2. It was broke my panel down, maybe you want to take a look for a while. It's working nicely on regular block. Btw how to hidden restricted content type at popular content?

You can use the Statistics Advanced settings to not allow 403 and 404's to be included in node_counter. Considering that the module is in a semi-abandoned (looking for adoption), should this maybe get marked won't fix since there is a solution provided in another (very similar) module?

I suspect Views can also do most if not everything that Statistics Filtering can (and probably much more flexibly), at least in 6.x.

I'm not sure whether this use case should be handled by this module.