chamilo / chamilo-lms

main/session/add_users_to_session.php

Indentation +38 added lines, -38 removed lines

@@ -38,7 +38,7 @@ 
 $add_type = 'unique';
 
 if (isset($_REQUEST['add_type']) && $_REQUEST['add_type'] != '') {
-	$add_type = Security::remove_XSS($_REQUEST['add_type']);
+    $add_type = Security::remove_XSS($_REQUEST['add_type']);
 }
 
 $page = isset($_GET['page']) ? Security::remove_XSS($_GET['page']) : null;
@@ -426,8 +426,8 @@ 
 
     if ($use_extra_fields) {
         $final_result = array();
-       	if (count($extra_field_result)>1) {
-	    for($i=0;$i<count($extra_field_result)-1;$i++) {
+            if (count($extra_field_result)>1) {
+        for($i=0;$i<count($extra_field_result)-1;$i++) {
                 if (is_array($extra_field_result[$i+1])) {
                     $final_result  = array_intersect($extra_field_result[$i],$extra_field_result[$i+1]);
                 }
@@ -503,7 +503,7 @@ 
                 'official_code' => $user['official_code']
             ) ;
             unset($users[$uid]);
-	}
+    }
     }
     unset($users); //clean to free memory
 
@@ -547,12 +547,12 @@ 
 }
 
 if ($add_type == 'multiple') {
-	$link_add_type_unique = '<a href="'.api_get_self().'?id_session='.$id_session.'&add='.$addProcess.'&add_type=unique">'.
+    $link_add_type_unique = '<a href="'.api_get_self().'?id_session='.$id_session.'&add='.$addProcess.'&add_type=unique">'.
         Display::return_icon('single.gif').get_lang('SessionAddTypeUnique').'</a>';
-	$link_add_type_multiple = Display::url(Display::return_icon('multiple.gif').get_lang('SessionAddTypeMultiple'), '');
+    $link_add_type_multiple = Display::url(Display::return_icon('multiple.gif').get_lang('SessionAddTypeMultiple'), '');
 } else {
-	$link_add_type_unique = Display::url(Display::return_icon('single.gif').get_lang('SessionAddTypeUnique'), '');
-	$link_add_type_multiple = '<a href="'.api_get_self().'?id_session='.$id_session.'&amp;add='.$addProcess.'&amp;add_type=multiple">'.Display::return_icon('multiple.gif').get_lang('SessionAddTypeMultiple').'</a>';
+    $link_add_type_unique = Display::url(Display::return_icon('single.gif').get_lang('SessionAddTypeUnique'), '');
+    $link_add_type_multiple = '<a href="'.api_get_self().'?id_session='.$id_session.'&amp;add='.$addProcess.'&amp;add_type=multiple">'.Display::return_icon('multiple.gif').get_lang('SessionAddTypeMultiple').'</a>';
 }
 $link_add_group = Display::url(
     Display::return_icon('multiple.gif',get_lang('RegistrationByUsersGroups')).get_lang('RegistrationByUsersGroups'),
@@ -580,17 +580,17 @@ 
 <?php echo '<legend>'.$tool_name.' ('.$session_info['name'].') </legend>'; ?>
 <?php
 if ($add_type=='multiple') {
-	if (is_array($extra_field_list)) {
-		if (is_array($new_field_list) && count($new_field_list)>0 ) {
-			echo '<h3>'.get_lang('FilterUsers').'</h3>';
-			foreach ($new_field_list as $new_field) {
-				echo $new_field['name'];
-				$varname = 'field_'.$new_field['variable'];
+    if (is_array($extra_field_list)) {
+        if (is_array($new_field_list) && count($new_field_list)>0 ) {
+            echo '<h3>'.get_lang('FilterUsers').'</h3>';
+            foreach ($new_field_list as $new_field) {
+                echo $new_field['name'];
+                $varname = 'field_'.$new_field['variable'];
                 $fieldtype = $new_field['type'];
-				echo '&nbsp;<select name="'.$varname.'">';
-				echo '<option value="0">--'.get_lang('Select').'--</option>';
-				foreach	($new_field['data'] as $option) {
-					$checked='';
+                echo '&nbsp;<select name="'.$varname.'">';
+                echo '<option value="0">--'.get_lang('Select').'--</option>';
+                foreach	($new_field['data'] as $option) {
+                    $checked='';
                     if ($fieldtype == ExtraField::FIELD_TYPE_TAG) {
                         if (isset($_POST[$varname])) {
                             if ($_POST[$varname] == $option['tag']) {
@@ -611,11 +611,11 @@ 
                 $extraHidden = $fieldtype == ExtraField::FIELD_TYPE_TAG ? '<input type="hidden" name="field_id" value="'.$option['field_id'].'" />' : '';
                 echo $extraHidden;
                 echo '&nbsp;&nbsp;';
-			}
-			echo '<input type="button" value="'.get_lang('Filter').'" onclick="validate_filter()" />';
-			echo '<br /><br />';
-		}
-	}
+            }
+            echo '<input type="button" value="'.get_lang('Filter').'" onclick="validate_filter()" />';
+            echo '<br /><br />';
+        }
+    }
 }
 ?>
 <input type="hidden" name="form_sent" value="1" />
@@ -623,7 +623,7 @@ 
 
 <?php
 if (!empty($errorMsg)) {
-	Display::display_normal_message($errorMsg); //main API
+    Display::display_normal_message($errorMsg); //main API
 }
 ?>
 <div id="multiple-add-session" class="row">
@@ -632,7 +632,7 @@ 
             <label><?php echo get_lang('UserListInPlatform') ?> </label>
             <?php
             if (!($add_type=='multiple')) {
-              ?>
+                ?>
               <input type="text" id="user_to_add" onkeyup="xajax_search_users(this.value,'single')" class="form-control" />
               <div id="ajax_list_users_single" class="select-list-ajax"></div>
               <?php
@@ -641,21 +641,21 @@ 
             <div id="ajax_list_users_multiple">
             <select id="origin_users" name="nosessionUsersList[]" multiple="multiple" size="15" class="form-control">
               <?php
-              foreach ($nosessionUsersList as $uid => $enreg) {
-              ?>
+                foreach ($nosessionUsersList as $uid => $enreg) {
+                ?>
                   <option value="<?php echo $uid; ?>" <?php if(in_array($uid,$UserList)) echo 'selected="selected"'; ?>>
                       <?php
-                      $personName = api_get_person_name($enreg['fn'], $enreg['ln']).' ('.$enreg['un'].') '.$enreg['official_code'];
-                      if ($showOfficialCode) {
-                          $officialCode = !empty($enreg['official_code']) ? $enreg['official_code'].' - ' : '? - ';
-                          $personName = $officialCode.api_get_person_name($enreg['fn'], $enreg['ln']).' ('.$enreg['un'].')';
-                      }
-                      echo $personName;
-                      ?>
+                        $personName = api_get_person_name($enreg['fn'], $enreg['ln']).' ('.$enreg['un'].') '.$enreg['official_code'];
+                        if ($showOfficialCode) {
+                            $officialCode = !empty($enreg['official_code']) ? $enreg['official_code'].' - ' : '? - ';
+                            $personName = $officialCode.api_get_person_name($enreg['fn'], $enreg['ln']).' ('.$enreg['un'].')';
+                        }
+                        echo $personName;
+                        ?>
                   </option>
               <?php
-              }
-              ?>
+                }
+                ?>
             </select>
             </div>
                 <input type="checkbox" onchange="checked_in_no_session(this.checked);" name="user_with_any_session" id="user_with_any_session_id">
@@ -663,7 +663,7 @@ 
             <?php
             }
             unset($nosessionUsersList);
-           ?>
+            ?>
         </div>
     </div>
 
@@ -707,7 +707,7 @@ 
               <?php
             }
             if (!empty($addProcess)) {
-		echo '<button class="btn btn-success" type="button" value="" onclick="valide()" >'.get_lang('FinishSessionCreation').'</button>';
+        echo '<button class="btn btn-success" type="button" value="" onclick="valide()" >'.get_lang('FinishSessionCreation').'</button>';
             } else {
                 echo '<button class="btn btn-success" type="button" value="" onclick="valide()" >'.get_lang('SubscribeUsersToSession').'</button>';
             }

Spacing +37 added lines, -37 removed lines

@@ -102,13 +102,13 @@ 
         if (api_is_session_admin() &&
             api_get_setting('prevent_session_admins_to_manage_all_users') === 'true'
         ) {
-            $order_clause = " AND user.creator_id = " . api_get_user_id() . $order_clause;
+            $order_clause = " AND user.creator_id = ".api_get_user_id().$order_clause;
         }
 
         $cond_user_id = '';
 
         // Only for single & multiple
-        if (in_array($type, array('single','multiple')))
+        if (in_array($type, array('single', 'multiple')))
         if (!empty($id_session)) {
             $id_session = intval($id_session);
             // check id_user from session_rel_user table
@@ -118,11 +118,11 @@ 
             $user_ids = array();
             if (Database::num_rows($res) > 0) {
                 while ($row = Database::fetch_row($res)) {
-                    $user_ids[] = (int)$row[0];
+                    $user_ids[] = (int) $row[0];
                 }
             }
             if (count($user_ids) > 0) {
-                $cond_user_id = ' AND user.id NOT IN('.implode(",",$user_ids).')';
+                $cond_user_id = ' AND user.id NOT IN('.implode(",", $user_ids).')';
             }
         }
 
@@ -164,7 +164,7 @@ 
         }
 
         if (api_is_multiple_url_enabled()) {
-            $tbl_user_rel_access_url= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
+            $tbl_user_rel_access_url = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
             $access_url_id = api_get_current_access_url_id();
             if ($access_url_id != -1) {
                 switch ($type) {
@@ -215,10 +215,10 @@ 
 
         $rs = Database::query($sql);
         $i = 0;
-        if ($type=='single') {
+        if ($type == 'single') {
             while ($user = Database :: fetch_array($rs)) {
                 $i++;
-                if ($i<=10) {
+                if ($i <= 10) {
                     $person_name = api_get_person_name($user['firstname'], $user['lastname']).' ('.$user['username'].') '.$user['official_code'];
                     if ($showOfficialCode) {
                         $officialCode = !empty($user['official_code']) ? $user['official_code'].' - ' : '? - ';
@@ -231,7 +231,7 @@ 
                 }
             }
 
-            $xajax_response -> addAssign('ajax_list_users_single','innerHTML',api_utf8_encode($return));
+            $xajax_response -> addAssign('ajax_list_users_single', 'innerHTML', api_utf8_encode($return));
         } else {
             global $nosessionUsersList;
             $return .= '<select id="origin_users" name="nosessionUsersList[]" multiple="multiple" size="15" style="width:360px;">';
@@ -244,7 +244,7 @@ 
                 $return .= '<option value="'.$user['id'].'">'.$person_name.' </option>';
             }
             $return .= '</select>';
-            $xajax_response -> addAssign('ajax_list_users_multiple','innerHTML',api_utf8_encode($return));
+            $xajax_response -> addAssign('ajax_list_users_multiple', 'innerHTML', api_utf8_encode($return));
         }
     }
     return $xajax_response;
@@ -355,10 +355,10 @@ 
             $order_clause";
 
     if (api_is_multiple_url_enabled()) {
-        $tbl_user_rel_access_url= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
+        $tbl_user_rel_access_url = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
         $access_url_id = api_get_current_access_url_id();
         if ($access_url_id != -1) {
-            $sql="SELECT u.id, lastname, firstname, username, session_id, official_code
+            $sql = "SELECT u.id, lastname, firstname, username, session_id, official_code
             FROM $tbl_user u
             INNER JOIN $tbl_session_rel_user
                 ON $tbl_session_rel_user.user_id = u.id AND
@@ -372,7 +372,7 @@ 
     $result = Database::query($sql);
     $users = Database::store_result($result);
     foreach ($users as $user) {
-        $sessionUsersList[$user['id']] = $user ;
+        $sessionUsersList[$user['id']] = $user;
     }
 
     $sessionUserInfo = SessionManager::getTotalUserCoursesInSession($id_session);
@@ -408,12 +408,12 @@ 
                     if (isset($_POST[$varname]) && $_POST[$varname] != '0') {
                         $use_extra_fields = true;
                         if ($fieldtype == ExtraField::FIELD_TYPE_TAG) {
-                            $extra_field_result[]= UserManager::get_extra_user_data_by_tags(
+                            $extra_field_result[] = UserManager::get_extra_user_data_by_tags(
                                 intval($_POST['field_id']),
                                 $_POST[$varname]
                             );
                         } else {
-                            $extra_field_result[]= UserManager::get_extra_user_data_by_value(
+                            $extra_field_result[] = UserManager::get_extra_user_data_by_value(
                                 $new_field['variable'],
                                 $_POST[$varname]
                             );
@@ -426,10 +426,10 @@ 
 
     if ($use_extra_fields) {
         $final_result = array();
-       	if (count($extra_field_result)>1) {
-	    for($i=0;$i<count($extra_field_result)-1;$i++) {
-                if (is_array($extra_field_result[$i+1])) {
-                    $final_result  = array_intersect($extra_field_result[$i],$extra_field_result[$i+1]);
+       	if (count($extra_field_result) > 1) {
+	    for ($i = 0; $i < count($extra_field_result) - 1; $i++) {
+                if (is_array($extra_field_result[$i + 1])) {
+                    $final_result = array_intersect($extra_field_result[$i], $extra_field_result[$i + 1]);
                 }
             }
         } else {
@@ -437,14 +437,14 @@ 
         }
 
         if (api_is_multiple_url_enabled()) {
-            if (is_array($final_result) && count($final_result)>0) {
-                $where_filter = " AND u.id IN  ('".implode("','",$final_result)."') ";
+            if (is_array($final_result) && count($final_result) > 0) {
+                $where_filter = " AND u.id IN  ('".implode("','", $final_result)."') ";
             } else {
                 //no results
                 $where_filter = " AND u.id  = -1";
             }
         } else {
-            if (is_array($final_result) && count($final_result)>0) {
+            if (is_array($final_result) && count($final_result) > 0) {
                 $where_filter = " WHERE u.id IN  ('".implode("','", $final_result)."') ";
             } else {
                 //no results
@@ -453,7 +453,7 @@ 
         }
     }
     if (api_is_session_admin() && api_get_setting('prevent_session_admins_to_manage_all_users') === 'true') {
-        $order_clause = " AND u.creator_id = " . api_get_user_id() . $order_clause;
+        $order_clause = " AND u.creator_id = ".api_get_user_id().$order_clause;
     }
     if ($use_extra_fields) {
         $sql = "SELECT  u.id, lastname, firstname, username, session_id, official_code
@@ -475,7 +475,7 @@ 
                 $order_clause";
     }
     if (api_is_multiple_url_enabled()) {
-        $tbl_user_rel_access_url= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
+        $tbl_user_rel_access_url = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
         $access_url_id = api_get_current_access_url_id();
         if ($access_url_id != -1) {
             $sql = "SELECT  u.id, lastname, firstname, username, session_id, official_code
@@ -492,7 +492,7 @@ 
     }
 
     $result = Database::query($sql);
-    $users = Database::store_result($result,'ASSOC');
+    $users = Database::store_result($result, 'ASSOC');
 
     foreach ($users as $uid => $user) {
         if ($user['session_id'] != $id_session) {
@@ -501,7 +501,7 @@ 
                 'ln' => $user['lastname'],
                 'un' => $user['username'],
                 'official_code' => $user['official_code']
-            ) ;
+            );
             unset($users[$uid]);
 	}
     }
@@ -517,7 +517,7 @@ 
           WHERE u.status<>".DRH." AND u.status<>6 $order_clause";
 
     if (api_is_multiple_url_enabled()) {
-        $tbl_user_rel_access_url= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
+        $tbl_user_rel_access_url = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
         $access_url_id = api_get_current_access_url_id();
         if ($access_url_id != -1) {
             $sql = "SELECT  u.id, lastname, firstname, username, session_id, official_code
@@ -533,11 +533,11 @@ 
     }
 
     $result = Database::query($sql);
-    $users = Database::store_result($result,'ASSOC');
+    $users = Database::store_result($result, 'ASSOC');
     foreach ($users as $uid => $user) {
         if ($user['session_id'] == $id_session) {
             $sessionUsersList[$user['id']] = $user;
-            if (array_key_exists($user['id'],$nosessionUsersList)) {
+            if (array_key_exists($user['id'], $nosessionUsersList)) {
                 unset($nosessionUsersList[$user['id']]);
             }
         }
@@ -555,7 +555,7 @@ 
 	$link_add_type_multiple = '<a href="'.api_get_self().'?id_session='.$id_session.'&amp;add='.$addProcess.'&amp;add_type=multiple">'.Display::return_icon('multiple.gif').get_lang('SessionAddTypeMultiple').'</a>';
 }
 $link_add_group = Display::url(
-    Display::return_icon('multiple.gif',get_lang('RegistrationByUsersGroups')).get_lang('RegistrationByUsersGroups'),
+    Display::return_icon('multiple.gif', get_lang('RegistrationByUsersGroups')).get_lang('RegistrationByUsersGroups'),
     api_get_path(WEB_CODE_PATH).'admin/usergroups.php'
 );
 
@@ -576,12 +576,12 @@ 
     echo $newLinks;
     ?>
 </div>
-<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>?page=<?php echo $page; ?>&id_session=<?php echo $id_session; ?><?php if(!empty($addProcess)) echo '&add=true' ; ?>"  <?php if ($ajax_search) { echo ' onsubmit="valide();"';}?>>
+<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>?page=<?php echo $page; ?>&id_session=<?php echo $id_session; ?><?php if (!empty($addProcess)) echo '&add=true'; ?>"  <?php if ($ajax_search) { echo ' onsubmit="valide();"'; }?>>
 <?php echo '<legend>'.$tool_name.' ('.$session_info['name'].') </legend>'; ?>
 <?php
-if ($add_type=='multiple') {
+if ($add_type == 'multiple') {
 	if (is_array($extra_field_list)) {
-		if (is_array($new_field_list) && count($new_field_list)>0 ) {
+		if (is_array($new_field_list) && count($new_field_list) > 0) {
 			echo '<h3>'.get_lang('FilterUsers').'</h3>';
 			foreach ($new_field_list as $new_field) {
 				echo $new_field['name'];
@@ -589,8 +589,8 @@ 
                 $fieldtype = $new_field['type'];
 				echo '&nbsp;<select name="'.$varname.'">';
 				echo '<option value="0">--'.get_lang('Select').'--</option>';
-				foreach	($new_field['data'] as $option) {
-					$checked='';
+				foreach ($new_field['data'] as $option) {
+					$checked = '';
                     if ($fieldtype == ExtraField::FIELD_TYPE_TAG) {
                         if (isset($_POST[$varname])) {
                             if ($_POST[$varname] == $option['tag']) {
@@ -631,7 +631,7 @@ 
         <div class="form-group">
             <label><?php echo get_lang('UserListInPlatform') ?> </label>
             <?php
-            if (!($add_type=='multiple')) {
+            if (!($add_type == 'multiple')) {
               ?>
               <input type="text" id="user_to_add" onkeyup="xajax_search_users(this.value,'single')" class="form-control" />
               <div id="ajax_list_users_single" class="select-list-ajax"></div>
@@ -643,7 +643,7 @@ 
               <?php
               foreach ($nosessionUsersList as $uid => $enreg) {
               ?>
-                  <option value="<?php echo $uid; ?>" <?php if(in_array($uid,$UserList)) echo 'selected="selected"'; ?>>
+                  <option value="<?php echo $uid; ?>" <?php if (in_array($uid, $UserList)) echo 'selected="selected"'; ?>>
                       <?php
                       $personName = api_get_person_name($enreg['fn'], $enreg['ln']).' ('.$enreg['un'].') '.$enreg['official_code'];
                       if ($showOfficialCode) {
@@ -668,7 +668,7 @@ 
     </div>
 
     <div class="col-md-4">
-        <?php if ($add_type=='multiple') { ?>
+        <?php if ($add_type == 'multiple') { ?>
             <?php echo get_lang('FirstLetterUser'); ?> :
 
             <select id="first_letter_user" name="firstLetterUser" onchange = "change_select(this.value);" >