Passed
Push — 1.10.x ( aae6b1...e5a590 )
by Yannick
131:05 queued 81:36
created
main/newscorm/lp_ajax_switch_item.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -198,7 +198,7 @@
 block discarded – undo
198 198
         //"lms_progress_bar_mode='".$myprogress_mode."';" .
199 199
         "olms.lms_view_id=".$view_id.";".
200 200
         "olms.lms_user_id=".$user_id.";".
201
-        "olms.next_item=".$new_item_id.";". // This one is very important to replace possible literal strings.
201
+        "olms.next_item=".$new_item_id.";".// This one is very important to replace possible literal strings.
202 202
         "olms.lms_next_item=".$mynext.";".
203 203
         "olms.lms_previous_item=".$myprevious.";".
204 204
         "olms.lms_item_type = '".$myitemtype."';".
Please login to merge, or discard this patch.
main/newscorm/resourcelinker.php 3 patches
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -308,11 +308,11 @@
 block discarded – undo
308 308
         //$_SESSION['addedresourceid']=null;
309 309
         // cleaning up the session once again
310 310
         $_SESSION['addedresource'] = null;
311
-           $_SESSION['addedresourceid'] = null;
312
-           $_SESSION['addedresourceassigned'] = null;
313
-           unset ($_SESSION['addedresource']);
314
-           unset ($_SESSION['addedresourceid']);
315
-           unset ($_SESSION['addedresourceassigned']);
311
+            $_SESSION['addedresourceid'] = null;
312
+            $_SESSION['addedresourceassigned'] = null;
313
+            unset ($_SESSION['addedresource']);
314
+            unset ($_SESSION['addedresourceid']);
315
+            unset ($_SESSION['addedresourceassigned']);
316 316
     }
317 317
 }
318 318
 
Please login to merge, or discard this patch.
Spacing   +28 added lines, -28 removed lines patch added patch discarded remove patch
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
     }*/
87 87
 
88 88
     // Get max display_order so far in this parent chapter.
89
-    $sql = "SELECT MAX(display_order) as maxi FROM $tbl_lp_item " .
89
+    $sql = "SELECT MAX(display_order) as maxi FROM $tbl_lp_item ".
90 90
             "WHERE c_id = $course_id AND lp_id = $learnpath_id ".
91 91
             " AND parent_item_id = $chapter_id";
92 92
     $res = Database::query($sql);
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
         $lastorder_item = 0;
97 97
         $previous = 0;
98 98
     } else {
99
-        $sql = "SELECT id FROM $tbl_lp_item " .
99
+        $sql = "SELECT id FROM $tbl_lp_item ".
100 100
                 "WHERE lp_id = $learnpath_id AND parent_item_id=$chapter_id AND display_order = $lastorder_item";
101 101
         $result = Database::query($sql);
102 102
         $row = Database::fetch_array($result);
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
     }
105 105
     $order = $lastorder_item + 1;
106 106
 
107
-    $sql = "INSERT INTO $tbl_lp_item "."(c_id, lp_id,item_type,title,parent_item_id,previous_item_id, next_item_id, display_order) " .
107
+    $sql = "INSERT INTO $tbl_lp_item "."(c_id, lp_id,item_type,title,parent_item_id,previous_item_id, next_item_id, display_order) ".
108 108
             "VALUES "."($course_id, $learnpath_id,'dokeos_chapter','$title', $chapter_id, $previous, 0, $order )";
109 109
     //error_log('New LP - Inserting new resource: '.$sql, 0);
