View GitHub Repository chamilo-lms

Schedule Inspection
Subscribe
Passed
Push — 1.10.x ( 2fb41b...669e36 )
by
unknown
52:15
created
Status
Category
Toggle Patches
main/newscorm/lp_list.php 1 patch
Spacing   +33 added lines, -33 removed lines patch added patch discarded remove patch 
@@ -331,7 +331,7 @@  discard block
 
 block discarded – undo
331 331 
                 if ($current_session == $details['lp_session']) {
332 332 
                     $dsp_edit_lp = Display::url(
333 333 
                         Display::return_icon('settings.png', get_lang('CourseSettings'), '', ICON_SIZE_SMALL),
334 
-                        "lp_controller.php?" . api_get_cidreq() . "&action=edit&lp_id=$id"
334 
+                        "lp_controller.php?".api_get_cidreq()."&action=edit&lp_id=$id"
335 335 
                     );
336 336 
                 } else {
337 337 
                     $dsp_edit_lp = Display::return_icon(
 
@@ -347,7 +347,7 @@  discard block
 
 block discarded – undo
347 347 
                     if ($details['lp_type'] == 1 || $details['lp_type'] == 2) {
348 348 
                         $dsp_build = Display::url(
349 349 
                             Display::return_icon('edit.png', get_lang('LearnpathEditLearnpath'), '', ICON_SIZE_SMALL),
350 
-                            'lp_controller.php?' . api_get_cidreq() . '&' . http_build_query([
350 
+                            'lp_controller.php?'.api_get_cidreq().'&'.http_build_query([
351 351 
                                 'action' => 'add_item',
352 352 
                                 'type' => 'step',
353 353 
                                 'lp_id' => $id,
 
@@ -381,19 +381,19 @@  discard block
 
 block discarded – undo
381 381 
                     if ($details['lp_visibility'] == 0) {
382 382 
                         $dsp_visible = Display::url(
383 383 
                             Display::return_icon('invisible.png', get_lang('Show'), '', ICON_SIZE_SMALL),
384 
-                            api_get_self() . '?' . api_get_cidreq() . "&lp_id=$id&action=toggle_visible&new_status=1"
384 
+                            api_get_self().'?'.api_get_cidreq()."&lp_id=$id&action=toggle_visible&new_status=1"
385 385 
                         );
386 386 
                     } else {
387 387 
                         $dsp_visible = Display::url(
388 388 
                             Display::return_icon('visible.png', get_lang('Hide'), '', ICON_SIZE_SMALL),
389 
-                            api_get_self() . '?' . api_get_cidreq() . "&lp_id=$id&action=toggle_visible&new_status=0"
389 
+                            api_get_self().'?'.api_get_cidreq()."&lp_id=$id&action=toggle_visible&new_status=0"
390 390 
                         );
391 391 
                     }
392 392 
                 }
393 393
394 394 
                 //Tracking command
395 395 
                 $trackingActionUrl = 'lp_controller.php?'
396 
-                    . api_get_cidreq() . '&'
396 
+                    . api_get_cidreq().'&'
397 397 
                     . http_build_query([
398 398 
                         'action' => 'report',
399 399 
                         'lp_id' => $id
 
@@ -414,7 +414,7 @@  discard block
 
 block discarded – undo
414 414 
                                 '',
415 415 
                                 ICON_SIZE_SMALL
416 416 
                             ),
417 
-                            api_get_self() . '?' . api_get_cidreq() . "&lp_id=$id&action=toggle_publish&new_status=v"
417 
+                            api_get_self().'?'.api_get_cidreq()."&lp_id=$id&action=toggle_publish&new_status=v"
418 418 
                         );
419 419 
                     } else {
420 420 
                         $dsp_publish = "<a href='".api_get_self()."?".api_get_cidreq()."&lp_id=$id&action=toggle_publish&new_status=i'>".
 
@@ -431,7 +431,7 @@  discard block
 
 block discarded – undo
431 431 
                                 '',
432 432 
                                 ICON_SIZE_SMALL
433 433 
                             ),
434 
-                            api_get_self() . '?' . api_get_cidreq() . "&lp_id=$id&action=toggle_publish&new_status=i"
434 
+                            api_get_self().'?'.api_get_cidreq()."&lp_id=$id&action=toggle_publish&new_status=i"
435 435 
                         );
436 436 
                     }
437 437 
                 } else {
 
@@ -459,7 +459,7 @@  discard block
 
 block discarded – undo
459 459 
                                 '',
460 460 
                                 ICON_SIZE_SMALL
461 461 
                             ),
462 
-                            "lp_controller.php?" . api_get_cidreq() . "&action=switch_attempt_mode&lp_id=$id"
462 
+                            "lp_controller.php?".api_get_cidreq()."&action=switch_attempt_mode&lp_id=$id"
463 463 
                         );
464 464 
                     }
465 465 
                     if ($details['seriousgame_mode'] == 0 && $details['lp_prevent_reinit'] == 1) { //single mode | next = multiple
 
@@ -470,7 +470,7 @@  discard block
 
 block discarded – undo
470 470 
                                 '',
471 471 
                                 ICON_SIZE_SMALL
472 472 
                             ),
473 
-                            "lp_controller.php?" . api_get_cidreq() . "&action=switch_attempt_mode&lp_id=$id"
473 
+                            "lp_controller.php?".api_get_cidreq()."&action=switch_attempt_mode&lp_id=$id"
474 474 
                         );
