Project: Drupal Version: cvs Component: aggregator.module Category: bug reports Priority: normal Assigned to: drumm Reported by: drumm Updated by: Dries Status: patch I'm OK with this patch but wouldn't it be better if we did not have to execute those queries for deleted/non-existing blocks? What do other modules do to prevent this kind of behavior? (_block_rehash() appears to be a private function.) Dries Previous comments: ------------------------------------------------------------------------ February 11, 2005 - 02:32 : drumm Attachment: http://drupal.org/files/issues/aggregator.module_1.diff (2.72 KB) Make a feed with the block enabled. Delete the feed. The block remains and spews a database error on every page view. This fixes that. As for 4.5, the diff file at http://trac.civicspacelabs.com/cgi-bin/trac.cgi/changeset/273?format=diff might apply if the attached diff does not. The indentation is the only thing which might have changed in the section of code from 4.5 to CVS that I can see. -- View: http://drupal.org/node/17160 Edit: http://drupal.org/project/comments/add/17160