chamilo / chamilo-lms

plugin/buycourses/database.php

Indentation +6 added lines, -6 removed lines

@@ -1,13 +1,13 @@
 <?php
 /* For license terms, see /license.txt */
 /**
- * Plugin database installation script. Can only be executed if included
- * inside another script loading global.inc.php
- * @package chamilo.plugin.buycourses
- */
+     * Plugin database installation script. Can only be executed if included
+     * inside another script loading global.inc.php
+     * @package chamilo.plugin.buycourses
+     */
 /**
- * Check if script can be called
- */
+     * Check if script can be called
+     */
 if (!function_exists('api_get_path')) {
     die('This script must be loaded through the Chamilo plugin installer sequence');
 }

plugin/buycourses/src/configuration.php

Indentation +5 added lines, -5 removed lines

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

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

 /**

- * Configuration script for the Buy Courses plugin

- * @package chamilo.plugin.buycourses

- */

+         * Configuration script for the Buy Courses plugin

+         * @package chamilo.plugin.buycourses

+         */

 /**

- * Initialization

- */

+         * Initialization

+         */

 $cidReset = true;

 

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

plugin/buycourses/src/session_catalog.php

Indentation +5 added lines, -5 removed lines

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

 /**

- * List of courses

- * @package chamilo.plugin.buycourses

- */

+     * List of courses

+     * @package chamilo.plugin.buycourses

+     */

 /**

- * Initialization

- */

+     * Initialization

+     */

 $cidReset = true;

 

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

plugin/buycourses/src/buy_course_plugin.class.php

Doc Comments +13 added lines, -13 removed lines

