chamilo / chamilo-lms

main/newscorm/lp_ajax_switch_item_toc.php

Spacing +16 added lines, -16 removed lines

@@ -124,22 +124,22 @@
     $coreExit = $myLPI->get_core_exit();
 
     $return .=
-        "olms.lms_lp_id=".$lpId.";" .
-        "olms.lms_item_id=".$newItemId.";" .
-        "olms.lms_old_item_id=".$oldItemId.";" .
-        "olms.lms_initialized=0;" .
-        "olms.lms_view_id=".$viewId.";" .
-        "olms.lms_user_id=".$userId.";" .
-        "olms.next_item=".$newItemId.";" . // This one is very important to replace possible literal strings.
-        "olms.lms_next_item=".$nextItemId.";" .
-        "olms.lms_previous_item=".$previousItemId.";" .
-        "olms.lms_item_type = '".$itemType."';" .
-        "olms.lms_item_credit = '".$credit."';" .
-        "olms.lms_item_lesson_mode = '".$lessonMode."';" .
-        "olms.lms_item_launch_data = '".$launchData."';" .
-        "olms.lms_item_interactions_count = '".$interactionsCount."';" .
-        "olms.lms_item_objectives_count = '".$objectivesCount."';" .
-        "olms.lms_item_core_exit = '".$coreExit."';" .
+        "olms.lms_lp_id=".$lpId.";".
+        "olms.lms_item_id=".$newItemId.";".
+        "olms.lms_old_item_id=".$oldItemId.";".
+        "olms.lms_initialized=0;".
+        "olms.lms_view_id=".$viewId.";".
+        "olms.lms_user_id=".$userId.";".
+        "olms.next_item=".$newItemId.";".// This one is very important to replace possible literal strings.
+        "olms.lms_next_item=".$nextItemId.";".
+        "olms.lms_previous_item=".$previousItemId.";".
+        "olms.lms_item_type = '".$itemType."';".
+        "olms.lms_item_credit = '".$credit."';".
+        "olms.lms_item_lesson_mode = '".$lessonMode."';".
+        "olms.lms_item_launch_data = '".$launchData."';".
+        "olms.lms_item_interactions_count = '".$interactionsCount."';".
+        "olms.lms_item_objectives_count = '".$objectivesCount."';".
+        "olms.lms_item_core_exit = '".$coreExit."';".
         "olms.asset_timer = 0;";
 
     $return .= "update_toc('unhighlight','".$currentItem."');".

main/newscorm/download.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
 
 $doc_url = str_replace(array('../', '\\..', '\\0', '..\\'), array('', '', '', ''), $doc_url); //echo $doc_url;
 
-if (strpos($doc_url,'../') OR strpos($doc_url,'/..')) {
+if (strpos($doc_url, '../') OR strpos($doc_url, '/..')) {
     $doc_url = '';
 }
 $sys_course_path = api_get_path(SYS_COURSE_PATH).$_course['path'].'/scorm';
@@ -50,5 +50,5 @@ 
     }
 }
 
-Display::display_error_message(get_lang('ProtectedDocument'));//api_not_allowed backbutton won't work.
+Display::display_error_message(get_lang('ProtectedDocument')); //api_not_allowed backbutton won't work.
 exit;

main/newscorm/aicc_api.php

Indentation +8 added lines, -8 removed lines

@@ -142,7 +142,7 @@ 
           <?php
             $who = addslashes(api_get_person_name($_user['firstName'], $_user['lastName']));
             echo "result='$who';";
