Completed
Pull Request — 1.11.x (#1628)
by José
97:30 queued 69:06
created
main/session/session_edit.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -56,8 +56,8 @@  discard block
 block discarded – undo
56 56
 $tool_name = get_lang('EditSession');
57 57
 
58 58
 //$interbreadcrumb[] = array('url' => 'index.php',"name" => get_lang('PlatformAdmin'));
59
-$interbreadcrumb[] = array('url' => "session_list.php","name" => get_lang('SessionList'));
60
-$interbreadcrumb[] = array('url' => "resume_session.php?id_session=".$id,"name" => get_lang('SessionOverview'));
59
+$interbreadcrumb[] = array('url' => "session_list.php", "name" => get_lang('SessionList'));
60
+$interbreadcrumb[] = array('url' => "resume_session.php?id_session=".$id, "name" => get_lang('SessionOverview'));
61 61
 
62 62
 if (isset($_POST['formSent']) && $_POST['formSent']) {
63 63
 	$formSent = 1;
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
         WHERE status='1'".$order_clause;
72 72
 
73 73
 if (api_is_multiple_url_enabled()) {
74
-	$table_access_url_rel_user= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
74
+	$table_access_url_rel_user = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
75 75
 	$access_url_id = api_get_current_access_url_id();
76 76
 	if ($access_url_id != -1) {
77 77
 		$sql = "SELECT DISTINCT u.user_id,lastname,firstname,username
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
 $thisYear = date('Y');
88 88
 
89 89
 $coachesOption = array(
90
-    '' => '----- ' . get_lang('None') . ' -----'
90
+    '' => '----- '.get_lang('None').' -----'
91 91
 );
92 92
 
93 93
 foreach ($coaches as $coach) {
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
     }
108 108
 }
109 109
 
110
-$formAction = api_get_self() . '?';
110
+$formAction = api_get_self().'?';
111 111
 $formAction .= http_build_query(array(
112 112
     'page' => Security::remove_XSS($_GET['page']),
113 113
     'id' => $id
@@ -158,7 +158,7 @@  discard block
 block discarded – undo
158 158
         $duration = null;
159 159
     }
160 160
     $description = $params['description'];
161
-    $showDescription = isset($params['show_description']) ? 1: 0;
161
+    $showDescription = isset($params['show_description']) ? 1 : 0;
162 162
     $sendSubscriptionNotification = isset($params['send_subscription_notification']);
163 163
     $isThisImageCropped = isset($params['picture_crop_result']);
164 164
 
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
 
196 196
     if ($return) {
197 197
         Display::addFlash(Display::return_message(get_lang('Updated')));
198
-        header('Location: resume_session.php?id_session=' . $return);
198
+        header('Location: resume_session.php?id_session='.$return);
199 199
         exit();
200 200
     }
201 201
 }
Please login to merge, or discard this patch.
main/session/session_add.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
 				' LIMIT 10';
52 52
 
53 53
 		if (api_is_multiple_url_enabled()) {
54
-			$tbl_user_rel_access_url= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
54
+			$tbl_user_rel_access_url = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
55 55
 			$access_url_id = api_get_current_access_url_id();
56 56
 			if ($access_url_id != -1) {
57 57
 				$sql = 'SELECT username, lastname, firstname
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
 			$return .= '<a href="javascript: void(0);" onclick="javascript: fill_coach_field(\''.$user['username'].'\')">'.api_get_person_name($user['firstname'], $user['lastname']).' ('.$user['username'].')</a><br />';
77 77
 		}
78 78
 	}
79
-	$xajax_response -> addAssign('ajax_list_coachs','innerHTML', api_utf8_encode($return));
79
+	$xajax_response -> addAssign('ajax_list_coachs', 'innerHTML', api_utf8_encode($return));
80 80
 	return $xajax_response;
81 81
 }
82 82
 $xajax -> processRequests();
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
 $form->addElement('header', $tool_name);
138 138
 $result = SessionManager::setForm($form);
139 139
 
140
-$htmlHeadXtra[] ='
140
+$htmlHeadXtra[] = '
141 141
 <script>
142 142
 $(function() {
143 143
     '.$result['js'].'
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
 $form->addButtonNext(get_lang('NextStep'));
148 148
 
149 149
 if (!$formSent) {
150
-    $formDefaults['access_start_date'] =  $formDefaults['display_start_date'] = api_get_local_time();
150
+    $formDefaults['access_start_date'] = $formDefaults['display_start_date'] = api_get_local_time();
151 151
     $formDefaults['coach_username'] = api_get_user_id();
152 152
 } else {
153 153
     $formDefaults['name'] = api_htmlentities($name, ENT_QUOTES, $charset);
@@ -172,7 +172,7 @@  discard block
 block discarded – undo
172 172
     $id_visibility = $params['session_visibility'];
173 173
     $duration = isset($params['duration']) ? $params['duration'] : null;
174 174
     $description = $params['description'];
175
-    $showDescription = isset($params['show_description']) ? 1: 0;
175
+    $showDescription = isset($params['show_description']) ? 1 : 0;
176 176
     $sendSubscriptionNotification = isset($params['send_subscription_notification']);
177 177
     $isThisImageCropped = isset($params['picture_crop_result']);
178 178
 
@@ -209,7 +209,7 @@  discard block
 block discarded – undo
209 209
 
210 210
     if ($return == strval(intval($return))) {
211 211
         // integer => no error on session creation
212
-        header('Location: add_courses_to_session.php?id_session=' . $return . '&add=true&msg=');
212
+        header('Location: add_courses_to_session.php?id_session='.$return.'&add=true&msg=');
213 213
         exit();
214 214
     }
215 215
 }
@@ -222,7 +222,7 @@  discard block
 block discarded – undo
222 222
 
223 223
 echo '<div class="actions">';
224 224
 echo '<a href="../session/session_list.php">'.
225
-    Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('PlatformAdmin'),'',ICON_SIZE_MEDIUM).'</a>';
225
+    Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('PlatformAdmin'), '', ICON_SIZE_MEDIUM).'</a>';
226 226
 echo '</div>';
227 227
 
228 228
 $form->display();
Please login to merge, or discard this patch.
main/inc/lib/online.inc.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -33,17 +33,17 @@  discard block
 block discarded – undo
33 33
 
34 34
         $login_date = api_get_utc_datetime();
35 35
         $access_url_id = 1;
36
-        if (api_get_multiple_access_url() && api_get_current_access_url_id()!=-1) {
36
+        if (api_get_multiple_access_url() && api_get_current_access_url_id() != -1) {
37 37
             $access_url_id = api_get_current_access_url_id();
38 38
         }
39 39
         $session_id = api_get_session_id();
40 40
         // if the $_course array exists this means we are in a course and we have to store this in the who's online table also
41 41
         // to have the x users in this course feature working
42
-        if (is_array($_course) && count($_course)>0 && !empty($_course['id'])) {
43
-            $query = "REPLACE INTO ".$online_table ." (login_id,login_user_id,login_date,user_ip, c_id, session_id, access_url_id)
42
+        if (is_array($_course) && count($_course) > 0 && !empty($_course['id'])) {
43
+            $query = "REPLACE INTO ".$online_table." (login_id,login_user_id,login_date,user_ip, c_id, session_id, access_url_id)
44 44
                       VALUES ($uid,$uid,'$login_date','$user_ip', '".$_course['real_id']."' , '$session_id' , '$access_url_id' )";
45 45
         } else {
46
-            $query = "REPLACE INTO ".$online_table ." (login_id,login_user_id,login_date,user_ip, c_id, session_id, access_url_id)
46
+            $query = "REPLACE INTO ".$online_table." (login_id,login_user_id,login_date,user_ip, c_id, session_id, access_url_id)
47 47
                       VALUES ($uid,$uid,'$login_date','$user_ip', 0, '$session_id', '$access_url_id')";
48 48
         }
49 49
         Database::query($query);
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
     		ORDER BY login_date DESC
118 118
     		LIMIT 0,1";
119 119
     $q_last_connection = Database::query($sql);
120
-    if (Database::num_rows($q_last_connection)>0) {
120
+    if (Database::num_rows($q_last_connection) > 0) {
121 121
         $i_id_last_connection = Database::result($q_last_connection, 0, "login_id");
122 122
     }
123 123
 
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
 {
168 168
     $online_table = Database::get_main_table(TABLE_STATISTIC_TRACK_E_ONLINE);
169 169
     $user_id = intval($user_id);
170
-    $query = "DELETE FROM " . $online_table . " WHERE login_user_id = $user_id";
170
+    $query = "DELETE FROM ".$online_table." WHERE login_user_id = $user_id";
171 171
     Database::query($query);
172 172
 }
173 173
 
@@ -248,7 +248,7 @@  discard block
 block discarded – undo
248 248
     $current_date = api_get_utc_datetime($online_time);
249 249
     $track_online_table = Database::get_main_table(TABLE_STATISTIC_TRACK_E_ONLINE);
250 250
     $friend_user_table = Database::get_main_table(TABLE_MAIN_USER_REL_USER);
251
-    $table_user	= Database::get_main_table(TABLE_MAIN_USER);
251
+    $table_user = Database::get_main_table(TABLE_MAIN_USER);
252 252
 
253 253
     if ($friends) {
254 254
         // 	who friends from social network is online
@@ -264,8 +264,8 @@  discard block
 block discarded – undo
264 264
                   LIMIT $from, $number_of_items";
265 265
     } else {
266 266
         $query = "SELECT DISTINCT login_user_id, login_date
267
-                    FROM ".$track_online_table ." e
268
-                    INNER JOIN ".$table_user ." u ON (u.id = e.login_user_id)
267
+                    FROM ".$track_online_table." e
268
+                    INNER JOIN ".$table_user." u ON (u.id = e.login_user_id)
269 269
                   WHERE u.status != ".ANONYMOUS." AND login_date >= '".$current_date."'
270 270
                   ORDER BY $column $direction
271 271
                   LIMIT $from, $number_of_items";
@@ -288,8 +288,8 @@  discard block
 block discarded – undo
288 288
             } else {
289 289
                 // all users online
290 290
                 $query = "SELECT login_user_id, login_date
291
-                          FROM ".$track_online_table ." track
292
-                          INNER JOIN ".$table_user ." u
291
+                          FROM ".$track_online_table." track
292
+                          INNER JOIN ".$table_user." u
293 293
                           ON (u.id=track.login_user_id)
294 294
                           WHERE u.status != ".ANONYMOUS." AND track.access_url_id =  $access_url_id AND
295 295
                                 login_date >= '".$current_date."'
@@ -426,7 +426,7 @@  discard block
 block discarded – undo
426 426
 	if ($result) {
427 427
 		$users_online = array();
428 428
 
429
-		while(list($login_user_id, $login_date) = Database::fetch_row($result)) {
429
+		while (list($login_user_id, $login_date) = Database::fetch_row($result)) {
430 430
             $users_online[] = $login_user_id;
431 431
 		}
432 432
 		return $users_online;
@@ -435,7 +435,7 @@  discard block
 block discarded – undo
435 435
 	}
436 436
 }
437 437
 
438
-function who_is_online_in_this_course_count($uid, $time_limit, $coursecode=null)
438
+function who_is_online_in_this_course_count($uid, $time_limit, $coursecode = null)
439 439
 {
440 440
 	if (empty($coursecode)) {
441 441
 		return false;
Please login to merge, or discard this patch.
plugin/bbb/listing.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -11,14 +11,14 @@  discard block
 block discarded – undo
11 11
 $tool_name = $plugin->get_lang('Videoconference');
12 12
 
13 13
 $htmlHeadXtra[] = api_get_js_simple(
14
-    api_get_path(WEB_PLUGIN_PATH) . 'bbb/resources/utils.js'
14
+    api_get_path(WEB_PLUGIN_PATH).'bbb/resources/utils.js'
15 15
 );
16
-$htmlHeadXtra[] = "<script>var _p = {web_plugin: '" . api_get_path(WEB_PLUGIN_PATH). "'}</script>";
16
+$htmlHeadXtra[] = "<script>var _p = {web_plugin: '".api_get_path(WEB_PLUGIN_PATH)."'}</script>";
17 17
 
18 18
 $tpl = new Template($tool_name);
19 19
 
20 20
 $isGlobal = isset($_GET['global']) ? true : false;
21
-$isGlobalPerUser = isset($_GET['user_id']) ? (int) $_GET['user_id']: false;
21
+$isGlobalPerUser = isset($_GET['user_id']) ? (int) $_GET['user_id'] : false;
22 22
 
23 23
 $bbb = new bbb('', '', $isGlobal, $isGlobalPerUser);
24 24
 $action = isset($_GET['action']) ? $_GET['action'] : null;
@@ -83,22 +83,22 @@  discard block
 block discarded – undo
83 83
         case 'end':
84 84
             $bbb->endMeeting($_GET['id']);
85 85
             $message = Display::return_message(
86
-                $plugin->get_lang('MeetingClosed') . '<br />' . $plugin->get_lang(
86
+                $plugin->get_lang('MeetingClosed').'<br />'.$plugin->get_lang(
87 87
                     'MeetingClosedComment'
88 88
                 ),
89 89
                 'success',
90 90
                 false
91 91
             );
92 92
 
93
-            if (file_exists(__DIR__ . '/config.vm.php')) {
94
-                require __DIR__ . '/../../vendor/autoload.php';
93
+            if (file_exists(__DIR__.'/config.vm.php')) {
94
+                require __DIR__.'/../../vendor/autoload.php';
95 95
 
96
-                require __DIR__ . '/lib/vm/AbstractVM.php';
97
-                require __DIR__ . '/lib/vm/VMInterface.php';
98
-                require __DIR__ . '/lib/vm/DigitalOceanVM.php';
99
-                require __DIR__ . '/lib/VM.php';
96
+                require __DIR__.'/lib/vm/AbstractVM.php';
97
+                require __DIR__.'/lib/vm/VMInterface.php';
98
+                require __DIR__.'/lib/vm/DigitalOceanVM.php';
99
+                require __DIR__.'/lib/VM.php';
100 100
 
101
-                $config = require __DIR__ . '/config.vm.php';
101
+                $config = require __DIR__.'/config.vm.php';
102 102
 
103 103
                 $vm = new VM($config);
104 104
                 $vm->resizeToMinLimit();
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
     $actionLinks = [
196 196
         Display::toolbarButton(
197 197
             $plugin->get_lang('AdminView'),
198
-            api_get_path(WEB_PLUGIN_PATH) . 'bbb/admin.php',
198
+            api_get_path(WEB_PLUGIN_PATH).'bbb/admin.php',
199 199
             'list',
200 200
             'primary'
201 201
         )
Please login to merge, or discard this patch.
main/admin/index.php 1 patch
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
 $nameTools = get_lang('PlatformAdmin');
20 20
 
21 21
 $accessUrlId = 0;
22
-$adminExtraContentDir = api_get_path(SYS_APP_PATH) . "home/admin/";
22
+$adminExtraContentDir = api_get_path(SYS_APP_PATH)."home/admin/";
23 23
 
24 24
 if (api_is_multiple_url_enabled()) {
25 25
     $accessUrlId = api_get_current_access_url_id();
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
         $urlInfo = api_get_access_url($accessUrlId);
29 29
         $url = api_remove_trailing_slash(preg_replace('/https?:\/\//i', '', $urlInfo['url']));
30 30
         $cleanUrl = str_replace('/', '-', $url);
31
-        $adminExtraContentDir = api_get_path(SYS_APP_PATH) . "home/$cleanUrl/admin/";
31
+        $adminExtraContentDir = api_get_path(SYS_APP_PATH)."home/$cleanUrl/admin/";
32 32
     }
33 33
 }
34 34
 
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
             <div class="form-group">
85 85
                 <input class="form-control" type="text" name="keyword" value="">
86 86
                 <button class="btn btn-default" type="submit">
87
-                    <em class="fa fa-search"></em> ' . get_lang('Search') . '
87
+                    <em class="fa fa-search"></em> ' . get_lang('Search').'
88 88
                 </button>
89 89
             </div>
90 90
         </form>';
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
             <div class="form-group">
138 138
                 <input class="form-control" type="text" name="keyword" value="">
139 139
                 <button class="btn btn-default" type="submit">
140
-                    <em class="fa fa-search"></em> ' . get_lang('Search') . '
140
+                    <em class="fa fa-search"></em> ' . get_lang('Search').'
141 141
                 </button>
142 142
             </div>
143 143
         </form>';
@@ -197,7 +197,7 @@  discard block
 block discarded – undo
197 197
                 <input class="form-control" type="text" name="search_field" value="" >
198 198
                 <input type="hidden" value="search_setting" name="category">
199 199
                 <button class="btn btn-default" type="submit">
200
-                    <em class="fa fa-search"></em> ' . get_lang('Search') . '
200
+                    <em class="fa fa-search"></em> ' . get_lang('Search').'
201 201
                 </button>
202 202
             </div>
203 203
         </form>';
@@ -210,24 +210,24 @@  discard block
 block discarded – undo
210 210
     $items[] = array('url' => 'settings.php?category=Regions', 'label' => get_lang('Regions'));
211 211
     $items[] = array('url' => 'system_announcements.php', 'label' => get_lang('SystemAnnouncements'));
212 212
     $items[] = array(
213
-        'url' => api_get_path(WEB_CODE_PATH) . 'calendar/agenda_js.php?type=admin',
213
+        'url' => api_get_path(WEB_CODE_PATH).'calendar/agenda_js.php?type=admin',
214 214
         'label' => get_lang('GlobalAgenda')
215 215
     );
216 216
     $items[] = array('url' => 'configure_homepage.php', 'label' => get_lang('ConfigureHomePage'));
217 217
     $items[] = array('url' => 'configure_inscription.php', 'label' => get_lang('ConfigureInscription'));
218 218
     $items[] = array('url' => 'statistics/index.php', 'label' => get_lang('Statistics'));
219 219
     $items[] = array(
220
-        'url' => api_get_path(WEB_CODE_PATH) . 'mySpace/company_reports.php',
220
+        'url' => api_get_path(WEB_CODE_PATH).'mySpace/company_reports.php',
221 221
         'label' => get_lang('Reports')
222 222
     );
223 223
     $items[] = array(
224
-        'url' => api_get_path(WEB_CODE_PATH) . 'admin/teacher_time_report.php',
224
+        'url' => api_get_path(WEB_CODE_PATH).'admin/teacher_time_report.php',
225 225
         'label' => get_lang('TeacherTimeReport')
226 226
     );
227 227
 
228 228
     if (api_get_configuration_value('chamilo_cms')) {
229 229
         $items[] = array(
230
-            'url' => api_get_path(WEB_PATH) . 'web/app_dev.php/administration/dashboard',
230
+            'url' => api_get_path(WEB_PATH).'web/app_dev.php/administration/dashboard',
231 231
             'label' => get_lang('CMS')
232 232
         );
233 233
     }
@@ -277,7 +277,7 @@  discard block
 block discarded – undo
277 277
                     <div class="form-group">
278 278
                         <input class="form-control" type="text" name="keyword" value="">
279 279
                         <button class="btn btn-default" type="submit">
280
-                            <em class="fa fa-search"></em> ' . get_lang('Search') . '
280
+                            <em class="fa fa-search"></em> ' . get_lang('Search').'
281 281
                         </button>
282 282
                     </div>
283 283
                 </form>';
@@ -305,7 +305,7 @@  discard block
 block discarded – undo
305 305
 
306 306
 if (api_is_platform_admin()) {
307 307
     // option only visible in development mode. Enable through code if required
308
-    if (is_dir(api_get_path(SYS_TEST_PATH) . 'datafiller/')) {
308
+    if (is_dir(api_get_path(SYS_TEST_PATH).'datafiller/')) {
309 309
         $items[] = array('url' => 'user_move_stats.php', 'label' => get_lang('MoveUserStats'));
310 310
     }
311 311
     $items[] = array('url' => 'career_dashboard.php', 'label' => get_lang('CareersAndPromotions'));
@@ -331,7 +331,7 @@  discard block
 block discarded – undo
331 331
     $items = array();
332 332
     $items[] = array('url' => 'special_exports.php', 'label' => get_lang('SpecialExports'));
333 333
     $items[] = array('url' => 'system_status.php', 'label' => get_lang('SystemStatus'));
334
-    if (is_dir(api_get_path(SYS_TEST_PATH) . 'datafiller/')) {
334
+    if (is_dir(api_get_path(SYS_TEST_PATH).'datafiller/')) {
335 335
         $items[] = array('url' => 'filler.php', 'label' => get_lang('DataFiller'));
336 336
     }
337 337
 
@@ -375,15 +375,15 @@  discard block
 block discarded – undo
375 375
         $items[] = array('url' => 'skills_wheel.php', 'label' => get_lang('SkillsWheel'));
376 376
         $items[] = array('url' => 'skills_import.php', 'label' => get_lang('SkillsImport'));
377 377
         $items[] = array('url' => 'skill_list.php', 'label' => get_lang('ManageSkills'));
378
-        $items[] = array('url'=>'skill.php',   'label' => get_lang('ManageSkillsLevels'));
378
+        $items[] = array('url'=>'skill.php', 'label' => get_lang('ManageSkillsLevels'));
379 379
         //$items[] = array('url'=>'skills_profile.php',   'label' => get_lang('SkillsProfile'));
380 380
         $items[] = array(
381
-            'url' => api_get_path(WEB_CODE_PATH) . 'social/skills_ranking.php',
381
+            'url' => api_get_path(WEB_CODE_PATH).'social/skills_ranking.php',
382 382
             'label' => get_lang('SkillsRanking')
383 383
         );
384 384
         $items[] = array('url' => 'skills_gradebook.php', 'label' => get_lang('SkillsAndGradebooks'));
385 385
         $items[] = array(
386
-            'url' => api_get_path(WEB_CODE_PATH) . 'admin/skill_badge.php',
386
+            'url' => api_get_path(WEB_CODE_PATH).'admin/skill_badge.php',
387 387
             'label' => get_lang('Badges')
388 388
         );
389 389
         $blocks['skills']['items'] = $items;
@@ -426,12 +426,12 @@  discard block
 block discarded – undo
426 426
                     $itemUrl = '/start.php';
427 427
                 }
428 428
 
429
-                if (!file_exists(api_get_path(SYS_PLUGIN_PATH) . $pluginName . $itemUrl)) {
429
+                if (!file_exists(api_get_path(SYS_PLUGIN_PATH).$pluginName.$itemUrl)) {
430 430
                     continue;
431 431
                 }
432 432
 
433 433
                 $items[] = array(
434
-                    'url' => api_get_path(WEB_PLUGIN_PATH) . $plugin_name . $itemUrl,
434
+                    'url' => api_get_path(WEB_PLUGIN_PATH).$plugin_name.$itemUrl,
435 435
                     'label' => $plugin_info['title']
436 436
                 );
437 437
             }
@@ -510,7 +510,7 @@  discard block
 block discarded – undo
510 510
     }
511 511
     //end hack
512 512
 }
513
-$admin_ajax_url = api_get_path(WEB_AJAX_PATH) . 'admin.ajax.php';
513
+$admin_ajax_url = api_get_path(WEB_AJAX_PATH).'admin.ajax.php';
514 514
 
515 515
 $tpl = new Template();
516 516
 
@@ -563,7 +563,7 @@  discard block
 block discarded – undo
563 563
                 die;
564 564
             }
565 565
 
566
-            $fullFilePath = $adminExtraContentDir . $extraData['block'];
566
+            $fullFilePath = $adminExtraContentDir.$extraData['block'];
567 567
             $fullFilePath .= "_extra.html";
568 568
 
569 569
             file_put_contents($fullFilePath, $extraData['extra_content']);
Please login to merge, or discard this patch.
main/messages/download.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -48,8 +48,8 @@  discard block
 block discarded – undo
48 48
 
49 49
 // get message user id for inbox/outbox
50 50
 $message_uid = '';
51
-$message_type = array('inbox','outbox');
52
-if (in_array($_GET['type'],$message_type)) {
51
+$message_type = array('inbox', 'outbox');
52
+if (in_array($_GET['type'], $message_type)) {
53 53
 	if ($_GET['type'] == 'inbox') {
54 54
 		$message_uid = $row_users['user_receiver_id'];
55 55
 	} else {
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 
64 64
 if (!empty($row_users['group_id'])) {
65 65
 	$users_group = $userGroup->get_all_users_by_group($row_users['group_id']);
66
-	if (!in_array($current_uid,array_keys($users_group))) {
66
+	if (!in_array($current_uid, array_keys($users_group))) {
67 67
 		$not_allowed_to_edit = true;
68 68
 	}
69 69
 } else {
Please login to merge, or discard this patch.
main/admin/sub_language_add.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -39,8 +39,8 @@  discard block
 block discarded – undo
39 39
     $sublanguage_available = Database::escape_string($sublanguage_available);
40 40
     $parent_id = intval($parent_id);
41 41
 
42
-    $sql = 'INSERT INTO ' . $tbl_admin_languages . '(original_name,english_name,isocode,dokeos_folder,available,parent_id)
43
-    	  VALUES ("' . $original_name . '","' . $english_name . '","' . $isocode . '","' . $english_name . '","' . $sublanguage_available . '","' . $parent_id . '")';
42
+    $sql = 'INSERT INTO '.$tbl_admin_languages.'(original_name,english_name,isocode,dokeos_folder,available,parent_id)
43
+    	  VALUES ("' . $original_name.'","'.$english_name.'","'.$isocode.'","'.$english_name.'","'.$sublanguage_available.'","'.$parent_id.'")';
44 44
     $res = Database::query($sql);
45 45
     if ($res === false) {
46 46
         return false;
@@ -65,8 +65,8 @@  discard block
 block discarded – undo
65 65
 function check_if_language_exist($original_name, $english_name, $isocode, $sublanguage_available)
66 66
 {
67 67
     $tbl_admin_languages = Database:: get_main_table(TABLE_MAIN_LANGUAGE);
68
-    $sql_original_name = 'SELECT count(*) AS count_original_name FROM ' . $tbl_admin_languages . ' WHERE original_name="' . Database::escape_string($original_name) . '" ';
69
-    $sql_english_name = 'SELECT count(*) AS count_english_name FROM ' . $tbl_admin_languages . ' WHERE english_name="' . Database::escape_string($english_name) . '" ';
68
+    $sql_original_name = 'SELECT count(*) AS count_original_name FROM '.$tbl_admin_languages.' WHERE original_name="'.Database::escape_string($original_name).'" ';
69
+    $sql_english_name = 'SELECT count(*) AS count_english_name FROM '.$tbl_admin_languages.' WHERE english_name="'.Database::escape_string($english_name).'" ';
70 70
     //$sql_isocode='SELECT count(*) AS count_isocode FROM '.$tbl_admin_languages.' WHERE isocode="'.Database::escape_string($isocode).'" ';
71 71
     $rs_original_name = Database::query($sql_original_name);
72 72
     $rs_english_name = Database::query($sql_english_name);
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
  */
123 123
 function ckeck_if_is_parent_of_sub_language($parent_id)
124 124
 {
125
-    $sql = 'SELECT count(*) AS count FROM language WHERE parent_id= ' . intval($parent_id) . '';
125
+    $sql = 'SELECT count(*) AS count FROM language WHERE parent_id= '.intval($parent_id).'';
126 126
     $rs = Database::query($sql);
127 127
     if (Database::num_rows($rs) > 0 && Database::result($rs, 0, 'count') == 1) {
128 128
         return true;
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
     }
184 184
 }
185 185
 
186
-$language_name = get_lang('CreateSubLanguageForLanguage') . ' ( ' . strtolower($language_name) . ' )';
186
+$language_name = get_lang('CreateSubLanguageForLanguage').' ( '.strtolower($language_name).' )';
187 187
 
188 188
 if (ckeck_if_is_parent_of_sub_language($parent_id) === true && isset($_GET['action']) && $_GET['action'] == 'deletesublanguage') {
189 189
     $language_name = get_lang('DeleteSubLanguage');
@@ -204,15 +204,15 @@  discard block
 block discarded – undo
204 204
     foreach ($check_information as $index_information => $value_information) {
205 205
         $allow_insert_info = false;
206 206
         if ($index_information == 'original_name') {
207
-            $msg .= Display::return_message(get_lang('AlreadyExists') . ' "' . get_lang('OriginalName') . '" ' . '(' . $original_name . ')',
207
+            $msg .= Display::return_message(get_lang('AlreadyExists').' "'.get_lang('OriginalName').'" '.'('.$original_name.')',
208 208
                 'error');
209 209
         }
210 210
         if ($index_information == 'english_name') {
211
-            $msg .= Display::return_message(get_lang('AlreadyExists') . ' "' . get_lang('EnglishName') . '" ' . '(' . $english_name . ')',
211
+            $msg .= Display::return_message(get_lang('AlreadyExists').' "'.get_lang('EnglishName').'" '.'('.$english_name.')',
212 212
                 'error');
213 213
         }
214 214
         if ($index_information == 'isocode') {
215
-            $msg .= Display::return_message(get_lang('CodeDoesNotExists') . ': ' . $isocode . '', 'error');
215
+            $msg .= Display::return_message(get_lang('CodeDoesNotExists').': '.$isocode.'', 'error');
216 216
         }
217 217
         if ($index_information == 'execute_add' && $value_information === true) {
218 218
             $allow_insert_info = true;
@@ -226,7 +226,7 @@  discard block
 block discarded – undo
226 226
             $english_name = api_strtolower($english_name);
227 227
 
228 228
             $isocode = str_replace(' ', '_', $isocode);
229
-            $str_info = '<br/>' . get_lang('OriginalName') . ' : ' . $original_name . '<br/>' . get_lang('EnglishName') . ' : ' . $english_name . '<br/>' . get_lang('PlatformCharsetTitle') . ' : ' . $isocode;
229
+            $str_info = '<br/>'.get_lang('OriginalName').' : '.$original_name.'<br/>'.get_lang('EnglishName').' : '.$english_name.'<br/>'.get_lang('PlatformCharsetTitle').' : '.$isocode;
230 230
 
231 231
             $mkdir_result = SubLanguageManager::add_language_directory($english_name);
232 232
             if ($mkdir_result) {
@@ -235,7 +235,7 @@  discard block
 block discarded – undo
235 235
                     SubLanguageManager::remove_language_directory($english_name);
236 236
                     $msg .= Display::return_message(get_lang('LanguageDirectoryNotWriteableContactAdmin'), 'error');
237 237
                 } else {
238
-                    Display::addFlash(Display::return_message(get_lang('TheNewSubLanguageHasBeenAdded') . $str_info, null, false));
238
+                    Display::addFlash(Display::return_message(get_lang('TheNewSubLanguageHasBeenAdded').$str_info, null, false));
239 239
                     unset($interbreadcrumb);
240 240
                     $_GET['sub_language_id'] = $_REQUEST['sub_language_id'] = $sl_id;
241 241
                     require 'sub_language.php';
@@ -271,7 +271,7 @@  discard block
 block discarded – undo
271 271
 if (isset($_GET['action']) && $_GET['action'] == 'definenewsublanguage') {
272 272
     $text = $language_name;
273 273
     $form = new FormValidator('addsublanguage', 'post',
274
-        'sub_language_add.php?id=' . Security::remove_XSS($_GET['id']) . '&action=definenewsublanguage');
274
+        'sub_language_add.php?id='.Security::remove_XSS($_GET['id']).'&action=definenewsublanguage');
275 275
     $class = 'add';
276 276
     $form->addElement('header', '', $text);
277 277
     $form->addElement('text', 'original_name', get_lang('OriginalName'), 'class="input_titles"');
@@ -284,7 +284,7 @@  discard block
 block discarded – undo
284 284
     $form->addElement('checkbox', 'sub_language_is_visible', '', get_lang('Visibility'));
285 285
     $form->addButtonCreate(get_lang('CreateSubLanguage'), 'SubmitAddNewLanguage');
286 286
     //$values['original_name'] = $language_details['original_name'].'...'; -> cannot be used because of quickform filtering (freeze)
287
-    $values['english_name'] = $language_details['english_name'] . '2';
287
+    $values['english_name'] = $language_details['english_name'].'2';
288 288
     $values['isocode'] = $language_details['isocode'];
289 289
     $form->setDefaults($values);
290 290
     $form->display();
@@ -292,7 +292,7 @@  discard block
 block discarded – undo
292 292
     if (isset($_GET['action']) && $_GET['action'] == 'deletesublanguage') {
293 293
         $text = $language_name;
294 294
         $form = new FormValidator('deletesublanguage', 'post',
295
-            'sub_language_add.php?id=' . Security::remove_XSS($_GET['id']) . '&sub_language_id=' . Security::remove_XSS($_GET['sub_language_id']));
295
+            'sub_language_add.php?id='.Security::remove_XSS($_GET['id']).'&sub_language_id='.Security::remove_XSS($_GET['sub_language_id']));
296 296
         $class = 'minus';
297 297
         $form->addElement('header', '', $text);
298 298
         $form->addElement('static', '', get_lang('OriginalName'), $original_name);
Please login to merge, or discard this patch.
main/ticket/ticket_details.php 1 patch
Spacing   +31 added lines, -31 removed lines patch added patch discarded remove patch
@@ -26,10 +26,10 @@  discard block
 block discarded – undo
26 26
 		width: 600,
27 27
 		modal: true,
28 28
 		buttons: {
29
-            ' . get_lang('Accept') . ': function(){
29
+            ' . get_lang('Accept').': function(){
30 30
                 $("#frmResponsable").submit()
31 31
             },
32
-            ' . ucfirst(get_lang('Close')) . ': function() {
32
+            ' . ucfirst(get_lang('Close')).': function() {
33 33
                 $(this).dialog("close");
34 34
             }
35 35
             }
@@ -40,25 +40,25 @@  discard block
 block discarded – undo
40 40
         });
41 41
 
42 42
         $(".responseyes").click(function () {
43
-            if(!confirm("' . get_lang('AreYouSure') . ' : ' . strtoupper(get_lang('Yes')) . '. ' . get_lang('IfYouAreSureTheTicketWillBeClosed') . '")){
43
+            if(!confirm("' . get_lang('AreYouSure').' : '.strtoupper(get_lang('Yes')).'. '.get_lang('IfYouAreSureTheTicketWillBeClosed').'")){
44 44
                 return false;
45 45
             }
46 46
         });
47 47
 
48 48
         $("input#responseno").click(function () {
49
-            if(!confirm("' . get_lang('AreYouSure') . ' : ' . strtoupper(get_lang('No')) . '")){
49
+            if(!confirm("' . get_lang('AreYouSure').' : '.strtoupper(get_lang('No')).'")){
50 50
                 return false;
51 51
             }
52 52
         });
53 53
 
54 54
         $("#unassign").click(function () {
55
-            if (!confirm("' . get_lang('AreYouSureYouWantToUnassignTheTicket') . '")) {
55
+            if (!confirm("' . get_lang('AreYouSureYouWantToUnassignTheTicket').'")) {
56 56
                 return false;
57 57
             }
58 58
         });
59 59
 
60 60
         $("#close").click(function () {
61
-            if (!confirm("' . get_lang('AreYouSureYouWantToCloseTheTicket') . '")) {
61
+            if (!confirm("' . get_lang('AreYouSureYouWantToCloseTheTicket').'")) {
62 62
                 return false;
63 63
             }
64 64
         });
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
     fckEditor1val = CKEDITOR.instances["content"].getData();
70 70
     document.getElementById("content").value= fckEditor1val;
71 71
     if (fckEditor1val == ""){
72
-        alert("' . get_lang('Filled') . '");
72
+        alert("' . get_lang('Filled').'");
73 73
         return false;
74 74
     }
75 75
 }
@@ -199,7 +199,7 @@  discard block
 block discarded – undo
199 199
             $user_id
200 200
         );
201 201
         Display::addFlash(Display::return_message(get_lang('Updated')));
202
-        header("Location:" . api_get_self() . "?ticket_id=" . $ticket_id);
202
+        header("Location:".api_get_self()."?ticket_id=".$ticket_id);
203 203
         exit;
204 204
 
205 205
     }
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
                 TicketManager::assign_ticket_user($_GET['ticket_id'], $_POST['admins']);
213 213
             }
214 214
             Display::addFlash(Display::return_message(get_lang('Updated')));
215
-            header("Location:" . api_get_self() . "?ticket_id=" . $ticket_id);
215
+            header("Location:".api_get_self()."?ticket_id=".$ticket_id);
216 216
             exit;
217 217
             break;
218 218
         case 'unassign':
@@ -220,7 +220,7 @@  discard block
 block discarded – undo
220 220
                 TicketManager::assign_ticket_user($_GET['ticket_id'], 0);
221 221
             }
222 222
             Display::addFlash(Display::return_message(get_lang('Updated')));
223
-            header("Location:" . api_get_self() . "?ticket_id=" . $ticket_id);
223
+            header("Location:".api_get_self()."?ticket_id=".$ticket_id);
224 224
             exit;
225 225
             break;
226 226
         default:
@@ -228,7 +228,7 @@  discard block
 block discarded – undo
228 228
     }
229 229
 }
230 230
 
231
-$title = 'Ticket #' . $ticket['ticket']['code'];
231
+$title = 'Ticket #'.$ticket['ticket']['code'];
232 232
 
233 233
 if (!isset($_POST['compose'])) {
234 234
     if (isset($_REQUEST['close'])) {
@@ -274,9 +274,9 @@  discard block
 block discarded – undo
274 274
               </style>";
275 275
     }
276 276
     if ($isAdmin) {
277
-        $senderData = get_lang('AddedBy'). ' '.$ticket['ticket']['user_url'].' (' . $ticket['usuario']['username'] . ').';
277
+        $senderData = get_lang('AddedBy').' '.$ticket['ticket']['user_url'].' ('.$ticket['usuario']['username'].').';
278 278
     } else {
279
-        $senderData = get_lang('AddedBy'). ' '.$ticket['usuario']['complete_name'].' (' . $ticket['usuario']['username']. ').';
279
+        $senderData = get_lang('AddedBy').' '.$ticket['usuario']['complete_name'].' ('.$ticket['usuario']['username'].').';
280 280
     }
281 281
 
282 282
     echo '<table width="100%" >
@@ -285,8 +285,8 @@  discard block
 block discarded – undo
285 285
               <h1>'.$title.' '.$form_close_ticket.'</h1>
286 286
               <h2>'.$ticket['ticket']['subject'].'</h2>
287 287
               <p>
288
-                '.$senderData.' ' .
289
-                get_lang('Created') . ' '.
288
+                '.$senderData.' '.
289
+                get_lang('Created').' '.
290 290
                 Display::url(
291 291
                     date_to_str_ago($ticket['ticket']['start_date_from_db']),
292 292
                     '#',
@@ -302,37 +302,37 @@  discard block
 block discarded – undo
302 302
               </td>
303 303
             </tr>
304 304
             <tr>
305
-               <td><p><b>' . get_lang('Category') . ': </b>' . $ticket['ticket']['name'] . '</p></td>
305
+               <td><p><b>' . get_lang('Category').': </b>'.$ticket['ticket']['name'].'</p></td>
306 306
             </tr>
307 307
             <tr>
308
-               <td><p ' . $bold . '><b>' . get_lang('Status') . ':</b> ' . $ticket['ticket']['status'] . '</p></td>
308
+               <td><p ' . $bold.'><b>'.get_lang('Status').':</b> '.$ticket['ticket']['status'].'</p></td>
309 309
             </tr>
310 310
             <tr>
311
-                <td><p><b>' . get_lang('Priority') . ': </b>' . $ticket['ticket']['priority'] . '<p></td>
311
+                <td><p><b>' . get_lang('Priority').': </b>'.$ticket['ticket']['priority'].'<p></td>
312 312
             </tr>';
313 313
 
314 314
     if (!empty($ticket['ticket']['assigned_last_user'])) {
315 315
         $assignedUser = api_get_user_info($ticket['ticket']['assigned_last_user']);
316 316
         echo '<tr>
317
-                <td><p><b>' . get_lang('AssignedTo') . ': </b>' . $assignedUser['complete_name'] . '<p></td>
317
+                <td><p><b>' . get_lang('AssignedTo').': </b>'.$assignedUser['complete_name'].'<p></td>
318 318
             </tr>';
319 319
     } else {
320 320
         echo '<tr>
321
-                <td><p><b>' . get_lang('AssignedTo') . ': </b>-<p></td>
321
+                <td><p><b>' . get_lang('AssignedTo').': </b>-<p></td>
322 322
             </tr>';
323 323
     }
324 324
     if ($ticket['ticket']['course_url'] != null) {
325 325
         if (!empty($ticket['ticket']['session_id'])) {
326 326
             $sessionInfo = api_get_session_info($ticket['ticket']['session_id']);
327 327
             echo '<tr>
328
-				<td><b>' . get_lang('Session') . ':</b> ' . $sessionInfo['name'] . ' </td>
328
+				<td><b>' . get_lang('Session').':</b> '.$sessionInfo['name'].' </td>
329 329
 			    <td></td>
330 330
 	            <td colspan="2"></td>
331 331
 	          </tr>';
332 332
         }
333 333
 
334 334
         echo '<tr>
335
-				<td><b>' . get_lang('Course') . ':</b> ' . $ticket['ticket']['course_url'] . ' </td>
335
+				<td><b>' . get_lang('Course').':</b> '.$ticket['ticket']['course_url'].' </td>
336 336
 			    <td></td>
337 337
 	            <td colspan="2"></td>
338 338
 	          </tr>';
@@ -340,7 +340,7 @@  discard block
 block discarded – undo
340 340
     echo '<tr>
341 341
             <td>
342 342
             <hr />
343
-            <b>' . get_lang('Description') . ':</b> <br />
343
+            <b>' . get_lang('Description').':</b> <br />
344 344
             '.$ticket['ticket']['message'].'
345 345
             <hr />
346 346
             </td>            
@@ -360,9 +360,9 @@  discard block
 block discarded – undo
360 360
 
361 361
         $receivedMessage = '';
362 362
         if (!empty($message['subject'])) {
363
-            $receivedMessage = '<b>'.get_lang('Subject') . ': </b> '.$message['subject'].'<br/>';
363
+            $receivedMessage = '<b>'.get_lang('Subject').': </b> '.$message['subject'].'<br/>';
364 364
         }
365
-        $receivedMessage = '<b>'.get_lang('Message') . ':</b><br/>'.$message['message'].'<br/>';
365
+        $receivedMessage = '<b>'.get_lang('Message').':</b><br/>'.$message['message'].'<br/>';
366 366
 
367 367
         $attachmentLinks = '';
368 368
         if (isset($message['attachments'])) {
@@ -374,9 +374,9 @@  discard block
 block discarded – undo
374 374
             }
375 375
         }
376 376
 
377
-        $entireMessage = $receivedMessage . $attachmentLinks;
377
+        $entireMessage = $receivedMessage.$attachmentLinks;
378 378
         $counterLink = Display::url('#'.$counter, api_get_self().'?ticket_id='.$ticket_id.'#note-'.$counter);
379
-        echo '<a id="note-'.$counter.'"> </a><h4>' . sprintf(get_lang('UpdatedByX'), $message['user_created']).' '.$date.
379
+        echo '<a id="note-'.$counter.'"> </a><h4>'.sprintf(get_lang('UpdatedByX'), $message['user_created']).' '.$date.
380 380
             ' <span class="pull-right">'.$counterLink.'</span></h4>';
381 381
         echo Display::div(
382 382
             $entireMessage,
@@ -385,7 +385,7 @@  discard block
 block discarded – undo
385 385
         $counter++;
386 386
     }
387 387
 
388
-    $subject = get_lang('ReplyShort') .': '.$ticket['ticket']['subject'];
388
+    $subject = get_lang('ReplyShort').': '.$ticket['ticket']['subject'];
389 389
 
390 390
     if ($ticket['ticket']['status_id'] != TicketManager::STATUS_FORWARDED &&
391 391
         $ticket['ticket']['status_id'] != TicketManager::STATUS_CLOSE
@@ -437,7 +437,7 @@  discard block
 block discarded – undo
437 437
         }
438 438
     }
439 439
     Display::addFlash(Display::return_message(get_lang('Saved')));
440
-    header("Location:" . api_get_self() . "?ticket_id=" . $ticket_id);
440
+    header("Location:".api_get_self()."?ticket_id=".$ticket_id);
441 441
     exit;
442 442
 }
443 443
 
@@ -452,7 +452,7 @@  discard block
 block discarded – undo
452 452
     $form = new FormValidator(
453 453
         'send_ticket',
454 454
         'POST',
455
-        api_get_self() . '?ticket_id=' . $ticket['id'],
455
+        api_get_self().'?ticket_id='.$ticket['id'],
456 456
         '',
457 457
         array(
458 458
             'enctype' => 'multipart/form-data',
@@ -550,7 +550,7 @@  discard block
 block discarded – undo
550 550
     $form->addLabel('', '<span id="filepaths"><div id="filepath_1"></div></span>');
551 551
     $form->addLabel('',
552 552
         '<span id="link-more-attach">
553
-         <span class="btn btn-success" onclick="return add_image_form()">' . get_lang('AddOneMoreFile') . '</span>
553
+         <span class="btn btn-success" onclick="return add_image_form()">' . get_lang('AddOneMoreFile').'</span>
554 554
          </span>
555 555
          ('.sprintf(get_lang('MaximunFileSizeX'), format_file_size(api_get_setting('message_max_upload_filesize'))).')
556 556
     ');
Please login to merge, or discard this patch.
main/inc/lib/grade_model.lib.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
     public function __construct()
17 17
     {
18 18
         parent::__construct();
19
-        $this->table =  Database::get_main_table(TABLE_GRADE_MODEL);
19
+        $this->table = Database::get_main_table(TABLE_GRADE_MODEL);
20 20
     }
21 21
 
22 22
     /**
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
     {
56 56
 		// action links
57 57
 		echo '<div class="actions" style="margin-bottom:20px">';
58
-        echo '<a href="grade_models.php">'.Display::return_icon('back.png',get_lang('Back'),'', ICON_SIZE_MEDIUM).'</a>';
58
+        echo '<a href="grade_models.php">'.Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM).'</a>';
59 59
 		echo '<a href="'.api_get_self().'?action=add">'.Display::return_icon('add.png', get_lang('Add'), '', ICON_SIZE_MEDIUM).'</a>';
60 60
 		echo '</div>';
61 61
         echo Display::grid_html('grade_model');
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
             $form->addElement('hidden', 'components['.$i.'][id]', null);
130 130
 
131 131
             $template_percentage =
132
-            '<div id=' . $i . ' style="display: '.(($i<=$nr_items)?'inline':'none').';" class="form-group">                
132
+            '<div id='.$i.' style="display: '.(($i <= $nr_items) ? 'inline' : 'none').';" class="form-group">                
133 133
                 <label for="" class="col-sm-2 control-label">
134 134
                     {label}
135 135
                 </label>
@@ -143,11 +143,11 @@  discard block
 block discarded – undo
143 143
 
144 144
             $template_title =
145 145
             '&nbsp{element} <!-- BEGIN error --> <span class="form_error">{error}</span><!-- END error -->
146
-             <a href="javascript:plusItem(' . ($counter+1) . ')">
147
-                '.Display::return_icon('add.png', get_lang('Add'), ['id' => 'plus-' . ($counter+1), 'style' => 'display: '.(($counter>=$nr_items) ? 'inline':'none') ]).'
146
+             <a href="javascript:plusItem(' . ($counter + 1).')">
147
+                '.Display::return_icon('add.png', get_lang('Add'), ['id' => 'plus-'.($counter + 1), 'style' => 'display: '.(($counter >= $nr_items) ? 'inline' : 'none')]).'
148 148
             </a>
149
-            <a href="javascript:minItem(' . ($counter) . ')">
150
-                '.Display::return_icon('delete.png', get_lang('Delete'), ['id' => 'min-' . ($counter), 'style' => 'display: '.(($counter>=$nr_items) ? 'inline':'none') ]).'
149
+            <a href="javascript:minItem(' . ($counter).')">
150
+                '.Display::return_icon('delete.png', get_lang('Delete'), ['id' => 'min-'.($counter), 'style' => 'display: '.(($counter >= $nr_items) ? 'inline' : 'none')]).'
151 151
             </a>
152 152
             </div></div>';
153 153
             $renderer->setElementTemplate($template_title, 'components['.$i.'][title]');
Please login to merge, or discard this patch.