110 110
     $res = Database::query($sql);
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
     if ($from_learnpath == 'yes') {
174 174
         $i = 0;
175 175
         // Calculating the last order of the items of this chapter.
176
-        $sql = "SELECT MAX(display_order) as maxi FROM $tbl_lp_item " .
176
+        $sql = "SELECT MAX(display_order) as maxi FROM $tbl_lp_item ".
177 177
                 "WHERE c_id = $course_id AND lp_id = $learnpath_id AND parent_item_id=$chapter_id";
178 178
         $result = Database::query($sql);
179 179
         $row = Database::fetch_array($result);
@@ -182,7 +182,7 @@  discard block
 block discarded – undo
182 182
             $lastorder_item = 0;
183 183
             $previous = 0;
184 184
         } else {
185
-            $sql = "SELECT id FROM $tbl_lp_item " .
185
+            $sql = "SELECT id FROM $tbl_lp_item ".
186 186
                     "WHERE lp_id = $learnpath_id AND parent_item_id=$chapter_id AND display_order = $lastorder_item";
187 187
             //error_log('New LP - resourcelinker.php - '.$sql, 0);
188 188
             $result = Database::query($sql);
@@ -193,8 +193,8 @@  discard block
 block discarded – undo
193 193
         foreach ($addedresource as $addedresource_item) {
194 194
             // In the case we added a chapter, add this into the chapters list with the correct parent_id.
195 195
             if ($addedresource_item == 'Chap') {
196
-                $sql = "INSERT INTO $tbl_lp_item " .
197
-                        "(c_id, lp_id,item_type,title,parent_item_id,previous_item_id,next_item_id,display_order) " .
196
+                $sql = "INSERT INTO $tbl_lp_item ".
197
+                        "(c_id, lp_id,item_type,title,parent_item_id,previous_item_id,next_item_id,display_order) ".
198 198
                         "VALUES ($course_id, ".$learnpath_id.",'dokeos_chapter','".$learnpath_chapter_name."',".$chapter_id.",$previous,0,".$lastorder.")";
199 199
                 //error_log('New LP - Inserting new resource: '.$sql, 0);
200 200
                 $res = Database::query($sql);
@@ -255,7 +255,7 @@  discard block
 block discarded – undo
255 255
                         //get title from tool-type table
256 256
                         $tooltable = Database::get_course_table(TABLE_DOCUMENT);
257 257
                         $result = Database::query("SELECT * FROM $tooltable WHERE id=".$addedresourceid[$i]);
258
-                        $myrow=Database::fetch_array($result);
258
+                        $myrow = Database::fetch_array($result);
259 259
                         $title = $myrow['title'];
260 260
                         break;
261 261
                     case 'Exercise':
@@ -263,7 +263,7 @@  discard block
 block discarded – undo
263 263
                         //get title from tool-type table
264 264
                         $tooltable = Database::get_course_table(TABLE_QUIZ_TEST);
265 265
                         $result = Database::query("SELECT * FROM $tooltable WHERE id=".$addedresourceid[$i]);
266
-                        $myrow=Database::fetch_array($result);
266
+                        $myrow = Database::fetch_array($result);
267 267
                         $title = $myrow['title'];
268 268
                         break;
269 269
                     case 'Forum':
@@ -275,7 +275,7 @@  discard block
 block discarded – undo
275 275
                         //get title from tool-type table
276 276
                         $tooltable = Database::get_course_table(TABLE_AGENDA);
277 277
                         $result = Database::query("SELECT * FROM $tooltable WHERE id=".$addedresourceid[$i]);
278
-                        $myrow=Database::fetch_array($result);
278
+                        $myrow = Database::fetch_array($result);
279 279
                         $title = $myrow['title'];
280 280
                         break;
281 281
                     case 'Ad_Valvas':
@@ -283,12 +283,12 @@  discard block
 block discarded – undo
283 283
                         //get title from tool-type table
284 284
                         $tooltable = Database::get_course_table(TABLE_ANNOUNCEMENT);
285 285
                         $result = Database::query("SELECT * FROM $tooltable WHERE id=".$addedresourceid[$i]);
286
-                        $myrow=Database::fetch_array($result);
286
+                        $myrow = Database::fetch_array($result);
287 287
                         $title = $myrow['title'];
288 288
                         break;
289 289
 
290 290
                 }
291
-                $sql = "INSERT INTO $tbl_lp_item (c_id, lp_id, title, parent_item_id, item_type, ref, previous_item_id, next_item_id, display_order) " .
291
+                $sql = "INSERT INTO $tbl_lp_item (c_id, lp_id, title, parent_item_id, item_type, ref, previous_item_id, next_item_id, display_order) ".
292 292
                         "VALUES ($course_id, $learnpath_id, '$title','$chapter_id', '$addedresource_item','$addedresourceid[$i]',$previous,0,'".$lastorder."')";
293 293
                 //error_log('New LP - Inserting new resource: '.$sql, 0);
294 294
                 $result = Database::query($sql);
@@ -301,8 +301,8 @@  discard block
 block discarded – undo
301 301
                 $addedresourceassigned[$i] = 1;
302 302
                 $resource_added = true;
303 303
             }
304
-            $i ++;
305
-            $lastorder ++;
304
+            $i++;
305
+            $lastorder++;
306 306
         }
307 307
         //$_SESSION['addedresource']=null;