-          ?>
+            ?>
     }else if(param == 'cmi.core.lesson_location'){
         result=lesson_location;
     }else if(param == 'cmi.core.total_time'){
@@ -198,7 +198,7 @@ 
           <?php
             $who = addslashes(api_get_person_name($_user['firstName'], $_user['lastName']));
             echo "result='$who';";
-          ?>	break;
+            ?>	break;
         case 'cmi.core.lesson_location'	:
             result='';
             break;
@@ -322,12 +322,12 @@ 
 }
 <?php
 /**
- * Chamilo-specific code that deals with event handling and inter-frames
- * messaging/refreshing.
- * Note that from now on, the Chamilo JS code in this library will act as
- * a controller, of the MVC pattern, and receive all requests for frame
- * updates, then redispatch to any frame concerned.
- */
+     * Chamilo-specific code that deals with event handling and inter-frames
+     * messaging/refreshing.
+     * Note that from now on, the Chamilo JS code in this library will act as
+     * a controller, of the MVC pattern, and receive all requests for frame
+     * updates, then redispatch to any frame concerned.
+     */
 ?>
 /**
  * Defining the AJAX-object class to be made available from other frames.

Spacing +23 added lines, -23 removed lines

@@ -78,33 +78,33 @@ 
 var commit = false ;
 
 // Strictly SCORM variables.
-var score=<?php echo $oItem->get_score();?>;
-var max=<?php echo $oItem->get_max();?>;
-var min=<?php echo $oItem->get_min();?>;
-var lesson_status='<?php echo $oItem->get_status();?>';
-var session_time='<?php echo $oItem->get_scorm_time('js');?>';
-var suspend_data = '<?php echo $oItem->get_suspend_data();?>';
-var lesson_location = '<?php echo $oItem->get_lesson_location();?>';
-var total_time = '<?php echo $oItem->get_scorm_time('js');?>';
+var score=<?php echo $oItem->get_score(); ?>;
+var max=<?php echo $oItem->get_max(); ?>;
+var min=<?php echo $oItem->get_min(); ?>;
+var lesson_status='<?php echo $oItem->get_status(); ?>';
+var session_time='<?php echo $oItem->get_scorm_time('js'); ?>';
+var suspend_data = '<?php echo $oItem->get_suspend_data(); ?>';
+var lesson_location = '<?php echo $oItem->get_lesson_location(); ?>';
+var total_time = '<?php echo $oItem->get_scorm_time('js'); ?>';
 
 // Chamilo internal variables.
 var saved_lesson_status = 'not attempted';
-var lms_lp_id = <?php echo $oLP->get_id();?>;
-var lms_item_id = <?php echo $oItem->get_id();?>;
+var lms_lp_id = <?php echo $oLP->get_id(); ?>;
+var lms_item_id = <?php echo $oItem->get_id(); ?>;
 //var lms_new_item_id = 0; //temporary value (only there between a load_item() and a LMSInitialize())
 var lms_been_synchronized = 0;
 var lms_initialized = 0;
 var lms_total_lessons = <?php echo $oLP->get_total_items_count(); ?>;
-var lms_complete_lessons = <?php echo $oLP->get_complete_items_count();?>;
-var lms_progress_bar_mode = '<?php echo $oLP->progress_bar_mode;?>';
+var lms_complete_lessons = <?php echo $oLP->get_complete_items_count(); ?>;
+var lms_progress_bar_mode = '<?php echo $oLP->progress_bar_mode; ?>';
 if(lms_progress_bar_mode == ''){lms_progress_bar_mode='%';}
-var lms_view_id = '<?php echo $oLP->get_view();?>';
+var lms_view_id = '<?php echo $oLP->get_view(); ?>';
 if(lms_view_id == ''){ lms_view_id = 1;}
-var lms_user_id = '<?php echo $_user['user_id'];?>';
-var lms_next_item = '<?php echo $oLP->get_next_item_id();?>';
-var lms_previous_item = '<?php echo $oLP->get_previous_item_id();?>';
-var lms_lp_type = '<?php echo $oLP->get_type();?>';
-var lms_item_type = '<?php echo $oItem->get_type();?>';
+var lms_user_id = '<?php echo $_user['user_id']; ?>';
+var lms_next_item = '<?php echo $oLP->get_next_item_id(); ?>';
+var lms_previous_item = '<?php echo $oLP->get_previous_item_id(); ?>';
+var lms_lp_type = '<?php echo $oLP->get_type(); ?>';
+var lms_item_type = '<?php echo $oItem->get_type(); ?>';
 
 // Backup for old values.
 var old_score = 0;
@@ -162,11 +162,11 @@ 
     }else if(param == 'cmi.core.lesson_mode'){
         result='normal';
     }else if(param == 'cmi.suspend_data'){
-        result='<?php echo $oItem->get_suspend_data();?>';
+        result='<?php echo $oItem->get_suspend_data(); ?>';
     }else if(param == 'cmi.launch_data'){
         result='';
     }else if(param == 'cmi.objectives._count'){
-        result='<?php echo $oItem->get_view_count();?>';
+        result='<?php echo $oItem->get_view_count(); ?>';
     }
     /*
     // Switch not working??? WTF???
@@ -227,13 +227,13 @@ 
             result='normal';
             break;
         case 'cmi.suspend_data'		:
-            result='<?php echo $oItem->get_suspend_data();?>';
+            result='<?php echo $oItem->get_suspend_data(); ?>';
             break;
         case 'cmi.launch_data'		:
             result='';
             break;
         case 'cmi.objectives._count':
-            result='<?php echo $oItem->get_view_count();?>';
+            result='<?php echo $oItem->get_view_count(); ?>';
             break;
         default 					:
             result='';
@@ -285,7 +285,7 @@ 
     $url = $_SERVER['HTTP_HOST'].$self;
     $url = substr($url, 0, -14); // 14 is the length of this file's name (/scorm_api.php).
     echo $url;
-    ?>/lp_controller.php?cidReq=<?php echo api_get_course_id();?>&action=save&lp_id=<?php echo $oLP->get_id();?>&" + param + "";
+    ?>/lp_controller.php?cidReq=<?php echo api_get_course_id(); ?>&action=save&lp_id=<?php echo $oLP->get_id(); ?>&" + param + "";
     logit_lms('saving data (status='+lesson_status+')',1);
     xajax_save_item(lms_lp_id, lms_user_id, lms_view_id, lms_item_id, score, max, min, lesson_status, session_time, suspend_data, lesson_location);
     //xajax_update_pgs();

main/newscorm/content_makers.inc.php

Indentation +5 added lines, -5 removed lines

@@ -13,11 +13,11 @@
     'Accent',
     'Accenture',
     'ADLNet',
-      'Articulate',
-      'ATutor',
-      'Blackboard',
-      'Calfat',
-      'Captivate',
+        'Articulate',
+        'ATutor',
+        'Blackboard',
+        'Calfat',
+        'Captivate',
     'Chamilo',
     'Chamilo 2',
     'Claroline',

main/newscorm/lp_ajax_initialize.php

Indentation +4 added lines, -4 removed lines

@@ -80,11 +80,11 @@ 
     if (!empty($myistring)) {
         $myistring = substr($myistring, 1);
     }
-	// Obtention des donnees d'objectifs
+    // Obtention des donnees d'objectifs
 
-	$mycoursedb = Database::get_course_table(TABLE_LP_IV_OBJECTIVE);
+    $mycoursedb = Database::get_course_table(TABLE_LP_IV_OBJECTIVE);
     $course_id = api_get_course_int_id();
-	$mylp_iv_id = $mylpi->db_item_view_id;
+    $mylp_iv_id = $mylpi->db_item_view_id;
 
     $phpobjectives = array();
 
@@ -98,7 +98,7 @@ 
             $phpobjectives[] = $row;
         }
     }
-	$myobjectives = json_encode($phpobjectives);
+    $myobjectives = json_encode($phpobjectives);
 
     $return .=
             "olms.score=".$myscore.";" .

Spacing +30 added lines, -30 removed lines

@@ -101,21 +101,21 @@ 
 	$myobjectives = json_encode($phpobjectives);
 
     $return .=
-            "olms.score=".$myscore.";" .
-            "olms.max=".$mymax.";" .
-            "olms.min=".$mymin.";" .
-            "olms.lesson_status='".$mylesson_status."';" .
-            "olms.lesson_location='".$mylesson_location."';" .
-            "olms.session_time='".$mysession_time."';" .
-            "olms.suspend_data='".$mysuspend_data."';" .
-            "olms.total_time = '".$mytotal_time."';" .
-            "olms.mastery_score = '".$mymastery_score."';" .
-            "olms.max_time_allowed = '".$mymax_time_allowed."';" .
-            "olms.launch_data = '".$mylaunch_data."';" .
-            "olms.interactions = new Array(".$myistring.");" .
+            "olms.score=".$myscore.";".
+            "olms.max=".$mymax.";".
+            "olms.min=".$mymin.";".
+            "olms.lesson_status='".$mylesson_status."';".
+            "olms.lesson_location='".$mylesson_location."';".
+            "olms.session_time='".$mysession_time."';".
+            "olms.suspend_data='".$mysuspend_data."';".
+            "olms.total_time = '".$mytotal_time."';".
+            "olms.mastery_score = '".$mymastery_score."';".
+            "olms.max_time_allowed = '".$mymax_time_allowed."';".
+            "olms.launch_data = '".$mylaunch_data."';".
+            "olms.interactions = new Array(".$myistring.");".
             //"olms.item_objectives = new Array();" .
-            "olms.item_objectives = ".$myobjectives.";" .
-            "olms.G_lastError = 0;" .
+            "olms.item_objectives = ".$myobjectives.";".
+            "olms.G_lastError = 0;".
             "olms.G_LastErrorMessage = 'No error';".
             "olms.finishSignalReceived = 0;";
     /*
@@ -144,22 +144,22 @@ 
     $mycore_exit = $mylpi->get_core_exit();
 
     $return .=
-            "olms.lms_lp_id=".$lp_id.";" .
-            "olms.lms_item_id=".$next_item.";" .
-            "olms.lms_old_item_id=0;" .
-            "olms.lms_initialized=0;" .
-            "olms.lms_view_id=".$view_id.";" .
-            "olms.lms_user_id=".$user_id.";" .
-            "olms.next_item=".$next_item.";" . // This one is very important to replace possible literal strings.
-            "olms.lms_next_item=".$mynext.";" .
-            "olms.lms_previous_item=".$myprevious.";" .
-            "olms.lms_item_type = '".$myitemtype."';" .
-            "olms.lms_item_credit = '".$mycredit."';" .
-            "olms.lms_item_lesson_mode = '".$mylesson_mode."';" .
-            "olms.lms_item_launch_data = '".$mylaunch_data."';" .
-            "olms.lms_item_interactions_count = '".$myinteractions_count."';" .
-            "olms.lms_item_objectives_count = '".$myinteractions_count."';" .
-            "olms.lms_item_core_exit = '".$mycore_exit."';" .
+            "olms.lms_lp_id=".$lp_id.";".
+            "olms.lms_item_id=".$next_item.";".
+            "olms.lms_old_item_id=0;".
+            "olms.lms_initialized=0;".
+            "olms.lms_view_id=".$view_id.";".
+            "olms.lms_user_id=".$user_id.";".
+            "olms.next_item=".$next_item.";".// This one is very important to replace possible literal strings.
+            "olms.lms_next_item=".$mynext.";".
+            "olms.lms_previous_item=".$myprevious.";".
+            "olms.lms_item_type = '".$myitemtype."';".
+            "olms.lms_item_credit = '".$mycredit."';".
+            "olms.lms_item_lesson_mode = '".$mylesson_mode."';".
+            "olms.lms_item_launch_data = '".$mylaunch_data."';".
+            "olms.lms_item_interactions_count = '".$myinteractions_count."';".
+            "olms.lms_item_objectives_count = '".$myinteractions_count."';".
+            "olms.lms_item_core_exit = '".$mycore_exit."';".
             "olms.asset_timer = 0;";
 
     $mylp->set_error_msg('');

main/newscorm/lp_add_audio.php

Spacing +8 added lines, -8 removed lines

@@ -17,7 +17,7 @@ 
 
 $isStudentView  = isset($_REQUEST['isStudentView']) ? (int) $_REQUEST['isStudentView'] : null;
 $learnpath_id   = (int) $_REQUEST['lp_id'];
-$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;
@@ -40,7 +40,7 @@ 
 }
 
 if (!empty($gradebook) && $gradebook == 'view') {
-    $interbreadcrumb[] = array (
+    $interbreadcrumb[] = array(
         'url' => '../gradebook/'.$_SESSION['gradebook_dest'],
         'name' => get_lang('ToolGradebook')
     );
@@ -51,19 +51,19 @@ 
 
 switch ($type) {
     case 'chapter':
-        $interbreadcrumb[]= array('url' => 'lp_controller.php?action=add_item&type=step&lp_id='.$_SESSION['oLP']->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='.$_SESSION['oLP']->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='.$_SESSION['oLP']->get_id(), 'name' => get_lang('NewStep'));
+        $interbreadcrumb[] = array('url' => 'lp_controller.php?action=add_item&type=step&lp_id='.$_SESSION['oLP']->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'));
+    $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('NewDocumentCreated'));
 }
 
 // Theme calls.
@@ -79,7 +79,7 @@ 
 $form = new FormValidator(
     'add_audio',
     'post',
-    api_get_self() . '?action=add_audio&id=' . $lp_item_id . '&' . api_get_cidreq().'&lp_id='.$learnpath_id,
+    api_get_self().'?action=add_audio&id='.$lp_item_id.'&'.api_get_cidreq().'&lp_id='.$learnpath_id,
     null,
     array('enctype' => 'multipart/form-data')
 );

Indentation +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 
 // Using the resource linker as a tool for adding resources to the learning path.
 if ($action == 'add' && $type == 'learnpathitem') {
-     $htmlHeadXtra[] = "<script>
+        $htmlHeadXtra[] = "<script>
      window.location=\"../resourcelinker/resourcelinker.php?source_id=5&action=$action&learnpath_id=$learnpath_id&chapter_id=$chapter_id&originalresource=no\";
      </script>";
 }

main/newscorm/openoffice_presentation.class.php

Spacing +7 added lines, -7 removed lines

@@ -85,7 +85,7 @@ 
             );
 
             // Generating the thumbnail.
-            $image = $this->base_work_dir.$this->created_dir .'/'. $file_name;
+            $image = $this->base_work_dir.$this->created_dir.'/'.$file_name;
 
             $pattern = '/(\w+)\.png$/';
             $replacement = '${1}_thumb.png';
@@ -101,7 +101,7 @@ 
 
             $my_new_image = new Image($image);
             $my_new_image->resize($thumb_width, $thumb_height);
-            $my_new_image->send_image($this->base_work_dir.$this->created_dir .'/'. $thumb_name, -1, 'png');
+            $my_new_image->send_image($this->base_work_dir.$this->created_dir.'/'.$thumb_name, -1, 'png');
 
             // Adding the thumbnail to documents.
             $document_id_thumb = add_document(
@@ -127,7 +127,7 @@ 
     <body>
         <img src="'.$slide_src.'" />
     </body>
-</html>');  // This indentation is to make the generated html files to look well.
+</html>'); // This indentation is to make the generated html files to look well.
 
             fclose($fp);
             $document_id = add_document(
@@ -162,7 +162,7 @@ 
                     foreach ($specific_fields as $specific_field) {
                         if (isset($_REQUEST[$specific_field['code']])) {
                             $sterms = trim($_REQUEST[$specific_field['code']]);
-                            $all_specific_terms .= ' '. $sterms;
+                            $all_specific_terms .= ' '.$sterms;
                             if (!empty($sterms)) {
                                 $sterms = explode(',', $sterms);
                                 foreach ($sterms as $sterm) {
@@ -171,7 +171,7 @@ 
                             }
                         }
                     }
-                    $slide_body = $all_specific_terms .' '. $slide_body;
+                    $slide_body = $all_specific_terms.' '.$slide_body;
                     $ic_slide->addValue('content', $slide_body);
                     /* FIXME:  cidReq:lp_id:doc_id al indexar  */
                     // Add a comment to say terms separated by commas.
