From aa6315b68491f5f7836db9685f838b841de5b2a4 Mon Sep 17 00:00:00 2001
From: Julio Montoya <gugli100@gmail.com>
Date: Tue, 22 Jan 2013 13:04:06 +0100
Subject: [PATCH] Minor removing old code + fixing code formatting, removing
 whitespaces

---
 main/newscorm/lp_list.php | 315 +++++++++++++++-----------------------
 1 file changed, 120 insertions(+), 195 deletions(-)

diff --git a/main/newscorm/lp_list.php b/main/newscorm/lp_list.php
index 0dc7382a76..d758e29df3 100644
--- a/main/newscorm/lp_list.php
+++ b/main/newscorm/lp_list.php
@@ -2,13 +2,13 @@
 /* For licensing terms, see /license.txt */
 
 /**
-* This file was origially the copy of document.php, but many modifications happened since then ;
-* the direct file view is not any more needed, if the user uploads a scorm zip file, a directory
-* will be automatically created for it, and the files will be uncompressed there for example ;
-*
-* @package chamilo.learnpath
-* @author Yannick Warnier <ywarnier@beeznest.org>
-*/
+ * This file was origially the copy of document.php, but many modifications happened since then ;
+ * the direct file view is not any more needed, if the user uploads a scorm zip file, a directory
+ * will be automatically created for it, and the files will be uncompressed there for example ;
+ *
+ * @package chamilo.learnpath
+ * @author Yannick Warnier <ywarnier@beeznest.org>
+ */
 use \ChamiloSession as Session;
 
 $this_section = SECTION_COURSES;
@@ -18,7 +18,7 @@ if (empty($lp_controller_touched) || $lp_controller_touched != 1) {
 }
 
 require_once 'back_compat.inc.php';
-$courseDir   = api_get_course_path().'/scorm';
+$courseDir = api_get_course_path().'/scorm';
 $baseWordDir = $courseDir;
 
 require_once 'learnpathList.class.php';
@@ -30,7 +30,7 @@ require_once 'learnpathItem.class.php';
  */
 // Extra javascript functions for in html head:
 $htmlHeadXtra[] =
-"<script>
+    "<script>
 function confirmation(name) {
     if (confirm(\" ".trim(get_lang('AreYouSureToDelete'))." \"+name+\"?\"))
         {return true;}
@@ -58,11 +58,10 @@ $current_session = api_get_session_id();
 /* Introduction section (editable by course admins) */
 
 Display::display_introduction_section(TOOL_LEARNPATH, array(
-        'CreateDocumentWebDir' => api_get_path(WEB_COURSE_PATH).api_get_course_path().'/document/',
-        'CreateDocumentDir' => '../../courses/'.api_get_course_path().'/document/',
-        'BaseHref' => api_get_path(WEB_COURSE_PATH).api_get_course_path().'/'
-    )
-);
+    'CreateDocumentWebDir' => api_get_path(WEB_COURSE_PATH).api_get_course_path().'/document/',
+    'CreateDocumentDir' => '../../courses/'.api_get_course_path().'/document/',
+    'BaseHref' => api_get_path(WEB_COURSE_PATH).api_get_course_path().'/'
+));
 
 $is_allowed_to_edit = api_is_allowed_to_edit(null, true);
 
@@ -90,46 +89,22 @@ if ($is_allowed_to_edit) {
     }
 
     echo '<div class="actions">';
-    echo '<a href="'.api_get_self().'?'.api_get_cidreq().'&action=add_lp">'.Display::return_icon('new_learnpath.png', get_lang('LearnpathAddLearnpath'),'',ICON_SIZE_MEDIUM).'</a>' .
-        str_repeat('&nbsp;', 3).
-        '<a href="../upload/index.php?'.api_get_cidreq().'&curdirpath=/&tool='.TOOL_LEARNPATH.'">'.Display::return_icon('import_scorm.png', get_lang('UploadScorm'),'',ICON_SIZE_MEDIUM).'</a>';
+    echo '<a href="'.api_get_self().'?'.api_get_cidreq().'&action=add_lp">'.Display::return_icon('new_learnpath.png', get_lang('LearnpathAddLearnpath'), '', ICON_SIZE_MEDIUM).'</a>'.
+    str_repeat('&nbsp;', 3).
+    '<a href="../upload/index.php?'.api_get_cidreq().'&curdirpath=/&tool='.TOOL_LEARNPATH.'">'.Display::return_icon('import_scorm.png', get_lang('UploadScorm'), '', ICON_SIZE_MEDIUM).'</a>';
     if (api_get_setting('service_ppt2lp', 'active') == 'true') {
         echo str_repeat('&nbsp;', 3).'<a href="../upload/upload_ppt.php?'.api_get_cidreq().'&curdirpath=/&tool='.TOOL_LEARNPATH.'">
-		'.Display::return_icon('import_powerpoint.png', get_lang('PowerPointConvert'),'',ICON_SIZE_MEDIUM).'</a>';
-           //echo  str_repeat('&nbsp;', 3).'<a href="../upload/upload_word.php?'.api_get_cidreq().'&curdirpath=/&tool='.TOOL_LEARNPATH.'"><img src="../img/word.gif" border="0" alt="'.get_lang('WordConvert').'" align="absmiddle">&nbsp;'.get_lang('WordConvert').'</a>';
+		'.Display::return_icon('import_powerpoint.png', get_lang('PowerPointConvert'), '', ICON_SIZE_MEDIUM).'</a>';
     }
     echo '</div>';
 }
 
 /* DISPLAY SCORM LIST */