475 475 
                     }
476 476 
                     if ($details['seriousgame_mode'] == 0 && $details['lp_prevent_reinit'] == 0) { //multiple mode | next = seriousgame
 
@@ -481,7 +481,7 @@  discard block
 
 block discarded – undo
481 481 
                                 '',
482 482 
                                 ICON_SIZE_SMALL
483 483 
                             ),
484 
-                            "lp_controller.php?" . api_get_cidreq() . "&action=switch_attempt_mode&lp_id=$id"
484 
+                            "lp_controller.php?".api_get_cidreq()."&action=switch_attempt_mode&lp_id=$id"
485 485 
                         );
486 486 
                     }
487 487 
                 } else {
 
@@ -504,8 +504,8 @@  discard block
 
 block discarded – undo
504 504 
                                     '',
505 505 
                                     ICON_SIZE_SMALL
506 506 
                                 ),
507 
-                                'lp_controller.php?' . api_get_cidreq()
508 
-                                . '&action=switch_view_mode&lp_id=' . $id . $token_parameter
507 
+                                'lp_controller.php?'.api_get_cidreq()
508 
+                                . '&action=switch_view_mode&lp_id='.$id.$token_parameter
509 509 
                             );
510 510 
                             break;
511 511 
                         case 'embedded':
 
@@ -516,8 +516,8 @@  discard block
 
 block discarded – undo
516 516 
                                     '',
517 517 
                                     ICON_SIZE_SMALL
518 518 
                                 ),
519 
-                                'lp_controller.php?' . api_get_cidreq()
520 
-                                . '&action=switch_view_mode&lp_id=' . $id . $token_parameter
519 
+                                'lp_controller.php?'.api_get_cidreq()
520 
+                                . '&action=switch_view_mode&lp_id='.$id.$token_parameter
521 521 
                             );
522 522 
                             break;
523 523 
                         case 'embedframe':
 
@@ -528,8 +528,8 @@  discard block
 
 block discarded – undo
528 528 
                                     '',
529 529 
                                     ICON_SIZE_SMALL
530 530 
                                 ),
531 
-                                'lp_controller.php?' . api_get_cidreq()
532 
-                                . '&action=switch_view_mode&lp_id=' . $id . $token_parameter
531 
+                                'lp_controller.php?'.api_get_cidreq()
532 
+                                . '&action=switch_view_mode&lp_id='.$id.$token_parameter
533 533 
                             );
534 534 
                             break;
535 535 
                         case 'impress':
 
@@ -540,8 +540,8 @@  discard block
 
 block discarded – undo
540 540 
                                     '',
541 541 
                                     ICON_SIZE_SMALL
542 542 
                                 ),
