In dit regeltje zie ik nergens je reclameboodschap.
Voordat je Curl-commando aanroept, zou je dit moeten uitvoeren:

$sMessage .= "\nVerstuurd vanaf website.nl";

Waarbij "\n" een Enter is, zodat jouw reclame op een nieuwe regel begint.

Bovenstaande werkt niet.
En de reden dat je de reclame boodschao niet ziet is, omdat ik die er juist in wil hebben.

Weet iemand het?

Hoezo: 'Bovenstaande werkt niet'?

Wat werkt er niet? Verschijnt er een foutmelding? Ziet de SMS er raar uit?

Heb je het commando dat ik gaf echt precies voor de curl-aanroep gegeven? Wordt sMessage misschien overschreven?

Overigens bedoelde ik: in jouw voorbeeld PHP zie ik de reclame boodschap nergens gezet worden. En dan verschijnt ie dus niet in de SMS.

Origineel gepost door Ewout Quax

Hoezo: 'Bovenstaande werkt niet'?

Wat werkt er niet? Verschijnt er een foutmelding? Ziet de SMS er raar uit?

Heb je het commando dat ik gaf echt precies voor de curl-aanroep gegeven? Wordt sMessage misschien overschreven?

Overigens bedoelde ik: in jouw voorbeeld PHP zie ik de reclame boodschap nergens gezet worden. En dan verschijnt ie dus niet in de SMS.

Kreeg een parse error.

Hieronder het hele script:

PHP Code:

<?php
class SmsController extends Zend_Controller_Action 
{
 /**
  * Index page (send sms)
  * Accessed by /sms/
  */
 public function indexAction()
 {  
  /** Registry Instance */
  $registry = Zend_Registry::getInstance();
  
  /** Variable */
  $sError = '';
  
  /** Check of user is loggedin */
  if(Users::loggedin())
  {     
   /** Create object */
   $oUsers = new Users();
   $oSMS = new Sms();
   
   /** getPhoneNumbers */
   $sNumbers = $oSMS -> getPhoneNumbers($registry['userdata'] -> user[0]);
   $aNumbers = $sNumbers -> fetchAll(PDO::FETCH_ASSOC);
    
   /** Send data to the view */
   $this -> view -> numbers = $aNumbers;
   
   /** Check for post */
   if($this -> _request -> isPost())
   {  
    /** Try Catch */
    try
    {
     /** getUserInfo */
     $sUserInfo = $oUsers -> getUserInfo($registry['userdata'] -> user[0]);
     $aUserInfo = $sUserInfo -> fetch(PDO::FETCH_ASSOC);
 
     /** Check of the user has enough credits */
     if($aUserInfo['user_credits'] >= $registry['config'] -> credits_sendsms)
     {
      /** Message */
      $sMessage = urlencode($this -> _request -> getPost('message'));
      $sMessage .= ' ';
      $sMessage .= urlencode($registry['config'] -> sms_ad);
      
      /** Recipient */
      $sRecipient = $this -> _request -> getPost('recipient');
      
      /** Gateway settings */
      $sUsername = $registry['config'] -> sms_username;
      $sPassword = $registry['config'] -> sms_password;
      $sFrom = $registry['config'] -> sms_username;
     
      /** Check gateway */
      if($registry['config'] -> sms_gateway == 0)
      {
       $sGateway = 'smsdiscount';
      }
      else
      {
       $sGateway = 'voipbuster';
      }
    
      /** Curl */
      $this -> curl = curl_init();
      curl_setopt ($this -> curl, CURLOPT_URL, "https://www.".$sGateway.".com/myaccount/sendsms.php?username=".$sUsername."&password=".$sPassword."&from=".$sFrom."&to=".$sRecipient."&text=".$sMessage."");
      curl_setopt ($this -> curl, CURLOPT_RETURNTRANSFER, 1);
      curl_setopt ($this -> curl, CURLOPT_CONNECTTIMEOUT, 0);
      $sFile = curl_exec($this -> curl);
      curl_close($this -> curl);
     
      /** File Get Contents */
      //$sFile = file_get_contents("https://www.".$sGateway.".com/myaccount/sendsms.php?username=".$sUsername."&password=".$sPassword."&from=".$sFrom."&to=".$sRecipient."&text=".$sMessage."");
     
      if(eregi("success", $sFile))
      {
       /** Update saldo */
       $oUsers -> updateMinCredits($registry['userdata'] -> user[0], $registry['config'] -> credits_sendsms);
       
       /** Save sms in the outbox */
       $oSMS -> saveOutbox($registry['userdata'] -> user[0], $sMessage, $sRecipient);
       
       /** Sms send */
       $sError .= $registry['Zend_Translate'] -> _('SmsController_SmsSend');
      }
      else
      {
       /** Sms not send */
       $sError .= $registry['Zend_Translate'] -> _('SmsController_SmsNotSend');
      }
     }
     else
     {
      /** To many credits */
      $sError .= $registry['Zend_Translate'] -> _('SmsController_Credits');
     }
    } 
    catch(Exception $e)
    {
     /** 
      * Log the error in an file
      * Show custom error 
      */
     $registry['Zend_Log'] -> log($e -> getMessage(), Zend_Log::WARN);
     
     $sError .= $registry['Zend_Translate'] -> _('customError');
    }
   }
  }
  else
  {
   /** Not Loggedin */
   throw new Zend_Exception($registry['Zend_Translate'] -> _('UserController_NotLoggedIn'));
  }
  
  /** Send data to the view */
  $this -> view -> pageTitle = $registry['Zend_Translate'] -> _('SmsController_SmsPageTitle');
  $this -> view -> error = $sError;
 }
 
