Brad Whitaker (whitaker) wrote in lj_dev,
Brad Whitaker
whitaker
lj_dev

bin/maint/clean_oldusers.pl

Confused.

I'm looking at this script. It checks to see if an account is over 2 months old and has never been posted in. Then it goes to check and see if they've ever posted comments. So then we have:


if ($commented) { next; }
print "$u->{'user'} ... $u->{'userid'}\n";


Am I missing what's going on here? There's no delete getting done? It seems like the "next;" should be skipping over a delete query, but there is none to be found. Has this been disabled or something? It wouldn't make sense to have it "disabled" but still running tons of useless queries on a cron job.


Also found that on line 14, the variable $posted seems useless?
Subscribe
  • Post a new comment

    Error

    Anonymous comments are disabled in this journal

    default userpic

    Your reply will be screened

    Your IP address will be recorded 

  • 9 comments