543 
-                                'lp_controller.php?' . api_get_cidreq()
544 
-                                . '&action=switch_view_mode&lp_id=' . $id . $token_parameter
543 
+                                'lp_controller.php?'.api_get_cidreq()
544 
+                                . '&action=switch_view_mode&lp_id='.$id.$token_parameter
545 545 
                             );
546 546 
                             break;
547 547 
                     }
 
@@ -573,7 +573,7 @@  discard block
 
 block discarded – undo
573 573 
                                 '',
574 574 
                                 ICON_SIZE_SMALL
575 575 
                             ),
576 
-                            "lp_controller.php?" . api_get_cidreq() . "&action=switch_scorm_debug&lp_id=$id"
576 
+                            "lp_controller.php?".api_get_cidreq()."&action=switch_scorm_debug&lp_id=$id"
577 577 
                         );
578 578 
                     } else {
579 579 
                         $dsp_debug = Display::url(
 
@@ -583,7 +583,7 @@  discard block
 
 block discarded – undo
583 583 
                                 '',
584 584 
                                 ICON_SIZE_SMALL
585 585 
                             ),
586 
-                            "lp_controller.php?" . api_get_cidreq() . "&action=switch_scorm_debug&lp_id=$id"
586 
+                            "lp_controller.php?".api_get_cidreq()."&action=switch_scorm_debug&lp_id=$id"
587 587 
                         );
588 588 
                     }
589 589 
                 }
 
@@ -608,7 +608,7 @@  discard block
 
 block discarded – undo
608 608 
                             array(),
609 609 
                             ICON_SIZE_SMALL
610 610 
                         ),
611 
-                        api_get_self()."?".api_get_cidreq()."&action=export&lp_id=$id&export_name=".api_replace_dangerous_char($name, 'strict' ).".zip"
611 
+                        api_get_self()."?".api_get_cidreq()."&action=export&lp_id=$id&export_name=".api_replace_dangerous_char($name, 'strict').".zip"
612 612 
                     );
613 613 
                 } else {
614 614 
                     $dsp_disk = Display::return_icon(
 
@@ -646,12 +646,12 @@  discard block
 
 block discarded – undo
646 646 
                         $autolaunch_exists = true;
647 647 
                         $lp_auto_launch_icon = Display::url(
648 648 
                             Display::return_icon('launch.png', get_lang('DisableLPAutoLaunch')),
649 
-                            api_get_self() . '?' . api_get_cidreq() . "&action=auto_launch&status=0&lp_id=$id"
649 
+                            api_get_self().'?'.api_get_cidreq()."&action=auto_launch&status=0&lp_id=$id"
650 650 
                         );
651 651 
                     } else {
652 652 
                         $lp_auto_launch_icon = Display::url(
653 653 
                             Display::return_icon('launch_na.png', get_lang('EnableLPAutoLaunch')),
654 
-                            api_get_self() . '?' . api_get_cidreq() . "&action=auto_launch&status=1&lp_id=$id"
654 
+                            api_get_self().'?'.api_get_cidreq()."&action=auto_launch&status=1&lp_id=$id"
655 655 
                         );
656 656 
                     }
657 657 
                 }
 
@@ -664,7 +664,7 @@  discard block
 
 block discarded – undo
664 664 
                         '',
665 665 
                         ICON_SIZE_SMALL
666 666 
                     ),
667 
-                    api_get_self() . '?' . api_get_cidreq() . "&action=export_to_pdf&lp_id=$id"
667 
+                    api_get_self().'?'.api_get_cidreq()."&action=export_to_pdf&lp_id=$id"
668 668 
                 );
669 669
670 670 
                 /* Delete */
 
@@ -676,8 +676,8 @@  discard block
 
 block discarded – undo
676 676 
                             '',
677 677 
                             ICON_SIZE_SMALL
678 678 
                         ),
679 
-                        'lp_controller.php?' . api_get_cidreq() . "&action=delete&lp_id=$id",
680 
-                        ['onclick' => "javascript: return confirmation('" . addslashes($name) . "');"]
679 
+                        'lp_controller.php?'.api_get_cidreq()."&action=delete&lp_id=$id",
680 
+                        ['onclick' => "javascript: return confirmation('".addslashes($name)."');"]
681 681 
                     );
