hi tokyoj, that is correct. i wrote it that way so that it can easily be configured

redjumpsuit is all about making people's lives a little bit better

hi, the only real purpose of the cron job is to automate running scripts for you. the php way is to just actually run/access the php file. if your cron_maintenance.php is on your root folder, then you can just access it like http://<yourjobberbasesite>/cron_maintenance.php

would be good though to name your cron_maintenance.php file into something unique to you (ike cron_maintenance_xxxxxx.php) where x represents values (like a code or pin) that only you would know, so that not just anybody could run your cron_maintenance.php file

i hope this answers your question.

RedJumpsuit

Last edited by redjumpsuit (2009-01-24 13:06:36)

yup no errors.

hi, you can echo the function and it should show you an array of the jobs that have been deactivated.

// deactivate jobs older than 30 days + notification
echo $janitor->DeactivateJobs();

I tried to implement this With error reporting off on our live site, I assumed it was running. I checked late at night with error reporting on. I get an error in class.Maintenance.php that stated unexpected T_Variable in on line 36 ....the sql statement...select * from.....I took the $notifmin & $notifmax vars out and hard coded the values. Only to end up with the same error on another line

       $adname = trim($rows['title']);
   
Any ideas?

jobshouts.com A Jobberbased Derivative

doh I can't believe i did not notice that the first time. must have been because it was late. I also just spotted another syntax issue. There is a close brace at the end of this statement

$sqlu = "UPDATE jobs SET is_active = 0 WHERE DATEDIFF(NOW(), created_on) > 30 AND is_active = 1 AND id = " . $rows['id']) ;

should read

$sqlu = "UPDATE jobs SET is_active = 0 WHERE DATEDIFF(NOW(), created_on) > 30 AND is_active = 1 AND id = " . $rows['id'] ;

This error is now prevalent

Warning: Invalid argument supplied for foreach() in /home/content/33/3945733/html/_includes/class.Maintenance.php on line 39

line 39 is

   foreach ( $row as $rows )

Last edited by PopaWoody (2009-02-25 15:23:35)

jobshouts.com A Jobberbased Derivative

i have this on production and is working fine, please update your code to this:

public function DeactivateJobs()
      {      

      $postMan = new Postman();

      global $db;

      $notifmin = 27; //the day when you want notification to be sent prior to the ad expiring
      $notifmax = 30; //the day the ad actually expires
      $dayscount = $notifmax - $notifmin;

      $sql = 'SELECT * FROM jobs WHERE DATEDIFF(NOW(), created_on) >= '. $notifmin .' AND is_active = 1';
      $row = $db->QueryArray($sql);
      print_r($row); // this will print the array of the ad that was deactivated
      foreach ( $row as $rows )
          {
          $ademail = trim(strtolower($rows['poster_email']));
          $adname = trim($rows['title']);
          $adcomp = trim($rows['company']);
          $adurl = BASE_URL . "job/" . $rows['id'] . "/";
          
          $sqlu = 'UPDATE jobs SET is_active = 0 WHERE DATEDIFF(NOW(), created_on) > '. $notifmax .' AND is_active = 1 AND id = ' . $rows['id'] ;
          $db->Execute($sqlu);
          $postMan->MailExpiringAd($ademail, $adname, $adcomp, $adurl, $dayscount);
          }
  }

Please take note also of EMAIL_HEADER AND EMAIL_FOOTER on /_includes/class.Postman.php, you need constants like these to your /config.php, like this:

define('EMAIL_HEADER','Welcome to RedJumpsuit's Site!');
define('EMAIL_FOOTER','http://www.redjump.co.cc');