308 308
         //$_SESSION['addedresourceid']=null;
@@ -330,7 +330,7 @@  discard block
 block discarded – undo
330 330
 }
331 331
 
332 332
 if (!empty($gradebook) && $gradebook == 'view') {
333
-    $interbreadcrumb[] = array (
333
+    $interbreadcrumb[] = array(
334 334
             'url' => '../gradebook/'.$_SESSION['gradebook_dest'],
335 335
             'name' => get_lang('ToolGradebook')
336 336
         );
@@ -348,34 +348,34 @@  discard block
 block discarded – undo
348 348
                 $url = "../calendar/agenda.php?action=add";
349 349
             }
350 350
             $originaltoolname = get_lang('Agenda');
351
-            $breadcrumbelement = array ('url' => $url, 'name' => $originaltoolname);
351
+            $breadcrumbelement = array('url' => $url, 'name' => $originaltoolname);
352 352
             session_unregister('from_learnpath');
353 353
             unset ($from_learnpath);
354 354
             break;
355 355
         case '2': // coming from forum: new topic
356 356
             $url = "../phpbb/newtopic.php?forum=$source_forum&md5=$md5";
357 357
             $originaltoolname = get_lang('ForumAddNewTopic');
358
-            $breadcrumbelement = array ('url' => $url, 'name' => $originaltoolname);
358
+            $breadcrumbelement = array('url' => $url, 'name' => $originaltoolname);
359 359
             session_unregister('from_learnpath');
360 360
             unset ($from_learnpath);
361 361
             break;
362 362
         case '3': // coming from forum: edit topic
363 363
             $url = "../phpbb/editpost.php?post_id=$post_id&topic=$topic&forum=$forum&md5=$md5&originalresource=no";
364 364
             $originaltoolname = get_lang('ForumEditTopic');
365
-            $breadcrumbelement = array ('url' => $url, 'name' => $originaltoolname);
365
+            $breadcrumbelement = array('url' => $url, 'name' => $originaltoolname);
366 366
             session_unregister('from_learnpath');
367 367
             unset ($from_learnpath);
368 368
             break;
369 369
         case '4': // coming from exercises: edit topic
370 370
             $url = "../exercice/admin.php?modifyAnswers=$modifyAnswers";
371 371
             $originaltoolname = get_lang('ExerciseAnswers');
372
-            $breadcrumbelement = array ('url' => $url, 'name' => $originaltoolname);
372
+            $breadcrumbelement = array('url' => $url, 'name' => $originaltoolname);
373 373
             session_unregister('from_learnpath');
374 374
             unset ($from_learnpath);
375 375
             break;
376 376
         case '5': // coming from learning path
377 377
             $from_learnpath = 'yes';
378
-            Session::write('from_learnpath',$from_learnpath);
378
+            Session::write('from_learnpath', $from_learnpath);
379 379
             break;
380 380
 
381 381
         /*  end add [email protected] */
@@ -396,7 +396,7 @@  discard block
 block discarded – undo
396 396
 // the information from the session. Else we use the information of the learningpath itself.
397 397
 if ($from_learnpath != 'yes') {
398 398
     $nameTools = get_lang('Attachment');
399
-    $interbreadcrumb[] = array ('url' => $_SESSION['origintoolurl'], 'name' => $_SESSION['origintoolname']);
399
+    $interbreadcrumb[] = array('url' => $_SESSION['origintoolurl'], 'name' => $_SESSION['origintoolname']);
400 400
 } else {
401 401
     $learnpath_select_query = "	SELECT * FROM $tbl_lp
402 402
                                           WHERE id=$learnpath_id";
@@ -410,9 +410,9 @@  discard block
 block discarded – undo
410 410
 
411 411
     $from_learnpath = 'yes';
412 412
     session_register('from_learnpath');
413
-    $interbreadcrumb[] = array ('url' => "../newscorm/lp_controller.php?action=list", 'name' => get_lang('LearningPath'));
414
-    $interbreadcrumb[] = array ('url' => "../newscorm/lp_controller.php?action=admin_view&lp_id=$learnpath_id", 'name' => stripslashes("{$therow['name']}"));
415
-    $interbreadcrumb[] = array ('url' => api_get_self()."?action=$action&lp_id=$learnpath_id&parent_item_id=$chapter_id&originalresource=no", 'name' => "{$therow2['title']}");
413
+    $interbreadcrumb[] = array('url' => "../newscorm/lp_controller.php?action=list", 'name' => get_lang('LearningPath'));
414
+    $interbreadcrumb[] = array('url' => "../newscorm/lp_controller.php?action=admin_view&lp_id=$learnpath_id", 'name' => stripslashes("{$therow['name']}"));
415
+    $interbreadcrumb[] = array('url' => api_get_self()."?action=$action&lp_id=$learnpath_id&parent_item_id=$chapter_id&originalresource=no", 'name' => "{$therow2['title']}");
416 416
 }
417 417
 
418 418
 $htmlHeadXtra[] = '<script type="text/javascript">
@@ -439,7 +439,7 @@  discard block
 block discarded – undo
439 439
 $tool_table = Database::get_course_table(TABLE_TOOL_LIST);
440 440
 $sql_select_active = "SELECT * FROM $tool_table WHERE visibility='1'";
441 441
 $result_select_active = Database::query($sql_select_active);
442
-while ($row=Database::fetch_array($result_select_active)) {
442
+while ($row = Database::fetch_array($result_select_active)) {
443 443
     $active_modules[] = $row['name'];
444 444
 }
445 445
 ?>
@@ -500,7 +500,7 @@  discard block
 block discarded – undo
500 500
         <!--tr>
501 501
           <td width="26%"><b><?php echo get_lang('ExportableCourseResources'); ?></b></td>
502 502
         </tr-->
503
-<?php if ($multi_level_learnpath === true ) { ?>
503
+<?php if ($multi_level_learnpath === true) { ?>
504 504
         <tr>
505 505
           <td><?php echo "<a href=\"".api_get_self()."?content=chapter&action=$action&id=$id&lp_id=$learnpath_id&parent_item_id=$chapter_id&source_forum=$source_forum&originalresource=no\">".get_lang('Chapter')."</a>"; ?></td>
506 506
         </tr>
@@ -857,9 +857,9 @@  discard block
 block discarded – undo
857 857
 ?>
858 858
     </tr>
859 859
     <tr>
860
-      <td><?php if ($is_allowedToEdit) {echo get_lang('AddToLinks');} ?></td>
860
+      <td><?php if ($is_allowedToEdit) {echo get_lang('AddToLinks'); } ?></td>
861 861
       <td>
862
-        <?php if ($is_allowedToEdit){?>
862
+        <?php if ($is_allowedToEdit) {?>
863 863
       <select name="add_2_links" id="add_2_links">
864 864
       <option value="niet toevoegen" selected="selected">-<?php echo get_lang('DontAdd'); ?>-</option>
865 865
       <option value="0"><?php echo get_lang('MainCategory'); ?></option>
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -341,8 +341,7 @@
 block discarded – undo
341 341
         case '1': // coming from Agenda
342 342
             if ($action == 'edit') {
343 343
                 $url = "../calendar/agenda.php?action=edit&id=49&originalresource=$originalresource";
344
-            }
345
-            elseif ($action == 'add') {
344
+            } elseif ($action == 'add') {
346 345
                 $url = "../calendar/agenda.php?action=add&originalresource=$originalresource";
347 346
             } else {
348 347
                 $url = "../calendar/agenda.php?action=add";
Please login to merge, or discard this patch.
main/newscorm/scorm_api.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -458,8 +458,8 @@
 block discarded – undo
458 458
     } else if(param == 'cmi.core.student_name'){
459 459
         // ---- cmi.core.student_name
460 460
         <?php
461
-          $who = addslashes($user['complete_name']);
462
-          echo "result='$who';";
461
+            $who = addslashes($user['complete_name']);
462
+            echo "result='$who';";
463 463
         ?>
464 464
     } else if(param == 'cmi.core.lesson_location'){
465 465
         // ---- cmi.core.lesson_location
Please login to merge, or discard this patch.
Spacing   +31 added lines, -31 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
 
26 26
 require_once '../inc/global.inc.php';
27 27
 
28
-$file   = (empty($_SESSION['file'])?'':$_SESSION['file']);
28
+$file = (empty($_SESSION['file']) ? '' : $_SESSION['file']);
29 29
 /** @var learnpath $oLP */
30 30
 $oLP = unserialize($_SESSION['lpobject']);
31 31
 /** @var learnpathItem $oItem */
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
 
42 42
 header('Content-type: text/javascript');
43 43
 
44
-?>var scorm_logs=<?php echo ((empty($oLP->scorm_debug) or (!api_is_course_admin() && !api_is_platform_admin()) )?'0':'3');?>; //debug log level for SCORM. 0 = none, 1=light, 2=a lot, 3=all - displays logs in log frame
44
+?>var scorm_logs=<?php echo ((empty($oLP->scorm_debug) or (!api_is_course_admin() && !api_is_platform_admin())) ? '0' : '3'); ?>; //debug log level for SCORM. 0 = none, 1=light, 2=a lot, 3=all - displays logs in log frame
45 45
 var lms_logs = 0; //debug log level for LMS actions. 0=none, 1=light, 2=a lot, 3=all
46 46
 
47 47
 // API Object initialization (eases access later on)
@@ -144,25 +144,25 @@  discard block
 block discarded – undo
144 144
 olms.statusSignalReceived = 0;
145 145
 
146 146
 // Strictly scorm variables
147
-olms.score=<?php echo $oItem->get_score();?>;
148
-olms.max='<?php echo $oItem->get_max();?>';
149
-olms.min='<?php echo $oItem->get_min();?>';
150
-olms.lesson_status='<?php echo $oItem->get_status();?>';
151
-olms.session_time='<?php echo $oItem->get_scorm_time('js');?>';
152
-olms.suspend_data = '<?php echo $oItem->get_suspend_data();?>';
153
-olms.lesson_location = '<?php echo $oItem->get_lesson_location();?>';
154
-olms.total_time = '<?php echo $oItem->get_scorm_time('js');?>';
155
-olms.mastery_score = '<?php echo $oItem->get_mastery_score();?>';
147
+olms.score=<?php echo $oItem->get_score(); ?>;
148
+olms.max='<?php echo $oItem->get_max(); ?>';
149
+olms.min='<?php echo $oItem->get_min(); ?>';
150
+olms.lesson_status='<?php echo $oItem->get_status(); ?>';
151
+olms.session_time='<?php echo $oItem->get_scorm_time('js'); ?>';
152
+olms.suspend_data = '<?php echo $oItem->get_suspend_data(); ?>';
153
+olms.lesson_location = '<?php echo $oItem->get_lesson_location(); ?>';
154
+olms.total_time = '<?php echo $oItem->get_scorm_time('js'); ?>';
155
+olms.mastery_score = '<?php echo $oItem->get_mastery_score(); ?>';
156 156
 olms.launch_data = '<?php echo $oItem->get_launch_data(); ?>';
157
-olms.max_time_allowed = '<?php echo $oItem->get_max_time_allowed();?>';
158
-olms.interactions = new Array(<?php echo $oItem->get_interactions_js_array();?>);
157
+olms.max_time_allowed = '<?php echo $oItem->get_max_time_allowed(); ?>';
158
+olms.interactions = new Array(<?php echo $oItem->get_interactions_js_array(); ?>);
159 159
 olms.item_objectives = new Array();
160 160
 olms.info_lms_item = new Array();
161 161
 
162 162
 // Chamilo internal variables (not SCORM)
163 163
 // olms.saved_lesson_status = 'not attempted';
164
-olms.lms_lp_id = <?php echo $oLP->get_id();?>;
165
-olms.lms_item_id = <?php echo $oItem->get_id();?>;
164
+olms.lms_lp_id = <?php echo $oLP->get_id(); ?>;
165
+olms.lms_item_id = <?php echo $oItem->get_id(); ?>;
166 166
 olms.lms_initialized = 0;
167 167
 // switch_finished indicates if the switch process is finished (if it has gone
168 168
 // through LMSInitialize() for the new item. Until then, all LMSSetValue()
@@ -171,26 +171,26 @@  discard block
 block discarded – undo
171 171
 olms.switch_finished = 0;
172 172
 
173 173
 //olms.lms_total_lessons = <?php echo $oLP->get_total_items_count(); ?>;
174
-//olms.lms_complete_lessons = <?php echo $oLP->get_complete_items_count();?>;
175
-//olms.lms_progress_bar_mode = '<?php echo $oLP->progress_bar_mode;?>';
174
+//olms.lms_complete_lessons = <?php echo $oLP->get_complete_items_count(); ?>;
175
+//olms.lms_progress_bar_mode = '<?php echo $oLP->progress_bar_mode; ?>';
176 176
 //if(lms_progress_bar_mode == ''){lms_progress_bar_mode='%';}
177 177
 
178
-olms.lms_view_id = '<?php echo $oLP->get_view();?>';
178
+olms.lms_view_id = '<?php echo $oLP->get_view(); ?>';
179 179
 if(olms.lms_view_id == ''){ olms.lms_view_id = 1;}
180
-olms.lms_user_id = '<?php echo $userId;?>';
181
-olms.lms_next_item = '<?php echo $oLP->get_next_item_id();?>';
182
-olms.lms_previous_item = '<?php echo $oLP->get_previous_item_id();?>';
183
-olms.lms_lp_type = '<?php echo $oLP->get_type();?>';
184
-olms.lms_item_type = '<?php echo $oItem->get_type();?>';
185
-olms.lms_item_credit = '<?php echo $oItem->get_credit();?>';
186
-olms.lms_item_lesson_mode = '<?php echo $oItem->get_lesson_mode();?>';
187
-olms.lms_item_launch_data = '<?php echo addslashes($oItem->get_launch_data());?>';
188
-olms.lms_item_core_exit = '<?php echo $oItem->get_core_exit();?>';
180
+olms.lms_user_id = '<?php echo $userId; ?>';
181
+olms.lms_next_item = '<?php echo $oLP->get_next_item_id(); ?>';
182
+olms.lms_previous_item = '<?php echo $oLP->get_previous_item_id(); ?>';
183
+olms.lms_lp_type = '<?php echo $oLP->get_type(); ?>';
184
+olms.lms_item_type = '<?php echo $oItem->get_type(); ?>';
185
+olms.lms_item_credit = '<?php echo $oItem->get_credit(); ?>';
186
+olms.lms_item_lesson_mode = '<?php echo $oItem->get_lesson_mode(); ?>';
187
+olms.lms_item_launch_data = '<?php echo addslashes($oItem->get_launch_data()); ?>';
188
+olms.lms_item_core_exit = '<?php echo $oItem->get_core_exit(); ?>';
189 189
 olms.lms_course_id = '<?php echo $oLP->get_course_int_id(); ?>';
190 190
 olms.lms_session_id = '<?php echo api_get_session_id(); ?>';
191 191
 olms.lms_course_code = '<?php echo $oLP->getCourseCode(); ?>';
192 192
 olms.lms_course_id =  '<?php echo $oLP->get_course_int_id(); ?>';
193
-<?php echo $oLP->get_items_details_as_js('olms.lms_item_types');?>
193
+<?php echo $oLP->get_items_details_as_js('olms.lms_item_types'); ?>
194 194
 
195 195
 // Following definition of cmi.core.score.raw in SCORM 1.2, "LMS should
196 196
 // initialize this to an empty string ("") upon initial launch of a SCO. The
@@ -227,8 +227,8 @@  discard block
 block discarded – undo
227 227
     logit_scorm('Other SCORM calls are shown in orange.', 1);
228 228
     logit_lms('To add new messages to these logs, use logit_lms() or logit_scorm().');
229 229
 
230
-    olms.info_lms_item[0] = '<?php echo $oItem->get_id();?>';
231
-    olms.info_lms_item[1] = '<?php echo $oItem->get_id();?>';
230
+    olms.info_lms_item[0] = '<?php echo $oItem->get_id(); ?>';
231
+    olms.info_lms_item[1] = '<?php echo $oItem->get_id(); ?>';
232 232
 
233 233
     $("#content_id").load(function() {
234 234
         logit_lms('#content_id load event starts');
@@ -499,7 +499,7 @@  discard block
 block discarded – undo
499 499
         result = 'id,score,status';
500 500
     } else if(param == 'cmi.objectives._count'){
501 501
     // ---- cmi.objectives._count
502
-        //result='<?php echo $oItem->get_view_count();?>';
502
+        //result='<?php echo $oItem->get_view_count(); ?>';
503 503
         result = olms.item_objectives.length;
504 504
     } else if(param.substring(0,15)== 'cmi.objectives.'){
505 505
         var myres = '';
Please login to merge, or discard this patch.
main/newscorm/lp_add_category.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@
 block discarded – undo
73 73
 
74 74
 echo '<div class="actions">';
75 75
 echo '<a href="lp_controller.php?'.api_get_cidreq().'">'.
76
-    Display::return_icon('back.png', get_lang('ReturnToLearningPaths'),'',ICON_SIZE_MEDIUM).'</a>';
76
+    Display::return_icon('back.png', get_lang('ReturnToLearningPaths'), '', ICON_SIZE_MEDIUM).'</a>';
77 77
 echo '</div>';
78 78
 
79 79
 $form->display();
Please login to merge, or discard this patch.
main/newscorm/lp_edit_item_prereq.php 2 patches
Indentation   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -1,14 +1,14 @@
 block discarded – undo
1 1
 <?php
2 2
 /* For licensing terms, see /license.txt */
3 3
 /**
4
- * This is a learning path creation and player tool in Chamilo - previously learnpath_handler.php
5
- *
6
- * @author Patrick Cool
7
- * @author Denes Nagy
8
- * @author Roan Embrechts, refactoring and code cleaning
9
- * @author Yannick Warnier <[email protected]> - cleaning and update for new SCORM tool
10
- * @package chamilo.learnpath
11
- */
4
+     * This is a learning path creation and player tool in Chamilo - previously learnpath_handler.php
5
+     *
6
+     * @author Patrick Cool
7
+     * @author Denes Nagy
8
+     * @author Roan Embrechts, refactoring and code cleaning
9
+     * @author Yannick Warnier <[email protected]> - cleaning and update for new SCORM tool
10
+     * @package chamilo.learnpath
11
+     */
12 12
 $this_section = SECTION_COURSES;
13 13
 
14 14
 api_protect_course_script();
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
 }
40 40
 if ((!$is_allowed_to_edit) || ($isStudentView)) {
41 41
     error_log('New LP - User not authorized in lp_edit_item_prereq.php');
42
-    header('location:lp_controller.php?action=view&lp_id=' . $learnpath_id);
42
+    header('location:lp_controller.php?action=view&lp_id='.$learnpath_id);
43 43
 }
44 44
 $course_id = api_get_course_int_id();
45 45
 
@@ -55,14 +55,14 @@  discard block
 block discarded – undo
55 55
 
56 56
 if (!empty($gradebook) && $gradebook == 'view') {
57 57
     $interbreadcrumb[] = array(
58
-        'url' => '../gradebook/' . $_SESSION['gradebook_dest'],
58
+        'url' => '../gradebook/'.$_SESSION['gradebook_dest'],
59 59
         'name' => get_lang('ToolGradebook')
60 60
     );
61 61
 }
62 62
 
63 63
 $interbreadcrumb[] = array('url' => 'lp_controller.php?action=list', 'name' => get_lang('LearningPaths'));
64
-$interbreadcrumb[] = array('url' => api_get_self() . "?action=build&lp_id=$learnpath_id", 'name' => stripslashes("{$therow['name']}"));
65
-$interbreadcrumb[] = array('url' => api_get_self() . "?action=add_item&type=step&lp_id=$learnpath_id", 'name' => get_lang('NewStep'));
64
+$interbreadcrumb[] = array('url' => api_get_self()."?action=build&lp_id=$learnpath_id", 'name' => stripslashes("{$therow['name']}"));
65
+$interbreadcrumb[] = array('url' => api_get_self()."?action=add_item&type=step&lp_id=$learnpath_id", 'name' => get_lang('NewStep'));
66 66
 
67 67
 // Theme calls.
68 68
 $show_learn_path = true;
Please login to merge, or discard this patch.
main/newscorm/lp_nav.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -46,9 +46,9 @@
 block discarded – undo
46 46
         $lp_theme_css = $my_style;
47 47
     }
48 48
 
49
-    $progress_bar 	= $_SESSION['oLP']->getProgressBar();
49
+    $progress_bar = $_SESSION['oLP']->getProgressBar();
50 50
     $navigation_bar = $_SESSION['oLP']->get_navigation_bar();
51
-    $mediaplayer 	= $_SESSION['oLP']->get_mediaplayer($autostart);
51
+    $mediaplayer = $_SESSION['oLP']->get_mediaplayer($autostart);
52 52
 }
53 53
 session_write_close();
54 54
 ?>
Please login to merge, or discard this patch.
main/newscorm/lp_report.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
 
107 107
 // View
108 108
 $interbreadcrumb[] = [
109
-    'url' => api_get_path(WEB_CODE_PATH) . 'newscorm/lp_controller.php?'.api_get_cidreq(),
109
+    'url' => api_get_path(WEB_CODE_PATH).'newscorm/lp_controller.php?'.api_get_cidreq(),
110 110
     'name' => get_lang('LearningPaths')
111 111
 ];
112 112
 
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
         array(),
118 118
         ICON_SIZE_MEDIUM
119 119
     ),
120
-    api_get_path(WEB_CODE_PATH) . 'newscorm/lp_controller.php?' . api_get_cidreq()
120
+    api_get_path(WEB_CODE_PATH).'newscorm/lp_controller.php?'.api_get_cidreq()
121 121
 );
122 122
 
123 123
 $template = new Template(get_lang('StudentScore'));
Please login to merge, or discard this patch.
main/newscorm/lp_ajax_save_objectives.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -28,10 +28,10 @@
 block discarded – undo
28 28
         error_log('In xajax_save_objectives('.$lp_id.','.$user_id.','.$view_id.','.$item_id.',"'.(count($objectives) > 0 ? count($objectives) : '').'")', 0);
29 29
     }
30 30
     $mylp = learnpath::getLpFromSession(api_get_course_id(), $lp_id, $user_id);
31
-    $mylpi =& $mylp->items[$item_id];
32
-    if (is_array($objectives) && count($objectives)>0){
33
-        foreach ($objectives as $index=>$objective){
34
-            $mylpi->add_objective($index,$objectives[$index]);
31
+    $mylpi = & $mylp->items[$item_id];
32
+    if (is_array($objectives) && count($objectives) > 0) {
33
+        foreach ($objectives as $index=>$objective) {
34
+            $mylpi->add_objective($index, $objectives[$index]);
35 35
         }
36 36
         $mylpi->write_objectives_to_db();
37 37
     }
Please login to merge, or discard this patch.
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -1,12 +1,12 @@
 block discarded – undo
1 1
 <?php
2 2
 /* For licensing terms, see /license.txt */
3 3
 /**
4
- * This script contains the server part of the xajax interaction process. The client part is located
5
- * in lp_api.php or other api's.
6
- * This is a first attempt at using xajax and AJAX in general, so the code might be a bit unsettling.
7
- * @package chamilo.learnpath
8
- * @author Yannick Warnier <[email protected]>
9
- */
4
+     * This script contains the server part of the xajax interaction process. The client part is located
5
+     * in lp_api.php or other api's.
6
+     * This is a first attempt at using xajax and AJAX in general, so the code might be a bit unsettling.
7
+     * @package chamilo.learnpath
8
+     * @author Yannick Warnier <[email protected]>
9
+     */
10 10
 
11 11
 // Flag to allow for anonymous user - needs to be set before global.inc.php.
12 12
 $use_anonymous = true;
Please login to merge, or discard this patch.
main/cron/cleanup.php 2 patches
Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -1,13 +1,13 @@
 block discarded – undo
1 1
 <?php
2 2
 /**
3
- * Automatic cleanup procedure. Cleans the archive directory of anything
4
- * older than 7 days
5
- * @package chamilo.cron
6
- * @author Yannick Warnier <[email protected]>
7
- */
3
+     * Automatic cleanup procedure. Cleans the archive directory of anything
4
+     * older than 7 days
5
+     * @package chamilo.cron
6
+     * @author Yannick Warnier <[email protected]>
7
+     */
8 8
 /**
9
- * Initialization
10
- */
9
+     * Initialization
10
+     */
11 11
 if (php_sapi_name() != 'cli') { 
12 12
     exit; //do not run from browser
13 13
 }
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -15,9 +15,9 @@  discard block
 block discarded – undo
15 15
 $a_dir = realpath($dir.'/../../archive/');
16 16
 $list = scandir($a_dir);
17 17
 // calculate 7 days
18
-$t = time()-(86400*7);
18
+$t = time() - (86400 * 7);
19 19
 foreach ($list as $item) {
20
-    if (substr($item,0,1) == '.') {
20
+    if (substr($item, 0, 1) == '.') {
21 21
         continue;
22 22
     }
23 23
     $stat = @stat($a_dir.'/'.$item);
@@ -34,11 +34,11 @@  discard block
 block discarded – undo
34 34
  * Delete a file or recursively delete a directory
35 35
  * @param string $str Path to file or directory
36 36
  */
37
-function recursive_delete($str){
37
+function recursive_delete($str) {
38 38
     if (is_file($str)) {
39 39
         return @unlink($str);
40 40
     } elseif (is_dir($str)) {
41
-        $scan = glob(rtrim($str,'/').'/*');
41
+        $scan = glob(rtrim($str, '/').'/*');
42 42
         foreach ($scan as $index=>$path) {
43 43
             recursive_delete($path);
44 44
         }
Please login to merge, or discard this patch.