 /**
  * Outbox page
  * Accessed by /sms/outbox/
  */
 public function outboxAction()
 {
  /** Registry Instance */
  $registry = Zend_Registry::getInstance();
  
  /** Variable */
  $sError = '';
  
  /** Check of user is loggedin */
  if(Users::loggedin())
  {
   /** Create object */
   $oSMS = new Sms();
   
   /** Try Catch */
   try
   {   
    /** Get pageNumber */
    $sParam = $this -> _request -> getParams();
    $iPageNumber = $sParam['page']; 
   
    /** Create object */
    $oSMS = new Sms();
   
    $sData = $oSMS -> getOutbox($registry['userdata'] -> user[0]);
    
    /** Paginator */
    $aData = $sData -> fetchAll(PDO::FETCH_ASSOC);    
    $oPaginator = new Zend_Paginator(new Zend_Paginator_Adapter_Array($aData)); 
     
    $oPaginator -> setItemCountPerPage($registry['config'] -> pageCount); 
    $oPaginator -> setCurrentPageNumber($iPageNumber);
    
    /** Send data to the view */
    $this -> view -> paginator = $oPaginator;  
   } 
   catch(Exception $e)
   {
    /** 
     * Log the error in an file
     * Show custom error 
     */
    $registry['Zend_Log'] -> log($e -> getMessage(), Zend_Log::WARN);
    
    $sError .= $registry['Zend_Translate'] -> _('customError');
   }
  }
  else
  {
   /** Not Loggedin */
   throw new Zend_Exception($registry['Zend_Translate'] -> _('UserController_NotLoggedIn'));
  }
  
  /** Send data to the view */
  $this -> view -> error = $sError;
  $this -> view -> pageTitle = $registry['Zend_Translate'] -> _('SmsController_OutboxPageTitle');
 }
}

Kan je niet ergens een ad toevoegen? Aangezien dit er staat;

PHP Code:

$sMessage .= urlencode($registry['config'] -> sms_ad); 


Malik,

Ik heb de gehele code door de PHP-parser heen gehaald, maar ik krijg geen parse-errors.
Verder heb ik even naar de code gekeken, maar in het stukje waar de advertentie achter de sms-tekst wordt geplaatst kan ik geen rare dingen zien.

't Zou gewoon moeten werken.

Ik raad je aan om het script te debuggen, door gewoon heel veel 'echo'-commando's te gebruiken, om te achterhalen waarmee de belangrijkste variabelen zijn gevuld en op die manier erachter te komen waar het fout gaat.

Origineel gepost door Robin L

Kan je niet ergens een ad toevoegen? Aangezien dit er staat;

PHP Code:

$sMessage .= urlencode($registry['config'] -> sms_ad); 


Ja dit kan maar de add komt niet in de sms. dit snap ik niet.
En zie daarom ook niet waar de fout is.
De opgeslagen berichtten hebben allemaal de add. maar op je telefoon komt het aan zonder add

Dus er moet toch iets verkeerds gaan bij het versturen
Aanvullend bericht:
Heb het inmiddels zelf opgelost.

Toch bedankt voor jullie hulp!