chamilo / chamilo-lms

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/process.php

Spacing +4 added lines, -4 removed lines

@@ -12,7 +12,7 @@ 
 $currentUserId = api_get_user_id();

 

 if (empty($currentUserId)) {

-    header('Location: ' . api_get_path(WEB_CODE_PATH) . 'auth/inscription.php');

+    header('Location: '.api_get_path(WEB_CODE_PATH).'auth/inscription.php');

     exit;

 }

 

@@ -27,7 +27,7 @@ 
 

 $buyingCourse = intval($_REQUEST['t']) === BuyCoursesPlugin::PRODUCT_TYPE_COURSE;

 $buyingSession = intval($_REQUEST['t']) === BuyCoursesPlugin::PRODUCT_TYPE_SESSION;

-$queryString = 'i=' . intval($_REQUEST['i']) . '&t=' . intval($_REQUEST['t']);

+$queryString = 'i='.intval($_REQUEST['i']).'&t='.intval($_REQUEST['t']);

 

 if ($buyingCourse) {

     $courseInfo = $plugin->getCourseInfo($_REQUEST['i']);

@@ -48,7 +48,7 @@ 
         Display::addFlash(

             Display::return_message($plugin->get_lang('NeedToSelectPaymentType'), 'error', false)

         );

-        header('Location:' . api_get_self() . '?' . $queryString);

+        header('Location:'.api_get_self().'?'.$queryString);

         exit;

     }

     

@@ -56,7 +56,7 @@ 
 

     if ($saleId !== false) {

         $_SESSION['bc_sale_id'] = $saleId;

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

+        header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/src/process_confirm.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;

 }

 

plugin/buycourses/src/process_confirm.php

Spacing +4 added lines, -4 removed lines

@@ -45,8 +45,8 @@ 
             $sale['price'],

             $currency['iso_code'],

             'paypal',

-            api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/success.php',

-            api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/error.php',

+            api_get_path(WEB_PLUGIN_PATH).'buycourses/src/success.php',

+            api_get_path(WEB_PLUGIN_PATH).'buycourses/src/error.php',

             $extra

         );

 

@@ -92,7 +92,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;

             }

 

@@ -129,7 +129,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/sales_report.php

Spacing +5 added lines, -5 removed lines

@@ -24,7 +24,7 @@ 
         api_not_allowed(true);

     }

 

-    $urlToRedirect = api_get_self() . '?';

+    $urlToRedirect = api_get_self().'?';

 

     switch ($_GET['action']) {

         case 'confirm':

@@ -93,10 +93,10 @@ 
     get_lang('Filter'),

     [$plugin->get_lang('ByStatus'), $plugin->get_lang('ByUser')]

 );

-$form->addHtml('<div id="report-by-status" ' . ($selectedFilterType !== '0' ? 'style="display:none"' : '') . '>');

+$form->addHtml('<div id="report-by-status" '.($selectedFilterType !== '0' ? 'style="display:none"' : '').'>');

 $form->addSelect('status', $plugin->get_lang('OrderStatus'), $saleStatuses);

 $form->addHtml('</div>');

-$form->addHtml('<div id="report-by-user" ' . ($selectedFilterType !== '1' ? 'style="display:none"' : '') . '>');

+$form->addHtml('<div id="report-by-user" '.($selectedFilterType !== '1' ? 'style="display:none"' : '').'>');

 $form->addText('user', get_lang('UserName'), false);

 $form->addHtml('</div>');

 $form->addButtonFilter(get_lang('Search'));

@@ -144,7 +144,7 @@ 
 

     $toolbar .= Display::toolbarButton(

         $plugin->get_lang('PaypalPayoutCommissions'),

-        api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/paypal_payout.php',

+        api_get_path(WEB_PLUGIN_PATH).'buycourses/src/paypal_payout.php',

         'paypal',

         'primary',

         ['title' => $plugin->get_lang('PaypalPayoutCommissions')]

@@ -158,7 +158,7 @@ 
 

     $toolbar .= Display::toolbarButton(

         $plugin->get_lang('PayoutReport'),

-        api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/payout_report.php',

+        api_get_path(WEB_PLUGIN_PATH).'buycourses/src/payout_report.php',

         'money',

         'info',

         ['title' => $plugin->get_lang('PayoutReport')]

plugin/buycourses/config.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
 <?php

 /* For licensing terms, see /license.txt */

 

-require_once __DIR__ . '/../../main/inc/global.inc.php';

+require_once __DIR__.'/../../main/inc/global.inc.php';

plugin/buycourses/install.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 /**

  * Initialization

  */

-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/ext_auth_chamilo_logout_button_behaviour/index.php

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
     api_get_setting('cas_activate') == 'true' &&
     $_user['auth_source'] == CAS_AUTH_SOURCE
 ) {
-    $_template['show_message']   = true;
+    $_template['show_message'] = true;
     // the link URL
     $link_url = "#";
     if (!empty($plugin_info['settings']['ext_auth_chamilo_logout_button_behaviour_eaclbb_form_link_url'])) {