chamilo / chamilo-lms

plugin/buycourses/src/buy_course_plugin.class.php

Doc Comments +13 added lines, -13 removed lines

@@ -45,7 +45,7 @@ 
 

     /**

      *

-     * @return StaticPlugin

+     * @return BuyCoursesPlugin

      */

     static function create()

     {

@@ -268,7 +268,7 @@ 
     /**

      * Save a transfer account information

      * @param array $params The transfer account

-     * @return int Rows affected. Otherwise return false

+     * @return false|string Rows affected. Otherwise return false

      */

     public function saveTransferAccount($params)

     {

@@ -680,7 +680,7 @@ 
     /**

      * Get session info

      * @param array $sessionId The session ID

-     * @return array

+     * @return Session

      */

     public function getSessionInfo($sessionId)

     {

@@ -773,7 +773,7 @@ 
      * Register a sale

      * @param int $itemId The product ID

      * @param int $paymentType The payment type

-     * @return boolean

+     * @return false|string

      */

     public function registerSale($itemId, $paymentType)

     {

@@ -955,7 +955,7 @@ 
 

     /**

      * Get payment types

-     * @return array

+     * @return string[]

      */

     public function getPaymentTypes()

     {

@@ -994,7 +994,7 @@ 
 

     /**

      * Get the statuses for sales

-     * @return array

+     * @return string[]

      */

     public function getSaleStatuses()

     {

@@ -1007,7 +1007,7 @@ 
 

     /**

      * Get the statuses for Payouts

-     * @return array

+     * @return string[]

      */

     public function getPayoutStatuses()

     {

@@ -1020,7 +1020,7 @@ 
 

     /**

      * Get the list of product types

-     * @return array

+     * @return string[]

      */

     public function getProductTypes()

     {

@@ -1401,7 +1401,7 @@ 
     /**

      * Register a item

      * @param array $itemData The item data

-     * @return int The item ID. Otherwise return false

+     * @return false|string The item ID. Otherwise return false

      */

     public function registerItem(array $itemData)

     {

@@ -1559,7 +1559,7 @@ 
     /**

      * Verify if the beneficiary have a paypal account

      * @param int $userId

-     * @return true if the user have a paypal account, false if not

+     * @return boolean if the user have a paypal account, false if not

      */

     public function verifyPaypalAccountByBeneficiary($userId)

     {

@@ -1681,7 +1681,7 @@ 
     /**

      * Register addicional service

      * @param array params $service

-     * @return mixed response

+     * @return string|false response

      */

     public function storeService($service)

     {

@@ -1838,7 +1838,7 @@ 
 

     /**

      * Get the statuses for sales

-     * @return array

+     * @return string[]

      */

     public function getServiceSaleStatuses()

     {

@@ -2094,7 +2094,7 @@ 
      * @param int $paymentType The payment type

      * @param int $infoSelect The ID for Service Type

      * @param int $trial trial mode

-     * @return boolean

+     * @return false|string

      */

     public function registerServiceSale($serviceId, $paymentType, $infoSelect, $trial = null)

     {

plugin/buycourses/src/service_process.php

Spacing +6 added lines, -6 removed lines

@@ -13,7 +13,7 @@ 
 require_once '../config.php';

 

 if (!isset($_REQUEST['t'], $_REQUEST['i'])) {

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

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

 }

 

 $currentUserId = api_get_user_id();

@@ -37,7 +37,7 @@ 
 $typeUser = intval($_REQUEST['t']) === BuyCoursesPlugin::SERVICE_TYPE_USER;

 $typeCourse = intval($_REQUEST['t']) === BuyCoursesPlugin::SERVICE_TYPE_COURSE;

 $typeSession = intval($_REQUEST['t']) === BuyCoursesPlugin::SERVICE_TYPE_SESSION;

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

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

 

 $serviceInfo = $plugin->getServices(intval($_REQUEST['i']));

 $userInfo = api_get_user_info($currentUserId);

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

     }

 

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

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

         );

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

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

         exit;

     }

     

@@ -68,7 +68,7 @@ 
     if ($serviceSaleId !== false) {

         $_SESSION['bc_service_sale_id'] = $serviceSaleId;

 

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

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

     }

 

     exit;

@@ -85,7 +85,7 @@ 
 

 if ($typeUser) {

     $users = $em->getRepository('ChamiloUserBundle:User')->findAll();

-    $selectOptions[$userInfo['user_id']] = api_get_person_name($userInfo['firstname'], $userInfo['lastname']) . ' (' . get_lang('Myself') . ')';

+    $selectOptions[$userInfo['user_id']] = api_get_person_name($userInfo['firstname'], $userInfo['lastname']).' ('.get_lang('Myself').')';

     if (!empty($users)) {

         foreach ($users as $user) {

             if (intval($userInfo['user_id']) !== intval($user->getId())) {

plugin/buycourses/src/service_success.php

Spacing +3 added lines, -3 removed lines

@@ -51,7 +51,7 @@ 
             Display::return_message($plugin->get_lang('OrderCancelled'), 'error', false)

         );

 

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

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

         exit;

     }

 

@@ -66,7 +66,7 @@ 
             Display::return_message($erroMessage, 'error', false)

         );

         unset($_SESSION['wizard']);

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

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

         exit;

     }

 

@@ -150,7 +150,7 @@ 
 

     unset($_SESSION['bc_service_sale_id']);

 

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

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

 

     exit;

 }

plugin/buycourses/src/service_process_confirm.php

Spacing +5 added lines, -5 removed lines

@@ -39,8 +39,8 @@ 
         // purchase-specific fields are ignored. This little condition handle this fact.

         $itemPrice = $serviceSale['price'];

 

-        $returnUrl = api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/service_success.php';

-        $cancelUrl = api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/service_error.php';

+        $returnUrl = api_get_path(WEB_PLUGIN_PATH).'buycourses/src/service_success.php';

+        $cancelUrl = api_get_path(WEB_PLUGIN_PATH).'buycourses/src/service_error.php';

 

         // The extra params for handle the hard job, this var is VERY IMPORTANT !!

         $extra = '';

@@ -72,7 +72,7 @@ 
 

             $plugin->cancelServiceSale(intval($serviceSale['id']));

 

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

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

             exit;

         }

 

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

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

                 );

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

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

                 exit;

             }

 

@@ -149,7 +149,7 @@ 
             );

 

             unset($_SESSION['bc_service_sale_id']);

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

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

             exit;

         }