chamilo / chamilo-lms

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();
 

plugin/advanced_subscription/src/terms_and_conditions.php

Spacing +2 added lines, -2 removed lines

@@ -7,7 +7,7 @@ 
 /**
  * Init
  */
-require_once __DIR__ . '/../config.php';
+require_once __DIR__.'/../config.php';
 // start plugin
 $plugin = AdvancedSubscriptionPlugin::create();
 // Session ID
@@ -24,7 +24,7 @@ 
 // Init template
 $tpl = new Template($plugin->get_lang('plugin_title'));
 
-$isAllowToDoRequest = $plugin->isAllowedToDoRequest( $data['studentUserId'], $data, true);
+$isAllowToDoRequest = $plugin->isAllowedToDoRequest($data['studentUserId'], $data, true);
 
 if (!$isAllowToDoRequest) {
     $tpl->assign('errorMessages', $plugin->getErrorMessages());

plugin/advanced_subscription/src/render_mail.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 /**
  * Init
  */
-require_once __DIR__ . '/../config.php';
+require_once __DIR__.'/../config.php';
 
 $plugin = AdvancedSubscriptionPlugin::create();
 // Get validation hash

plugin/advanced_subscription/test/mails.php

Spacing +9 added lines, -10 removed lines

@@ -5,7 +5,7 @@ 
  * @package chamilo.plugin.advanced_subscription
  */
 
-require_once __DIR__ . '/../config.php';
+require_once __DIR__.'/../config.php';
 
 // Protect test
 api_protect_admin_script();
@@ -78,8 +78,7 @@ 
 $isWesternNameOrder = api_is_western_name_order();
 foreach ($adminsArray as &$admin) {
     $admin['complete_name'] = $isWesternNameOrder ?
-        $admin['firstname'] . ', ' . $admin['lastname'] :
-        $admin['lastname'] . ', ' . $admin['firstname']
+        $admin['firstname'].', '.$admin['lastname'] : $admin['lastname'].', '.$admin['firstname']
     ;
 }
 unset($admin);
@@ -91,8 +90,8 @@ 
 $data['admin'] = current($adminsArray);
 $data['session'] = $sessionArray;
 $data['signature'] = api_get_setting('Institution');
-$data['admin_view_url'] = api_get_path(WEB_PLUGIN_PATH) .
-    'advanced_subscription/src/admin_view.php?s=' . $data['sessionId'];
+$data['admin_view_url'] = api_get_path(WEB_PLUGIN_PATH).
+    'advanced_subscription/src/admin_view.php?s='.$data['sessionId'];
 $data['newStatus'] = ADVANCED_SUBSCRIPTION_QUEUE_STATUS_BOSS_APPROVED;
 $data['student']['acceptUrl'] = $plugin->getQueueUrl($data);
 $data['newStatus'] = ADVANCED_SUBSCRIPTION_QUEUE_STATUS_BOSS_DISAPPROVED;
@@ -116,21 +115,21 @@ 
     $tpl->assign($tplParam, $data[$tplParam]);
 }
 
-$dir = __DIR__ . '/../views/';
+$dir = __DIR__.'/../views/';
 $files = scandir($dir);
 
-echo '<br>', '<pre>' , print_r($files, 1) , '</pre>';
+echo '<br>', '<pre>', print_r($files, 1), '</pre>';
 
 foreach ($files as $k =>&$file) {
     if (
-        is_file($dir . $file) &&
+        is_file($dir.$file) &&
         strpos($file, '.tpl') &&
         $file != 'admin_view.tpl'
     ) {
         echo '<pre>', $file, '</pre>';
-        echo $tpl->fetch('/advanced_subscription/views/' . $file);
+        echo $tpl->fetch('/advanced_subscription/views/'.$file);
     } else {
         unset($files[$k]);
     }
 }
-echo '<br>', '<pre>' , print_r($files, 1) , '</pre>';
+echo '<br>', '<pre>', print_r($files, 1), '</pre>';