My cron doesn't work as expected (should post)

[Altari]

Hello,

(Sorry for my bad english)

I did a cron to post in each thread which didn't have any activity since 1 month, and send a mail to the author.
When i manually execute this cron, it work quite good : posts are posted, mails are sent etc.
But when the cron automatically execute, all the script is done without problems except the posts : No posts are posted, mails are sent, and no errors.

Another problem, is when i manually execute the cron posts are posted since my account, and not from the account id 1 like i wish.

Here is the portion of code which shoud post :

PHP Code:

$post = array();  
            $postinfo = array();  
            $errors = array();  

            $userid = 1; 
            $forumid = $donnees['forumid']; 

            $foruminfo = fetch_foruminfo($forumid);   
            $bbuserinfo = fetch_userinfo($userid);   
             
            $post['message'] =  
                '[B][COLOR=#008000]=================== 
                Ceci est un message automatique 
                =================== 

                Blablabla[/COLOR][/B]';   
                 
            $post['userid'] = 1; 
            $post['username'] = 'WEBSITE'; 
            $post['title'] = 'Plus de nouvelles depuis 1 mois'; 
             
             
            $post['poststarttime'] = TIMENOW;   
            $post['posthash'] = md5($post['poststarttime'] . $bbuserinfo['userid'] . $bbuserinfo['salt']);  

            $threadinfo = fetch_threadinfo($donnees['threadid']); 
            $postinfo = array(); 
            $postinfo['visible'] = 1; 
            $postinfo['specifiedpost'] = false; 
            $postinfo['postid'] = 0; 

             
            build_new_post('reply', $foruminfo, $threadinfo, $postinfo, $post, $errors); 


Here is all the cron :

PHP Code:

<?php
error_reporting(E_ALL);
ini_set('display_errors', 'On');

set_time_limit(0);
if (!is_object($vbulletin->db))
{
    exit;
}

require_once('./global.php');
require_once('includes/functions_newpost.php'); 
$deux_mois = date("Y-m-d", mktime(0,0,0,date("m"),date("d")-60,date("Y")));
function unix_timestamp($date)
{
    $date = str_replace(array(' ', ':'), '-', $date);
    $c    = explode('-', $date);
    $c    = array_pad($c, 6, 0);
    array_walk($c, 'intval');
 
    return mktime($c[3], $c[4], $c[5], $c[1], $c[2], $c[0]);
}
$deux_mois = unix_timestamp($deux_mois);

