chamilo / chamilo-lms

plugin/buycourses/src/buy_course_plugin.class.php

Doc Comments +14 added lines, -14 removed lines

@@ -49,7 +49,7 @@ 
 

     /**

      *

-     * @return StaticPlugin

+     * @return BuyCoursesPlugin

      */

     static function create()

     {

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

     {

@@ -690,7 +690,7 @@ 
     /**

      * Get session info

      * @param array $sessionId The session ID

-     * @return array

+     * @return Session

      */

     public function getSessionInfo($sessionId)

     {

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

     {

@@ -965,7 +965,7 @@ 
 

     /**

      * Get payment types

-     * @return array

+     * @return string[]

      */

     public function getPaymentTypes()

     {

@@ -1004,7 +1004,7 @@ 
 

     /**

      * Get the statuses for sales

-     * @return array

+     * @return string[]

      */

     public function getSaleStatuses()

     {

@@ -1017,7 +1017,7 @@ 
 

     /**

      * Get the statuses for Payouts

-     * @return array

+     * @return string[]

      */

     public function getPayoutStatuses()

     {

@@ -1030,7 +1030,7 @@ 
 

     /**

      * Get the list of product types

-     * @return array

+     * @return string[]

      */

     public function getProductTypes()

     {

@@ -1042,7 +1042,7 @@ 
 

     /**

      * Get the list of service types

-     * @return array

+     * @return string[]

      */

     public function getServiceTypes()

     {

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

     {

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

     {

@@ -1704,7 +1704,7 @@ 
     /**

      * Register addicional service

      * @param array params $service

-     * @return mixed response

+     * @return string|false response

      */

     public function storeService($service)

     {

@@ -1861,7 +1861,7 @@ 
 

     /**

      * Get the statuses for sales

-     * @return array

+     * @return string[]

      */

     public function getServiceSaleStatuses()

     {

@@ -2118,7 +2118,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/Culqi/Client.php

Doc Comments +3 added lines

@@ -17,6 +17,9 @@
     const BASE_URL = "https://integ-pago.culqi.com/api/v1";
 
 
+    /**
+     * @param string $method
+     */
     public function request($method, $url, $api_key, $data = NULL, $headers= array("Content-Type" => "application/json", "Accept" => "application/json") ) {
         try {
             $options = array(

Spacing +9 added lines, -9 removed lines

@@ -17,22 +17,22 @@
     const BASE_URL = "https://integ-pago.culqi.com/api/v1";
 
 
-    public function request($method, $url, $api_key, $data = NULL, $headers= array("Content-Type" => "application/json", "Accept" => "application/json") ) {
+    public function request($method, $url, $api_key, $data = NULL, $headers = array("Content-Type" => "application/json", "Accept" => "application/json")) {
         try {
             $options = array(
                 'auth' => new AuthBearer($api_key),
                 'timeout' => 120
             );
-            if($method == "GET") {
-                $url_params = is_array($data) ? '?' . http_build_query($data) : '';
-                $response = \Requests::get(Culqi::$api_base . $url . $url_params, $headers, $options);
-            } else if($method == "POST") {
-                $response = \Requests::post(Culqi::$api_base . $url, $headers, json_encode($data), $options);
+            if ($method == "GET") {
+                $url_params = is_array($data) ? '?'.http_build_query($data) : '';
+                $response = \Requests::get(Culqi::$api_base.$url.$url_params, $headers, $options);
+            } else if ($method == "POST") {
+                $response = \Requests::post(Culqi::$api_base.$url, $headers, json_encode($data), $options);
 
 
-            } else if($method == "PATCH") {
-                $response = \Requests::patch(Culqi::$api_base . $url, $headers, json_encode($data), $options);
-            } else if($method == "DELETE") {
+            } else if ($method == "PATCH") {
+                $response = \Requests::patch(Culqi::$api_base.$url, $headers, json_encode($data), $options);
+            } else if ($method == "DELETE") {
                 $response = \Requests::delete(Culqi::$api_base, $options);
             }
         } catch (\Exception $e) {

plugin/buycourses/src/Culqi/Resource.php

Doc Comments +3 added lines

@@ -3,6 +3,9 @@
 
 class Resource extends Client {
 
+    /**
+     * @param Culqi $culqi
+     */
     public function __construct($culqi)
     {
         $this->culqi = $culqi;

plugin/buycourses/src/Requests.php

Doc Comments +4 added lines

@@ -226,6 +226,7 @@ 
 	 */
 	/**
 	 * Send a GET request
+	 * @param string $url
 	 */
 	public static function get($url, $headers = array(), $options = array()) {
 		return self::request($url, $headers, null, self::GET, $options);
@@ -240,6 +241,7 @@ 
 
 	/**
 	 * Send a DELETE request
+	 * @param string $url
 	 */
 	public static function delete($url, $headers = array(), $options = array()) {
 		return self::request($url, $headers, null, self::DELETE, $options);
@@ -263,6 +265,7 @@ 
 	 */
 	/**
 	 * Send a POST request
+	 * @param string $url
 	 */
 	public static function post($url, $headers = array(), $data = array(), $options = array()) {
 		return self::request($url, $headers, $data, self::POST, $options);
@@ -288,6 +291,7 @@ 
 	 * specification recommends that should send an ETag
 	 *
 	 * @link https://tools.ietf.org/html/rfc5789
+	 * @param string $url
 	 */
 	public static function patch($url, $headers, $data = array(), $options = array()) {
 		return self::request($url, $headers, $data, self::PATCH, $options);

Spacing +6 added lines, -6 removed lines

@@ -142,8 +142,8 @@ 
 		}
 
 		$file = str_replace('_', '/', $class);
-		if (file_exists(dirname(__FILE__) . '/' . $file . '.php')) {
-			require_once(dirname(__FILE__) . '/' . $file . '.php');
+		if (file_exists(dirname(__FILE__).'/'.$file.'.php')) {
+			require_once(dirname(__FILE__).'/'.$file.'.php');
 		}
 	}
 
@@ -503,7 +503,7 @@ 
 		$defaults = array(
 			'timeout' => 10,
 			'connect_timeout' => 10,
-			'useragent' => 'php-requests/' . self::VERSION,
+			'useragent' => 'php-requests/'.self::VERSION,
 			'protocol_version' => 1.1,
 			'redirected' => 0,
 			'redirects' => 10,
@@ -533,11 +533,11 @@ 
 	 * @return string Default certificate path.
 	 */
 	public static function get_certificate_path() {
-		if ( ! empty( Requests::$certificate_path ) ) {
+		if (!empty(Requests::$certificate_path)) {
 			return Requests::$certificate_path;
 		}
 
-		return dirname(__FILE__) . '/Requests/Transport/cacert.pem';
+		return dirname(__FILE__).'/Requests/Transport/cacert.pem';
 	}
 
 	/**
@@ -545,7 +545,7 @@ 
 	 *
 	 * @param string $path Certificate path, pointing to a PEM file.
 	 */
-	public static function set_certificate_path( $path ) {
+	public static function set_certificate_path($path) {
 		Requests::$certificate_path = $path;
 	}
 

plugin/buycourses/src/Requests/Cookie.php

Doc Comments +2 added lines, -1 removed lines

@@ -62,7 +62,7 @@ 
 	 *
 	 * @param string $name
 	 * @param string $value
-	 * @param array|Requests_Utility_CaseInsensitiveDictionary $attributes Associative array of attribute data
+	 * @param Requests_Utility_CaseInsensitiveDictionary $attributes Associative array of attribute data
 	 */
 	public function __construct($name, $value, $attributes = array(), $flags = array(), $reference_time = null) {
 		$this->name = $name;
@@ -377,6 +377,7 @@ 
 	 * specifies some of this handling, but not in a thorough manner.
 	 *
 	 * @param string Cookie header value (from a Set-Cookie header)
+	 * @param integer $reference_time
 	 * @return Requests_Cookie Parsed cookie object
 	 */
 	public static function parse($string, $name = '', $reference_time = null) {

Spacing +1 added lines, -1 removed lines

@@ -344,7 +344,7 @@
 				}
 			}
 
-			$header_value .= '; ' . implode('; ', $parts);
+			$header_value .= '; '.implode('; ', $parts);
 		}
 		return $header_value;
 	}

plugin/buycourses/src/Requests/IDNAEncoder.php

Doc Comments +1 added lines, -1 removed lines

@@ -357,7 +357,7 @@
 	 * @param int $delta
 	 * @param int $numpoints
 	 * @param bool $firsttime
-	 * @return int New bias
+	 * @return double New bias
 	 */
 	protected static function adapt($delta, $numpoints, $firsttime) {
 #	function adapt(delta,numpoints,firsttime):

Spacing +1 added lines, -1 removed lines

@@ -93,7 +93,7 @@
 		$string = self::punycode_encode($string);
 
 		// Step 7: Prepend ACE prefix
-		$string = self::ACE_PREFIX . $string;
+		$string = self::ACE_PREFIX.$string;
 
 		// Step 8: Check size
 		if (strlen($string) < 64) {

plugin/buycourses/src/Requests/IRI.php

Doc Comments +2 added lines, -1 removed lines

@@ -249,7 +249,7 @@ 
 	 *
 	 * Returns false if $base is not absolute, otherwise an IRI.
 	 *
-	 * @param IRI|string $base (Absolute) Base IRI
+	 * @param string $base (Absolute) Base IRI
 	 * @param IRI|string $relative Relative IRI
 	 * @return IRI|false
 	 */
@@ -984,6 +984,7 @@ 
 	 * Convert an IRI to a URI (or parts thereof)
 	 *
 	 * @param string|bool IRI to convert (or false from {@see get_iri})
+	 * @param false|string $string
 	 * @return string|false URI if IRI is valid, false otherwise.
 	 */
 	protected function to_uri($string) {

Spacing +21 added lines, -21 removed lines

@@ -153,8 +153,8 @@ 
 	 * @param mixed $value Property value
 	 */
 	public function __set($name, $value) {
-		if (method_exists($this, 'set_' . $name)) {
-			call_user_func(array($this, 'set_' . $name), $value);
+		if (method_exists($this, 'set_'.$name)) {
+			call_user_func(array($this, 'set_'.$name), $value);
 		}
 		elseif (
 			   $name === 'iauthority'
@@ -164,7 +164,7 @@ 
 			|| $name === 'iquery'
 			|| $name === 'ifragment'
 		) {
-			call_user_func(array($this, 'set_' . substr($name, 1)), $value);
+			call_user_func(array($this, 'set_'.substr($name, 1)), $value);
 		}
 	}
 
@@ -185,14 +185,14 @@ 
 			$name === 'iauthority' ||
 			$name === 'authority'
 		) {
-			$method = 'get_' . $name;
+			$method = 'get_'.$name;
 			$return = $this->$method();
 		}
 		elseif (array_key_exists($name, $props)) {
 			$return = $this->$name;
 		}
 		// host -> ihost
-		elseif (($prop = 'i' . $name) && array_key_exists($prop, $props)) {
+		elseif (($prop = 'i'.$name) && array_key_exists($prop, $props)) {
 			$name = $prop;
 			$return = $this->$prop;
 		}
@@ -202,7 +202,7 @@ 
 			$return = $this->$prop;
 		}
 		else {
-			trigger_error('Undefined property: ' . get_class($this) . '::' . $name, E_USER_NOTICE);
+			trigger_error('Undefined property: '.get_class($this).'::'.$name, E_USER_NOTICE);
 			$return = null;
 		}
 
@@ -221,7 +221,7 @@ 
 	 * @return bool
 	 */
 	public function __isset($name) {
-		return (method_exists($this, 'get_' . $name) || isset($this->$name));
+		return (method_exists($this, 'get_'.$name) || isset($this->$name));
 	}
 
 	/**
@@ -230,8 +230,8 @@ 
 	 * @param string $name Property name
 	 */
 	public function __unset($name) {
-		if (method_exists($this, 'set_' . $name)) {
-			call_user_func(array($this, 'set_' . $name), '');
+		if (method_exists($this, 'set_'.$name)) {
+			call_user_func(array($this, 'set_'.$name), '');
 		}
 	}
 
@@ -287,10 +287,10 @@ 
 						$target->ipath = $relative->ipath;
 					}
 					elseif (($base->iuserinfo !== null || $base->ihost !== null || $base->port !== null) && $base->ipath === '') {
-						$target->ipath = '/' . $relative->ipath;
+						$target->ipath = '/'.$relative->ipath;
 					}
 					elseif (($last_segment = strrpos($base->ipath, '/')) !== false) {
-						$target->ipath = substr($base->ipath, 0, $last_segment + 1) . $relative->ipath;
+						$target->ipath = substr($base->ipath, 0, $last_segment + 1).$relative->ipath;
 					}
 					else {
 						$target->ipath = $relative->ipath;
@@ -405,7 +405,7 @@ 
 				$input = '';
 			}
 		}
-		return $output . $input;
+		return $output.$input;
 	}
 
 	/**
@@ -632,7 +632,7 @@ 
 					|| $character > 0xD7FF && $character < 0xF900
 				) {
 					for ($j = $start; $j <= $i; $j++) {
-						$string .= '%' . strtoupper($bytes[$j]);
+						$string .= '%'.strtoupper($bytes[$j]);
 					}
 				}
 				else {
@@ -647,7 +647,7 @@ 
 		// mid-way through a multi-byte sequence)
 		if ($remaining) {
 			for ($j = $start; $j < $len; $j++) {
-				$string .= '%' . strtoupper($bytes[$j]);
+				$string .= '%'.strtoupper($bytes[$j]);
 			}
 		}
 
@@ -862,7 +862,7 @@ 
 		}
 		if (substr($ihost, 0, 1) === '[' && substr($ihost, -1) === ']') {
 			if (Requests_IPv6::check_ipv6(substr($ihost, 1, -1))) {
-				$this->ihost = '[' . Requests_IPv6::compress(substr($ihost, 1, -1)) . ']';
+				$this->ihost = '['.Requests_IPv6::compress(substr($ihost, 1, -1)).']';
 			}
 			else {
 				$this->ihost = null;
@@ -1019,17 +1019,17 @@ 
 
 		$iri = '';
 		if ($this->scheme !== null) {
-			$iri .= $this->scheme . ':';
+			$iri .= $this->scheme.':';
 		}
 		if (($iauthority = $this->get_iauthority()) !== null) {
-			$iri .= '//' . $iauthority;
+			$iri .= '//'.$iauthority;
 		}
 		$iri .= $this->ipath;
 		if ($this->iquery !== null) {
-			$iri .= '?' . $this->iquery;
+			$iri .= '?'.$this->iquery;
 		}
 		if ($this->ifragment !== null) {
-			$iri .= '#' . $this->ifragment;
+			$iri .= '#'.$this->ifragment;
 		}
 
 		return $iri;
@@ -1056,13 +1056,13 @@ 
 
 		$iauthority = '';
 		if ($this->iuserinfo !== null) {
-			$iauthority .= $this->iuserinfo . '@';
+			$iauthority .= $this->iuserinfo.'@';
 		}
 		if ($this->ihost !== null) {
 			$iauthority .= $this->ihost;
 		}
 		if ($this->port !== null) {
-			$iauthority .= ':' . $this->port;
+			$iauthority .= ':'.$this->port;
 		}
 		return $iauthority;
 	}

plugin/buycourses/src/Requests/Session.php

Doc Comments +1 added lines, -1 removed lines

@@ -232,7 +232,7 @@
 	 *
 	 * @param array $request Request data (same form as {@see request_multiple})
 	 * @param boolean $merge_options Should we merge options as well?
-	 * @return array Request data
+	 * @return string Request data
 	 */
 	protected function merge_request($request, $merge_options = true) {
 		if ($this->url !== null) {

plugin/buycourses/src/Requests/Proxy/HTTP.php

Spacing +2 added lines, -2 removed lines

@@ -82,7 +82,7 @@ 
 	 * @see fsockopen_header
 	 * @param Requests_Hooks $hooks Hook system
 	 */
-	public function register(Requests_Hooks &$hooks) {
+	public function register(Requests_Hooks & $hooks) {
 		$hooks->register('curl.before_send', array(&$this, 'curl_before_send'));
 
 		$hooks->register('fsockopen.remote_socket', array(&$this, 'fsockopen_remote_socket'));
@@ -146,6 +146,6 @@ 
 	 * @return string
 	 */
 	public function get_auth_string() {
-		return $this->user . ':' . $this->pass;
+		return $this->user.':'.$this->pass;
 	}
 }
\ No newline at end of file