Completed
Push — 1.10.x ( c82817...6a93c3 )
by Yannick
202:42 queued 149:59
created
main/survey/survey_list.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
     if (api_is_allowed_to_edit()) {
76 76
         SurveyManager::copy_survey($_GET['survey_id']);
77 77
         $message = get_lang('SurveyCopied');
78
-        header('Location: ' . api_get_path(WEB_CODE_PATH) . 'survey/survey_list.php?' . api_get_cidreq());
78
+        header('Location: '.api_get_path(WEB_CODE_PATH).'survey/survey_list.php?'.api_get_cidreq());
79 79
         exit;
80 80
     }
81 81
 }
@@ -162,10 +162,10 @@  discard block
 block discarded – undo
162 162
 if (!api_is_course_coach() || $extend_rights_for_coachs == 'true') {
163 163
     // Action links
164 164
     echo '<a href="'.api_get_path(WEB_CODE_PATH).'survey/create_new_survey.php?'.api_get_cidreq().'&amp;action=add">'.
165
-        Display::return_icon('new_survey.png', get_lang('CreateNewSurvey'),'',ICON_SIZE_MEDIUM).'</a> ';
165
+        Display::return_icon('new_survey.png', get_lang('CreateNewSurvey'), '', ICON_SIZE_MEDIUM).'</a> ';
166 166
 }
167 167
 echo '<a href="'.api_get_self().'?'.api_get_cidreq().'&amp;search=advanced">'.
168
-    Display::return_icon('search.png', get_lang('Search'),'',ICON_SIZE_MEDIUM).'</a>';
168
+    Display::return_icon('search.png', get_lang('Search'), '', ICON_SIZE_MEDIUM).'</a>';
169 169
 echo '</div>';
170 170
 
171 171
 // Load main content
Please login to merge, or discard this patch.
main/survey/create_new_survey.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -216,7 +216,7 @@
 block discarded – undo
216 216
             foreach ($field_list as $key => & $field) {
217 217
                 if ($field['visibility'] == 1) {
218 218
                     $form->addElement('checkbox', 'profile_'.$key, ' ', '&nbsp;&nbsp;'.$field['name']);
219
-                    $input_name_list.= 'profile_'.$key.',';
219
+                    $input_name_list .= 'profile_'.$key.',';
220 220
                 }
221 221
             }
222 222
 
Please login to merge, or discard this patch.
main/session/session_course_user_list.php 1 patch
Spacing   +35 added lines, -35 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
 $id_session = intval($_GET['id_session']);
19 19
 SessionManager::protectSession($id_session);
20 20
 