try
    {
    $pdo_options[PDO::ATTR_ERRMODE] = PDO::ERRMODE_EXCEPTION;
    $bdd = new PDO('mysql:host=localhost;dbname=XXX', 'root', 'XXXXXX', $pdo_options);
    $req = $bdd->query('SELECT threadid, postuserid, forumid, lastpost, lastposter FROM vb_thread WHERE lastpost < (UNIX_TIMESTAMP() - (30*24*3600)) AND visible = 1 AND open = 1 AND (forumid = 17 OR forumid = 30 OR forumid = 42)');
    while ($donnees = $req->fetch()){
        if($donnees['lastpost'] < $deux_mois OR $donnees['lastposter'] == 'WEBSITE'){
            $changement = $bdd->prepare("UPDATE vb_thread SET forumid = '179' WHERE threadid = :id");
            $changement->execute(array('id' => $donnees['threadid']));
            $changement->closeCursor();
            
            $membre = $bdd->prepare('SELECT userid, username, email FROM vb_user WHERE userid = :memberid');
            $membre->execute(array('memberid' => $donnees['postuserid']));
            $final_membre = $membre->fetch();
            $membre->closeCursor();
            
            if(!preg_match("#^[a-z0-9._-]+@(hotmail|live|msn).[a-z]{2,4}$#", $final_membre['email'])){
                $passage_ligne = "\r\n";
            }
            else{
                $passage_ligne = "\n";
            }
            
            $sujet = 'Votre annonce a ?t? archiv?e automatiquement';
            $contenu_mail_txt = 'Bonjour '.$final_membre['username'].','.$passage_ligne.'Nous vous informons que l\'annonce dont vous trouverez le lien ci-dessous a ?t? automatiquement archiv?e en l\absence de nouvelles depuis deux mois.'.$passage_ligne.'Si cette annonce est toujours d\'actualit?, nous vous invitons ? faire un rapport aupr?s des mod?rateurs ou ? r?diger une nouvelle annonce.'.$passage_ligne;
            $contenu_mail_txt .= $passage_ligne.'Lien de votre annonce : http://www.website.com/'.$donnees['threadid'].$passage_ligne;
            $contenu_mail_txt .= $passage_ligne.'Cordialement,'.$passage_ligne.'L\'?quipe Website';
            
            echo 'Mail envoy? ? '.$final_membre['username'].' avec succ?s ('.$final_membre['email'].')<br />';
            vbmail($final_membre['email'], $sujet, $contenu_mail_txt);
            
        }
        else{
            $post = array(); 
            $postinfo = array(); 
            $errors = array(); 

            $userid = 1;
            $forumid = $donnees['forumid'];

            $foruminfo = fetch_foruminfo($forumid);  
            $bbuserinfo = fetch_userinfo($userid);  
            
            $post['message'] = 
                '[B][COLOR=#008000]===================
                Ceci est un message automatique
                ===================

                Bonjour,

                Merci de nous donner des nouvelles concernant cette annonce,
                Est elle toujours d\'actualit? ? 
                Nous vous invitons ? faire un rapport de mod?ration si elle n\'est plus d\'actualit?, ou ? simplement r?pondre ? ce message le cas ?ch?ant.
                Cordialement,
                L\'?quipe Website[/COLOR][/B]';  
                
            $post['userid'] = 1;
            $post['username'] = 'Website';
            $post['title'] = 'Plus de nouvelles depuis 1 mois';
            
            
            $post['poststarttime'] = TIMENOW;  
            $post['posthash'] = md5($post['poststarttime'] . $bbuserinfo['userid'] . $bbuserinfo['salt']); 

            $threadinfo = fetch_threadinfo($donnees['threadid']);
            $postinfo = array();
            $postinfo['visible'] = 1;
            $postinfo['specifiedpost'] = false;
            $postinfo['postid'] = 0;

            
            build_new_post('reply', $foruminfo, $threadinfo, $postinfo, $post, $errors);
        
            $membre = $bdd->prepare('SELECT userid, username, email FROM vb_user WHERE userid = :memberid');
            $membre->execute(array('memberid' => $donnees['postuserid']));
            $final_membre = $membre->fetch();
            $membre->closeCursor();
            
            if(!preg_match("#^[a-z0-9._-]+@(hotmail|live|msn).[a-z]{2,4}$#", $final_membre['email'])){
                $passage_ligne = "\r\n";
            }
            else{
                $passage_ligne = "\n";
            }
            
            $sujet = 'Demande de nouvelles pour votre annonce';
            $contenu_mail_txt = 'Bonjour '.$final_membre['username'].','.$passage_ligne.'Voil? un mois que vous avez post? une annonce d\'adoption, sans qu\'il n\'y ait de nouvelles depuis. Afin de pr?server la qualit? des annonces sur Websiteet leur pertinence, merci de passer poster un message sur l\'annonce, ne serait ce que pour signaler qu\'elle est toujours d\'actualit?'.$passage_ligne.'Si cette annonce est obsol?te, nous vous invitons ? faire un rapport aux mod?rateurs ou ? le signaler dans le message afin qu\'elle soit archiv?e. Sans nouvelles de votre part ou d\'un autre membre, cette annonce sera automatiquement archiv?e dans 1 mois.';
            $contenu_mail_txt .= $passage_ligne.'Lien de votre annonce : http://www.rescue-forum.com/discussions/'.$donnees['threadid'].$passage_ligne;
            $contenu_mail_txt .= $passage_ligne.'Merci de votre compr?hension,'.$passage_ligne.'L\'?quipe RESCUE';
            
            echo 'Mail envoy? ? '.$final_membre['username'].' avec succ?s ('.$final_membre['email'].')<br />';
            vbmail($final_membre['email'], $sujet, $contenu_mail_txt);
            
        }
    
    }
    $req->closeCursor();
    }
    catch(Exception $e){
        die('Erreur : '.$e->getMessage());
    }

?>

Thank you a lot

[kh99]

The function build_new_post() uses some info from $vbulletin->userinfo, which will be whatever random user happens to run the cron job. Specifically, the from userid is taken from there even if you've set it already. So you either need to figure out a way around that, or else (re)write your own version.