chamilo / chamilo-lms

main/coursecopy/copy_course_session.php

Spacing +4 added lines, -4 removed lines

@@ -85,7 +85,7 @@ 
 
 function display_form()
 {
-    $html  = '';
+    $html = '';
     $sessions = SessionManager::get_sessions_list(array(), array('name', 'ASC'));
 
     // Link back to the documents overview
@@ -109,7 +109,7 @@ 
 
     //destination
     $html .= '<div class="form-group">';
-    $html .= '<label class="col-sm-2 control-label">' . get_lang('DestinationCoursesFromSession') . ': </label>';
+    $html .= '<label class="col-sm-2 control-label">'.get_lang('DestinationCoursesFromSession').': </label>';
     $html .= '<div class="col-sm-5" id="ajax_sessions_list_destination">';
     $html .= '<select class="form-control" name="sessions_list_destination" onchange="javascript: xajax_search_courses(this.value,\'destination\');">';
     $html .= '<option value = "0">'.get_lang('ThereIsNotStillASession').'</option></select ></div>';
@@ -131,7 +131,7 @@ 
     $html .= '<button class="btn btn-success" type="submit" onclick="javascript:if(!confirm('."'".addslashes(api_htmlentities(get_lang('ConfirmYourChoice'), ENT_QUOTES))."'".')) return false;"><em class="fa fa-files-o"></em> '.get_lang('CopyCourse').'</button>';
 
     // Add Security token
-    $html .= '<input type="hidden" value="' . Security::get_token() . '" name="sec_token">';
+    $html .= '<input type="hidden" value="'.Security::get_token().'" name="sec_token">';
     $html .= '</div></div>';
     $html .= '</form>';
 
@@ -227,7 +227,7 @@ 
 
 /* HTML head extra */
 
-$htmlHeadXtra[] = $xajax->getJavascript( api_get_path(WEB_LIBRARY_PATH).'xajax/');
+$htmlHeadXtra[] = $xajax->getJavascript(api_get_path(WEB_LIBRARY_PATH).'xajax/');
 $htmlHeadXtra[] = '<script>
 function checkSelected(id_select,id_radio,id_title,id_destination) {
    var num=0;

main/template/default/auth/courses_list.php

Spacing +23 added lines, -23 removed lines

@@ -18,7 +18,7 @@ 
 <div id="actions" class="actions">
     <?php if ($action != 'createcoursecategory') { ?>
 	<a href="<?php echo api_get_self(); ?>?action=createcoursecategory">
-        <?php echo Display::return_icon('new_folder.png', get_lang('CreateCourseCategory'),'','32'); ?></a>
+        <?php echo Display::return_icon('new_folder.png', get_lang('CreateCourseCategory'), '', '32'); ?></a>
     <?php } ?>
 </div>
 
@@ -50,26 +50,26 @@ 
             $max_category_key = count($user_course_categories);
             if ($action != 'unsubscribe') { ?>
                 <a href="courses.php?action=sortmycourses&amp;categoryid=<?php echo $row['id']; ?>&amp;sec_token=<?php echo $stok; ?>#category<?php echo $row['id']; ?>">
-                <?php echo Display::display_icon('edit.png', get_lang('Edit'),'',22); ?>
+                <?php echo Display::display_icon('edit.png', get_lang('Edit'), '', 22); ?>
                 </a>
 
                 <?php if ($row['id'] != $user_course_categories[0]['id']) { ?>
                         <a href="courses.php?action=<?php echo $action ?>&amp;move=up&amp;category=<?php echo $row['id']; ?>&amp;sec_token=<?php echo $stok; ?>">
-                        <?php echo Display::return_icon('up.png', get_lang('Up'),'',22); ?>
+                        <?php echo Display::return_icon('up.png', get_lang('Up'), '', 22); ?>
                         </a>
                 <?php } else { ?>
-                    <?php echo Display::return_icon('up_na.png', get_lang('Up'),'',22); ?>
+                    <?php echo Display::return_icon('up_na.png', get_lang('Up'), '', 22); ?>
                <?php } ?>
 
                 <?php if ($row['id'] != $user_course_categories[$max_category_key - 1]['id']) { ?>
                     <a href="courses.php?action=<?php echo $action; ?>&amp;move=down&amp;category=<?php echo $row['id']; ?>&amp;sec_token=<?php echo $stok; ?>">
-                    <?php echo Display::return_icon('down.png', get_lang('Down'),'',22); ?>
+                    <?php echo Display::return_icon('down.png', get_lang('Down'), '', 22); ?>
                     </a>
                 <?php } else { ?>
-                    <?php echo Display::return_icon('down_na.png', get_lang('Down'),'',22); ?>
+                    <?php echo Display::return_icon('down_na.png', get_lang('Down'), '', 22); ?>
                 <?php } ?>
                 <a href="courses.php?action=deletecoursecategory&amp;id=<?php echo $row['id']; ?>&amp;sec_token=<?php echo $stok; ?>">
-                <?php echo Display::display_icon('delete.png', get_lang('Delete'), array('onclick' => "javascript: if (!confirm('".addslashes(api_htmlentities(get_lang("CourseCategoryAbout2bedeleted"), ENT_QUOTES, api_get_system_encoding()))."')) return false;"),22) ?>
+                <?php echo Display::display_icon('delete.png', get_lang('Delete'), array('onclick' => "javascript: if (!confirm('".addslashes(api_htmlentities(get_lang("CourseCategoryAbout2bedeleted"), ENT_QUOTES, api_get_system_encoding()))."')) return false;"), 22) ?>
                 </a>
             <?php }
             echo '<br /><br />';
@@ -123,35 +123,35 @@ 
                     <div style="float:left;width:110px;">
                     <?php
                         if (api_get_setting('show_courses_descriptions_in_catalog') == 'true') {
-                            $icon_title = get_lang('CourseDetails') . ' - ' . $course['title'];
+                            $icon_title = get_lang('CourseDetails').' - '.$course['title'];
                     ?>
                     <a href="<?php echo api_get_path(WEB_CODE_PATH); ?>inc/ajax/course_home.ajax.php?a=show_course_information&code=<?php echo $course['code'] ?>" data-title="<?php echo $icon_title ?>" title="<?php echo $icon_title ?>" class="ajax">
-                        <?php echo Display::return_icon('info.png', $icon_title,'','22') ?>
+                        <?php echo Display::return_icon('info.png', $icon_title, '', '22') ?>
                        <?php } ?>
                     </a>
 
                     <?php if (isset($_GET['edit']) && $course['code'] == $_GET['edit']) { ?>
-                          <?php echo Display::display_icon('edit_na.png', get_lang('Edit'),'',22); ?>
+                          <?php echo Display::display_icon('edit_na.png', get_lang('Edit'), '', 22); ?>
                     <?php } else { ?>
                         <a href="courses.php?action=<?php echo $action; ?>&amp;edit=<?php echo $course['code']; ?>&amp;sec_token=<?php echo $stok; ?>">
-                            <?php echo Display::display_icon('edit.png', get_lang('Edit'),'',22); ?>
+                            <?php echo Display::display_icon('edit.png', get_lang('Edit'), '', 22); ?>
                             </a>
                     <?php } ?>
 
                     <?php if ($key > 0) { ?>
                         <a href="courses.php?action=<?php echo $action; ?>&amp;move=up&amp;course=<?php echo $course['code']; ?>&amp;category=<?php echo $course['user_course_cat']; ?>&amp;sec_token=<?php echo $stok; ?>">
-                        <?php echo Display::display_icon('up.png', get_lang('Up'),'',22); ?>
+                        <?php echo Display::display_icon('up.png', get_lang('Up'), '', 22); ?>
                         </a>
                     <?php } else { ?>
-                        <?php echo Display::display_icon('up_na.png', get_lang('Up'),'',22); ?>
+                        <?php echo Display::display_icon('up_na.png', get_lang('Up'), '', 22); ?>
                     <?php } ?>
 
                     <?php if ($key < $number_of_courses - 1) { ?>
                         <a href="courses.php?action=<?php echo $action; ?>&amp;move=down&amp;course=<?php echo $course['code']; ?>&amp;category=<?php echo $course['user_course_cat']; ?>&amp;sec_token=<?php echo $stok; ?>">
-                        <?php echo Display::display_icon('down.png', get_lang('Down'),'',22); ?>
+                        <?php echo Display::display_icon('down.png', get_lang('Down'), '', 22); ?>
                         </a>
                     <?php } else { ?>
-                        <?php echo Display::display_icon('down_na.png', get_lang('Down'),'',22); ?>
+                        <?php echo Display::display_icon('down_na.png', get_lang('Down'), '', 22); ?>
                     <?php } ?>
 
                   </div>
@@ -219,36 +219,36 @@ 
             <div style="float:left; width:110px">
             <?php
             if (api_get_setting('show_courses_descriptions_in_catalog') == 'true') {
-            $icon_title = get_lang('CourseDetails') . ' - ' . $course['title'];
+            $icon_title = get_lang('CourseDetails').' - '.$course['title'];
             ?>
             <a href="<?php echo api_get_path(WEB_CODE_PATH); ?>inc/ajax/course_home.ajax.php?a=show_course_information&code=<?php echo $course['code'] ?>" data-title="<?php echo $icon_title ?>" title="<?php echo $icon_title ?>" class="ajax">
-                <?php echo Display::return_icon('info.png', $icon_title, '','22'); ?>
+                <?php echo Display::return_icon('info.png', $icon_title, '', '22'); ?>
             </a>
             <?php } ?>
 
              <?php if (isset($_GET['edit']) && $course['code'] == $_GET['edit']) {   ?>
-                    <?php echo Display::display_icon('edit_na.png', get_lang('Edit'),'',22); ?>
+                    <?php echo Display::display_icon('edit_na.png', get_lang('Edit'), '', 22); ?>
                   <?php } else { ?>
                     <a href="courses.php?action=<?php echo $action; ?>&amp;edit=<?php echo $course['code']; ?>&amp;sec_token=<?php echo $stok; ?>">
-                    <?php echo Display::display_icon('edit.png', get_lang('Edit'),'',22); ?>
+                    <?php echo Display::display_icon('edit.png', get_lang('Edit'), '', 22); ?>
                     </a>
              <?php } ?>
 
             <!-- up /down icons-->
             <?php if ($key > 0) { ?>
                     <a href="courses.php?action=<?php echo $action; ?>&amp;move=up&amp;course=<?php echo $course['code']; ?>&amp;category=<?php echo $course['user_course_cat']; ?>&amp;sec_token=<?php echo $stok; ?>">
-                    <?php echo Display::display_icon('up.png', get_lang('Up'),'',22) ?>
+                    <?php echo Display::display_icon('up.png', get_lang('Up'), '', 22) ?>
                     </a>
             <?php } else {
-                    echo Display::display_icon('up_na.png', get_lang('Up'),'',22);
+                    echo Display::display_icon('up_na.png', get_lang('Up'), '', 22);
                   }
 
                   if ($key < $number_of_courses - 1) { ?>
                     <a href="courses.php?action=<?php echo $action; ?>&amp;move=down&amp;course=<?php echo $course['code']; ?>&amp;category=<?php echo $course['user_course_cat']; ?>&amp;sec_token=<?php echo $stok; ?>">
-                    <?php echo Display::display_icon('down.png', get_lang('Down'),'',22); ?>
+                    <?php echo Display::display_icon('down.png', get_lang('Down'), '', 22); ?>
                     </a>
             <?php } else {
-                    echo Display::display_icon('down_na.png', get_lang('Down'),'',22);
+                    echo Display::display_icon('down_na.png', get_lang('Down'), '', 22);
                   }?>
                 </div>
                  <div style="float:left; margin-right:10px;">

main/inc/lib/legal.lib.php

Spacing +8 added lines, -8 removed lines

@@ -80,7 +80,7 @@ 
     public static function get_last_condition_version($language)
     {
         $legal_conditions_table = Database::get_main_table(TABLE_MAIN_LEGAL);
-        $language= Database::escape_string($language);
+        $language = Database::escape_string($language);
         $sql = "SELECT version FROM $legal_conditions_table
                 WHERE language_id = '".$language."'
                 ORDER BY id DESC LIMIT 1 ";
@@ -101,7 +101,7 @@ 
 	public static function get_last_condition($language)
     {
 		$legal_conditions_table = Database::get_main_table(TABLE_MAIN_LEGAL);
-		$language= Database::escape_string($language);
+		$language = Database::escape_string($language);
 		$sql = "SELECT * FROM $legal_conditions_table
                 WHERE language_id = '".$language."'
                 ORDER BY version DESC
@@ -151,9 +151,9 @@ 
                 ORDER BY version DESC
                 LIMIT 1 ";
         $result = Database::query($sql);
-        if (Database::num_rows($result)>0){
+        if (Database::num_rows($result) > 0) {
             $version = Database::fetch_array($result);
-            $version = explode(':',$version[0]);
+            $version = explode(':', $version[0]);
 
             return $version[0];
         } else {
@@ -184,7 +184,7 @@ 
                 break;
                 // Page link
             case 1:
-                $preview ='<fieldset>
+                $preview = '<fieldset>
                              <legend>'.get_lang('TermsAndConditions').'</legend>';
                 $preview .= '<div id="legal-accept-wrapper" class="form-item">
                 <label class="option" for="legal-accept">
@@ -217,7 +217,7 @@ 
 		$number_of_items = intval($number_of_items);
 		$column = intval($column);
 
- 		$sql  = "SELECT version, original_name as language, content, changes, type, FROM_UNIXTIME(date)
+ 		$sql = "SELECT version, original_name as language, content, changes, type, FROM_UNIXTIME(date)
 				FROM $legal_conditions_table inner join $lang_table l on(language_id = l.id) ";
 		$sql .= "ORDER BY language, version ASC ";
 		$sql .= "LIMIT $from, $number_of_items ";
@@ -252,7 +252,7 @@ 
 		        FROM $legal_conditions_table
 		        ORDER BY id DESC ";
 		$result = Database::query($sql);
-		$url = Database::fetch_array($result,'ASSOC');
+		$url = Database::fetch_array($result, 'ASSOC');
 		$result = $url['count_result'];
 
 		return $result;
@@ -273,7 +273,7 @@ 
 		        WHERE id =  "'.$legal_id.'" AND language_id="'.$language_id.'"';
 		$rs = Database::query($sql);
 
-		return Database::result($rs,0,'type');
+		return Database::result($rs, 0, 'type');
 	}
 
     /**

main/inc/lib/pdf.lib.php

Spacing +29 added lines, -29 removed lines

@@ -33,7 +33,7 @@ 
         /* More info @ http://mpdf1.com/manual/index.php?tid=184&searchstring=mPDF
          * mPDF ([ string $mode [, mixed $format [, float $default_font_size [, string $default_font [, float $margin_left , float $margin_right , float $margin_top , float $margin_bottom , float $margin_header , float $margin_footer [, string $orientation ]]]]]])
          */
-        if (!in_array($orientation, array('P','L'))) {
+        if (!in_array($orientation, array('P', 'L'))) {
             $orientation = 'P';
         }
         //$this->pdf = $pdf = new mPDF('UTF-8', $pageFormat, '', '', 30, 20, 27, 25, 16, 13, $orientation);
@@ -100,7 +100,7 @@ 
             $visualTheme = api_get_visual_theme();
             $img = api_get_path(SYS_CSS_PATH).'themes/'.$visualTheme.'/images/pdf_logo_header.png';
             if (file_exists($img)) {
-                $img = api_get_path(WEB_CSS_PATH) . 'themes/' . $visualTheme . '/images/pdf_logo_header.png';
+                $img = api_get_path(WEB_CSS_PATH).'themes/'.$visualTheme.'/images/pdf_logo_header.png';
                 $organization = "<img src='$img'>";
             }
         }
@@ -237,7 +237,7 @@ 
             // then print the title in the PDF
             if (is_array($file) && isset($file['title'])) {
                 $html_title = $file['title'];
-                $file  = $file['path'];
+                $file = $file['path'];
             } else {
                 //we suppose we've only been sent a file path
                 $html_title = basename($file);
@@ -268,7 +268,7 @@ 
             //it's not a chapter but the file exists, print its title
             if ($print_title) {
                 $this->pdf->WriteHTML(
-                    '<html><body><h3>' . $html_title . '</h3></body></html>'
+                    '<html><body><h3>'.$html_title.'</h3></body></html>'
                 );
             }
 
@@ -282,7 +282,7 @@ 
 
                 if ($extension === 'html') {
                     $filename = basename($filename, '.html');
-                } elseif($extension === 'htm'){
+                } elseif ($extension === 'htm') {
                     $filename = basename($filename, '.htm');
                 }
 
@@ -294,7 +294,7 @@ 
                 $document_html = str_replace('href="./css/frames.css"', $absolute_css_path, $document_html);
 
                 if (!empty($course_data['path'])) {
-                    $document_html= str_replace('../', '', $document_html);
+                    $document_html = str_replace('../', '', $document_html);
                     $document_path = api_get_path(SYS_COURSE_PATH).$course_data['path'].'/document/';
 
                     $doc = new DOMDocument();
@@ -372,7 +372,7 @@ 
                 if (!empty($document_html)) {
                     $this->pdf->WriteHTML($document_html.$page_break);
                 }
-            } elseif (in_array($extension, array('jpg','jpeg','png','gif'))) {
+            } elseif (in_array($extension, array('jpg', 'jpeg', 'png', 'gif'))) {
                 //Images
                 $image = Display::img($file);
                 $this->pdf->WriteHTML('<html><body>'.$image.'</body></html>'.$page_break);
@@ -435,7 +435,7 @@ 
 
         //absolute path for frames.css //TODO: necessary?
         $absolute_css_path = api_get_path(WEB_CSS_PATH).api_get_setting('stylesheets').'/frames.css';
-        $document_html = str_replace('href="./css/frames.css"','href="'.$absolute_css_path.'"', $document_html);
+        $document_html = str_replace('href="./css/frames.css"', 'href="'.$absolute_css_path.'"', $document_html);
 
         $document_html = str_replace('../../', '', $document_html);
         $document_html = str_replace('../', '', $document_html);
@@ -463,7 +463,7 @@ 
                                 $old_src_fixed = str_replace(api_get_path(REL_COURSE_PATH).$course_data['path'].'/document/', '', $old_src);
                                 $old_src_fixed = str_replace('courses/'.$course_data['path'].'/document/', '', $old_src_fixed);
                                 $new_path = $document_path.$old_src_fixed;
-                                $document_html= str_replace($old_src, $new_path, $document_html);
+                                $document_html = str_replace($old_src, $new_path, $document_html);
 
                             }
                         }
@@ -482,7 +482,7 @@ 
         //$document_html= str_replace('temp_template_path', 'src="/main/default_course_document/', $document_html);// restore src templates
 
         api_set_encoding_html($document_html, 'UTF-8'); // The library mPDF expects UTF-8 encoded input data.
-        $title = api_get_title_html($document_html, 'UTF-8', 'UTF-8');  // TODO: Maybe it is better idea the title to be passed through
+        $title = api_get_title_html($document_html, 'UTF-8', 'UTF-8'); // TODO: Maybe it is better idea the title to be passed through
         // $_GET[] too, as it is done with file name.
         // At the moment the title is retrieved from the html document itself.
 
@@ -504,7 +504,7 @@ 
         //$this->pdf->Output($output_file, $outputMode); // F to save the pdf in a file
 
         if ($outputMode == 'F') {
-            $output_file = api_get_path(SYS_ARCHIVE_PATH) . $output_file;
+            $output_file = api_get_path(SYS_ARCHIVE_PATH).$output_file;
         }
 
         if ($saveInFile) {
@@ -537,14 +537,14 @@ 
         $web_path = false;
         if (!empty($course_code) && api_get_setting('pdf_export_watermark_by_course') == 'true') {
             $course_info = api_get_course_info($course_code);
-            $store_path = api_get_path(SYS_COURSE_PATH).$course_info['path'].'/'.api_get_current_access_url_id().'_pdf_watermark.png';   // course path
+            $store_path = api_get_path(SYS_COURSE_PATH).$course_info['path'].'/'.api_get_current_access_url_id().'_pdf_watermark.png'; // course path
             if (file_exists($store_path)) {
-                $web_path   = api_get_path(WEB_COURSE_PATH).$course_info['path'].'/'.api_get_current_access_url_id().'_pdf_watermark.png';
+                $web_path = api_get_path(WEB_COURSE_PATH).$course_info['path'].'/'.api_get_current_access_url_id().'_pdf_watermark.png';
             }
         } else {
-            $store_path = api_get_path(SYS_CODE_PATH).'default_course_document/images/'.api_get_current_access_url_id().'_pdf_watermark.png';   // course path
+            $store_path = api_get_path(SYS_CODE_PATH).'default_course_document/images/'.api_get_current_access_url_id().'_pdf_watermark.png'; // course path
             if (file_exists($store_path))
-                $web_path   = api_get_path(WEB_CODE_PATH).'default_course_document/images/'.api_get_current_access_url_id().'_pdf_watermark.png';
+                $web_path = api_get_path(WEB_CODE_PATH).'default_course_document/images/'.api_get_current_access_url_id().'_pdf_watermark.png';
         }
         return $web_path;
     }
@@ -584,10 +584,10 @@ 
     {
         if (!empty($course_code) && api_get_setting('pdf_export_watermark_by_course') == 'true') {
             $course_info = api_get_course_info($course_code);
-            $store_path = api_get_path(SYS_COURSE_PATH).$course_info['path'];   // course path
+            $store_path = api_get_path(SYS_COURSE_PATH).$course_info['path']; // course path
             $web_path   = api_get_path(WEB_COURSE_PATH).$course_info['path'].'/pdf_watermark.png';
         } else {
-            $store_path = api_get_path(SYS_CODE_PATH).'default_course_document/images';   // course path
+            $store_path = api_get_path(SYS_CODE_PATH).'default_course_document/images'; // course path
             $web_path   = api_get_path(WEB_CODE_PATH).'default_course_document/images/'.api_get_current_access_url_id().'_pdf_watermark.png';
         }
         $course_image = $store_path.'/'.api_get_current_access_url_id().'_pdf_watermark.png';
@@ -620,9 +620,9 @@ 
      */
     public function set_footer()
     {
-        $this->pdf->defaultfooterfontsize = 12;   // in pts
-        $this->pdf->defaultfooterfontstyle = 'B';   // blank, B, I, or BI
-        $this->pdf->defaultfooterline = 1;        // 1 to include line below header/above footer
+        $this->pdf->defaultfooterfontsize = 12; // in pts
+        $this->pdf->defaultfooterfontstyle = 'B'; // blank, B, I, or BI
+        $this->pdf->defaultfooterline = 1; // 1 to include line below header/above footer
         $platform_name   = api_get_setting('Institution');
         $left_content    = $platform_name;
         $center_content  = '';
@@ -689,9 +689,9 @@ 
      */
     public function set_header($course_data)
     {
-        $this->pdf->defaultheaderfontsize   = 10;   // in pts
-        $this->pdf->defaultheaderfontstyle  = 'BI';   // blank, B, I, or BI
-        $this->pdf->defaultheaderline       = 1;    // 1 to include line below header/above footer
+        $this->pdf->defaultheaderfontsize   = 10; // in pts
+        $this->pdf->defaultheaderfontstyle  = 'BI'; // blank, B, I, or BI
+        $this->pdf->defaultheaderline       = 1; // 1 to include line below header/above footer
 
         if (!empty($course_data['code'])) {
             $teacher_list = CourseManager::get_teacher_list_from_course_code($course_data['code']);
@@ -699,7 +699,7 @@ 
             $teachers = '';
             if (!empty($teacher_list)) {
                 foreach ($teacher_list as $teacher) {
-                    $teachers[]= $teacher['firstname'].' '.$teacher['lastname'];
+                    $teachers[] = $teacher['firstname'].' '.$teacher['lastname'];
                 }
                 if (count($teachers) > 1) {
                     $teachers = get_lang('Teachers').': '.implode(', ', $teachers);
@@ -766,7 +766,7 @@ 
                     'line' => 1,
                 ),
             );
-            $this->pdf->SetHeader($header);// ('{DATE j-m-Y}|{PAGENO}/{nb}|'.$title);
+            $this->pdf->SetHeader($header); // ('{DATE j-m-Y}|{PAGENO}/{nb}|'.$title);
         }
     }
 
@@ -811,7 +811,7 @@ 
         $this->pdf->directionality = api_get_text_direction();
         $this->pdf->useOnlyCoreFonts = true;
         // Use different Odd/Even headers and footers and mirror margins
-        $this->pdf->mirrorMargins       = 1;
+        $this->pdf->mirrorMargins = 1;
 
         // Add decoration only if not stated otherwise
         if ($complete) {
@@ -839,7 +839,7 @@ 
                     $watermark_text = api_get_setting('pdf_export_watermark_text');
                 }
                 if (!empty($watermark_text)) {
-                    $this->pdf->SetWatermarkText(strcode2utf($watermark_text),0.1);
+                    $this->pdf->SetWatermarkText(strcode2utf($watermark_text), 0.1);
                     $this->pdf->showWatermarkText = true;
                 }
             }
@@ -847,8 +847,8 @@ 
             if (empty($this->custom_header)) {
                 self::set_header($course_data);
             } else {
-                $this->pdf->SetHTMLHeader($this->custom_header,'E');
-                $this->pdf->SetHTMLHeader($this->custom_header,'O');
+                $this->pdf->SetHTMLHeader($this->custom_header, 'E');
+                $this->pdf->SetHTMLHeader($this->custom_header, 'O');
             }
 
             if (empty($this->custom_footer)) {

main/inc/lib/myspace.lib.php

Spacing +103 added lines, -103 removed lines

@@ -134,7 +134,7 @@ 
         $session_id  = intval($session_id);
 
         $sql = 'SELECT login_course_date, logout_course_date
-                FROM ' . $tbl_track_course . '
+                FROM ' . $tbl_track_course.'
                 WHERE
                     user_id = '.$user_id.' AND
                     c_id = '.$courseId.' AND
@@ -173,7 +173,7 @@ 
         foreach ($course_list as $course_item) {
             $courseInfo = api_get_course_info($course_item['code']);
             $courseId = $courseInfo['real_id'];
-            $new_course_list[] =  '"'.$courseId.'"';
+            $new_course_list[] = '"'.$courseId.'"';
         }
         $course_list = implode(', ', $new_course_list);
 
@@ -181,7 +181,7 @@ 
             return false;
         }
         $sql = 'SELECT login_course_date, logout_course_date, c_id
-                FROM ' . $tbl_track_course . '
+                FROM ' . $tbl_track_course.'
                 WHERE
                     user_id = '.$user_id.' AND
                     c_id IN ('.$course_list.') AND
@@ -259,7 +259,7 @@ 
             // student score
             $avg_score = Tracking :: get_avg_student_score($user_id, $courseCode);
             if (is_numeric($avg_score)) {
-                $avg_score = round($avg_score,2);
+                $avg_score = round($avg_score, 2);
             } else {
                 $$avg_score = '-';
             }
@@ -295,7 +295,7 @@ 
 
         $t_head = '    <table style="width: 100%;border:0;padding:0;border-collapse:collapse;table-layout: fixed">';
         //$t_head .= '  <caption>'.get_lang('CourseInformation').'</caption>';
-        $t_head .=      '<tr>';
+        $t_head .= '<tr>';
         $t_head .= '        <th width="155px" style="border-left:0;border-bottom:0"><span>'.get_lang('Course').'</span></th>';
         $t_head .= '        <th style="padding:0;border-bottom:0"><span>'.cut(get_lang('AvgTimeSpentInTheCourse'), 6, true).'</span></th>';
         $t_head .= '        <th style="padding:0;border-bottom:0"><span>'.cut(get_lang('AvgStudentsProgress'), 6, true).'</span></th>';
@@ -313,7 +313,7 @@ 
 
         $addparams = array('view' => 'admin', 'display' => 'user');
 
-        $table = new SortableTable('tracking_user_overview', array('MySpace','get_number_of_users_tracking_overview'), array('MySpace','get_user_data_tracking_overview'), 0);
+        $table = new SortableTable('tracking_user_overview', array('MySpace', 'get_number_of_users_tracking_overview'), array('MySpace', 'get_user_data_tracking_overview'), 0);
         $table->additional_parameters = $addparams;
 
         $table->set_header(0, get_lang('OfficialCode'), true, array('style' => 'font-size:8pt'), array('style' => 'font-size:8pt'));
@@ -326,7 +326,7 @@ 
         }
         $table->set_header(3, get_lang('LoginName'), true, array('style' => 'font-size:8pt'), array('style' => 'font-size:8pt'));
         $table->set_header(4, $t_head, false, array('style' => 'width:90%;border:0;padding:0;font-size:7.5pt;'), array('style' => 'width:90%;padding:0;font-size:7.5pt;'));
-        $table->set_column_filter(4, array('MySpace','course_info_tracking_filter'));
+        $table->set_column_filter(4, array('MySpace', 'course_info_tracking_filter'));
         $table->display();
     }
 
@@ -386,7 +386,7 @@ 
         $table -> set_header(7, get_lang('Sessions'), false);
 
         if ($is_western_name_order) {
-            $csv_header[] = array (
+            $csv_header[] = array(
                 get_lang('FirstName'),
                 get_lang('LastName'),
                 get_lang('TimeSpentOnThePlatform'),
@@ -396,7 +396,7 @@ 
                 get_lang('NumberOfSessions')
             );
         } else {
-            $csv_header[] = array (
+            $csv_header[] = array(
                 get_lang('LastName'),
                 get_lang('FirstName'),
                 get_lang('TimeSpentOnThePlatform'),
@@ -536,9 +536,9 @@ 
 
         if ($tracking_column != 3) {
             if ($tracking_direction == 'DESC') {
-                usort($all_datas, array('MySpace','rsort_users'));
+                usort($all_datas, array('MySpace', 'rsort_users'));
             } else {
-                usort($all_datas, array('MySpace','sort_users'));
+                usort($all_datas, array('MySpace', 'sort_users'));
             }
         }
 
@@ -597,7 +597,7 @@ 
         /**
          * Column config
          */
-        $column_model   = array(
+        $column_model = array(
             array(
                 'name' => 'username',
                 'index' => 'username',
@@ -638,7 +638,7 @@ 
 
         $action_links = '';
         // jqgrid will use this URL to do the selects
-        $url = api_get_path(WEB_AJAX_PATH).'model.ajax.php?a=get_session_lp_progress&session_id=' . $sessionId . '&course_id=' . $courseId . '&date_to=' . $date_to . '&date_from=' . $date_from;
+        $url = api_get_path(WEB_AJAX_PATH).'model.ajax.php?a=get_session_lp_progress&session_id='.$sessionId.'&course_id='.$courseId.'&date_to='.$date_to.'&date_from='.$date_from;
 
         //Table Id
         $tableId = 'lpProgress';
@@ -660,11 +660,11 @@ 
             true
         );
 
-        $return = '<script>$(function() {'. $table .
+        $return = '<script>$(function() {'.$table.
             'jQuery("#'.$tableId.'").jqGrid("navGrid","#'.$tableId.'_pager",{view:false, edit:false, add:false, del:false, search:false, excel:true});
                 jQuery("#'.$tableId.'").jqGrid("navButtonAdd","#'.$tableId.'_pager",{
                        caption:"",
-                       title:"' . get_lang('ExportExcel') . '",
+                       title:"' . get_lang('ExportExcel').'",
                        onClickButton : function () {
                            jQuery("#'.$tableId.'").jqGrid("excelExport",{"url":"'.$url.'&export_format=xls"});
                        }
@@ -714,7 +714,7 @@ 
         /**
          * Column config
          */
-        $column_model   = array(
+        $column_model = array(
             array('name'=>'session', 'index'=>'session', 'align'=>'left', 'search' => 'true', 'wrap_cell' => "true"),
             array('name'=>'exercise_id', 'index'=>'exercise_id', 'align'=>'left', 'search' => 'true'),
             array('name'=>'quiz_title', 'index'=>'quiz_title', 'align'=>'left', 'search' => 'true'),
@@ -731,7 +731,7 @@ 
         //get dynamic column names
 
         // jqgrid will use this URL to do the selects
-        $url = api_get_path(WEB_AJAX_PATH).'model.ajax.php?a=get_exercise_progress&session_id=' . $sessionId . '&course_id=' . $courseId  . '&exercise_id=' . $exerciseId . '&date_to=' . $date_to . '&date_from=' . $date_from;
+        $url = api_get_path(WEB_AJAX_PATH).'model.ajax.php?a=get_exercise_progress&session_id='.$sessionId.'&course_id='.$courseId.'&exercise_id='.$exerciseId.'&date_to='.$date_to.'&date_from='.$date_from;
 
         // Autowidth
         $extra_params['autowidth'] = 'true';
@@ -742,11 +742,11 @@ 
         $tableId = 'exerciseProgressOverview';
         $table = Display::grid_js($tableId, $url, $columns, $column_model, $extra_params, array(), '', true);
 
-        $return = '<script>$(function() {'. $table .
+        $return = '<script>$(function() {'.$table.
             'jQuery("#'.$tableId.'").jqGrid("navGrid","#'.$tableId.'_pager",{view:false, edit:false, add:false, del:false, search:false, excel:true});
                 jQuery("#'.$tableId.'").jqGrid("navButtonAdd","#'.$tableId.'_pager",{
                        caption:"",
-                       title:"' . get_lang('ExportExcel') . '",
+                       title:"' . get_lang('ExportExcel').'",
                        onClickButton : function () {
                            jQuery("#'.$tableId.'").jqGrid("excelExport",{"url":"'.$url.'&export_format=xls"});
                        }
@@ -806,7 +806,7 @@ 
                 // adding the fields that are checked to the session
                 $message = '';
                 foreach ($values as $field_ids => $value) {
-                    if ($value == 1 && strstr($field_ids,'extra_export_field')) {
+                    if ($value == 1 && strstr($field_ids, 'extra_export_field')) {
                         $_SESSION['additional_export_fields'][] = str_replace('extra_export_field', '', $field_ids);
                     }
                 }
@@ -821,7 +821,7 @@ 
                 // Displaying a feedback message
                 if (!empty($_SESSION['additional_export_fields'])) {
                     Display::display_confirmation_message(get_lang('FollowingFieldsWillAlsoBeExported').': <br /><ul>'.$message.'</ul>', false);
-                } else  {
+                } else {
                     Display::display_confirmation_message(get_lang('NoAdditionalFieldsWillBeExported'), false);
                 }
             } else {
@@ -849,7 +849,7 @@ 
     {
         $t_head = '    <table style="width: 100%;border:0;padding:0;border-collapse:collapse;table-layout: fixed">';
         //$t_head .= '  <caption>'.get_lang('CourseInformation').'</caption>';
-        $t_head .=      '<tr>';
+        $t_head .= '<tr>';
         $t_head .= '        <th style="padding:0;border-bottom:0"><span>'.cut(get_lang('AvgTimeSpentInTheCourse'), 6, true).'</span></th>';
         $t_head .= '        <th style="padding:0;border-bottom:0"><span>'.cut(get_lang('AvgStudentsProgress'), 6, true).'</span></th>';
         $t_head .= '        <th style="padding:0;border-bottom:0"><span>'.cut(get_lang('AvgCourseScore'), 6, true).'</span></th>';
@@ -866,13 +866,13 @@ 
 
         $addparams = array('view' => 'admin', 'display' => 'courseoverview');
 
-        $table = new SortableTable('tracking_session_overview', array('MySpace', 'get_total_number_courses'), array('MySpace','get_course_data_tracking_overview'), 1);
+        $table = new SortableTable('tracking_session_overview', array('MySpace', 'get_total_number_courses'), array('MySpace', 'get_course_data_tracking_overview'), 1);
         $table->additional_parameters = $addparams;
 
         $table->set_header(0, '', false, null, array('style' => 'display: none'));
         $table->set_header(1, get_lang('Course'), true, array('style' => 'font-size:8pt'), array('style' => 'font-size:8pt'));
         $table->set_header(2, $t_head, false, array('style' => 'width:90%;border:0;padding:0;font-size:7.5pt;'), array('style' => 'width:90%;padding:0;font-size:7.5pt;'));
-        $table->set_column_filter(2, array('MySpace','course_tracking_filter'));
+        $table->set_column_filter(2, array('MySpace', 'course_tracking_filter'));
         $table->display();
     }
 
@@ -908,7 +908,7 @@ 
         $sql .= " ORDER BY col$column $direction ";
         $sql .= " LIMIT $from,$number_of_items";
         $result = Database::query($sql);
-        $return = array ();
+        $return = array();
         while ($course = Database::fetch_row($result)) {
             $return[] = $course;
         }
@@ -961,18 +961,18 @@ 
             $progress += $progress_tmp[0];
             $nb_progress_lp += $progress_tmp[1];
             $score_tmp = Tracking :: get_avg_student_score($row->user_id, $course_code, array(), null, true);
-            if(is_array($score_tmp)) {
+            if (is_array($score_tmp)) {
                 $score += $score_tmp[0];
                 $nb_score_lp += $score_tmp[1];
             }
             $nb_messages += Tracking::count_student_messages($row->user_id, $course_code);
             $nb_assignments += Tracking::count_student_assignments($row->user_id, $course_code);
             $last_login_date_tmp = Tracking :: get_last_connection_date_on_the_course($row->user_id, $courseInfo, null, false);
-            if($last_login_date_tmp != false && $last_login_date == false) { // TODO: To be cleaned
+            if ($last_login_date_tmp != false && $last_login_date == false) { // TODO: To be cleaned
                 $last_login_date = $last_login_date_tmp;
-            } else if($last_login_date_tmp != false && $last_login_date != false) { // TODO: Repeated previous condition. To be cleaned.
+            } else if ($last_login_date_tmp != false && $last_login_date != false) { // TODO: Repeated previous condition. To be cleaned.
                 // Find the max and assign it to first_login_date
-                if(strtotime($last_login_date_tmp) > strtotime($last_login_date)) {
+                if (strtotime($last_login_date_tmp) > strtotime($last_login_date)) {
                     $last_login_date = $last_login_date_tmp;
                 }
             }
@@ -1110,7 +1110,7 @@ 
                 $progress += $progress_tmp[0];
                 $nb_progress_lp += $progress_tmp[1];
                 $score_tmp = Tracking :: get_avg_student_score($row->user_id, $course_code, array(), null, true);
-                if(is_array($score_tmp)) {
+                if (is_array($score_tmp)) {
                     $score += $score_tmp[0];
                     $nb_score_lp += $score_tmp[1];
                 }
@@ -1118,11 +1118,11 @@ 
                 $nb_assignments += Tracking::count_student_assignments($row->user_id, $course_code);
 
                 $last_login_date_tmp = Tracking::get_last_connection_date_on_the_course($row->user_id, $courseInfo, null, false);
-                if($last_login_date_tmp != false && $last_login_date == false) { // TODO: To be cleaned.
+                if ($last_login_date_tmp != false && $last_login_date == false) { // TODO: To be cleaned.
                     $last_login_date = $last_login_date_tmp;
-                } else if($last_login_date_tmp != false && $last_login_date == false) { // TODO: Repeated previous condition. To be cleaned.
+                } else if ($last_login_date_tmp != false && $last_login_date == false) { // TODO: Repeated previous condition. To be cleaned.
                     // Find the max and assign it to first_login_date
-                    if(strtotime($last_login_date_tmp) > strtotime($last_login_date)) {
+                    if (strtotime($last_login_date_tmp) > strtotime($last_login_date)) {
                         $last_login_date = $last_login_date_tmp;
                     }
                 }
@@ -1132,22 +1132,22 @@ 
                 $total_score_possible += $exercise_results_tmp['score_possible'];
                 $total_questions_answered += $exercise_results_tmp['questions_answered'];
             }
-            if($nb_progress_lp > 0) {
+            if ($nb_progress_lp > 0) {
                 $avg_progress = round($progress / $nb_progress_lp, 2);
             } else {
                 $avg_progress = 0;
             }
-            if($nb_score_lp > 0) {
+            if ($nb_score_lp > 0) {
                 $avg_score = round($score / $nb_score_lp, 2);
             } else {
                 $avg_score = '-';
             }
-            if($last_login_date) {
+            if ($last_login_date) {
                 $last_login_date = api_convert_and_format_date($last_login_date, DATE_FORMAT_SHORT, date_default_timezone_get());
             } else {
                 $last_login_date = '-';
             }
-            if($total_score_possible > 0) {
+            if ($total_score_possible > 0) {
                 $total_score_percentage = round($total_score_obtained / $total_score_possible * 100, 2);
             } else {
                 $total_score_percentage = 0;
@@ -1183,7 +1183,7 @@ 
     {
         $t_head = '    <table style="width: 100%;border:0;padding:0;border-collapse:collapse;table-layout: fixed">';
         //$t_head .= '  <caption>'.get_lang('CourseInformation').'</caption>';
-        $t_head .=      '<tr>';
+        $t_head .= '<tr>';
         $t_head .= '        <th width="155px" style="border-left:0;border-bottom:0"><span>'.get_lang('Course').'</span></th>';
         $t_head .= '        <th style="padding:0;border-bottom:0"><span>'.cut(get_lang('AvgTimeSpentInTheCourse'), 6, true).'</span></th>';
         $t_head .= '        <th style="padding:0;border-bottom:0"><span>'.cut(get_lang('AvgStudentsProgress'), 6, true).'</span></th>';
@@ -1201,7 +1201,7 @@ 
 
         $addparams = array('view' => 'admin', 'display' => 'sessionoverview');
 
-        $table = new SortableTable('tracking_session_overview', array('MySpace','get_total_number_sessions'), array('MySpace','get_session_data_tracking_overview'), 1);
+        $table = new SortableTable('tracking_session_overview', array('MySpace', 'get_total_number_sessions'), array('MySpace', 'get_session_data_tracking_overview'), 1);
         $table->additional_parameters = $addparams;
 
         $table->set_header(0, '', false, null, array('style' => 'display: none'));
@@ -1240,7 +1240,7 @@ 
         $sql .= " ORDER BY col$column $direction ";
         $sql .= " LIMIT $from,$number_of_items";
         $result = Database::query($sql);
-        $return = array ();
+        $return = array();
         while ($session = Database::fetch_row($result)) {
             $return[] = $session;
         }
@@ -1329,10 +1329,10 @@ 
                 if ($last_login_date_tmp != false && $last_login_date == false) {
                     // TODO: To be cleaned.
                     $last_login_date = $last_login_date_tmp;
-                } else if($last_login_date_tmp != false && $last_login_date != false) {
+                } else if ($last_login_date_tmp != false && $last_login_date != false) {
                     // TODO: Repeated previous condition! To be cleaned.
                     // Find the max and assign it to first_login_date
-                    if(strtotime($last_login_date_tmp) > strtotime($last_login_date)) {
+                    if (strtotime($last_login_date_tmp) > strtotime($last_login_date)) {
                         $last_login_date = $last_login_date_tmp;
                     }
                 }
@@ -1342,27 +1342,27 @@ 
                 $total_score_possible += $exercise_results_tmp['score_possible'];
                 $total_questions_answered += $exercise_results_tmp['questions_answered'];
             }
-            if($nb_progress_lp > 0) {
+            if ($nb_progress_lp > 0) {
                 $avg_progress = round($progress / $nb_progress_lp, 2);
             } else {
                 $avg_progress = 0;
             }
-            if($nb_score_lp > 0) {
+            if ($nb_score_lp > 0) {
                 $avg_score = round($score / $nb_score_lp, 2);
             } else {
                 $avg_score = '-';
             }
-            if($last_login_date) {
+            if ($last_login_date) {
                 $last_login_date = api_convert_and_format_date($last_login_date, DATE_FORMAT_SHORT, date_default_timezone_get());
             } else {
                 $last_login_date = '-';
             }
-            if($total_score_possible > 0) {
+            if ($total_score_possible > 0) {
                 $total_score_percentage = round($total_score_obtained / $total_score_possible * 100, 2);
             } else {
                 $total_score_percentage = 0;
             }
-            if($total_score_percentage > 0) {
+            if ($total_score_percentage > 0) {
                 $total_score = $total_score_obtained.'/'.$total_score_possible.' ('.$total_score_percentage.' %)';
             } else {
                 $total_score = '-';
@@ -1473,7 +1473,7 @@ 
                 $total_score_obtained = 0;
                 $total_score_possible = 0;
                 $total_questions_answered = 0;
-                while($row_user = Database::fetch_object($result_users)) {
+                while ($row_user = Database::fetch_object($result_users)) {
                     // get time spent in the course and session
                     $time_spent += Tracking::get_time_spent_on_the_course($row_user->user_id, $courseId, $session_id);
                     $progress_tmp = Tracking::get_avg_student_progress($row_user->user_id, $row->code, array(), $session_id, true);
@@ -1502,11 +1502,11 @@ 
                         $session_id,
                         false
                     );
-                    if($last_login_date_tmp != false && $last_login_date == false) { // TODO: To be cleaned.
+                    if ($last_login_date_tmp != false && $last_login_date == false) { // TODO: To be cleaned.
                         $last_login_date = $last_login_date_tmp;
-                    } else if($last_login_date_tmp != false && $last_login_date == false) { // TODO: Repeated previous condition. To be cleaned.
+                    } else if ($last_login_date_tmp != false && $last_login_date == false) { // TODO: Repeated previous condition. To be cleaned.
                         // Find the max and assign it to first_login_date
-                        if(strtotime($last_login_date_tmp) > strtotime($last_login_date)) {
+                        if (strtotime($last_login_date_tmp) > strtotime($last_login_date)) {
                             $last_login_date = $last_login_date_tmp;
                         }
                     }
@@ -1516,27 +1516,27 @@ 
                     $total_score_possible += $exercise_results_tmp['score_possible'];
                     $total_questions_answered += $exercise_results_tmp['questions_answered'];
                 }
-                if($nb_progress_lp > 0) {
+                if ($nb_progress_lp > 0) {
                     $avg_progress = round($progress / $nb_progress_lp, 2);
                 } else {
                     $avg_progress = 0;
                 }
-                if($nb_score_lp > 0) {
+                if ($nb_score_lp > 0) {
                     $avg_score = round($score / $nb_score_lp, 2);
                 } else {
                     $avg_score = '-';
                 }
-                if($last_login_date) {
+                if ($last_login_date) {
                     $last_login_date = api_convert_and_format_date($last_login_date, DATE_FORMAT_SHORT, date_default_timezone_get());
                 } else {
                     $last_login_date = '-';
                 }
-                if($total_score_possible > 0) {
+                if ($total_score_possible > 0) {
                     $total_score_percentage = round($total_score_obtained / $total_score_possible * 100, 2);
                 } else {
                     $total_score_percentage = 0;
                 }
-                if($total_score_percentage > 0) {
+                if ($total_score_percentage > 0) {
                     $total_score = $total_score_obtained.'/'.$total_score_possible.' ('.$total_score_percentage.' %)';
                 } else {
                     $total_score = '-';
@@ -1596,7 +1596,7 @@ 
         while ($row = Database::fetch_array($result)) {
             $score_obtained += $row['exe_result'];
             $score_possible += $row['exe_weighting'];
-            $questions_answered ++;
+            $questions_answered++;
         }
 
         if ($score_possible != 0) {
@@ -1716,9 +1716,9 @@ 
                     }
                 }
                 // time spent in the course
-                $csv_row[] = api_time_to_hms(Tracking::get_time_spent_on_the_course ($user[4], $courseId));
+                $csv_row[] = api_time_to_hms(Tracking::get_time_spent_on_the_course($user[4], $courseId));
                 // student progress in course
-                $csv_row[] = round(Tracking::get_avg_student_progress ($user[4], $row[0]), 2);
+                $csv_row[] = round(Tracking::get_avg_student_progress($user[4], $row[0]), 2);
                 // student score
                 $csv_row[] = round(Tracking::get_avg_student_score($user[4], $row[0]), 2);
                 // student tes score
@@ -1726,7 +1726,7 @@ 
                 // student messages
                 $csv_row[] = Tracking::count_student_messages($user[4], $row[0]);
                 // student assignments
-                $csv_row[] = Tracking::count_student_assignments ($user[4], $row[0]);
+                $csv_row[] = Tracking::count_student_assignments($user[4], $row[0]);
                 // student exercises results
                 $exercises_results = MySpace::exercises_results($user[4], $row[0]);
                 $csv_row[] = $exercises_results['score_obtained'];
@@ -1734,7 +1734,7 @@ 
                 $csv_row[] = $exercises_results['questions_answered'];
                 $csv_row[] = $exercises_results['percentage'];
                 // first connection
-                $csv_row[] = Tracking::get_first_connection_date_on_the_course ($user[4], $courseId);
+                $csv_row[] = Tracking::get_first_connection_date_on_the_course($user[4], $courseId);
                 // last connection
                 $csv_row[] = strip_tags(Tracking::get_last_connection_date_on_the_course($user[4], $courseInfo));
 
@@ -1768,9 +1768,9 @@ 
         // get all courses with limit
         $sql = "SELECT course.code as col1, course.title as col2
                 FROM $tbl_course course
-                WHERE course.code IN (".implode(',',$courses_code).")";
+                WHERE course.code IN (".implode(',', $courses_code).")";
 
-        if (!in_array($direction, array('ASC','DESC'))) $direction = 'ASC';
+        if (!in_array($direction, array('ASC', 'DESC'))) $direction = 'ASC';
 
         $column = intval($column);
         $from = intval($from);
@@ -1839,7 +1839,7 @@ 
             $table_row[] = $avg_assignments_in_course;
 
             //set the "from" value to know if I access the Reporting by the chamilo tab or the course link
-            $table_row[] = '<center><a href="../../tracking/courseLog.php?cidReq=' .$course_code.'&from=myspace&id_session='.$session_id.'">
+            $table_row[] = '<center><a href="../../tracking/courseLog.php?cidReq='.$course_code.'&from=myspace&id_session='.$session_id.'">
                              '.Display::return_icon('2rightarrow.png').'
                              </a>
                             </center>';
@@ -1848,7 +1848,7 @@ 
                 $nb_students_in_course,
                 $avg_time_spent_in_course,
                 is_null($avg_progress_in_course) ? null : $avg_progress_in_course.'%',
-                is_null($avg_score_in_course) ? null : is_numeric($avg_score_in_course) ? $avg_score_in_course.'%' : $avg_score_in_course ,
+                is_null($avg_score_in_course) ? null : is_numeric($avg_score_in_course) ? $avg_score_in_course.'%' : $avg_score_in_course,
                 is_null($avg_score_in_exercise) ? null : $avg_score_in_exercise.'%',
                 $avg_messages_in_course,
                 $avg_assignments_in_course,
@@ -1916,7 +1916,7 @@ 
         $sql .= " ORDER BY col$column $direction ";
         $sql .= " LIMIT $from,$number_of_items";
         $result = Database::query($sql);
-        $return = array ();
+        $return = array();
         while ($user = Database::fetch_row($result)) {
             $return[] = $user;
         }
@@ -1971,7 +1971,7 @@ 
                     $i++;
                 }
             }
-            $username_array = array('username' => $desired_username , 'sufix' => $sufix);
+            $username_array = array('username' => $desired_username, 'sufix' => $sufix);
             return $username_array;
         } else {
             $username_array = array('username' => $username, 'sufix' => '');
@@ -2246,7 +2246,7 @@ 
         foreach ($users as $index => $user) {
             $userid = $user['id'];
             $sql_insert = "INSERT IGNORE INTO $tbl_session_rel_user(session_id, user_id, registered_at)
-                           VALUES ('$id_session','$userid', '" . api_get_utc_datetime() . "')";
+                           VALUES ('$id_session','$userid', '".api_get_utc_datetime()."')";
             Database::query($sql_insert);
             $user['added_at_session'] = 1;
             $new_users[] = $user;
@@ -2280,10 +2280,10 @@ 
                 );
                 $userInfo = api_get_user_info($user['id']);
 
-                if (($user['added_at_platform'] == 1  && $user['added_at_session'] == 1) || $user['added_at_session'] == 1) {
+                if (($user['added_at_platform'] == 1 && $user['added_at_session'] == 1) || $user['added_at_session'] == 1) {
                     if ($user['added_at_platform'] == 1) {
                         $addedto = get_lang('UserCreatedPlatform');
-                    } else  {
+                    } else {
                         $addedto = '          ';
                     }
 
@@ -2346,7 +2346,7 @@ 
         global $current_tag;
         switch ($data) {
             case 'Contact' :
-                $user = array ();
+                $user = array();
                 break;
             default :
                 $current_tag = $data;
@@ -2393,9 +2393,9 @@ 
         global $current_value;
         global $user;
         global $users;
-        $users = array ();
+        $users = array();
         $parser = xml_parser_create('UTF-8');
-        xml_set_element_handler($parser, array('MySpace','element_start'), array('MySpace','element_end'));
+        xml_set_element_handler($parser, array('MySpace', 'element_start'), array('MySpace', 'element_end'));
         xml_set_character_data_handler($parser, "character_data");
         xml_parser_set_option($parser, XML_OPTION_CASE_FOLDING, false);
         xml_parse($parser, api_utf8_encode_xml(file_get_contents($file)));
@@ -2441,7 +2441,7 @@ 
             get_lang('SearchCourse'),
             $courseList,
             [
-                'url' => api_get_path(WEB_AJAX_PATH) . 'course.ajax.php?' . http_build_query([
+                'url' => api_get_path(WEB_AJAX_PATH).'course.ajax.php?'.http_build_query([
                     'a' => 'search_course_by_session_all',
                     'session_id' => $sessionId
                 ])
@@ -2460,7 +2460,7 @@ 
                             course_id: $('#course_id').val() || 0
                         });
 
-                        return '" . api_get_path(WEB_AJAX_PATH) . "session.ajax.php?' + params;
+                        return '" . api_get_path(WEB_AJAX_PATH)."session.ajax.php?' + params;
                     }
                 "
             ]
@@ -2491,7 +2491,7 @@ 
                             course_id: $('#course_id').val()
                         });
 
-                        return '" . api_get_path(WEB_AJAX_PATH) . "course.ajax.php?' + params;
+                        return '" . api_get_path(WEB_AJAX_PATH)."course.ajax.php?' + params;
                     }
                 "
             ]
@@ -2579,7 +2579,7 @@ 
                     " : "
                         u.lastname AS col2,
                         u.firstname AS col3,
-                " ) . "
+                " )."
                 a.logout_course_date,
                 c.title,
                 c.code,
@@ -2590,7 +2590,7 @@ 
 
         if (isset($_GET['session_id']) && !empty($_GET['session_id'])) {
             $sessionId = intval($_GET['session_id']);
-            $sql .= " WHERE a.session_id = " . $sessionId;
+            $sql .= " WHERE a.session_id = ".$sessionId;
         }
 
         $sql .= " ORDER BY col$column $orderDirection ";
@@ -2696,13 +2696,13 @@ 
 function get_stats($user_id, $courseId, $start_date = null, $end_date = null)
 {
     // Database table definitions
-    $tbl_track_course   = Database :: get_main_table(TABLE_STATISTIC_TRACK_E_COURSE_ACCESS);
+    $tbl_track_course = Database :: get_main_table(TABLE_STATISTIC_TRACK_E_COURSE_ACCESS);
 
     $course_info = api_get_course_info_by_id($courseId);
     if (!empty($course_info)) {
         $strg_sd = '';
         $strg_ed = '';
-        if ($start_date != null && $end_date != null){
+        if ($start_date != null && $end_date != null) {
             $end_date = add_day_to($end_date);
             $strg_sd = "AND login_course_date BETWEEN '$start_date' AND '$end_date'";
             $strg_ed = "AND logout_course_date BETWEEN '$start_date' AND '$end_date'";
@@ -2711,10 +2711,10 @@ 
                 SEC_TO_TIME(avg(time_to_sec(timediff(logout_course_date,login_course_date)))) as avrg,
                 SEC_TO_TIME(sum(time_to_sec(timediff(logout_course_date,login_course_date)))) as total,
                 count(user_id) as times
-                FROM ' . $tbl_track_course . '
+                FROM ' . $tbl_track_course.'
                 WHERE
-                    user_id = ' . intval($user_id) . ' AND
-                    c_id = ' . intval($courseId) . ' '.$strg_sd.' '.$strg_ed.' '.'
+                    user_id = ' . intval($user_id).' AND
+                    c_id = ' . intval($courseId).' '.$strg_sd.' '.$strg_ed.' '.'
                 ORDER BY login_course_date ASC';
 
         $rs = Database::query($sql);
@@ -2732,7 +2732,7 @@ 
 }
 
 function add_day_to($end_date) {
-    $foo_date = strtotime( $end_date );
+    $foo_date = strtotime($end_date);
     $foo_date = strtotime(" +1 day", $foo_date);
     $foo_date = date("Y-m-d", $foo_date);
     return $foo_date;
@@ -2746,7 +2746,7 @@ 
  * @version OCT-22- 2010
  * @return array
  */
-function convert_to_array($sql_result){
+function convert_to_array($sql_result) {
     $result_to_print = '<table>';
     foreach ($sql_result as $key => $data) {
         $result_to_print .= '<tr><td>'.date('d-m-Y (H:i:s)', $data['login']).'</td><td>'.api_time_to_hms($data['logout'] - $data['login']).'</tr></td>'."\n";
@@ -2765,7 +2765,7 @@ 
  * @version OCT-22- 2010
  * @return string
  */
-function convert_to_string($sql_result){
+function convert_to_string($sql_result) {
     $result_to_print = '<table>';
     if (!empty($sql_result)) {
         foreach ($sql_result as $key => $data) {
@@ -2790,18 +2790,18 @@ 
  */
 function grapher($sql_result, $start_date, $end_date, $type = "")
 {
-    if (empty($start_date)) { $start_date =""; }
-    if (empty($end_date)) { $end_date =""; }
-    if ($type == ""){ $type = 'day'; }
-    $main_year  = $main_month_year = $main_day = array();
+    if (empty($start_date)) { $start_date = ""; }
+    if (empty($end_date)) { $end_date = ""; }
+    if ($type == "") { $type = 'day'; }
+    $main_year = $main_month_year = $main_day = array();
     // get last 8 days/months
     $last_days      = 5;
     $last_months    = 3;
     for ($i = $last_days; $i >= 0; $i--) {
-        $main_day[date ('d-m-Y', time () - $i * 3600 * 24)] = 0;
+        $main_day[date('d-m-Y', time() - $i * 3600 * 24)] = 0;
     }
     for ($i = $last_months; $i >= 0; $i--) {
-        $main_month_year[date ('m-Y', time () - $i * 30 * 3600 * 24)] = 0;
+        $main_month_year[date('m-Y', time() - $i * 30 * 3600 * 24)] = 0;
     }
 
     $i = 0;
@@ -2837,7 +2837,7 @@ 
         /* Create and populate the pData object */
         $myData = new pData();
         $myData->addPoints($main_date, 'Serie1');
-        if (count($main_date)!= 1) {
+        if (count($main_date) != 1) {
             $myData->addPoints($labels, 'Labels');
             $myData->setSerieDescription('Labels', 'Months');
             $myData->setAbscissa('Labels');
@@ -2845,7 +2845,7 @@ 
         $myData->setSerieWeight('Serie1', 1);
         $myData->setSerieDescription('Serie1', get_lang('MyResults'));
         $myData->setAxisName(0, get_lang('Minutes'));
-        $myData->loadPalette(api_get_path(SYS_CODE_PATH) . 'palettes/pchart/default.color', true);
+        $myData->loadPalette(api_get_path(SYS_CODE_PATH).'palettes/pchart/default.color', true);
 
         // Cache definition
         $cachePath = api_get_path(SYS_ARCHIVE_PATH);
@@ -2854,9 +2854,9 @@ 
 
         if ($myCache->isInCache($chartHash)) {
             //if we already created the img
-            $imgPath = api_get_path(SYS_ARCHIVE_PATH) . $chartHash;
+            $imgPath = api_get_path(SYS_ARCHIVE_PATH).$chartHash;
             $myCache->saveFromCache($chartHash, $imgPath);
-            $imgPath = api_get_path(WEB_ARCHIVE_PATH) . $chartHash;
+            $imgPath = api_get_path(WEB_ARCHIVE_PATH).$chartHash;
         } else {
             /* Define width, height and angle */
             $mainWidth = 760;
@@ -2885,7 +2885,7 @@ 
             /* Set the default font */
             $myPicture->setFontProperties(
                 array(
-                    "FontName" => api_get_path(SYS_FONTS_PATH) . 'opensans/OpenSans-Regular.ttf',
+                    "FontName" => api_get_path(SYS_FONTS_PATH).'opensans/OpenSans-Regular.ttf',
                     "FontSize" => 10)
             );
             /* Write the chart title */
@@ -2902,7 +2902,7 @@ 
             /* Set the default font */
             $myPicture->setFontProperties(
                 array(
-                    "FontName" => api_get_path(SYS_FONTS_PATH) . 'opensans/OpenSans-Regular.ttf',
+                    "FontName" => api_get_path(SYS_FONTS_PATH).'opensans/OpenSans-Regular.ttf',
                     "FontSize" => 8
                 )
             );
@@ -2944,7 +2944,7 @@ 
             /* Draw the line chart */
             $myPicture->setFontProperties(
                 array(
-                    "FontName" => api_get_path(SYS_FONTS_PATH) . 'opensans/OpenSans-Regular.ttf',
+                    "FontName" => api_get_path(SYS_FONTS_PATH).'opensans/OpenSans-Regular.ttf',
                     "FontSize" => 10
                 )
             );
@@ -2963,15 +2963,15 @@ 
 
             /* Write and save into cache */
             $myCache->writeToCache($chartHash, $myPicture);
-            $imgPath = api_get_path(SYS_ARCHIVE_PATH) . $chartHash;
+            $imgPath = api_get_path(SYS_ARCHIVE_PATH).$chartHash;
             $myCache->saveFromCache($chartHash, $imgPath);
-            $imgPath = api_get_path(WEB_ARCHIVE_PATH) . $chartHash;
+            $imgPath = api_get_path(WEB_ARCHIVE_PATH).$chartHash;
         }
-        $html = '<img src="' . $imgPath . '">';
+        $html = '<img src="'.$imgPath.'">';
 
         return $html;
     } else {
-        $foo_img = api_convert_encoding('<div id="messages" class="warning-message">'.get_lang('GraphicNotAvailable').'</div>','UTF-8');
+        $foo_img = api_convert_encoding('<div id="messages" class="warning-message">'.get_lang('GraphicNotAvailable').'</div>', 'UTF-8');
 
         return $foo_img;
     }

main/mySpace/access_details.php

Spacing +2 added lines, -2 removed lines

@@ -126,7 +126,7 @@ 
 
 echo Display::page_header(get_lang('DetailsStudentInCourse'));
 echo Display::page_subheader(
-    get_lang('User').': '.$userInfo['complete_name'].' - '.get_lang('Course').': '.$courseInfo['title'] . ' (' . $course_code . ')'
+    get_lang('User').': '.$userInfo['complete_name'].' - '.get_lang('Course').': '.$courseInfo['title'].' ('.$course_code.')'
 );
 
 $form->setDefaults(array('from' => $from, 'to' => $to));
@@ -138,7 +138,7 @@ 
     </div><br />
     <div id="cev_cont_stats">
     <?php
-    if ($result_to_print != "")  {
+    if ($result_to_print != "") {
         $rst = get_stats($user_id, $courseId);
         $foo_stats = '<strong>'.get_lang('Total').': </strong>'.$rst['total'].'<br />';
         $foo_stats .= '<strong>'.get_lang('Average').': </strong>'.$rst['avg'].'<br />';

main/lp/lp_list_search.php

Spacing +13 added lines, -13 removed lines

@@ -22,7 +22,7 @@ 
 }
 
 if (!empty($gradebook) && $gradebook == 'view') {
-    $interbreadcrumb[]= array(
+    $interbreadcrumb[] = array(
         'url' => '../gradebook/'.$_SESSION['gradebook_dest'],
         'name' => get_lang('ToolGradebook')
     );
@@ -62,8 +62,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) {
@@ -75,7 +75,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);
                 }
             }
         }
@@ -93,9 +93,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)) {
@@ -145,12 +145,12 @@ 
         }
 
         if ($mode == 'gallery') {
-            $title = $a_prefix.str_replace('_', ' ', $result['title']). $a_suffix;
+            $title = $a_prefix.str_replace('_', ' ', $result['title']).$a_suffix;
             $blocks[] = array(1 =>
-                $a_prefix .'<img src="'.$result['thumbnail'].'" />'. $a_suffix .'<br />'.$title.'<br />'.$result['author'],
+                $a_prefix.'<img src="'.$result['thumbnail'].'" />'.$a_suffix.'<br />'.$title.'<br />'.$result['author'],
             );
         } else {
-            $title = '<div style="text-align:left;">'. $a_prefix . $result['title']. $a_suffix .(!empty($result['author']) ? ' '.$result['author'] : '').'<div>';
+            $title = '<div style="text-align:left;">'.$a_prefix.$result['title'].$a_suffix.(!empty($result['author']) ? ' '.$result['author'] : '').'<div>';
             $blocks[] = array(1 => $title);
         }
     }
@@ -169,12 +169,12 @@ 
     );
     $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 .= '&';
         }

main/lp/learnpathItem.class.php

Spacing +243 added lines, -243 removed lines

@@ -199,13 +199,13 @@ 
             $res = Database::query($sql);
             if (Database::num_rows($res) > 0) {
                 $se_ref = Database::fetch_array($res);
-                $this->search_did = (int)$se_ref['search_did'];
+                $this->search_did = (int) $se_ref['search_did'];
             }
         }
         $this->audio = $row['audio'];
         if (self::debug > 0) {
             error_log(
-                'New LP - End of learnpathItem constructor for item ' . $id,
+                'New LP - End of learnpathItem constructor for item '.$id,
                 0
             );
         }
@@ -314,7 +314,7 @@ 
     public function delete()
     {
         if (self::debug > 0) {
-            error_log('learnpath_item::delete() for item ' . $this->db_id, 0);
+            error_log('learnpath_item::delete() for item '.$this->db_id, 0);
         }
         $lp_item_view = Database::get_course_table(TABLE_LP_ITEM_VIEW);
         $lp_item = Database::get_course_table(TABLE_LP_ITEM);
@@ -322,29 +322,29 @@ 
         $course_id = api_get_course_int_id();
 
         $sql = "DELETE FROM $lp_item_view
-                WHERE c_id = $course_id AND lp_item_id = " . $this->db_id;
+                WHERE c_id = $course_id AND lp_item_id = ".$this->db_id;
         if (self::debug > 0) {
-            error_log('Deleting from lp_item_view: ' . $sql, 0);
+            error_log('Deleting from lp_item_view: '.$sql, 0);
         }
         Database::query($sql);
 
         $sql = "SELECT * FROM $lp_item
-                WHERE c_id = $course_id AND id = " . $this->db_id;
+                WHERE c_id = $course_id AND id = ".$this->db_id;
         $res_sel = Database::query($sql);
         if (Database::num_rows($res_sel) < 1) {
             return false;
         }
 
         $sql = "DELETE FROM $lp_item
-                WHERE c_id = $course_id AND id = " . $this->db_id;
+                WHERE c_id = $course_id AND id = ".$this->db_id;
         Database::query($sql);
         if (self::debug > 0) {
-            error_log('Deleting from lp_item: ' . $sql);
+            error_log('Deleting from lp_item: '.$sql);
         }
 
         if (api_get_setting('search_enabled') == 'true') {
             if (!is_null($this->search_did)) {
-                require_once api_get_path(LIBRARY_PATH) . 'search/ChamiloIndexer.class.php';
+                require_once api_get_path(LIBRARY_PATH).'search/ChamiloIndexer.class.php';
                 $di = new ChamiloIndexer();
                 $di->remove_document($this->search_did);
             }
@@ -380,7 +380,7 @@ 
     {
         if (self::debug > 0) {
             error_log(
-                'learnpathItem::get_attempt_id() on item ' . $this->db_id,
+                'learnpathItem::get_attempt_id() on item '.$this->db_id,
                 0
             );
         }
@@ -390,8 +390,8 @@ 
         }
         if (self::debug > 0) {
             error_log(
-                'New LP - End of learnpathItem::get_attempt_id() on item ' .
-                $this->db_id . ' - Returning ' . $res,
+                'New LP - End of learnpathItem::get_attempt_id() on item '.
+                $this->db_id.' - Returning '.$res,
                 0
             );
         }
@@ -453,12 +453,12 @@ 
             if (self::debug > 2) {
                 error_log(
                     'learnpathItem::get_credit() - get_prevent_reinit!=0 and '.
-                    'status is ' . $status,
+                    'status is '.$status,
                     0
                 );
             }
             //0=not attempted - 1 = incomplete
-            if ($status != $this->possible_status[0]&&
+            if ($status != $this->possible_status[0] &&
                 $status != $this->possible_status[1]
             ) {
                 $credit = 'no-credit';
@@ -530,13 +530,13 @@ 
                 case TOOL_DOCUMENT:
                     $table_doc = Database::get_course_table(TABLE_DOCUMENT);
                     $sql = 'SELECT path
-                            FROM ' . $table_doc . '
+                            FROM ' . $table_doc.'
                             WHERE
-                                c_id = ' . $course_id . ' AND
+                                c_id = ' . $course_id.' AND
                                 id = ' . $path;
                     $res = Database::query($sql);
                     $row = Database::fetch_array($res);
-                    $real_path = 'document' . $row['path'];
+                    $real_path = 'document'.$row['path'];
                     return $real_path;
                 case TOOL_STUDENTPUBLICATION:
                 case TOOL_QUIZ:
@@ -548,7 +548,7 @@ 
             }
         } else {
             if (!empty($path_to_scorm_dir)) {
-                $path = $path_to_scorm_dir . $path;
+                $path = $path_to_scorm_dir.$path;
             }
             return $path;
         }
@@ -584,8 +584,8 @@ 
         $sql = "SELECT id FROM $tbl
                 WHERE
                     c_id = $course_id AND
-                    lp_item_id = " . $this->db_id . " AND
-                    lp_view_id = " . $this->view_id . " AND
+                    lp_item_id = ".$this->db_id." AND
+                    lp_view_id = " . $this->view_id." AND
                     view_count = " . $this->attempt_id;
         $res = Database::query($sql);
         if (Database::num_rows($res) > 0) {
@@ -628,8 +628,8 @@ 
             $sql = "SELECT id FROM $tbl
                     WHERE
                         c_id = $course_id AND
-                        lp_item_id = " . $this->db_id . " AND
-                        lp_view_id = " . $this->view_id ." AND
+                        lp_item_id = ".$this->db_id." AND
+                        lp_view_id = " . $this->view_id." AND
                         view_count = " . $this->get_attempt_id();
             $res = Database::query($sql);
             if (Database::num_rows($res) > 0) {
@@ -669,13 +669,13 @@ 
         foreach ($this->interactions as $id => $in) {
             $return .= "[
                 '$id',
-                '" . $in[1] . "',
-                '" . $in[2] . "',
-                '" . $in[3] . "',
-                '" . $in[4] . "',
-                '" . $in[5] . "',
-                '" . $in[6] . "',
-                '" . $in[7] . "'],";
+                '".$in[1]."',
+                '" . $in[2]."',
+                '" . $in[3]."',
+                '" . $in[4]."',
+                '" . $in[5]."',
+                '" . $in[6]."',
+                '" . $in[7]."'],";
         }
         if (!empty($return)) {
             $return = substr($return, 0, -1);
@@ -917,7 +917,7 @@ 
                 $table = Database::get_course_table(TABLE_LP_MAIN);
                 $sql = "SELECT prevent_reinit
                     FROM $table
-                    WHERE c_id = $course_id AND id = " . $this->lp_id;
+                    WHERE c_id = $course_id AND id = ".$this->lp_id;
                 $res = Database::query($sql);
                 if (Database::num_rows($res) < 1) {
                     $this->error = "Could not find parent learnpath in lp table";
@@ -939,7 +939,7 @@ 
         }
         if (self::debug > 2) {
             error_log(
-                'New LP - End of learnpathItem::get_prevent_reinit() - Returned ' . $this->prevent_reinit,
+                'New LP - End of learnpathItem::get_prevent_reinit() - Returned '.$this->prevent_reinit,
                 0
             );
         }
@@ -964,7 +964,7 @@ 
                 $table = Database::get_course_table(TABLE_LP_MAIN);
                 $sql = "SELECT seriousgame_mode
                         FROM $table
-                        WHERE c_id = $course_id AND id = " . $this->lp_id;
+                        WHERE c_id = $course_id AND id = ".$this->lp_id;
                 $res = Database::query($sql);
                 if (Database::num_rows($res) < 1) {
                     $this->error = "Could not find parent learnpath in learnpath table";
@@ -985,7 +985,7 @@ 
         }
         if (self::debug > 2) {
             error_log(
-                'New LP - End of learnpathItem::get_seriousgame_mode() - Returned ' . $this->seriousgame_mode,
+                'New LP - End of learnpathItem::get_seriousgame_mode() - Returned '.$this->seriousgame_mode,
                 0
             );
         }
@@ -1030,7 +1030,7 @@ 
         }
         if (!isset($abs_path)) {
             $path = $this->get_file_path();
-            $abs_path = api_get_path(SYS_COURSE_PATH) . api_get_course_path() . '/' . $path;
+            $abs_path = api_get_path(SYS_COURSE_PATH).api_get_course_path().'/'.$path;
         }
 
         $files_list = array();
@@ -1305,7 +1305,7 @@ 
                                                             $abs_path
                                                         );
                                                         $new_abs_path = realpath(
-                                                            $dir . '/' . $second_part
+                                                            $dir.'/'.$second_part
                                                         );
                                                         $in_files_list[] = learnpathItem::get_resources_from_source(
                                                             TOOL_DOCUMENT,
@@ -1343,7 +1343,7 @@ 
                                                             $abs_path
                                                         );
                                                         $new_abs_path = realpath(
-                                                            $dir . '/' . $second_part
+                                                            $dir.'/'.$second_part
                                                         );
                                                         $in_files_list[] = learnpathItem::get_resources_from_source(
                                                             TOOL_DOCUMENT,
@@ -1450,7 +1450,7 @@ 
                                                             $abs_path
                                                         );
                                                         $new_abs_path = realpath(
-                                                            $dir . '/' . $source
+                                                            $dir.'/'.$source
                                                         );
                                                         $in_files_list[] = learnpathItem::get_resources_from_source(
                                                             TOOL_DOCUMENT,
@@ -1488,7 +1488,7 @@ 
                                                             $abs_path
                                                         );
                                                         $new_abs_path = realpath(
-                                                            $dir . '/' . $source
+                                                            $dir.'/'.$source
                                                         );
                                                         $in_files_list[] = learnpathItem::get_resources_from_source(
                                                             TOOL_DOCUMENT,
@@ -1572,7 +1572,7 @@ 
                                                 );
                                                 $dir = dirname($abs_path);
                                                 $new_abs_path = realpath(
-                                                    $dir . '/' . $source
+                                                    $dir.'/'.$source
                                                 );
                                                 $in_files_list[] = learnpathItem::get_resources_from_source(
                                                     TOOL_DOCUMENT,
@@ -1617,7 +1617,7 @@ 
                                                 );
                                                 $dir = dirname($abs_path);
                                                 $new_abs_path = realpath(
-                                                    $dir . '/' . $source
+                                                    $dir.'/'.$source
                                                 );
                                                 $in_files_list[] = learnpathItem::get_resources_from_source(
                                                     TOOL_DOCUMENT,
@@ -1677,7 +1677,7 @@ 
         }
         if (self::debug > 1) {
             error_log(
-                'New LP - Out of learnpathItem::get_score() - returning ' . $res,
+                'New LP - Out of learnpathItem::get_score() - returning '.$res,
                 0
             );
         }
@@ -1697,7 +1697,7 @@ 
         $course_id = api_get_course_int_id();
         $debug = self::debug;
         if ($debug > 0) {
-            error_log('learnpathItem::get_status() on item ' . $this->db_id, 0);
+            error_log('learnpathItem::get_status() on item '.$this->db_id, 0);
         }
         if ($check_db) {
             if ($debug > 2) {
@@ -1708,12 +1708,12 @@ 
                 $sql = "SELECT status FROM $table
                         WHERE
                             c_id = $course_id AND
-                            id = '" . $this->db_item_view_id . "' AND
-                            view_count = '" . $this->get_attempt_id() . "'";
+                            id = '".$this->db_item_view_id."' AND
+                            view_count = '" . $this->get_attempt_id()."'";
 
                 if ($debug > 2) {
                     error_log(
-                        'learnpathItem::get_status() - Checking DB: ' . $sql,
+                        'learnpathItem::get_status() - Checking DB: '.$sql,
                         0
                     );
                 }
@@ -1724,7 +1724,7 @@ 
                     if ($update_local) {
                         if ($debug > 2) {
                             error_log(
-                                'learnpathItem::set_status() :' . $row['status'],
+                                'learnpathItem::set_status() :'.$row['status'],
                                 0
                             );
                         }
@@ -1732,7 +1732,7 @@ 
                     }
                     if ($debug > 2) {
                         error_log(
-                            'learnpathItem::get_status() - Returning db value ' . $row['status'],
+                            'learnpathItem::get_status() - Returning db value '.$row['status'],
                             0
                         );
                     }
@@ -1749,7 +1749,7 @@ 
             if (!empty($this->status)) {
                 if ($debug > 2) {
                     error_log(
-                        'learnpathItem::get_status() - Returning attrib: ' . $this->status,
+                        'learnpathItem::get_status() - Returning attrib: '.$this->status,
                         0
                     );
                 }
@@ -1759,7 +1759,7 @@ 
 
         if ($debug > 2) {
             error_log(
-                'learnpathItem::get_status() - Returning default ' . $this->possible_status[0],
+                'learnpathItem::get_status() - Returning default '.$this->possible_status[0],
                 0
             );
         }
@@ -1799,7 +1799,7 @@ 
             if ($origin == 'js') {
                 return '00 : 00: 00';
             } else {
-                return '00 ' . $h . ' 00 \' 00"';
+                return '00 '.$h.' 00 \' 00"';
             }
         } else {
             return api_format_time($time, $origin);
@@ -1824,7 +1824,7 @@ 
         if (!isset($given_time)) {
             if (self::debug > 2) {
                 error_log(
-                    'learnpathItem::get_scorm_time(): given time empty, current_start_time = ' . $this->current_start_time,
+                    'learnpathItem::get_scorm_time(): given time empty, current_start_time = '.$this->current_start_time,
                     0
                 );
             }
@@ -1834,8 +1834,8 @@ 
                         FROM $table
                         WHERE
                             c_id = $course_id AND
-                            id = '" . $this->db_item_view_id . "' AND
-                            view_count = '" . $this->get_attempt_id() . "'";
+                            id = '".$this->db_item_view_id."' AND
+                            view_count = '" . $this->get_attempt_id()."'";
                 $res = Database::query($sql);
                 $row = Database::fetch_array($res);
                 $start = $row['start_time'];
@@ -1856,7 +1856,7 @@ 
         }
         if (self::debug > 2) {
             error_log(
-                'learnpathItem::get_scorm_time(): intermediate = ' . $time,
+                'learnpathItem::get_scorm_time(): intermediate = '.$time,
                 0
             );
         }
@@ -1876,7 +1876,7 @@ 
         $sql = "SELECT * FROM $lp_item
                 WHERE
                     c_id = $course_id AND
-                    id='" . intval($this->db_id) . "'";
+                    id='".intval($this->db_id)."'";
         $res = Database::query($sql);
         $row = Database::fetch_array($res);
         return $row['terms'];
@@ -1905,9 +1905,9 @@ 
     {
         if (self::debug > 0) {
             error_log(
-                'learnpathItem::get_total_time() for item ' . $this->db_id .
-                ' - Initially, current_start_time = ' . $this->current_start_time .
-                ' and current_stop_time = ' . $this->current_stop_time,
+                'learnpathItem::get_total_time() for item '.$this->db_id.
+                ' - Initially, current_start_time = '.$this->current_start_time.
+                ' and current_stop_time = '.$this->current_stop_time,
                 0
             );
         }
@@ -1948,9 +1948,9 @@ 
         } else {
             if (self::debug > 2) {
                 error_log(
-                    'learnpathItem::get_total_time() - Current start time = ' .
-                    $this->current_start_time . ', current stop time = ' .
-                    $this->current_stop_time . ' Returning ' . $time . "-----------\n",
+                    'learnpathItem::get_total_time() - Current start time = '.
+                    $this->current_start_time.', current stop time = '.
+                    $this->current_stop_time.' Returning '.$time."-----------\n",
                     0
                 );
             }
@@ -1970,7 +1970,7 @@ 
         }
         if (self::debug > 2) {
             error_log(
-                'learnpathItem::get_type() - Returning ' . $res . ' for item ' . $this->db_id,
+                'learnpathItem::get_type() - Returning '.$res.' for item '.$this->db_id,
                 0
             );
         }
@@ -2010,8 +2010,8 @@ 
         if ($this->status_is($completedStatusList)) {
             if (self::debug > 2) {
                 error_log(
-                    'learnpath::is_done() - Item ' . $this->get_id(
-                    ) . ' is complete',
+                    'learnpath::is_done() - Item '.$this->get_id(
+                    ).' is complete',
                     0
                 );
             }
@@ -2020,8 +2020,8 @@ 
         } else {
             if (self::debug > 2) {
                 error_log(
-                    'learnpath::is_done() - Item ' . $this->get_id(
-                    ) . ' is not complete',
+                    'learnpath::is_done() - Item '.$this->get_id(
+                    ).' is not complete',
                     0
                 );
             }
@@ -2058,7 +2058,7 @@ 
         }
         if (self::debug > 2) {
             error_log(
-                'New LP - End of learnpathItem::is_restart_allowed() - Returning ' . $restart,
+                'New LP - End of learnpathItem::is_restart_allowed() - Returning '.$restart,
                 0
             );
         }
@@ -2137,7 +2137,7 @@ 
     {
         if (self::debug > 0) {
             error_log(
-                'learnpathItem::parse_prereq() for learnpath ' . $this->lp_id . ' with string ' . $prereqs_string,
+                'learnpathItem::parse_prereq() for learnpath '.$this->lp_id.' with string '.$prereqs_string,
                 0
             );
         }
@@ -2338,7 +2338,7 @@ 
                                 // Strange...
                                 if (self::debug > 1) {
                                     error_log(
-                                        'New LP - Found ~ but strange string: ' . $prereqs_string,
+                                        'New LP - Found ~ but strange string: '.$prereqs_string,
                                         0
                                     );
                                 }
@@ -2366,7 +2366,7 @@ 
                                     //  the big brackets-enclosed condition.
                                     if (self::debug > 1) {
                                         error_log(
-                                            'New LP - Dealing with group ' . $gr,
+                                            'New LP - Dealing with group '.$gr,
                                             0
                                         );
                                     }
@@ -2380,7 +2380,7 @@ 
                                     ) {
                                         if (self::debug > 1) {
                                             error_log(
-                                                'New LP - Found multiplier ' . $multi[0],
+                                                'New LP - Found multiplier '.$multi[0],
                                                 0
                                             );
                                         }
@@ -2396,7 +2396,7 @@ 
                                                     $mytrue++;
                                                     if (self::debug > 1) {
                                                         error_log(
-                                                            'New LP - Found true item, counting.. (' . ($mytrue) . ')',
+                                                            'New LP - Found true item, counting.. ('.($mytrue).')',
                                                             0
                                                         );
                                                     }
@@ -2404,7 +2404,7 @@ 
                                             } else {
                                                 if (self::debug > 1) {
                                                     error_log(
-                                                        'New LP - item ' . $cond . ' does not exist in items list',
+                                                        'New LP - item '.$cond.' does not exist in items list',
                                                         0
                                                     );
                                                 }
@@ -2451,7 +2451,7 @@ 
                                                 } else {
                                                     if (self::debug > 1) {
                                                         error_log(
-                                                            'New LP - ' .
+                                                            'New LP - '.
                                                             ' Found false item, the set is not true, return false',
                                                             0
                                                         );
@@ -2462,7 +2462,7 @@ 
                                             } else {
                                                 if (self::debug > 1) {
                                                     error_log(
-                                                        'New LP - item ' . $cond . ' does not exist in items list',
+                                                        'New LP - item '.$cond.' does not exist in items list',
                                                         0
                                                     );
                                                 }
@@ -2488,7 +2488,7 @@ 
                                 //  value of the corresponding resource completion status.
                                 if (self::debug > 1) {
                                     error_log(
-                                        'New LP - Didnt find any group, returning value for ' . $prereqs_string,
+                                        'New LP - Didnt find any group, returning value for '.$prereqs_string,
                                         0
                                     );
                                 }
@@ -2505,14 +2505,14 @@ 
                                         if (!$returnstatus) {
                                             if (self::debug > 1) {
                                                 error_log(
-                                                    'New LP - Prerequisite ' . $prereqs_string . ' not complete',
+                                                    'New LP - Prerequisite '.$prereqs_string.' not complete',
                                                     0
                                                 );
                                             }
                                         } else {
                                             if (self::debug > 1) {
                                                 error_log(
-                                                    'New LP - Prerequisite ' . $prereqs_string . ' complete',
+                                                    'New LP - Prerequisite '.$prereqs_string.' complete',
                                                     0
                                                 );
                                             }
@@ -2524,12 +2524,12 @@ 
                                             if ($returnstatus) {
                                                 //AND origin_lp_item_id = '.$user_id.'
                                                 $sql = 'SELECT exe_result, exe_weighting
-                                                        FROM ' . Database :: get_main_table(TABLE_STATISTIC_TRACK_E_EXERCISES) . '
+                                                        FROM ' . Database :: get_main_table(TABLE_STATISTIC_TRACK_E_EXERCISES).'
                                                         WHERE
-                                                            exe_exo_id = ' . $items[$refs_list[$prereqs_string]]->path . ' AND
-                                                            exe_user_id = ' . $user_id . ' AND
-                                                            orig_lp_id = ' . $this->lp_id . ' AND
-                                                            orig_lp_item_id = ' . $prereqs_string . ' AND
+                                                            exe_exo_id = ' . $items[$refs_list[$prereqs_string]]->path.' AND
+                                                            exe_user_id = ' . $user_id.' AND
+                                                            orig_lp_id = ' . $this->lp_id.' AND
+                                                            orig_lp_item_id = ' . $prereqs_string.' AND
                                                             status <> "incomplete"
                                                         ORDER BY exe_date DESC
                                                         LIMIT 0, 1';
@@ -2567,12 +2567,12 @@ 
 
                                             // Checking in the database.
                                             $sql = 'SELECT exe_result, exe_weighting
-                                                    FROM ' . Database :: get_main_table(TABLE_STATISTIC_TRACK_E_EXERCISES) . '
+                                                    FROM ' . Database :: get_main_table(TABLE_STATISTIC_TRACK_E_EXERCISES).'
                                                     WHERE
-                                                        exe_exo_id = ' . $items[$refs_list[$prereqs_string]]->path . ' AND
-                                                        exe_user_id = ' . $user_id . ' AND
-                                                        orig_lp_id = ' . $this->lp_id . ' AND
-                                                        orig_lp_item_id = ' . $prereqs_string . ' ';
+                                                        exe_exo_id = ' . $items[$refs_list[$prereqs_string]]->path.' AND
+                                                        exe_user_id = ' . $user_id.' AND
+                                                        orig_lp_id = ' . $this->lp_id.' AND
+                                                        orig_lp_item_id = ' . $prereqs_string.' ';
 
                                             $rs_quiz = Database::query($sql);
                                             if (Database::num_rows($rs_quiz) > 0) {
@@ -2619,14 +2619,14 @@ 
                                         if (!$returnstatus) {
                                             if (self::debug > 1) {
                                                 error_log(
-                                                    'New LP - Prerequisite ' . $prereqs_string . ' not complete',
+                                                    'New LP - Prerequisite '.$prereqs_string.' not complete',
                                                     0
                                                 );
                                             }
                                         } else {
                                             if (self::debug > 1) {
                                                 error_log(
-                                                    'New LP - Prerequisite ' . $prereqs_string . ' complete',
+                                                    'New LP - Prerequisite '.$prereqs_string.' complete',
                                                     0
                                                 );
                                             }
@@ -2641,11 +2641,11 @@ 
                                                 TABLE_LP_VIEW
                                             );
 
-                                            $sql = 'SELECT id FROM ' . $lp_view . '
+                                            $sql = 'SELECT id FROM '.$lp_view.'
                                                     WHERE
-                                                        c_id = ' . $course_id . ' AND
-                                                        user_id = ' . $user_id . '  AND
-                                                        lp_id = ' . $this->lp_id . ' AND
+                                                        c_id = ' . $course_id.' AND
+                                                        user_id = ' . $user_id.'  AND
+                                                        lp_id = ' . $this->lp_id.' AND
                                                         session_id = '.$sessionId.'
                                                     LIMIT 0, 1';
                                             $rs_lp = Database::query($sql);
@@ -2654,11 +2654,11 @@ 
                                             );
                                             $my_lp_id = $lp_id[0];
 
-                                            $sql = 'SELECT status FROM ' . $lp_item_view . '
+                                            $sql = 'SELECT status FROM '.$lp_item_view.'
                                                    WHERE
-                                                        c_id = ' . $course_id . ' AND
-                                                        lp_view_id = ' . $my_lp_id . ' AND
-                                                        lp_item_id = ' . $refs_list[$prereqs_string] . '
+                                                        c_id = ' . $course_id.' AND
+                                                        lp_view_id = ' . $my_lp_id.' AND
+                                                        lp_item_id = ' . $refs_list[$prereqs_string].'
                                                     LIMIT 0, 1';
                                             $rs_lp = Database::query($sql);
                                             $status_array = Database :: fetch_row(
@@ -2675,14 +2675,14 @@ 
                                             if (!$returnstatus) {
                                                 if (self::debug > 1) {
                                                     error_log(
-                                                        'New LP - Prerequisite ' . $prereqs_string . ' not complete',
+                                                        'New LP - Prerequisite '.$prereqs_string.' not complete',
                                                         0
                                                     );
                                                 }
                                             } else {
                                                 if (self::debug > 1) {
                                                     error_log(
-                                                        'New LP - Prerequisite ' . $prereqs_string . ' complete',
+                                                        'New LP - Prerequisite '.$prereqs_string.' complete',
                                                         0
                                                     );
                                                 }
@@ -2693,7 +2693,7 @@ 
                                 } else {
                                     if (self::debug > 1) {
                                         error_log(
-                                            'New LP - Could not find ' . $prereqs_string . ' in ' . print_r(
+                                            'New LP - Could not find '.$prereqs_string.' in '.print_r(
                                                 $refs_list,
                                                 true
                                             ),
@@ -2716,7 +2716,7 @@ 
                 foreach ($list as $condition) {
                     if (self::debug > 1) {
                         error_log(
-                            'New LP - Found OR, adding it (' . $condition . ')',
+                            'New LP - Found OR, adding it ('.$condition.')',
                             0
                         );
                     }
@@ -2770,7 +2770,7 @@ 
 
         if (self::debug > 1) {
             error_log(
-                'New LP - End of parse_prereq. Error code is now ' . $this->prereq_alert,
+                'New LP - End of parse_prereq. Error code is now '.$this->prereq_alert,
                 0
             );
         }
@@ -2888,7 +2888,7 @@ 
                                 $this->set_score($value);
                                 if (self::debug > 2) {
                                     error_log(
-                                        'learnpathItem::save() - setting score to ' . $value,
+                                        'learnpathItem::save() - setting score to '.$value,
                                         0
                                     );
                                 }
@@ -2897,7 +2897,7 @@ 
                                 $this->set_max_score($value);
                                 if (self::debug > 2) {
                                     error_log(
-                                        'learnpathItem::save() - setting view_max_score to ' . $value,
+                                        'learnpathItem::save() - setting view_max_score to '.$value,
                                         0
                                     );
                                 }
@@ -2906,7 +2906,7 @@ 
                                 $this->min_score = $value;
                                 if (self::debug > 2) {
                                     error_log(
-                                        'learnpathItem::save() - setting min_score to ' . $value,
+                                        'learnpathItem::save() - setting min_score to '.$value,
                                         0
                                     );
                                 }
@@ -2916,7 +2916,7 @@ 
                                     $this->set_status($value);
                                     if (self::debug > 2) {
                                         error_log(
-                                            'learnpathItem::save() - setting status to ' . $value,
+                                            'learnpathItem::save() - setting status to '.$value,
                                             0
                                         );
                                     }
@@ -2926,7 +2926,7 @@ 
                                 $this->set_time($value);
                                 if (self::debug > 2) {
                                     error_log(
-                                        'learnpathItem::save() - setting time to ' . $value,
+                                        'learnpathItem::save() - setting time to '.$value,
                                         0
                                     );
                                 }
@@ -2935,7 +2935,7 @@ 
                                 $this->current_data = $value;
                                 if (self::debug > 2) {
                                     error_log(
-                                        'learnpathItem::save() - setting suspend_data to ' . $value,
+                                        'learnpathItem::save() - setting suspend_data to '.$value,
                                         0
                                     );
                                 }
@@ -2944,7 +2944,7 @@ 
                                 $this->set_lesson_location($value);
                                 if (self::debug > 2) {
                                     error_log(
-                                        'learnpathItem::save() - setting lesson_location to ' . $value,
+                                        'learnpathItem::save() - setting lesson_location to '.$value,
                                         0
                                     );
                                 }
@@ -2953,7 +2953,7 @@ 
                                 $this->set_core_exit($value);
                                 if (self::debug > 2) {
                                     error_log(
-                                        'learnpathItem::save() - setting core_exit to ' . $value,
+                                        'learnpathItem::save() - setting core_exit to '.$value,
                                         0
                                     );
                                 }
@@ -3100,7 +3100,7 @@ 
     public function set_level($int = 0)
     {
         if (self::debug > 0) {
-            error_log('learnpathItem::set_level(' . $int . ')', 0);
+            error_log('learnpathItem::set_level('.$int.')', 0);
         }
         if (!empty($int) AND $int == strval(intval($int))) {
             $this->level = $int;
@@ -3129,7 +3129,7 @@ 
         if (empty($lpItemId)) {
             if (self::debug > 0) {
                 error_log(
-                    'learnpathItem::set_lp_view(' . $lp_view_id . ') $lpItemId is empty',
+                    'learnpathItem::set_lp_view('.$lp_view_id.') $lpItemId is empty',
                     0
                 );
             }
@@ -3140,7 +3140,7 @@ 
         if (empty($lp_view_id)) {
             if (self::debug > 0) {
                 error_log(
-                    'learnpathItem::set_lp_view(' . $lp_view_id . ') $lp_view_id is empty',
+                    'learnpathItem::set_lp_view('.$lp_view_id.') $lp_view_id is empty',
                     0
                 );
             }
@@ -3149,7 +3149,7 @@ 
         }
 
         if (self::debug > 0) {
-            error_log('learnpathItem::set_lp_view(' . $lp_view_id . ')', 0);
+            error_log('learnpathItem::set_lp_view('.$lp_view_id.')', 0);
         }
 
         $this->view_id = $lp_view_id;
@@ -3159,13 +3159,13 @@ 
         $sql = "SELECT * FROM $item_view_table
                 WHERE
                     c_id = $course_id AND
-                    lp_item_id = " . $lpItemId . " AND
-                    lp_view_id = " . $lp_view_id . "
+                    lp_item_id = ".$lpItemId." AND
+                    lp_view_id = " . $lp_view_id."
                 ORDER BY view_count DESC";
 
         if (self::debug > 2) {
             error_log(
-                'learnpathItem::set_lp_view() - Querying lp_item_view: ' . $sql,
+                'learnpathItem::set_lp_view() - Querying lp_item_view: '.$sql,
                 0
             );
         }
@@ -3197,7 +3197,7 @@ 
             $sql = "SELECT * FROM $item_view_interaction_table
                     WHERE
                         c_id = $course_id AND
-                        lp_iv_id = '" . $this->db_item_view_id . "'";
+                        lp_iv_id = '".$this->db_item_view_id."'";
 
             $res = Database::query($sql);
             if ($res !== false) {
@@ -3212,7 +3212,7 @@ 
             $sql = "SELECT * FROM $item_view_objective_table
                     WHERE
                         c_id = $course_id AND
-                        lp_iv_id = '" . $this->db_item_view_id . "'";
+                        lp_iv_id = '".$this->db_item_view_id."'";
 
             $res = Database::query($sql);
             if ($res !== false) {
@@ -3273,7 +3273,7 @@ 
     {
         $debug = self::debug;
         if ($debug > 0) {
-            error_log('learnpathItem::set_score(' . $score . ')', 0);
+            error_log('learnpathItem::set_score('.$score.')', 0);
         }
         if (($this->max_score <= 0 || $score <= $this->max_score) && ($score >= $this->min_score)) {
             $this->current_score = $score;
@@ -3286,9 +3286,9 @@ 
             }
 
             if ($debug > 0) {
-                error_log('get_mastery_score: ' . $masteryScore);
-                error_log('current_status: ' . $current_status);
-                error_log('current score : ' . $this->current_score);
+                error_log('get_mastery_score: '.$masteryScore);
+                error_log('current_status: '.$current_status);
+                error_log('current score : '.$this->current_score);
             }
 
             // If mastery_score is set AND the current score reaches the mastery
@@ -3316,15 +3316,15 @@ 
     public function set_max_score($score)
     {
         if (self::debug > 0) {
-            error_log('learnpathItem::set_max_score(' . $score . ')', 0);
+            error_log('learnpathItem::set_max_score('.$score.')', 0);
         }
         if (is_int($score) || $score == '') {
             $this->view_max_score = $score;
             if (self::debug > 1) {
                 error_log(
-                    'learnpathItem::set_max_score() - ' .
-                    'Updated object score of item ' . $this->db_id .
-                    ' to ' . $this->view_max_score,
+                    'learnpathItem::set_max_score() - '.
+                    'Updated object score of item '.$this->db_id.
+                    ' to '.$this->view_max_score,
                     0
                 );
             }
@@ -3342,12 +3342,12 @@ 
     public function set_status($status)
     {
         if (self::debug > 0) {
-            error_log('learnpathItem::set_status(' . $status . ')', 0);
+            error_log('learnpathItem::set_status('.$status.')', 0);
         }
 
         $found = false;
         foreach ($this->possible_status as $possible) {
-            if (preg_match('/^' . $possible . '$/i', $status)) {
+            if (preg_match('/^'.$possible.'$/i', $status)) {
                 $found = true;
             }
         }
@@ -3357,8 +3357,8 @@ 
             if (self::debug > 1) {
                 error_log(
                     'learnpathItem::set_status() - '.
-                        'Updated object status of item ' . $this->db_id .
-                        ' to ' . $this->status,
+                        'Updated object status of item '.$this->db_id.
+                        ' to '.$this->status,
                     0
                 );
             }
@@ -3380,7 +3380,7 @@ 
         global $charset;
         $course_id = api_get_course_int_id();
         $lp_item = Database::get_course_table(TABLE_LP_ITEM);
-        require_once api_get_path(LIBRARY_PATH) . 'search/ChamiloIndexer.class.php';
+        require_once api_get_path(LIBRARY_PATH).'search/ChamiloIndexer.class.php';
         $a_terms = preg_split('/,/', $terms);
         $i_terms = preg_split('/,/', $this->get_terms());
         foreach ($i_terms as $term) {
@@ -3397,7 +3397,7 @@ 
                 SET terms = '$terms'
                 WHERE
                     c_id = $course_id AND
-                    id=" . $this->get_id();
+                    id=".$this->get_id();
         Database::query($sql);
         // Save it to search engine.
         if (api_get_setting('search_enabled') == 'true') {
@@ -3425,7 +3425,7 @@ 
     public function set_time($scorm_time, $format = 'scorm')
     {
         if (self::debug > 0) {
-            error_log('learnpathItem::set_time(' . $scorm_time . ')', 0);
+            error_log('learnpathItem::set_time('.$scorm_time.')', 0);
         }
         if ($scorm_time == '0'
                 and ($this->type != 'sco')
@@ -3436,8 +3436,8 @@ 
                 $this->update_time($my_time);
                 if (self::debug > 0) {
                     error_log(
-                        'learnpathItem::set_time(' . $scorm_time . ') - ' .
-                            'found asset - set time to ' . $my_time,
+                        'learnpathItem::set_time('.$scorm_time.') - '.
+                            'found asset - set time to '.$my_time,
                         0
                     );
                 }
@@ -3510,10 +3510,10 @@ 
     {
         if (self::debug > 1) {
             error_log(
-                'learnpathItem::status_is(' . print_r(
+                'learnpathItem::status_is('.print_r(
                     $list,
                     true
-                ) . ') on item ' . $this->db_id,
+                ).') on item '.$this->db_id,
                 0
             );
         }
@@ -3523,11 +3523,11 @@ 
         }
         $found = false;
         foreach ($list as $status) {
-            if (preg_match('/^' . $status . '$/i', $currentStatus)) {
+            if (preg_match('/^'.$status.'$/i', $currentStatus)) {
                 if (self::debug > 2) {
                     error_log(
-                        'New LP - learnpathItem::status_is() - Found status ' .
-                            $status . ' corresponding to current status',
+                        'New LP - learnpathItem::status_is() - Found status '.
+                            $status.' corresponding to current status',
                         0
                     );
                 }
@@ -3538,8 +3538,8 @@ 
         }
         if (self::debug > 2) {
             error_log(
-                'New LP - learnpathItem::status_is() - Status ' .
-                    $currentStatus . ' did not match request',
+                'New LP - learnpathItem::status_is() - Status '.
+                    $currentStatus.' did not match request',
                 0
             );
         }
@@ -3556,7 +3556,7 @@ 
     public function update_time($total_sec = 0)
     {
         if (self::debug > 0) {
-            error_log('learnpathItem::update_time(' . $total_sec . ')', 0);
+            error_log('learnpathItem::update_time('.$total_sec.')', 0);
         }
         if ($total_sec >= 0) {
             // Getting start time from finish time. The only problem in the calculation is it might be
@@ -3606,12 +3606,12 @@ 
         $item_view_table = Database::get_course_table(TABLE_LP_ITEM_VIEW);
         $course_id = api_get_course_int_id();
 
-        $sql = 'SELECT total_time, status FROM ' . $item_view_table . '
+        $sql = 'SELECT total_time, status FROM '.$item_view_table.'
                  WHERE 
-                    c_id = ' . $course_id . ' AND 
-                    lp_item_id="' . $this->db_id . '" AND 
-                    lp_view_id="' . $this->view_id . '" AND 
-                    view_count="' . $this->get_attempt_id() . '"';
+                    c_id = ' . $course_id.' AND 
+                    lp_item_id="' . $this->db_id.'" AND 
+                    lp_view_id="' . $this->view_id.'" AND 
+                    view_count="' . $this->get_attempt_id().'"';
         $result = Database::query($sql);
         $row = Database::fetch_array($result);
 
@@ -3681,13 +3681,13 @@ 
     {
         $item_view_table = Database::get_course_table(TABLE_LP_ITEM_VIEW);
         $course_id = api_get_course_int_id();
-        $sql = 'UPDATE ' . $item_view_table . '
+        $sql = 'UPDATE '.$item_view_table.'
                 SET total_time = 0, 
-                    start_time=' . time() . '
-                WHERE c_id = ' . $course_id . '
-                    AND lp_item_id="' . $this->db_id . '"
-                    AND lp_view_id="' . $this->view_id . '"
-                    AND view_count="' . $this->attempt_id . '" ;';
+                    start_time=' . time().'
+                WHERE c_id = ' . $course_id.'
+                    AND lp_item_id="' . $this->db_id.'"
+                    AND lp_view_id="' . $this->view_id.'"
+                    AND view_count="' . $this->attempt_id.'" ;';
         Database::query($sql);
     }
 
@@ -3709,8 +3709,8 @@ 
                     FROM $tbl
                     WHERE
                         c_id = $course_id AND
-                        lp_item_id = " . $this->db_id . " AND
-                        lp_view_id = " . $this->view_id . " AND
+                        lp_item_id = ".$this->db_id." AND
+                        lp_view_id = " . $this->view_id." AND
                         view_count = " . $this->attempt_id;
             $res = Database::query($sql);
             if (Database::num_rows($res) > 0) {
@@ -3718,8 +3718,8 @@ 
                 $lp_iv_id = $row[0];
                 if (self::debug > 2) {
                     error_log(
-                        'learnpathItem::write_to_db() - Got item_view_id ' .
-                            $lp_iv_id . ', now checking objectives ',
+                        'learnpathItem::write_to_db() - Got item_view_id '.
+                            $lp_iv_id.', now checking objectives ',
                         0
                     );
                 }
@@ -3731,7 +3731,7 @@ 
                                 WHERE
                                     c_id = $course_id AND
                                     lp_iv_id = $lp_iv_id AND
-                                    objective_id = '" . Database::escape_string($objective[0]) . "'";
+                                    objective_id = '".Database::escape_string($objective[0])."'";
                     $iva_res = Database::query($iva_sql);
                     // id(0), type(1), time(2), weighting(3),
                     // correct_responses(4), student_response(5),
@@ -3740,12 +3740,12 @@ 
                         // Update (or don't).
                         $iva_row = Database::fetch_array($iva_res);
                         $iva_id = $iva_row[0];
-                        $ivau_sql = "UPDATE $iva_table " .
-                            "SET objective_id = '" . Database::escape_string($objective[0]) . "'," .
-                            "status = '" . Database::escape_string($objective[1]) . "'," .
-                            "score_raw = '" . Database::escape_string($objective[2]) . "'," .
-                            "score_min = '" . Database::escape_string($objective[4]) . "'," .
-                            "score_max = '" . Database::escape_string($objective[3]) . "' " .
+                        $ivau_sql = "UPDATE $iva_table ".
+                            "SET objective_id = '".Database::escape_string($objective[0])."',".
+                            "status = '".Database::escape_string($objective[1])."',".
+                            "score_raw = '".Database::escape_string($objective[2])."',".
+                            "score_min = '".Database::escape_string($objective[4])."',".
+                            "score_max = '".Database::escape_string($objective[3])."' ".
                             "WHERE c_id = $course_id AND id = $iva_id";
                         Database::query($ivau_sql);
                     } else {
@@ -3797,12 +3797,12 @@ 
         $my_status = ' ';
 
         $item_view_table = Database::get_course_table(TABLE_LP_ITEM_VIEW);
-        $sql = 'SELECT status FROM ' . $item_view_table . '
+        $sql = 'SELECT status FROM '.$item_view_table.'
                 WHERE
-                    c_id = ' . $course_id . ' AND
-                    lp_item_id="' . $this->db_id . '" AND
-                    lp_view_id="' . $this->view_id . '" AND
-                    view_count="' . $this->get_attempt_id() . '" ';
+                    c_id = ' . $course_id.' AND
+                    lp_item_id="' . $this->db_id.'" AND
+                    lp_view_id="' . $this->view_id.'" AND
+                    view_count="' . $this->get_attempt_id().'" ';
         $rs_verified = Database::query($sql);
         $row_verified = Database::fetch_array($rs_verified);
 
@@ -3830,8 +3830,8 @@ 
                     "This info shouldn't be saved as the credit or lesson mode info prevent it"
                 );
                 error_log(
-                    'learnpathItem::write_to_db() - credit(' . $credit . ') or'.
-                        ' lesson_mode(' . $mode . ') prevent recording!',
+                    'learnpathItem::write_to_db() - credit('.$credit.') or'.
+                        ' lesson_mode('.$mode.') prevent recording!',
                     0
                 );
             }
@@ -3855,14 +3855,14 @@ 
                     "max_score" => $this->get_max(),
                     "lp_item_id" => $this->db_id,
                     "lp_view_id" => $this->view_id,
-                    "view_count" => $this->get_attempt_id() ,
+                    "view_count" => $this->get_attempt_id(),
                     "suspend_data" => $this->current_data,
                     //"max_time_allowed" => ,
                     "lesson_location" => $this->lesson_location
                 );
                 if (self::debug > 2) {
                     error_log(
-                        'learnpathItem::write_to_db() - Inserting into item_view forced: ' . print_r($params, 1),
+                        'learnpathItem::write_to_db() - Inserting into item_view forced: '.print_r($params, 1),
                         0
                     );
                 }
@@ -3870,7 +3870,7 @@ 
 
                 if ($this->db_item_view_id) {
                     $sql = "UPDATE $item_view_table SET id = iid
-                            WHERE iid = " . $this->db_item_view_id;
+                            WHERE iid = ".$this->db_item_view_id;
                     Database::query($sql);
                     $inserted = true;
                 }
@@ -3880,12 +3880,12 @@ 
             $sql = "SELECT * FROM $item_view_table
                     WHERE
                         c_id = $course_id AND
-                        lp_item_id = " . $this->db_id . " AND
-                        lp_view_id = " . $this->view_id . " AND
+                        lp_item_id = ".$this->db_id." AND
+                        lp_view_id = " . $this->view_id." AND
                         view_count = " . intval($this->get_attempt_id());
             if (self::debug > 2) {
                 error_log(
-                    'learnpathItem::write_to_db() - Querying item_view: ' . $sql,
+                    'learnpathItem::write_to_db() - Querying item_view: '.$sql,
                     0
                 );
             }
@@ -3902,7 +3902,7 @@ 
                     "max_score" => $this->get_max(),
                     "lp_item_id" => $this->db_id,
                     "lp_view_id" => $this->view_id,
-                    "view_count" => $this->get_attempt_id() ,
+                    "view_count" => $this->get_attempt_id(),
                     "suspend_data" => $this->current_data,
                     //"max_time_allowed" => ,$this->get_max_time_allowed()
                     "lesson_location" => $this->lesson_location
@@ -3910,7 +3910,7 @@ 
 
                 if (self::debug > 2) {
                     error_log(
-                        'learnpathItem::write_to_db() - Inserting into item_view forced: ' . print_r($params, 1),
+                        'learnpathItem::write_to_db() - Inserting into item_view forced: '.print_r($params, 1),
                         0
                     );
                 }
@@ -3919,7 +3919,7 @@ 
 
                 if ($this->db_item_view_id) {
                     $sql = "UPDATE $item_view_table SET id = iid
-                            WHERE iid = " . $this->db_item_view_id;
+                            WHERE iid = ".$this->db_item_view_id;
                     Database::query($sql);
                 }
             } else {
@@ -3959,8 +3959,8 @@ 
                             $time_exe_date = convert_sql_date(
                                 $row_dates['exe_date']
                             );
-                            $mytime = ((int)$time_exe_date - (int)$time_start_date);
-                            $total_time = " total_time = " . $mytime . ", ";
+                            $mytime = ((int) $time_exe_date - (int) $time_start_date);
+                            $total_time = " total_time = ".$mytime.", ";
                         }
                     } else {
                         $my_type_lp = learnpath::get_type_static($this->lp_id);
@@ -3974,16 +3974,16 @@ 
 
                         // Is not multiple attempts
                         if ($this->seriousgame_mode == 1 && $this->type == 'sco') {
-                            $total_time = " total_time = total_time +" . $this->get_total_time() . ", ";
-                            $my_status = " status = '" . $this->get_status(false) . "' ,";
+                            $total_time = " total_time = total_time +".$this->get_total_time().", ";
+                            $my_status = " status = '".$this->get_status(false)."' ,";
                         } elseif ($this->get_prevent_reinit() == 1) {
                             // Process of status verified into data base.
-                            $sql = 'SELECT status FROM ' . $item_view_table . '
+                            $sql = 'SELECT status FROM '.$item_view_table.'
                                     WHERE
-                                        c_id = ' . $course_id . ' AND
-                                        lp_item_id="' . $this->db_id . '" AND
-                                        lp_view_id="' . $this->view_id . '" AND
-                                        view_count="' . $this->get_attempt_id() . '"
+                                        c_id = ' . $course_id.' AND
+                                        lp_item_id="' . $this->db_id.'" AND
+                                        lp_view_id="' . $this->view_id.'" AND
+                                        view_count="' . $this->get_attempt_id().'"
                                     ';
                             $rs_verified = Database::query($sql);
                             $row_verified = Database::fetch_array($rs_verified);
@@ -3993,26 +3993,26 @@ 
                             if (!in_array($this->get_status(false), $case_completed) &&
                                 $my_type_lp == 2
                             ) {
-                                $total_time = " total_time = total_time +" . $this->get_total_time() . ", ";
-                                $my_status = " status = '" . $this->get_status(false) . "' ,";
+                                $total_time = " total_time = total_time +".$this->get_total_time().", ";
+                                $my_status = " status = '".$this->get_status(false)."' ,";
                             } else {
                                 // Verified into database.
                                 if (!in_array($row_verified['status'], $case_completed) &&
                                     $my_type_lp == 2
                                 ) {
-                                    $total_time = " total_time = total_time +" . $this->get_total_time() . ", ";
-                                    $my_status = " status = '" . $this->get_status(false) . "' ,";
-                                } elseif (in_array($row_verified['status'], $case_completed ) &&
+                                    $total_time = " total_time = total_time +".$this->get_total_time().", ";
+                                    $my_status = " status = '".$this->get_status(false)."' ,";
+                                } elseif (in_array($row_verified['status'], $case_completed) &&
                                     $my_type_lp == 2 && $this->type != 'sco'
                                 ) {
-                                    $total_time = " total_time = total_time +" . $this->get_total_time() . ", ";
-                                    $my_status = " status = '" . $this->get_status(false) . "' ,";
+                                    $total_time = " total_time = total_time +".$this->get_total_time().", ";
+                                    $my_status = " status = '".$this->get_status(false)."' ,";
                                 } else {
                                     if (($my_type_lp == 3 && $this->type == 'au') ||
                                         ($my_type_lp == 1 && $this->type != 'dir')) {
                                         // Is AICC or Chamilo LP
-                                        $total_time = " total_time = total_time + " . $this->get_total_time() . ", ";
-                                        $my_status = " status = '" . $this->get_status(false) . "' ,";
+                                        $total_time = " total_time = total_time + ".$this->get_total_time().", ";
+                                        $my_status = " status = '".$this->get_status(false)."' ,";
                                     }
                                 }
                             }
@@ -4024,27 +4024,27 @@ 
                                 ) && $my_type_lp == 2
                             ) {
                                 // Reset zero new attempt ?
-                                $my_status = " status = '" . $this->get_status(false) . "' ,";
+                                $my_status = " status = '".$this->get_status(false)."' ,";
                             } elseif (!in_array($this->get_status(false), $case_completed) &&
                                 $my_type_lp == 2
                             ) {
-                                $total_time = " total_time = " . $this->get_total_time() . ", ";
-                                $my_status = " status = '" . $this->get_status(false) . "' ,";
+                                $total_time = " total_time = ".$this->get_total_time().", ";
+                                $my_status = " status = '".$this->get_status(false)."' ,";
                             } else {
                                 // It is chamilo LP.
-                                $total_time = " total_time = total_time +" . $this->get_total_time() . ", ";
-                                $my_status = " status = '" . $this->get_status(false) . "' ,";
+                                $total_time = " total_time = total_time +".$this->get_total_time().", ";
+                                $my_status = " status = '".$this->get_status(false)."' ,";
                             }
 
                             // This code line fixes the problem of wrong status.
                             if ($my_type_lp == 2) {
                                 // Verify current status in multiples attempts.
-                                $sql = 'SELECT status FROM ' . $item_view_table . '
+                                $sql = 'SELECT status FROM '.$item_view_table.'
                                         WHERE
-                                            c_id = ' . $course_id . ' AND
-                                            lp_item_id="' . $this->db_id . '" AND
-                                            lp_view_id="' . $this->view_id . '" AND
-                                            view_count="' . $this->get_attempt_id() . '" ';
+                                            c_id = ' . $course_id.' AND
+                                            lp_item_id="' . $this->db_id.'" AND
+                                            lp_view_id="' . $this->view_id.'" AND
+                                            view_count="' . $this->get_attempt_id().'" ';
                                 $rs_status = Database::query($sql);
                                 $current_status = Database::result(
                                     $rs_status,
@@ -4055,7 +4055,7 @@ 
                                     $my_status = '';
                                     $total_time = '';
                                 } else {
-                                    $total_time = " total_time = total_time +" . $this->get_total_time() . ", ";
+                                    $total_time = " total_time = total_time +".$this->get_total_time().", ";
                                 }
                             }
                         }
@@ -4066,38 +4066,38 @@ 
                         //" . //start_time = ".$this->get_current_start_time().", " . //scorm_init_time does it
                         ////" max_time_allowed = '".$this->get_max_time_allowed()."'," .
                         $sql = "UPDATE $item_view_table SET
-                                    score = " . $this->get_score() . ",
+                                    score = ".$this->get_score().",
                                     $my_status
-                                    max_score = '" . $this->get_max() . "',
-                                    suspend_data = '" . Database::escape_string($this->current_data) . "',
-                                    lesson_location = '" . $this->lesson_location . "'
+                                    max_score = '".$this->get_max()."',
+                                    suspend_data = '" . Database::escape_string($this->current_data)."',
+                                    lesson_location = '" . $this->lesson_location."'
                                 WHERE
                                     c_id = $course_id AND
-                                    lp_item_id = " . $this->db_id . " AND
-                                    lp_view_id = " . $this->view_id . "  AND
+                                    lp_item_id = ".$this->db_id." AND
+                                    lp_view_id = " . $this->view_id."  AND
                                     view_count = " . $this->get_attempt_id();
 
                     } else {
                         //" max_time_allowed = '".$this->get_max_time_allowed()."'," .
                         $sql = "UPDATE $item_view_table SET
                                     $total_time
-                                    start_time = " . $this->get_current_start_time() . ",
-                                    score = " . $this->get_score() . ",
+                                    start_time = ".$this->get_current_start_time().",
+                                    score = " . $this->get_score().",
                                     $my_status
-                                    max_score = '" . $this->get_max() . "',
-                                    suspend_data = '" . Database::escape_string($this->current_data) . "',
-                                    lesson_location = '" . $this->lesson_location . "'
+                                    max_score = '".$this->get_max()."',
+                                    suspend_data = '" . Database::escape_string($this->current_data)."',
+                                    lesson_location = '" . $this->lesson_location."'
                                 WHERE
                                     c_id = $course_id AND
-                                    lp_item_id = " . $this->db_id . " AND
-                                    lp_view_id = " . $this->view_id . " AND
+                                    lp_item_id = ".$this->db_id." AND
+                                    lp_view_id = " . $this->view_id." AND
                                     view_count = " . $this->get_attempt_id();
                     }
                     $this->current_start_time = time();
                 }
                 if (self::debug > 2) {
                     error_log(
-                        'learnpathItem::write_to_db() - Updating item_view: ' . $sql,
+                        'learnpathItem::write_to_db() - Updating item_view: '.$sql,
                         0
                     );
                 }
@@ -4110,8 +4110,8 @@ 
                 $sql = "SELECT id FROM $tbl
                         WHERE
                             c_id = $course_id AND
-                            lp_item_id = " . $this->db_id . " AND
-                            lp_view_id = " . $this->view_id . " AND
+                            lp_item_id = ".$this->db_id." AND
+                            lp_view_id = " . $this->view_id." AND
                             view_count = " . $this->get_attempt_id();
                 $res = Database::query($sql);
                 if (Database::num_rows($res) > 0) {
@@ -4119,16 +4119,16 @@ 
                     $lp_iv_id = $row[0];
                     if (self::debug > 2) {
                         error_log(
-                            'learnpathItem::write_to_db() - Got item_view_id ' .
-                                $lp_iv_id . ', now checking interactions ',
+                            'learnpathItem::write_to_db() - Got item_view_id '.
+                                $lp_iv_id.', now checking interactions ',
                             0
                         );
                     }
                     foreach ($this->interactions as $index => $interaction) {
                         $correct_resp = '';
-                        if (is_array($interaction[4] ) && !empty($interaction[4][0])) {
+                        if (is_array($interaction[4]) && !empty($interaction[4][0])) {
                             foreach ($interaction[4] as $resp) {
-                                $correct_resp .= $resp . ',';
+                                $correct_resp .= $resp.',';
                             }
                             $correct_resp = substr(
                                 $correct_resp,
@@ -4147,7 +4147,7 @@ 
                                         lp_iv_id = $lp_iv_id AND
                                         (
                                             order_id = $index OR
-                                            interaction_id = '" . Database::escape_string($interaction[0]) . "'
+                                            interaction_id = '".Database::escape_string($interaction[0])."'
                                         )
                                     ";
                         $iva_res = Database::query($iva_sql);
@@ -4229,11 +4229,11 @@ 
     public function add_audio()
     {
         $course_info = api_get_course_info();
-        $filepath = api_get_path(SYS_COURSE_PATH) . $course_info['path'] . '/document/';
+        $filepath = api_get_path(SYS_COURSE_PATH).$course_info['path'].'/document/';
 
-        if (!is_dir($filepath . 'audio')) {
+        if (!is_dir($filepath.'audio')) {
             mkdir(
-                $filepath . 'audio',
+                $filepath.'audio',
                 api_get_permissions_for_new_directories()
             );
             $audio_id = add_document(
@@ -4292,10 +4292,10 @@ 
             // Store the mp3 file in the lp_item table.
             $tbl_lp_item = Database::get_course_table(TABLE_LP_ITEM);
             $sql = "UPDATE $tbl_lp_item SET
-                        audio = '" . Database::escape_string($file_path) . "'
+                        audio = '".Database::escape_string($file_path)."'
                     WHERE
                         c_id = {$course_info['real_id']} AND
-                        id = '" . intval($this->db_id) . "'";
+                        id = '".intval($this->db_id)."'";
             Database::query($sql);
         }
 
@@ -4320,10 +4320,10 @@ 
             // Store the mp3 file in the lp_item table.
             $tbl_lp_item = Database::get_course_table(TABLE_LP_ITEM);
             $sql = "UPDATE $tbl_lp_item SET
-                        audio = '" . Database::escape_string($file_path) . "'
+                        audio = '".Database::escape_string($file_path)."'
                     WHERE
                         c_id = {$course_info['real_id']} AND
-                        id = " . intval($this->db_id);
+                        id = ".intval($this->db_id);
             Database::query($sql);
         }
         return $file_path;
@@ -4344,7 +4344,7 @@ 
         }
         $sql = "UPDATE $tbl_lp_item SET
                 audio = ''
-                WHERE c_id = $course_id AND id IN (" . $this->db_id . ")";
+                WHERE c_id = $course_id AND id IN (".$this->db_id.")";
         Database::query($sql);
     }
 
@@ -4390,7 +4390,7 @@ 
 
         if ($type == 'simple') {
             if (in_array($status, array('failed', 'passed', 'browsed'))) {
-                $myLessonStatus = get_lang('ScormIncomplete');;
+                $myLessonStatus = get_lang('ScormIncomplete'); ;
                 $classStatus = 'warning';
             }
         }
@@ -4539,7 +4539,7 @@ 
      */
     public function createForumThread($currentForumId)
     {
-        require_once api_get_path(SYS_CODE_PATH) . '/forum/forumfunction.inc.php';
+        require_once api_get_path(SYS_CODE_PATH).'/forum/forumfunction.inc.php';
 
         $em = Database::getManager();
         $threadRepo = $em->getRepository('ChamiloCourseBundle:CForumThread');

main/lp/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,'../') || strpos($doc_url,'/..')) {
+if (strpos($doc_url, '../') || 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;