chamilo / chamilo-lms

plugin/pens/lib/pens/pens_response.php

Spacing +14 added lines, -14 removed lines

@@ -26,9 +26,9 @@ 
  * @licence http://www.gnu.org/licenses/gpl.txt
  */
 
-require_once __DIR__ . '/pens_config.php';
-require_once __DIR__ . '/pens_message.php';
-require_once __DIR__ . '/pens_exception.php';
+require_once __DIR__.'/pens_config.php';
+require_once __DIR__.'/pens_message.php';
+require_once __DIR__.'/pens_exception.php';
 
 /**
  * PENSResponse
@@ -73,13 +73,13 @@ 
 	  */
 	public function __construct($error, $error_text = null, $pens_data = null) {
 		$this->_pens_version = PENSConfig::$version;
-		if($error instanceof PENSException) {
+		if ($error instanceof PENSException) {
 			$this->_error = $error->getCode();
 			$this->_error_text = $error->getMessage();
-		} else if(is_string($error)){
+		} else if (is_string($error)) {
 			// Parse the string
 			$this->parseResponse($error);
-		} else if(is_array($error)) {
+		} else if (is_array($error)) {
 			// Try to build from array
 			$this->_error = $error["error"];
 			$this->_error_text = $error["error-text"];
@@ -100,22 +100,22 @@ 
 	protected function parseResponse($response) {
 		$lines = explode(PENSConfig::$eol, $response);
 		$i = 1;
-		foreach($lines as $line) {
-			if($i < 5) {
+		foreach ($lines as $line) {
+			if ($i < 5) {
 				$pair = explode("=", $line);
-				if($pair[0] == "error") {
+				if ($pair[0] == "error") {
 					$this->_error = intval($pair[1]);
-				} else if($pair[0] == "error-text") {
+				} else if ($pair[0] == "error-text") {
 					$this->_error_text = $pair[1];
-				} else if($pair[0] == "version") {
+				} else if ($pair[0] == "version") {
 					$this->_pens_version = $pair[1];
-				} else if($pair[0] == "pens-data") {
-					if(!empty($pair[1])) {
+				} else if ($pair[0] == "pens-data") {
+					if (!empty($pair[1])) {
 						$this->_pens_data = $pair[1].PENSConfig::$eol;
 					}
 				}
 			} else {
-				if(!empty($line)) {
+				if (!empty($line)) {
 					$this->_pens_data .= $line.PENSConfig::$eol;
 				}
 			}

plugin/pens/lib/pens.php

Spacing +5 added lines, -5 removed lines

@@ -1,7 +1,7 @@
 <?php
 
-require_once __DIR__ . '/pens/pens_request_factory.php';
-require_once __DIR__ . '/pens/pens_response.php';
-require_once __DIR__ . '/pens/pens_package_handler.php';
-require_once __DIR__ . '/pens/pens_server.php';
-require_once __DIR__ . '/pens/pens_client.php';
+require_once __DIR__.'/pens/pens_request_factory.php';
+require_once __DIR__.'/pens/pens_response.php';
+require_once __DIR__.'/pens/pens_package_handler.php';
+require_once __DIR__.'/pens/pens_server.php';
+require_once __DIR__.'/pens/pens_client.php';

plugin/pens/lib/pens_server_test.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
  * @licence http://www.gnu.org/licenses/gpl.txt
  */
 
-require_once __DIR__ . '/pens.php';
+require_once __DIR__.'/pens.php';
 
 class MyPackageHandler extends PENSPackageHandler {
 	public function processPackage($request, $path_to_package) {

plugin/pens/lib/tests/test_pens_request.php

Spacing +4 added lines, -4 removed lines

@@ -79,14 +79,14 @@ 
 	public function exceptionTestForValue($key, $value, $code) {
 		try {
 			$myargs = $this->args;
-			if($value === null) {
+			if ($value === null) {
 				unset($myargs[$key]);
 			} else {
 				$myargs[$key] = $value;
 			}
 			$object = $this->createObject($myargs);
 			$this->fail();
-		} catch(PENSException $e) {
+		} catch (PENSException $e) {
 			$this->assertEqual($e->getCode(), $code);
 		}
 	}
@@ -178,7 +178,7 @@ 
 			$myargs["receipt"] = "mailto:name@domain.com,name2@domain.com";
 			$object = $this->createObject($myargs);
 			$this->pass();
-		} catch(PENSException $e) {
+		} catch (PENSException $e) {
 			$this->fail();
 		}
 	}
@@ -204,7 +204,7 @@ 
 			$this->assertEqual($object->getAlerts(), "http://myurl.com/alerts");
 			$this->assertEqual($object->getVendorData(), "here are my data");
 			
-		} catch(PENSException $e) {
+		} catch (PENSException $e) {
 			$this->fail();
 		}
 	}

plugin/tour/ajax/save.ajax.php

Spacing +2 added lines, -2 removed lines

@@ -8,8 +8,8 @@
 /**
  * Init
  */
-require_once __DIR__ . '/../../../main/inc/global.inc.php';
-require_once __DIR__ . '/../config.php';
+require_once __DIR__.'/../../../main/inc/global.inc.php';
+require_once __DIR__.'/../config.php';
 
 if (!api_is_anonymous()) {
     $currentPageClass = isset($_POST['page_class']) ? $_POST['page_class'] : '';

plugin/advanced_subscription/src/AdvancedSubscriptionPlugin.php

Spacing +41 added lines, -42 removed lines

@@ -106,15 +106,15 @@ 
     {
         $advancedSubscriptionQueueTable = Database::get_main_table(TABLE_ADVANCED_SUBSCRIPTION_QUEUE);
 
-        $sql = "CREATE TABLE IF NOT EXISTS $advancedSubscriptionQueueTable (" .
-            "id int UNSIGNED NOT NULL AUTO_INCREMENT, " .
-            "session_id int UNSIGNED NOT NULL, " .
-            "user_id int UNSIGNED NOT NULL, " .
-            "status int UNSIGNED NOT NULL, " .
-            "last_message_id int UNSIGNED NOT NULL, " .
-            "created_at datetime NOT NULL, " .
-            "updated_at datetime NULL, " .
-            "PRIMARY KEY PK_advanced_subscription_queue (id), " .
+        $sql = "CREATE TABLE IF NOT EXISTS $advancedSubscriptionQueueTable (".
+            "id int UNSIGNED NOT NULL AUTO_INCREMENT, ".
+            "session_id int UNSIGNED NOT NULL, ".
+            "user_id int UNSIGNED NOT NULL, ".
+            "status int UNSIGNED NOT NULL, ".
+            "last_message_id int UNSIGNED NOT NULL, ".
+            "created_at datetime NOT NULL, ".
+            "updated_at datetime NULL, ".
+            "PRIMARY KEY PK_advanced_subscription_queue (id), ".
             "UNIQUE KEY UK_advanced_subscription_queue (user_id, session_id)); ";
         Database::query($sql);
     }
@@ -265,11 +265,10 @@ 
         $now = new DateTime(api_get_utc_datetime());
         $newYearDate = $plugin->get('course_session_credit_year_start_date');
         $newYearDate = !empty($newYearDate) ?
-            new \DateTime($newYearDate . $now->format('/Y')) :
-            $now;
+            new \DateTime($newYearDate.$now->format('/Y')) : $now;
         $extra = new ExtraFieldValue('session');
-        $joinSessionTable = Database::get_main_table(TABLE_MAIN_SESSION_USER) . ' su INNER JOIN ' .
-            Database::get_main_table(TABLE_MAIN_SESSION) . ' s ON s.id = su.session_id';
+        $joinSessionTable = Database::get_main_table(TABLE_MAIN_SESSION_USER).' su INNER JOIN '.
+            Database::get_main_table(TABLE_MAIN_SESSION).' s ON s.id = su.session_id';
         $whereSessionParams = 'su.relation_type = ? AND s.access_start_date >= ? AND su.user_id = ?';
         $whereSessionParamsValues = array(
             0,
@@ -734,14 +733,14 @@ 
                     $tpl->assign('termsContent', $termsAndConditions);
                     $termsAndConditions = $tpl->fetch('/advanced_subscription/views/terms_and_conditions_to_pdf.tpl');
                     $pdf = new PDF();
-                    $filename = 'terms' . sha1(rand(0,99999));
+                    $filename = 'terms'.sha1(rand(0, 99999));
                     $pdf->content_to_pdf($termsAndConditions, null, $filename, null, 'F');
                     $fileAttachments['file'][] = array(
-                        'name' => $filename . '.pdf',
-                        'application/pdf' => $filename . '.pdf',
-                        'tmp_name' => api_get_path(SYS_ARCHIVE_PATH) . $filename . '.pdf',
+                        'name' => $filename.'.pdf',
+                        'application/pdf' => $filename.'.pdf',
+                        'tmp_name' => api_get_path(SYS_ARCHIVE_PATH).$filename.'.pdf',
                         'error' => UPLOAD_ERR_OK,
-                        'size' => filesize(api_get_path(SYS_ARCHIVE_PATH) . $filename . '.pdf'),
+                        'size' => filesize(api_get_path(SYS_ARCHIVE_PATH).$filename.'.pdf'),
                     );
                     $fileAttachments['comments'][] = get_lang('TermsAndConditions');
                 }
@@ -1032,7 +1031,7 @@ 
 
             $mergedArray = array_merge(array($sessionId), array_keys($fields));
 
-            $sql = "SELECT * FROM " . Database::get_main_table(TABLE_EXTRA_FIELD_VALUES) ."
+            $sql = "SELECT * FROM ".Database::get_main_table(TABLE_EXTRA_FIELD_VALUES)."
                     WHERE item_id = %d AND field_id IN (%d, %d, %d, %d, %d, %d, %d)";
             $sql = vsprintf($sql, $mergedArray);
             $sessionFieldValueList = Database::query($sql);
@@ -1048,10 +1047,10 @@ 
             $sessionArray['description'] = SessionManager::getDescriptionFromSessionId($sessionId);
 
             if (isset($sessionArray['brochure'])) {
-                $sessionArray['brochure'] = api_get_path(WEB_UPLOAD_PATH) . $sessionArray['brochure'];
+                $sessionArray['brochure'] = api_get_path(WEB_UPLOAD_PATH).$sessionArray['brochure'];
             }
             if (isset($sessionArray['banner'])) {
-                $sessionArray['banner'] = api_get_path(WEB_UPLOAD_PATH) . $sessionArray['banner'];
+                $sessionArray['banner'] = api_get_path(WEB_UPLOAD_PATH).$sessionArray['banner'];
             }
 
             return $sessionArray;
@@ -1108,7 +1107,7 @@ 
      */
     public function getSessionUrl($sessionId)
     {
-        $url = api_get_path(WEB_CODE_PATH) . 'session/?session_id=' . intval($sessionId);
+        $url = api_get_path(WEB_CODE_PATH).'session/?session_id='.intval($sessionId);
 
         return $url;
     }
@@ -1157,16 +1156,16 @@ 
      */
     public function getQueueUrl($params)
     {
-        $url = api_get_path(WEB_PLUGIN_PATH) . 'advanced_subscription/ajax/advanced_subscription.ajax.php?' .
-            'a=' . Security::remove_XSS($params['action']) . '&' .
-            's=' . intval($params['sessionId']) . '&' .
-            'current_user_id=' . intval($params['currentUserId']) . '&' .
-            'e=' . intval($params['newStatus']) . '&' .
-            'u=' . intval($params['studentUserId']) . '&' .
-            'q=' . intval($params['queueId']) . '&' .
-            'is_connected=' . 1 . '&' .
-            'profile_completed=' . intval($params['profile_completed']) . '&' .
-            'v=' . $this->generateHash($params);
+        $url = api_get_path(WEB_PLUGIN_PATH).'advanced_subscription/ajax/advanced_subscription.ajax.php?'.
+            'a='.Security::remove_XSS($params['action']).'&'.
+            's='.intval($params['sessionId']).'&'.
+            'current_user_id='.intval($params['currentUserId']).'&'.
+            'e='.intval($params['newStatus']).'&'.
+            'u='.intval($params['studentUserId']).'&'.
+            'q='.intval($params['queueId']).'&'.
+            'is_connected='.1.'&'.
+            'profile_completed='.intval($params['profile_completed']).'&'.
+            'v='.$this->generateHash($params);
 
         return $url;
     }
@@ -1219,7 +1218,7 @@ 
         }
         $queueTable = Database::get_main_table(TABLE_ADVANCED_SUBSCRIPTION_QUEUE);
         $userTable = Database::get_main_table(TABLE_MAIN_USER);
-        $userJoinTable = $queueTable . ' q INNER JOIN ' . $userTable . ' u ON q.user_id = u.user_id';
+        $userJoinTable = $queueTable.' q INNER JOIN '.$userTable.' u ON q.user_id = u.user_id';
         $where = array(
             'where' => array(
                 'q.session_id = ?' => array(
@@ -1232,7 +1231,7 @@ 
         $students = Database::select($select, $userJoinTable, $where);
         foreach ($students as &$student) {
             $status = intval($student['status']);
-            switch($status) {
+            switch ($status) {
                 case ADVANCED_SUBSCRIPTION_QUEUE_STATUS_NO_QUEUE:
                 case ADVANCED_SUBSCRIPTION_QUEUE_STATUS_START:
                     $student['validation'] = '';
@@ -1246,7 +1245,7 @@ 
                     $student['validation'] = 'Yes';
                     break;
                 default:
-                    error_log(__FILE__ . ' ' . __FUNCTION__ . ' Student status no detected');
+                    error_log(__FILE__.' '.__FUNCTION__.' Student status no detected');
             }
         }
         $return = array(
@@ -1295,7 +1294,7 @@ 
         $dataPrepared['queueId'] = intval($data['queueId']);
         $dataPrepared['newStatus'] = intval($data['newStatus']);
         $dataPrepared = serialize($dataPrepared);
-        return sha1($dataPrepared . $key);
+        return sha1($dataPrepared.$key);
     }
 
     /**
@@ -1350,12 +1349,12 @@ 
                 break;
         }
 
-        $url = api_get_path(WEB_PLUGIN_PATH) . "advanced_subscription/src/terms_and_conditions.php?";
+        $url = api_get_path(WEB_PLUGIN_PATH)."advanced_subscription/src/terms_and_conditions.php?";
         $url .= http_build_query($urlParams);
 
         // Launch popup
         if ($mode == ADVANCED_SUBSCRIPTION_TERMS_MODE_POPUP) {
-            $url = 'javascript:void(window.open(\'' . $url .'\',\'AdvancedSubscriptionTerms\', \'toolbar=no,location=no,status=no,menubar=no,scrollbars=yes,resizable=yes,width=700px,height=600px\', \'100\' ))';
+            $url = 'javascript:void(window.open(\''.$url.'\',\'AdvancedSubscriptionTerms\', \'toolbar=no,location=no,status=no,menubar=no,scrollbars=yes,resizable=yes,width=700px,height=600px\', \'100\' ))';
         }
         return $url;
     }
@@ -1367,9 +1366,9 @@ 
      */
     public function getRenderMailUrl($params)
     {
-        $url = api_get_path(WEB_PLUGIN_PATH) . 'advanced_subscription/src/render_mail.php?' .
-            'q=' . $params['queueId'] . '&' .
-            'v=' . $this->generateHash($params);
+        $url = api_get_path(WEB_PLUGIN_PATH).'advanced_subscription/src/render_mail.php?'.
+            'q='.$params['queueId'].'&'.
+            'v='.$this->generateHash($params);
         return $url;
     }
 
@@ -1440,7 +1439,7 @@ 
                 sf.extra_field_type = $extraFieldType AND
                 sf.variable = 'is_induction_session' AND
                 su.relation_type = 0 AND
-                su.user_id = " . intval($userId);
+                su.user_id = ".intval($userId);
 
         $result = Database::query($sql);
 

plugin/advanced_subscription/src/HookAdvancedSubscription.php

Spacing +4 added lines, -4 removed lines

@@ -6,7 +6,7 @@ 
  * @package chamilo.plugin.advanced_subscription
  */
 
-require_once __DIR__ . '/../config.php';
+require_once __DIR__.'/../config.php';
 
 /**
  * Class HookAdvancedSubscription extends the HookObserver to implements
@@ -420,7 +420,7 @@ 
 
         if ($debug) {
             error_log(__FUNCTION__);
-            error_log('Params ' . print_r($params, 1));
+            error_log('Params '.print_r($params, 1));
         }
         if (!WSHelperVerifyKey($params)) {
 
@@ -471,7 +471,7 @@ 
 
         if ($debug) {
             error_log('WSUserSubscribedInCourse');
-            error_log('Params ' . print_r($params, 1));
+            error_log('Params '.print_r($params, 1));
         }
         if (!WSHelperVerifyKey($params)) {
             return return_error(WS_ERROR_SECRET_KEY);
@@ -576,7 +576,7 @@ 
 
         if ($debug) {
             error_log('WSListSessionsDetailsByCategory');
-            error_log('Params ' . print_r($params, 1));
+            error_log('Params '.print_r($params, 1));
         }
         $secretKey = $params['secret_key'];
 

plugin/advanced_subscription/src/admin_view.php

Spacing +2 added lines, -3 removed lines

@@ -7,7 +7,7 @@ 
 /**
  * Init
  */
-require_once __DIR__ . '/../config.php';
+require_once __DIR__.'/../config.php';
 // protect
 api_protect_admin_script();
 // start plugin
@@ -85,8 +85,7 @@ 
         $data['newStatus'] = ADVANCED_SUBSCRIPTION_QUEUE_STATUS_ADMIN_DISAPPROVED;
         $student['rejectUrl'] = $plugin->getQueueUrl($data);
         $student['complete_name'] = $isWesternNameOrder ?
-            $student['firstname'] . ', ' . $student['lastname'] :
-            $student['lastname'] . ', ' . $student['firstname'];
+            $student['firstname'].', '.$student['lastname'] : $student['lastname'].', '.$student['firstname'];
     }
     $tpl->assign('session', $sessionArray);
     $tpl->assign('students', $studentList['students']);

plugin/advanced_subscription/src/scripts/insert_session_fields.php

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
 
 //exit;
 
-require_once __DIR__ . '/../../config.php';
+require_once __DIR__.'/../../config.php';
 
 api_protect_admin_script();