chamilo / chamilo-lms

plugin/kannelsms/vendor/kannelsms_api.php

Spacing +25 added lines, -25 removed lines

@@ -9,7 +9,7 @@ 
 * @version     1.3.0
 */
 
-if ( !class_exists('KannelException') ) {
+if (!class_exists('KannelException')) {
   require_once('exception.php');
 }
 
@@ -24,25 +24,25 @@ 
   /*
   * Version of this class
   */
-  const VERSION           = '1.3.1';
+  const VERSION = '1.3.1';
 
   /**
   * All Kannel API calls start with BASE_URL
   * @author  Martin Steel
   */
-  const API_BASE_URL      = 'api.kannelsms.com/xml/';
+  const API_BASE_URL = 'api.kannelsms.com/xml/';
 
   /**
   * string to append to API_BASE_URL to check authentication
   * @author  Martin Steel
   */
-  const API_AUTH_METHOD   = 'authenticate';
+  const API_AUTH_METHOD = 'authenticate';
 
   /**
   * string to append to API_BASE_URL for sending SMS
   * @author  Martin Steel
   */
-  const API_SMS_METHOD    = 'sms';
+  const API_SMS_METHOD = 'sms';
 
   /**
   * string to append to API_BASE_URL for checking message credit
@@ -258,13 +258,13 @@ 
     $err_no = null;
     $err_desc = null;
 
-    foreach($resp_doc->documentElement->childNodes AS $doc_child) {
-      switch(strtolower($doc_child->nodeName)) {
+    foreach ($resp_doc->documentElement->childNodes AS $doc_child) {
+      switch (strtolower($doc_child->nodeName)) {
         case 'sms_resp':
         $resp = array();
         $wrapper_id = null;
-        foreach($doc_child->childNodes AS $resp_node) {
-          switch(strtolower($resp_node->nodeName)) {
+        foreach ($doc_child->childNodes AS $resp_node) {
+          switch (strtolower($resp_node->nodeName)) {
             case 'messageid':
             $resp['id'] = $resp_node->nodeValue;
             break;
@@ -279,7 +279,7 @@ 
             break;
           }
         }
-        if( array_key_exists('error_code', $resp ) ) 
+        if (array_key_exists('error_code', $resp)) 
         {
           $resp['success'] = 0;
         } else {
@@ -415,7 +415,7 @@ 
       throw new KannelException($err_desc, $err_no);
     }
         
-    return array( 'symbol' => $symbol, 'balance' => $balance, 'code' => $code );
+    return array('symbol' => $symbol, 'balance' => $balance, 'code' => $code);
   }
 
   /**
@@ -480,14 +480,14 @@ 
       $this->logXML("API $method Request XML", $data);
     }
     
-    if( isset( $this->ssl ) ) {
+    if (isset($this->ssl)) {
       $ssl = $this->ssl;
     } else {
       $ssl = $this->sslSupport();
     }
 
     $url = $ssl ? 'https://' : 'http://';
-    $url .= self::API_BASE_URL . $method;
+    $url .= self::API_BASE_URL.$method;
 
     $response = $this->xmlPost($url, $data);
 
@@ -509,12 +509,12 @@ 
   * @author  Martin Steel
   */
   protected function xmlPost($url, $data) {
-    if(extension_loaded('curl')) {
+    if (extension_loaded('curl')) {
       $ch = curl_init($url);
       curl_setopt($ch, CURLOPT_POST, 1);
       curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
       curl_setopt($ch, CURLOPT_HTTPHEADER, Array("Content-Type: text/xml"));
-      curl_setopt($ch, CURLOPT_USERAGENT, 'Kannel PHP Wrapper/1.0' . self::VERSION);
+      curl_setopt($ch, CURLOPT_USERAGENT, 'Kannel PHP Wrapper/1.0'.self::VERSION);
       curl_setopt($ch, CURLOPT_POSTFIELDS, $data);
       if (isset($this->proxy_host) && isset($this->proxy_port)) {
         curl_setopt($ch, CURLOPT_PROXY, $this->proxy_host);
@@ -525,9 +525,9 @@ 
       $info = curl_getinfo($ch);
 
       if ($response === false || $info['http_code'] != 200) {
-        throw new \Exception('HTTP Error calling Kannel API - HTTP Status: ' . $info['http_code'] . ' - cURL Erorr: ' . curl_error($ch));
+        throw new \Exception('HTTP Error calling Kannel API - HTTP Status: '.$info['http_code'].' - cURL Erorr: '.curl_error($ch));
       } elseif (curl_errno($ch) > 0) {
-        throw new \Exception('HTTP Error calling Kannel API - cURL Error: ' . curl_error($ch));
+        throw new \Exception('HTTP Error calling Kannel API - cURL Error: '.curl_error($ch));
       }
 
       curl_close($ch);
@@ -536,31 +536,31 @@ 
     } elseif (function_exists('stream_get_contents')) {
       // Enable error Track Errors
       $track = ini_get('track_errors');
-      ini_set('track_errors',true);
+      ini_set('track_errors', true);
 
       $params = array('http' => array(
       'method'  => 'POST',
-      'header'  => "Content-Type: text/xml\r\nUser-Agent: mediaburst PHP Wrapper/" . self::VERSION . "\r\n",
+      'header'  => "Content-Type: text/xml\r\nUser-Agent: mediaburst PHP Wrapper/".self::VERSION."\r\n",
       'content' => $data
       ));
 
       if (isset($this->proxy_host) && isset($this->proxy_port)) {
-        $params['http']['proxy'] = 'tcp://'.$this->proxy_host . ':' . $this->proxy_port;
+        $params['http']['proxy'] = 'tcp://'.$this->proxy_host.':'.$this->proxy_port;
         $params['http']['request_fulluri'] = True;
       }
 
       $ctx = stream_context_create($params);
       $fp = @fopen($url, 'rb', false, $ctx);
       if (!$fp) {
-        ini_set('track_errors',$track);
+        ini_set('track_errors', $track);
         throw new \Exception("HTTP Error calling Kannel API - fopen Error: $php_errormsg");
       }
       $response = @stream_get_contents($fp);
       if ($response === false) {
-        ini_set('track_errors',$track);
+        ini_set('track_errors', $track);
         throw new \Exception("HTTP Error calling Kannel API - stream Error: $php_errormsg");
       }
-      ini_set('track_errors',$track);
+      ini_set('track_errors', $track);
       return $response;
     } else {
       throw new \Exception("Kannel requires PHP5 with cURL or HTTP stream support");
@@ -624,7 +624,7 @@ 
   * @author  Martin Steel
   */
   protected function is_assoc($array) {
-    return (bool)count(array_filter(array_keys($array), 'is_string'));
+    return (bool) count(array_filter(array_keys($array), 'is_string'));
   }
   
   /**
@@ -637,7 +637,7 @@ 
    * @todo Take an optional country code and check that the number starts with it
    */
   public static function is_valid_msisdn($val) {
-    return preg_match( '/^[1-9][0-9]{7,12}$/', $val );
+    return preg_match('/^[1-9][0-9]{7,12}$/', $val);
   }
 
 }

plugin/kannelsms/lib/kannelsms.lib.php

Spacing +3 added lines, -4 removed lines

@@ -75,14 +75,13 @@
         $trimmedKey = trim(CONFIG_SECURITY_API_KEY);
         if (!empty($trimmedKey)) {
             $message = array(
-                "to" => array_key_exists("mobilePhoneNumber",$additionalParameters) ?
-                    $additionalParameters['mobilePhoneNumber'] :
-                    $this->getMobilePhoneNumberById($additionalParameters['userId']),
+                "to" => array_key_exists("mobilePhoneNumber", $additionalParameters) ?
+                    $additionalParameters['mobilePhoneNumber'] : $this->getMobilePhoneNumberById($additionalParameters['userId']),
                 "message" => $this->getSms($additionalParameters)
             );
 
             if (!empty($message['message'])) {
-                if(extension_loaded('curl')) {
+                if (extension_loaded('curl')) {
                     $url = $this->hostAddress.'?username='.
                         $this->username.'&password='.$this->password.'&from='.
                         $this->from.'&to='.$message['to'].'&msg='.urlencode($message['message']);

plugin/rss/plugin.php

Spacing +1 added lines, -1 removed lines

@@ -9,6 +9,6 @@
  * @author Laurent Opprecht

  */

 

-require_once dirname(__FILE__) . '/lib/rss_plugin.class.php';

+require_once dirname(__FILE__).'/lib/rss_plugin.class.php';

 

 $plugin_info = RssPlugin::create()->get_info();

plugin/courselegal/user_list.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 /* For license terms, see /license.txt */
-require_once dirname(__FILE__) . '/config.php';
+require_once dirname(__FILE__).'/config.php';
 
 // Course legal
 $enabled = api_get_plugin_setting('courselegal', 'tool_enable');
@@ -19,7 +19,7 @@ 
 $url = api_get_self().'?'.api_get_cidreq();
 
 $action = isset($_GET['action']) ? $_GET['action'] : null;
-switch($action) {
+switch ($action) {
     case 'resend':
         if (isset($_GET['user_id'])) {
             $legal->updateMailAgreementLink($_GET['user_id'], $courseId, $sessionId);

plugin/courselegal/CourseLegalPlugin.php

Spacing +1 added lines, -1 removed lines

@@ -421,7 +421,7 @@
                 array('id = ? ' => $id)
             );
             if (!empty($legalData['filename'])) {
-                $fileToDelete = $coursePath . '/' . $legalData['filename'];
+                $fileToDelete = $coursePath.'/'.$legalData['filename'];
                 if (file_exists($fileToDelete)) {
                     unlink($fileToDelete);
                 }

plugin/courselegal/install.php

Spacing +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?php
 /* For license terms, see /license.txt */
 
-require_once dirname(__FILE__) . '/config.php';
+require_once dirname(__FILE__).'/config.php';
 
 if (!api_is_platform_admin()) {
     die ('You must have admin permissions to install plugins');

plugin/buycourses/src/expresscheckout.php

Spacing +4 added lines, -4 removed lines

@@ -40,8 +40,8 @@
     $ErrorSeverityCode = urldecode($resArray["L_SEVERITYCODE0"]);

 

     echo "SetExpressCheckout API call failed. ";

-    echo "Detailed Error Message: " . $ErrorLongMsg;

-    echo "Short Error Message: " . $ErrorShortMsg;

-    echo "Error Code: " . $ErrorCode;

-    echo "Error Severity Code: " . $ErrorSeverityCode;

+    echo "Detailed Error Message: ".$ErrorLongMsg;

+    echo "Short Error Message: ".$ErrorShortMsg;

+    echo "Error Code: ".$ErrorCode;

+    echo "Error Severity Code: ".$ErrorSeverityCode;

 }

plugin/buycourses/src/success.php

Spacing +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
 

         unset($_SESSION['bc_sale_id']);

 

-        header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'buycourses/index.php');

+        header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/index.php');

         exit;

     }

 

@@ -166,7 +166,7 @@ 
     }

 

     unset($_SESSION['bc_sale_id']);

-    header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/course_catalog.php');

+    header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/src/course_catalog.php');

     exit;

 }

 

plugin/buycourses/src/paypalfunctions.php

Spacing +44 added lines, -44 removed lines

@@ -67,12 +67,12 @@ 
 function CallShortcutExpressCheckout($paymentAmount, $currencyCodeType, $paymentType, $returnURL, $cancelURL, $extra)

 {

     // Construct the parameter string that describes the SetExpressCheckout API call in the shortcut implementation

-    $nvpstr = "&PAYMENTREQUEST_0_AMT=" . $paymentAmount;

-    $nvpstr .= "&PAYMENTREQUEST_0_ITEMAMT=" . $paymentAmount;

-    $nvpstr .= "&PAYMENTREQUEST_0_PAYMENTACTION=" . $paymentType;

-    $nvpstr .= "&RETURNURL=" . $returnURL;

-    $nvpstr .= "&CANCELURL=" . $cancelURL;

-    $nvpstr .= "&PAYMENTREQUEST_0_CURRENCYCODE=" . $currencyCodeType;

+    $nvpstr = "&PAYMENTREQUEST_0_AMT=".$paymentAmount;

+    $nvpstr .= "&PAYMENTREQUEST_0_ITEMAMT=".$paymentAmount;

+    $nvpstr .= "&PAYMENTREQUEST_0_PAYMENTACTION=".$paymentType;

+    $nvpstr .= "&RETURNURL=".$returnURL;

+    $nvpstr .= "&CANCELURL=".$cancelURL;

+    $nvpstr .= "&PAYMENTREQUEST_0_CURRENCYCODE=".$currencyCodeType;

     $nvpstr .= $extra;

 

     $_SESSION["currencyCodeType"] = $currencyCodeType;

@@ -116,20 +116,20 @@ 
 )

 {

     // Construct the parameter string that describes the SetExpressCheckout API call in the shortcut implementation

-    $nvpstr = "&PAYMENTREQUEST_0_AMT=" . $paymentAmount;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_PAYMENTACTION=" . $paymentType;

-    $nvpstr = $nvpstr . "&RETURNURL=" . $returnURL;

-    $nvpstr = $nvpstr . "&CANCELURL=" . $cancelURL;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_CURRENCYCODE=" . $currencyCodeType;

-    $nvpstr = $nvpstr . "&ADDROVERRIDE=1";

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTONAME=" . $shipToName;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTOSTREET=" . $shipToStreet;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTOSTREET2=" . $shipToStreet2;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTOCITY=" . $shipToCity;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTOSTATE=" . $shipToState;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTOCOUNTRYCODE=" . $shipToCountryCode;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTOZIP=" . $shipToZip;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTOPHONENUM=" . $phoneNum;

+    $nvpstr = "&PAYMENTREQUEST_0_AMT=".$paymentAmount;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_PAYMENTACTION=".$paymentType;

+    $nvpstr = $nvpstr."&RETURNURL=".$returnURL;

+    $nvpstr = $nvpstr."&CANCELURL=".$cancelURL;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_CURRENCYCODE=".$currencyCodeType;

+    $nvpstr = $nvpstr."&ADDROVERRIDE=1";

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTONAME=".$shipToName;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTOSTREET=".$shipToStreet;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTOSTREET2=".$shipToStreet2;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTOCITY=".$shipToCity;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTOSTATE=".$shipToState;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTOCOUNTRYCODE=".$shipToCountryCode;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTOZIP=".$shipToZip;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTOPHONENUM=".$phoneNum;

 

     $_SESSION["currencyCodeType"] = $currencyCodeType;

     $_SESSION["PaymentType"] = $paymentType;

@@ -172,7 +172,7 @@ 
      * Build a second API request to PayPal, using the token as the

      * ID to get the details on the payment authorization

      */

-    $nvpstr = "&TOKEN=" . $token;

+    $nvpstr = "&TOKEN=".$token;

 

     /**

      * Make the API call and store the results in an array.

@@ -213,9 +213,9 @@ 
     $payerID = urlencode($_SESSION['payer_id']);

     $serverName = urlencode($_SERVER['SERVER_NAME']);

 

-    $nvpstr = '&TOKEN=' . $token . '&PAYERID=' . $payerID . '&PAYMENTREQUEST_0_PAYMENTACTION=' . $paymentType . '&PAYMENTREQUEST_0_AMT=' . $FinalPaymentAmt;

-    $nvpstr .= '&PAYMENTREQUEST_0_CURRENCYCODE=' . $currencyCodeType . '&IPADDRESS=' . $serverName;

-    $nvpstr = '&' . http_build_query([

+    $nvpstr = '&TOKEN='.$token.'&PAYERID='.$payerID.'&PAYMENTREQUEST_0_PAYMENTACTION='.$paymentType.'&PAYMENTREQUEST_0_AMT='.$FinalPaymentAmt;

+    $nvpstr .= '&PAYMENTREQUEST_0_CURRENCYCODE='.$currencyCodeType.'&IPADDRESS='.$serverName;

+    $nvpstr = '&'.http_build_query([

         'TOKEN' => $token,

         'PAYERID' => $payerID,

         'PAYMENTACTION' => $paymentType,

@@ -269,20 +269,20 @@ 
                        $countryCode, $currencyCode)

 {

     //Construct the parameter string that describes DoDirectPayment

-    $nvpstr = "&AMT=" . $paymentAmount;

-    $nvpstr = $nvpstr . "&CURRENCYCODE=" . $currencyCode;

-    $nvpstr = $nvpstr . "&PAYMENTACTION=" . $paymentType;

-    $nvpstr = $nvpstr . "&CREDITCARDTYPE=" . $creditCardType;

-    $nvpstr = $nvpstr . "&ACCT=" . $creditCardNumber;

-    $nvpstr = $nvpstr . "&EXPDATE=" . $expDate;

-    $nvpstr = $nvpstr . "&CVV2=" . $cvv2;

-    $nvpstr = $nvpstr . "&FIRSTNAME=" . $firstName;

-    $nvpstr = $nvpstr . "&LASTNAME=" . $lastName;

-    $nvpstr = $nvpstr . "&STREET=" . $street;

-    $nvpstr = $nvpstr . "&CITY=" . $city;

-    $nvpstr = $nvpstr . "&STATE=" . $state;

-    $nvpstr = $nvpstr . "&COUNTRYCODE=" . $countryCode;

-    $nvpstr = $nvpstr . "&IPADDRESS=" . $_SERVER['REMOTE_ADDR'];

+    $nvpstr = "&AMT=".$paymentAmount;

+    $nvpstr = $nvpstr."&CURRENCYCODE=".$currencyCode;

+    $nvpstr = $nvpstr."&PAYMENTACTION=".$paymentType;

+    $nvpstr = $nvpstr."&CREDITCARDTYPE=".$creditCardType;

+    $nvpstr = $nvpstr."&ACCT=".$creditCardNumber;

+    $nvpstr = $nvpstr."&EXPDATE=".$expDate;

+    $nvpstr = $nvpstr."&CVV2=".$cvv2;

+    $nvpstr = $nvpstr."&FIRSTNAME=".$firstName;

+    $nvpstr = $nvpstr."&LASTNAME=".$lastName;

+    $nvpstr = $nvpstr."&STREET=".$street;

+    $nvpstr = $nvpstr."&CITY=".$city;

+    $nvpstr = $nvpstr."&STATE=".$state;

+    $nvpstr = $nvpstr."&COUNTRYCODE=".$countryCode;

+    $nvpstr = $nvpstr."&IPADDRESS=".$_SERVER['REMOTE_ADDR'];

 

     $resArray = hash_call("DoDirectPayment", $nvpstr);

 

@@ -345,13 +345,13 @@ 
     //if USE_PROXY constant set to TRUE in Constants.php, then only proxy will be enabled.

     //Set proxy name to PROXY_HOST and port number to PROXY_PORT in constants.php

     if ($USE_PROXY) {

-        curl_setopt($ch, CURLOPT_PROXY, $PROXY_HOST . ":" . $PROXY_PORT);

+        curl_setopt($ch, CURLOPT_PROXY, $PROXY_HOST.":".$PROXY_PORT);

     }

 

     //NVPRequest for submitting to server

-    $nvpreq = "METHOD=" . urlencode($methodName) . "&VERSION=" . urlencode($version) .

-        "&PWD=" . urlencode($API_Password) . "&USER=" . urlencode($API_UserName) .

-        "&SIGNATURE=" . urlencode($API_Signature) . $nvpStr . "&BUTTONSOURCE=" . urlencode($sBNCode);

+    $nvpreq = "METHOD=".urlencode($methodName)."&VERSION=".urlencode($version).

+        "&PWD=".urlencode($API_Password)."&USER=".urlencode($API_UserName).

+        "&SIGNATURE=".urlencode($API_Signature).$nvpStr."&BUTTONSOURCE=".urlencode($sBNCode);

 

     //setting the nvpreq as POST FIELD to curl

     curl_setopt($ch, CURLOPT_POSTFIELDS, $nvpreq);

@@ -385,8 +385,8 @@ 
 {

     global $PAYPAL_URL;

     // Redirect to paypal.com here

-    $payPalURL = $PAYPAL_URL . $token;

-    header("Location: " . $payPalURL);

+    $payPalURL = $PAYPAL_URL.$token;

+    header("Location: ".$payPalURL);

     exit;

 }