@@ -183,7 +183,7 @@ 
                         SE_COURSE_ID => $courseid,
                         SE_TOOL_ID => TOOL_LEARNPATH,
                         SE_DATA => array('lp_id' => $lp_id, 'lp_item' => $previous, 'document_id' => $document_id),
-                        SE_USER => (int)api_get_user_id(),
+                        SE_USER => (int) api_get_user_id(),
                     );
                     $ic_slide->xapian_data = serialize($xapian_data);
                     $di->addChunk($ic_slide);
@@ -221,7 +221,7 @@ 
         $_course = api_get_course_info();
         foreach ($files as $file) {
             // '||' is used as separator between slide name (with accents) and file name (without accents).
-            list($slide_name,$file_name) = explode('||',$file);
+            list($slide_name, $file_name) = explode('||', $file);
             $slide_name = api_htmlentities($slide_name, ENT_COMPAT, $this->original_charset);
             $slide_name = str_replace('&rsquo;', '\'', $slide_name);
             $slide_name = api_convert_encoding($slide_name, api_get_system_encoding(), $this->original_charset);

main/newscorm/blank.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 Display::display_reduced_header();
 
 if (isset($_GET['error'])) {
-    switch ($_GET['error']){
+    switch ($_GET['error']) {
         case 'document_deleted':
             echo '<br /><br />';
             Display::display_error_message(get_lang('DocumentHasBeenDeleted'));

main/newscorm/lp_impress.php

Indentation +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     api_get_group_id()
 );
 if (!api_is_allowed_to_edit(null, true) && intval($visibility) == 0 ) {
-     api_not_allowed();
+        api_not_allowed();
 }
 
 if (empty($_SESSION['oLP'])) {

Spacing +5 added lines, -5 removed lines

@@ -35,7 +35,7 @@ 
     null,
     api_get_group_id()
 );
-if (!api_is_allowed_to_edit(null, true) && intval($visibility) == 0 ) {
+if (!api_is_allowed_to_edit(null, true) && intval($visibility) == 0) {
      api_not_allowed();
 }
 
@@ -73,22 +73,22 @@ 
 $html = '';
 $step = 1;
 foreach ($list as $toc) {
-    $x = 1000*$step;
+    $x = 1000 * $step;
     $html .= '<div id="step-'.$step.'" class="step slide" data-x="'.$x.'" data-y="-1500"  >';
     $html .= '<div class="impress-content">';
     $src = $_SESSION['oLP']->get_link('http', $toc['id']);
     if ($toc['type'] !== 'dokeos_chapter') {
         //just showing the src in a iframe ...
         $html .= '<h2>'.$toc['title'].'</h2>';
-        $html .= '<iframe border="0" frameborder="0" style="width:100%;height:600px" src="' . $src . '"></iframe>';
-    }else{
+        $html .= '<iframe border="0" frameborder="0" style="width:100%;height:600px" src="'.$src.'"></iframe>';
+    } else {
         $html .= "<div class='impress-title'>";
         $html .= '<h1>'.$toc['title'].'</h1>';
         $html .= "</div>";
     }
     $html .= "</div>";
     $html .= "</div>";
-    $step ++;
+    $step++;
 }
 
 //Setting the template

Braces +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
         //just showing the src in a iframe ...
         $html .= '<h2>'.$toc['title'].'</h2>';
         $html .= '<iframe border="0" frameborder="0" style="width:100%;height:600px" src="' . $src . '"></iframe>';
-    }else{
+    } else{
         $html .= "<div class='impress-title'>";
         $html .= '<h1>'.$toc['title'].'</h1>';
         $html .= "</div>";