delete items on a per feed basis

Thx.
I'll test it soon.

Perhaps it would be better to aka the table names in the query. Check the snippet below.

  // Delete too old items
  	   $result = db_query("SELECT f.items_delete, n.created, i.nid
  	            FROM {node} AS n, {leech_news_item} AS i, {leech_news_feed} AS f
  	            WHERE ( n.nid = i.nid AND
  	                    f.nid = i.fid)"
  	   );
  	   $now = time();
  	   while ($node = db_fetch_object($result))  {
  	     if (abs($now - $node->created) > $node->items_delete) {
  	       node_delete($node->nid);
  	     }
  	   }

I tested it, and it is working allright.
But, i think it would be better to check for old items for each feed at a time rather than checking the whole database at once. This way, after a feed is refreshed, only that feed is checked for old items..

Because I had a couple thousand old items, the script timmed out a couple of times before deleting them all.
One more thing, i can't seem to find a hook_delete function for feed items, i might have missed it though.

I really think it should be set up to delete items on a per feed basis.

If a site aggregates 100 items/day, 1 cron run will have to delete 100 items in 1 time.. which is very time consuming..
if it is set to items on a per feed basis, the 100 items will spread to multiple runs..