chamilo / chamilo-lms

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/admin/update_schema.php

Spacing +1 added lines, -1 removed lines

@@ -60,5 +60,5 @@
 }
 
 Display::display_header(get_lang('SyncDatabaseWithSchema'));
-echo $content ;
+echo $content;
 Display::display_footer();

main/inc/lib/blog.lib.php

Spacing +370 added lines, -370 removed lines

@@ -29,8 +29,8 @@ 
             $tbl_blogs = Database::get_course_table(TABLE_BLOGS);
 
             $sql = "SELECT blog_name
-                    FROM " . $tbl_blogs . "
-                    WHERE c_id = $course_id AND blog_id = " . intval($blog_id);
+                    FROM " . $tbl_blogs."
+                    WHERE c_id = $course_id AND blog_id = ".intval($blog_id);
 
             $result = Database::query($sql);
             $blog = Database::fetch_array($result);
@@ -82,9 +82,9 @@ 
                 ON (blogs_rel_user.user_id = user.user_id)
                 WHERE
                     blogs_rel_user.c_id = $course_id AND
-                    blogs_rel_user.blog_id = '" . (int)$blog_id."'";
+                    blogs_rel_user.blog_id = '".(int) $blog_id."'";
         $result = Database::query($sql);
-        $blog_members = array ();
+        $blog_members = array();
         while ($user = Database::fetch_array($result)) {
             $blog_members[$user['user_id']] = api_get_person_name(
                 $user['firstname'],
@@ -128,7 +128,7 @@ 
                 'blog_name' => $title,
                 'blog_subtitle' =>  $subtitle,
                 'date_creation' => $current_date,
-                'visibility' => 1 ,
+                'visibility' => 1,
                 'session_id' => $session_id,
             ];
             $this_blog_id = Database::insert($tbl_blogs, $params);
@@ -165,7 +165,7 @@ 
 
             // Put it on course homepage
             $sql = "INSERT INTO $tbl_tool (c_id, name, link, image, visibility, admin, address, added_tool, session_id, target)
-                    VALUES ($course_id, '".Database::escape_string($title)."','blog/blog.php?blog_id=".(int)$this_blog_id."','blog.gif','1','0','pastillegris.gif',0,'$session_id', '')";
+                    VALUES ($course_id, '".Database::escape_string($title)."','blog/blog.php?blog_id=".(int) $this_blog_id."','blog.gif','1','0','pastillegris.gif',0,'$session_id', '')";
             Database::query($sql);
 
             $toolId = Database::insert_id();
@@ -198,7 +198,7 @@ 
                 blog_subtitle = '".Database::escape_string($subtitle)."'
                 WHERE
                     c_id = $course_id AND
-                    blog_id ='".Database::escape_string((int)$blog_id)."'
+                    blog_id ='".Database::escape_string((int) $blog_id)."'
                 LIMIT 1";
         Database::query($sql);
 
@@ -214,7 +214,7 @@ 
         // Update course homepage link
         $sql = "UPDATE $tbl_tool SET
                 name = '".Database::escape_string($title)."'
-                WHERE c_id = $course_id AND link = 'blog/blog.php?blog_id=".(int)$blog_id."' 
+                WHERE c_id = $course_id AND link = 'blog/blog.php?blog_id=".(int) $blog_id."' 
                 LIMIT 1";
         Database::query($sql);
     }
@@ -257,7 +257,7 @@ 
         Database::query($sql);
 
         // Delete blog
-        $sql ="DELETE FROM $tbl_blogs WHERE c_id = $course_id AND blog_id ='".$blog_id."'";
+        $sql = "DELETE FROM $tbl_blogs WHERE c_id = $course_id AND blog_id ='".$blog_id."'";
         Database::query($sql);
 
         // Delete from course homepage
@@ -288,13 +288,13 @@ 
         $course_id = $_course['real_id'];
 
         $blog_table_attachment = Database::get_course_table(TABLE_BLOGS_ATTACHMENT);
-        $upload_ok=true;
-        $has_attachment=false;
+        $upload_ok = true;
+        $has_attachment = false;
         $current_date = api_get_utc_datetime();
 
         if (!empty($_FILES['user_upload']['name'])) {
             $upload_ok = process_uploaded_file($_FILES['user_upload']);
-            $has_attachment=true;
+            $has_attachment = true;
         }
 
         if ($upload_ok) {
@@ -303,7 +303,7 @@ 
 
             // Create the post
             $sql = "INSERT INTO $tbl_blogs_posts (c_id, title, full_text, date_creation, blog_id, author_id )
-                    VALUES ($course_id, '".Database::escape_string($title)."', '".Database::escape_string($full_text)."','".$current_date."', '".(int)$blog_id."', '".(int)$_user['user_id']."');";
+                    VALUES ($course_id, '".Database::escape_string($title)."', '".Database::escape_string($full_text)."','".$current_date."', '".(int) $blog_id."', '".(int) $_user['user_id']."');";
 
             Database::query($sql);
             $last_post_id = Database::insert_id();
@@ -314,7 +314,7 @@ 
             }
 
             if ($has_attachment) {
-                $courseDir   = $_course['path'].'/upload/blog';
+                $courseDir = $_course['path'].'/upload/blog';
                 $sys_course_path = api_get_path(SYS_COURSE_PATH);
                 $updir = $sys_course_path.$courseDir;
 
@@ -365,9 +365,9 @@ 
 
         // Create the post
         $sql = "UPDATE $tbl_blogs_posts SET
-                title = '" . Database::escape_string($title)."',
+                title = '".Database::escape_string($title)."',
                 full_text = '" . Database::escape_string($full_text)."'
-                WHERE c_id = $course_id AND post_id ='".(int)$post_id."' AND blog_id ='".(int)$blog_id."'
+                WHERE c_id = $course_id AND post_id ='".(int) $post_id."' AND blog_id ='".(int) $blog_id."'
                 LIMIT 1 ";
         Database::query($sql);
     }
@@ -388,17 +388,17 @@ 
 
         // Delete ratings on this comment
         $sql = "DELETE FROM $tbl_blogs_rating
-                WHERE c_id = $course_id AND blog_id = '".(int)$blog_id."' AND item_id = '".(int)$post_id."' AND rating_type = 'post'";
+                WHERE c_id = $course_id AND blog_id = '".(int) $blog_id."' AND item_id = '".(int) $post_id."' AND rating_type = 'post'";
         Database::query($sql);
 
         // Delete the post
         $sql = "DELETE FROM $tbl_blogs_posts
-                WHERE c_id = $course_id AND post_id = '".(int)$post_id."'";
+                WHERE c_id = $course_id AND post_id = '".(int) $post_id."'";
         Database::query($sql);
 
         // Delete the comments
         $sql = "DELETE FROM $tbl_blogs_comments
-                WHERE c_id = $course_id AND post_id = '".(int)$post_id."' AND blog_id = '".(int)$blog_id."'";
+                WHERE c_id = $course_id AND post_id = '".(int) $post_id."' AND blog_id = '".(int) $blog_id."'";
         Database::query($sql);
 
         // Delete posts and attachments
@@ -425,7 +425,7 @@ 
     ) {
         $_user = api_get_user_info();
         $_course = api_get_course_info();
-        $blog_table_attachment 	= Database::get_course_table(TABLE_BLOGS_ATTACHMENT);
+        $blog_table_attachment = Database::get_course_table(TABLE_BLOGS_ATTACHMENT);
 
         $upload_ok = true;
         $has_attachment = false;
@@ -434,7 +434,7 @@ 
 
         if (!empty($_FILES['user_upload']['name'])) {
             $upload_ok = process_uploaded_file($_FILES['user_upload']);
-            $has_attachment=true;
+            $has_attachment = true;
         }
 
         if ($upload_ok) {
@@ -443,7 +443,7 @@ 
 
             // Create the comment
             $sql = "INSERT INTO $tbl_blogs_comments (c_id, title, comment, author_id, date_creation, blog_id, post_id, parent_comment_id, task_id )
-                    VALUES ($course_id, '".Database::escape_string($title)."', '".Database::escape_string($full_text)."', '".(int)$_user['user_id']."','".$current_date."', '".(int)$blog_id."', '".(int)$post_id."', '".(int)$parent_id."', '".(int)$task_id."')";
+                    VALUES ($course_id, '".Database::escape_string($title)."', '".Database::escape_string($full_text)."', '".(int) $_user['user_id']."','".$current_date."', '".(int) $blog_id."', '".(int) $post_id."', '".(int) $parent_id."', '".(int) $task_id."')";
             Database::query($sql);
 
             // Empty post values, or they are shown on the page again
@@ -455,7 +455,7 @@ 
             }
 
             if ($has_attachment) {
-                $courseDir   = $_course['path'].'/upload/blog';
+                $courseDir = $_course['path'].'/upload/blog';
                 $sys_course_path = api_get_path(SYS_COURSE_PATH);
                 $updir = $sys_course_path.$courseDir;
 
@@ -466,19 +466,19 @@ 
                 );
 
                 // user's file name
-                $file_name =$_FILES['user_upload']['name'];
+                $file_name = $_FILES['user_upload']['name'];
 
                 if (!filter_extension($new_file_name)) {
                     Display :: display_error_message(get_lang('UplUnableToSaveFileFilteredExtension'));
                 } else {
                     $new_file_name = uniqid('');
-                    $new_path=$updir.'/'.$new_file_name;
-                    $result= @move_uploaded_file($_FILES['user_upload']['tmp_name'], $new_path);
+                    $new_path = $updir.'/'.$new_file_name;
+                    $result = @move_uploaded_file($_FILES['user_upload']['tmp_name'], $new_path);
                     $comment = Database::escape_string($file_comment);
 
                     // Storing the attachments if any
                     if ($result) {
-                        $sql='INSERT INTO '.$blog_table_attachment.'(c_id, filename,comment, path, post_id,size,blog_id,comment_id) '.
+                        $sql = 'INSERT INTO '.$blog_table_attachment.'(c_id, filename,comment, path, post_id,size,blog_id,comment_id) '.
                              "VALUES ($course_id, '".Database::escape_string($file_name)."', '".$comment."', '".Database::escape_string($new_file_name)."' , '".$post_id."', '".$_FILES['user_upload']['size']."',  '".$blog_id."', '".$last_id."'  )";
                         Database::query($sql);
 
@@ -553,7 +553,7 @@ 
 
         // Create the task
         $sql = "INSERT INTO $tbl_blogs_tasks (c_id, blog_id, title, description, color, system_task)
-                VALUES ($course_id , '".(int)$blog_id."', '" . Database::escape_string($title)."', '" . Database::escape_string($description)."', '" . Database::escape_string($color)."', '0');";
+                VALUES ($course_id , '".(int) $blog_id."', '".Database::escape_string($title)."', '".Database::escape_string($description)."', '".Database::escape_string($color)."', '0');";
         Database::query($sql);
 
         $task_id = Database::insert_id();
@@ -563,13 +563,13 @@ 
             Database::query($sql);
         }
 
-        $tool = 'BLOG_' . $blog_id;
+        $tool = 'BLOG_'.$blog_id;
 
         if ($articleDelete == 'on') {
-            $sql = " INSERT INTO " . $tbl_tasks_permissions . " ( c_id,  task_id, tool, action) VALUES (
-                    '" . (int)$course_id . "',
-                    '" . (int)$task_id . "',
-                    '" . Database::escape_string($tool) . "',
+            $sql = " INSERT INTO ".$tbl_tasks_permissions." ( c_id,  task_id, tool, action) VALUES (
+                    '" . (int) $course_id."',
+                    '" . (int) $task_id."',
+                    '" . Database::escape_string($tool)."',
                     'article_delete'
                 )";
             Database::query($sql);
@@ -584,10 +584,10 @@ 
 
         if ($articleEdit == 'on') {
             $sql = "
-                INSERT INTO " . $tbl_tasks_permissions . " (c_id, task_id, tool, action ) VALUES (
-                    '" . (int)$course_id . "',
-                    '" . (int)$task_id . "',
-                    '" . Database::escape_string($tool) . "',
+                INSERT INTO " . $tbl_tasks_permissions." (c_id, task_id, tool, action ) VALUES (
+                    '" . (int) $course_id."',
+                    '" . (int) $task_id."',
+                    '" . Database::escape_string($tool)."',
                     'article_edit'
                 )";
             Database::query($sql);
@@ -601,10 +601,10 @@ 
 
         if ($commentsDelete == 'on') {
             $sql = "
-                INSERT INTO " . $tbl_tasks_permissions . " (c_id, task_id, tool, action ) VALUES (
-                    '" . (int)$course_id . "',
-                    '" . (int)$task_id . "',
-                    '" . Database::escape_string($tool) . "',
+                INSERT INTO " . $tbl_tasks_permissions." (c_id, task_id, tool, action ) VALUES (
+                    '" . (int) $course_id."',
+                    '" . (int) $task_id."',
+                    '" . Database::escape_string($tool)."',
                     'article_comments_delete'
                 )";
             Database::query($sql);
@@ -645,20 +645,20 @@ 
                     title = '".Database::escape_string($title)."',
                     description = '".Database::escape_string($description)."',
                     color = '".Database::escape_string($color)."'
-                WHERE c_id = $course_id AND task_id ='".(int)$task_id."' LIMIT 1";
+                WHERE c_id = $course_id AND task_id ='".(int) $task_id."' LIMIT 1";
         Database::query($sql);
 
-        $tool = 'BLOG_' . $blog_id;
+        $tool = 'BLOG_'.$blog_id;
 
-        $sql = "DELETE FROM " . $tbl_tasks_permissions . "
-                WHERE c_id = $course_id AND task_id = '" . (int)$task_id."'";
+        $sql = "DELETE FROM ".$tbl_tasks_permissions."
+                WHERE c_id = $course_id AND task_id = '".(int) $task_id."'";
         Database::query($sql);
 
         if ($articleDelete == 'on') {
-            $sql = "INSERT INTO " . $tbl_tasks_permissions . " ( c_id, task_id, tool, action) VALUES (
-                    '" . (int)$course_id . "',
-                    '" . (int)$task_id . "',
-                    '" . Database::escape_string($tool) . "',
+            $sql = "INSERT INTO ".$tbl_tasks_permissions." ( c_id, task_id, tool, action) VALUES (
+                    '" . (int) $course_id."',
+                    '" . (int) $task_id."',
+                    '" . Database::escape_string($tool)."',
                     'article_delete'
                 )";
             Database::query($sql);
@@ -671,10 +671,10 @@ 
         }
 
         if ($articleEdit == 'on') {
-            $sql = "INSERT INTO " . $tbl_tasks_permissions . " (c_id, task_id, tool, action) VALUES (
-                    '" . (int)$course_id . "',
-                    '" . (int)$task_id . "',
-                    '" . Database::escape_string($tool) . "',
+            $sql = "INSERT INTO ".$tbl_tasks_permissions." (c_id, task_id, tool, action) VALUES (
+                    '" . (int) $course_id."',
+                    '" . (int) $task_id."',
+                    '" . Database::escape_string($tool)."',
                     'article_edit'
                 )";
             Database::query($sql);
@@ -687,10 +687,10 @@ 
         }
 
         if ($commentsDelete == 'on') {
-            $sql = " INSERT INTO " . $tbl_tasks_permissions . " (c_id, task_id, tool, action) VALUES (
-                    '" . (int)$course_id . "',
-                    '" . (int)$task_id . "',
-                    '" . Database::escape_string($tool) . "',
+            $sql = " INSERT INTO ".$tbl_tasks_permissions." (c_id, task_id, tool, action) VALUES (
+                    '" . (int) $course_id."',
+                    '" . (int) $task_id."',
+                    '" . Database::escape_string($tool)."',
                     'article_comments_delete'
                 )";
             Database::query($sql);
@@ -715,7 +715,7 @@ 
 
         // Delete posts
         $sql = "DELETE FROM $tbl_blogs_tasks
-                WHERE c_id = $course_id AND blog_id = '".(int)$blog_id."' AND task_id = '".(int)$task_id."'";
+                WHERE c_id = $course_id AND blog_id = '".(int) $blog_id."' AND task_id = '".(int) $task_id."'";
         Database::query($sql);
     }
 
@@ -732,9 +732,9 @@ 
         $sql = "DELETE FROM $tbl_blogs_tasks_rel_user
                 WHERE
                     c_id = $course_id AND
-                    blog_id = '".(int)$blog_id."' AND
-                    task_id = '".(int)$task_id."' AND
-                    user_id = '".(int)$user_id."'";
+                    blog_id = '".(int) $blog_id."' AND
+                    task_id = '".(int) $task_id."' AND
+                    user_id = '".(int) $user_id."'";
         Database::query($sql);
     }
 
@@ -748,7 +748,7 @@ 
         $_user = api_get_user_info();
 
         $tbl_blogs = Database::get_course_table(TABLE_BLOGS);
-        $tbl_blogs_tasks_rel_user 	= Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER);
+        $tbl_blogs_tasks_rel_user = Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER);
         $tbl_blogs_tasks = Database::get_course_table(TABLE_BLOGS_TASKS);
 
         $course_id = api_get_course_int_id();
@@ -765,7 +765,7 @@ 
                         task.c_id = $course_id AND
                         blog.c_id = $course_id AND
                         task_rel_user.c_id = $course_id AND
-                        task_rel_user.user_id = ".(int)$_user['user_id']."
+                        task_rel_user.user_id = ".(int) $_user['user_id']."
                     ORDER BY target_date ASC";
 
             $result = Database::query($sql);
