chamilo / chamilo-lms

main/permissions/user_permissions.inc.php

Braces +11 added lines, -22 removed lines

@@ -19,12 +19,10 @@ 
     }
 }
 if (isset($_GET['action'])) {
-    if (isset($_GET['permission']) AND isset($_GET['tool']) AND ($_GET['action'] == 'grant' OR $_GET['action'] == 'revoke'))
-    {
+    if (isset($_GET['permission']) AND isset($_GET['tool']) AND ($_GET['action'] == 'grant' OR $_GET['action'] == 'revoke')) {
         $result_message = store_one_permission('user', $_GET['action'], $user_id, $_GET['tool'], $_GET['permission']);
     }
-    if (isset($_GET['role']) AND ($_GET['action'] == 'grant' OR $_GET['action'] == 'revoke'))
-    {
+    if (isset($_GET['role']) AND ($_GET['action'] == 'grant' OR $_GET['action'] == 'revoke')) {
         $result_message = assign_role('user', $_GET['action'], $user_id, $_GET['role'], $_GET['scope']);
     }
 }
@@ -45,13 +43,10 @@ 
 // 			RETRIEVING THE PERMISSIONS OF THE GROUPS OF THE USER
 $groups_of_user = array();
 $groups_of_user = GroupManager::get_group_ids($_course['real_id'], $user_id);
-foreach ($groups_of_user as $group)
-{
+foreach ($groups_of_user as $group) {
     $this_group_permissions = get_permissions('group', $group);
-    foreach ($this_group_permissions as $tool=>$permissions)
-    {
-        foreach ($permissions as $permission)
-        {
+    foreach ($this_group_permissions as $tool=>$permissions) {
+        foreach ($permissions as $permission) {
             $inherited_group_permissions[$tool][] = $permission;
         }
     }
@@ -76,11 +71,9 @@ 
     $inherited_permissions = permission_array_merge($inherited_permissions, $current_user_role_permissions_of_user);
 }
 //	RETRIEVING THE PERMISSIONS OF THE ROLES OF THE GROUPS OF THE USER
-if (api_get_setting('group_roles') == 'true')
-{
+if (api_get_setting('group_roles') == 'true') {
     // NOTE: DIT MOET NOG VERDER UITGEWERKT WORDEN
-    foreach ($groups_of_user as $group)
-    {
+    foreach ($groups_of_user as $group) {
         $this_current_group_role_permissions_of_user = get_roles_permissions('user', $user_id);
         //$inherited_permissions[$tool][]=$permission;
     }
@@ -89,20 +82,17 @@ 
 // 			LIMITED OR FULL
 $current_user_permissions = limited_or_full($current_user_permissions);
 $inherited_permissions = limited_or_full($inherited_permissions);
-if (api_get_setting('permissions') == 'limited')
-{
+if (api_get_setting('permissions') == 'limited') {
     $header_array = $rights_limited;
 }
-if (api_get_setting('permissions') == 'full')
-{
+if (api_get_setting('permissions') == 'full') {
     $header_array = $rights_full;
 }
 
 echo "<form method=\"post\" action=\"".str_replace('&', '&amp;', $_SERVER['REQUEST_URI'])."\">";
 // 		DISPLAYING THE ROLES LIST
 
-if (api_get_setting('user_roles') == 'true')
-{
+if (api_get_setting('user_roles') == 'true') {
     // the list of the roles for the user
     echo '<strong>'.get_lang('UserRoles').'</strong><br />';
     $current_user_course_roles = get_roles('user', $user_id);
@@ -120,8 +110,7 @@ 
 // the header
 echo "\t<tr>\n";
 echo "\t\t<th>".get_lang('Module')."</th>\n";
-foreach ($header_array as $header_key=>$header_value)
-{
+foreach ($header_array as $header_key=>$header_value) {
     echo "\t\t<th>".get_lang($header_value)."</th>\n";
 }
 echo "\t</tr>\n";

main/admin/ldap_import_students_to_session.php

Braces +7 added lines, -16 removed lines

@@ -51,8 +51,7 @@ 
 
 // form1 annee = 0; composante= 0 etape = 0
 //if ($annee == "" && $composante == "" && $etape == "") {
-if (empty($annee) && empty($id_session))
-{
+if (empty($annee) && empty($id_session)) {
 		Display::display_header($tool_name);
 		echo '<div style="align:center">';
 		echo Display::return_icon('group.gif', get_lang('LDAPSelectFilterOnUsersOU')).' '.get_lang('LDAPSelectFilterOnUsersOU');
@@ -63,9 +62,7 @@ 
 		echo '</form>';
 		echo '</div>';
 
-}
-elseif (!empty($annee) && empty($id_session))
-{
+} elseif (!empty($annee) && empty($id_session)) {
 	Display::display_header($tool_name);
 	echo '<div style="align:center">';
 	echo Display::return_icon(
@@ -92,8 +89,7 @@ 
 }
 // form4  annee != 0; composante != 0 etape != 0
 //elseif ($annee <> "" && $composante <> "" && $etape <> "" && $listeok != 'yes') {
-elseif (!empty($annee) && !empty($id_session) && empty($_POST['confirmed']))
-{
+elseif (!empty($annee) && !empty($id_session) && empty($_POST['confirmed'])) {
 	Display::display_header($tool_name);
 	echo '<div style="align: center;">';
 	echo '<br />';
@@ -144,16 +140,12 @@ 
     echo '<br /><br />';
     echo '</div>';
 
-}
-elseif (!empty($annee) && !empty($id_session) && ($_POST['confirmed'] == 'yes'))
-{
+} elseif (!empty($annee) && !empty($id_session) && ($_POST['confirmed'] == 'yes')) {
 	$id = $_POST['username_form'];
 	$UserList = array();
 	$userid_match_login = array();
-	foreach ($id as $form_index=>$user_id)
-	{
-		if (is_array($_POST['checkboxes']) && in_array($form_index, array_values($_POST['checkboxes'])))
-		{
+	foreach ($id as $form_index=>$user_id) {
+		if (is_array($_POST['checkboxes']) && in_array($form_index, array_values($_POST['checkboxes']))) {
 			$tmp = ldap_add_user($user_id);
 			$UserList[] = $tmp;
 			$userid_match_login[$tmp] = $user_id;
@@ -202,8 +194,7 @@ 
 		Display::addFlash(Display::return_message($message, 'normal', false));
 	}
 	*/
-	else
-	{
+	else {
 		$message = get_lang('NoUserAdded');
 		Display::addFlash(Display::return_message($message, 'normal', false));
         Display::display_header($tool_name);

plugin/vchamilo/cli/clilib.php

Braces +28 added lines, -12 removed lines

@@ -7,7 +7,8 @@ 
  * @param string $plugin
  * @return mixed
  */
-function vchamilo_parse_csv_nodelist($nodelistlocation = '', $plugin = null) {
+function vchamilo_parse_csv_nodelist($nodelistlocation = '', $plugin = null)
+{
     global $_configuration;
 
     $vnodes = array();
@@ -102,7 +103,8 @@ 
 
     // Check for required fields.
     foreach ($required as $key => $value) {
-        if ($value) { // Required field missing.
+        if ($value) {
+// Required field missing.
             cli_error("fieldrequired $key");
             return;
         }
@@ -136,7 +138,9 @@ 
             $key = $headers[$f];
             if (preg_match('/\|/', $key)) {
                 list($plugin, $variable) = explode('|', str_replace('plugin_', '', $key));
-                if (empty($variable)) die("Key error in CSV : $key ");
+                if (empty($variable)) {
+                    die("Key error in CSV : $key ");
+                }
                 if (!isset($vnode->$plugin)) {
                     $vnode->$plugin = new StdClass();
                 }
@@ -169,15 +173,21 @@ 
  * @param bool $resetfirst
  * @return bool
  */
-function vchamilo_is_empty_line_or_format(&$text, $resetfirst = false) {
+function vchamilo_is_empty_line_or_format(&$text, $resetfirst = false)
+{
     global $CFG;
 
     static $textlib;
     static $first = true;
 
     // We may have a risk the BOM is present on first line
-    if ($resetfirst) $first = true;
-    if (!isset($textlib)) $textlib = new textlib(); // Singleton
+    if ($resetfirst) {
+        $first = true;
+    }
+    if (!isset($textlib)) {
+        $textlib = new textlib();
+    }
+    // Singleton
     $text = $textlib->trim_utf8_bom($text);
     $first = false;
 
@@ -200,7 +210,8 @@ 
  * @param bool $casesensitiveoptions true if options are case sensitive
  * @return string entered text
  */
-function cli_input($prompt, $default = '', array $options = null, $casesensitiveoptions = false) {
+function cli_input($prompt, $default = '', array $options = null, $casesensitiveoptions = false)
+{
     echo $prompt;
     echo "\n: ";
     $input = fread(STDIN, 2048);
@@ -226,7 +237,8 @@ 
  * @param array $shortmapping array describing mapping of short to long style options ex:('h'=>'help', 'v'=>'verbose')
  * @return array array of arrays, options, unrecognised as optionlongname=>value
  */
-function cli_get_params(array $longoptions, array $shortmapping = null) {
+function cli_get_params(array $longoptions, array $shortmapping = null)
+{
     $shortmapping = (array) $shortmapping;
     $options      = array();
     $unrecognized = array();
@@ -296,7 +308,8 @@ 
  * @param bool $return false means print, true return as string
  * @return mixed void or string
  */
-function cli_separator($return = false) {
+function cli_separator($return = false)
+{
     $separator = str_repeat('-', 79)."\n";
     if ($return) {
         return $separator;
@@ -311,7 +324,8 @@ 
  * @param bool $return false means print, true return as string
  * @return mixed void or string
  */
-function cli_heading($string, $return = false) {
+function cli_heading($string, $return = false)
+{
     $string = "== $string ==\n";
     if ($return) {
         return $string;
@@ -325,7 +339,8 @@ 
  * @param $text
  * @return void
  */
-function cli_problem($text) {
+function cli_problem($text)
+{
     fwrite(STDERR, $text."\n");
 }
 
@@ -336,7 +351,8 @@ 
  * @param int $errorCode
  * @return void (does not return)
  */
-function cli_error($text, $errorCode = 1) {
+function cli_error($text, $errorCode = 1)
+{
     fwrite(STDERR, $text);
     fwrite(STDERR, "\n");
     die($errorCode);

main/auth/gotocourse.php

Braces +4 added lines, -4 removed lines

@@ -14,7 +14,7 @@ 
 require_once __DIR__.'/../inc/global.inc.php';

 require_once api_get_path(SYS_PATH).'main/auth/cas/authcas.php';

 $msg = null;

-if (isset($_GET['firstpage'])) {

+if (isset($_GET['firstpage'])) {
     $firstpage = $_GET['firstpage'];

 

     // if course is public, go to course without auth

@@ -41,7 +41,7 @@ 
     );

     $form->addButtonNext(get_lang('LoginEnter'), 'submitAuth');

     // see same text in main_api.lib.php function api_not_allowed

-    if (api_is_cas_activated()) {

+    if (api_is_cas_activated()) {
         $msg .= Display::return_message(sprintf(get_lang('YouHaveAnInstitutionalAccount'), api_get_setting("Institution")), '', false);

         $msg .= Display::div("<br/><a href='".get_cas_direct_URL(api_get_course_id())."'>".getCASLogoHTML()." ".sprintf(get_lang('LoginWithYourAccount'), api_get_setting("Institution"))."</a><br/><br/>", array('align'=>'center'));

         $msg .= Display::return_message(get_lang('YouDontHaveAnInstitutionAccount'));

@@ -52,7 +52,7 @@ 
     $msg .= '<div class="well_login">';

     $msg .= $form->returnForm();

     $msg .= '</div>';

-    if (api_is_cas_activated()) {

+    if (api_is_cas_activated()) {
         $msg .= "</div>";

     }

     $msg .= '<hr/><p style="text-align:center"><a href="'.api_get_path(WEB_PATH).'">'.get_lang('ReturnToCourseHomepage').'</a></p>';

@@ -60,7 +60,7 @@ 
     $tpl->assign('content', '<h4>'.get_lang('LoginToGoToThisCourse').'</h4>'.$msg);

     $tpl->display_one_col_template();

 

-} else {

+} else {
     api_delete_firstpage_parameter();

     header('Location: '.api_get_path(WEB_PATH).'index.php');

     exit;

main/admin/ldap_users_list.php

Braces +2 added lines, -4 removed lines

@@ -15,8 +15,7 @@ 
 $login_as_user_id = @$_GET["user_id"] ?: null;
 
 // Login as ...
-if ($action == "login_as" && !empty ($login_as_user_id))
-{
+if ($action == "login_as" && !empty ($login_as_user_id)) {
     login_user($login_as_user_id);
 }
 
@@ -32,8 +31,7 @@ 
 //Display::display_header($tool_name); //cannot display now as we need to redirect
 //api_display_tool_title($tool_name);
 
-if (isset ($_GET['action']))
-{
+if (isset ($_GET['action'])) {
     $check = Security::check_token('get');
     if ($check) {
         switch ($_GET['action']) {

plugin/buycourses/src/service_success.php

Braces +9 added lines, -9 removed lines

@@ -10,7 +10,7 @@ 
 $plugin = BuyCoursesPlugin::create();

 $paypalEnabled = $plugin->get('paypal_enable') === 'true';

 

-if (!$paypalEnabled) {

+if (!$paypalEnabled) {
     api_not_allowed(true);

 }

 

@@ -18,7 +18,7 @@ 
 $serviceSale = $plugin->getServiceSale($serviceSaleId);

 $itemPrice = $serviceSale['price'];

 

-if (empty($serviceSale)) {

+if (empty($serviceSale)) {
     api_not_allowed(true);

 }

 

@@ -49,9 +49,9 @@ 
 );

 $form->addButtonCancel($plugin->get_lang('CancelOrder'), 'cancel');

 

-if ($form->validate()) {

+if ($form->validate()) {
     $formValues = $form->getSubmitValues();

-    if (isset($formValues['cancel'])) {

+    if (isset($formValues['cancel'])) {
         $plugin->cancelServiceSale($serviceSale['id']);

 

         unset($_SESSION['bc_service_sale_id']);

@@ -65,7 +65,7 @@ 
     }

 

     $confirmPayments = ConfirmPayment($itemPrice);

-    if ($confirmPayments['ACK'] !== 'Success') {

+    if ($confirmPayments['ACK'] !== 'Success') {
         $erroMessage = vsprintf(

             $plugin->get_lang('ErrorOccurred'),

             [$expressCheckout['L_ERRORCODE0'], $confirmPayments['L_LONGMESSAGE0']]

@@ -78,11 +78,11 @@ 
         exit;

     }

 

-    switch ($confirmPayments["PAYMENTINFO_0_PAYMENTSTATUS"]) {

+    switch ($confirmPayments["PAYMENTINFO_0_PAYMENTSTATUS"]) {
         case 'Completed':

             $serviceSaleIsCompleted = $plugin->completeServiceSale($serviceSale['id']);

 

-            if ($serviceSaleIsCompleted) {

+            if ($serviceSaleIsCompleted) {
                 Display::addFlash(

                     Display::return_message(

                         sprintf($plugin->get_lang('SubscriptionToServiceXSuccessful'), $serviceSale['service']['name']),

@@ -98,7 +98,7 @@ 
             );

             break;

         case 'Pending':

-            switch ($confirmPayments["PAYMENTINFO_0_PENDINGREASON"]) {

+            switch ($confirmPayments["PAYMENTINFO_0_PENDINGREASON"]) {
                 case 'address':

                     $purchaseStatus = $plugin->get_lang('PendingReasonByAddress');

                     break;

@@ -164,7 +164,7 @@ 
 }

 

 $token = isset($_GET['token']) ? Security::remove_XSS($_GET['token']) : null;

-if (empty($token)) {

+if (empty($token)) {
     api_not_allowed(true);

 }

 

main/survey/surveyUtil.class.php

Braces +2 added lines, -1 removed lines

@@ -2099,7 +2099,8 @@
 
         // Remind unanswered is a special version of remind all reminder
         $exclude_users = array();
-        if ($remindUnAnswered == 1) { // Remind only unanswered users
+        if ($remindUnAnswered == 1) {
+// Remind only unanswered users
             $reminder = 1;
             $exclude_users = SurveyManager::get_people_who_filled_survey($_GET['survey_id']);
         }

main/exercise/question_pool.php

Braces +6 added lines, -3 removed lines

@@ -675,7 +675,8 @@ 
         $actionIcon2 = "delete";
         // We are in the course, question title can be a link to the question edit page
         $questionTagA = 1;
-    } else { // NOT IN A TEST - NOT IN THE COURSE
+    } else {
+// NOT IN A TEST - NOT IN THE COURSE
         $actionLabel = get_lang('Reuse');
         $actionIcon1 = get_lang('MustBeInATest');
         $actionIcon2 = "";
@@ -828,7 +829,8 @@ 
     $in_questiontype,
     $in_questionname,
     $sessionId
-) {
+)
+{
     $res = $in_questionname;
     $sessionIcon = null;
     if ($in_addA) {
@@ -868,7 +870,8 @@ 
     $in_answerType,
     $in_session_id,
     $in_exercise_id
-) {
+)
+{
     $res = "";
     $getParams = "&selected_course=$in_selected_course&courseCategoryId=$in_courseCategoryId&exerciseId=$in_exercise_id&exerciseLevel=$in_exerciseLevel&answerType=$in_answerType&session_id=$in_session_id";
     switch ($in_action) {

main/inc/lib/fileUpload.lib.php

Braces +19 added lines, -9 removed lines

@@ -223,7 +223,8 @@ 
     $comment = null,
     $sessionId = null,
     $treat_spaces_as_hyphens = true
-) {
+)
+{
     if (!$userId) {
         return false;
     }
@@ -985,7 +986,8 @@ 
             - add it to the database
             - parse & change relative html links
         */
-        if (PHP_OS == 'Linux' && !get_cfg_var('safe_mode') && false) { // *** UGent, changed by OC ***
+        if (PHP_OS == 'Linux' && !get_cfg_var('safe_mode') && false) {
+// *** UGent, changed by OC ***
             // Shell Method - if this is possible, it gains some speed
             exec("unzip -d \"".$base_work_dir.$upload_path."/\"".$uploaded_file['name']." ".$uploaded_file['tmp_name']);
         } else {
@@ -1004,7 +1006,9 @@ 
                     if ($file != '.' && $file != '..') {
 
                         $filetype = 'file';
-                        if (is_dir($base_work_dir.$upload_path.'/'.$file)) $filetype = 'folder';
+                        if (is_dir($base_work_dir.$upload_path.'/'.$file)) {
+                            $filetype = 'folder';
+                        }
 
                         $safe_file = api_replace_dangerous_char($file);
                         @rename($base_work_dir.$upload_path.'/'.$file, $base_work_dir.$upload_path.'/'.$safe_file);
@@ -1056,7 +1060,8 @@ 
     $groupId = 0,
     $output = true,
     $onlyUploadFile = false
-) {
+)
+{
     $zip = new PclZip($uploaded_file['tmp_name']);
 
     // Check the zip content (real size and file extension)
@@ -1170,7 +1175,8 @@ 
         return 1; // Authorize directories
     }
     $blacklist = api_get_setting('upload_extensions_list_type');
-    if ($blacklist != 'whitelist') { // if = blacklist
+    if ($blacklist != 'whitelist') {
+// if = blacklist
         $extensions = explode(';', strtolower(api_get_setting('upload_extensions_blacklist')));
 
         $skip = api_get_setting('upload_extensions_skip');
@@ -1241,7 +1247,8 @@ 
     $group_id = null,
     $session_id = 0,
     $userId = 0
-) {
+)
+{
     $session_id = empty($session_id) ? api_get_session_id() : $session_id;
     $userId = empty($userId) ? api_get_user_id() : $userId;
 
@@ -1509,7 +1516,8 @@ 
     $title = null,
     $visibility = null,
     $generateNewNameIfExists = false
-) {
+)
+{
     $course_id = $_course['real_id'];
     $session_id = intval($session_id);
 
@@ -1683,7 +1691,8 @@ 
     $to_group_id,
     $to_user_id,
     $max_filled_space
-) {
+)
+{
     $number_of_uploaded_images = count($uploaded_file_collection['name']);
     $new_file_list = array();
     for ($i = 0; $i < $number_of_uploaded_images; $i++) {
@@ -1856,7 +1865,8 @@ 
     $groupId = 0,
     $output = false,
     $parent = array()
-) {
+)
+{
     if (empty($userInfo) || empty($courseInfo)) {
 
         return false;