chamilo / chamilo-lms

main/newscorm/lp_edit_item.php

Indentation +15 added lines, -15 removed lines

@@ -28,21 +28,21 @@ 
 <script>'.$_SESSION['oLP']->get_js_dropdown_array().
 
 'function load_cbo(id){' ."\n" .
-  'if (!id) {return false;}'.
-  'var cbo = document.getElementById(\'previous\');' .
-  'for(var i = cbo.length - 1; i > 0; i--) {' .
+    'if (!id) {return false;}'.
+    'var cbo = document.getElementById(\'previous\');' .
+    'for(var i = cbo.length - 1; i > 0; i--) {' .
     'cbo.options[i] = null;' .
-  '}' ."\n" .
-  'var k=0;' .
-  'for(var i = 1; i <= child_name[id].length; i++){' ."\n" .
-  '  cbo.options[i] = new Option(child_name[id][i-1], child_value[id][i-1]);' ."\n" .
-  '  k=i;' ."\n" .
-  '}' ."\n" .
-  //'if( typeof cbo != "undefined" ) {'."\n" .
-  'cbo.options[k].selected = true;'."\n" .
-   //'}'."\n" .
-
-   '$(\'#previous\').selectpicker(\'refresh\');' .
+    '}' ."\n" .
+    'var k=0;' .
+    'for(var i = 1; i <= child_name[id].length; i++){' ."\n" .
+    '  cbo.options[i] = new Option(child_name[id][i-1], child_value[id][i-1]);' ."\n" .
+    '  k=i;' ."\n" .
+    '}' ."\n" .
+    //'if( typeof cbo != "undefined" ) {'."\n" .
+    'cbo.options[k].selected = true;'."\n" .
+    //'}'."\n" .
+
+    '$(\'#previous\').selectpicker(\'refresh\');' .
 '}
 
 $(document).on("ready", function() {
@@ -65,7 +65,7 @@ 
 
 // Using the resource linker as a tool for adding resources to the learning path.
 if ($action == 'add' && $type == 'learnpathitem') {
-     $htmlHeadXtra[] = "<script language='JavaScript' type='text/javascript'> window.location=\"../resourcelinker/resourcelinker.php?source_id=5&action=$action&learnpath_id=$learnpath_id&chapter_id=$chapter_id&originalresource=no\"; </script>";
+        $htmlHeadXtra[] = "<script language='JavaScript' type='text/javascript'> window.location=\"../resourcelinker/resourcelinker.php?source_id=5&action=$action&learnpath_id=$learnpath_id&chapter_id=$chapter_id&originalresource=no\"; </script>";
 }
 if ((!$is_allowed_to_edit) || ($isStudentView)) {
     error_log('New LP - User not authorized in lp_add_item.php');

Spacing +16 added lines, -16 removed lines

@@ -27,22 +27,22 @@ 
 $htmlHeadXtra[] = '
 <script>'.$_SESSION['oLP']->get_js_dropdown_array().
 
-'function load_cbo(id){' ."\n" .
+'function load_cbo(id){'."\n".
   'if (!id) {return false;}'.
-  'var cbo = document.getElementById(\'previous\');' .
-  'for(var i = cbo.length - 1; i > 0; i--) {' .
-    'cbo.options[i] = null;' .
-  '}' ."\n" .
-  'var k=0;' .
-  'for(var i = 1; i <= child_name[id].length; i++){' ."\n" .
-  '  cbo.options[i] = new Option(child_name[id][i-1], child_value[id][i-1]);' ."\n" .
-  '  k=i;' ."\n" .
-  '}' ."\n" .
+  'var cbo = document.getElementById(\'previous\');'.
+  'for(var i = cbo.length - 1; i > 0; i--) {'.
+    'cbo.options[i] = null;'.
+  '}'."\n".
+  'var k=0;'.
+  'for(var i = 1; i <= child_name[id].length; i++){'."\n".
+  '  cbo.options[i] = new Option(child_name[id][i-1], child_value[id][i-1]);'."\n".
+  '  k=i;'."\n".
+  '}'."\n".
   //'if( typeof cbo != "undefined" ) {'."\n" .
-  'cbo.options[k].selected = true;'."\n" .
+  'cbo.options[k].selected = true;'."\n".
    //'}'."\n" .
 
-   '$(\'#previous\').selectpicker(\'refresh\');' .
+   '$(\'#previous\').selectpicker(\'refresh\');'.
 '}
 
 $(document).on("ready", function() {
@@ -91,7 +91,7 @@ 
 }
 
 if (!empty($gradebook) && $gradebook == 'view') {
-    $interbreadcrumb[] = array (
+    $interbreadcrumb[] = array(
         'url' => '../gradebook/'.$_SESSION['gradebook_dest'],
         'name' => get_lang('ToolGradebook')
     );
@@ -104,7 +104,7 @@ 
 $show_learn_path = true;
 $lp_theme_css = $_SESSION['oLP']->get_theme();
 
-Display::display_header(get_lang('Edit'),'Path');
+Display::display_header(get_lang('Edit'), 'Path');
 $suredel = trim(get_lang('AreYouSureToDeleteJS'));
 
 ?>
@@ -157,8 +157,8 @@ 
 $path_item = isset($_GET['path_item']) ? $_GET['path_item'] : 0;
 $path_item = Database::escape_string($path_item);
 $tbl_doc = Database :: get_course_table(TABLE_DOCUMENT);
-$sql_doc = "SELECT path FROM " . $tbl_doc . "
-            WHERE c_id = $course_id AND id = '". $path_item."' ";
+$sql_doc = "SELECT path FROM ".$tbl_doc."
+            WHERE c_id = $course_id AND id = '".$path_item."' ";
 
 $res_doc = Database::query($sql_doc);
 $path_file = Database::result($res_doc, 0, 0);

main/newscorm/lp_add.php

Spacing +11 added lines, -11 removed lines

@@ -68,12 +68,12 @@ 
     Course admin section
     - all the functions not available for students - always available in this case (page only shown to admin)
 */
-if (isset($_SESSION['gradebook'])){
+if (isset($_SESSION['gradebook'])) {
     $gradebook = $_SESSION['gradebook'];
 }
 
-if (!empty($gradebook) && $gradebook=='view') {
-    $interbreadcrumb[]= array (
+if (!empty($gradebook) && $gradebook == 'view') {
+    $interbreadcrumb[] = array(
         'url' => '../gradebook/'.$_SESSION['gradebook_dest'],
         'name' => get_lang('ToolGradebook')
     );
@@ -85,7 +85,7 @@ 
 
 echo '<div class="actions">';
 echo '<a href="lp_controller.php?cidReq='.$_course['sysCode'].'">'.
-        Display::return_icon('back.png', get_lang('ReturnToLearningPaths'),'',ICON_SIZE_MEDIUM).'</a>';
+        Display::return_icon('back.png', get_lang('ReturnToLearningPaths'), '', ICON_SIZE_MEDIUM).'</a>';
 echo '</div>';
 
 Display::display_normal_message(get_lang('AddLpIntro'), false);
@@ -119,22 +119,22 @@ 
 
 // Start date
 $form->addElement('checkbox', 'activate_start_date_check', null, get_lang('EnableStartTime'), array('onclick' => 'activate_start_date()'));
-$form->addElement('html','<div id="start_date_div" style="display:block;">');
+$form->addElement('html', '<div id="start_date_div" style="display:block;">');
 $form->addElement('DatePicker', 'publicated_on', get_lang('PublicationDate'));
-$form->addElement('html','</div>');
+$form->addElement('html', '</div>');
 
 //End date
 $form->addElement('checkbox', 'activate_end_date_check', null, get_lang('EnableEndTime'), array('onclick' => 'activate_end_date()'));
-$form->addElement('html','<div id="end_date_div" style="display:none;">');
+$form->addElement('html', '<div id="end_date_div" style="display:none;">');
 $form->addElement('DatePicker', 'expired_on', get_lang('ExpirationDate'));
-$form->addElement('html','</div>');
+$form->addElement('html', '</div>');
 
-$form->addElement('html','</div>');
+$form->addElement('html', '</div>');
 
-$defaults['activate_start_date_check']  = 1;
+$defaults['activate_start_date_check'] = 1;
 
 $defaults['publicated_on'] = date('Y-m-d 08:00:00');
-$defaults['expired_on'] = date('Y-m-d 08:00:00',time()+86400);
+$defaults['expired_on'] = date('Y-m-d 08:00:00', time() + 86400);
 
 $form->setDefaults($defaults);
 $form->addButtonCreate(get_lang('CreateLearningPath'));

main/newscorm/audiorecorder.inc.php

Indentation +5 added lines, -5 removed lines

@@ -58,11 +58,11 @@
 if (!empty($path_to_lzx)) {
     $recorder_content = sprintf("<object type=\"application/x-shockwave-flash\" data=\"%s\" ".
             "width='$width' height='$height'>".
-             "<param name=\"movie\" value=\"%s\">".
-         "<param name=\"quality\" value=\"high\">".
-         "<param name=\"scale\" value=\"noscale\">".
-         "<param name=\"salign\" value=\"LT\">".
-         "<param name=\"menu\" value=\"false\"></object>", $path_to_lzx, $path_to_lzx);
+                "<param name=\"movie\" value=\"%s\">".
+            "<param name=\"quality\" value=\"high\">".
+            "<param name=\"scale\" value=\"noscale\">".
+            "<param name=\"salign\" value=\"LT\">".
+            "<param name=\"menu\" value=\"false\"></object>", $path_to_lzx, $path_to_lzx);
     if ($audio_recorder_studentview == 'false') {
         echo '<script type="text/javascript">
         function show_audiorecorder() {

Spacing +3 added lines, -3 removed lines

@@ -30,7 +30,7 @@ 
 
 if ($audio_recorder_studentview == 'false') {
     $width = 295;
-    $height= 90;
+    $height = 90;
     $player = 'recorder2.swf';
     $server = (api_get_setting('service_visio', 'visio_use_rtmpt') == 'true' ? 'rtmpt://' : 'rtmp://').api_get_setting('service_visio', 'visio_host').':'.(api_get_setting('service_visio', 'visio_use_rtmpt') == 'true' ? '80' : api_get_setting('service_visio', 'visio_port')).'/recorder';
     $post_uri = urlencode($web_path.'conference/audiopost.php?course_code='.api_get_course_id().'&user_id='.api_get_user_id().'&checker='.md5(api_get_course_id().api_get_user_id().gmdate('Ymd').$_configuration['security_key']));
@@ -42,8 +42,8 @@ 
     $player = 'player2.swf';
     $cp = api_get_course_path();
     $docs = Database::get_course_table(TABLE_DOCUMENT);
-    $select = "SELECT * FROM $docs " .
-            " WHERE c_id = $course_id AND path like BINARY '/audio/lpi".intval($audio_recorder_item_id)."-%' AND filetype='file' " .
+    $select = "SELECT * FROM $docs ".
+            " WHERE c_id = $course_id AND path like BINARY '/audio/lpi".intval($audio_recorder_item_id)."-%' AND filetype='file' ".
             " ORDER BY path DESC";
     $res = Database::query($select);
     if (Database::num_rows($res) > 0) {

main/newscorm/lp_add_item.php

Indentation +15 added lines, -15 removed lines

@@ -27,21 +27,21 @@ 
 $learnPath->get_js_dropdown_array() .
 
 'function load_cbo(id){' ."\n" .
-  'if (!id) {return false;}'.
-  'var cbo = document.getElementById(\'previous\');' .
-  'for(var i = cbo.length - 1; i > 0; i--) {' .
+    'if (!id) {return false;}'.
+    'var cbo = document.getElementById(\'previous\');' .
+    'for(var i = cbo.length - 1; i > 0; i--) {' .
     'cbo.options[i] = null;' .
-  '}' ."\n" .
-  'var k=0;' .
-  'for(var i = 1; i <= child_name[id].length; i++){' ."\n" .
-  '  cbo.options[i] = new Option(child_name[id][i-1], child_value[id][i-1]);' ."\n" .
-  '  k=i;' ."\n" .
-  '}' ."\n" .
-  //'if( typeof cbo != "undefined" ) {'."\n" .
-  'cbo.options[k].selected = true;'."\n" .
-   //'}'."\n" .
-
-   '$(\'#previous\').selectpicker(\'refresh\');' .
+    '}' ."\n" .
+    'var k=0;' .
+    'for(var i = 1; i <= child_name[id].length; i++){' ."\n" .
+    '  cbo.options[i] = new Option(child_name[id][i-1], child_value[id][i-1]);' ."\n" .
+    '  k=i;' ."\n" .
+    '}' ."\n" .
+    //'if( typeof cbo != "undefined" ) {'."\n" .
+    'cbo.options[k].selected = true;'."\n" .
+    //'}'."\n" .
+
+    '$(\'#previous\').selectpicker(\'refresh\');' .
 '}
 
 $(function() {
@@ -73,7 +73,7 @@ 
 
 // Using the resource linker as a tool for adding resources to the learning path.
 if ($action == 'add' && $type == 'learnpathitem') {
-     $htmlHeadXtra[] = "<script type='text/javascript'> window.location=\"../resourcelinker/resourcelinker.php?source_id=5&action=$action&learnpath_id=$learnpath_id&chapter_id=$chapter_id&originalresource=no\"; </script>";
+        $htmlHeadXtra[] = "<script type='text/javascript'> window.location=\"../resourcelinker/resourcelinker.php?source_id=5&action=$action&learnpath_id=$learnpath_id&chapter_id=$chapter_id&originalresource=no\"; </script>";
 }
 if ((!$is_allowed_to_edit)) {
     error_log('New LP - User not authorized in lp_add_item.php');

Spacing +21 added lines, -21 removed lines

@@ -24,24 +24,24 @@ 
 
 $htmlHeadXtra[] = '<script>'.
 
-$learnPath->get_js_dropdown_array() .
+$learnPath->get_js_dropdown_array().
 
-'function load_cbo(id){' ."\n" .
+'function load_cbo(id){'."\n".
   'if (!id) {return false;}'.
-  'var cbo = document.getElementById(\'previous\');' .
-  'for(var i = cbo.length - 1; i > 0; i--) {' .
-    'cbo.options[i] = null;' .
-  '}' ."\n" .
-  'var k=0;' .
-  'for(var i = 1; i <= child_name[id].length; i++){' ."\n" .
-  '  cbo.options[i] = new Option(child_name[id][i-1], child_value[id][i-1]);' ."\n" .
-  '  k=i;' ."\n" .
-  '}' ."\n" .
+  'var cbo = document.getElementById(\'previous\');'.
+  'for(var i = cbo.length - 1; i > 0; i--) {'.
+    'cbo.options[i] = null;'.
+  '}'."\n".
+  'var k=0;'.
+  'for(var i = 1; i <= child_name[id].length; i++){'."\n".
+  '  cbo.options[i] = new Option(child_name[id][i-1], child_value[id][i-1]);'."\n".
+  '  k=i;'."\n".
+  '}'."\n".
   //'if( typeof cbo != "undefined" ) {'."\n" .
-  'cbo.options[k].selected = true;'."\n" .
+  'cbo.options[k].selected = true;'."\n".
    //'}'."\n" .
 
-   '$(\'#previous\').selectpicker(\'refresh\');' .
+   '$(\'#previous\').selectpicker(\'refresh\');'.
 '}
 
 $(function() {
@@ -66,7 +66,7 @@ 
 
 $isStudentView  = isset($_REQUEST['isStudentView']) ? $_REQUEST['isStudentView'] : null;
 $learnpath_id   = isset($_REQUEST['lp_id']) ? intval($_REQUEST['lp_id']) : null;
-$submit			= isset($_POST['submit_button']) ? $_POST['submit_button'] : null;
+$submit = isset($_POST['submit_button']) ? $_POST['submit_button'] : null;
 
 $type = isset($_GET['type']) ? $_GET['type'] : null;
 $action = isset($_GET['action']) ? $_GET['action'] : null;
@@ -87,7 +87,7 @@ 
 }
 
 if (!empty($gradebook) && $gradebook == 'view') {
-    $interbreadcrumb[] = array (
+    $interbreadcrumb[] = array(
         'url' => '../gradebook/'.$_SESSION['gradebook_dest'],
         'name' => get_lang('ToolGradebook')
     );
@@ -98,19 +98,19 @@ 
 
 switch ($type) {
     case 'chapter':
-        $interbreadcrumb[]= array('url' => 'lp_controller.php?action=add_item&type=step&lp_id='.$learnPath->get_id(), 'name' => get_lang('NewStep'));
-        $interbreadcrumb[]= array('url' => '#', 'name' => get_lang('NewChapter'));
+        $interbreadcrumb[] = array('url' => 'lp_controller.php?action=add_item&type=step&lp_id='.$learnPath->get_id(), 'name' => get_lang('NewStep'));
+        $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('NewChapter'));
         break;
     case 'document':
-        $interbreadcrumb[]= array('url' => 'lp_controller.php?action=add_item&type=step&lp_id='.$learnPath->get_id(), 'name' => get_lang('NewStep'));
+        $interbreadcrumb[] = array('url' => 'lp_controller.php?action=add_item&type=step&lp_id='.$learnPath->get_id(), 'name' => get_lang('NewStep'));
         break;
     default:
-        $interbreadcrumb[]= array('url' => '#', 'name' => get_lang('NewStep'));
+        $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('NewStep'));
         break;
 }
 
-if ($action == 'add_item' && $type == 'document' ) {
-    $interbreadcrumb[]= array ('url' => '#', 'name' => get_lang('NewDocumentCreated'));
+if ($action == 'add_item' && $type == 'document') {
+    $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('NewDocumentCreated'));
 }
 
 // Theme calls.

main/newscorm/lp_list_search.php

Spacing +18 added lines, -18 removed lines

@@ -15,15 +15,15 @@ 
 require_once api_get_path(LIBRARY_PATH).'search/IndexableChunk.class.php';
 require_once api_get_path(LIBRARY_PATH).'specific_fields_manager.lib.php';
 
-$htmlHeadXtra[] = '<link rel="stylesheet" type="text/css" href="'. api_get_path(WEB_PATH) .'main/newscorm/lp_list_search.css" />';
+$htmlHeadXtra[] = '<link rel="stylesheet" type="text/css" href="'.api_get_path(WEB_PATH).'main/newscorm/lp_list_search.css" />';
 Event::event_access_tool(TOOL_SEARCH);
 
-if (isset($_SESSION['gradebook'])){
+if (isset($_SESSION['gradebook'])) {
     $gradebook = $_SESSION['gradebook'];
 }
 
 if (!empty($gradebook) && $gradebook == 'view') {
-    $interbreadcrumb[]= array (
+    $interbreadcrumb[] = array(
         'url' => '../gradebook/'.$_SESSION['gradebook_dest'],
         'name' => get_lang('ToolGradebook')
     );
@@ -59,8 +59,8 @@ 
 $term_array = array();
 $specific_fields = get_specific_field_list();
 foreach ($specific_fields as $specific_field) {
-    if (!empty($_REQUEST[ 'sf_'. $specific_field['code'] ])) {
-        $values = $_REQUEST[ 'sf_'. $specific_field['code'] ];
+    if (!empty($_REQUEST['sf_'.$specific_field['code']])) {
+        $values = $_REQUEST['sf_'.$specific_field['code']];
         if (in_array('__all__', $values)) {
             $sf_terms_for_code = xapian_get_all_terms(1000, $specific_field['code']);
             foreach ($sf_terms_for_code as $term) {
@@ -72,7 +72,7 @@ 
             foreach ($values as $term) {
                 if (!empty($term)) {
                     $prefix = $specific_field['code'];
-                    $term_array[] = chamilo_get_boolean_query($prefix . $term);
+                    $term_array[] = chamilo_get_boolean_query($prefix.$term);
                 }
             }
         }
@@ -90,9 +90,9 @@ 
 // Get right group of terms to show on multiple select.
 $fixed_queries = array();
 $course_filter = null;
-if ( ($cid=api_get_course_id()) != -1 ) {
+if (($cid = api_get_course_id()) != -1) {
     // Results only from actual course.
-    $course_filter = chamilo_get_boolean_query(XAPIAN_PREFIX_COURSEID . $cid);
+    $course_filter = chamilo_get_boolean_query(XAPIAN_PREFIX_COURSEID.$cid);
 }
 
 if (count($term_array)) {
@@ -137,12 +137,12 @@ 
         }
 
         if ($mode == 'gallery') {
-            $title = $a_prefix.str_replace('_',' ',$result['title']). $a_sufix;
+            $title = $a_prefix.str_replace('_', ' ', $result['title']).$a_sufix;
             $blocks[] = array(
-                $a_prefix .'<img src="'.$result['thumbnail'].'" />'. $a_sufix .'<br />'.$title.'<br />'.$result['author'],
+                $a_prefix.'<img src="'.$result['thumbnail'].'" />'.$a_sufix.'<br />'.$title.'<br />'.$result['author'],
             );
         } else {
-            $title = '<div style="text-align:left;">'. $a_prefix . $result['title']. $a_sufix .(!empty($result['author']) ? ' '.$result['author'] : '').'<div>';
+            $title = '<div style="text-align:left;">'.$a_prefix.$result['title'].$a_sufix.(!empty($result['author']) ? ' '.$result['author'] : '').'<div>';
             $blocks[] = array($title);
         }
     }
@@ -154,19 +154,19 @@ 
     $s->display_mode = $mode; // default
     $s->display_mode_params = 3;
     $s->per_page = 9;
-    $additional_parameters = array (
+    $additional_parameters = array(
         'mode' => $mode,
         'action' => 'search',
         'query' => Security::remove_XSS($_REQUEST['query']),
     );
     $get_params = '';
     foreach ($specific_fields as $specific_field) {
-        if (isset($_REQUEST[ 'sf_'. $specific_field['code'] ])) {
-            $values = $_REQUEST[ 'sf_'. $specific_field['code'] ];
+        if (isset($_REQUEST['sf_'.$specific_field['code']])) {
+            $values = $_REQUEST['sf_'.$specific_field['code']];
             //Sortable additional_parameters doesn't accept multi dimensional arrays
             //$additional_parameters[ 'sf_'. $specific_field['code'] ] = $values;
-            foreach ( $values as $value ) {
-                $get_params .= '&sf_' . $specific_field['code'] .'[]='. $value;
+            foreach ($values as $value) {
+                $get_params .= '&sf_'.$specific_field['code'].'[]='.$value;
             }
             $get_params .= '&';
         }
@@ -183,9 +183,9 @@ 
 
     $mode_selector = '<div id="mode-selector">';
     $mode_selector .= sprintf($search_link, api_get_path(WEB_CODE_PATH), 'gallery', $_REQUEST['query'], $get_params);
-    $mode_selector .= '<img src="../img/'. (($mode=='gallery')?'ButtonGallOn':'ButtonGallOff') .'.png" /></a>';
+    $mode_selector .= '<img src="../img/'.(($mode == 'gallery') ? 'ButtonGallOn' : 'ButtonGallOff').'.png" /></a>';
     $mode_selector .= sprintf($search_link, api_get_path(WEB_CODE_PATH), 'default', $_REQUEST['query'], $get_params);
-    $mode_selector .= '<img src="../img/'.(($mode=='default')?'ButtonListOn':'ButtonListOff').'.png" /></a>';
+    $mode_selector .= '<img src="../img/'.(($mode == 'default') ? 'ButtonListOn' : 'ButtonListOff').'.png" /></a>';
     $mode_selector .= '</div>';
 
     echo '<div id="search-results-container">';

main/newscorm/lp_build.php

Indentation +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
 
 // Using the resource linker as a tool for adding resources to the learning path.
 if ($action=="add" and $type=="learnpathitem") {
-     $htmlHeadXtra[] = "<script language='JavaScript' type='text/javascript'> window.location=\"../resourcelinker/resourcelinker.php?source_id=5&action=$action&learnpath_id=$learnpath_id&chapter_id=$chapter_id&originalresource=no\"; </script>";
+        $htmlHeadXtra[] = "<script language='JavaScript' type='text/javascript'> window.location=\"../resourcelinker/resourcelinker.php?source_id=5&action=$action&learnpath_id=$learnpath_id&chapter_id=$chapter_id&originalresource=no\"; </script>";
 }
 if ((!$is_allowed_to_edit) || ($isStudentView)) {
     error_log('New LP - User not authorized in lp_build.php');

Spacing +17 added lines, -17 removed lines

@@ -29,12 +29,12 @@ 
 
 $isStudentView  = (int) $_REQUEST['isStudentView'];
 $learnpath_id   = (int) $_REQUEST['lp_id'];
-$submit			= $_POST['submit_button'];
+$submit = $_POST['submit_button'];
 
 /* MAIN CODE */
 
 // Using the resource linker as a tool for adding resources to the learning path.
-if ($action=="add" and $type=="learnpathitem") {
+if ($action == "add" and $type == "learnpathitem") {
      $htmlHeadXtra[] = "<script language='JavaScript' type='text/javascript'> window.location=\"../resourcelinker/resourcelinker.php?source_id=5&action=$action&learnpath_id=$learnpath_id&chapter_id=$chapter_id&originalresource=no\"; </script>";
 }
 if ((!$is_allowed_to_edit) || ($isStudentView)) {
@@ -51,7 +51,7 @@ 
 if ($learnpath_id == 0) {
     $is_new = true;
 
-    $sql        = "SELECT id FROM " . $tbl_lp . " WHERE c_id = $course_id ORDER BY id DESC LIMIT 0, 1";
+    $sql        = "SELECT id FROM ".$tbl_lp." WHERE c_id = $course_id ORDER BY id DESC LIMIT 0, 1";
     $result     = Database::query($sql);
     $row        = Database::fetch_array($result);
     $learnpath_id = $row['id'];
@@ -73,8 +73,8 @@ 
 }
 
 if (!empty($gradebook) && $gradebook == 'view') {
-    $interbreadcrumb[] = array (
-        'url' => '../gradebook/' . $_SESSION['gradebook_dest'],
+    $interbreadcrumb[] = array(
+        'url' => '../gradebook/'.$_SESSION['gradebook_dest'],
         'name' => get_lang('ToolGradebook')
     );
 }
@@ -82,7 +82,7 @@ 
 $interbreadcrumb[] = array('url' => '#', "name" => $therow['name']);
 
 // Theme calls.
-$lp_theme_css=$_SESSION['oLP']->get_theme();
+$lp_theme_css = $_SESSION['oLP']->get_theme();
 $show_learn_path = true;
 Display::display_header('', 'Path');
 $suredel = trim(get_lang('AreYouSureToDeleteJS'));
@@ -132,11 +132,11 @@ 
 
     echo '<ul id="lp_overview" class="thumbnails">';
 
-    echo show_block('lp_controller.php?'.api_get_cidreq().'&gradebook='.$gradebook.'&action=add_item&type=step&lp_id=' . $_SESSION['oLP']->lp_id, get_lang("NewStep"), get_lang('NewStepComment'), 'tools.png');
+    echo show_block('lp_controller.php?'.api_get_cidreq().'&gradebook='.$gradebook.'&action=add_item&type=step&lp_id='.$_SESSION['oLP']->lp_id, get_lang("NewStep"), get_lang('NewStepComment'), 'tools.png');
 
 //    echo show_block('lp_controller.php?'.api_get_cidreq().'&gradebook='.$gradebook.'&action=admin_view&updateaudio=true&lp_id=' . $_SESSION['oLP']->lp_id, get_lang("BasicOverview"), get_lang('BasicOverviewComment'), 'audio.png');
 
-    echo show_block('lp_controller.php?'.api_get_cidreq().'&gradebook='.$gradebook.'&action=view&lp_id=' . $_SESSION['oLP']->lp_id, get_lang("Display"), get_lang('DisplayComment'), 'view.png');
+    echo show_block('lp_controller.php?'.api_get_cidreq().'&gradebook='.$gradebook.'&action=view&lp_id='.$_SESSION['oLP']->lp_id, get_lang("Display"), get_lang('DisplayComment'), 'view.png');
 
     //echo show_block('lp_controller.php?'.api_get_cidreq().'&gradebook='.$gradebook.'&action=edit&lp_id=' . $_SESSION['oLP']->lp_id, get_lang("Settings"), null, 'reference.png');
 
@@ -148,15 +148,15 @@ 
 
 function show_block($link, $title, $subtitle, $icon) {
     $html = '<li class="col-md-4">';
-        $html .=  '<div class="thumbnail">';
-        $html .=  '<a href="'.$link.'" title="'.$title.'">';
-        $html .=  Display::return_icon($icon, $title, array(), ICON_SIZE_BIG);
-        $html .=  '</a>';
-        $html .=  '<div class="caption">';
-        $html .=  '<strong>'.$title.'</strong></a> '.$subtitle;
-        $html .=  '</div>';
-        $html .=  '</div>';
-    $html .=  '</li>';
+        $html .= '<div class="thumbnail">';
+        $html .= '<a href="'.$link.'" title="'.$title.'">';
+        $html .= Display::return_icon($icon, $title, array(), ICON_SIZE_BIG);
+        $html .= '</a>';
+        $html .= '<div class="caption">';
+        $html .= '<strong>'.$title.'</strong></a> '.$subtitle;
+        $html .= '</div>';
+        $html .= '</div>';
+    $html .= '</li>';
     return $html;
 }
 

main/newscorm/aicc_hacp.php

Spacing +3 added lines, -3 removed lines

@@ -55,7 +55,7 @@ 
 //$_user							= $_SESSION['_user'];
 $file = $_SESSION['file'];
 $oLP = unserialize($_SESSION['lpobject']);
-$oItem =& $oLP->items[$oLP->current];
+$oItem = & $oLP->items[$oLP->current];
 if (!is_object($oItem)) {
     error_log('New LP - aicc_hacp - Could not load oItem item', 0);
     exit;
@@ -69,8 +69,8 @@ 
     'evaluation' => array(),
     'student_data' => array(),
 );
-$convert_enc = array('%25','%0D','%0A','%09','%20','%2D','%2F','%3B','%3F','%7B','%7D','%7C','%5C','%5E','%7E','%5B','%5D','%60','%23','%3E','%3C','%22');
-$convert_dec = array('%',"\r","\n","\t",' ','-','/',';','?','{','}','|','\\','^','~','[',']','`','#','>','<','"');
+$convert_enc = array('%25', '%0D', '%0A', '%09', '%20', '%2D', '%2F', '%3B', '%3F', '%7B', '%7D', '%7C', '%5C', '%5E', '%7E', '%5B', '%5D', '%60', '%23', '%3E', '%3C', '%22');
+$convert_dec = array('%', "\r", "\n", "\t", ' ', '-', '/', ';', '?', '{', '}', '|', '\\', '^', '~', '[', ']', '`', '#', '>', '<', '"');
 $crlf = "\r\n";
 //$tab = "\t";
 $tab = "";

main/newscorm/aiccItem.class.php

Indentation +3 added lines, -3 removed lines

@@ -48,7 +48,7 @@ 
                 default:
                      //if($first_item->type == XML_ELEMENT_NODE) this is already check prior to the call to this function
                      foreach ($params as $a => $value) {
-                         switch ($a) {
+                            switch ($a) {
                             case 'system_id':
                                 $this->identifier = Database::escape_string(strtolower($value));
                                 break;
@@ -82,8 +82,8 @@ 
                             case 'web_launch':
                                 $this->parameters = Database::escape_string($value);
                                 break;
-                         }
-                     }
+                            }
+                        }
 
                     return true;
             }

Spacing +1 added lines, -1 removed lines

@@ -117,7 +117,7 @@
         $abs_order++;
         $i = 1;
         foreach ($this->sub_items as $id => $dummy) {
-            $oSubitem =& $this->sub_items[$id];
+            $oSubitem = & $this->sub_items[$id];
             $oSubitem->get_flat_list($list, $abs_order, $i, $level + 1);
             $i++;
         }

main/newscorm/lp_subscribe_users.php

Spacing +1 added lines, -1 removed lines

@@ -163,7 +163,7 @@
     exit;
 } else {
     $headers = [get_lang('SubscribeUsersToLp'), get_lang('SubscribeGroupsToLp')];
-    $tabs = Display::tabs($headers, [$formUsers->toHtml(),$form->toHtml()]);
+    $tabs = Display::tabs($headers, [$formUsers->toHtml(), $form->toHtml()]);
     $tpl->assign('tabs', $tabs);
 }