@@ -773,7 +773,7 @@ 
             if (Database::num_rows($result) > 0) {
                 echo '<ul>';
                 while ($mytask = Database::fetch_array($result)) {
-                    echo '<li><a href="blog.php?action=execute_task&blog_id=' . $mytask['blog_id'] . '&task_id='.stripslashes($mytask['task_id']) . '" title="[Blog: '.stripslashes($mytask['blog_name']) . '] ' . get_lang('ExecuteThisTask') . '">'.stripslashes($mytask['title']) . '</a></li>';
+                    echo '<li><a href="blog.php?action=execute_task&blog_id='.$mytask['blog_id'].'&task_id='.stripslashes($mytask['task_id']).'" title="[Blog: '.stripslashes($mytask['blog_name']).'] '.get_lang('ExecuteThisTask').'">'.stripslashes($mytask['title']).'</a></li>';
                 }
                 echo '<ul>';
             } else {
@@ -797,7 +797,7 @@ 
 
         // Get blog properties
         $sql = "SELECT blog_name, visibility FROM $tbl_blogs
-                WHERE c_id = $course_id AND blog_id='".(int)$blog_id."'";
+                WHERE c_id = $course_id AND blog_id='".(int) $blog_id."'";
         $result = Database::query($sql);
         $blog = Database::fetch_array($result);
         $visibility = $blog['visibility'];
@@ -806,7 +806,7 @@ 
         if ($visibility == 1) {
             // Change visibility state, remove from course home.
             $sql = "UPDATE $tbl_blogs SET visibility = '0'
-                    WHERE c_id = $course_id AND blog_id ='".(int)$blog_id."' LIMIT 1";
+                    WHERE c_id = $course_id AND blog_id ='".(int) $blog_id."' LIMIT 1";
             Database::query($sql);
 
             $sql = "DELETE FROM $tbl_tool
@@ -815,11 +815,11 @@ 
         } else {
             // Change visibility state, add to course home.
             $sql = "UPDATE $tbl_blogs SET visibility = '1'
-                    WHERE c_id = $course_id AND blog_id ='".(int)$blog_id."' LIMIT 1";
+                    WHERE c_id = $course_id AND blog_id ='".(int) $blog_id."' LIMIT 1";
             Database::query($sql);
 
             $sql = "INSERT INTO $tbl_tool (c_id, name, link, image, visibility, admin, address, added_tool, target )
-                    VALUES ($course_id, '".Database::escape_string($title)."', 'blog/blog.php?blog_id=".(int)$blog_id."', 'blog.gif', '1', '0', 'pastillegris.gif', '0', '_self')";
+                    VALUES ($course_id, '".Database::escape_string($title)."', 'blog/blog.php?blog_id=".(int) $blog_id."', 'blog.gif', '1', '0', 'pastillegris.gif', '0', '_self')";
             Database::query($sql);
             $id = Database::insert_id();
 
@@ -850,14 +850,14 @@ 
                 FROM $tbl_blogs_posts post
                 INNER JOIN $tbl_users user
                 ON post.author_id = user.user_id
-                WHERE 	post.blog_id = '".(int)$blog_id."' AND
+                WHERE 	post.blog_id = '".(int) $blog_id."' AND
                         post.c_id = $course_id AND
                         $filter
-                ORDER BY post_id DESC LIMIT 0,".(int)$max_number_of_posts;
+                ORDER BY post_id DESC LIMIT 0,".(int) $max_number_of_posts;
         $result = Database::query($sql);
 
         // Display
-        if(Database::num_rows($result) > 0) {
+        if (Database::num_rows($result) > 0) {
             $limit = 200;
             while ($blog_post = Database::fetch_array($result)) {
                 // Get number of comments
@@ -865,8 +865,8 @@ 
                         FROM $tbl_blogs_comments
                         WHERE
                             c_id = $course_id AND
-                            blog_id = '".(int)$blog_id."' AND
-                            post_id = '" . (int)$blog_post['post_id']."'";
+                            blog_id = '".(int) $blog_id."' AND
+                            post_id = '" . (int) $blog_post['post_id']."'";
                 $tmp = Database::query($sql);
                 $blog_post_comments = Database::fetch_array($tmp);
 
@@ -877,11 +877,11 @@ 
 
                 // Create an introduction text (but keep FULL sentences)
                 $words = 0;
-                $blog_post_text_cut = cut($blog_post_text, $limit) ;
+                $blog_post_text_cut = cut($blog_post_text, $limit);
                 $words = strlen($blog_post_text);
 
                 if ($words >= $limit) {
-                    $readMoreLink = ' <div class="link" onclick="document.getElementById(\'blogpost_text_' . $blog_post_id . '\').style.display=\'block\'; document.getElementById(\'blogpost_introduction_' . $blog_post_id . '\').style.display=\'none\'">' . get_lang('ReadMore') . '</div>';
+                    $readMoreLink = ' <div class="link" onclick="document.getElementById(\'blogpost_text_'.$blog_post_id.'\').style.display=\'block\'; document.getElementById(\'blogpost_introduction_'.$blog_post_id.'\').style.display=\'none\'">'.get_lang('ReadMore').'</div>';
                     $introduction_text = $blog_post_text_cut;
                 } else {
                     $introduction_text = $blog_post_text;
@@ -891,27 +891,27 @@ 
                 $introduction_text = stripslashes($introduction_text);
 
                 echo '<div class="blogpost">';
-                echo '<span class="blogpost_title"><a href="blog.php?action=view_post&blog_id=' . $blog_id . '&post_id=' . $blog_post['post_id'] . '#add_comment" title="' . get_lang('ReadPost') . '" >'.stripslashes($blog_post['title']) . '</a></span>';
-                echo '<span class="blogpost_date"><a href="blog.php?action=view_post&blog_id=' . $blog_id . '&post_id=' . $blog_post['post_id'] . '#add_comment" title="' . get_lang('ReadPost') . '" >' . $blog_post_date . '</a></span>';
-                echo '<div class="blogpost_introduction" id="blogpost_introduction_'.$blog_post_id.'">' . $introduction_text.$readMoreLink.'</div>';
-                echo '<div class="blogpost_text" id="blogpost_text_' . $blog_post_id . '" style="display: none">' . $blog_post_text . '</div>';
+                echo '<span class="blogpost_title"><a href="blog.php?action=view_post&blog_id='.$blog_id.'&post_id='.$blog_post['post_id'].'#add_comment" title="'.get_lang('ReadPost').'" >'.stripslashes($blog_post['title']).'</a></span>';
+                echo '<span class="blogpost_date"><a href="blog.php?action=view_post&blog_id='.$blog_id.'&post_id='.$blog_post['post_id'].'#add_comment" title="'.get_lang('ReadPost').'" >'.$blog_post_date.'</a></span>';
+                echo '<div class="blogpost_introduction" id="blogpost_introduction_'.$blog_post_id.'">'.$introduction_text.$readMoreLink.'</div>';
+                echo '<div class="blogpost_text" id="blogpost_text_'.$blog_post_id.'" style="display: none">'.$blog_post_text.'</div>';
 
-                $file_name_array = get_blog_attachment($blog_id,$blog_post_id,0);
+                $file_name_array = get_blog_attachment($blog_id, $blog_post_id, 0);
 
                 if (!empty($file_name_array)) {
                     echo '<br /><br />';
-                    echo Display::return_icon('attachment.gif',get_lang('Attachment'));
+                    echo Display::return_icon('attachment.gif', get_lang('Attachment'));
                     echo '<a href="download.php?file=';
                     echo $file_name_array['path'];
                     echo ' "> '.$file_name_array['filename'].' </a><br />';
                     echo '</span>';
                 }
                 $username = api_htmlentities(sprintf(get_lang('LoginX'), $blog_post['username']), ENT_QUOTES);
-                echo '<span class="blogpost_info">' . get_lang('Author') . ': ' . Display::tag('span', api_get_person_name($blog_post['firstname'], $blog_post['lastname']), array('title'=>$username)) .' - <a href="blog.php?action=view_post&blog_id=' . $blog_id . '&post_id=' . $blog_post['post_id'] . '#add_comment" title="' . get_lang('ReadPost') . '" >' . get_lang('Comments') . ': ' . $blog_post_comments['number_of_comments'] . '</a></span>';
+                echo '<span class="blogpost_info">'.get_lang('Author').': '.Display::tag('span', api_get_person_name($blog_post['firstname'], $blog_post['lastname']), array('title'=>$username)).' - <a href="blog.php?action=view_post&blog_id='.$blog_id.'&post_id='.$blog_post['post_id'].'#add_comment" title="'.get_lang('ReadPost').'" >'.get_lang('Comments').': '.$blog_post_comments['number_of_comments'].'</a></span>';
                 echo '</div>';
             }
         } else {
-            if($filter == '1=1') {
+            if ($filter == '1=1') {
                 echo get_lang('NoArticles');
             } else {
                 echo get_lang('NoArticleMatches');
@@ -925,19 +925,19 @@ 
      * @param Integer $blog_id
      * @param String $query_string
      */
-    public static function display_search_results ($blog_id, $query_string)
+    public static function display_search_results($blog_id, $query_string)
     {
         // Init
         $query_string = Database::escape_string($query_string);
-        $query_string_parts = explode(' ',$query_string);
+        $query_string_parts = explode(' ', $query_string);
         $query_string = array();
         foreach ($query_string_parts as $query_part) {
-            $query_string[] = " full_text LIKE '%" . $query_part."%' OR title LIKE '%" . $query_part."%' ";
+            $query_string[] = " full_text LIKE '%".$query_part."%' OR title LIKE '%".$query_part."%' ";
         }
-        $query_string = '('.implode('OR',$query_string) . ')';
+        $query_string = '('.implode('OR', $query_string).')';
 
         // Display the posts
-        echo '<span class="blogpost_title">' . get_lang('SearchResults') . '</span>';
+        echo '<span class="blogpost_title">'.get_lang('SearchResults').'</span>';
         Blog::display_blog_posts($blog_id, $query_string);
     }
 
@@ -950,14 +950,14 @@ 
     public static function display_day_results($blog_id, $query_string)
     {
         $date_output = $query_string;
-        $date = explode('-',$query_string);
-        $query_string = ' DAYOFMONTH(date_creation) =' . intval($date[2]) . ' AND MONTH(date_creation) =' . intval($date[1]) . ' AND YEAR(date_creation) =' . intval($date[0]);
+        $date = explode('-', $query_string);
+        $query_string = ' DAYOFMONTH(date_creation) ='.intval($date[2]).' AND MONTH(date_creation) ='.intval($date[1]).' AND YEAR(date_creation) ='.intval($date[0]);
 
         // Put date in correct output format
         $date_output = api_format_date($date_output, DATE_FORMAT_LONG);
 
         // Display the posts
-        echo '<span class="blogpost_title">' . get_lang('PostsOf') . ': ' . $date_output . '</span>';
+        echo '<span class="blogpost_title">'.get_lang('PostsOf').': '.$date_output.'</span>';
         Blog::display_blog_posts($blog_id, $query_string);
     }
 
@@ -984,8 +984,8 @@ 
                     ON post.author_id = user.user_id
                 WHERE
                     post.c_id = $course_id AND
-                    post.blog_id = '".(int)$blog_id."' AND
-                    post.post_id = '".(int)$post_id."'
+                    post.blog_id = '".(int) $blog_id."' AND
+                    post.post_id = '".(int) $post_id."'
                 ORDER BY post_id DESC";
         $result = Database::query($sql);
         $blog_post = Database::fetch_array($result);
@@ -993,7 +993,7 @@ 
         // Get number of comments
         $sql = "SELECT COUNT(1) as number_of_comments
                 FROM $tbl_blogs_comments
-                WHERE c_id = $course_id AND blog_id = '".(int)$blog_id."' AND post_id = '".(int)$post_id."'";
+                WHERE c_id = $course_id AND blog_id = '".(int) $blog_id."' AND post_id = '".(int) $post_id."'";
         $result = Database::query($sql);
         $blog_post_comments = Database::fetch_array($result);
 
@@ -1004,34 +1004,34 @@ 
 
         $task_id = (isset($_GET['task_id']) && is_numeric($_GET['task_id'])) ? intval($_GET['task_id']) : 0;
 
-        if (api_is_allowed('BLOG_' . $blog_id, 'article_edit', $task_id)) {
-            $blog_post_actions .= '<a href="blog.php?action=edit_post&blog_id=' . $blog_id . '&post_id=' . $post_id . '&article_id=' . $blog_post['post_id'] . '&task_id=' . $task_id . '" title="' . get_lang('EditThisPost') . '">';
-            $blog_post_actions .=  Display::return_icon('edit.png');
+        if (api_is_allowed('BLOG_'.$blog_id, 'article_edit', $task_id)) {
+            $blog_post_actions .= '<a href="blog.php?action=edit_post&blog_id='.$blog_id.'&post_id='.$post_id.'&article_id='.$blog_post['post_id'].'&task_id='.$task_id.'" title="'.get_lang('EditThisPost').'">';
+            $blog_post_actions .= Display::return_icon('edit.png');
             $blog_post_actions .= '</a>';
         }
 
-        if (api_is_allowed('BLOG_' . $blog_id, 'article_delete', $task_id)) {
-            $blog_post_actions .= '<a href="blog.php?action=view_post&blog_id=' . $blog_id . '&post_id=' . $post_id . '&do=delete_article&article_id=' . $blog_post['post_id'] . '&task_id=' . $task_id . '" title="' . get_lang('DeleteThisArticle') . '" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"),ENT_QUOTES,$charset)). '\')) return false;">';
+        if (api_is_allowed('BLOG_'.$blog_id, 'article_delete', $task_id)) {
+            $blog_post_actions .= '<a href="blog.php?action=view_post&blog_id='.$blog_id.'&post_id='.$post_id.'&do=delete_article&article_id='.$blog_post['post_id'].'&task_id='.$task_id.'" title="'.get_lang('DeleteThisArticle').'" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"), ENT_QUOTES, $charset)).'\')) return false;">';
             $blog_post_actions .= Display::return_icon('delete.png');
             $blog_post_actions .= '</a>';
         }
 
-        if (api_is_allowed('BLOG_' . $blog_id, 'article_rate'))
-            $rating_select = Blog::display_rating_form('post',$blog_id,$post_id);
+        if (api_is_allowed('BLOG_'.$blog_id, 'article_rate'))
+            $rating_select = Blog::display_rating_form('post', $blog_id, $post_id);
 
-        $blog_post_text=stripslashes($blog_post_text);
+        $blog_post_text = stripslashes($blog_post_text);
 
         // Display post
         echo '<div class="blogpost">';
-        echo '<span class="blogpost_title"><a href="blog.php?action=view_post&blog_id=' . $blog_id . '&post_id=' . $blog_post['post_id'] . '" title="' . get_lang('ReadPost') . '" >'.stripslashes($blog_post['title']) . '</a></span>';
-        echo '<span class="blogpost_date">' . $blog_post_date . '</span>';
-        echo '<span class="blogpost_text">' . $blog_post_text . '</span><br />';
+        echo '<span class="blogpost_title"><a href="blog.php?action=view_post&blog_id='.$blog_id.'&post_id='.$blog_post['post_id'].'" title="'.get_lang('ReadPost').'" >'.stripslashes($blog_post['title']).'</a></span>';
+        echo '<span class="blogpost_date">'.$blog_post_date.'</span>';
+        echo '<span class="blogpost_text">'.$blog_post_text.'</span><br />';
 
         $file_name_array = get_blog_attachment($blog_id, $post_id);
 
         if (!empty($file_name_array)) {
             echo ' <br />';
-            echo Display::return_icon('attachment.gif',get_lang('Attachment'));
+            echo Display::return_icon('attachment.gif', get_lang('Attachment'));
             echo '<a href="download.php?file=';
             echo $file_name_array['path'];
             echo ' "> '.$file_name_array['filename'].' </a>';
@@ -1042,20 +1042,20 @@ 
             echo '<br />';
         }
         $username = api_htmlentities(sprintf(get_lang('LoginX'), $blog_post['username']), ENT_QUOTES);
-        echo '<span class="blogpost_info">'.get_lang('Author').': ' .Display::tag('span', api_get_person_name($blog_post['firstname'], $blog_post['lastname']), array('title'=>$username)).' - '.get_lang('Comments').': '.$blog_post_comments['number_of_comments'].' - '.get_lang('Rating').': '.Blog::display_rating('post',$blog_id,$post_id).$rating_select.'</span>';
-        echo '<span class="blogpost_actions">' . $blog_post_actions . '</span>';
+        echo '<span class="blogpost_info">'.get_lang('Author').': '.Display::tag('span', api_get_person_name($blog_post['firstname'], $blog_post['lastname']), array('title'=>$username)).' - '.get_lang('Comments').': '.$blog_post_comments['number_of_comments'].' - '.get_lang('Rating').': '.Blog::display_rating('post', $blog_id, $post_id).$rating_select.'</span>';
+        echo '<span class="blogpost_actions">'.$blog_post_actions.'</span>';
         echo '</div>';
 
         // Display comments if there are any
-        if($blog_post_comments['number_of_comments'] > 0) {
+        if ($blog_post_comments['number_of_comments'] > 0) {
             echo '<div class="comments">';
-                echo '<span class="blogpost_title">' . get_lang('Comments') . '</span><br />';
+                echo '<span class="blogpost_title">'.get_lang('Comments').'</span><br />';
                 Blog::get_threaded_comments(0, 0, $blog_id, $post_id, $task_id);
             echo '</div>';
         }
 
         // Display comment form
-        if (api_is_allowed('BLOG_' . $blog_id, 'article_comments_add')) {
+        if (api_is_allowed('BLOG_'.$blog_id, 'article_comments_add')) {
             Blog::display_new_comment_form($blog_id, $post_id, $blog_post['title']);
         }
     }
@@ -1083,16 +1083,16 @@ 
         $sql = "SELECT rating_id FROM $tbl_blogs_rating
                 WHERE
                     c_id = $course_id AND
-                    blog_id = '".(int)$blog_id."' AND
-                    item_id = '".(int)$item_id."' AND
+                    blog_id = '".(int) $blog_id."' AND
+                    item_id = '".(int) $item_id."' AND
                     rating_type = '".Database::escape_string($type)."' AND
-                    user_id = '".(int)$_user['user_id']."'";
+                    user_id = '".(int) $_user['user_id']."'";
         $result = Database::query($sql);
 
         // Add rating
         if (Database::num_rows($result) == 0) {
             $sql = "INSERT INTO $tbl_blogs_rating (c_id, blog_id, rating_type, item_id, user_id, rating )
-                    VALUES ($course_id, '".(int)$blog_id."', '".Database::escape_string($type)."', '".(int)$item_id."', '".(int)$_user['user_id']."', '".Database::escape_string($rating)."')";
+                    VALUES ($course_id, '".(int) $blog_id."', '".Database::escape_string($type)."', '".(int) $item_id."', '".(int) $_user['user_id']."', '".Database::escape_string($rating)."')";
             Database::query($sql);
 
             $id = Database::insert_id();
@@ -1123,8 +1123,8 @@ 
         $sql = "SELECT AVG(rating) as rating FROM $tbl_blogs_rating
                 WHERE
                     c_id = $course_id AND
-                    blog_id = '".(int)$blog_id."' AND
-                    item_id = '".(int)$item_id."' AND
+                    blog_id = '".(int) $blog_id."' AND
+                    item_id = '".(int) $item_id."' AND
                     rating_type = '".Database::escape_string($type)."' ";
         $result = Database::query($sql);
         $result = Database::fetch_array($result);
@@ -1139,7 +1139,7 @@ 
      * @param Integer $blog_id
      * @param integer $post_id
      */
-    public static function display_rating_form ($type, $blog_id, $post_id, $comment_id = NULL)
+    public static function display_rating_form($type, $blog_id, $post_id, $comment_id = NULL)
     {
         $_user = api_get_user_info();
         $tbl_blogs_rating = Database::get_course_table(TABLE_BLOGS_RATING);
@@ -1149,14 +1149,14 @@ 
             // Check if the user has already rated this post
             $sql = "SELECT rating_id FROM $tbl_blogs_rating
                     WHERE c_id = $course_id AND
-                    blog_id = '".(int)$blog_id."'
-                    AND item_id = '".(int)$post_id."'
+                    blog_id = '".(int) $blog_id."'
+                    AND item_id = '".(int) $post_id."'
                     AND rating_type = '".Database::escape_string($type)."'
-                    AND user_id = '".(int)$_user['user_id']."'";
+                    AND user_id = '".(int) $_user['user_id']."'";
             $result = Database::query($sql);
             // Add rating
             if (Database::num_rows($result) == 0) {
-                return ' - ' . get_lang('RateThis') . ': <form method="get" action="blog.php" style="display: inline" id="frm_rating_' . $type . '_' . $post_id . '" name="frm_rating_' . $type . '_' . $post_id . '"><select name="rating" onchange="document.forms[\'frm_rating_' . $type . '_' . $post_id . '\'].submit()"><option value="">-</option><option value="1">1</option><option value="2">2</option><option value="3">3</option><option value="4">4</option><option value="5">5</option><option value="6">6</option><option value="7">7</option><option value="8">8</option><option value="9">9</option><option value="10">10</option></select><input type="hidden" name="action" value="view_post" /><input type="hidden" name="type" value="' . $type . '" /><input type="hidden" name="do" value="rate" /><input type="hidden" name="blog_id" value="' . $blog_id . '" /><input type="hidden" name="post_id" value="' . $post_id . '" /></form>';
+                return ' - '.get_lang('RateThis').': <form method="get" action="blog.php" style="display: inline" id="frm_rating_'.$type.'_'.$post_id.'" name="frm_rating_'.$type.'_'.$post_id.'"><select name="rating" onchange="document.forms[\'frm_rating_'.$type.'_'.$post_id.'\'].submit()"><option value="">-</option><option value="1">1</option><option value="2">2</option><option value="3">3</option><option value="4">4</option><option value="5">5</option><option value="6">6</option><option value="7">7</option><option value="8">8</option><option value="9">9</option><option value="10">10</option></select><input type="hidden" name="action" value="view_post" /><input type="hidden" name="type" value="'.$type.'" /><input type="hidden" name="do" value="rate" /><input type="hidden" name="blog_id" value="'.$blog_id.'" /><input type="hidden" name="post_id" value="'.$post_id.'" /></form>';
             } else {
                 return '';
             }
@@ -1165,14 +1165,14 @@ 
         if ($type = 'comment') {
             // Check if the user has already rated this comment
             $sql = "SELECT rating_id FROM $tbl_blogs_rating
-                    WHERE c_id = $course_id AND blog_id = '".(int)$blog_id ."'
-                    AND item_id = '".(int)$comment_id."'
+                    WHERE c_id = $course_id AND blog_id = '".(int) $blog_id."'
+                    AND item_id = '".(int) $comment_id."'
                     AND rating_type = '".Database::escape_string($type)."'
-                    AND user_id = '".(int)$_user['user_id']."'";
+                    AND user_id = '".(int) $_user['user_id']."'";
             $result = Database::query($sql);
 
             if (Database::num_rows($result) == 0) {
-                return ' - ' . get_lang('RateThis') . ': <form method="get" action="blog.php" style="display: inline" id="frm_rating_' . $type . '_' . $comment_id . '" name="frm_rating_' . $type . '_' . $comment_id . '"><select name="rating" onchange="document.forms[\'frm_rating_' . $type . '_' . $comment_id . '\'].submit()"><option value="">-</option><option value="1">1</option><option value="2">2</option><option value="3">3</option><option value="4">4</option><option value="5">5</option><option value="6">6</option><option value="7">7</option><option value="8">8</option><option value="9">9</option><option value="10">10</option></select><input type="hidden" name="action" value="view_post" /><input type="hidden" name="type" value="' . $type . '" /><input type="hidden" name="do" value="rate" /><input type="hidden" name="blog_id" value="' . $blog_id . '" /><input type="hidden" name="post_id" value="' . $post_id . '" /><input type="hidden" name="comment_id" value="' . $comment_id . '" /></form>';
+                return ' - '.get_lang('RateThis').': <form method="get" action="blog.php" style="display: inline" id="frm_rating_'.$type.'_'.$comment_id.'" name="frm_rating_'.$type.'_'.$comment_id.'"><select name="rating" onchange="document.forms[\'frm_rating_'.$type.'_'.$comment_id.'\'].submit()"><option value="">-</option><option value="1">1</option><option value="2">2</option><option value="3">3</option><option value="4">4</option><option value="5">5</option><option value="6">6</option><option value="7">7</option><option value="8">8</option><option value="9">9</option><option value="10">10</option></select><input type="hidden" name="action" value="view_post" /><input type="hidden" name="type" value="'.$type.'" /><input type="hidden" name="do" value="rate" /><input type="hidden" name="blog_id" value="'.$blog_id.'" /><input type="hidden" name="post_id" value="'.$post_id.'" /><input type="hidden" name="comment_id" value="'.$comment_id.'" /></form>';
             } else {
                 return '';
             }
@@ -1207,11 +1207,11 @@ 
                 WHERE
                     comments.c_id = $course_id AND
                     parent_comment_id = $current AND
-                    comments.blog_id = '".(int)$blog_id."' AND
-                    comments.post_id = '".(int)$post_id."'";
+                    comments.blog_id = '".(int) $blog_id."' AND
+                    comments.post_id = '".(int) $post_id."'";
         $result = Database::query($sql);
 
-        while($comment = Database::fetch_array($result)) {
+        while ($comment = Database::fetch_array($result)) {
             // Select the children recursivly
             $tmp = "SELECT comments.*, user.lastname, user.firstname, user.username
                     FROM $tbl_blogs_comments comments
@@ -1220,8 +1220,8 @@ 
                     WHERE
                         comments.c_id = $course_id AND
                         comment_id = $current
-                        AND blog_id = '".(int)$blog_id."'
-                        AND post_id = '".(int)$post_id."'";
+                        AND blog_id = '".(int) $blog_id."'
+                        AND post_id = '".(int) $post_id."'";
             $tmp = Database::query($tmp);
             $tmp = Database::fetch_array($tmp);
             $parent_cat = $tmp['parent_comment_id'];
@@ -1246,22 +1246,22 @@ 
             }
 
             if (!is_null($comment['task_id'])) {
-                $border_color = ' border-left: 3px solid #' . $comment['color'];
+                $border_color = ' border-left: 3px solid #'.$comment['color'];
             }
 
             $comment_text = stripslashes($comment_text);
 
             // Output...
             $margin = $current_level * 30;
-            echo '<div class="blogpost_comment" style="margin-left: ' . $margin . 'px;' . $border_color . '">';
-                echo '<span class="blogpost_comment_title"><a href="#add_comment" onclick="document.getElementById(\'comment_parent_id\').value=\'' . $comment['comment_id'] . '\'; document.getElementById(\'comment_title\').value=\'Re: '.addslashes($comment['title']) . '\'" title="' . get_lang('ReplyToThisComment') . '" >'.stripslashes($comment['title']) . '</a></span>';
-                echo '<span class="blogpost_comment_date">' . $blog_comment_date . '</span>';
-                echo '<span class="blogpost_text">' . $comment_text . '</span>';
+            echo '<div class="blogpost_comment" style="margin-left: '.$margin.'px;'.$border_color.'">';
+                echo '<span class="blogpost_comment_title"><a href="#add_comment" onclick="document.getElementById(\'comment_parent_id\').value=\''.$comment['comment_id'].'\'; document.getElementById(\'comment_title\').value=\'Re: '.addslashes($comment['title']).'\'" title="'.get_lang('ReplyToThisComment').'" >'.stripslashes($comment['title']).'</a></span>';
+                echo '<span class="blogpost_comment_date">'.$blog_comment_date.'</span>';
+                echo '<span class="blogpost_text">'.$comment_text.'</span>';
 
-                $file_name_array = get_blog_attachment($blog_id,$post_id, $comment['comment_id']);
+                $file_name_array = get_blog_attachment($blog_id, $post_id, $comment['comment_id']);
                 if (!empty($file_name_array)) {
                     echo '<br /><br />';
-                    echo Display::return_icon('attachment.gif',get_lang('Attachment'));
+                    echo Display::return_icon('attachment.gif', get_lang('Attachment'));
                     echo '<a href="download.php?file=';
                     echo $file_name_array['path'];
                     echo ' "> '.$file_name_array['filename'].' </a>';
@@ -1271,7 +1271,7 @@ 
                 }
                 $username = api_htmlentities(sprintf(get_lang('LoginX'), $comment['username']), ENT_QUOTES);
                 echo '<span class="blogpost_comment_info">'.get_lang('Author').': '.Display::tag('span', api_get_person_name($comment['firstname'], $comment['lastname']), array('title'=>$username)).' - '.get_lang('Rating').': '.Blog::display_rating('comment', $blog_id, $comment['comment_id']).$rating_select.'</span>';
-                echo '<span class="blogpost_actions">' . $blog_comment_actions . '</span>';
+                echo '<span class="blogpost_actions">'.$blog_comment_actions.'</span>';
             echo '</div>';
 
             // Go further down the tree.
@@ -1287,11 +1287,11 @@ 
      */
     public static function display_form_new_post($blog_id)
     {
-        if (api_is_allowed('BLOG_' . $blog_id, 'article_add')) {
+        if (api_is_allowed('BLOG_'.$blog_id, 'article_add')) {
             $form = new FormValidator(
                 'add_post',
                 'post',
-                api_get_path(WEB_CODE_PATH)."blog/blog.php?action=new_post&blog_id=" . $blog_id . "&" . api_get_cidreq(),
+                api_get_path(WEB_CODE_PATH)."blog/blog.php?action=new_post&blog_id=".$blog_id."&".api_get_cidreq(),
                 null,
                 array('enctype' => 'multipart/form-data')
             );
@@ -1335,8 +1335,8 @@ 
                 INNER JOIN $tbl_users user ON post.author_id = user.user_id
                 WHERE
                 post.c_id 			= $course_id AND
-                post.blog_id 		= '".(int)$blog_id ."'
-                AND post.post_id	= '".(int)$post_id."'
+                post.blog_id 		= '".(int) $blog_id."'
+                AND post.post_id	= '".(int) $post_id."'
                 ORDER BY post_id DESC";
         $result = Database::query($sql);
         $blog_post = Database::fetch_array($result);
@@ -1345,7 +1345,7 @@ 
         $form = new FormValidator(
             'edit_post',
             'post',
-            api_get_path(WEB_CODE_PATH).'blog/blog.php?action=edit_post&post_id=' . intval($_GET['post_id']) . '&blog_id=' . intval($blog_id) . '&article_id='.intval($_GET['article_id']).'&task_id='.intval($_GET['task_id'])
+            api_get_path(WEB_CODE_PATH).'blog/blog.php?action=edit_post&post_id='.intval($_GET['post_id']).'&blog_id='.intval($blog_id).'&article_id='.intval($_GET['article_id']).'&task_id='.intval($_GET['task_id'])
         );
 
         $form->addHeader(get_lang('EditPost'));
@@ -1377,31 +1377,31 @@ 
         global $charset;
         $course_id = api_get_course_int_id();
 
-        if (api_is_allowed('BLOG_' . $blog_id, 'article_add')) {
+        if (api_is_allowed('BLOG_'.$blog_id, 'article_add')) {
             $tbl_blogs_tasks = Database::get_course_table(TABLE_BLOGS_TASKS);
             $counter = 0;
             global $color2;
 
             echo '<div class="actions">';
-            echo '<a href="' .api_get_self(). '?action=manage_tasks&blog_id=' . $blog_id . '&do=add">';
+            echo '<a href="'.api_get_self().'?action=manage_tasks&blog_id='.$blog_id.'&do=add">';
             echo Display::return_icon('blog_newtasks.gif', get_lang('AddTasks'));
-            echo get_lang('AddTasks') . '</a> ';
-            echo '<a href="' .api_get_self(). '?action=manage_tasks&blog_id=' . $blog_id . '&do=assign">';
+            echo get_lang('AddTasks').'</a> ';
+            echo '<a href="'.api_get_self().'?action=manage_tasks&blog_id='.$blog_id.'&do=assign">';
             echo Display::return_icon('blog_task.gif', get_lang('AssignTasks'));
-            echo get_lang('AssignTasks') . '</a>';
+            echo get_lang('AssignTasks').'</a>';
             ?>
                 <a href="<?php echo api_get_self(); ?>?action=manage_rights&blog_id=<?php echo $blog_id ?>" title="<?php echo get_lang('ManageRights') ?>">
-                    <?php echo Display::return_icon('blog_admin_users.png', get_lang('RightsManager'),'',ICON_SIZE_SMALL). get_lang('RightsManager') ?></a>
+                    <?php echo Display::return_icon('blog_admin_users.png', get_lang('RightsManager'), '', ICON_SIZE_SMALL).get_lang('RightsManager') ?></a>
             <?php
             echo '</div>';
 
-            echo '<span class="blogpost_title">' . get_lang('TaskList') . '</span><br />';
+            echo '<span class="blogpost_title">'.get_lang('TaskList').'</span><br />';
             echo "<table class=\"data_table\">";
             echo	"<tr bgcolor=\"$color2\" align=\"center\" valign=\"top\">",
-                     "<th width='240'><b>",get_lang('Title'),"</b></th>",
-                     "<th><b>",get_lang('Description'),"</b></th>",
-                     "<th><b>",get_lang('Color'),"</b></th>",
-                     "<th width='50'><b>",get_lang('Modify'),"</b></th>",
+                     "<th width='240'><b>", get_lang('Title'), "</b></th>",
+                     "<th><b>", get_lang('Description'), "</b></th>",
+                     "<th><b>", get_lang('Color'), "</b></th>",
+                     "<th width='50'><b>", get_lang('Modify'), "</b></th>",
                 "</tr>";
 
 
@@ -1413,8 +1413,8 @@ 
                         description,
                         color,
                         system_task
-                    FROM " . $tbl_blogs_tasks . "
-                    WHERE c_id = $course_id AND blog_id = " . (int)$blog_id . "
+                    FROM " . $tbl_blogs_tasks."
+                    WHERE c_id = $course_id AND blog_id = ".(int) $blog_id."
                     ORDER BY system_task, title";
             $result = Database::query($sql);
 
@@ -1423,10 +1423,10 @@ 
                 $css_class = (($counter % 2) == 0) ? "row_odd" : "row_even";
                 $delete_icon = ($task['system_task'] == '1') ? "delete_na.png" : "delete.png";
                 $delete_title = ($task['system_task'] == '1') ? get_lang('DeleteSystemTask') : get_lang('DeleteTask');
-                $delete_link = ($task['system_task'] == '1') ? '#' : api_get_self() . '?action=manage_tasks&blog_id=' . $task['blog_id'] . '&do=delete&task_id=' . $task['task_id'];
-                $delete_confirm = ($task['system_task'] == '1') ? '' : 'onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"),ENT_QUOTES,$charset)). '\')) return false;"';
+                $delete_link = ($task['system_task'] == '1') ? '#' : api_get_self().'?action=manage_tasks&blog_id='.$task['blog_id'].'&do=delete&task_id='.$task['task_id'];
+                $delete_confirm = ($task['system_task'] == '1') ? '' : 'onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"), ENT_QUOTES, $charset)).'\')) return false;"';
 
-                echo '<tr class="' . $css_class . '" valign="top">';
+                echo '<tr class="'.$css_class.'" valign="top">';
                 echo '<td width="240">'.Security::remove_XSS($task['title']).'</td>';
                 echo '<td>'.Security::remove_XSS($task['description']).'</td>';
                 echo '<td><span style="background-color: #'.$task['color'].'">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</span></td>';
@@ -1452,23 +1452,23 @@ 
      *
      * @param Integer $blog_id
      */
-    public static function display_assigned_task_list ($blog_id)
+    public static function display_assigned_task_list($blog_id)
     {
         // Init
         $tbl_users = Database::get_main_table(TABLE_MAIN_USER);
         $tbl_blogs_tasks = Database::get_course_table(TABLE_BLOGS_TASKS);
         $tbl_blogs_tasks_rel_user = Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER);
         $counter = 0;
-        global $charset,$color2;
+        global $charset, $color2;
 
-        echo '<span class="blogpost_title">' . get_lang('AssignedTasks') . '</span><br />';
+        echo '<span class="blogpost_title">'.get_lang('AssignedTasks').'</span><br />';
         echo "<table class=\"data_table\">";
         echo	"<tr bgcolor=\"$color2\" align=\"center\" valign=\"top\">",
-                 "<th width='240'><b>",get_lang('Member'),"</b></th>",
-                 "<th><b>",get_lang('Task'),"</b></th>",
-                 "<th><b>",get_lang('Description'),"</b></th>",
-                 "<th><b>",get_lang('TargetDate'),"</b></th>",
-                 "<th width='50'><b>",get_lang('Modify'),"</b></th>",
+                 "<th width='240'><b>", get_lang('Member'), "</b></th>",
+                 "<th><b>", get_lang('Task'), "</b></th>",
+                 "<th><b>", get_lang('Description'), "</b></th>",
+                 "<th><b>", get_lang('TargetDate'), "</b></th>",
+                 "<th width='50'><b>", get_lang('Modify'), "</b></th>",
             "</tr>";
 
         $course_id = api_get_course_int_id();
@@ -1480,17 +1480,17 @@ 
                 WHERE
                     task_rel_user.c_id = $course_id AND
                     task.c_id = $course_id AND
-                    task_rel_user.blog_id = '".(int)$blog_id."'
+                    task_rel_user.blog_id = '".(int) $blog_id."'
                 ORDER BY target_date ASC";
         $result = Database::query($sql);
 
         while ($assignment = Database::fetch_array($result)) {
             $counter++;
-            $css_class = (($counter % 2)==0) ? "row_odd" : "row_even";
+            $css_class = (($counter % 2) == 0) ? "row_odd" : "row_even";
             $delete_icon = ($assignment['system_task'] == '1') ? "delete_na.png" : "delete.png";
             $delete_title = ($assignment['system_task'] == '1') ? get_lang('DeleteSystemTask') : get_lang('DeleteTask');
-            $delete_link = ($assignment['system_task'] == '1') ? '#' : api_get_self() . '?action=manage_tasks&blog_id=' . $assignment['blog_id'] . '&do=delete&task_id=' . $assignment['task_id'];
-            $delete_confirm = ($assignment['system_task'] == '1') ? '' : 'onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"),ENT_QUOTES,$charset)). '\')) return false;"';
+            $delete_link = ($assignment['system_task'] == '1') ? '#' : api_get_self().'?action=manage_tasks&blog_id='.$assignment['blog_id'].'&do=delete&task_id='.$assignment['task_id'];
+            $delete_confirm = ($assignment['system_task'] == '1') ? '' : 'onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"), ENT_QUOTES, $charset)).'\')) return false;"';
 
             $username = api_htmlentities(sprintf(get_lang('LoginX'), $assignment['username']), ENT_QUOTES);
 
@@ -1522,7 +1522,7 @@ 
      * @author Toon Keppens
      *
      */
-    public static function display_new_task_form ($blog_id)
+    public static function display_new_task_form($blog_id)
     {
         // Init
         $colors = array(
@@ -1544,7 +1544,7 @@ 
         );
 
         // form
-        echo '<form name="add_task" method="post" action="blog.php?action=manage_tasks&blog_id=' . $blog_id . '">';
+        echo '<form name="add_task" method="post" action="blog.php?action=manage_tasks&blog_id='.$blog_id.'">';
 
         // form title
         echo '<legend>'.get_lang('AddTask').'</legend>';
@@ -1552,7 +1552,7 @@ 
         // task title
         echo '	<div class="control-group">
                     <label class="control-label">
-                        <span class="form_required">*</span>' . get_lang('Title') . '
+                        <span class="form_required">*</span>' . get_lang('Title').'
                     </label>
                     <div class="controls">
                         <input name="task_name" type="text" size="70" />
@@ -1562,7 +1562,7 @@ 
         // task comment
         echo '	<div class="control-group">
                     <label class="control-label">
-                        ' . get_lang('Description') . '
+                        ' . get_lang('Description').'
                     </label>
                     <div class="controls">
                         <textarea name="task_description" cols="45"></textarea>
@@ -1572,18 +1572,18 @@ 
         // task management
         echo '	<div class="control-group">
                     <label class="control-label">
-                        ' . get_lang('TaskManager') . '
+                        ' . get_lang('TaskManager').'
                     </label>
                     <div class="controls">';
                 echo '<table class="data_table" cellspacing="0" style="border-collapse:collapse; width:446px;">';
                     echo '<tr>';
-                        echo '<th colspan="2" style="width:223px;">' . get_lang('ArticleManager') . '</th>';
-                        echo '<th width:223px;>' . get_lang('CommentManager') . '</th>';
+                        echo '<th colspan="2" style="width:223px;">'.get_lang('ArticleManager').'</th>';
+                        echo '<th width:223px;>'.get_lang('CommentManager').'</th>';
                     echo '</tr>';
                     echo '<tr>';
-                        echo '<th style="width:111px;"><label for="articleDelete">' . get_lang('Delete') . '</label></th>';
-                        echo '<th style="width:112px;"><label for="articleEdit">' . get_lang('Edit') . '</label></th>';
-                        echo '<th style="width:223px;"><label for="commentsDelete">' . get_lang('Delete') . '</label></th>';
+                        echo '<th style="width:111px;"><label for="articleDelete">'.get_lang('Delete').'</label></th>';
+                        echo '<th style="width:112px;"><label for="articleEdit">'.get_lang('Edit').'</label></th>';
+                        echo '<th style="width:223px;"><label for="commentsDelete">'.get_lang('Delete').'</label></th>';
                     echo '</tr>';
                     echo '<tr>';
                         echo '<td style="text-align:center;"><input id="articleDelete" name="chkArticleDelete" type="checkbox" /></td>';
@@ -1598,13 +1598,13 @@ 
         // task color
         echo '	<div class="control-group">
                     <label class="control-label">
-                        ' . get_lang('Color') . '
+                        ' . get_lang('Color').'
                     </label>
                     <div class="controls">';
         echo '<select name="task_color" id="color" style="width: 150px; background-color: #eeeeee" onchange="document.getElementById(\'color\').style.backgroundColor=\'#\'+document.getElementById(\'color\').value" onkeypress="document.getElementById(\'color\').style.backgroundColor=\'#\'+document.getElementById(\'color\').value">';
                 foreach ($colors as $color) {
-                    $style = 'style="background-color: #' . $color . '"';
-                    echo '<option value="' . $color . '" ' . $style . '>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</option>';
+                    $style = 'style="background-color: #'.$color.'"';
+                    echo '<option value="'.$color.'" '.$style.'>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</option>';
                 }
         echo '</select>';
         echo '		</div>
@@ -1615,7 +1615,7 @@ 
                     <div class="controls">
                             <input type="hidden" name="action" value="" />
                             <input type="hidden" name="new_task_submit" value="true" />
-                        <button class="save" type="submit" name="Submit">' . get_lang('Save') . '</button>
+                        <button class="save" type="submit" name="Submit">' . get_lang('Save').'</button>
                     </div>
                 </div>';
         echo '</form>';
@@ -1629,34 +1629,34 @@ 
      * @author Toon Keppens
      *
      */
-    public static function display_edit_task_form ($blog_id, $task_id) {
+    public static function display_edit_task_form($blog_id, $task_id) {
         $tbl_blogs_tasks = Database::get_course_table(TABLE_BLOGS_TASKS);
         $course_id = api_get_course_int_id();
 
-        $colors = array('FFFFFF','FFFF99','FFCC99','FF9933','FF6699','CCFF99','CC9966','66FF00', '9966FF', 'CF3F3F', '990033','669933','0033FF','003366','000000');
+        $colors = array('FFFFFF', 'FFFF99', 'FFCC99', 'FF9933', 'FF6699', 'CCFF99', 'CC9966', '66FF00', '9966FF', 'CF3F3F', '990033', '669933', '0033FF', '003366', '000000');
 
-        $sql = "SELECT blog_id, task_id, title, description, color FROM $tbl_blogs_tasks WHERE c_id = $course_id AND task_id = '".(int)$task_id."'";
+        $sql = "SELECT blog_id, task_id, title, description, color FROM $tbl_blogs_tasks WHERE c_id = $course_id AND task_id = '".(int) $task_id."'";
         $result = Database::query($sql);
         $task = Database::fetch_array($result);
 
         // Display
-        echo '<form name="edit_task" method="post" action="blog.php?action=manage_tasks&blog_id=' . $blog_id . '">
-                    <legend>' . get_lang('EditTask') . '</legend>
+        echo '<form name="edit_task" method="post" action="blog.php?action=manage_tasks&blog_id='.$blog_id.'">
+                    <legend>' . get_lang('EditTask').'</legend>
                     <table width="100%" border="0" cellspacing="2">
                         <tr>
-                       <td align="right">' . get_lang('Title') . ':&nbsp;&nbsp;</td>
-                       <td><input name="task_name" type="text" size="70" value="'.Security::remove_XSS($task['title']) . '" /></td>
+                       <td align="right">' . get_lang('Title').':&nbsp;&nbsp;</td>
+                       <td><input name="task_name" type="text" size="70" value="'.Security::remove_XSS($task['title']).'" /></td>
                         </tr>
                         <tr>
-                       <td align="right">' . get_lang('Description') . ':&nbsp;&nbsp;</td>
+                       <td align="right">' . get_lang('Description').':&nbsp;&nbsp;</td>
                        <td><textarea name="task_description" cols="45">'.Security::remove_XSS($task['description']).'</textarea></td>
                         </tr>';
 
                         /* edit by Kevin Van Den Haute (kevin@develop-it.be) */
                         $tbl_tasks_permissions = Database::get_course_table(TABLE_BLOGS_TASKS_PERMISSIONS);
 
-                        $sql = " SELECT id, action FROM " . $tbl_tasks_permissions . "
-                                 WHERE c_id = $course_id AND task_id = '" . (int)$task_id."'";
+                        $sql = " SELECT id, action FROM ".$tbl_tasks_permissions."
+                                 WHERE c_id = $course_id AND task_id = '".(int) $task_id."'";
                         $result = Database::query($sql);
 
                         $arrPermissions = array();
@@ -1665,22 +1665,22 @@ 
                             $arrPermissions[] = $row['action'];
 
                             echo '<tr>';
-                            echo '<td style="text-align:right; vertical-align:top;">' . get_lang('TaskManager') . ':&nbsp;&nbsp;</td>';
+                            echo '<td style="text-align:right; vertical-align:top;">'.get_lang('TaskManager').':&nbsp;&nbsp;</td>';
                             echo '<td>';
                                 echo '<table  class="data_table" cellspacing="0" style="border-collapse:collapse; width:446px;">';
                                     echo '<tr>';
-                                        echo '<th colspan="2" style="width:223px;">' . get_lang('ArticleManager') . '</th>';
-                                        echo '<th width:223px;>' . get_lang('CommentManager') . '</th>';
+                                        echo '<th colspan="2" style="width:223px;">'.get_lang('ArticleManager').'</th>';
+                                        echo '<th width:223px;>'.get_lang('CommentManager').'</th>';
                                     echo '</tr>';
                                     echo '<tr>';
-                                        echo '<th style="width:111px;"><label for="articleDelete">' . get_lang('Delete') . '</label></th>';
-                                        echo '<th style="width:112px;"><label for="articleEdit">' . get_lang('Edit') . '</label></th>';
-                                        echo '<th style="width:223px;"><label for="commentsDelete">' . get_lang('Delete') . '</label></th>';
+                                        echo '<th style="width:111px;"><label for="articleDelete">'.get_lang('Delete').'</label></th>';
+                                        echo '<th style="width:112px;"><label for="articleEdit">'.get_lang('Edit').'</label></th>';
+                                        echo '<th style="width:223px;"><label for="commentsDelete">'.get_lang('Delete').'</label></th>';
                                     echo '</tr>';
                                     echo '<tr>';
-                                        echo '<td style="text-align:center;"><input ' . ((in_array('article_delete', $arrPermissions)) ? 'checked ' : '') . 'id="articleDelete" name="chkArticleDelete" type="checkbox" /></td>';
-                                        echo '<td style="text-align:center;"><input ' . ((in_array('article_edit', $arrPermissions)) ? 'checked ' : '') . 'id="articleEdit" name="chkArticleEdit" type="checkbox" /></td>';
-                                        echo '<td style="text-align:center;"><input ' . ((in_array('article_comments_delete', $arrPermissions)) ? 'checked ' : '') . 'id="commentsDelete" name="chkCommentsDelete" type="checkbox" /></td>';
+                                        echo '<td style="text-align:center;"><input '.((in_array('article_delete', $arrPermissions)) ? 'checked ' : '').'id="articleDelete" name="chkArticleDelete" type="checkbox" /></td>';
+                                        echo '<td style="text-align:center;"><input '.((in_array('article_edit', $arrPermissions)) ? 'checked ' : '').'id="articleEdit" name="chkArticleEdit" type="checkbox" /></td>';
+                                        echo '<td style="text-align:center;"><input '.((in_array('article_comments_delete', $arrPermissions)) ? 'checked ' : '').'id="commentsDelete" name="chkCommentsDelete" type="checkbox" /></td>';
                                     echo '</tr>';
                                 echo '</table>';
                             echo '</td>';
@@ -1688,13 +1688,13 @@ 
                         /* end of edit */
 
                         echo '<tr>
-                       <td align="right">' . get_lang('Color') . ':&nbsp;&nbsp;</td>
+                       <td align="right">' . get_lang('Color').':&nbsp;&nbsp;</td>
                        <td>
-                        <select name="task_color" id="color" style="width: 150px; background-color: #' . $task['color'] . '" onchange="document.getElementById(\'color\').style.backgroundColor=\'#\'+document.getElementById(\'color\').value" onkeypress="document.getElementById(\'color\').style.backgroundColor=\'#\'+document.getElementById(\'color\').value">';
+                        <select name="task_color" id="color" style="width: 150px; background-color: #' . $task['color'].'" onchange="document.getElementById(\'color\').style.backgroundColor=\'#\'+document.getElementById(\'color\').value" onkeypress="document.getElementById(\'color\').style.backgroundColor=\'#\'+document.getElementById(\'color\').value">';
                             foreach ($colors as $color) {
                                 $selected = ($color == $task['color']) ? ' selected' : '';
-                                $style = 'style="background-color: #' . $color . '"';
-                                echo '<option value="' . $color . '" ' . $style . ' ' . $selected . ' >&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</option>';
+                                $style = 'style="background-color: #'.$color.'"';
+                                echo '<option value="'.$color.'" '.$style.' '.$selected.' >&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</option>';
                             }
         echo '			   </select>
                           </td>
@@ -1703,9 +1703,9 @@ 
                             <td align="right">&nbsp;</td>
                             <td><br /><input type="hidden" name="action" value="" />
                             <input type="hidden" name="edit_task_submit" value="true" />
-                            <input type="hidden" name="task_id" value="' . $task['task_id'] . '" />
-                            <input type="hidden" name="blog_id" value="' . $task['blog_id'] . '" />
-                            <button class="save" type="submit" name="Submit">' . get_lang('Save') . '</button></td>
+                            <input type="hidden" name="task_id" value="' . $task['task_id'].'" />
+                            <input type="hidden" name="blog_id" value="' . $task['blog_id'].'" />
+                            <button class="save" type="submit" name="Submit">' . get_lang('Save').'</button></td>
                         </tr>
                     </table>
                 </form>';
@@ -1727,7 +1727,7 @@ 
                 FROM $tbl_users user
                 INNER JOIN $tbl_blogs_rel_user blogs_rel_user
                 ON user.user_id = blogs_rel_user.user_id
-                WHERE blogs_rel_user.c_id = $course_id AND blogs_rel_user.blog_id = '".(int)$blog_id."'";
+                WHERE blogs_rel_user.c_id = $course_id AND blogs_rel_user.blog_id = '".(int) $blog_id."'";
         $result = Database::query($sql);
 
         $options = array();
@@ -1746,7 +1746,7 @@ 
                 color,
                 system_task
             FROM $tbl_blogs_tasks
-            WHERE c_id = $course_id AND blog_id = " . (int)$blog_id . "
+            WHERE c_id = $course_id AND blog_id = ".(int) $blog_id."
             ORDER BY system_task, title";
         $result = Database::query($sql);
 
@@ -1794,7 +1794,7 @@ 
      */
     public static function display_edit_assigned_task_form($blog_id, $task_id, $user_id)
     {
-        $tbl_blogs_tasks_rel_user 	= Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER);
+        $tbl_blogs_tasks_rel_user = Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER);
 
         $course_id = api_get_course_int_id();
 
@@ -1803,9 +1803,9 @@ 
             SELECT target_date
             FROM $tbl_blogs_tasks_rel_user
             WHERE c_id = $course_id AND
-                  blog_id = '".(int)$blog_id."' AND
-                  user_id = '".(int)$user_id."' AND
-                  task_id = '".(int)$task_id."'";
+                  blog_id = '".(int) $blog_id."' AND
+                  user_id = '".(int) $user_id."' AND
+                  task_id = '".(int) $task_id."'";
         $result = Database::query($sql);
         $row = Database::fetch_assoc($result);
 
@@ -1840,11 +1840,11 @@ 
 
         $sql = "
             SELECT COUNT(*) as 'number'
-            FROM " . $tbl_blogs_tasks_rel_user . "
+            FROM " . $tbl_blogs_tasks_rel_user."
             WHERE c_id = $course_id AND
-            blog_id = " . (int)$blog_id . "
-            AND	user_id = " . (int)$user_id . "
-            AND	task_id = " . (int)$task_id . "
+            blog_id = ".(int) $blog_id."
+            AND	user_id = " . (int) $user_id."
+            AND	task_id = " . (int) $task_id."
         ";
 
         $result = Database::query($sql);
@@ -1852,18 +1852,18 @@ 
 
         if ($row['number'] == 0) {
             $sql = "
-                INSERT INTO " . $tbl_blogs_tasks_rel_user . " (
+                INSERT INTO " . $tbl_blogs_tasks_rel_user." (
                     c_id,
                     blog_id,
                     user_id,
                     task_id,
                     target_date
                 ) VALUES (
-                    '" . (int)$course_id . "',
-                    '" . (int)$blog_id . "',
-                    '" . (int)$user_id . "',
-                    '" . (int)$task_id . "',
-                    '" . Database::escape_string($target_date) . "'
+                    '" . (int) $course_id."',
+                    '" . (int) $blog_id."',
+                    '" . (int) $user_id."',
+                    '" . (int) $task_id."',
+                    '" . Database::escape_string($target_date)."'
                 )";
 
             Database::query($sql);
@@ -1893,12 +1893,12 @@ 
         $course_id = api_get_course_int_id();
 
         $sql = "SELECT COUNT(*) as 'number'
-                FROM " . $tbl_blogs_tasks_rel_user . "
+                FROM " . $tbl_blogs_tasks_rel_user."
                 WHERE
                     c_id = $course_id AND
-                    blog_id = " . (int)$blog_id . " AND
-                    user_id = " . (int)$user_id . " AND
-                    task_id = " . (int)$task_id . "
+                    blog_id = ".(int) $blog_id." AND
+                    user_id = " . (int) $user_id." AND
+                    task_id = " . (int) $task_id."
             ";
 
         $result = Database::query($sql);
@@ -1906,17 +1906,17 @@ 
 
         if ($row['number'] == 0 || ($row['number'] != 0 && $task_id == $old_task_id && $user_id == $old_user_id)) {
             $sql = "
-                UPDATE " . $tbl_blogs_tasks_rel_user . "
+                UPDATE " . $tbl_blogs_tasks_rel_user."
                 SET
-                    user_id = " . (int)$user_id . ",
-                    task_id = " . (int)$task_id . ",
-                    target_date = '" . Database::escape_string($target_date) . "'
+                    user_id = " . (int) $user_id.",
+                    task_id = " . (int) $task_id.",
+                    target_date = '" . Database::escape_string($target_date)."'
                 WHERE
                     c_id = $course_id AND
-                    blog_id = " . (int)$blog_id . " AND
-                    user_id = " . (int)$old_user_id . " AND
-                    task_id = " . (int)$old_task_id . " AND
-                    target_date = '" . Database::escape_string($old_target_date) . "'
+                    blog_id = ".(int) $blog_id." AND
+                    user_id = " . (int) $old_user_id." AND
+                    task_id = " . (int) $old_task_id." AND
+                    target_date = '" . Database::escape_string($old_target_date)."'
             ";
             Database::query($sql);
         }
@@ -1937,7 +1937,7 @@ 
 
 
         $sql = "SELECT title, description FROM $tbl_blogs_tasks
-                WHERE task_id = '".(int)$task_id."'
+                WHERE task_id = '".(int) $task_id."'
                 AND c_id = $course_id";
         $result = Database::query($sql);
         $row = Database::fetch_assoc($result);
@@ -1945,19 +1945,19 @@ 
         $sql = "SELECT post.*, user.lastname, user.firstname, user.username
                 FROM $tbl_blogs_posts post
                 INNER JOIN $tbl_users user ON post.author_id = user.user_id
-                WHERE post.blog_id = '".(int)$blog_id."' AND post.c_id = $course_id
+                WHERE post.blog_id = '".(int) $blog_id."' AND post.c_id = $course_id
                 ORDER BY post_id DESC
                 LIMIT 0, 100";
         $result = Database::query($sql);
 
         // Display
-        echo '<span class="blogpost_title">' . get_lang('SelectTaskArticle') . ' "' . stripslashes($row['title']) . '"</span>';
-        echo '<span style="font-style: italic;"">'.stripslashes($row['description']) . '</span><br><br>';
+        echo '<span class="blogpost_title">'.get_lang('SelectTaskArticle').' "'.stripslashes($row['title']).'"</span>';
+        echo '<span style="font-style: italic;"">'.stripslashes($row['description']).'</span><br><br>';
 
         if (Database::num_rows($result) > 0) {
-            while($blog_post = Database::fetch_array($result)) {
+            while ($blog_post = Database::fetch_array($result)) {
                 $username = api_htmlentities(sprintf(get_lang('LoginX'), $blog_post['username']), ENT_QUOTES);
-                echo '<a href="blog.php?action=execute_task&blog_id=' . $blog_id . '&task_id=' . $task_id . '&post_id=' . $blog_post['post_id'] . '#add_comment">'.stripslashes($blog_post['title']) . '</a>, ' . get_lang('WrittenBy') . ' ' . stripslashes(Display::tag('span', api_get_person_name($blog_post['firstname'], $blog_post['lastname']), array('title'=>$username))) . '<br />';
+                echo '<a href="blog.php?action=execute_task&blog_id='.$blog_id.'&task_id='.$task_id.'&post_id='.$blog_post['post_id'].'#add_comment">'.stripslashes($blog_post['title']).'</a>, '.get_lang('WrittenBy').' '.stripslashes(Display::tag('span', api_get_person_name($blog_post['firstname'], $blog_post['lastname']), array('title'=>$username))).'<br />';
             }
         } else {
             echo get_lang('NoArticles');
@@ -1981,12 +1981,12 @@ 
 
         // Subscribe the user
         $sql = "INSERT INTO $tbl_blogs_rel_user (c_id, blog_id, user_id )
-                VALUES ($course_id, '".(int)$blog_id."', '".(int)$user_id."');";
+                VALUES ($course_id, '".(int) $blog_id."', '".(int) $user_id."');";
         Database::query($sql);
 
         // Give this user basic rights
         $sql = "INSERT INTO $tbl_user_permissions (c_id, user_id,tool,action)
-                VALUES ($course_id, '".(int)$user_id."','BLOG_" . (int)$blog_id."','article_add')";
+                VALUES ($course_id, '".(int) $user_id."','BLOG_".(int) $blog_id."','article_add')";
         Database::query($sql);
 
         $id = Database::insert_id();
@@ -1996,7 +1996,7 @@ 
         }
 
         $sql = "INSERT INTO $tbl_user_permissions (c_id, user_id,tool,action)
-                VALUES ($course_id, '".(int)$user_id."','BLOG_" . (int)$blog_id."','article_comments_add')";
+                VALUES ($course_id, '".(int) $user_id."','BLOG_".(int) $blog_id."','article_comments_add')";
         Database::query($sql);
 
         $id = Database::insert_id();
@@ -2022,12 +2022,12 @@ 
 
         // Unsubscribe the user
         $sql = "DELETE FROM $tbl_blogs_rel_user
-                WHERE blog_id = '".(int)$blog_id."' AND user_id = '".(int)$user_id."'";
+                WHERE blog_id = '".(int) $blog_id."' AND user_id = '".(int) $user_id."'";
         Database::query($sql);
 
         // Remove this user's permissions.
         $sql = "DELETE FROM $tbl_user_permissions
-                WHERE user_id = '".(int)$user_id."'";
+                WHERE user_id = '".(int) $user_id."'";
         Database::query($sql);
     }
 
@@ -2048,7 +2048,7 @@ 
         $course_id = $_course['real_id'];
 
         $currentCourse = $_course['code'];
-        $tbl_users 			= Database::get_main_table(TABLE_MAIN_USER);
+        $tbl_users = Database::get_main_table(TABLE_MAIN_USER);
         $tbl_blogs_rel_user = Database::get_course_table(TABLE_BLOGS_REL_USER);
 
         echo '<legend>'.get_lang('SubscribeMembers').'</legend>';
@@ -2063,12 +2063,12 @@ 
         $result = Database::query($sql);
 
         $blog_member_ids = array();
-        while($user = Database::fetch_array($result)) {
+        while ($user = Database::fetch_array($result)) {
             $blog_member_ids[] = $user['user_id'];
         }
 
         // Set table headers
-        $column_header[] = array ('', false, '');
+        $column_header[] = array('', false, '');
         if ($is_western_name_order) {
             $column_header[] = array(get_lang('FirstName'), true, '');
             $column_header[] = array(get_lang('LastName'), true, '');
@@ -2088,13 +2088,13 @@ 
 
         // Add users that are not in this blog to the list.
         foreach ($student_list as $key=>$user) {
-            if(isset($user['id_user'])) {
+            if (isset($user['id_user'])) {
                 $user['user_id'] = $user['id_user'];
             }
-            if(!in_array($user['user_id'],$blog_member_ids)) {
+            if (!in_array($user['user_id'], $blog_member_ids)) {
                 $a_infosUser = api_get_user_info($user['user_id']);
-                $row = array ();
-                $row[] = '<input type="checkbox" name="user[]" value="' . $a_infosUser['user_id'] . '" '.((isset($_GET['selectall']) && $_GET['selectall'] == "subscribe") ? ' checked="checked" ' : '') . '/>';
+                $row = array();
+                $row[] = '<input type="checkbox" name="user[]" value="'.$a_infosUser['user_id'].'" '.((isset($_GET['selectall']) && $_GET['selectall'] == "subscribe") ? ' checked="checked" ' : '').'/>';
                 $username = api_htmlentities(sprintf(get_lang('LoginX'), $a_infosUser["username"]), ENT_QUOTES);
                 if ($is_western_name_order) {
                     $row[] = $a_infosUser["firstname"];
@@ -2106,8 +2106,8 @@ 
                 $row[] = Display::icon_mailto_link($a_infosUser["email"]);
 
                 //Link to register users
-                if ($a_infosUser["user_id"] != $_SESSION['_user']['user_id']){
-                    $row[] = "<a class=\"btn btn-primary \" href=\"" .api_get_self()."?action=manage_members&blog_id=$blog_id&register=yes&user_id=" . $a_infosUser["user_id"]."\">" . get_lang('Register')."</a>";
+                if ($a_infosUser["user_id"] != $_SESSION['_user']['user_id']) {
+                    $row[] = "<a class=\"btn btn-primary \" href=\"".api_get_self()."?action=manage_members&blog_id=$blog_id&register=yes&user_id=".$a_infosUser["user_id"]."\">".get_lang('Register')."</a>";
                 } else {
                     $row[] = '';
                 }
@@ -2118,20 +2118,20 @@ 
         // Display
         $query_vars['action'] = 'manage_members';
         $query_vars['blog_id'] = $blog_id;
-        echo '<form method="post" action="blog.php?action=manage_members&blog_id=' . $blog_id . '">';
-            Display::display_sortable_table($column_header, $user_data,null,null,$query_vars);
+        echo '<form method="post" action="blog.php?action=manage_members&blog_id='.$blog_id.'">';
+            Display::display_sortable_table($column_header, $user_data, null, null, $query_vars);
             $link = '';
-            $link .= isset ($_GET['action']) ? 'action=' . Security::remove_XSS($_GET['action']) . '&' : '';
+            $link .= isset ($_GET['action']) ? 'action='.Security::remove_XSS($_GET['action']).'&' : '';
             $link .= "blog_id=$blog_id&";
 
-            echo '<a href="blog.php?' . $link . 'selectall=subscribe">' . get_lang('SelectAll') . '</a> - ';
-            echo '<a href="blog.php?' . $link . '">' . get_lang('UnSelectAll') . '</a> ';
-            echo get_lang('WithSelected') . ' : ';
+            echo '<a href="blog.php?'.$link.'selectall=subscribe">'.get_lang('SelectAll').'</a> - ';
+            echo '<a href="blog.php?'.$link.'">'.get_lang('UnSelectAll').'</a> ';
+            echo get_lang('WithSelected').' : ';
             echo '<select name="action">';
-            echo '<option value="select_subscribe">' . get_lang('Register') . '</option>';
+            echo '<option value="select_subscribe">'.get_lang('Register').'</option>';
             echo '</select>';
             echo '<input type="hidden" name="register" value="true" />';
-            echo '<button class="save" type="submit">' . get_lang('Ok') . '</button>';
+            echo '<button class="save" type="submit">'.get_lang('Ok').'</button>';
         echo '</form>';
     }
 
@@ -2144,7 +2144,7 @@ 
      *
      * @return false|null Form with sortable table with users to unsubcribe from a blog.
      */
-    public static function display_form_user_unsubscribe ($blog_id)
+    public static function display_form_user_unsubscribe($blog_id)
     {
         $_user = api_get_user_info();
         $is_western_name_order = api_is_western_name_order();
@@ -2157,34 +2157,34 @@ 
 
         $properties["width"] = "100%";
         //table column titles
-        $column_header[] = array ('', false, '');
+        $column_header[] = array('', false, '');
         if ($is_western_name_order) {
-            $column_header[] = array (get_lang('FirstName'), true, '');
-            $column_header[] = array (get_lang('LastName'), true, '');
+            $column_header[] = array(get_lang('FirstName'), true, '');
+            $column_header[] = array(get_lang('LastName'), true, '');
         } else {
-            $column_header[] = array (get_lang('LastName'), true, '');
-            $column_header[] = array (get_lang('FirstName'), true, '');
+            $column_header[] = array(get_lang('LastName'), true, '');
+            $column_header[] = array(get_lang('FirstName'), true, '');
         }
-        $column_header[] = array (get_lang('Email'), false, '');
-        $column_header[] = array (get_lang('TaskManager'), true, '');
-        $column_header[] = array (get_lang('UnRegister'), false, '');
+        $column_header[] = array(get_lang('Email'), false, '');
+        $column_header[] = array(get_lang('TaskManager'), true, '');
+        $column_header[] = array(get_lang('UnRegister'), false, '');
 
         $course_id = api_get_course_int_id();
 
         $sql = "SELECT user.user_id, user.lastname, user.firstname, user.email, user.username
                 FROM $tbl_users user INNER JOIN $tbl_blogs_rel_user blogs_rel_user
                 ON user.user_id = blogs_rel_user.user_id
-                WHERE blogs_rel_user.c_id = $course_id AND  blogs_rel_user.blog_id = '".(int)$blog_id."'";
+                WHERE blogs_rel_user.c_id = $course_id AND  blogs_rel_user.blog_id = '".(int) $blog_id."'";
 
         if (!($sql_result = Database::query($sql))) {
             return false;
         }
 
-        $user_data = array ();
+        $user_data = array();
 
         while ($myrow = Database::fetch_array($sql_result)) {
-            $row = array ();
-            $row[] = '<input type="checkbox" name="user[]" value="' . $myrow['user_id'] . '" '.((isset($_GET['selectall']) && $_GET['selectall'] == "unsubscribe") ? ' checked="checked" ' : '') . '/>';
+            $row = array();
+            $row[] = '<input type="checkbox" name="user[]" value="'.$myrow['user_id'].'" '.((isset($_GET['selectall']) && $_GET['selectall'] == "unsubscribe") ? ' checked="checked" ' : '').'/>';
             $username = api_htmlentities(sprintf(get_lang('LoginX'), $myrow["username"]), ENT_QUOTES);
             if ($is_western_name_order) {
                 $row[] = $myrow["firstname"];
@@ -2196,19 +2196,19 @@ 
             $row[] = Display::icon_mailto_link($myrow["email"]);
 
             $sql = "SELECT bt.title task
-                    FROM " . Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER) . " btu
-                    INNER JOIN " . Database::get_course_table(TABLE_BLOGS_TASKS) . " bt
+                    FROM " . Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER)." btu
+                    INNER JOIN " . Database::get_course_table(TABLE_BLOGS_TASKS)." bt
                     ON btu.task_id = bt.task_id
                     WHERE 	btu.c_id 	= $course_id  AND
                             bt.c_id 	= $course_id  AND
                             btu.blog_id = $blog_id AND
-                            btu.user_id = " . $myrow['user_id'];
+                            btu.user_id = ".$myrow['user_id'];
             $sql_res = Database::query($sql);
 
             $task = '';
 
-            while($r = Database::fetch_array($sql_res)) {
-                $task .= stripslashes($r['task']) . ', ';
+            while ($r = Database::fetch_array($sql_res)) {
+                $task .= stripslashes($r['task']).', ';
             }
             //echo $task;
             $task = (api_strlen(trim($task)) != 0) ? api_substr($task, 0, api_strlen($task) - 2) : get_lang('Reader');
@@ -2216,7 +2216,7 @@ 
             //Link to register users
 
             if ($myrow["user_id"] != $_user['user_id']) {
-                $row[] = "<a class=\"btn btn-primary\" href=\"" .api_get_self()."?action=manage_members&blog_id=$blog_id&unregister=yes&user_id=" . $myrow['user_id']."\">" . get_lang('UnRegister')."</a>";
+                $row[] = "<a class=\"btn btn-primary\" href=\"".api_get_self()."?action=manage_members&blog_id=$blog_id&unregister=yes&user_id=".$myrow['user_id']."\">".get_lang('UnRegister')."</a>";
             } else {
                 $row[] = '';
             }
@@ -2226,20 +2226,20 @@ 
 
         $query_vars['action'] = 'manage_members';
         $query_vars['blog_id'] = $blog_id;
-        echo '<form method="post" action="blog.php?action=manage_members&blog_id=' . $blog_id . '">';
-        Display::display_sortable_table($column_header, $user_data,null,null,$query_vars);
+        echo '<form method="post" action="blog.php?action=manage_members&blog_id='.$blog_id.'">';
+        Display::display_sortable_table($column_header, $user_data, null, null, $query_vars);
         $link = '';
-        $link .= isset ($_GET['action']) ? 'action=' . Security::remove_XSS($_GET['action']). '&' : '';
+        $link .= isset ($_GET['action']) ? 'action='.Security::remove_XSS($_GET['action']).'&' : '';
         $link .= "blog_id=$blog_id&";
 
-        echo '<a href="blog.php?' . $link . 'selectall=unsubscribe">' . get_lang('SelectAll') . '</a> - ';
-        echo '<a href="blog.php?' . $link . '">' . get_lang('UnSelectAll') . '</a> ';
-        echo get_lang('WithSelected') . ' : ';
+        echo '<a href="blog.php?'.$link.'selectall=unsubscribe">'.get_lang('SelectAll').'</a> - ';
+        echo '<a href="blog.php?'.$link.'">'.get_lang('UnSelectAll').'</a> ';
+        echo get_lang('WithSelected').' : ';
         echo '<select name="action">';
-        echo '<option value="select_unsubscribe">' . get_lang('UnRegister') . '</option>';
+        echo '<option value="select_unsubscribe">'.get_lang('UnRegister').'</option>';
         echo '</select>';
         echo '<input type="hidden" name="unregister" value="true" />';
-        echo '<button class="save" type="submit">' . get_lang('Ok') . '</button>';
+        echo '<button class="save" type="submit">'.get_lang('Ok').'</button>';
         echo '</form>';
     }
 
@@ -2249,7 +2249,7 @@ 
      *
      * @param Integer $blog_id
      */
-    public static function display_form_user_rights ($blog_id)
+    public static function display_form_user_rights($blog_id)
     {
         echo '<legend>'.get_lang('RightsManager').'</legend>';
         echo '<br />';
@@ -2270,7 +2270,7 @@ 
         $form = new FormValidator(
             'add_post',
             'post',
-            api_get_path(WEB_CODE_PATH)."blog/blog.php?action=view_post&blog_id=" . intval($blog_id)  . "&post_id=".intval($post_id)."&".api_get_cidreq(),
+            api_get_path(WEB_CODE_PATH)."blog/blog.php?action=view_post&blog_id=".intval($blog_id)."&post_id=".intval($post_id)."&".api_get_cidreq(),
             null,
             array('enctype' => 'multipart/form-data')
         );
@@ -2336,21 +2336,21 @@ 
         $course_id = api_get_course_int_id();
 
         //Handle leap year
-        $numberofdays = array (0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31);
+        $numberofdays = array(0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31);
 
-        if(($year % 400 == 0) or ($year % 4 == 0 and $year % 100 <> 0))
+        if (($year % 400 == 0) or ($year % 4 == 0 and $year % 100 <> 0))
             $numberofdays[2] = 29;
 
         //Get the first day of the month
         $dayone = getdate(mktime(0, 0, 0, $month, 1, $year));
-        $monthName = $MonthsLong[$month-1];
+        $monthName = $MonthsLong[$month - 1];
 
         //Start the week on monday
         $startdayofweek = $dayone['wday'] <> 0 ? ($dayone['wday'] - 1) : 6;
         $blogId = isset($_GET['blog_id']) ? intval($_GET['blog_id']) : null;
         $filter = isset($_GET['filter']) ? Security::remove_XSS($_GET['filter']) : null;
-        $backwardsURL = api_get_self()."?blog_id=" . $blogId."&filter=" . $filter."&month=". ($month == 1 ? 12 : $month -1)."&year=". ($month == 1 ? $year -1 : $year);
-        $forewardsURL = api_get_self()."?blog_id=" . $blogId."&filter=" . $filter."&month=". ($month == 12 ? 1 : $month +1)."&year=". ($month == 12 ? $year +1 : $year);
+        $backwardsURL = api_get_self()."?blog_id=".$blogId."&filter=".$filter."&month=".($month == 1 ? 12 : $month - 1)."&year=".($month == 1 ? $year - 1 : $year);
+        $forewardsURL = api_get_self()."?blog_id=".$blogId."&filter=".$filter."&month=".($month == 12 ? 1 : $month + 1)."&year=".($month == 12 ? $year + 1 : $year);
 
         // Get posts for this month
         $sql = "SELECT post.*, DAYOFMONTH(date_creation) as post_day, user.lastname, user.firstname
@@ -2359,15 +2359,15 @@ 
                 ON post.author_id = user.user_id
                 WHERE
                     post.c_id = $course_id AND
-                    post.blog_id = '".(int)$blog_id."' AND
-                    MONTH(date_creation) = '".(int)$month."' AND
-                    YEAR(date_creation) = '".(int)$year."'
+                    post.blog_id = '".(int) $blog_id."' AND
+                    MONTH(date_creation) = '".(int) $month."' AND
+                    YEAR(date_creation) = '".(int) $year."'
                 ORDER BY date_creation";
         $result = Database::query($sql);
 
         // We will create an array of days on which there are posts.
-        if( Database::num_rows($result) > 0) {
-            while($blog_post = Database::fetch_array($result)) {
+        if (Database::num_rows($result) > 0) {
+            while ($blog_post = Database::fetch_array($result)) {
                 // If the day of this post is not yet in the array, add it.
                 if (!in_array($blog_post['post_day'], $posts))
                     $posts[] = $blog_post['post_day'];
@@ -2384,9 +2384,9 @@ 
                     task_rel_user.c_id = $course_id AND
                     task.c_id = $course_id AND
                     blog.c_id = $course_id AND
-                    task_rel_user.user_id = '".(int)$_user['user_id']."' AND
-                    MONTH(target_date) = '".(int)$month."' AND
-                    YEAR(target_date) = '".(int)$year."'
+                    task_rel_user.user_id = '".(int) $_user['user_id']."' AND
+                    MONTH(target_date) = '".(int) $month."' AND
+                    YEAR(target_date) = '".(int) $year."'
                 ORDER BY target_date ASC";
             $result = Database::query($sql);
 
@@ -2409,7 +2409,7 @@ 
 
         echo "<tr>";
 
-        for($ii = 1; $ii < 8; $ii ++)
+        for ($ii = 1; $ii < 8; $ii++)
             echo "<td class=\"weekdays\">", $DaysShort[$ii % 7], "</td>";
 
         echo "</tr>";
@@ -2419,24 +2419,24 @@ 
 
         while ($curday <= $numberofdays[$month]) {
             echo "<tr>";
-            for ($ii = 0; $ii < 7; $ii ++) {
+            for ($ii = 0; $ii < 7; $ii++) {
                 if (($curday == -1) && ($ii == $startdayofweek))
                     $curday = 1;
 
                 if (($curday > 0) && ($curday <= $numberofdays[$month])) {
-                    $bgcolor = $ii < 5 ? $class="class=\"days_week\"" : $class="class=\"days_weekend\"";
+                    $bgcolor = $ii < 5 ? $class = "class=\"days_week\"" : $class = "class=\"days_weekend\"";
                     $dayheader = "$curday";
 
-                    if(($curday == $today['mday']) && ($year == $today['year']) && ($month == $today['mon'])) {
+                    if (($curday == $today['mday']) && ($year == $today['year']) && ($month == $today['mon'])) {
                         $dayheader = "$curday";
                         $class = "class=\"days_today\"";
                     }
 
-                    echo "<td " . $class.">";
+                    echo "<td ".$class.">";
 
                     // If there are posts on this day, create a filter link.
-                    if(in_array($curday, $posts))
-                        echo '<a href="blog.php?blog_id=' . $blog_id . '&filter=' . $year . '-' . $month . '-' . $curday . '&month=' . $month . '&year=' . $year . '" title="' . get_lang('ViewPostsOfThisDay') . '">' . $curday . '</a>';
+                    if (in_array($curday, $posts))
+                        echo '<a href="blog.php?blog_id='.$blog_id.'&filter='.$year.'-'.$month.'-'.$curday.'&month='.$month.'&year='.$year.'" title="'.get_lang('ViewPostsOfThisDay').'">'.$curday.'</a>';
                     else
                         echo $dayheader;
 
@@ -2444,7 +2444,7 @@ 
                         if (isset($tasks[$curday]) && is_array($tasks[$curday])) {
                             // Add tasks to calendar
                             foreach ($tasks[$curday] as $task) {
-                                echo '<a href="blog.php?action=execute_task&blog_id=' . $task['blog_id'] . '&task_id='.stripslashes($task['task_id']) . '" title="' . $task['title'] . ' : ' . get_lang('InBlog') . ' : ' . $task['blog_name'] . ' - ' . get_lang('ExecuteThisTask') . '">';
+                                echo '<a href="blog.php?action=execute_task&blog_id='.$task['blog_id'].'&task_id='.stripslashes($task['task_id']).'" title="'.$task['title'].' : '.get_lang('InBlog').' : '.$task['blog_name'].' - '.get_lang('ExecuteThisTask').'">';
                                 echo Display::return_icon('blog_task.gif', get_lang('ExecuteThisTask'));
                                 echo '</a>';
                             }
@@ -2452,7 +2452,7 @@ 
                     }
 
                     echo "</td>";
-                    $curday ++;
+                    $curday++;
                 } else
                     echo "<td>&nbsp;</td>";
             }
@@ -2490,7 +2490,7 @@ 
     public static function display_edit_blog_form($blog_id)
     {
         $course_id = api_get_course_int_id();
-        $blog_id= intval($blog_id);
+        $blog_id = intval($blog_id);
         $tbl_blogs = Database::get_course_table(TABLE_BLOGS);
 
         $sql = "SELECT blog_id, blog_name, blog_subtitle
@@ -2505,7 +2505,7 @@ 
             $blog['blog_subtitle'] = Security::remove_XSS($_POST['blog_subtitle']);
         }
 
-        $form = new FormValidator('edit_blog', 'post','blog_admin.php?action=edit&blog_id='.intval($_GET['blog_id']));
+        $form = new FormValidator('edit_blog', 'post', 'blog_admin.php?action=edit&blog_id='.intval($_GET['blog_id']));
         $form->addElement('header', get_lang('EditBlog'));
         $form->addElement('text', 'blog_name', get_lang('Title'));
         $form->addElement('textarea', 'blog_subtitle', get_lang('SubTitle'));
@@ -2541,8 +2541,8 @@ 
         $result = Database::query($sql);
         $list_info = array();
         if (Database::num_rows($result)) {
-            while ($row_project=Database::fetch_row($result)) {
-                $list_info[]=$row_project;
+            while ($row_project = Database::fetch_row($result)) {
+                $list_info[] = $row_project;
             }
         }
 
@@ -2554,35 +2554,35 @@ 
                 // Validation when belongs to a session
                 $session_img = api_get_session_image($info_log[4], $_user['status']);
 
-                $url_start_blog = 'blog.php' ."?". "blog_id=".$info_log[3]. "&".api_get_cidreq();
+                $url_start_blog = 'blog.php'."?"."blog_id=".$info_log[3]."&".api_get_cidreq();
                 $title = $info_log[0];
                         $image = Display::return_icon('blog.png', $title);
-                $list_name = '<div style="float: left; width: 35px; height: 22px;"><a href="'.$url_start_blog.'">' . $image . '</a></div><a href="'.$url_start_blog.'">' .$title. '</a>' . $session_img;
+                $list_name = '<div style="float: left; width: 35px; height: 22px;"><a href="'.$url_start_blog.'">'.$image.'</a></div><a href="'.$url_start_blog.'">'.$title.'</a>'.$session_img;
 
                 $list_body_blog[] = $list_name;
                 $list_body_blog[] = $info_log[1];
 
-                $visibility_icon=($info_log[2]==0) ? 'invisible' : 'visible';
-                $visibility_info=($info_log[2]==0) ? 'Visible' : 'Invisible';
-                $my_image = '<a href="' .api_get_self(). '?action=edit&blog_id=' . $info_log[3] . '">';
-                                $my_image.= Display::return_icon('edit.png', get_lang('EditBlog'));
+                $visibility_icon = ($info_log[2] == 0) ? 'invisible' : 'visible';
+                $visibility_info = ($info_log[2] == 0) ? 'Visible' : 'Invisible';
+                $my_image = '<a href="'.api_get_self().'?action=edit&blog_id='.$info_log[3].'">';
+                                $my_image .= Display::return_icon('edit.png', get_lang('EditBlog'));
 
-                $my_image.= "</a>";
-                $my_image.= '<a href="' .api_get_self(). '?action=delete&blog_id=' . $info_log[3] . '" ';
-                $my_image.= 'onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"),ENT_QUOTES,$charset)). '\')) return false;" >';
-                                $my_image.= Display::return_icon('delete.png', get_lang('DeleteBlog'));
+                $my_image .= "</a>";
+                $my_image .= '<a href="'.api_get_self().'?action=delete&blog_id='.$info_log[3].'" ';
+                $my_image .= 'onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"), ENT_QUOTES, $charset)).'\')) return false;" >';
+                                $my_image .= Display::return_icon('delete.png', get_lang('DeleteBlog'));
 
-                $my_image.= "</a>";
-                $my_image.= '<a href="' .api_get_self(). '?action=visibility&blog_id=' . $info_log[3] . '">';
-                                $my_image.= Display::return_icon($visibility_icon . '.gif', get_lang($visibility_info));
+                $my_image .= "</a>";
+                $my_image .= '<a href="'.api_get_self().'?action=visibility&blog_id='.$info_log[3].'">';
+                                $my_image .= Display::return_icon($visibility_icon.'.gif', get_lang($visibility_info));
 
-                $my_image.= "</a>";
-                $list_body_blog[]=$my_image;
-                $list_content_blog[]=$list_body_blog;
+                $my_image .= "</a>";
+                $list_body_blog[] = $my_image;
+                $list_content_blog[] = $list_body_blog;
                 $list_body_blog = array();
             }
 
-            $table = new SortableTableFromArrayConfig($list_content_blog, 1,20,'project');
+            $table = new SortableTableFromArrayConfig($list_content_blog, 1, 20, 'project');
             $table->set_header(0, get_lang('Title'));
             $table->set_header(1, get_lang('SubTitle'));
             $table->set_header(2, get_lang('Modify'));
@@ -2607,34 +2607,34 @@ 
  * @author Julio Montoya
  * @version avril 2008, dokeos 1.8.5
  */
-function get_blog_attachment($blog_id, $post_id=null,$comment_id=null)
+function get_blog_attachment($blog_id, $post_id = null, $comment_id = null)
 {
 	$blog_table_attachment = Database::get_course_table(TABLE_BLOGS_ATTACHMENT);
 
 	$blog_id = intval($blog_id);
 	$comment_id = intval($comment_id);
 	$post_id = intval($post_id);
-	$row=array();
-	$where='';
+	$row = array();
+	$where = '';
 	if (!empty ($post_id) && is_numeric($post_id)) {
-		$where.=' AND post_id ="'.$post_id.'" ';
+		$where .= ' AND post_id ="'.$post_id.'" ';
 	}
 
 	if (!empty ($comment_id) && is_numeric($comment_id)) {
 		if (!empty ($post_id)) {
-			$where.= ' AND ';
+			$where .= ' AND ';
 		}
-		$where.=' comment_id ="'.$comment_id.'" ';
+		$where .= ' comment_id ="'.$comment_id.'" ';
 	}
 
     $course_id = api_get_course_int_id();
 
-	$sql = 'SELECT path, filename, comment FROM '. $blog_table_attachment.'
+	$sql = 'SELECT path, filename, comment FROM '.$blog_table_attachment.'
 	        WHERE c_id = '.$course_id.' AND blog_id ="'.intval($blog_id).'"  '.$where;
 
-	$result=Database::query($sql);
-	if (Database::num_rows($result)!=0) {
-		$row=Database::fetch_array($result);
+	$result = Database::query($sql);
+	if (Database::num_rows($result) != 0) {
+		$row = Database::fetch_array($result);
 	}
 	return $row;
 }
@@ -2674,22 +2674,22 @@ 
     }
 
 	// delete all files in directory
-	$courseDir   = $_course['path'].'/upload/blog';
+	$courseDir = $_course['path'].'/upload/blog';
 	$sys_course_path = api_get_path(SYS_COURSE_PATH);
 	$updir = $sys_course_path.$courseDir;
 
 	$sql = 'SELECT path FROM '.$blog_table_attachment.'
 	        WHERE c_id = '.$course_id.' AND blog_id ="'.intval($blog_id).'"  '.$where;
-	$result=Database::query($sql);
+	$result = Database::query($sql);
 
-	while ($row=Database::fetch_row($result)) {
-		$file=$updir.'/'.$row[0];
-		if (Security::check_abs_path($file,$updir) )
+	while ($row = Database::fetch_row($result)) {
+		$file = $updir.'/'.$row[0];
+		if (Security::check_abs_path($file, $updir))
 		{
 			@ unlink($file);
 		}
 	}
-	$sql = 'DELETE FROM '. $blog_table_attachment.'
+	$sql = 'DELETE FROM '.$blog_table_attachment.'
 	        WHERE c_id = '.$course_id.' AND  blog_id ="'.intval($blog_id).'"  '.$where;
 	Database::query($sql);
 }
@@ -2719,13 +2719,13 @@ 
 	$result = Database::query($sql);
 	$return_data = '';
 
-	if (Database::num_rows($result)!=0) {
-		while ($row=Database::fetch_array($result)) {
-			$return_data.=  '<div class="clear"></div><br />';
-			$return_data.=  '<div class="actions" style="margin-left:5px;margin-right:5px;">'.Display::return_icon('blog_article.png',get_lang('BlogPosts')).' '.$row['title'].'&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<div style="float:right;margin-top:-18px"><a href="../blog/blog.php?blog_id='.$row['blog_id'].'&gidReq=&cidReq='.$my_course_id.' " >'.get_lang('SeeBlog').'</a></div></div>';
-			$return_data.=  '<br / >';
-			$return_data.= $row['full_text'];
-			$return_data.= '<br /><br />';
+	if (Database::num_rows($result) != 0) {
+		while ($row = Database::fetch_array($result)) {
+			$return_data .= '<div class="clear"></div><br />';
+			$return_data .= '<div class="actions" style="margin-left:5px;margin-right:5px;">'.Display::return_icon('blog_article.png', get_lang('BlogPosts')).' '.$row['title'].'&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<div style="float:right;margin-top:-18px"><a href="../blog/blog.php?blog_id='.$row['blog_id'].'&gidReq=&cidReq='.$my_course_id.' " >'.get_lang('SeeBlog').'</a></div></div>';
+			$return_data .= '<br / >';
+			$return_data .= $row['full_text'];
+			$return_data .= '<br /><br />';
 		}
 	}
 	return $return_data;
@@ -2755,13 +2755,13 @@ 
 			ORDER BY blog_name";
 	$result = Database::query($sql);
 	$return_data = '';
-	if (Database::num_rows($result)!=0) {
-		while ($row=Database::fetch_array($result)) {
-			$return_data.=  '<div class="clear"></div><br />';
-			$return_data.=  '<div class="actions" style="margin-left:5px;margin-right:5px;">'.$row['title'].'&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<div style="float:right;margin-top:-18px"><a href="../blog/blog.php?blog_id='.$row['blog_id'].'&gidReq=&cidReq='.Security::remove_XSS($course_code).' " >'.get_lang('SeeBlog').'</a></div></div>';
-			$return_data.=  '<br / >';
-			$return_data.=  $row['comment'];
-			$return_data.=  '<br />';
+	if (Database::num_rows($result) != 0) {
+		while ($row = Database::fetch_array($result)) {
+			$return_data .= '<div class="clear"></div><br />';
+			$return_data .= '<div class="actions" style="margin-left:5px;margin-right:5px;">'.$row['title'].'&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<div style="float:right;margin-top:-18px"><a href="../blog/blog.php?blog_id='.$row['blog_id'].'&gidReq=&cidReq='.Security::remove_XSS($course_code).' " >'.get_lang('SeeBlog').'</a></div></div>';
+			$return_data .= '<br / >';
+			$return_data .= $row['comment'];
+			$return_data .= '<br />';
 		}
 	}
 	return $return_data;

main/survey/survey.lib.php

Spacing +151 added lines, -152 removed lines

@@ -35,7 +35,7 @@ 
             $result = Database::query($sql);
             if (Database::num_rows($result)) {
                 $num++;
-                $new_code = $code . $num;
+                $new_code = $code.$num;
             } else {
                 break;
             }
@@ -63,7 +63,7 @@ 
         $sql = "SELECT survey_invitation_id, survey_code
                 FROM $table_survey_invitation WHERE user = '$user_id' AND c_id <> 0 ";
         $result = Database::query($sql);
-        while ($row = Database::fetch_array($result ,'ASSOC')){
+        while ($row = Database::fetch_array($result, 'ASSOC')) {
             $survey_invitation_id = $row['survey_invitation_id'];
             $survey_code = $row['survey_code'];
             $sql2 = "DELETE FROM $table_survey_invitation
@@ -129,7 +129,7 @@ 
         $table_survey = Database :: get_course_table(TABLE_SURVEY);
 
         if ($shared != 0) {
-            $table_survey	= Database :: get_main_table(TABLE_MAIN_SHARED_SURVEY_QUESTION);
+            $table_survey = Database :: get_main_table(TABLE_MAIN_SHARED_SURVEY_QUESTION);
             $sql = "SELECT * FROM $table_survey
                     WHERE survey_id='".intval($survey_id)."' ";
         } else {
@@ -142,8 +142,8 @@ 
         $result = Database::query($sql);
         $return = array();
 
-        if (Database::num_rows($result)> 0) {
-            $return = Database::fetch_array($result,'ASSOC');
+        if (Database::num_rows($result) > 0) {
+            $return = Database::fetch_array($result, 'ASSOC');
             if ($simple_return) {
                 return $return;
             }
@@ -187,7 +187,7 @@ 
         $course_id = api_get_course_int_id();
         $session_id = api_get_session_id();
         $courseCode = api_get_course_id();
-        $table_survey 	= Database :: get_course_table(TABLE_SURVEY);
+        $table_survey = Database :: get_course_table(TABLE_SURVEY);
         $shared_survey_id = 0;
 
         if (!isset($values['survey_id'])) {
@@ -233,7 +233,7 @@ 
                             if ($values[$field] == '') {
                                 $values[$field] = 0;
                             }
-                            $field_values.= $field.':'.$values[$field].'@';
+                            $field_values .= $field.':'.$values[$field].'@';
                         }
                     }
                     $extraParams['form_fields'] = $field_values;
@@ -385,7 +385,7 @@ 
             if ($values['anonymous'] == 0) {
                 $extraParams['show_form_profile'] = $values['show_form_profile'];
                 if ($values['show_form_profile'] == 1) {
-                    $fields = explode(',',$values['input_name_list']);
+                    $fields = explode(',', $values['input_name_list']);
                     $field_values = '';
                     foreach ($fields as &$field) {
                         if ($field != '') {
@@ -394,7 +394,7 @@ 
                             ) {
                                 $values[$field] = 0;
                             }
-                            $field_values.= $field.':'.$values[$field].'@';
+                            $field_values .= $field.':'.$values[$field].'@';
                         }
                     }
                     $extraParams['form_fields'] = $field_values;
@@ -526,7 +526,7 @@ 
         $_course = api_get_course_info();
 
         // Table definitions
-        $table_survey	= Database :: get_main_table(TABLE_MAIN_SHARED_SURVEY);
+        $table_survey = Database :: get_main_table(TABLE_MAIN_SHARED_SURVEY);
 
         if (!$values['survey_id'] ||
             !is_numeric($values['survey_id']) ||
@@ -544,7 +544,7 @@ 
                     '".api_get_utc_datetime()."',
                     '".$_course['id']."')";
             Database::query($sql);
-            $return	= Database::insert_id();
+            $return = Database::insert_id();
 
             $sql = "UPDATE $table_survey SET survey_id = $return WHERE iid = $return";
             Database::query($sql);
@@ -561,7 +561,7 @@ 
                         surveythanks	= '".Database::escape_string($values['survey_thanks'])."'
 					WHERE survey_id = '".Database::escape_string($values['survey_share']['survey_share'])."'";
             Database::query($sql);
-            $return	= $values['survey_share']['survey_share'];
+            $return = $values['survey_share']['survey_share'];
         }
 
         return $return;
@@ -659,7 +659,7 @@ 
             $params['c_id'] = $targetCourseId;
             unset($params['survey_id']);
             $params['session_id'] = api_get_session_id();
-            $params['title'] = $params['title'] . ' ' . get_lang('Copy');
+            $params['title'] = $params['title'].' '.get_lang('Copy');
             unset($params['iid']);
             Database::insert($table_survey, $params);
             $new_survey_id = Database::insert_id();
@@ -685,7 +685,7 @@ 
         $sql = "SELECT * FROM $table_survey_question_group
                 WHERE c_id = $course_id AND  survey_id='".$survey_id."'";
         $res = Database::query($sql);
-        while($row = Database::fetch_array($res, 'ASSOC')) {
+        while ($row = Database::fetch_array($res, 'ASSOC')) {
             $params = array(
                 'c_id' =>  $targetCourseId,
                 'name' => $row['name'],
@@ -732,7 +732,7 @@ 
                 WHERE c_id = $course_id AND survey_id='".$survey_id."'";
 
         $res = Database::query($sql);
-        while ($row = Database::fetch_array($res ,'ASSOC')) {
+        while ($row = Database::fetch_array($res, 'ASSOC')) {
             $params = array(
                 'c_id' =>  $targetCourseId,
                 'question_id' => $question_id[$row['question_id']],
@@ -897,7 +897,7 @@ 
     {
         // Table definitions
         $tbl_survey_question = Database :: get_course_table(TABLE_SURVEY_QUESTION);
-        $table_survey_question_option 	= Database :: get_course_table(TABLE_SURVEY_QUESTION_OPTION);
+        $table_survey_question_option = Database :: get_course_table(TABLE_SURVEY_QUESTION_OPTION);
         $course_id = api_get_course_int_id();
 
         $sql = "SELECT * FROM $tbl_survey_question
@@ -923,7 +923,7 @@ 
         // Getting the information of the question
 
         $result = Database::query($sql);
-        $row = Database::fetch_array($result,'ASSOC');
+        $row = Database::fetch_array($result, 'ASSOC');
 
         $return['survey_id'] = $row['survey_id'];
         $return['question_id'] = $row['question_id'];
@@ -971,8 +971,8 @@ 
     public static function get_questions($survey_id, $course_id = '')
     {
         // Table definitions
-        $tbl_survey_question 			= Database :: get_course_table(TABLE_SURVEY_QUESTION);
-        $table_survey_question_option 	= Database :: get_course_table(TABLE_SURVEY_QUESTION_OPTION);
+        $tbl_survey_question = Database :: get_course_table(TABLE_SURVEY_QUESTION);
+        $table_survey_question_option = Database :: get_course_table(TABLE_SURVEY_QUESTION_OPTION);
 
         if (empty($course_id)) {
             $course_id = api_get_course_int_id();
@@ -1076,7 +1076,7 @@ 
 					        FROM $tbl_survey_question
                             WHERE c_id = $course_id AND survey_id='".intval($form_content['survey_id'])."'";
                     $result = Database::query($sql);
-                    $row = Database::fetch_array($result,'ASSOC');
+                    $row = Database::fetch_array($result, 'ASSOC');
                     $max_sort = $row['max_sort'];
 
                     // Some variables defined for survey-test type
@@ -1213,7 +1213,7 @@ 
                     WHERE survey_id='".intval($survey_data['survey_share'])."'
                     AND code='".Database::escape_string($_course['id'])."'";
             $result = Database::query($sql);
-            $row = Database::fetch_array($result,'ASSOC');
+            $row = Database::fetch_array($result, 'ASSOC');
             $max_sort = $row['max_sort'];
 
             // Adding the question to the survey_question table
@@ -1223,11 +1223,11 @@ 
                     '".Database::escape_string($form_content['question_comment'])."',
                     '".Database::escape_string($form_content['type'])."',
                     '".Database::escape_string($form_content['horizontalvertical'])."',
-                    '".Database::escape_string($max_sort+1)."',
+                    '".Database::escape_string($max_sort + 1)."',
                     '".Database::escape_string($_course['id'])."')";
             Database::query($sql);
             $shared_question_id = Database::insert_id();
-        }  else {
+        } else {
             // Updating an existing question
             // adding the question to the survey_question table
             $sql = "UPDATE $tbl_survey_question SET
@@ -1257,7 +1257,7 @@ 
     public static function move_survey_question($direction, $survey_question_id, $survey_id)
     {
         // Table definition
-        $table_survey_question 	= Database :: get_course_table(TABLE_SURVEY_QUESTION);
+        $table_survey_question = Database :: get_course_table(TABLE_SURVEY_QUESTION);
         $course_id = api_get_course_int_id();
 
         if ($direction == 'moveup') {
@@ -1309,11 +1309,11 @@ 
         $course_id = api_get_course_int_id();
 
         // Table definitions
-        $table_survey_question 	= Database :: get_course_table(TABLE_SURVEY_QUESTION);
+        $table_survey_question = Database :: get_course_table(TABLE_SURVEY_QUESTION);
         $course_condition = " c_id = $course_id AND ";
         if ($shared) {
             $course_condition = "";
-            $table_survey_question 	= Database :: get_main_table(TABLE_MAIN_SHARED_SURVEY_QUESTION);
+            $table_survey_question = Database :: get_main_table(TABLE_MAIN_SHARED_SURVEY_QUESTION);
         }
 
         $sql = "DELETE FROM $table_survey_question
@@ -1346,7 +1346,7 @@ 
     {
         $course_id = api_get_course_int_id();
         // Table definitions
-        $table_survey_question 	= Database :: get_course_table(TABLE_SURVEY_QUESTION);
+        $table_survey_question = Database :: get_course_table(TABLE_SURVEY_QUESTION);
         if ($shared) {
             SurveyManager::delete_shared_survey_question($survey_id, $question_id);
         }
@@ -1408,7 +1408,7 @@ 
         $course_id = api_get_course_int_id();
         // A percentage question type has options 1 -> 100
         if ($form_content['type'] == 'percentage') {
-            for($i = 1; $i < 101; $i++) {
+            for ($i = 1; $i < 101; $i++) {
                 $form_content['answers'][] = $i;
             }
         }
@@ -1589,7 +1589,7 @@ 
     {
         $table_survey_answer = Database :: get_course_table(TABLE_SURVEY_ANSWER);
 
-        $user_id	= intval($user_id);
+        $user_id = intval($user_id);
         $course_id	= intval($course_id);
         $survey_id	= intval($survey_id);
 
@@ -1774,7 +1774,7 @@ 
     public static function remove_answer($user, $survey_id, $question_id, $course_id) {
         $course_id = intval($course_id);
         // table definition
-        $table_survey_answer 		= Database :: get_course_table(TABLE_SURVEY_ANSWER);
+        $table_survey_answer = Database :: get_course_table(TABLE_SURVEY_ANSWER);
         $sql = "DELETE FROM $table_survey_answer
 				WHERE
 				    c_id = $course_id AND
@@ -2009,19 +2009,19 @@ 
         // Actions bar
         echo '<div class="actions">';
         echo '<a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?survey_id='.$surveyId.'&'.api_get_cidreq().'">'.
-            Display::return_icon('back.png',get_lang('BackTo').' '.get_lang('ReportingOverview'),'',ICON_SIZE_MEDIUM).'</a>';
+            Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('ReportingOverview'), '', ICON_SIZE_MEDIUM).'</a>';
         if (isset($_GET['user'])) {
             if (api_is_allowed_to_edit()) {
                 // The delete link
                 echo '<a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?action=deleteuserreport&survey_id='.$surveyId.'&'.api_get_cidreq().'&user='.Security::remove_XSS($_GET['user']).'" >'.
-                    Display::return_icon('delete.png', get_lang('Delete'),'',ICON_SIZE_MEDIUM).'</a>';
+                    Display::return_icon('delete.png', get_lang('Delete'), '', ICON_SIZE_MEDIUM).'</a>';
             }
 
             // Export the user report
             echo '<a href="javascript: void(0);" onclick="document.form1a.submit();">'.
-                Display::return_icon('export_csv.png', get_lang('ExportAsCSV'),'',ICON_SIZE_MEDIUM).'</a> ';
+                Display::return_icon('export_csv.png', get_lang('ExportAsCSV'), '', ICON_SIZE_MEDIUM).'</a> ';
             echo '<a href="javascript: void(0);" onclick="document.form1b.submit();">'.
-                Display::return_icon('export_excel.png', get_lang('ExportAsXLS'),'',ICON_SIZE_MEDIUM).'</a> ';
+                Display::return_icon('export_excel.png', get_lang('ExportAsXLS'), '', ICON_SIZE_MEDIUM).'</a> ';
             echo '<form id="form1a" name="form1a" method="post" action="'.api_get_self().'?action='.Security::remove_XSS($_GET['action']).'&survey_id='.$surveyId.'&'.api_get_cidreq().'&user_id='.Security::remove_XSS($_GET['user']).'">';
             echo '<input type="hidden" name="export_report" value="export_report">';
             echo '<input type="hidden" name="export_format" value="csv">';
@@ -2054,7 +2054,7 @@ 
                     $name = $person['invited_user'];
                 }
             } else {
-                $name  = get_lang('Anonymous') . ' ' . ($key + 1);
+                $name = get_lang('Anonymous').' '.($key + 1);
                 $id = $person;
             }
             echo '<option value="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?action='.Security::remove_XSS($_GET['action']).'&survey_id='.Security::remove_XSS($_GET['survey_id']).'&user='.Security::remove_XSS($id).'" ';
@@ -2197,7 +2197,7 @@ 
 
         echo '<div class="actions">';
         echo '<a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?survey_id='.$surveyId.'">'.
-            Display::return_icon('back.png',get_lang('BackTo').' '.get_lang('ReportingOverview'),'',ICON_SIZE_MEDIUM).'</a>';
+            Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('ReportingOverview'), '', ICON_SIZE_MEDIUM).'</a>';
         echo '</div>';
 
         if ($survey_data['number_of_questions'] > 0) {
@@ -2205,18 +2205,18 @@ 
             if (!$singlePage) {
                 echo '<div id="question_report_questionnumbers" class="pagination">';
                 if ($currentQuestion != 0) {
-                    echo '<li><a href="' . api_get_path(WEB_CODE_PATH) . 'survey/reporting.php?action=' . $action . '&' . api_get_cidreq() . '&survey_id=' . $surveyId . '&question=' . ($offset - 1) . '">' . get_lang('PreviousQuestion') . '</a></li>';
+                    echo '<li><a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?action='.$action.'&'.api_get_cidreq().'&survey_id='.$surveyId.'&question='.($offset - 1).'">'.get_lang('PreviousQuestion').'</a></li>';
                 }
 
                 for ($i = 1; $i <= $survey_data['number_of_questions']; $i++) {
                     if ($offset != $i - 1) {
-                        echo '<li><a href="' . api_get_path(WEB_CODE_PATH) . 'survey/reporting.php?action=' . $action . '&' . api_get_cidreq() . '&survey_id=' . $surveyId . '&question=' . ($i - 1) . '">' . $i . '</a></li>';
+                        echo '<li><a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?action='.$action.'&'.api_get_cidreq().'&survey_id='.$surveyId.'&question='.($i - 1).'">'.$i.'</a></li>';
                     } else {
-                        echo '<li class="disabled"s><a href="#">' . $i . '</a></li>';
+                        echo '<li class="disabled"s><a href="#">'.$i.'</a></li>';
                     }
                 }
                 if ($currentQuestion < ($survey_data['number_of_questions'] - 1)) {
-                    echo '<li><a href="' . api_get_path(WEB_CODE_PATH) . 'survey/reporting.php?action=' . $action . '&' . api_get_cidreq() . '&survey_id=' . $surveyId . '&question=' . ($offset + 1) . '">' . get_lang('NextQuestion') . '</li></a>';
+                    echo '<li><a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?action='.$action.'&'.api_get_cidreq().'&survey_id='.$surveyId.'&question='.($offset + 1).'">'.get_lang('NextQuestion').'</li></a>';
                 }
                 echo '</ul>';
                 echo '</div>';
@@ -2254,19 +2254,19 @@ 
                 $sql = "SELECT * FROM $table_survey_answer
                         WHERE
                             c_id = $course_id AND
-                            survey_id='" . intval($_GET['survey_id']) . "' AND
-                            question_id = '" . intval($question['question_id']) . "'";
+                            survey_id='".intval($_GET['survey_id'])."' AND
+                            question_id = '" . intval($question['question_id'])."'";
                 $result = Database::query($sql);
                 while ($row = Database::fetch_array($result, 'ASSOC')) {
-                    echo $row['option_id'] . '<hr noshade="noshade" size="1" />';
+                    echo $row['option_id'].'<hr noshade="noshade" size="1" />';
                 }
             } else {
                 // Getting the options ORDER BY sort ASC
                 $sql = "SELECT * FROM $table_survey_question_option
                         WHERE
                             c_id = $course_id AND
-                            survey_id='" . intval($_GET['survey_id']) . "'
-                            AND question_id = '" . intval($question['question_id']) . "'
+                            survey_id='".intval($_GET['survey_id'])."'
+                            AND question_id = '" . intval($question['question_id'])."'
                         ORDER BY sort ASC";
                 $result = Database::query($sql);
                 while ($row = Database::fetch_array($result, 'ASSOC')) {
@@ -2276,8 +2276,8 @@ 
                 $sql = "SELECT *, count(answer_id) as total FROM $table_survey_answer
                         WHERE
                             c_id = $course_id AND
-                            survey_id='" . intval($_GET['survey_id']) . "'
-                            AND question_id = '" . intval($question['question_id']) . "'
+                            survey_id='".intval($_GET['survey_id'])."'
+                            AND question_id = '" . intval($question['question_id'])."'
                         GROUP BY option_id, value";
                 $result = Database::query($sql);
                 $number_of_answers = array();
@@ -2294,8 +2294,7 @@ 
                     $optionText = strip_tags($option['option_text']);
                     $optionText = html_entity_decode($optionText);
                     $votes = isset($data[$option['question_option_id']]['total']) ?
-                        $data[$option['question_option_id']]['total'] :
-                        '0';
+                        $data[$option['question_option_id']]['total'] : '0';
                     array_push($chartData, array('option' => $optionText, 'votes' => $votes));
                 }
                 $chartContainerId = 'chartContainer'.$question['question_id'];
@@ -2307,9 +2306,9 @@ 
                 echo '<table class="display-survey table">';
                 echo '	<tr>';
                 echo '		<th>&nbsp;</th>';
-                echo '		<th>' . get_lang('AbsoluteTotal') . '</th>';
-                echo '		<th>' . get_lang('Percentage') . '</th>';
-                echo '		<th>' . get_lang('VisualRepresentation') . '</th>';
+                echo '		<th>'.get_lang('AbsoluteTotal').'</th>';
+                echo '		<th>'.get_lang('Percentage').'</th>';
+                echo '		<th>'.get_lang('VisualRepresentation').'</th>';
                 echo '	<tr>';
 
                 // Displaying the table: the content
@@ -2329,22 +2328,22 @@ 
                             $answers_number = $absolute_number / $number_of_answers[$option['question_id']] * 100;
                         }
                         echo '	<tr>';
-                        echo '		<td class="center">' . $value['option_text'] . '</td>';
+                        echo '		<td class="center">'.$value['option_text'].'</td>';
                         echo '		<td class="center">';
                         if ($absolute_number != 0) {
-                            echo '<a href="' . api_get_path(WEB_CODE_PATH) . 'survey/reporting.php?action=' . $action . '&survey_id=' . $surveyId . '&question=' . $offset . '&viewoption=' . $value['question_option_id'] . '">' . $absolute_number . '</a>';
+                            echo '<a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?action='.$action.'&survey_id='.$surveyId.'&question='.$offset.'&viewoption='.$value['question_option_id'].'">'.$absolute_number.'</a>';
                         } else {
                             echo '0';
                         }
 
                         echo '      </td>';
-                        echo '		<td class="center">' . round($answers_number, 2) . ' %</td>';
+                        echo '		<td class="center">'.round($answers_number, 2).' %</td>';
                         echo '		<td class="center">';
                         $size = $answers_number * 2;
                         if ($size > 0) {
-                            echo '<div style="border:1px solid #264269; background-color:#aecaf4; height:10px; width:' . $size . 'px">&nbsp;</div>';
+                            echo '<div style="border:1px solid #264269; background-color:#aecaf4; height:10px; width:'.$size.'px">&nbsp;</div>';
                         } else {
-                            echo '<div style="text-align: left;">' . get_lang("NoDataAvailable") . '</div>';
+                            echo '<div style="text-align: left;">'.get_lang("NoDataAvailable").'</div>';
                         }
                         echo ' </td>';
                         echo ' </tr>';
@@ -2352,8 +2351,8 @@ 
                 }
                 // displaying the table: footer (totals)
                 echo '	<tr>';
-                echo '		<td class="total"><b>' . get_lang('Total') . '</b></td>';
-                echo '		<td class="total"><b>' . ($number_of_answers[$option['question_id']] == 0 ? '0' : $number_of_answers[$option['question_id']]) . '</b></td>';
+                echo '		<td class="total"><b>'.get_lang('Total').'</b></td>';
+                echo '		<td class="total"><b>'.($number_of_answers[$option['question_id']] == 0 ? '0' : $number_of_answers[$option['question_id']]).'</b></td>';
                 echo '		<td class="total">&nbsp;</td>';
                 echo '		<td class="total">&nbsp;</td>';
                 echo '	</tr>';
@@ -2397,8 +2396,8 @@ 
     public static function display_question_report_score($survey_data, $question, $offset)
     {
         // Database table definitions
-        $table_survey_question_option 	= Database :: get_course_table(TABLE_SURVEY_QUESTION_OPTION);
-        $table_survey_answer 			= Database :: get_course_table(TABLE_SURVEY_ANSWER);
+        $table_survey_question_option = Database :: get_course_table(TABLE_SURVEY_QUESTION_OPTION);
+        $table_survey_answer = Database :: get_course_table(TABLE_SURVEY_ANSWER);
 
         $course_id = api_get_course_int_id();
 
@@ -2468,9 +2467,9 @@ 
                 echo '		<td>'.$value['option_text'].'</td>';
                 echo '		<td>'.$i.'</td>';
                 echo '		<td><a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?action='.$action.'&survey_id='.Security::remove_XSS($_GET['survey_id']).'&question='.Security::remove_XSS($offset).'&viewoption='.$value['question_option_id'].'&value='.$i.'">'.$absolute_number.'</a></td>';
-                echo '		<td>'.round($absolute_number/$number_of_answers*100, 2).' %</td>';
+                echo '		<td>'.round($absolute_number / $number_of_answers * 100, 2).' %</td>';
                 echo '		<td>';
-                $size = ($absolute_number/$number_of_answers*100*2);
+                $size = ($absolute_number / $number_of_answers * 100 * 2);
                 if ($size > 0) {
                     echo '			<div style="border:1px solid #264269; background-color:#aecaf4; height:10px; width:'.$size.'px">&nbsp;</div>';
                 }
@@ -2510,11 +2509,11 @@ 
         // Actions bar
         echo '<div class="actions">';
         echo '<a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?survey_id='.Security::remove_XSS($_GET['survey_id']).'">
-		'.Display::return_icon('back.png',get_lang('BackTo').' '.get_lang('ReportingOverview'),'',ICON_SIZE_MEDIUM).'</a>';
+		'.Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('ReportingOverview'), '', ICON_SIZE_MEDIUM).'</a>';
         echo '<a class="survey_export_link" href="javascript: void(0);" onclick="document.form1a.submit();">
-		'.Display::return_icon('export_csv.png',get_lang('ExportAsCSV'),'',ICON_SIZE_MEDIUM).'</a>';
+		'.Display::return_icon('export_csv.png', get_lang('ExportAsCSV'), '', ICON_SIZE_MEDIUM).'</a>';
         echo '<a class="survey_export_link" href="javascript: void(0);" onclick="document.form1b.submit();">
-		'.Display::return_icon('export_excel.png',get_lang('ExportAsXLS'),'',ICON_SIZE_MEDIUM).'</a>';
+		'.Display::return_icon('export_excel.png', get_lang('ExportAsXLS'), '', ICON_SIZE_MEDIUM).'</a>';
         echo '</div>';
 
         // The form
@@ -2549,8 +2548,8 @@ 
             // Show user fields section with a big th colspan that spans over all fields
             $extra_user_fields = UserManager::get_extra_fields(0, 0, 5, 'ASC', false, true);
             $num = count($extra_user_fields);
-            if ($num > 0 ) {
-                echo '<th '.($num>0?' colspan="'.$num.'"':'').'>';
+            if ($num > 0) {
+                echo '<th '.($num > 0 ? ' colspan="'.$num.'"' : '').'>';
                 echo '<label><input type="checkbox" name="fields_filter" value="1" checked="checked"/> ';
                 echo get_lang('UserFields');
                 echo '</label>';
@@ -2606,7 +2605,7 @@ 
         if (!(isset($_POST['submit_question_filter']) && $_POST['submit_question_filter'] ||
                 isset($_POST['export_report']) && $_POST['export_report']) || !empty($_POST['fields_filter'])) {
             //show the fields names for user fields
-            foreach($extra_user_fields as & $field) {
+            foreach ($extra_user_fields as & $field) {
                 echo '<th>'.$field[3].'</th>';
             }
         }
@@ -2689,7 +2688,7 @@ 
                     $questions,
                     $display_extra_user_fields
                 );
-                $answers_of_user=array();
+                $answers_of_user = array();
             }
             if (isset($questions[$row['question_id']]) && $questions[$row['question_id']]['type'] != 'open') {
                 $answers_of_user[$row['question_id']][$row['option_id']] = $row;
@@ -2751,7 +2750,7 @@ 
                 echo '<th>'.$user.'</th>'; // the user column
             }
         } else {
-            echo '<th>' . get_lang('Anonymous') . ' ' . $user . '</th>';
+            echo '<th>'.get_lang('Anonymous').' '.$user.'</th>';
         }
 
         if ($display_extra_user_fields) {
@@ -2847,10 +2846,10 @@ 
                 // We do not show comment and pagebreak question types
                 if ($row['type'] != 'comment' && $row['type'] != 'pagebreak') {
                     if ($row['number_of_options'] == 0 && $row['type'] == 'open') {
-                        $return .= str_replace("\r\n",'  ', api_html_entity_decode(strip_tags($row['survey_question']), ENT_QUOTES)).';';
+                        $return .= str_replace("\r\n", '  ', api_html_entity_decode(strip_tags($row['survey_question']), ENT_QUOTES)).';';
                     } else {
                         for ($ii = 0; $ii < $row['number_of_options']; $ii++) {
-                            $return .= str_replace("\r\n",'  ', api_html_entity_decode(strip_tags($row['survey_question']), ENT_QUOTES)).';';
+                            $return .= str_replace("\r\n", '  ', api_html_entity_decode(strip_tags($row['survey_question']), ENT_QUOTES)).';';
                         }
                     }
                 }
@@ -2864,7 +2863,7 @@ 
         // Show the fields names for user fields
         if (!empty($extra_user_fields)) {
             foreach ($extra_user_fields as & $field) {
-                $return .= '"'.str_replace("\r\n",'  ',api_html_entity_decode(strip_tags($field[3]), ENT_QUOTES)).'";';
+                $return .= '"'.str_replace("\r\n", '  ', api_html_entity_decode(strip_tags($field[3]), ENT_QUOTES)).'";';
             }
         }
 
@@ -2900,7 +2899,7 @@ 
             ) {
                 // We do not show comment and pagebreak question types
                 if ($row['type'] != 'comment' && $row['type'] != 'pagebreak') {
-                    $row['option_text'] = str_replace(array("\r","\n"),array('',''),$row['option_text']);
+                    $row['option_text'] = str_replace(array("\r", "\n"), array('', ''), $row['option_text']);
                     $return .= api_html_entity_decode(strip_tags($row['option_text']), ENT_QUOTES).';';
                     $possible_answers[$row['question_id']][$row['question_option_id']] = $row['question_option_id'];
                     $possible_answers_type[$row['question_id']] = $row['type'];
@@ -2930,9 +2929,9 @@ 
                     $old_user,
                     true
                 );
-                $answers_of_user=array();
+                $answers_of_user = array();
             }
-            if($possible_answers_type[$row['question_id']] == 'open') {
+            if ($possible_answers_type[$row['question_id']] == 'open') {
                 $temp_id = 'open'.$open_question_iterator;
                 $answers_of_user[$row['question_id']][$temp_id] = $row;
                 $open_question_iterator++;
@@ -2991,7 +2990,7 @@ 
 
         if ($display_extra_user_fields) {
             // Show user fields data, if any, for this user
-            $user_fields_values = UserManager::get_extra_user_data($user,false,false, false, true);
+            $user_fields_values = UserManager::get_extra_user_data($user, false, false, false, true);
             foreach ($user_fields_values as & $value) {
                 $return .= '"'.str_replace('"', '""', api_html_entity_decode(strip_tags($value), ENT_QUOTES)).'";';
             }
@@ -3104,7 +3103,7 @@ 
                         );
                         $column++;
                     } else {
-                        for ($ii = 0; $ii < $row['number_of_options']; $ii ++) {
+                        for ($ii = 0; $ii < $row['number_of_options']; $ii++) {
                             $worksheet->setCellValueByColumnAndRow(
                                 $column,
                                 $line,
@@ -3185,7 +3184,7 @@ 
         }
 
         // Getting all the answers of the users
-        $line ++;
+        $line++;
         $column = 0;
         $old_user = '';
         $answers_of_user = array();
@@ -3194,7 +3193,7 @@ 
         if ($user_id != 0) {
             $sql .= " AND user='".intval($user_id)."' ";
         }
-        $sql .=	" ORDER BY user ASC";
+        $sql .= " ORDER BY user ASC";
 
         $open_question_iterator = 1;
         $result = Database::query($sql);
@@ -3270,7 +3269,7 @@ 
                         FROM '.Database::get_main_table(TABLE_MAIN_USER).'
                         WHERE user_id='.intval($user);
                 $rs = Database::query($sql);
-                if($row = Database::fetch_array($rs)) {
+                if ($row = Database::fetch_array($rs)) {
                     $user_displayed = api_get_person_name($row['firstname'], $row['lastname']);
                 } else {
                     $user_displayed = '-';
@@ -3285,7 +3284,7 @@ 
 
         if ($display_extra_user_fields) {
             //show user fields data, if any, for this user
-            $user_fields_values = UserManager::get_extra_user_data(intval($user),false,false, false, true);
+            $user_fields_values = UserManager::get_extra_user_data(intval($user), false, false, false, true);
             foreach ($user_fields_values as $value) {
                 $return[] = api_html_entity_decode(strip_tags($value), ENT_QUOTES);
             }
@@ -3346,7 +3345,7 @@ 
         // Actions bar
         echo '<div class="actions">';
         echo '<a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?survey_id='.$surveyId.'&'.api_get_cidreq().'">'.
-                Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('ReportingOverview'),'',ICON_SIZE_MEDIUM).'</a>';
+                Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('ReportingOverview'), '', ICON_SIZE_MEDIUM).'</a>';
         echo '</div>';
 
         // Displaying an information message that only the questions with predefined answers can be used in a comparative report
@@ -3411,17 +3410,17 @@ 
             $tableHtml .= '<tr>';
             for ($ii = 0; $ii <= count($question_x['answers']); $ii++) {
                 if ($ii == 0) {
-                    $tableHtml .=  '<th>&nbsp;</th>';
+                    $tableHtml .= '<th>&nbsp;</th>';
                 } else {
                     if ($question_x['type'] == 'score') {
                         for ($x = 1; $x <= $question_x['maximum_score']; $x++) {
-                            $tableHtml .= '<th>'.$question_x['answers'][($ii-1)].'<br />'.$x.'</th>';
+                            $tableHtml .= '<th>'.$question_x['answers'][($ii - 1)].'<br />'.$x.'</th>';
                         }
                         $x = '';
                     } else {
-                        $tableHtml .= '<th>'.$question_x['answers'][($ii-1)].'</th>';
+                        $tableHtml .= '<th>'.$question_x['answers'][($ii - 1)].'</th>';
                     }
-                    $optionText = strip_tags($question_x['answers'][$ii-1]);
+                    $optionText = strip_tags($question_x['answers'][$ii - 1]);
                     $optionText = html_entity_decode($optionText);
                     array_push($xOptions, trim($optionText));
                 }
@@ -3436,15 +3435,15 @@ 
                 // The Y axis is a scoring question type so we have more rows than the options (actually options * maximum score)
                 if ($question_y['type'] == 'score') {
                     for ($y = 1; $y <= $question_y['maximum_score']; $y++) {
-                        $tableHtml .=  '<tr>';
+                        $tableHtml .= '<tr>';
                         for ($ii = 0; $ii <= count($question_x['answers']); $ii++) {
                             if ($question_x['type'] == 'score') {
                                 for ($x = 1; $x <= $question_x['maximum_score']; $x++) {
                                     if ($ii == 0) {
-                                        $tableHtml .=  '<th>'.$question_y['answers'][($ij)].' '.$y.'</th>';
+                                        $tableHtml .= '<th>'.$question_y['answers'][($ij)].' '.$y.'</th>';
                                         break;
                                     } else {
-                                        $tableHtml .=  '<td align="center">';
+                                        $tableHtml .= '<td align="center">';
                                         $votes = SurveyUtil::comparative_check(
                                             $answers_x,
                                             $answers_y,
@@ -3453,16 +3452,16 @@ 
                                             $x,
                                             $y
                                         );
-                                        $tableHtml .=  $votes;
+                                        $tableHtml .= $votes;
                                         array_push(
                                             $chartData,
                                             array(
-                                                'serie' => array($currentYQuestion, $xOptions[$ii-1]),
+                                                'serie' => array($currentYQuestion, $xOptions[$ii - 1]),
                                                 'option' => $x,
                                                 'votes' => $votes
                                             )
                                         );
-                                        $tableHtml .=  '</td>';
+                                        $tableHtml .= '</td>';
                                     }
                                 }
                             } else {
@@ -3482,33 +3481,33 @@ 
                                     array_push(
                                         $chartData,
                                         array(
-                                            'serie' => array($currentYQuestion, $xOptions[$ii-1]),
+                                            'serie' => array($currentYQuestion, $xOptions[$ii - 1]),
                                             'option' => $y,
                                             'votes' => $votes
                                         )
                                     );
-                                    $tableHtml .=  '</td>';
+                                    $tableHtml .= '</td>';
                                 }
                             }
                         }
-                        $tableHtml .=  '</tr>';
+                        $tableHtml .= '</tr>';
                     }
                 }
                 // The Y axis is NOT a score question type so the number of rows = the number of options
                 else {
-                    $tableHtml .=  '<tr>';
+                    $tableHtml .= '<tr>';
                     for ($ii = 0; $ii <= count($question_x['answers']); $ii++) {
                         if ($question_x['type'] == 'score') {
                             for ($x = 1; $x <= $question_x['maximum_score']; $x++) {
                                 if ($ii == 0) {
-                                    $tableHtml .=  '<th>'.$question_y['answers'][$ij].'</th>';
+                                    $tableHtml .= '<th>'.$question_y['answers'][$ij].'</th>';
                                     break;
                                 } else {
-                                    $tableHtml .=  '<td align="center">';
-                                    $votes =  SurveyUtil::comparative_check(
+                                    $tableHtml .= '<td align="center">';
+                                    $votes = SurveyUtil::comparative_check(
                                         $answers_x,
                                         $answers_y,
-                                        $question_x['answersid'][($ii-1)],
+                                        $question_x['answersid'][($ii - 1)],
                                         $question_y['answersid'][($ij)],
                                         $x,
                                         0
@@ -3517,25 +3516,25 @@ 
                                     array_push(
                                         $chartData,
                                         array(
-                                            'serie' => array($currentYQuestion, $xOptions[$ii-1]),
+                                            'serie' => array($currentYQuestion, $xOptions[$ii - 1]),
                                             'option' => $x,
                                             'votes' => $votes
                                         )
                                     );
-                                    $tableHtml .=  '</td>';
+                                    $tableHtml .= '</td>';
                                 }
                             }
                         } else {
                             if ($ii == 0) {
-                                $tableHtml .=  '<th>'.$question_y['answers'][($ij)].'</th>';
+                                $tableHtml .= '<th>'.$question_y['answers'][($ij)].'</th>';
                             } else {
-                                $tableHtml .=  '<td align="center">';
-                                $votes = SurveyUtil::comparative_check($answers_x, $answers_y, $question_x['answersid'][($ii-1)], $question_y['answersid'][($ij)]);
+                                $tableHtml .= '<td align="center">';
+                                $votes = SurveyUtil::comparative_check($answers_x, $answers_y, $question_x['answersid'][($ii - 1)], $question_y['answersid'][($ij)]);
                                 $tableHtml .= $votes;
                                 array_push(
                                     $chartData,
                                     array(
-                                        'serie' => $xOptions[$ii-1],
+                                        'serie' => $xOptions[$ii - 1],
                                         'option' => $currentYQuestion,
                                         'votes' => $votes
                                     )
@@ -3547,7 +3546,7 @@ 
                     $tableHtml .= '</tr>';
                 }
             }
-            $tableHtml .=  '</table>';
+            $tableHtml .= '</table>';
             echo '<div id="chartContainer" class="col-md-12">';
             echo self::drawChart($chartData, true);
             echo '</div>';
@@ -3690,7 +3689,7 @@ 
                     survey_id='".intval($_GET['survey_id'])."' AND
                     session_id='".api_get_session_id()."' ";
         $res = Database::query($sql);
-        $row = Database::fetch_array($res,'ASSOC');
+        $row = Database::fetch_array($res, 'ASSOC');
 
         return $row['total'];
     }
@@ -3764,7 +3763,7 @@ 
             $exclude_users = SurveyManager::get_people_who_filled_survey($_GET['survey_id']);
         }
 
-        $counter = 0;  // Nr of invitations "sent" (if sendmail option)
+        $counter = 0; // Nr of invitations "sent" (if sendmail option)
         $course_id = api_get_course_int_id();
         $session_id = api_get_session_id();
 
@@ -3920,7 +3919,7 @@ 
         $text_link = '<a href="'.$survey_link.'">'.get_lang('ClickHereToAnswerTheSurvey')."</a><br />\r\n<br />\r\n".get_lang('OrCopyPasteTheFollowingUrl')." <br />\r\n ".$survey_link;
 
         $replace_count = 0;
-        $full_invitation_text = api_str_ireplace('**link**', $text_link ,$invitation_text, $replace_count);
+        $full_invitation_text = api_str_ireplace('**link**', $text_link, $invitation_text, $replace_count);
         if ($replace_count < 1) {
             $full_invitation_text = $full_invitation_text."<br />\r\n<br />\r\n".$text_link;
         }
@@ -3992,8 +3991,8 @@ 
         $course_id = api_get_course_int_id();
 
         // Database table definition
-        $table_survey_invitation 	= Database :: get_course_table(TABLE_SURVEY_INVITATION);
-        $table_survey 				= Database :: get_course_table(TABLE_SURVEY);
+        $table_survey_invitation = Database :: get_course_table(TABLE_SURVEY_INVITATION);
+        $table_survey = Database :: get_course_table(TABLE_SURVEY);
 
         // Counting the number of people that are invited
         $sql = "SELECT count(user) as total
@@ -4107,7 +4106,7 @@ 
     {
         $course_id = api_get_course_int_id();
         // Database table definition
-        $table_survey_invitation 	= Database :: get_course_table(TABLE_SURVEY_INVITATION);
+        $table_survey_invitation = Database :: get_course_table(TABLE_SURVEY_INVITATION);
 
         $sql = "SELECT * FROM $table_survey_invitation
 		        WHERE
@@ -4216,7 +4215,7 @@ 
     public static function display_survey_list_for_coach()
     {
         $parameters = array();
-        $parameters['cidReq']=api_get_course_id();
+        $parameters['cidReq'] = api_get_course_id();
         if (isset($_GET['do_search'])) {
             $message = get_lang('DisplaySearchResults').'<br />';
             $message .= '<a href="'.api_get_self().'?'.api_get_cidreq().'">'.get_lang('DisplayAll').'</a>';
@@ -4260,17 +4259,17 @@ 
 
         if ($drh) {
             return '<a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?'.api_get_cidreq().'&survey_id='.$survey_id.'">'.
-            Display::return_icon('stats.png', get_lang('Reporting'),'',ICON_SIZE_SMALL).'</a>';
+            Display::return_icon('stats.png', get_lang('Reporting'), '', ICON_SIZE_SMALL).'</a>';
         }
 
         // Coach can see that only if the survey is in his session
         if (api_is_allowed_to_edit() ||
             api_is_element_in_the_session(TOOL_SURVEY, $survey_id)
         ) {
-            $return .= '<a href="'.api_get_path(WEB_CODE_PATH).'survey/create_new_survey.php?'.api_get_cidreq().'&action=edit&survey_id='.$survey_id.'">'.Display::return_icon('edit.png', get_lang('Edit'),'',ICON_SIZE_SMALL).'</a>';
+            $return .= '<a href="'.api_get_path(WEB_CODE_PATH).'survey/create_new_survey.php?'.api_get_cidreq().'&action=edit&survey_id='.$survey_id.'">'.Display::return_icon('edit.png', get_lang('Edit'), '', ICON_SIZE_SMALL).'</a>';
             if (SurveyManager::survey_generation_hash_available()) {
-                $return .=  Display::url(
-                    Display::return_icon('new_link.png', get_lang('GenerateSurveyAccessLink'),'',ICON_SIZE_SMALL),
+                $return .= Display::url(
+                    Display::return_icon('new_link.png', get_lang('GenerateSurveyAccessLink'), '', ICON_SIZE_SMALL),
                     api_get_path(WEB_CODE_PATH).'survey/generate_link.php?survey_id='.$survey_id.'&'.api_get_cidreq()
                 );
             }
@@ -4280,20 +4279,20 @@ 
             );
 
             $return .= ' <a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_list.php?'.api_get_cidreq().'&action=empty&survey_id='.$survey_id.'" onclick="javascript: if(!confirm(\''.addslashes(api_htmlentities(get_lang("EmptySurvey").'?')).'\')) return false;">'.
-                Display::return_icon('clean.png', get_lang('EmptySurvey'),'',ICON_SIZE_SMALL).'</a>&nbsp;';
+                Display::return_icon('clean.png', get_lang('EmptySurvey'), '', ICON_SIZE_SMALL).'</a>&nbsp;';
         }
         $return .= '<a href="'.api_get_path(WEB_CODE_PATH).'survey/preview.php?'.api_get_cidreq().'&survey_id='.$survey_id.'">'.
-            Display::return_icon('preview_view.png', get_lang('Preview'),'',ICON_SIZE_SMALL).'</a>&nbsp;';
+            Display::return_icon('preview_view.png', get_lang('Preview'), '', ICON_SIZE_SMALL).'</a>&nbsp;';
         $return .= '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_invite.php?'.api_get_cidreq().'&survey_id='.$survey_id.'">'.
-            Display::return_icon('mail_send.png', get_lang('Publish'),'',ICON_SIZE_SMALL).'</a>&nbsp;';
+            Display::return_icon('mail_send.png', get_lang('Publish'), '', ICON_SIZE_SMALL).'</a>&nbsp;';
         $return .= '<a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?'.api_get_cidreq().'&survey_id='.$survey_id.'">'.
-            Display::return_icon('stats.png', get_lang('Reporting'),'',ICON_SIZE_SMALL).'</a>';
+            Display::return_icon('stats.png', get_lang('Reporting'), '', ICON_SIZE_SMALL).'</a>';
 
         if (api_is_allowed_to_edit() ||
             api_is_element_in_the_session(TOOL_SURVEY, $survey_id)
         ) {
             $return .= '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_list.php?'.api_get_cidreq().'&action=delete&survey_id='.$survey_id.'" onclick="javascript: if(!confirm(\''.addslashes(api_htmlentities(get_lang("DeleteSurvey").'?', ENT_QUOTES)).'\')) return false;">'.
-                Display::return_icon('delete.png', get_lang('Delete'),'',ICON_SIZE_SMALL).'</a>&nbsp;';
+                Display::return_icon('delete.png', get_lang('Delete'), '', ICON_SIZE_SMALL).'</a>&nbsp;';
         }
 
         return $return;
@@ -4306,9 +4305,9 @@ 
         //$return .= '<a href="survey_list.php?'.api_get_cidreq().'&action=delete&survey_id='.$survey_id.'" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("DeleteSurvey").'?', ENT_QUOTES)).'\')) return false;">'.Display::return_icon('delete.gif', get_lang('Delete')).'</a>';
         //$return .= '<a href="create_survey_in_another_language.php?id_survey='.$survey_id.'">'.Display::return_icon('copy.gif', get_lang('Copy')).'</a>';
         //$return .= '<a href="survey.php?survey_id='.$survey_id.'">'.Display::return_icon('add.gif', get_lang('Add')).'</a>';
-        $return = '<a href="'.api_get_path(WEB_CODE_PATH).'survey/preview.php?'.api_get_cidreq().'&survey_id='.$survey_id.'">'.Display::return_icon('preview_view.png', get_lang('Preview'),'',ICON_SIZE_SMALL).'</a>&nbsp;';
-        $return .= '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_invite.php?'.api_get_cidreq().'&survey_id='.$survey_id.'">'.Display::return_icon('mail_send.png', get_lang('Publish'),'',ICON_SIZE_SMALL).'</a>&nbsp;';
-        $return .= '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_list.php?'.api_get_cidreq().'&action=empty&survey_id='.$survey_id.'" onclick="javascript: if(!confirm(\''.addslashes(api_htmlentities(get_lang("EmptySurvey").'?', ENT_QUOTES)).'\')) return false;">'.Display::return_icon('clean.png', get_lang('EmptySurvey'),'',ICON_SIZE_SMALL).'</a>&nbsp;';
+        $return = '<a href="'.api_get_path(WEB_CODE_PATH).'survey/preview.php?'.api_get_cidreq().'&survey_id='.$survey_id.'">'.Display::return_icon('preview_view.png', get_lang('Preview'), '', ICON_SIZE_SMALL).'</a>&nbsp;';
+        $return .= '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_invite.php?'.api_get_cidreq().'&survey_id='.$survey_id.'">'.Display::return_icon('mail_send.png', get_lang('Publish'), '', ICON_SIZE_SMALL).'</a>&nbsp;';
+        $return .= '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_list.php?'.api_get_cidreq().'&action=empty&survey_id='.$survey_id.'" onclick="javascript: if(!confirm(\''.addslashes(api_htmlentities(get_lang("EmptySurvey").'?', ENT_QUOTES)).'\')) return false;">'.Display::return_icon('clean.png', get_lang('EmptySurvey'), '', ICON_SIZE_SMALL).'</a>&nbsp;';
 
         return $return;
     }
@@ -4463,7 +4462,7 @@ 
 
             // Validation when belonging to a session
             $session_img = api_get_session_image($survey['session_id'], $_user['status']);
-            $array[2] = $survey[2] . $session_img;
+            $array[2] = $survey[2].$session_img;
             $array[3] = $survey[3];
             $array[4] = $survey[4];
             $array[5] = $survey[5];
@@ -4505,10 +4504,10 @@ 
         $last_version_surveys = $survey_tree->surveylist;
         $list = array();
         foreach ($last_version_surveys as & $survey) {
-            $list[]=$survey['id'];
+            $list[] = $survey['id'];
         }
         if (count($list) > 0) {
-            $list_condition = " AND survey.survey_id IN (".implode(',',$list).") ";
+            $list_condition = " AND survey.survey_id IN (".implode(',', $list).") ";
         } else {
             $list_condition = '';
         }
@@ -4636,11 +4635,11 @@ 
             echo '<tr>';
             if ($row['answered'] == 0) {
                 echo '<td>';
-                echo Display::return_icon('statistics.png', get_lang('CreateNewSurvey'), array(),ICON_SIZE_TINY);
+                echo Display::return_icon('statistics.png', get_lang('CreateNewSurvey'), array(), ICON_SIZE_TINY);
                 echo '<a href="'.api_get_path(WEB_CODE_PATH).'survey/fillsurvey.php?course='.$_course['sysCode'].'&invitationcode='.$row['invitation_code'].'&cidReq='.$_course['sysCode'].'">'.$row['title'].'</a></td>';
             } else {
                 echo '<td>';
-                echo Display::return_icon('statistics_na.png', get_lang('CreateNewSurvey'), array(),ICON_SIZE_TINY);
+                echo Display::return_icon('statistics_na.png', get_lang('CreateNewSurvey'), array(), ICON_SIZE_TINY);
                 echo '<a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?action=questionreport&cidReq='.$_course['sysCode'].'&id_session='.$row['session_id'].'&gidReq=0&origin=&survey_id='.$row['survey_id'].'">'.$row['title'].'</a></td>';
             }
             echo '<td class="center">';
@@ -4787,7 +4786,7 @@ 
                     if ($field_details[7] == 0) {
                         $field_list_array['extra_'.$field_details[1]]['visibility'] = 0;
                     } else {
-                        $field_list_array['extra_'.$field_details[1]]['visibility']=1;
+                        $field_list_array['extra_'.$field_details[1]]['visibility'] = 1;
                     }
                     break;
                 case UserManager::USER_FIELD_TYPE_DIVIDER:
@@ -4873,7 +4872,7 @@ 
         $htmlChart = '';
         if (api_browser_support("svg")) {
             $htmlChart .= api_get_js("d3/d3.v3.5.4.min.js");
-            $htmlChart .= api_get_js("dimple.v2.1.2.min.js") . '
+            $htmlChart .= api_get_js("dimple.v2.1.2.min.js").'
             <script type="text/javascript">
             var svg = dimple.newSvg("#'.$chartContainerId.'", "100%", 400);
             var data = [';
@@ -4882,40 +4881,40 @@ 
             foreach ($chartData as $chartDataElement) {
                 $htmlChart .= '{"';
                 if (!$hasSerie) {
-                    $htmlChart .= get_lang("Option") . '":"' . $chartDataElement['option'] . '", "';
+                    $htmlChart .= get_lang("Option").'":"'.$chartDataElement['option'].'", "';
                     array_push($order, $chartDataElement['option']);
                 } else {
                     if (!is_array($chartDataElement['serie'])) {
-                        $htmlChart .= get_lang("Option") . '":"' . $chartDataElement['serie'] . '", "' .
-                            get_lang("Score") . '":"' . $chartDataElement['option'] . '", "';
+                        $htmlChart .= get_lang("Option").'":"'.$chartDataElement['serie'].'", "'.
+                            get_lang("Score").'":"'.$chartDataElement['option'].'", "';
                         array_push($serie, $chartDataElement['serie']);
                     } else {
-                        $htmlChart .= get_lang("Serie") . '":"' . $chartDataElement['serie'][0] . '", "' .
-                            get_lang("Option") . '":"' . $chartDataElement['serie'][1] . '", "' .
-                            get_lang("Score") . '":"' . $chartDataElement['option'] . '", "';
+                        $htmlChart .= get_lang("Serie").'":"'.$chartDataElement['serie'][0].'", "'.
+                            get_lang("Option").'":"'.$chartDataElement['serie'][1].'", "'.
+                            get_lang("Score").'":"'.$chartDataElement['option'].'", "';
                     }
                 }
-                $htmlChart .= get_lang("Votes") . '":"' . $chartDataElement['votes'] .
+                $htmlChart .= get_lang("Votes").'":"'.$chartDataElement['votes'].
                     '"},';
             }
             rtrim($htmlChart, ",");
             $htmlChart .= '];
                 var myChart = new dimple.chart(svg, data);
-                myChart.addMeasureAxis("y", "' . get_lang("Votes") . '");';
+                myChart.addMeasureAxis("y", "' . get_lang("Votes").'");';
             if (!$hasSerie) {
-                $htmlChart .= 'var xAxisCategory = myChart.addCategoryAxis("x", "' . get_lang("Option") . '");
-                    xAxisCategory.addOrderRule(' . json_encode($order) . ');
-                    myChart.addSeries("' . get_lang("Option") . '", dimple.plot.bar);';
+                $htmlChart .= 'var xAxisCategory = myChart.addCategoryAxis("x", "'.get_lang("Option").'");
+                    xAxisCategory.addOrderRule(' . json_encode($order).');
+                    myChart.addSeries("' . get_lang("Option").'", dimple.plot.bar);';
             } else {
                 if (!is_array($chartDataElement['serie'])) {
                     $serie = array_values(array_unique($serie));
-                    $htmlChart .= 'var xAxisCategory = myChart.addCategoryAxis("x", ["' . get_lang("Option") . '","' . get_lang("Score") . '"]);
-                        xAxisCategory.addOrderRule(' . json_encode($serie) . ');
-                        xAxisCategory.addGroupOrderRule("' . get_lang("Score") . '");
-                        myChart.addSeries("' . get_lang("Option") . '", dimple.plot.bar);';
+                    $htmlChart .= 'var xAxisCategory = myChart.addCategoryAxis("x", ["'.get_lang("Option").'","'.get_lang("Score").'"]);
+                        xAxisCategory.addOrderRule(' . json_encode($serie).');
+                        xAxisCategory.addGroupOrderRule("' . get_lang("Score").'");
+                        myChart.addSeries("' . get_lang("Option").'", dimple.plot.bar);';
                 } else {
-                    $htmlChart .= 'myChart.addCategoryAxis("x", ["' . get_lang("Option") . '","' . get_lang("Score") . '"]);
-                        myChart.addSeries("' . get_lang("Serie") . '", dimple.plot.bar);';
+                    $htmlChart .= 'myChart.addCategoryAxis("x", ["'.get_lang("Option").'","'.get_lang("Score").'"]);
+                        myChart.addSeries("' . get_lang("Serie").'", dimple.plot.bar);';
                 }
             }
             $htmlChart .= 'myChart.draw();

plugin/vchamilo/views/syncparams.php

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
         isset($param['subkey']) && !empty($param['subkey']) ? $param['variable'].' ['.$param['subkey'].']' : $param['variable'],
         $param['category'],
         $param['access_url'],
-        '<input type="text" disabled name="value_'.$param['id'].'" value="'.htmlspecialchars($param['selected_value'], ENT_COMPAT, 'UTF-8' ).'" />'.
+        '<input type="text" disabled name="value_'.$param['id'].'" value="'.htmlspecialchars($param['selected_value'], ENT_COMPAT, 'UTF-8').'" />'.
         '<br />Master value: '.$param['selected_value'],
         $syncButton,
     );
@@ -56,8 +56,8 @@ 
 
 $content  = '<form name="settingsform" action="'.$thisurl.'">';
 $content .= '<input type="hidden" name="what" value="" />';
-$content .=  $table->toHtml();
-$content .=  '</form>';
+$content .= $table->toHtml();
+$content .= '</form>';
 $actions = '';
 
 Display::addFlash(Display::return_message($plugin->get_lang('Sync your master settings to all instances.')));

main/inc/introductionSection.inc.php

Spacing +25 added lines, -25 removed lines

@@ -203,7 +203,7 @@ 
     if ($displayMode == '1') {
         // Show only the current course progress step
         // $information_title = get_lang('InfoAboutLastDoneAdvance');
-        $last_done_advance =  $thematic->get_last_done_thematic_advance();
+        $last_done_advance = $thematic->get_last_done_thematic_advance();
         $thematic_advance_info = $thematic->get_thematic_advance_list($last_done_advance);
         $subTitle1 = get_lang('CurrentTopic');
         $class1 = ' current';
@@ -218,7 +218,7 @@ 
     } else if ($displayMode == '3') {
         // Show the current and next course progress steps
         // $information_title = get_lang('InfoAboutLastDoneAdvanceAndNextAdvanceNotDone');
-        $last_done_advance =  $thematic->get_last_done_thematic_advance();
+        $last_done_advance = $thematic->get_last_done_thematic_advance();
         $next_advance_not_done = $thematic->get_next_thematic_advance_not_done();
         $thematic_advance_info = $thematic->get_thematic_advance_list($last_done_advance);
         $thematic_advance_info2 = $thematic->get_thematic_advance_list($next_advance_not_done);
@@ -229,7 +229,7 @@ 
 
     if (!empty($thematic_advance_info)) {
         $thematic_advance = get_lang('CourseThematicAdvance');
-        $thematicScore = $thematic->get_total_average_of_thematic_advances() . '%';
+        $thematicScore = $thematic->get_total_average_of_thematic_advances().'%';
         $thematicUrl = api_get_path(WEB_CODE_PATH).'course_progress/index.php?action=thematic_details&'.api_get_cidreq();
         $thematic_info = $thematic->get_thematic_list(
             $thematic_advance_info['thematic_id']
@@ -244,11 +244,11 @@ 
         );
         $userInfo = api_get_user_info();
         $courseInfo = api_get_course_info();
-        $titleThematic = $thematic_advance .' : '. $courseInfo['name'] . ' <b>( '. $thematicScore .' )</b>';
+        $titleThematic = $thematic_advance.' : '.$courseInfo['name'].' <b>( '.$thematicScore.' )</b>';
 
-        $infoUser = '<div class="thematic-avatar"><img src="' . $userInfo['avatar'] . '" class="img-circle img-responsive"></div>';
+        $infoUser = '<div class="thematic-avatar"><img src="'.$userInfo['avatar'].'" class="img-circle img-responsive"></div>';
         $infoUser .= '<div class="progress">
-                        <div class="progress-bar progress-bar-danger" role="progressbar" style="width: ' . $thematicScore . ';">
+                        <div class="progress-bar progress-bar-danger" role="progressbar" style="width: ' . $thematicScore.';">
                         '.$thematicScore.'
                         </div>
                     </div>';
@@ -256,11 +256,11 @@ 
         $thematicItemOne = '
         <div class="col-md-6 items-progress">
             <div class="thematic-cont '.$class1.'">
-            <div class="topics">' . $subTitle1 . '</div>
-            <h4 class="title-topics">' . Display::returnFontAwesomeIcon('book') . $thematic_info['title'] . '</h4>
-            <p class="date">' .  Display::returnFontAwesomeIcon('calendar-o') . $thematic_advance_info['start_date'] . '</p>
-            <div class="views">' . Display::returnFontAwesomeIcon('file-text-o')  . strip_tags($thematic_advance_info['content']). '</div>
-            <p class="time">'. Display::returnFontAwesomeIcon('clock-o') . get_lang('DurationInHours') . ' : ' . $thematic_advance_info['duration'] . ' - <a href="' . $thematicUrl . '">' . get_lang('SeeDetail') . '</a></p>
+            <div class="topics">' . $subTitle1.'</div>
+            <h4 class="title-topics">' . Display::returnFontAwesomeIcon('book').$thematic_info['title'].'</h4>
+            <p class="date">' .  Display::returnFontAwesomeIcon('calendar-o').$thematic_advance_info['start_date'].'</p>
+            <div class="views">' . Display::returnFontAwesomeIcon('file-text-o').strip_tags($thematic_advance_info['content']).'</div>
+            <p class="time">'. Display::returnFontAwesomeIcon('clock-o').get_lang('DurationInHours').' : '.$thematic_advance_info['duration'].' - <a href="'.$thematicUrl.'">'.get_lang('SeeDetail').'</a></p>
             </div>
         </div>';
 
@@ -273,19 +273,19 @@ 
                 <div class="col-md-6 items-progress">
                     <div class="thematic-cont">
                     <div class="topics">'.$subTitle2.'</div>
-                    <h4 class="title-topics">'. Display::returnFontAwesomeIcon('book')  . $thematic_info2['title'].'</h4>
-                    <p class="date">' . Display::returnFontAwesomeIcon('calendar-o') .$thematic_advance_info2['start_date'].'</p>
-                    <div class="views">' . Display::returnFontAwesomeIcon('file-text-o')  . strip_tags($thematic_advance_info2['content']).'</div>
-                    <p class="time">'. Display::returnFontAwesomeIcon('clock-o') .get_lang('DurationInHours').' : '.$thematic_advance_info2['duration'].' - <a href="'.$thematicUrl.'">'.get_lang('SeeDetail').'</a></p>
+                    <h4 class="title-topics">'. Display::returnFontAwesomeIcon('book').$thematic_info2['title'].'</h4>
+                    <p class="date">' . Display::returnFontAwesomeIcon('calendar-o').$thematic_advance_info2['start_date'].'</p>
+                    <div class="views">' . Display::returnFontAwesomeIcon('file-text-o').strip_tags($thematic_advance_info2['content']).'</div>
+                    <p class="time">'. Display::returnFontAwesomeIcon('clock-o').get_lang('DurationInHours').' : '.$thematic_advance_info2['duration'].' - <a href="'.$thematicUrl.'">'.get_lang('SeeDetail').'</a></p>
                     </div>
                 </div>';
         }
         $thematicPanel = '<div class="row">';
-        $thematicPanel .= '<div class="col-md-2">' . $infoUser . '</div>';
-        $thematicPanel .= '<div class="col-md-10"><div class="row">' . $thematicItemOne . $thematicItemTwo . '</div></div>';
+        $thematicPanel .= '<div class="col-md-2">'.$infoUser.'</div>';
+        $thematicPanel .= '<div class="col-md-10"><div class="row">'.$thematicItemOne.$thematicItemTwo.'</div></div>';
         $thematicPanel .= '</div>';
         $thematicPanel .= '<div class="separate">
-                        <a href="' . $thematicUrl . '" class="btn btn-default btn-block">' . get_lang('ShowFullCourseAdvance') . '</a>
+                        <a href="' . $thematicUrl.'" class="btn btn-default btn-block">'.get_lang('ShowFullCourseAdvance').'</a>
                     </div>';
         $thematicProgress = Display::panelCollapse($titleThematic, $thematicPanel, 'thematic', null, 'accordion-thematic', 'collapse-thematic', false);
     }
@@ -302,7 +302,7 @@ 
     $editIconButton = Display::url(
         '<em class="fa fa-wrench"></em> ',
         api_get_path(WEB_CODE_PATH).'course_info/tools.php?'.api_get_cidreq(),
-        ['class' => 'btn btn-default', 'title' => get_lang('CustomizeIcons') ]
+        ['class' => 'btn btn-default', 'title' => get_lang('CustomizeIcons')]
     );
 }
 
@@ -313,12 +313,12 @@ 
         // Displays "Add intro" commands
         $toolbar = '<div class="btn-group pull-right" role="group">';
         if (!empty($courseId)) {
-            $textIntro  = '<a class="btn btn-default" title="' . addslashes(get_lang('AddIntro')) . '" href="'.api_get_self().'?' . api_get_cidreq().'&intro_cmdAdd=1">';
+            $textIntro  = '<a class="btn btn-default" title="'.addslashes(get_lang('AddIntro')).'" href="'.api_get_self().'?'.api_get_cidreq().'&intro_cmdAdd=1">';
             $textIntro .= '<em class="fa fa-file-text"></em> ';
             $textIntro .= "</a>";
-            $toolbar .= $textIntro . $editIconButton;
+            $toolbar .= $textIntro.$editIconButton;
         } else {
-            $toolbar .= '<a class="btn btn-default" href="' . api_get_self() . '?intro_cmdAdd=1">' . get_lang('AddIntro') . '</a>';
+            $toolbar .= '<a class="btn btn-default" href="'.api_get_self().'?intro_cmdAdd=1">'.get_lang('AddIntro').'</a>';
             $toolbar .= $editIconButton;
         }
         $toolbar .= '</div>';
@@ -331,7 +331,7 @@ 
                 '<a  class="btn btn-default" href="'.api_get_self().'?'.api_get_cidreq().'&intro_cmdEdit=1" title="'.get_lang('Modify').'">
                 <em class="fa fa-pencil"></em></a>';
             $toolbar .= $editIconButton;
-            $toolbar .="<a class=\"btn btn-default\" href=\"".api_get_self()."?".api_get_cidreq()."&intro_cmdDel=1\" onclick=\"javascript:
+            $toolbar .= "<a class=\"btn btn-default\" href=\"".api_get_self()."?".api_get_cidreq()."&intro_cmdDel=1\" onclick=\"javascript:
                 if(!confirm('".addslashes(api_htmlentities(get_lang('ConfirmYourChoice'), ENT_QUOTES, $charset)).
                 "')) return false;\"><em class=\"fa fa-trash-o\"></em></a>";
 
@@ -345,7 +345,7 @@ 
                 if(!confirm('".addslashes(api_htmlentities(get_lang('ConfirmYourChoice'), ENT_QUOTES, $charset)).
                 "')) return false;\"><em class=\"fa fa-trash-o\"></em></a>";
         }
-        $toolbar .=  "</div>";
+        $toolbar .= "</div>";
         // Fix for chrome XSS filter for videos in iframes - BT#7930
         $browser = api_get_navigator();
         if (strpos($introduction_section, '<iframe') !== false && $browser['name'] == 'Chrome') {
@@ -364,7 +364,7 @@ 
     } else {
         if (api_is_allowed_to_edit()) {
             $introduction_section .= '<div class="help-course">';
-            $introduction_section .= get_lang('AddCustomCourseIntro') . ' ' . $textIntro;
+            $introduction_section .= get_lang('AddCustomCourseIntro').' '.$textIntro;
             $introduction_section .= '</div>';
         }
     }

main/inc/lib/database.lib.php

Spacing +9 added lines, -10 removed lines

@@ -115,8 +115,7 @@ 
      */
     public static function getUTCDateTimeTypeClass()
     {
-        return isset(self::$utcDateTimeClass) ? self::$utcDateTimeClass :
-        'Application\DoctrineExtensions\DBAL\Types\UTCDateTimeType';
+        return isset(self::$utcDateTimeClass) ? self::$utcDateTimeClass : 'Application\DoctrineExtensions\DBAL\Types\UTCDateTimeType';
     }
 
     /**
@@ -163,7 +162,7 @@ 
         );*/
 
         AnnotationRegistry::registerLoader(
-            function ($class) use ($sysPath) {
+            function($class) use ($sysPath) {
                 $file = str_replace("\\", DIRECTORY_SEPARATOR, $class).".php";
                 $file = str_replace('Symfony/Component/Validator', '', $file);
                 $file = $sysPath.'vendor/symfony/validator'.$file;
@@ -428,7 +427,7 @@ 
 
         if (!empty($params)) {
             $sql = 'INSERT INTO '.$table_name.' ('.implode(',', $params).')
-                    VALUES (:'.implode(', :' ,$params).')';
+                    VALUES (:'.implode(', :', $params).')';
 
             $statement = self::getManager()->getConnection()->prepare($sql);
             $result = $statement->execute($attributes);
@@ -472,7 +471,7 @@ 
             foreach ($attributes as $key => $value) {
                 $updateSql .= "$key = :$key ";
                 if ($count < count($attributes)) {
-                    $updateSql.= ', ';
+                    $updateSql .= ', ';
                 }
                 $count++;
             }
@@ -527,7 +526,7 @@ 
             if ($columns == '*') {
                 $clean_columns = '*';
             } else {
-                $clean_columns = (string)$columns;
+                $clean_columns = (string) $columns;
             }
         }
 
@@ -574,7 +573,7 @@ 
                             $clean_values = array();
                             foreach ($value_array as $item) {
                                 $item = Database::escape_string($item);
-                                $clean_values[]= $item;
+                                $clean_values[] = $item;
                             }
                         } else {
                             $value_array = Database::escape_string($value_array);
@@ -598,7 +597,7 @@ 
                     }
 
                     if (!empty($where_return)) {
-                        $return_value = " WHERE $where_return" ;
+                        $return_value = " WHERE $where_return";
                     }
                     break;
                 case 'order':
@@ -621,10 +620,10 @@ 
                                 if (in_array($element[1], array('desc', 'asc'))) {
                                     $order = $element[1];
                                 }
-                                $temp_value[]= $element[0].' '.$order.' ';
+                                $temp_value[] = $element[0].' '.$order.' ';
                             } else {
                                 //by default DESC
-                                $temp_value[]= $element[0].' DESC ';
+                                $temp_value[] = $element[0].' DESC ';
                             }
                         }
                         if (!empty($temp_value)) {

main/inc/lib/webservices/Rest.php

Spacing +23 added lines, -23 removed lines

@@ -238,7 +238,7 @@ 
             'urlPicture' => $this->course->getPicturePath(true),
             'teachers' => $teachers,
             'tools' => array_map(
-                function ($tool) {
+                function($tool) {
                     return ['type' => $tool['name']];
                 },
                 $tools
@@ -261,7 +261,7 @@ 
             $results[] = [
                 'id' => $description->get_description_type(),
                 'title' => $description->get_title(),
-                'content' => str_replace('src="/', 'src="' . api_get_path(WEB_PATH), $description->get_content())
+                'content' => str_replace('src="/', 'src="'.api_get_path(WEB_PATH), $description->get_content())
             ];
         }
 
@@ -294,7 +294,7 @@ 
 
             $path = $directory['path'];
         }
-        require_once api_get_path(LIBRARY_PATH) . 'fileDisplay.lib.php';
+        require_once api_get_path(LIBRARY_PATH).'fileDisplay.lib.php';
 
         $courseInfo = api_get_course_info_by_id($this->course->getId());
         $documents = DocumentManager::get_all_document_data(
@@ -309,7 +309,7 @@ 
         $results = [];
 
         if (is_array($documents)) {
-            $webPath = api_get_path(WEB_CODE_PATH) . 'document/document.php?';
+            $webPath = api_get_path(WEB_CODE_PATH).'document/document.php?';
 
             /** @var array $document */
             foreach ($documents as $document) {
@@ -326,7 +326,7 @@ 
                     'type' => $document['filetype'],
                     'title' => $document['title'],
                     'path' => $document['path'],
-                    'url' => $webPath . http_build_query([
+                    'url' => $webPath.http_build_query([
                         'username' => $this->user->getUsername(),
                         'api_key' => $this->apiKey,
                         'cidReq' => $this->course->getCode(),
@@ -370,7 +370,7 @@ 
             $sessionId
         );
 
-        $announcements = array_map(function ($announcement) {
+        $announcements = array_map(function($announcement) {
             return [
                 'id' => intval($announcement['id']),
                 'title' => strip_tags($announcement['title']),
@@ -452,11 +452,11 @@ 
         $webPath = api_get_path(WEB_PATH);
 
         return array_map(
-            function ($event) use ($webPath) {
+            function($event) use ($webPath) {
                 return [
                     'id' => intval($event['unique_id']),
                     'title' => $event['title'],
-                    'content' => str_replace('src="/', 'src="' . $webPath, $event['description']),
+                    'content' => str_replace('src="/', 'src="'.$webPath, $event['description']),
                     'startDate' => $event['start_date_localtime'],
                     'endDate' => $event['end_date_localtime'],
                     'isAllDay' => $event['allDay'] ? true : false
@@ -478,7 +478,7 @@ 
         $notebooks = $notebooksRepo->findByUser($this->user, $this->course, $this->session);
 
         return array_map(
-            function (\Chamilo\CourseBundle\Entity\CNotebook $notebook) {
+            function(\Chamilo\CourseBundle\Entity\CNotebook $notebook) {
                 return [
                     'id' => $notebook->getIid(),
                     'title' => $notebook->getTitle(),
@@ -502,9 +502,9 @@ 
     public function getCourseForumCategories()
     {
         $sessionId = $this->session ? $this->session->getId() : 0;
-        $webCoursePath = api_get_path(WEB_COURSE_PATH) . $this->course->getDirectory() . '/upload/forum/images/';
+        $webCoursePath = api_get_path(WEB_COURSE_PATH).$this->course->getDirectory().'/upload/forum/images/';
 
-        require_once api_get_path(SYS_CODE_PATH) . 'forum/forumfunction.inc.php';
+        require_once api_get_path(SYS_CODE_PATH).'forum/forumfunction.inc.php';
 
         $categoriesFullData = get_forum_categories('', $this->course->getId(), $sessionId);
         $categories = [];
@@ -518,7 +518,7 @@ 
                 'catId' => intval($forumInfo['forum_category']),
                 'title' => $forumInfo['forum_title'],
                 'description' => $forumInfo['forum_comment'],
-                'image' => $forumInfo['forum_image'] ? ($webCoursePath . $forumInfo['forum_image']) : '',
+                'image' => $forumInfo['forum_image'] ? ($webCoursePath.$forumInfo['forum_image']) : '',
                 'numberOfThreads' => isset($forumInfo['number_of_threads']) ? intval($forumInfo['number_of_threads']) : 0,
                 'lastPost' => null
             ];
@@ -541,7 +541,7 @@ 
         foreach ($categoriesFullData as $category) {
             $categoryForums = array_filter(
                 $forums,
-                function (array $forum) use ($category) {
+                function(array $forum) use ($category) {
                     if ($forum['catId'] != $category['cat_id']) {
                         return false;
                     }
@@ -570,7 +570,7 @@ 
      */
     public function getCourseForum($forumId)
     {
-        require_once api_get_path(SYS_CODE_PATH) . 'forum/forumfunction.inc.php';
+        require_once api_get_path(SYS_CODE_PATH).'forum/forumfunction.inc.php';
 
         $forumInfo = get_forums($forumId, $this->course->getCode());
 
@@ -578,12 +578,12 @@ 
             throw new Exception(get_lang('NoForum'));
         }
 
-        $webCoursePath = api_get_path(WEB_COURSE_PATH) . $this->course->getDirectory() . '/upload/forum/images/';
+        $webCoursePath = api_get_path(WEB_COURSE_PATH).$this->course->getDirectory().'/upload/forum/images/';
         $forum = [
             'id' => $forumInfo['iid'],
             'title' => $forumInfo['forum_title'],
             'description' => $forumInfo['forum_comment'],
-            'image' => $forumInfo['forum_image'] ? ($webCoursePath . $forumInfo['forum_image']) : '',
+            'image' => $forumInfo['forum_image'] ? ($webCoursePath.$forumInfo['forum_image']) : '',
             'threads' => []
         ];
 
@@ -610,7 +610,7 @@ 
      */
     public function getCourseForumThread($forumId, $threadId)
     {
-        require_once api_get_path(SYS_CODE_PATH) . 'forum/forumfunction.inc.php';
+        require_once api_get_path(SYS_CODE_PATH).'forum/forumfunction.inc.php';
 
         $threadInfo = get_thread_information($forumId, $threadId);
 
@@ -648,7 +648,7 @@ 
         $pictureInfo = UserManager::get_user_picture_path_by_id($this->user->getId(), 'web');
 
         $result = [
-            'pictureUri' => $pictureInfo['dir'] . $pictureInfo['file'],
+            'pictureUri' => $pictureInfo['dir'].$pictureInfo['file'],
             'fullName' => $this->user->getCompleteName(),
             'username' => $this->user->getUsername(),
             'officialCode' => $this->user->getOfficialCode(),
@@ -758,7 +758,7 @@ 
                     'id' => $lpId,
                     'title' => Security::remove_XSS($lpDetails['lp_name']),
                     'progress' => intval($progress),
-                    'url' => api_get_path(WEB_CODE_PATH) . 'webservices/api/v2.php?' . http_build_query([
+                    'url' => api_get_path(WEB_CODE_PATH).'webservices/api/v2.php?'.http_build_query([
                         'hash' => $this->encodeParams([
                             'action' => 'course_learnpath',
                             'lp_id' => $lpId,
@@ -805,7 +805,7 @@ 
      * @param string $encoded
      * @return array
      */
-    public static function decodeParams($encoded){
+    public static function decodeParams($encoded) {
         $decoded = str_replace(['-', '_', '.'], ['+', '/', '='], $encoded);
         $mod4 = strlen($decoded) % 4;
 
@@ -833,7 +833,7 @@ 
         ChamiloSession::write('_user', $loggedUser);
         Login::init_user($this->user->getId(), true);
 
-        $url = api_get_path(WEB_CODE_PATH) . 'lp/lp_controller.php?' . http_build_query([
+        $url = api_get_path(WEB_CODE_PATH).'lp/lp_controller.php?'.http_build_query([
             'cidReq' => $this->course->getCode(),
             'id_session' => $sessionId,
             'gidReq' => 0,
@@ -855,7 +855,7 @@ 
      */
     public function saveForumPost(array $postValues, $forumId)
     {
-        require_once api_get_path(SYS_CODE_PATH) . 'forum/forumfunction.inc.php';
+        require_once api_get_path(SYS_CODE_PATH).'forum/forumfunction.inc.php';
 
         $forum = get_forums($forumId, $this->course->getCode());
 
@@ -992,7 +992,7 @@ 
      */
     public function saveForumThread(array $values, $forumId)
     {
-        require_once api_get_path(SYS_CODE_PATH) . 'forum/forumfunction.inc.php';
+        require_once api_get_path(SYS_CODE_PATH).'forum/forumfunction.inc.php';
 
         $forum = get_forums($forumId, $this->course->getCode());
         $courseInfo = api_get_course_info($this->course->getCode());