682 682 
                 } else {
683 683 
                     $dsp_delete = Display::return_icon(
 
@@ -694,23 +694,23 @@  discard block
 
 block discarded – undo
694 694 
                     if ($details['lp_display_order'] == 1 && $max != 1) {
695 695 
                         $dsp_order .= Display::url(
696 696 
                             Display::return_icon('down.png', get_lang('MoveDown'), '', ICON_SIZE_SMALL),
697 
-                            "lp_controller.php?" . api_get_cidreq() . "&action=move_lp_down&lp_id=$id"
697 
+                            "lp_controller.php?".api_get_cidreq()."&action=move_lp_down&lp_id=$id"
698 698 
                         );
699 699 
                     } elseif ($current == $max - 1 && $max != 1) {
700 700 
                         $dsp_order .= Display::url(
701 701 
                             Display::return_icon('up.png', get_lang('MoveUp'), '', ICON_SIZE_SMALL),
702 
-                            "lp_controller.php?" . api_get_cidreq() . "&action=move_lp_up&lp_id=$id"
702 
+                            "lp_controller.php?".api_get_cidreq()."&action=move_lp_up&lp_id=$id"
703 703 
                         );
704 704 
                     } elseif ($max == 1) {
705 705 
                         $dsp_order = '';
706 706 
                     } else {
707 707 
                         $dsp_order .= Display::url(
708 708 
                             Display::return_icon('down.png', get_lang('MoveDown'), '', ICON_SIZE_SMALL),
709 
-                            "lp_controller.php?" . api_get_cidreq() . "&action=move_lp_down&lp_id=$id"
709 
+                            "lp_controller.php?".api_get_cidreq()."&action=move_lp_down&lp_id=$id"
710 710 
                         );
711 711 
                         $dsp_order .= Display::url(
712 712 
                             Display::return_icon('up.png', get_lang('MoveUp'), '', ICON_SIZE_SMALL),
713 
-                            "lp_controller.php?" . api_get_cidreq() . "&action=move_lp_up&lp_id=$id"
713 
+                            "lp_controller.php?".api_get_cidreq()."&action=move_lp_up&lp_id=$id"
714 714 
                         );
715 715 
                     }
716 716 
                 }
 
@@ -725,7 +725,7 @@  discard block
 
 block discarded – undo
725 725 
                     if ($details['seriousgame_mode'] == 0) {
726 726 
                         $actionSeriousGame = Display::toolbarButton(
727 727 
                             null,
728 
-                            api_get_self() . '?' . api_get_cidreq() . "&lp_id=$id&action=toggle_seriousgame",
728 
+                            api_get_self().'?'.api_get_cidreq()."&lp_id=$id&action=toggle_seriousgame",
729 729 
                             'trophy',
730 730 
                             'default',
731 731 
                             [
 
@@ -736,7 +736,7 @@  discard block
 
 block discarded – undo
736 736 
                     } else {
737 737 
                         $actionSeriousGame = Display::toolbarButton(
738 738 
                             null,
739 
-                            api_get_self() . '?' . api_get_cidreq() . "&lp_id=$id&action=toggle_seriousgame",
739 
+                            api_get_self().'?'.api_get_cidreq()."&lp_id=$id&action=toggle_seriousgame",
740 740 
                             'trophy',
741 741 
                             'warning',
742 742 
                             [
 
@@ -750,7 +750,7 @@  discard block
 
 block discarded – undo
750 750 
                 // Student
751 751 
                 $export_icon = Display::url(
752 752 
                     Display::return_icon('pdf.png', get_lang('ExportToPDF'), '', ICON_SIZE_SMALL),
753 
-                    api_get_self() . '?' . api_get_cidreq() . "&action=export_to_pdf&lp_id=$id"
753 
+                    api_get_self().'?'.api_get_cidreq()."&action=export_to_pdf&lp_id=$id"
754 754 
                 );
755 755 
             }
756 756
Please login to merge, or discard this patch.