21
-if (empty($id_session )) {
21
+if (empty($id_session)) {
22 22
     api_not_allowed();
23 23
 }
24 24
 
@@ -28,16 +28,16 @@  discard block
 block discarded – undo
28 28
 
29 29
 $page           = isset($_GET['page']) ? intval($_GET['page']) : null;
30 30
 $action         = isset($_REQUEST['action']) ? $_REQUEST['action'] : null;
31
-$default_sort   = api_sort_by_first_name() ? 'firstname':'lastname';
32
-$sort           = isset($_GET['sort']) && in_array($_GET['sort'], array('lastname','firstname','username')) ? $_GET['sort'] : $default_sort;
31
+$default_sort   = api_sort_by_first_name() ? 'firstname' : 'lastname';
32
+$sort           = isset($_GET['sort']) && in_array($_GET['sort'], array('lastname', 'firstname', 'username')) ? $_GET['sort'] : $default_sort;
33 33
 $idChecked      = isset($_GET['idChecked']) && is_array($_GET['idChecked']) ? $_GET['idChecked'] : (isset($_POST['idChecked']) && is_array($_POST['idChecked']) ? $_POST['idChecked'] : null);
34
-$direction      = isset($_GET['direction']) && in_array($_GET['direction'], array('desc','asc')) ? $_GET['direction'] : 'desc';
34
+$direction      = isset($_GET['direction']) && in_array($_GET['direction'], array('desc', 'asc')) ? $_GET['direction'] : 'desc';
35 35
 
36 36
 if (is_array($idChecked)) {
37 37
     $my_temp = array();
38 38
     foreach ($idChecked as $id) {
39 39
         // forcing the intval
40
-        $my_temp[]= intval($id);
40
+        $my_temp[] = intval($id);
41 41
     }
42 42
     $idChecked = $my_temp;
43 43
 }
@@ -49,14 +49,14 @@  discard block
 block discarded – undo
49 49
 		WHERE src.session_id='$id_session' AND src.c_id='$courseId' ";
50 50
 
51 51
 $result = Database::query($sql);
52
-if (!list($session_name,$course_title) = Database::fetch_row($result)) {
52
+if (!list($session_name, $course_title) = Database::fetch_row($result)) {
53 53
 	header('Location: session_course_list.php?id_session='.$id_session);
54 54
 	exit();
55 55
 }
56 56
 
57 57
 switch ($action) {
58 58
     case 'delete':
59
-        if (is_array($idChecked) && count($idChecked)>0) {
59
+        if (is_array($idChecked) && count($idChecked) > 0) {
60 60
             array_map('intval', $idChecked);
61 61
             $idChecked = implode(',', $idChecked);
62 62
         }
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
          ON (s.session_id = scru.session_id AND s.user_id = scru.user_id AND scru.c_id = '".$courseId."' )
92 92
          WHERE s.session_id='$id_session'
93 93
          ORDER BY $sort $direction
94
-         LIMIT $from,".($limit+1);
94
+         LIMIT $from,".($limit + 1);
95 95
 
96 96
 if ($direction == 'desc') {
97 97
     $direction = 'asc';
@@ -107,8 +107,8 @@  discard block
 block discarded – undo
107 107
 $tool_name = get_lang('Session').': '.$session_name.' - '.get_lang('Course').': '.$course_title;
108 108
 
109 109
 //$interbreadcrumb[] = array("url" => "index.php","name" => get_lang('PlatformAdmin'));
110
-$interbreadcrumb[] = array("url" => "session_list.php","name" => get_lang('SessionList'));
111
-$interbreadcrumb[] = array('url' => "resume_session.php?id_session=".$id_session,"name" => get_lang('SessionOverview'));
110
+$interbreadcrumb[] = array("url" => "session_list.php", "name" => get_lang('SessionList'));
111
+$interbreadcrumb[] = array('url' => "resume_session.php?id_session=".$id_session, "name" => get_lang('SessionOverview'));
112 112
 
113 113
 Display::display_header($tool_name);
114 114
 
@@ -117,9 +117,9 @@  discard block
 block discarded – undo
117 117
 <form method="post" action="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=<?php echo $sort; ?>" onsubmit="javascript:if(!confirm('<?php echo get_lang('ConfirmYourChoice'); ?>')) return false;">
118 118
 <div align="right">
119 119
 <?php
120
-if($page) {
120
+if ($page) {
121 121
 ?>
122
-<a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page-1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Previous');?></a>
122
+<a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page - 1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Previous'); ?></a>
123 123
 <?php
124 124
 } else {
125 125
 	echo get_lang('Previous');
@@ -127,9 +127,9 @@  discard block
 block discarded – undo
127 127
 ?>
128 128
 |
129 129
 <?php
130
-if($nbr_results > $limit) {
130
+if ($nbr_results > $limit) {
131 131
 ?>
132
-<a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page+1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Next');?></a>
132
+<a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page + 1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Next'); ?></a>
133 133
 <?php
134 134
 } else {
135 135
 	echo get_lang('Next');
@@ -141,39 +141,39 @@  discard block
 block discarded – undo
141 141
 <tr>
142 142
   <th>&nbsp;</th>
143 143
   <?php if ($is_western_name_order) { ?>
144
-  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=firstname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('FirstName');?></a></th>
145
-  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=lastname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('LastName');?></a></th>
144
+  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=firstname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('FirstName'); ?></a></th>
145
+  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=lastname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('LastName'); ?></a></th>
146 146
   <?php } else { ?>
147
-  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=lastname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('LastName');?></a></th>
148
-  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=firstname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('FirstName');?></a></th>
147
+  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=lastname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('LastName'); ?></a></th>
148
+  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=firstname&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('FirstName'); ?></a></th>
149 149
   <?php } ?>
150
-  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=username&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('Login');?></a></th>
151
-  <th><?php echo get_lang('Actions');?></th>
150
+  <th><a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=username&direction=<?php echo urlencode($direction); ?>"><?php echo get_lang('Login'); ?></a></th>
151
+  <th><?php echo get_lang('Actions'); ?></th>
152 152
 </tr>
153 153
 <?php
154
-$i=0;
154
+$i = 0;
155 155
 foreach ($users as $key => $enreg) {
156 156
 
157 157
 	if ($key == $limit) {
158 158
 		break;
159 159
 	}
160 160
     ?>
161
-    <tr class="<?php echo $i?'row_odd':'row_even'; ?>">
161
+    <tr class="<?php echo $i ? 'row_odd' : 'row_even'; ?>">
162 162
         <td><input type="checkbox" name="idChecked[]" value="<?php echo $enreg['user_id']; ?>"></td>
163 163
         <?php if ($is_western_name_order) { ?>
164
-        <td><?php echo api_htmlentities($enreg['firstname'],ENT_QUOTES,$charset); ?></td>
165
-        <td><?php echo api_htmlentities($enreg['lastname'],ENT_QUOTES,$charset); ?></td>
164
+        <td><?php echo api_htmlentities($enreg['firstname'], ENT_QUOTES, $charset); ?></td>
165
+        <td><?php echo api_htmlentities($enreg['lastname'], ENT_QUOTES, $charset); ?></td>
166 166
         <?php } else { ?>
167
-        <td><?php echo api_htmlentities($enreg['lastname'],ENT_QUOTES,$charset); ?></td>
168
-        <td><?php echo api_htmlentities($enreg['firstname'],ENT_QUOTES,$charset); ?></td>
167
+        <td><?php echo api_htmlentities($enreg['lastname'], ENT_QUOTES, $charset); ?></td>
168
+        <td><?php echo api_htmlentities($enreg['firstname'], ENT_QUOTES, $charset); ?></td>
169 169
         <?php } ?>
170
-        <td><?php echo api_htmlentities($enreg['username'],ENT_QUOTES,$charset); ?></td>
170
+        <td><?php echo api_htmlentities($enreg['username'], ENT_QUOTES, $charset); ?></td>
171 171
         <td>
172 172
         <?php if ($enreg['is_subscribed']) { ?>
173 173
             <a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=<?php echo $sort; ?>&action=delete&idChecked[]=<?php echo $enreg['user_id']; ?>" onclick="javascript:if(!confirm('<?php echo get_lang('ConfirmYourChoice'); ?>')) return false;">
174 174
                 <?php Display::display_icon('delete.png', get_lang('Delete')); ?>
175 175
             </a>
176
-        <?php } else  { ?>
176
+        <?php } else { ?>
177 177
             <a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&sort=<?php echo $sort; ?>&action=add&idChecked[]=<?php echo $enreg['user_id']; ?>" onclick="javascript:if(!confirm('<?php echo get_lang('ConfirmYourChoice'); ?>')) return false;">
178 178
                 <?php Display::display_icon('add.png', get_lang('Add'), array(), ICON_SIZE_SMALL); ?>
179 179
             </a>
@@ -182,7 +182,7 @@  discard block
 block discarded – undo
182 182
         </td>
183 183
     </tr>
184 184
     <?php
185
-        $i=$i ? 0 : 1;
185
+        $i = $i ? 0 : 1;
186 186
 }
187 187
 unset($users);
188 188
 ?>
@@ -190,9 +190,9 @@  discard block
 block discarded – undo
190 190
 <br />
191 191
 <div align="left">
192 192
     <?php
193
-    if($page) {
193
+    if ($page) {
194 194
     ?>
195
-    <a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page-1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Previous'); ?></a>
195
+    <a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page - 1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Previous'); ?></a>
196 196
     <?php
197 197
     } else {
198 198
         echo get_lang('Previous');
@@ -200,9 +200,9 @@  discard block
 block discarded – undo
200 200
     ?>
201 201
     |
202 202
     <?php
203
-    if($nbr_results > $limit) {
203
+    if ($nbr_results > $limit) {
204 204
     ?>
205
-    <a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page+1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Next'); ?></a>
205
+    <a href="<?php echo api_get_self(); ?>?id_session=<?php echo $id_session; ?>&course_code=<?php echo urlencode($course_code); ?>&page=<?php echo $page + 1; ?>&sort=<?php echo $sort; ?>"><?php echo get_lang('Next'); ?></a>
206 206
     <?php
207 207
     } else {
208 208
         echo get_lang('Next');
@@ -211,8 +211,8 @@  discard block
 block discarded – undo
211 211
 </div>
212 212
 <br />
213 213
 <select name="action">
214
-<option value="delete"><?php echo get_lang('UnsubscribeSelectedUsersFromSession');?></option>
215
-<option value="add"><?php echo get_lang('AddUsers');?></option>
214
+<option value="delete"><?php echo get_lang('UnsubscribeSelectedUsersFromSession'); ?></option>
215
+<option value="add"><?php echo get_lang('AddUsers'); ?></option>
216 216
 </select>
217 217
 <button class="save" type="submit"> <?php echo get_lang('Ok'); ?></button>
218 218
 </form>
Please login to merge, or discard this patch.
main/session/session_course_user.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
 SessionManager::protectSession($id_session);
17 17
 
18 18
 // setting breadcrumbs
19
-$interbreadcrumb[] = array('url' => 'session_list.php','name' => get_lang('SessionList'));
19
+$interbreadcrumb[] = array('url' => 'session_list.php', 'name' => get_lang('SessionList'));
20 20
 $interbreadcrumb[] = array(
21 21
     'url' => 'resume_session.php?id_session='.$id_session,
22 22
     'name' => get_lang('SessionOverview'),
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
             FROM '.Database :: get_main_table(TABLE_MAIN_SESSION).'
44 44
             WHERE id='.$id_session;
45 45
     $rs = Database::query($sql);
46
-    if (Database::result($rs,0,0)!=$_user['user_id']) {
46
+    if (Database::result($rs, 0, 0) != $_user['user_id']) {
47 47
         api_not_allowed(true);
48 48
     }
49 49
 }
@@ -88,9 +88,9 @@  discard block
 block discarded – undo
88 88
 
89 89
     foreach ($CourseList as $enreg_course) {
90 90
         $exists = false;
91
-        foreach($existingCourses as $existingCourse) {
91
+        foreach ($existingCourses as $existingCourse) {
92 92
             if ($enreg_course == $existingCourse['id']) {
93
-                $exists=true;
93
+                $exists = true;
94 94
             }
95 95
         }
96 96
 
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
     }
111 111
 
112 112
     foreach ($existingCourses as $existingCourse) {
113
-        if (!in_array($existingCourse['id'], $CourseList)){
113
+        if (!in_array($existingCourse['id'], $CourseList)) {
114 114
             $existingCourse = Database::escape_string($existingCourse['id']);
115 115
             $sql = "INSERT IGNORE INTO $tbl_session_rel_course_rel_user (session_id,c_id,user_id)
116 116
                     VALUES ('$id_session','$existingCourse','$id_user')";
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
         WHERE srcru.user_id = $id_user AND session_id = $id_session";
147 147
 
148 148
 //all
149
-$sql_all="SELECT course.id, code, title, visual_code, src.session_id
149
+$sql_all = "SELECT course.id, code, title, visual_code, src.session_id
150 150
         FROM $tbl_course course
151 151
         INNER JOIN $tbl_session_rel_course  as src
152 152
         ON course.id = src.c_id AND session_id = $id_session";
@@ -163,9 +163,9 @@  discard block
 block discarded – undo
163 163
 
164 164
 foreach ($CoursesAll as $course) {
165 165
     if (in_array($course['id'], $course_temp)) {
166
-        $nosessionCourses[$course['id']] = $course ;
166
+        $nosessionCourses[$course['id']] = $course;
167 167
     } else {
168
-        $sessionCourses[$course['id']] = $course ;
168
+        $sessionCourses[$course['id']] = $course;
169 169
     }
170 170
 }
171 171
 unset($Courses);
@@ -184,9 +184,9 @@  discard block
 block discarded – undo
184 184
             <td width="45%" align="center">
185 185
                 <div id="ajax_list_courses_multiple">
186 186
                     <select id="origin" name="NoSessionCoursesList[]" multiple="multiple" size="20" style="width:320px;"> <?php
187
-                        foreach($nosessionCourses as $enreg) {
187
+                        foreach ($nosessionCourses as $enreg) {
188 188
                             ?>
189
-                            <option value="<?php echo $enreg['id']; ?>" <?php echo 'title="'.htmlspecialchars($enreg['title'].' ('.$enreg['visual_code'].')',ENT_QUOTES).'"'; if(in_array($enreg['code'],$CourseList)) echo 'selected="selected"'; ?>><?php echo $enreg['title'].' ('.$enreg['visual_code'].')'; ?></option>
189
+                            <option value="<?php echo $enreg['id']; ?>" <?php echo 'title="'.htmlspecialchars($enreg['title'].' ('.$enreg['visual_code'].')', ENT_QUOTES).'"'; if (in_array($enreg['code'], $CourseList)) echo 'selected="selected"'; ?>><?php echo $enreg['title'].' ('.$enreg['visual_code'].')'; ?></option>
190 190
                         <?php
191 191
                         }
192 192
                         ?> </select></div> <?php
@@ -211,9 +211,9 @@  discard block
 block discarded – undo
211 211
             <td width="45%" align="center">
212 212
                 <select id='destination' name="SessionCoursesList[]" multiple="multiple" size="20" style="width:320px;">
213 213
             <?php
214
-            foreach($sessionCourses as $enreg) {
214
+            foreach ($sessionCourses as $enreg) {
215 215
                 ?>
216
-                <option value="<?php echo $enreg['id']; ?>" title="<?php echo htmlspecialchars($enreg['title'].' ('.$enreg['visual_code'].')',ENT_QUOTES); ?>"><?php echo $enreg['title'].' ('.$enreg['visual_code'].')'; ?></option>
216
+                <option value="<?php echo $enreg['id']; ?>" title="<?php echo htmlspecialchars($enreg['title'].' ('.$enreg['visual_code'].')', ENT_QUOTES); ?>"><?php echo $enreg['title'].' ('.$enreg['visual_code'].')'; ?></option>
217 217
             <?php
218 218
             }
219 219
             unset($sessionCourses);
Please login to merge, or discard this patch.
main/session/session_course_list.php 1 patch
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@  discard block
 block discarded – undo
8 8
 require_once '../inc/global.inc.php';
9 9
 
10 10
 // setting the section (for the tabs)
11
-$this_section=SECTION_PLATFORM_ADMIN;
11
+$this_section = SECTION_PLATFORM_ADMIN;
12 12
 
13 13
 $id_session = intval($_GET['id_session']);
14 14
 SessionManager::protectSession($id_session);
@@ -16,8 +16,8 @@  discard block
 block discarded – undo
16 16
 // Database Table Definitions
17 17
 $tbl_course							= Database::get_main_table(TABLE_MAIN_COURSE);
18 18
 $tbl_session						= Database::get_main_table(TABLE_MAIN_SESSION);
19
-$tbl_session_rel_course				= Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
20
-$tbl_session_rel_course_rel_user	= Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
19
+$tbl_session_rel_course = Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
20
+$tbl_session_rel_course_rel_user = Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
21 21
 
22 22
 if (empty($id_session)) {
23 23
     api_not_allowed();
@@ -25,26 +25,26 @@  discard block
 block discarded – undo
25 25
 
26 26
 $page   = intval($_GET['page']);
27 27
 $action = $_REQUEST['action'];
28
-$sort   = in_array($_GET['sort'],array('title','nbr_users'))?$_GET['sort']:'title';
28
+$sort   = in_array($_GET['sort'], array('title', 'nbr_users')) ? $_GET['sort'] : 'title';
29 29
 
30 30
 $result = Database::query("SELECT name FROM $tbl_session WHERE id='$id_session'");
31 31
 
32
-if (!list($session_name)=Database::fetch_row($result)) {
32
+if (!list($session_name) = Database::fetch_row($result)) {
33 33
 	header('Location: session_list.php');
34 34
 	exit;
35 35
 }
36 36
 
37 37
 if ($action == 'delete') {
38 38
 	$idChecked = $_REQUEST['idChecked'];
39
-	if (is_array($idChecked) && count($idChecked)>0) {
39
+	if (is_array($idChecked) && count($idChecked) > 0) {
40 40
 		$my_temp = array();
41
-		foreach ($idChecked as $id){
42
-			$my_temp[]= Database::escape_string($id);// forcing the escape_string
41
+		foreach ($idChecked as $id) {
42
+			$my_temp[] = Database::escape_string($id); // forcing the escape_string
43 43
 		}
44 44
 		$idChecked = $my_temp;
45
-		$idChecked="'".implode("','", $idChecked)."'";
45
+		$idChecked = "'".implode("','", $idChecked)."'";
46 46
 		$result = Database::query("DELETE FROM $tbl_session_rel_course WHERE session_id='$id_session' AND c_id IN($idChecked)");
47
-		$nbr_affected_rows=Database::affected_rows($result);
47
+		$nbr_affected_rows = Database::affected_rows($result);
48 48
 		Database::query("DELETE FROM $tbl_session_rel_course_rel_user WHERE session_id='$id_session' AND c_id IN($idChecked)");
49 49
 		Database::query("UPDATE $tbl_session SET nbr_courses=nbr_courses-$nbr_affected_rows WHERE id='$id_session'");
50 50
 	}
@@ -59,14 +59,14 @@  discard block
 block discarded – undo
59 59
 		FROM $tbl_session_rel_course, $tbl_course c
60 60
 		WHERE c_id = c.id AND session_id='$id_session'
61 61
 		ORDER BY $sort
62
-		LIMIT $from,".($limit+1);
63
-$result=Database::query($sql);
64
-$Courses=Database::store_result($result);
65
-$tool_name = api_htmlentities($session_name,ENT_QUOTES,$charset).' : '.get_lang('CourseListInSession');
62
+		LIMIT $from,".($limit + 1);
63
+$result = Database::query($sql);
64
+$Courses = Database::store_result($result);
65
+$tool_name = api_htmlentities($session_name, ENT_QUOTES, $charset).' : '.get_lang('CourseListInSession');
66 66
 
67 67
 //$interbreadcrumb[]=array('url' => 'index.php',"name" => get_lang('PlatformAdmin'));
68
-$interbreadcrumb[] = array('url' => "session_list.php","name" => get_lang('SessionList'));
69
-$interbreadcrumb[] = array('url' => "resume_session.php?id_session=".Security::remove_XSS($_REQUEST['id_session']),"name" => get_lang('SessionOverview'));
68
+$interbreadcrumb[] = array('url' => "session_list.php", "name" => get_lang('SessionList'));
69
+$interbreadcrumb[] = array('url' => "resume_session.php?id_session=".Security::remove_XSS($_REQUEST['id_session']), "name" => get_lang('SessionOverview'));
70 70
 
71 71
 Display::display_header($tool_name);
72 72
 echo Display::page_header($tool_name);
@@ -84,18 +84,18 @@  discard block
 block discarded – undo
84 84
 foreach ($Courses as $key=>$enreg) {
85 85
 	$course = array();
86 86
 	$course[] = '<input type="checkbox" name="idChecked[]" value="'.$enreg['id'].'">';
87
-	$course[] = api_htmlentities($enreg['title'],ENT_QUOTES,$charset);
87
+	$course[] = api_htmlentities($enreg['title'], ENT_QUOTES, $charset);
88 88
 	$course[] = '<a href="session_course_user_list.php?id_session='.$id_session.'&course_code='.$enreg['code'].'">'.$enreg['nbr_users'].' '.get_lang('Users').'</a>';
89 89
 	$course[] = '<a href="'.api_get_path(WEB_COURSE_PATH).$enreg['code'].'/?id_session='.$id_session.'">'.
90 90
 		Display::return_icon('course_home.gif', get_lang('Course')).'</a>
91 91
 			<a href="session_course_edit.php?id_session='.$id_session.'&page=session_course_list.php&course_code='.$enreg['code'].'">'.
92 92
 		Display::return_icon('edit.png', get_lang('Edit')).'</a>
93
-			<a href="'.api_get_self().'?id_session='.$id_session.'&sort='.$sort.'&action=delete&idChecked[]='.$enreg['id'].'" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"),ENT_QUOTES,$charset)).'\')) return false;">'.
93
+			<a href="'.api_get_self().'?id_session='.$id_session.'&sort='.$sort.'&action=delete&idChecked[]='.$enreg['id'].'" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"), ENT_QUOTES, $charset)).'\')) return false;">'.
94 94
 		Display::return_icon('delete.png', get_lang('Delete')).'</a>';
95 95
 	$tableCourses[] = $course;
96 96
 }
97 97
 echo '<form method="post" action="'.api_get_self().'">';
98
-Display :: display_sortable_table($tableHeader, $tableCourses, array (), array ());
98
+Display :: display_sortable_table($tableHeader, $tableCourses, array(), array());
99 99
 echo '<select name="action">
100 100
 	<option value="delete">'.get_lang('UnsubscribeCoursesFromSession').'</option>
101 101
 	</select>
Please login to merge, or discard this patch.
main/session/session_category_list.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
 }
44 44
 
45 45
 //$interbreadcrumb[] = array("url" => "index.php", "name" => get_lang('PlatformAdmin'));
46
-$interbreadcrumb[] = array('url' => 'session_list.php','name' => get_lang('SessionList'));
46
+$interbreadcrumb[] = array('url' => 'session_list.php', 'name' => get_lang('SessionList'));
47 47
 
48 48
 if (isset($_GET['search']) && $_GET['search'] == 'advanced') {
49 49
     $interbreadcrumb[] = array("url" => 'session_category_list.php', "name" => get_lang('ListSessionCategory'));
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
         $where .= " AND access_url_id = ".api_get_current_access_url_id()." ";
77 77
     }
78 78
 
79
-    $table_access_url_rel_session= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_SESSION);
79
+    $table_access_url_rel_session = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_SESSION);
80 80
     $query = "SELECT sc.*, (
81 81
                 SELECT count(id) FROM $tbl_session s
82 82
                 INNER JOIN $table_access_url_rel_session us
Please login to merge, or discard this patch.
main/session/add_students_to_session.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -14,12 +14,12 @@
 block discarded – undo
14 14
 
15 15
 // setting breadcrumbs
16 16
 //$interbreadcrumb[] = array('url' => 'index.php','name' => get_lang('PlatformAdmin'));
17
-$interbreadcrumb[] = array('url' => 'session_list.php','name' => get_lang('SessionList'));
17
+$interbreadcrumb[] = array('url' => 'session_list.php', 'name' => get_lang('SessionList'));
18 18
 
19 19
 // Setting the name of the tool
20 20
 $tool_name = get_lang('SubscribeStudentsToSession');
21 21
 $add_type = 'multiple';
22
-if (isset($_REQUEST['add_type']) && $_REQUEST['add_type']!='') {
22
+if (isset($_REQUEST['add_type']) && $_REQUEST['add_type'] != '') {
23 23
     $add_type = Security::remove_XSS($_REQUEST['add_type']);
24 24
 }
25 25
 $form_sent  = 0;
Please login to merge, or discard this patch.
main/session/index.php 1 patch
Spacing   +40 added lines, -40 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
     api_not_allowed();
17 17
 }
18 18
 
19
-$session_id = isset($_GET['session_id']) ? intval($_GET['session_id']): null;
19
+$session_id = isset($_GET['session_id']) ? intval($_GET['session_id']) : null;
20 20
 
21 21
 $sessionField = new ExtraFieldValue('session');
22 22
 $valueAllowVisitors = $sessionField->get_values_by_handler_and_field_variable($session_id, 'allow_visitors');
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 $this_section = SECTION_COURSES;
32 32
 $htmlHeadXtra[] = api_get_jqgrid_js();
33 33
 
34
-$course_id  = isset($_GET['course_id'])  ? intval($_GET['course_id']) : null;
34
+$course_id = isset($_GET['course_id']) ? intval($_GET['course_id']) : null;
35 35
 
36 36
 $_SESSION['id_session'] = $session_id;
37 37
 
@@ -245,12 +245,12 @@  discard block
 block discarded – undo
245 245
     $sessionCourses = $session->getCourses();
246 246
     $sessionCourse = $sessionCourses[0]->getCourse();
247 247
 
248
-    $courseUrl = $sessionCourse->getDirectory() . '/index.php?';
248
+    $courseUrl = $sessionCourse->getDirectory().'/index.php?';
249 249
     $courseUrl .= http_build_query([
250 250
         'id_session' => $session->getId()
251 251
     ]);
252 252
 
253
-    header('Location: ' . api_get_path(WEB_COURSE_PATH) . $courseUrl);
253
+    header('Location: '.api_get_path(WEB_COURSE_PATH).$courseUrl);
254 254
     exit;
255 255
 }
256 256
 
@@ -258,7 +258,7 @@  discard block
 block discarded – undo
258 258
 
259 259
 $session_select = array();
260 260
 foreach ($session_list as $item) {
261
-    $session_select[$item['id']] =  $item['name'];
261
+    $session_select[$item['id']] = $item['name'];
262 262
 }
263 263
 
264 264
 // Session list form
@@ -280,7 +280,7 @@  discard block
 block discarded – undo
280 280
 $my_real_array = $new_exercises = array();
281 281
 $now = time();
282 282
 foreach ($final_array as $session_data) {
283
-    $my_course_list = isset($session_data['data']) ? $session_data['data']: array();
283
+    $my_course_list = isset($session_data['data']) ? $session_data['data'] : array();
284 284
     if (!empty($my_course_list)) {
285 285
         foreach ($my_course_list as $my_course_code=>$course_data) {
286 286
             $courseInfo = api_get_course_info($my_course_code);
@@ -318,8 +318,8 @@  discard block
 block discarded – undo
318 318
                             foreach ($result_list as $exercise_result) {
319 319
                                 $platform_score = ExerciseLib::show_score($exercise_result['exe_result'], $exercise_result['exe_weighting']);
320 320
                                 $my_score = 0;
321
-                                if(!empty($exercise_result['exe_weighting']) && intval($exercise_result['exe_weighting']) != 0) {
322
-                                    $my_score = $exercise_result['exe_result']/$exercise_result['exe_weighting'];
321
+                                if (!empty($exercise_result['exe_weighting']) && intval($exercise_result['exe_weighting']) != 0) {
322
+                                    $my_score = $exercise_result['exe_result'] / $exercise_result['exe_weighting'];
323 323
                                 }
324 324
                                 $position = ExerciseLib::get_exercise_result_ranking(
325 325
                                     $my_score,
@@ -333,11 +333,11 @@  discard block
 block discarded – undo
333 333
                                 $exercise_info->exercise = Display::url(
334 334
                                     $exercise_info->exercise,
335 335
                                     api_get_path(WEB_CODE_PATH)."exercice/result.php?cidReq=$my_course_code&id={$exercise_result['exe_id']}&id_session=$session_id&show_headers=1",
336
-                                    array('target'=>SESSION_LINK_TARGET,'class'=>'exercise-result-link')
336
+                                    array('target'=>SESSION_LINK_TARGET, 'class'=>'exercise-result-link')
337 337
                                 );
338 338
 
339
-                                $my_real_array[]= array(
340
-                                    'status'      => Display::return_icon('quiz.gif', get_lang('Attempted'),'', ICON_SIZE_SMALL),
339
+                                $my_real_array[] = array(
340
+                                    'status'      => Display::return_icon('quiz.gif', get_lang('Attempted'), '', ICON_SIZE_SMALL),
341 341
                                     'date'        => $start_date,
342 342
                                     'course'      => $course_data['name'],
343 343
                                     'exercise'    => $exercise_info->exercise,
@@ -362,7 +362,7 @@  discard block
 block discarded – undo
362 362
                                 array('target'=>SESSION_LINK_TARGET)
363 363
                             );
364 364
 
365
-                            $new_exercises[]= array(
365
+                            $new_exercises[] = array(
366 366
                                 'status'      => Display::return_icon('star.png', get_lang('New'), array('width'=>ICON_SIZE_SMALL)),
367 367
                                 'date'        => $start_date,
368 368
                                 'course'      => $course_data['name'],
@@ -386,7 +386,7 @@  discard block
 block discarded – undo
386 386
     $my_real_array = array_merge($new_exercises, $my_real_array);
387 387
 }
388 388
 
389
-$start = $end = $start_only = $end_only ='';
389
+$start = $end = $start_only = $end_only = '';
390 390
 
391 391
 if (!empty($session_info['access_start_date']) && $session_info['access_start_date'] != '0000-00-00') {
392 392
     $start = api_convert_and_format_date($session_info['access_start_date'], DATE_FORMAT_SHORT);
@@ -424,11 +424,11 @@  discard block
 block discarded – undo
424 424
     get_lang('Title'), get_lang('NumberOfPublishedExercises'), get_lang('NumberOfPublishedLps')
425 425
 );
426 426
 $column_model_courses = array(
427
-    array('name'=>'title',              'index'=>'title',               'width'=>'400px',  'align'=>'left',  'sortable'=>'true'),
427
+    array('name'=>'title', 'index'=>'title', 'width'=>'400px', 'align'=>'left', 'sortable'=>'true'),
428 428
     //array('name'=>'recent_lps',         'index'=>'recent_lps',          'width'=>'10px',  'align'=>'left',  'sortable'=>'false'),
429 429
 //    array('name'=>'max_mutation_date',  'index'=>'max_mutation_date',   'width'=>'120px',  'align'=>'left',  'sortable'=>'true'),
430
-    array('name'=>'exercise_count',     'index'=>'exercise_count',      'width'=>'180px',  'align'=>'left',  'sortable'=>'true'),
431
-    array('name'=>'lp_count',           'index'=>'lp_count',            'width'=>'180px',  'align'=>'left',  'sortable'=>'true')
430
+    array('name'=>'exercise_count', 'index'=>'exercise_count', 'width'=>'180px', 'align'=>'left', 'sortable'=>'true'),
431
+    array('name'=>'lp_count', 'index'=>'lp_count', 'width'=>'180px', 'align'=>'left', 'sortable'=>'true')
432 432
 );
433 433
 
434 434
 $extra_params_courses['height'] = '100%';
@@ -452,10 +452,10 @@  discard block
 block discarded – undo
452 452
     get_lang('LearningPaths')
453 453
 );
454 454
 
455
-$column_model   = array(
456
-    array('name'=>'date',   'index'=>'date',   'width'=>'120', 'align'=>'left', 'sortable'=>'true'),
455
+$column_model = array(
456
+    array('name'=>'date', 'index'=>'date', 'width'=>'120', 'align'=>'left', 'sortable'=>'true'),
457 457
     array('name'=>'course', 'index'=>'course', 'width'=>'300', 'align'=>'left', 'sortable'=>'true', 'wrap_cell' => 'true'),
458
-    array('name'=>'lp',     'index'=>'lp',     'width'=>'440', 'align'=>'left', 'sortable'=>'true')
458
+    array('name'=>'lp', 'index'=>'lp', 'width'=>'440', 'align'=>'left', 'sortable'=>'true')
459 459
 );
460 460
 
461 461
 $extra_params = array();
@@ -501,11 +501,11 @@  discard block
 block discarded – undo
501 501
     get_lang('LearningPaths')
502 502
 );
503 503
 
504
-$column_week_model  = array(
505
-    array('name'=>'week',     'index'=>'week',    'width'=>'40',  'align'=>'left', 'sortable'=>'false'),
506
-    array('name'=>'date',     'index'=>'date',    'width'=>'120', 'align'=>'left', 'sortable'=>'false'),
507
-    array('name'=>'course',   'index'=>'course',  'width'=>'300', 'align'=>'left', 'sortable'=>'true', 'wrap_cell' => 'true'),
508
-    array('name'=>'lp',       'index'=>'lp',      'width'=>'440', 'align'=>'left', 'sortable'=>'true')
504
+$column_week_model = array(
505
+    array('name'=>'week', 'index'=>'week', 'width'=>'40', 'align'=>'left', 'sortable'=>'false'),
506
+    array('name'=>'date', 'index'=>'date', 'width'=>'120', 'align'=>'left', 'sortable'=>'false'),
507
+    array('name'=>'course', 'index'=>'course', 'width'=>'300', 'align'=>'left', 'sortable'=>'true', 'wrap_cell' => 'true'),
508
+    array('name'=>'lp', 'index'=>'lp', 'width'=>'440', 'align'=>'left', 'sortable'=>'true')
509 509
 );
510 510
 
511 511
 $extra_params_week = array();
@@ -517,7 +517,7 @@  discard block
 block discarded – undo
517 517
     'groupField' => array('week'),
518 518
     'groupOrder' => array('desc'),
519 519
     'groupColumnShow' => 'false',
520
-    'groupText' => array('<b>' . get_lang('PeriodWeek') . ' {0}</b>')
520
+    'groupText' => array('<b>'.get_lang('PeriodWeek').' {0}</b>')
521 521
 );
522 522
 $extra_params_week['autowidth'] = 'true'; //use the width of the parent
523 523
 $extra_params_week['height'] = '100%';
@@ -534,15 +534,15 @@  discard block
 block discarded – undo
534 534
         get_lang('BestResultInCourse'),
535 535
         get_lang('Ranking')
536 536
     );
537
-    $column_exercise_model  = array(
538
-        array('name'=>'status',     'index'=>'status',     'width'=>'40', 'align'=>'left',   'sortable'=>'false'),
539
-        array('name'=>'date',       'index'=>'date',       'width'=>'130','align'=>'left',   'sortable'=>'true'),
540
-        array('name'=>'course',     'index'=>'course',     'width'=>'200','align'=>'left',   'sortable'=>'true', 'wrap_cell' => 'true'),
541
-        array('name'=>'exercise',   'index'=>'exercise',   'width'=>'200','align'=>'left',   'sortable'=>'false'),
542
-        array('name'=>'attempt',    'index'=>'attempt',    'width'=>'60', 'align'=>'center', 'sortable'=>'true'),
543
-        array('name'=>'result',     'index'=>'result',     'width'=>'120','align'=>'center', 'sortable'=>'true'),
544
-        array('name'=>'best_result','index'=>'best_result','width'=>'140','align'=>'center', 'sortable'=>'true'),
545
-        array('name'=>'position',   'index'=>'position',   'width'=>'55', 'align'=>'center', 'sortable'=>'true')
537
+    $column_exercise_model = array(
538
+        array('name'=>'status', 'index'=>'status', 'width'=>'40', 'align'=>'left', 'sortable'=>'false'),
539
+        array('name'=>'date', 'index'=>'date', 'width'=>'130', 'align'=>'left', 'sortable'=>'true'),
540
+        array('name'=>'course', 'index'=>'course', 'width'=>'200', 'align'=>'left', 'sortable'=>'true', 'wrap_cell' => 'true'),
541
+        array('name'=>'exercise', 'index'=>'exercise', 'width'=>'200', 'align'=>'left', 'sortable'=>'false'),
542
+        array('name'=>'attempt', 'index'=>'attempt', 'width'=>'60', 'align'=>'center', 'sortable'=>'true'),
543
+        array('name'=>'result', 'index'=>'result', 'width'=>'120', 'align'=>'center', 'sortable'=>'true'),
544
+        array('name'=>'best_result', 'index'=>'best_result', 'width'=>'140', 'align'=>'center', 'sortable'=>'true'),
545
+        array('name'=>'position', 'index'=>'position', 'width'=>'55', 'align'=>'center', 'sortable'=>'true')
546 546
     );
547 547
     $extra_params_exercise['height'] = '100%';
548 548
     $extra_params_exercise['autowidth'] = 'true';
@@ -584,10 +584,10 @@  discard block
 block discarded – undo
584 584
     });
585 585
 <?php
586 586
      //Displays js code to use a jqgrid
587
-     echo Display::grid_js('courses',       '',             $columns_courses, $column_model_courses, $extra_params_courses, $new_course_list);
588
-     echo Display::grid_js('list_default',  $url,           $columns,         $column_model,$extra_params,array(), '');
589
-     echo Display::grid_js('list_course',   $url_by_course, $columns,         $column_model,$extra_params_course,array(),'');
590
-     echo Display::grid_js('list_week',     $url_week,      $column_week,     $column_week_model, $extra_params_week,array(),'');
587
+     echo Display::grid_js('courses', '', $columns_courses, $column_model_courses, $extra_params_courses, $new_course_list);
588
+     echo Display::grid_js('list_default', $url, $columns, $column_model, $extra_params, array(), '');
589
+     echo Display::grid_js('list_course', $url_by_course, $columns, $column_model, $extra_params_course, array(), '');
590
+     echo Display::grid_js('list_week', $url_week, $column_week, $column_week_model, $extra_params_week, array(), '');
591 591
 
592 592
     if (!api_is_anonymous()) {
593 593
         echo Display::grid_js('exercises', '', $column_exercise, $column_exercise_model, $extra_params_exercise, $my_real_array);
@@ -608,7 +608,7 @@  discard block
 block discarded – undo
608 608
         false,
609 609
         false
610 610
     );
611
-    if (!empty($reportingTab))  {
611
+    if (!empty($reportingTab)) {
612 612
         $reportingTab .= '<br />'.Tracking::show_course_detail(
613 613
                 api_get_user_id(),
614 614
                 $courseCode,
@@ -616,7 +616,7 @@  discard block
 block discarded – undo
616 616
             );
617 617
     }
618 618
     if (empty($reportingTab)) {
619
-        $reportingTab  = Display::return_message(get_lang('NoDataAvailable'), 'warning');
619
+        $reportingTab = Display::return_message(get_lang('NoDataAvailable'), 'warning');
620 620
     }
621 621
 }
622 622
 
Please login to merge, or discard this patch.
main/session/session_import_drh.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
 $tool_name = get_lang('ImportSessionDrhList');
17 17
 
18 18
 //$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
19
-$interbreadcrumb[]=array('url' => 'session_list.php','name' => get_lang('SessionList'));
19
+$interbreadcrumb[] = array('url' => 'session_list.php', 'name' => get_lang('SessionList'));
20 20
 
21 21
 set_time_limit(0);
22 22
 
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
 
28 28
 echo '<div class="actions">';
29 29
 echo '<a href="../session/session_list.php">'.
30
-    Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('PlatformAdmin'),'',ICON_SIZE_MEDIUM).'</a>';
30
+    Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('PlatformAdmin'), '', ICON_SIZE_MEDIUM).'</a>';
31 31
 echo '</div>';
32 32
 
33 33
 if (!empty($error_message)) {
Please login to merge, or discard this patch.