chamilo / chamilo-lms

main/lp/lp_ajax_save_item.php

Spacing +3 added lines, -3 removed lines

@@ -163,7 +163,7 @@ 
         // Set status to completed for hotpotatoes if score > 80%.
         if ($my_type == 'hotpotatoes') {
             if ((empty($status) || $status == 'undefined' || $status == 'not attempted') && $max > 0) {
-                if (($score/$max) > 0.8) {
+                if (($score / $max) > 0.8) {
                     $myStatus = 'completed';
                     if ($debug > 1) {
                         error_log('Calling set_status('.$myStatus.') for hotpotatoes', 0);
@@ -174,7 +174,7 @@ 
                         error_log('Done calling set_status for hotpotatoes - now '.$myLPI->get_status(false), 0);
                     }
                 }
-            } elseif ($status == 'completed' && $max > 0 && ($score/$max) < 0.8) {
+            } elseif ($status == 'completed' && $max > 0 && ($score / $max) < 0.8) {
                 $myStatus = 'failed';
                 if ($debug > 1) {
                     error_log('Calling set_status('.$myStatus.') for hotpotatoes', 0);
@@ -367,7 +367,7 @@ 
             $myLPI->current_data = $suspend;
         }
 
-        if (isset($location) && $location != '' && $location!='undefined') {
+        if (isset($location) && $location != '' && $location != 'undefined') {
             $myLPI->set_lesson_location($location);
         }
 

main/lp/lp_ajax_initialize.php

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/lp/storageapi.php

Spacing +2 added lines, -2 removed lines

@@ -103,7 +103,7 @@ 
         and sco_id = '$sv_sco'
         and course_id = '$sv_course'
         and sv_key = '$sv_key'
-        order by sv_value ".($sv_asc ? "ASC": "DESC")." limit $sv_length";
+        order by sv_value ".($sv_asc ? "ASC" : "DESC")." limit $sv_length";
 //	$sql_data = "select sv.user_id as user_id, sv_key as variable, sv_value as value
 //		from ".Database::get_main_table(TABLE_TRACK_STORED_VALUES)." sv
 //		where sv.user_id in (select u2.user_id from ($sql_leaders) u2)
@@ -135,7 +135,7 @@ 
         and search.sco_id = '$sv_sco'
         and search.course_id = '$sv_course'
         and search.sv_key = '$sv_key'
-        and list.sv_value ".($sv_asc ? "<=": ">=")." search.sv_value
+        and list.sv_value ".($sv_asc ? "<=" : ">=")." search.sv_value
         and list.sco_id = search.sco_id
         and list.course_id = search.course_id
         and list.sv_key = search.sv_key

main/lp/lp_content.php

Spacing +1 added lines, -1 removed lines

@@ -106,7 +106,7 @@
 // Define the 'doc.inc.php' as language file.
 $nameTools = $_SESSION['oLP']->get_name();
 $interbreadcrumb[] = array(
-    'url' => api_get_path(WEB_CODE_PATH) . 'lp/lp_list.php?'.api_get_cidreq(),
+    'url' => api_get_path(WEB_CODE_PATH).'lp/lp_list.php?'.api_get_cidreq(),
     'name' => get_lang('Doc'),
 );
 // Update global setting to avoid displaying right menu.

main/lp/scormItem.class.php

Spacing +5 added lines, -5 removed lines

@@ -100,7 +100,7 @@ 
                                         }
                                         break;
                                     case 'item':
-                                        $oItem = new scormItem('manifest',$child);
+                                        $oItem = new scormItem('manifest', $child);
                                         if ($oItem->identifier != '') {
                                             $this->sub_items[$oItem->identifier] = $oItem;
                                         }
@@ -119,7 +119,7 @@ 
                         $attributes = $element->attributes;
                         //$keep_href = '';
                         foreach ($attributes as $attrib) {
-                            switch($attrib->name){
+                            switch ($attrib->name) {
                                 case 'identifier':
                                     $this->identifier = $attrib->value;
                                     break;
@@ -172,8 +172,8 @@ 
         );
         $abs_order++;
         $i = 1;
-        foreach($this->sub_items as $id => $dummy) {
-            $oSubitem =& $this->sub_items[$id];
+        foreach ($this->sub_items as $id => $dummy) {
+            $oSubitem = & $this->sub_items[$id];
             $oSubitem->get_flat_list($list, $abs_order, $i, $level + 1);
             $i++;
         }
@@ -188,7 +188,7 @@ 
         parent::save($from_outside, $prereqs_complete);
         // Under certain conditions, the scorm_contact should not be set, because no scorm signal was sent.
         $this->scorm_contact = true;
-        if (!$this->scorm_contact){
+        if (!$this->scorm_contact) {
             //error_log('New LP - was expecting SCORM message but none received', 0);
         }
     }

main/lp/lp_report.php

Spacing +2 added lines, -2 removed lines

@@ -106,7 +106,7 @@ 
 
 // View
 $interbreadcrumb[] = [
-    'url' => api_get_path(WEB_CODE_PATH) . 'lp/lp_controller.php?'.api_get_cidreq(),
+    'url' => api_get_path(WEB_CODE_PATH).'lp/lp_controller.php?'.api_get_cidreq(),
     'name' => get_lang('LearningPaths')
 ];
 
@@ -117,7 +117,7 @@ 
         array(),
         ICON_SIZE_MEDIUM
     ),
-    api_get_path(WEB_CODE_PATH) . 'lp/lp_controller.php?' . api_get_cidreq()
+    api_get_path(WEB_CODE_PATH).'lp/lp_controller.php?'.api_get_cidreq()
 );
 
 $template = new Template(get_lang('StudentScore'));

main/lp/aiccItem.class.php

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/lp/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);
 }
 

main/lp/scormOrganization.class.php

Spacing +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
         foreach ($this->items as $id => $dummy) {
             $abs_order = 0;
             // Passes the array as a pointer so it is modified in $list directly.
-            $this->items[$id]->get_flat_list($list,$abs_order, $i, 0);
+            $this->items[$id]->get_flat_list($list, $abs_order, $i, 0);
             $i++;
         }