-$list       = new LearnpathList(api_get_user_id());
-$flat_list  = $list->get_flat_list();
-
+$list = new LearnpathList(api_get_user_id());
+$flat_list = $list->get_flat_list();
 
 if (!empty($flat_list)) {
-
     echo '<table class="data_table">';
-    $is_allowed_to_edit ? $colspan = 9 : $colspan = 3;
-
-    if (!empty($curDirPath)) {
-        if (substr($curDirPath, 1, 1) == '/') {
-            $tmpcurDirPath=substr($curDirPath,1,strlen($curDirPath));
-        } else {
-            $tmpcurDirPath = $curDirPath;
-        }
-        ?>
-        <!-- current dir name -->
-        <tr>
-            <td colspan="<?php echo $colspan; ?>" align="left" bgcolor="#4171B5">
-                <img src="../img/opendir.gif" align="absbottom" vspace="2" hspace="3" alt="open_dir" />
-                <font color="#ffffff"><b><?php echo $tmpcurDirPath; ?></b></font>
-            </td>
-        </tr>
-        <?php
-    }
-
-    /* CURRENT DIRECTORY */
-
     echo '<tr>';
 
     if ($is_allowed_to_edit) {
@@ -145,10 +120,10 @@ if (!empty($flat_list)) {
     }
     echo '</tr>';
 
-    $test_mode      = api_get_setting('server_type');
-    $max            = count($flat_list);
-    $counter        = 0;
-    $current        = 0;
+    $test_mode = api_get_setting('server_type');
+    $max = count($flat_list);
+    $counter = 0;
+    $current = 0;
     $autolunch_exists = false;
     foreach ($flat_list as $id => $details) {
 
@@ -164,12 +139,12 @@ if (!empty($flat_list)) {
         if (!$is_allowed_to_edit && !learnpath::is_lp_visible_for_student($id, api_get_user_id())) {
             continue;
         }
-        $start_time =  $end_time = '';
+        $start_time = $end_time = '';
         if (!$is_allowed_to_edit) {
             $time_limits = false;
 
             //This is an old LP (from a migration 1.8.7) so we do nothing
-            if ((empty($details['created_on']) ||  $details['created_on'] == '0000-00-00 00:00:00') && (empty($details['modified_on']) || $details['modified_on'] == '0000-00-00 00:00:00')) {
+            if ((empty($details['created_on']) || $details['created_on'] == '0000-00-00 00:00:00') && (empty($details['modified_on']) || $details['modified_on'] == '0000-00-00 00:00:00')) {
                 $time_limits = false;
             }
 
@@ -180,53 +155,54 @@ if (!empty($flat_list)) {
             if ($time_limits) {
                 // check if start time
                 if (!empty($details['publicated_on']) && $details['publicated_on'] != '0000-00-00 00:00:00' &&
-                    !empty($details['expired_on'])    && $details['expired_on'] != '0000-00-00 00:00:00') {
-                    $start_time = api_strtotime($details['publicated_on'],'UTC');
-                    $end_time   = api_strtotime($details['expired_on'],'UTC');
-                    $now        = time();
+                    !empty($details['expired_on']) && $details['expired_on'] != '0000-00-00 00:00:00') {
+                    $start_time = api_strtotime($details['publicated_on'], 'UTC');
+                    $end_time = api_strtotime($details['expired_on'], 'UTC');
+                    $now = time();
                     $is_actived_time = false;
 
-                    if ($now > $start_time && $end_time > $now ) {
+                    if ($now > $start_time && $end_time > $now) {
                         $is_actived_time = true;
                     }
 
                     if (!$is_actived_time) {
-                    	continue;
+                        continue;
                     }
                 }
             }
-            $start_time =  $end_time = '';
+            $start_time = $end_time = '';
         } else {
             if (!empty($details['publicated_on'])) {
                 $start_time = api_convert_and_format_date($details['publicated_on'], DATE_TIME_FORMAT_LONG);
             }
             if (!empty($details['expired_on'])) {
-                $end_time   = api_convert_and_format_date($details['expired_on'], DATE_TIME_FORMAT_LONG);
+                $end_time = api_convert_and_format_date($details['expired_on'], DATE_TIME_FORMAT_LONG);
             }
         }
 
         $counter++;
-        if (($counter % 2) == 0) { $oddclass = 'row_odd'; } else { $oddclass = 'row_even'; }
+        if (($counter % 2) == 0) {
+            $oddclass = 'row_odd';
+        } else {
+            $oddclass = 'row_even';
+        }
 
         $url_start_lp = 'lp_controller.php?'.api_get_cidreq().'&action=view&lp_id='.$id;
         $name = Security::remove_XSS($details['lp_name']);
         if ($is_allowed_to_edit) {
             $url_start_lp .= '&isStudentView=true';
-            $dsp_desc = '<em>'.$details['lp_maker'].'</em>   '.(learnpath::is_lp_visible_for_student($id, api_get_user_id())?'':' - ('.get_lang('LPNotVisibleToStudent').')');
-            $extra = '<div class ="lp_content_type_label">'.$dsp_desc .'</div>';
+            $dsp_desc = '<em>'.$details['lp_maker'].'</em>   '.(learnpath::is_lp_visible_for_student($id, api_get_user_id()) ? '' : ' - ('.get_lang('LPNotVisibleToStudent').')');
+            $extra = '<div class ="lp_content_type_label">'.$dsp_desc.'</div>';
         }
 
-        /*$image = '<img src="../img/icons/22/learnpath.png" border="0" align="absmiddle" alt="' . $name . '">';
-        <div style="float: left; width: 35px; height: 22px;"><a href="'.$url_start_lp.'">' .
-                $image . '</a></div>*/
         $my_title = $name;
-        if ($details['lp_visibility'] == 0 ) {
-            $my_title = Display::tag('font', $name, array('style'=>'color:grey'));
+        if ($details['lp_visibility'] == 0) {
+            $my_title = Display::tag('font', $name, array('style' => 'color:grey'));
         }
 
-        $dsp_line =	'<tr align="center" class="'.$oddclass.'">'.
-            		'<td align="left" valign="top">'.Display::return_icon('learnpath.png', get_lang('LPName'),'',ICON_SIZE_SMALL).'
-                     <a href="'.$url_start_lp.'">' . $my_title . '</a>' . $session_img .$extra."</td>";
+        $dsp_line = '<tr align="center" class="'.$oddclass.'">'.
+            '<td align="left" valign="top">'.Display::return_icon('learnpath.png', get_lang('LPName'), '', ICON_SIZE_SMALL).'
+                     <a href="'.$url_start_lp.'">'.$my_title.'</a>'.$session_img.$extra."</td>";
 
         $dsp_desc = '';
         $dsp_export = '';
@@ -244,7 +220,7 @@ if (!empty($flat_list)) {
         if ($is_allowed_to_edit) {
             $dsp_progress = '<td>'.$progress.'</td>';
         } else {
-            $dsp_progress = '<td>'.learnpath::get_progress_bar('%',learnpath::get_db_progress($id, api_get_user_id(), '%', '', false, api_get_session_id())).'</td>';
+            $dsp_progress = '<td>'.learnpath::get_progress_bar('%', learnpath::get_db_progress($id, api_get_user_id(), '%', '', false, api_get_session_id())).'</td>';
         }
 
         $dsp_edit = '<td class="td_actions">';
@@ -252,57 +228,22 @@ if (!empty($flat_list)) {
 
         if ($is_allowed_to_edit) {
 
-            /*
-              if ($current_session == $details['lp_session']) {
-                    $dsp_desc = '<td valign="middle" style="color: grey; padding-top:1em;"><em>'.$details['lp_maker'].'</em>  &nbsp;&nbsp; '.$details['lp_proximity'].' &nbsp;&nbsp; '.$details['lp_encoding'].'<a href="lp_controller.php?'.api_get_cidreq().'&action=edit&lp_id='.$id.'">&nbsp;&nbsp;<img src="../img/edit.gif" border="0" title="'.get_lang('LearnpathEditLearnpath').'"></a></td>'."\n";
-            } else {
-                $dsp_desc = '<td valign="middle" style="color: grey; padding-top:1em;"><em>'.$details['lp_maker'].'</em>  &nbsp;&nbsp; '.$details['lp_proximity'].' &nbsp;&nbsp; '.$details['lp_encoding'].'<img src="../img/edit_na.gif" border="0" title="'.get_lang('LearnpathEditLearnpath').'"></td>'."	";
-            }
-            */
-
-            /* // Deprecated code, Chamilo 1.8.8.
-            $dsp_desc = '<td valign="middle" style="color: grey; padding-top:1em;"><em>'.$details['lp_maker'].'</em>  &nbsp;&nbsp; '.$details['lp_proximity'].' &nbsp;&nbsp; '.$details['lp_encoding'].'</td>'."\n";
-            */
-
-            //$dsp_desc = '<td valign="middle" style="color: grey; padding-top:1em;"><em>'.$details['lp_maker'].'</em>  &nbsp;&nbsp; '.$details['lp_proximity'].'<br />'.(learnpath::is_lp_visible_for_student($id,api_get_user_id())?'':'('.get_lang('LPNotVisibleToStudent').')').'</td>'."\n";
-
-            /* Export */
-
-            // Export is inside "Edit"
-            // export not available for normal lps yet
-            /*if ($details['lp_type'] == 1) {
-                $dsp_export = '<td align="center">' .
-                    "<a href='".api_get_self()."?".api_get_cidreq()."&action=export&lp_id=$id'>" .
-                    "<img src=\"../img/cd.gif\" border=\"0\" title=\"".get_lang('Export')."\">" .
-                    "</a>";
-            } elseif ($details['lp_type'] == 2) {
-                $dsp_export = '<td align="center">' .
-                    "<a href='".api_get_self()."?".api_get_cidreq()."&action=export&lp_id=$id&export_name=".replace_dangerous_char($name,'strict').".zip'>" .
-                    "<img src=\"../img/cd.gif\" border=\"0\" title=\"".get_lang('Export')."\">" .
-                    "</a>";
-            } else {
-                $dsp_export = '<td align="center">' .
-                    //"<a href='".api_get_self()."?".api_get_cidreq()."&action=export&lp_id=$id'>" .
-                    "<img src=\"../img/cd_gray.gif\" border=\"0\" title=\"".get_lang('Export')."\">" .
-                    //"</a>";
-            }*/
-
             // EDIT LP
             if ($current_session == $details['lp_session']) {
-                $dsp_edit_lp = '<a href="lp_controller.php?'.api_get_cidreq().'&action=edit&lp_id='.$id.'">'.Display::return_icon('settings.png', get_lang('CourseSettings'),'',ICON_SIZE_SMALL).'</a>';
+                $dsp_edit_lp = '<a href="lp_controller.php?'.api_get_cidreq().'&action=edit&lp_id='.$id.'">'.Display::return_icon('settings.png', get_lang('CourseSettings'), '', ICON_SIZE_SMALL).'</a>';
             } else {
-                $dsp_edit_lp = Display::return_icon('settings_na.png', get_lang('CourseSettings'),'',ICON_SIZE_SMALL);
+                $dsp_edit_lp = Display::return_icon('settings_na.png', get_lang('CourseSettings'), '', ICON_SIZE_SMALL);
             }
 
             // BUILD
             if ($current_session == $details['lp_session']) {
                 if ($details['lp_type'] == 1 || $details['lp_type'] == 2) {
-                    $dsp_build = '<a href="lp_controller.php?'.api_get_cidreq().'&amp;action=add_item&amp;type=step&amp;lp_id='.$id.'">'.Display::return_icon('edit.png', get_lang('LearnpathEditLearnpath'),'',ICON_SIZE_SMALL).'</a>';
+                    $dsp_build = '<a href="lp_controller.php?'.api_get_cidreq().'&amp;action=add_item&amp;type=step&amp;lp_id='.$id.'">'.Display::return_icon('edit.png', get_lang('LearnpathEditLearnpath'), '', ICON_SIZE_SMALL).'</a>';
                 } else {
-                    $dsp_build = Display::return_icon('edit_na.png', get_lang('LearnpathEditLearnpath'),'',ICON_SIZE_SMALL);
+                    $dsp_build = Display::return_icon('edit_na.png', get_lang('LearnpathEditLearnpath'), '', ICON_SIZE_SMALL);
                 }
             } else {
-                $dsp_build = Display::return_icon('edit_na.png', get_lang('LearnpathEditLearnpath'),'',ICON_SIZE_SMALL);
+                $dsp_build = Display::return_icon('edit_na.png', get_lang('LearnpathEditLearnpath'), '', ICON_SIZE_SMALL);
             }
 
             /* VISIBILITY COMMAND */
@@ -310,9 +251,9 @@ if (!empty($flat_list)) {
             // Session test not necessary if we want to show base course learning paths inside the session (see http://support.chamilo.org/projects/chamilo-18/wiki/Tools_and_sessions).
             //if ($current_session == $details['lp_session']) {
             if ($details['lp_visibility'] == 0) {
-                $dsp_visible =	"<a href=\"".api_get_self()."?".api_get_cidreq()."&lp_id=$id&action=toggle_visible&new_status=1\">".Display::return_icon('invisible.png', get_lang('Show'),'',ICON_SIZE_SMALL)."</a>";
+                $dsp_visible = "<a href=\"".api_get_self()."?".api_get_cidreq()."&lp_id=$id&action=toggle_visible&new_status=1\">".Display::return_icon('invisible.png', get_lang('Show'), '', ICON_SIZE_SMALL)."</a>";
             } else {
-                $dsp_visible =	"<a href='".api_get_self()."?".api_get_cidreq()."&lp_id=$id&action=toggle_visible&new_status=0'>" .Display::return_icon('visible.png', get_lang('Hide'),'',ICON_SIZE_SMALL)."</a>";
+                $dsp_visible = "<a href='".api_get_self()."?".api_get_cidreq()."&lp_id=$id&action=toggle_visible&new_status=0'>".Display::return_icon('visible.png', get_lang('Hide'), '', ICON_SIZE_SMALL)."</a>";
             }
             //} else {
             //	$dsp_visible = '<img src="../img/invisible.gif" border="0" title="'.get_lang('Show').'" />';
@@ -321,103 +262,87 @@ if (!empty($flat_list)) {
             /* PUBLISH COMMAND */
             if ($current_session == $details['lp_session']) {
                 if ($details['lp_published'] == "i") {
-                    $dsp_publish =	"<a href=\"".api_get_self()."?".api_get_cidreq()."&lp_id=$id&action=toggle_publish&new_status=v\">" .
-                    Display::return_icon('lp_publish_na.png', get_lang('LearnpathPublish'),'',ICON_SIZE_SMALL)."</a>";
-
+                    $dsp_publish = "<a href=\"".api_get_self()."?".api_get_cidreq()."&lp_id=$id&action=toggle_publish&new_status=v\">".
+                        Display::return_icon('lp_publish_na.png', get_lang('LearnpathPublish'), '', ICON_SIZE_SMALL)."</a>";
                 } else {
-                    $dsp_publish =	"<a href='".api_get_self()."?".api_get_cidreq()."&lp_id=$id&action=toggle_publish&new_status=i'>" .Display::return_icon('lp_publish.png', get_lang('LearnpathDoNotPublish'),'',ICON_SIZE_SMALL)."</a>";
+                    $dsp_publish = "<a href='".api_get_self()."?".api_get_cidreq()."&lp_id=$id&action=toggle_publish&new_status=i'>".Display::return_icon('lp_publish.png', get_lang('LearnpathDoNotPublish'), '', ICON_SIZE_SMALL)."</a>";
                 }
             } else {
-                $dsp_publish = Display::return_icon('lp_publish_na.png', get_lang('LearnpathDoNotPublish'),'',ICON_SIZE_SMALL);
+                $dsp_publish = Display::return_icon('lp_publish_na.png', get_lang('LearnpathDoNotPublish'), '', ICON_SIZE_SMALL);
             }
 
-      /*  MULTIPLE ATTEMPTS OR SERIOUS GAME MODE
+            /*  MULTIPLE ATTEMPTS OR SERIOUS GAME MODE
 
-        SERIOUSGAME MODE is a special mode where :
-           * If a user exits the learning path before finishing it, he comes back where he left next time he tries
-           * When lp status is completed, user can still modify the attempt (adds/time change score, and browse it)
-           * It is thus a mix betwenn multiple attempt and mono attempt
-      */
+              SERIOUSGAME MODE is a special mode where :
+             * If a user exits the learning path before finishing it, he comes back where he left next time he tries
+             * When lp status is completed, user can still modify the attempt (adds/time change score, and browse it)
+             * It is thus a mix betwenn multiple attempt and mono attempt
+             */
             if ($current_session == $details['lp_session']) {
-              if ($details['seriousgame_mode'] == 1 && $details['lp_prevent_reinit'] == 1) { //seriousgame mode | next = single
-                $dsp_reinit = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_attempt_mode&lp_id='.$id.'">' .
-                    Display::return_icon('reload.png', get_lang('PreventMultipleAttempts'),'',ICON_SIZE_SMALL).
-                  '</a>';
-              }
-              if ($details['seriousgame_mode'] == 0 && $details['lp_prevent_reinit'] == 1) { //single mode | next = multiple
-                $dsp_reinit = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_attempt_mode&lp_id='.$id.'">' .
-                   Display::return_icon('reload_na.png', get_lang('AllowMultipleAttempts'),'',ICON_SIZE_SMALL).
-                  '</a>';
-              }
-              if ($details['seriousgame_mode'] == 0 && $details['lp_prevent_reinit'] == 0) { //multiple mode | next = seriousgame
-                $dsp_reinit = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_attempt_mode&lp_id='.$id.'">' .Display::return_icon('reload.png', get_lang('AllowMultipleAttempts'),'',ICON_SIZE_SMALL).
-                  '</a>';
-              }
+                if ($details['seriousgame_mode'] == 1 && $details['lp_prevent_reinit'] == 1) { //seriousgame mode | next = single
+                    $dsp_reinit = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_attempt_mode&lp_id='.$id.'">'.
+                        Display::return_icon('reload.png', get_lang('PreventMultipleAttempts'), '', ICON_SIZE_SMALL).
+                        '</a>';
+                }
+                if ($details['seriousgame_mode'] == 0 && $details['lp_prevent_reinit'] == 1) { //single mode | next = multiple
+                    $dsp_reinit = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_attempt_mode&lp_id='.$id.'">'.
+                        Display::return_icon('reload_na.png', get_lang('AllowMultipleAttempts'), '', ICON_SIZE_SMALL).
+                        '</a>';
+                }
+                if ($details['seriousgame_mode'] == 0 && $details['lp_prevent_reinit'] == 0) { //multiple mode | next = seriousgame
+                    $dsp_reinit = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_attempt_mode&lp_id='.$id.'">'.Display::return_icon('reload.png', get_lang('AllowMultipleAttempts'), '', ICON_SIZE_SMALL).
+                        '</a>';
+                }
             } else {
-                $dsp_reinit = Display::return_icon('reload_na.png', get_lang('AllowMultipleAttempts'),'',ICON_SIZE_SMALL);
+                $dsp_reinit = Display::return_icon('reload_na.png', get_lang('AllowMultipleAttempts'), '', ICON_SIZE_SMALL);
             }
 
             /* FUll screen VIEW */
             if ($current_session == $details['lp_session']) {
 
-                switch($details['lp_view_mode']) {
+                switch ($details['lp_view_mode']) {
                     case 'fullscreen':
-                    $dsp_default_view = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_view_mode&lp_id='.$id.'">' .
-                        Display::return_icon('view_fullscreen.png', get_lang('ViewModeFullScreen'),'',ICON_SIZE_SMALL).'</a>';
+                        $dsp_default_view = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_view_mode&lp_id='.$id.'">'.
+                            Display::return_icon('view_fullscreen.png', get_lang('ViewModeFullScreen'), '', ICON_SIZE_SMALL).'</a>';
                         break;
                     case 'embedded':
-                        $dsp_default_view = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_view_mode&lp_id='.$id.'">' .
-                         Display::return_icon('view_left_right.png', get_lang('ViewModeEmbedded'),'',ICON_SIZE_SMALL).'</a>';
+                        $dsp_default_view = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_view_mode&lp_id='.$id.'">'.
+                            Display::return_icon('view_left_right.png', get_lang('ViewModeEmbedded'), '', ICON_SIZE_SMALL).'</a>';
                         break;
                     case 'embedframe':
-                        $dsp_default_view = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_view_mode&lp_id='.$id.'">' .
-                        Display::return_icon('view_nofullscreen.png', get_lang('ViewModeEmbedFrame'),'',ICON_SIZE_SMALL).'</a>';
+                        $dsp_default_view = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_view_mode&lp_id='.$id.'">'.
+                            Display::return_icon('view_nofullscreen.png', get_lang('ViewModeEmbedFrame'), '', ICON_SIZE_SMALL).'</a>';
                         break;
                     case 'impress':
-                        $dsp_default_view = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_view_mode&lp_id='.$id.'">' .
-                        Display::return_icon('window_list_slide.png', get_lang('ViewModeImpress'),'',ICON_SIZE_SMALL).'</a>';
+                        $dsp_default_view = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_view_mode&lp_id='.$id.'">'.
+                            Display::return_icon('window_list_slide.png', get_lang('ViewModeImpress'), '', ICON_SIZE_SMALL).'</a>';
                         break;
                 }
             } else {
-                if ($details['lp_view_mode'] == 'fullscreen'){
-                    $dsp_default_view = Display::return_icon('view_fullscreen_na.png', get_lang('ViewModeEmbedded'),'',ICON_SIZE_SMALL);
-				}
-				else{
-                    $dsp_default_view = Display::return_icon('view_left_right_na.png', get_lang('ViewModeEmbedded'),'',ICON_SIZE_SMALL);
-				}
-            }
-
-            /* Increase SCORM recording */
-            /*
-            if ($details['lp_force_commit'] == 1) {
-                $dsp_force_commit = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_force_commit&lp_id='.$id.'">' .
-                    '<img src="../img/clock.gif" border="0" alt="Normal SCORM recordings" title="'.get_lang('MakeScormRecordingNormal').'"/>' .
-                    '</a>&nbsp;';
-            }else{
-                $dsp_force_commit = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_force_commit&lp_id='.$id.'">' .
-                    '<img src="../img/clock_gray.gif" border="0" alt="Extra SCORM recordings" title="'.get_lang('MakeScormRecordingExtra').'"/>' .
-                    '</a>&nbsp;';
+                if ($details['lp_view_mode'] == 'fullscreen') {
+                    $dsp_default_view = Display::return_icon('view_fullscreen_na.png', get_lang('ViewModeEmbedded'), '', ICON_SIZE_SMALL);
+                } else {
+                    $dsp_default_view = Display::return_icon('view_left_right_na.png', get_lang('ViewModeEmbedded'), '', ICON_SIZE_SMALL);
+                }
             }
-            */
 
             /*  DEBUG  */
 
             if ($test_mode == 'test' or api_is_platform_admin()) {
                 if ($details['lp_scorm_debug'] == 1) {
-                    $dsp_debug = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_scorm_debug&lp_id='.$id.'">' .
-                        Display::return_icon('bug.png', get_lang('HideDebug'),'',ICON_SIZE_SMALL).'</a>';
-                }else{
-                    $dsp_debug = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_scorm_debug&lp_id='.$id.'">' .
-					Display::return_icon('bug_na.png', get_lang('ShowDebug'),'',ICON_SIZE_SMALL).'</a>';
+                    $dsp_debug = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_scorm_debug&lp_id='.$id.'">'.
+                        Display::return_icon('bug.png', get_lang('HideDebug'), '', ICON_SIZE_SMALL).'</a>';
+                } else {
+                    $dsp_debug = '<a href="lp_controller.php?'.api_get_cidreq().'&action=switch_scorm_debug&lp_id='.$id.'">'.
+                        Display::return_icon('bug_na.png', get_lang('ShowDebug'), '', ICON_SIZE_SMALL).'</a>';
                 }
-             }
+            }
 
 
-             /* Export */
+            /* Export */
 
             if ($details['lp_type'] == 1) {
                 $dsp_disk = Display::url(Display::return_icon('cd.gif', get_lang('Export'), array(), ICON_SIZE_SMALL), api_get_self()."?".api_get_cidreq()."&action=export&lp_id=$id");
-
             } elseif ($details['lp_type'] == 2) {
                 $dsp_disk = Display::url(Display::return_icon('cd.gif', get_lang('Export'), array(), ICON_SIZE_SMALL), api_get_self()."?".api_get_cidreq()."&action=export&lp_id=$id&export_name=".replace_dangerous_char($name, 'strict').".zip");
             } else {
@@ -427,7 +352,7 @@ if (!empty($flat_list)) {
             //Copy
             $copy = Display::url(Display::return_icon('cd_copy.png', get_lang('Copy'), array(), ICON_SIZE_SMALL), api_get_self()."?".api_get_cidreq()."&action=copy&lp_id=$id");
 
-            /* Auto Lunch LP code*/
+            /* Auto Lunch LP code */
             $lp_auto_lunch_icon = '';
             if (api_get_course_setting('enable_lp_auto_launch') == 1) {
                 if ($details['autolaunch'] == 1 && $autolunch_exists == false) {
@@ -441,18 +366,18 @@ if (!empty($flat_list)) {
             }
 
             //if (api_get_setting('pdf_export_watermark_enable') == 'true') {
-            	  $export_icon = ' <a href="'.api_get_self().'?'.api_get_cidreq().'&action=export_to_pdf&lp_id='.$id.'">
-				  '.Display::return_icon('pdf.png', get_lang('ExportToPDFOnlyHTMLAndImages'),'',ICON_SIZE_SMALL).'</a>';
+            $export_icon = ' <a href="'.api_get_self().'?'.api_get_cidreq().'&action=export_to_pdf&lp_id='.$id.'">
+				  '.Display::return_icon('pdf.png', get_lang('ExportToPDFOnlyHTMLAndImages'), '', ICON_SIZE_SMALL).'</a>';
             //}
 
             /* DELETE COMMAND */
 
             if ($current_session == $details['lp_session']) {
-                $dsp_delete = "<a href=\"lp_controller.php?".api_get_cidreq()."&action=delete&lp_id=$id\" " .
-                "onclick=\"javascript: return confirmation('".addslashes($name)."');\">" .
-				Display::return_icon('delete.png', get_lang('LearnpathDeleteLearnpath'),'',ICON_SIZE_SMALL).'</a>';
+                $dsp_delete = "<a href=\"lp_controller.php?".api_get_cidreq()."&action=delete&lp_id=$id\" ".
+                    "onclick=\"javascript: return confirmation('".addslashes($name)."');\">".
+                    Display::return_icon('delete.png', get_lang('LearnpathDeleteLearnpath'), '', ICON_SIZE_SMALL).'</a>';
             } else {
-                $dsp_delete = Display::return_icon('delete_na.png', get_lang('LearnpathDeleteLearnpath'),'',ICON_SIZE_SMALL);
+                $dsp_delete = Display::return_icon('delete_na.png', get_lang('LearnpathDeleteLearnpath'), '', ICON_SIZE_SMALL);
             }
 
 
@@ -463,36 +388,36 @@ if (!empty($flat_list)) {
             if ($current_session == 0) {
                 if ($details['lp_display_order'] == 1 && $max != 1) {
                     $dsp_order .= '<a href="lp_controller.php?'.api_get_cidreq().'&action=move_lp_down&lp_id='.$id.'">
-                         '.Display::return_icon('down.png', get_lang('MoveDown'),'',ICON_SIZE_SMALL).'</a>
+                         '.Display::return_icon('down.png', get_lang('MoveDown'), '', ICON_SIZE_SMALL).'</a>
 						<img src="../img/blanco.png" border="0" alt="" title="" />';
-                } elseif ($current == $max-1 && $max != 1) {
+                } elseif ($current == $max - 1 && $max != 1) {
                     $dsp_order .= '<img src="../img/blanco.png" border="0" alt="" title="" /><a href="lp_controller.php?'.api_get_cidreq().'&action=move_lp_up&lp_id='.$id.'">
-					'.Display::return_icon('up.png', get_lang('MoveUp'),'',ICON_SIZE_SMALL).'</a>';
+					'.Display::return_icon('up.png', get_lang('MoveUp'), '', ICON_SIZE_SMALL).'</a>';
                 } elseif ($max == 1) {
                     $dsp_order = '';
                 } else {
-                    $dsp_order .= '<a href="lp_controller.php?'.api_get_cidreq().'&action=move_lp_down&lp_id='.$id.'">' .
-                        Display::return_icon('down.png', get_lang('MoveDown'),'',ICON_SIZE_SMALL).'</a>';
-                    $dsp_order .= '<a href="lp_controller.php?'.api_get_cidreq().'&action=move_lp_up&lp_id='.$id.'">' .
-                        Display::return_icon('up.png', get_lang('MoveUp'),'',ICON_SIZE_SMALL).'</a>';
+                    $dsp_order .= '<a href="lp_controller.php?'.api_get_cidreq().'&action=move_lp_down&lp_id='.$id.'">'.
+                        Display::return_icon('down.png', get_lang('MoveDown'), '', ICON_SIZE_SMALL).'</a>';
+                    $dsp_order .= '<a href="lp_controller.php?'.api_get_cidreq().'&action=move_lp_up&lp_id='.$id.'">'.
+                        Display::return_icon('up.png', get_lang('MoveUp'), '', ICON_SIZE_SMALL).'</a>';
                 }
             }
             if ($is_allowed_to_edit) {
-                $start_time = Display::tag('td', Display::div($start_time, array('class'=>'small')));
-                $end_time   = Display::tag('td', Display::div($end_time,   array('class'=>'small')));
+                $start_time = Display::tag('td', Display::div($start_time, array('class' => 'small')));
+                $end_time = Display::tag('td', Display::div($end_time, array('class' => 'small')));
             } else {
-                $start_time  = $end_time= '';
+                $start_time = $end_time = '';
             }
         } else { // end if ($is_allowedToEdit)
             //Student
-            $export_icon = ' <a href="'.api_get_self().'?'.api_get_cidreq().'&action=export_to_pdf&lp_id='.$id.'">'.Display::return_icon('pdf.png', get_lang('ExportToPDF'),'',ICON_SIZE_SMALL).'</a>';
+            $export_icon = ' <a href="'.api_get_self().'?'.api_get_cidreq().'&action=export_to_pdf&lp_id='.$id.'">'.Display::return_icon('pdf.png', get_lang('ExportToPDF'), '', ICON_SIZE_SMALL).'</a>';
         }
 
         echo $dsp_line.$start_time.$end_time.$dsp_progress.$dsp_desc.$dsp_export.$dsp_edit.$dsp_build.$dsp_edit_lp.$dsp_visible.$dsp_publish.$dsp_reinit.
-             $dsp_default_view.$dsp_debug.$dsp_disk.$copy.$lp_auto_lunch_icon.$export_icon.$dsp_delete.$dsp_order.$dsp_edit_close;
+        $dsp_default_view.$dsp_debug.$dsp_disk.$copy.$lp_auto_lunch_icon.$export_icon.$dsp_delete.$dsp_order.$dsp_edit_close;
 
         echo "</tr>";
-        $current ++; //counter for number of elements treated
+        $current++; //counter for number of elements treated
     } // end foreach ($flat_list)
     // TODO: Erint some user-friendly message if counter is still = 0 to tell nothing can be display yet.
     echo "</table>";
@@ -502,7 +427,7 @@ if (!empty($flat_list)) {
         echo '<h2>'.get_lang('LearningPaths').'</h2>';
         echo Display::return_icon('scorms.png', '', array(), 64);
         echo '<div class="controls">';
-        echo Display::url(get_lang('LearnpathAddLearnpath'), api_get_self().'?'.api_get_cidreq().'&action=add_lp' , array('class' => 'btn'));
+        echo Display::url(get_lang('LearnpathAddLearnpath'), api_get_self().'?'.api_get_cidreq().'&action=add_lp', array('class' => 'btn'));
         echo '</div>';
         echo '</div>';
     }