@@ -44,7 +44,7 @@ 
 

     /**

      *

-     * @return StaticPlugin

+     * @return BuyCoursesPlugin

      */

     static function create()

     {

@@ -216,7 +216,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)

     {

@@ -626,7 +626,7 @@ 
     /**

      * Get session info

      * @param array $sessionId The session ID

-     * @return array

+     * @return Chamilo\CoreBundle\Entity\Session

      */

     public function getSessionInfo($sessionId)

     {

@@ -719,7 +719,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)

     {

@@ -918,7 +918,7 @@ 
 

     /**

      * Get payment types

-     * @return array

+     * @return string[]

      */

     public function getPaymentTypes()

     {

@@ -956,7 +956,7 @@ 
 

     /**

      * Get the statuses for sales

-     * @return array

+     * @return string[]

      */

     public function getSaleStatuses()

     {

@@ -969,7 +969,7 @@ 
     

     /**

      * Get the statuses for sales

-     * @return array

+     * @return string[]

      */

     public function getServiceSaleStatuses()

     {

@@ -982,7 +982,7 @@ 
     

     /**

      * Get the statuses for Payouts

-     * @return array

+     * @return string[]

      */

     public function getPayoutStatuses()

     {

@@ -995,7 +995,7 @@ 
     

     /**

      * Get the list of service types

-     * @return array

+     * @return string[]

      */

     public function getServiceTypes()

     {

@@ -1008,7 +1008,7 @@ 
 

     /**

      * Get the list of product types

-     * @return array

+     * @return string[]

      */

     public function getProductTypes()

     {

@@ -1389,7 +1389,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)

     {

@@ -1674,7 +1674,7 @@ 
     /**

      * Register addicional service

      * @param array params $service

-     * @return database response

+     * @return false|string response

      */

     public function storeService($service)

     {

@@ -1872,7 +1872,7 @@ 
      * @param int $serviceId The service ID

      * @param int $paymentType The payment type

      * @param int $infoSelect The ID for Service Type

-     * @return boolean

+     * @return false|string

      */

     public function registerServiceSale($serviceId, $paymentType, $infoSelect)

     {

plugin/buycourses/src/paypalfunctions.php

Doc Comments +1 added lines

@@ -397,6 +397,7 @@
  * @nvpStr is nvp string.

  * returns an associtive array containing the response from the server.

  *

+ * @param string $methodName
  */

 function hash_call($methodName, $nvpStr)

 {

Indentation +4 added lines, -5 removed lines

@@ -111,8 +111,8 @@ 
  *		phoneNum:			the phoneNum  entered on the merchant's site

  */

 function CallMarkExpressCheckout($paymentAmount, $currencyCodeType, $paymentType, $returnURL,

-                                 $cancelURL, $shipToName, $shipToStreet, $shipToCity, $shipToState,

-                                 $shipToCountryCode, $shipToZip, $shipToStreet2, $phoneNum

+                                    $cancelURL, $shipToName, $shipToStreet, $shipToCity, $shipToState,

+                                    $shipToCountryCode, $shipToZip, $shipToStreet2, $phoneNum

 )

 {

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

@@ -265,8 +265,8 @@ 
  */

 

 function DirectPayment($paymentType, $paymentAmount, $creditCardType, $creditCardNumber,

-                       $expDate, $cvv2, $firstName, $lastName, $street, $city, $state, $zip,

-                       $countryCode, $currencyCode)

+                        $expDate, $cvv2, $firstName, $lastName, $street, $city, $state, $zip,

+                        $countryCode, $currencyCode)

 {

     //Construct the parameter string that describes DoDirectPayment

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

@@ -328,7 +328,6 @@ 
  * @param   CurrencyCode: Currency code (default is USD).

  * @param   CustomerPaypalAccount:   Email address of customer. 

  * @param   ExtraParams:    Extra Info params.

-      

  * @return: The NVP Collection object of the CreateRecurringPaymentsProfile Call Response.

  *

  * 

plugin/buycourses/src/services_edit.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 /* For licensing terms, see /license.txt */
 
 use ChamiloSession as Session;
-use Doctrine\Common\Collections\Criteria;
 
 /**
  *  Class DocumentManager

plugin/buycourses/src/service_success.php

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
 

         unset($_SESSION['bc_service_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;

     }

 

@@ -179,7 +179,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 +4 added lines, -4 removed lines

@@ -47,8 +47,8 @@ 
             $serviceSale['price'],

             $currency['iso_code'],

             'paypal',

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

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

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

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

             $extra,

             true

         );

@@ -97,7 +97,7 @@ 
 

                 unset($_SESSION['bc_service_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;

             }

 

@@ -136,7 +136,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.php

Spacing +9 added lines, -9 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;

 }

 

@@ -35,7 +35,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']);

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

 

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

 $userInfo = api_get_user_info();

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

     }

     

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

     }

     

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

@@ -82,7 +82,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())) {

@@ -90,7 +90,7 @@ 
             }

         }

     }

-    $form->addSelect('info_select',get_lang('User'), $selectOptions);

+    $form->addSelect('info_select', get_lang('User'), $selectOptions);

 } elseif ($typeCourse) {

     $user = $em->getRepository('ChamiloUserBundle:User')->find($currentUserId);

     $courses = $user->getCourses();

@@ -99,7 +99,7 @@ 
             $selectOptions[$course->getCourse()->getId()] = $course->getCourse()->getTitle();

         }

     }

-    $form->addSelect('info_select',get_lang('Course'), $selectOptions);

+    $form->addSelect('info_select', get_lang('Course'), $selectOptions);

 } elseif ($typeSession) {

     $user = $em->getRepository('ChamiloUserBundle:User')->find($currentUserId);

     $sessions = $user->getSessionCourseSubscriptions();

@@ -108,7 +108,7 @@ 
             $selectOptions[$session->getSession()->getId()] = $session->getSession()->getName();

         }

     }

-    $form->addSelect('info_select',get_lang('Session'), $selectOptions);

+    $form->addSelect('info_select', get_lang('Session'), $selectOptions);

 }

 

 $form->addHeader($plugin->get_lang('PaymentMethods'));