chamilo / chamilo-lms

main/mySpace/admin.php

Spacing +10 added lines, -10 removed lines

@@ -35,13 +35,13 @@ 
 }
 $sql = "SELECT user.user_id,lastname,firstname,email
         FROM $tbl_user as user, $tbl_admin as admin
-        WHERE admin.user_id=user.user_id" . $order_clause;
+        WHERE admin.user_id=user.user_id".$order_clause;
 $result_admins = Database::query($sql);
 
 if (api_is_western_name_order()) {
-    echo '<table class="data_table"><tr><th>' . get_lang('FirstName') . '</th><th>' . get_lang('LastName') . '</th><th>' . get_lang('Email') . '</th></tr>';
+    echo '<table class="data_table"><tr><th>'.get_lang('FirstName').'</th><th>'.get_lang('LastName').'</th><th>'.get_lang('Email').'</th></tr>';
 } else {
-    echo '<table class="data_table"><tr><th>' . get_lang('LastName') . '</th><th>' . get_lang('FirstName') . '</th><th>' . get_lang('Email') . '</th></tr>';
+    echo '<table class="data_table"><tr><th>'.get_lang('LastName').'</th><th>'.get_lang('FirstName').'</th><th>'.get_lang('Email').'</th></tr>';
 }
 
 if (api_is_western_name_order(PERSON_NAME_DATA_EXPORT)) {
@@ -65,9 +65,9 @@ 
             $css_class = "row_odd";
             if ($i % 20 == 0 && $i != 0) {
                 if (api_is_western_name_order()) {
-                    echo '<tr><th>' . get_lang('FirstName') . '</th><th>' . get_lang('LastName') . '</th><th>' . get_lang('Email') . '</th></tr>';
+                    echo '<tr><th>'.get_lang('FirstName').'</th><th>'.get_lang('LastName').'</th><th>'.get_lang('Email').'</th></tr>';
                 } else {
-                    echo '<tr><th>' . get_lang('LastName') . '</th><th>' . get_lang('FirstName') . '</th><th>' . get_lang('Email') . '</th></tr>';
+                    echo '<tr><th>'.get_lang('LastName').'</th><th>'.get_lang('FirstName').'</th><th>'.get_lang('Email').'</th></tr>';
                 }
             }
         } else {
@@ -77,9 +77,9 @@ 
         $i++;
 
         if (api_is_western_name_order()) {
-            echo "<tr class=" . $css_class . "><td>$firstname</td><td>$lastname</td><td><a href='mailto:" . $email . "'>$email</a></td></tr>";
+            echo "<tr class=".$css_class."><td>$firstname</td><td>$lastname</td><td><a href='mailto:".$email."'>$email</a></td></tr>";
         } else {
-            echo "<tr class=" . $css_class . "><td>$lastname</td><td>$firstname</td><td><a href='mailto:" . $email . "'>$email</a></td></tr>";
+            echo "<tr class=".$css_class."><td>$lastname</td><td>$firstname</td><td><a href='mailto:".$email."'>$email</a></td></tr>";
         }
 
         if (api_is_western_name_order(PERSON_NAME_DATA_EXPORT)) {
@@ -93,7 +93,7 @@ 
     }
 } else {
     // No results
-    echo '<tr><td colspan="3">' . get_lang('NoResults') . '</td></tr>';
+    echo '<tr><td colspan="3">'.get_lang('NoResults').'</td></tr>';
 }
 echo '</table>';
 
@@ -104,8 +104,8 @@ 
 echo "
     <br /><br />
     <form method='post' action='admin.php'>
-        <button type='submit' class='save' name='export' value='" . get_lang('ExportExcel') . "'>
-            " . get_lang('ExportExcel') . "
+        <button type='submit' class='save' name='export' value='" . get_lang('ExportExcel')."'>
+            " . get_lang('ExportExcel')."
         </button>
     <form>
 ";

Indentation +3 added lines, -3 removed lines

@@ -1,9 +1,9 @@
 <?php
 /* For licensing terms, see /license.txt */
 /**
- * Special reporting page for admins
- * @package chamilo.reporting
- */
+     * Special reporting page for admins
+     * @package chamilo.reporting
+     */
 
 ob_start();
 $cidReset = true;

main/mySpace/teachers.php

Spacing +9 added lines, -9 removed lines

@@ -20,14 +20,14 @@ 
 
 $this_section = SECTION_TRACKING;
 
-$interbreadcrumb[] = array ("url" => "index.php", "name" => get_lang('MySpace'));
+$interbreadcrumb[] = array("url" => "index.php", "name" => get_lang('MySpace'));
 
 if (isset($_GET["user_id"]) && $_GET["user_id"] != "" && !isset($_GET["type"])) {
-    $interbreadcrumb[] = array ("url" => "teachers.php", "name" => get_lang('Teachers'));
+    $interbreadcrumb[] = array("url" => "teachers.php", "name" => get_lang('Teachers'));
 }
 
-if (isset($_GET["user_id"]) && $_GET["user_id"]!="" && isset($_GET["type"]) && $_GET["type"] == "coach") {
-    $interbreadcrumb[] = array ("url" => "coaches.php", "name" => get_lang('Tutors'));
+if (isset($_GET["user_id"]) && $_GET["user_id"] != "" && isset($_GET["type"]) && $_GET["type"] == "coach") {
+    $interbreadcrumb[] = array("url" => "coaches.php", "name" => get_lang('Tutors'));
 }
 
 function get_count_users()
@@ -122,8 +122,8 @@ 
                 $courseInfo = api_get_course_info($course_code);
                 $courseId = $courseInfo['real_id'];
                 if (CourseManager :: is_user_subscribed_in_course($student_id, $course_code, true)) {
-                    $avg_time_spent 	+= Tracking :: get_time_spent_on_the_course($student_id, $courseId, $_GET['id_session']);
-                    $my_average 		 = Tracking :: get_avg_student_score($student_id, $course_code);
+                    $avg_time_spent += Tracking :: get_time_spent_on_the_course($student_id, $courseId, $_GET['id_session']);
+                    $my_average = Tracking :: get_avg_student_score($student_id, $course_code);
                     if (is_numeric($my_average)) {
                         $avg_student_score += $my_average;
                     }
@@ -162,7 +162,7 @@ 
             $detailsLink = '<a href="myStudents.php?student='.$student_id.'&id_coach='.$coach_id.'&id_session='.$_GET['id_session'].'">
 				          '.Display::return_icon('2rightarrow.png', get_lang('Details')).'</a>';
         } else {
-            $detailsLink =  '<a href="myStudents.php?student='.$student_id.'">
+            $detailsLink = '<a href="myStudents.php?student='.$student_id.'">
 				            '.Display::return_icon('2rightarrow.png', get_lang('Details')).'</a>';
         }
         $row[] = $detailsLink;
@@ -181,7 +181,7 @@ 
 $actionsLeft = '';
 if (api_is_drh()) {
     $menu_items = array(
-        Display::url(Display::return_icon('stats.png', get_lang('MyStats'), '', ICON_SIZE_MEDIUM), api_get_path(WEB_CODE_PATH)."auth/my_progress.php" ),
+        Display::url(Display::return_icon('stats.png', get_lang('MyStats'), '', ICON_SIZE_MEDIUM), api_get_path(WEB_CODE_PATH)."auth/my_progress.php"),
         Display::url(Display::return_icon('user.png', get_lang('Students'), array(), ICON_SIZE_MEDIUM), 'student.php'),
         Display::url(Display::return_icon('teacher_na.png', get_lang('Trainers'), array(), ICON_SIZE_MEDIUM), 'teachers.php'),
         Display::url(Display::return_icon('course.png', get_lang('Courses'), array(), ICON_SIZE_MEDIUM), 'course.php'),
@@ -208,7 +208,7 @@ 
 );
 
 
-$toolbar = Display::toolbarAction('toolbar-teachers', $content = array( 0 => $actionsLeft, 1 => $actionsRight ));
+$toolbar = Display::toolbarAction('toolbar-teachers', $content = array(0 => $actionsLeft, 1 => $actionsRight));
 
 $table = new SortableTable(
     'tracking_teachers',

Indentation +3 added lines, -3 removed lines

@@ -1,9 +1,9 @@
 <?php
 /* For licensing terms, see /license.txt */
 /**
- * Teacher report
- * @package chamilo.reporting
- */
+     * Teacher report
+     * @package chamilo.reporting
+     */
 
 $cidReset = true;
 

main/mySpace/access_details.php

Spacing +1 added lines, -1 removed lines

@@ -138,7 +138,7 @@
     </div><br />
     <div id="cev_cont_stats">
     <?php
-    if ($result_to_print != "")  {
+    if ($result_to_print != "") {
         $rst = get_stats($user_id, $courseId);
         $foo_stats = '<strong>'.get_lang('Total').': </strong>'.$rst['total'].'<br />';
         $foo_stats .= '<strong>'.get_lang('Average').': </strong>'.$rst['avg'].'<br />';

main/wiki/diff.inc.php

Spacing +8 added lines, -8 removed lines

@@ -137,19 +137,19 @@ 
     switch ($type) {
         case DIFF_EQUAL:
             // return $line. ' : ' . ' = <span class="diffEqual" >' . $value . '</span><br />' . "\n" ;
-            return '<span class="diffEqual" >' . $value . '</span><br />' . "\n"; //juan carlos muestra solo color
+            return '<span class="diffEqual" >'.$value.'</span><br />'."\n"; //juan carlos muestra solo color
             break;
         case DIFF_MOVED:
             //return $line. ' : ' . ' M <span class="diffMoved" >' . $value . '</span><br />' . "\n" ; //juan carlos ra�a  la sustitye la inverior
-            return '<span class="diffMoved" >' . $value . '</span><br />' . "\n"; //juan carlos muestra solo color
+            return '<span class="diffMoved" >'.$value.'</span><br />'."\n"; //juan carlos muestra solo color
             break;
         case DIFF_ADDED:
             //return $line . ' : ' . ' + <span class="diffAdded" >' . $value . '</span><br />' . "\n" ;
-            return '<span class="diffAdded" >' . $value . '</span><br />' . "\n"; //juan carlos muestra solo color
+            return '<span class="diffAdded" >'.$value.'</span><br />'."\n"; //juan carlos muestra solo color
             break;
         case DIFF_DELETED:
             //return $line . ' : ' . ' - <span class="diffDeleted" >' . $value . '</span><br />' . "\n" ; //juan carlos ra�a  la sustitye la inverior
-            return '<span class="diffDeleted" >' . $value . '</span><br />' . "\n"; //juan carlos muestra solo color
+            return '<span class="diffDeleted" >'.$value.'</span><br />'."\n"; //juan carlos muestra solo color
             break;
     }
 }
@@ -169,19 +169,19 @@ 
     switch ($type) {
         case DIFF_EQUAL:
             //return '<tr><td>' . $line. '&nbsp;:&nbsp;' . '&nbsp;=</td><td><span class="diffEqual" >'                    . $value . '</span></td></tr>' . "\n"; //juan carlos comentado
-            return '<tr><td></td><td bgcolor="#FFFFFF">' . $value . '</td></tr>' . "\n"; //juan carlos muestra solo color (no tambi�n la l�nea). Adem�s EN IEXPLORER VA BIEN PERO EN FIREFOX 3 la etiqueta span no muestra el color de fondo que est� definido en la hoja de estilos como background-color, aceptando s�lo la propiedad color pero esta solo da color al texto con lo cual los cambios quedan poco resaltados, adem�s los cambios de otros objetos que no sean texto no se indican por ej. a�adir una imagen, por esta raz�n doy el color de fondo al td directamente.
+            return '<tr><td></td><td bgcolor="#FFFFFF">'.$value.'</td></tr>'."\n"; //juan carlos muestra solo color (no tambi�n la l�nea). Adem�s EN IEXPLORER VA BIEN PERO EN FIREFOX 3 la etiqueta span no muestra el color de fondo que est� definido en la hoja de estilos como background-color, aceptando s�lo la propiedad color pero esta solo da color al texto con lo cual los cambios quedan poco resaltados, adem�s los cambios de otros objetos que no sean texto no se indican por ej. a�adir una imagen, por esta raz�n doy el color de fondo al td directamente.
             break;
         case DIFF_MOVED:
             // return '<tr><td>' . $line. '&nbsp;:&nbsp;' . '&nbsp;M</td><td><span class="diffMoved" >'                    . $value . '</span></td></tr>' . "\n" //juan carlos comenta
 
-            return '<tr><td></td><td bgcolor="#FFFFAA">' . $value . '</td></tr>' . "\n"; //juan carlos muestra solo color (no tambi�n la l�nea). Adem�s EN IEXPLORER VA BIEN PERO EN FIREFOX 3 la etiqueta span no muestra el color de fondo que est� definido en la hoja de estilos como background-color, aceptando s�lo la propiedad color pero esta solo da color al texto con lo cual los cambios quedan poco resaltados, adem�s los cambios de otros objetos que no sean texto no se indican por ej. a�adir una imagen, por esta raz�n doy el color de fondo al td directamente.
+            return '<tr><td></td><td bgcolor="#FFFFAA">'.$value.'</td></tr>'."\n"; //juan carlos muestra solo color (no tambi�n la l�nea). Adem�s EN IEXPLORER VA BIEN PERO EN FIREFOX 3 la etiqueta span no muestra el color de fondo que est� definido en la hoja de estilos como background-color, aceptando s�lo la propiedad color pero esta solo da color al texto con lo cual los cambios quedan poco resaltados, adem�s los cambios de otros objetos que no sean texto no se indican por ej. a�adir una imagen, por esta raz�n doy el color de fondo al td directamente.
             break;
         case DIFF_ADDED:
             // return '<tr><td>' . $line. '&nbsp;:&nbsp;' . '&nbsp;+</td><td><span class="diffAdded" >'                    . $value . '</span></td></tr>' . "\n" ; //juan carlos comentado
-            return '<tr><td></td><td bgcolor="#CCFFCC">' . $value . '</td></tr>' . "\n"; //juan carlos muestra solo color (no tambi�n la l�nea). Adem�s EN IEXPLORER VA BIEN PERO EN FIREFOX 3 la etiqueta span no muestra el color de fondo que est� definido en la hoja de estilos como background-color, aceptando s�lo la propiedad color pero esta solo da color al texto con lo cual los cambios quedan poco resaltados, adem�s los cambios de otros objetos que no sean texto no se indican por ej. a�adir una imagen, por esta raz�n doy el color de fondo al td directamente.
+            return '<tr><td></td><td bgcolor="#CCFFCC">'.$value.'</td></tr>'."\n"; //juan carlos muestra solo color (no tambi�n la l�nea). Adem�s EN IEXPLORER VA BIEN PERO EN FIREFOX 3 la etiqueta span no muestra el color de fondo que est� definido en la hoja de estilos como background-color, aceptando s�lo la propiedad color pero esta solo da color al texto con lo cual los cambios quedan poco resaltados, adem�s los cambios de otros objetos que no sean texto no se indican por ej. a�adir una imagen, por esta raz�n doy el color de fondo al td directamente.
             break;
         case DIFF_DELETED:
             // return '<tr><td>' . $line. '&nbsp;:&nbsp;' . '&nbsp;-</td><td><span class="diffDeleted" >'                    . $value . '</span></td></tr>' . "\n" ; //juan carlos comentado
-            return '<tr><td></td><td bgcolor="#FFAAAA">' . $value . '</td></tr>' . "\n"; //juan carlos muestra solo color (no tambi�n la l�nea). Adem�s EN IEXPLORER VA BIEN PERO EN FIREFOX 3 la etiqueta span no muestra el color de fondo que est� definido en la hoja de estilos como background-color, aceptando s�lo la propiedad color pero esta solo da color al texto con lo cual los cambios quedan poco resaltados, adem�s los cambios de otros objetos que no sean texto no se indican por ej. a�adir una imagen, por esta raz�n doy el color de fondo al td directamente.
+            return '<tr><td></td><td bgcolor="#FFAAAA">'.$value.'</td></tr>'."\n"; //juan carlos muestra solo color (no tambi�n la l�nea). Adem�s EN IEXPLORER VA BIEN PERO EN FIREFOX 3 la etiqueta span no muestra el color de fondo que est� definido en la hoja de estilos como background-color, aceptando s�lo la propiedad color pero esta solo da color al texto con lo cual los cambios quedan poco resaltados, adem�s los cambios de otros objetos que no sean texto no se indican por ej. a�adir una imagen, por esta raz�n doy el color de fondo al td directamente.
     }
 }

main/wiki/index.php

Spacing +4 added lines, -4 removed lines

@@ -18,7 +18,7 @@ 
 
 // section (for the tabs)
 $this_section = SECTION_COURSES;
-$current_course_tool  = TOOL_WIKI;
+$current_course_tool = TOOL_WIKI;
 
 $course_id = api_get_course_int_id();
 $session_id = api_get_session_id();
@@ -27,7 +27,7 @@ 
 $groupId = api_get_group_id();
 
 // additional style information
-$htmlHeadXtra[] ='<link rel="stylesheet" type="text/css" href="'.api_get_path(WEB_CODE_PATH).'wiki/css/default.css"/>';
+$htmlHeadXtra[] = '<link rel="stylesheet" type="text/css" href="'.api_get_path(WEB_CODE_PATH).'wiki/css/default.css"/>';
 
 // javascript for advanced parameters menu
 $htmlHeadXtra[] = '<script>
@@ -72,7 +72,7 @@ 
     //ensure this tool in groups whe it's private or deactivated
     if ($group_properties['wiki_state'] == 0) {
         api_not_allowed();
-    } elseif ($group_properties['wiki_state']==2) {
+    } elseif ($group_properties['wiki_state'] == 2) {
         if (!api_is_allowed_to_edit(false, true) and
             !GroupManager :: is_user_in_group(api_get_user_id(), api_get_group_id())
         ) {
@@ -84,7 +84,7 @@ 
 $is_allowed_to_edit = api_is_allowed_to_edit(false, true);
 
 // The page we are dealing with
-$page = isset($_GET['title']) ? $_GET['title']: 'index';
+$page = isset($_GET['title']) ? $_GET['title'] : 'index';
 $action = isset($_GET['action']) ? Security::remove_XSS($_GET['action']) : 'showpage';
 $view = isset($_GET['view']) ? Security::remove_XSS($_GET['view']) : null;
 

main/forum/newthread.php

Spacing +8 added lines, -8 removed lines

@@ -54,12 +54,12 @@ 
 
 /* Breadcrumbs */
 
-if (isset($_SESSION['gradebook'])){
+if (isset($_SESSION['gradebook'])) {
     $gradebook = Security::remove_XSS($_SESSION['gradebook']);
 }
 
 if (!empty($gradebook) && $gradebook == 'view') {
-    $interbreadcrumb[] = array (
+    $interbreadcrumb[] = array(
         'url' => '../gradebook/'.Security::remove_XSS($_SESSION['gradebook_dest']),
         'name' => get_lang('ToolGradebook')
     );
@@ -67,7 +67,7 @@ 
 
 if (!empty($_GET['gidReq'])) {
     $toolgroup = intval($_GET['gidReq']);
-    Session::write('toolgroup',$toolgroup);
+    Session::write('toolgroup', $toolgroup);
 }
 
 /* Is the user allowed here? */
@@ -122,7 +122,7 @@ 
     $interbreadcrumb[] = array('url' => '../group/group.php?'.$cidreq, 'name' => get_lang('Groups'));
     $interbreadcrumb[] = array('url' => '../group/group_space.php?'.$cidreq, 'name' => get_lang('GroupSpace').' '.$groupProperties['name']);
     $interbreadcrumb[] = array('url' => 'viewforum.php?'.$cidreq.'&forum='.Security::remove_XSS($_GET['forum']), 'name' => $current_forum['forum_title']);
-    $interbreadcrumb[] = array('url' => 'newthread.php?'.$cidreq.'&forum='.Security::remove_XSS($_GET['forum']),'name' => get_lang('NewTopic'));
+    $interbreadcrumb[] = array('url' => 'newthread.php?'.$cidreq.'&forum='.Security::remove_XSS($_GET['forum']), 'name' => get_lang('NewTopic'));
 } else {
     $interbreadcrumb[] = array('url' => 'index.php?'.$cidreq, 'name' => $nameTools);
     $interbreadcrumb[] = array('url' => 'viewforumcategory.php?'.$cidreq.'&forumcategory='.$current_forum_category['cat_id'], 'name' => $current_forum_category['cat_title']);
@@ -133,9 +133,9 @@ 
 /* Resource Linker */
 
 if (isset($_POST['add_resources']) AND $_POST['add_resources'] == get_lang('Resources')) {
-    $_SESSION['formelements']	= $_POST;
-    $_SESSION['origin']			= $_SERVER['REQUEST_URI'];
-    $_SESSION['breadcrumbs']	= $interbreadcrumb;
+    $_SESSION['formelements'] = $_POST;
+    $_SESSION['origin'] = $_SERVER['REQUEST_URI'];
+    $_SESSION['breadcrumbs'] = $interbreadcrumb;
     header('Location: ../resourcelinker/resourcelinker.php');
     exit;
 }
@@ -170,7 +170,7 @@ 
 echo '<div class="actions">';
 echo '<span style="float:right;">'.search_link().'</span>';
 echo '<a href="viewforum.php?forum='.Security::remove_XSS($_GET['forum']).'&'.$cidreq.'">'.
-    Display::return_icon('back.png',get_lang('BackToForum'),'',ICON_SIZE_MEDIUM).'</a>';
+    Display::return_icon('back.png', get_lang('BackToForum'), '', ICON_SIZE_MEDIUM).'</a>';
 echo '</div>';
 
 // Set forum attachment data into $_SESSION

main/forum/index.php

Spacing +39 added lines, -39 removed lines

@@ -64,7 +64,7 @@ 
 
 if (!empty($gradebook) && $gradebook == 'view') {
     $interbreadcrumb[] = array(
-        'url' => '../gradebook/' . $_SESSION['gradebook_dest'],
+        'url' => '../gradebook/'.$_SESSION['gradebook_dest'],
         'name' => get_lang('ToolGradebook'),
     );
 }
@@ -79,7 +79,7 @@ 
     switch ($_GET['content']) {
         case 'forum':
             $interbreadcrumb[] = array(
-                'url' => 'index.php?search=' . $search_forum . '&' . api_get_cidreq(),
+                'url' => 'index.php?search='.$search_forum.'&'.api_get_cidreq(),
                 'name' => get_lang('Forum'),
             );
             $interbreadcrumb[] = array(
@@ -89,7 +89,7 @@ 
             break;
         case 'forumcategory':
             $interbreadcrumb[] = array(
-                'url' => 'index.php?search=' . $search_forum . '&' . api_get_cidreq(),
+                'url' => 'index.php?search='.$search_forum.'&'.api_get_cidreq(),
                 'name' => get_lang('Forum'),
             );
             $interbreadcrumb[] = array(
@@ -185,7 +185,7 @@ 
         . "&gradebook=&action=add_item&type=step&lp_id=$lp_id#resource_tab-5\">"
         . Display::return_icon(
             'back.png',
-            get_lang("BackTo") . ' ' . get_lang("LearningPaths"),
+            get_lang("BackTo").' '.get_lang("LearningPaths"),
             '',
             ICON_SIZE_MEDIUM
         )
@@ -197,8 +197,8 @@ 
 
 if (api_is_allowed_to_edit(false, true)) {
     echo '<a href="'
-        . api_get_self() . '?' . api_get_cidreq()
-        . '&action=add&content=forumcategory&lp_id=' . $lp_id . '"> ' .
+        . api_get_self().'?'.api_get_cidreq()
+        . '&action=add&content=forumcategory&lp_id='.$lp_id.'"> '.
         Display::return_icon(
             'new_folder.png', get_lang('AddForumCategory'), '', ICON_SIZE_MEDIUM
         )
@@ -248,7 +248,7 @@ 
         );
 
         if (empty($sessionId) && !empty($forumCategory['session_name'])) {
-            $session_displayed = ' (' . Security::remove_XSS($forumCategory['session_name']) . ')';
+            $session_displayed = ' ('.Security::remove_XSS($forumCategory['session_name']).')';
         } else {
             $session_displayed = '';
         }
@@ -260,7 +260,7 @@ 
         $html = '';
         $iconsEdit = '';
         $idCategory = $forumCategory['cat_id'];
-        $urlCategory = 'viewforumcategory.php?' . api_get_cidreq() . '&forumcategory=' . intval($idCategory);
+        $urlCategory = 'viewforumcategory.php?'.api_get_cidreq().'&forumcategory='.intval($idCategory);
         $titleCategory = Display::tag(
             'a',
             $forumCategory['cat_title'],
@@ -278,9 +278,9 @@ 
                 !($forumCategory['session_id'] == 0 &&
                 intval($sessionId) != 0)
             ) {
-                $iconsEdit .= '<a href="' . api_get_self() . '?' . api_get_cidreq()
-                    . '&action=edit&content=forumcategory&id=' . intval($idCategory)
-                    . '">' . Display::return_icon(
+                $iconsEdit .= '<a href="'.api_get_self().'?'.api_get_cidreq()
+                    . '&action=edit&content=forumcategory&id='.intval($idCategory)
+                    . '">'.Display::return_icon(
                         'edit.png',
                         get_lang('Edit'),
                         array(),
@@ -288,8 +288,8 @@ 
                     )
                     . '</a>';
 
-                $iconsEdit .='<a href="' . api_get_self() . '?' . api_get_cidreq()
-                    . '&action=delete&content=forumcategory&id=' . intval($idCategory)
+                $iconsEdit .= '<a href="'.api_get_self().'?'.api_get_cidreq()
+                    . '&action=delete&content=forumcategory&id='.intval($idCategory)
                     . "\" onclick=\"javascript:if(!confirm('"
                     . addslashes(api_htmlentities(
                         get_lang('DeleteForumCategory'),
@@ -303,7 +303,7 @@ 
                         ICON_SIZE_SMALL
                     )
                     . '</a>';
-                $iconsEdit .=  return_visible_invisible_icon(
+                $iconsEdit .= return_visible_invisible_icon(
                     'forumcategory',
                     strval(intval($idCategory)),
                     strval(intval($forumCategory['visibility']))
@@ -337,7 +337,7 @@ 
             array(),
             ICON_SIZE_MEDIUM
         );
-        $html .= '<h3>' . $icoCategory . $titleCategory . '</h3>';
+        $html .= '<h3>'.$icoCategory.$titleCategory.'</h3>';
 
         if ($descriptionCategory != '' && trim($descriptionCategory) != '&nbsp;') {
             $html .= '<div class="forum-description">'.$descriptionCategory.'</div>';
@@ -470,15 +470,15 @@ 
                                 30
                             );
                             $forum_title_group_addition = ' (<a href="../group/group_space.php?'
-                                . api_get_cidreq() . '&gidReq=' . $forum['forum_of_group']
-                                . '" class="forum_group_link">' . get_lang('GoTo') . ' ' . $group_title . '</a>)'
+                                . api_get_cidreq().'&gidReq='.$forum['forum_of_group']
+                                . '" class="forum_group_link">'.get_lang('GoTo').' '.$group_title.'</a>)'
                                 . $session_img;
                         } else {
                             $forum_title_group_addition = '';
                         }
 
                         if (empty($sessionId) && !empty($forum['session_name'])) {
-                            $session_displayed = ' (' . $forum['session_name'] . ')';
+                            $session_displayed = ' ('.$forum['session_name'].')';
                         } else {
                             $session_displayed = '';
                         }
@@ -490,7 +490,7 @@ 
                         $html .= '<div class="row">';
                         $html .= '<div class="col-md-6">';
                         $html .= '<div class="col-md-3">';
-                        $html .= '<div class="number-post">'.$forum_image.'<p>'.$number_threads.' ' . get_lang('ForumThreads') . '</p>'
+                        $html .= '<div class="number-post">'.$forum_image.'<p>'.$number_threads.' '.get_lang('ForumThreads').'</p>'
                             . '</div>';
                         $html .= '</div>';
 
@@ -505,15 +505,15 @@ 
                         $linkForum .= Display::tag(
                             'a',
                             $forum['forum_title'],
-                            array (
-                                'href' => 'viewforum.php?' . api_get_cidreq()
-                                    . '&gidReq=' . intval($groupid)
-                                    . '&forum=' . intval($forum['forum_id']),
-                                'class' => return_visible_invisible( strval( intval($forum['visibility']) ) )
+                            array(
+                                'href' => 'viewforum.php?'.api_get_cidreq()
+                                    . '&gidReq='.intval($groupid)
+                                    . '&forum='.intval($forum['forum_id']),
+                                'class' => return_visible_invisible(strval(intval($forum['visibility'])))
                             )
                         );
 
-                        $html .= '<h3 class="title">' . $iconForum . $linkForum . '</h3>';
+                        $html .= '<h3 class="title">'.$iconForum.$linkForum.'</h3>';
 
                         $html .= Display::tag(
                             'p',
@@ -525,13 +525,13 @@ 
                         $html .= '</div>';
                         $html .= '</div>';
 
-                        $iconEmpty='';
+                        $iconEmpty = '';
 
                         // The number of topics and posts.
                         if ($forum['forum_of_group'] !== '0') {
-                            $newPost='';
+                            $newPost = '';
                             if (is_array($mywhatsnew_post_info) && !empty($mywhatsnew_post_info)) {
-                                $newPost = ' ' . Display::return_icon(
+                                $newPost = ' '.Display::return_icon(
                                     'alert.png',
                                     get_lang('Forum'),
                                     null,
@@ -542,7 +542,7 @@ 
                             }
                         } else {
                             if (is_array($mywhatsnew_post_info) && !empty($mywhatsnew_post_info)) {
-                                $newPost = ' ' . Display::return_icon(
+                                $newPost = ' '.Display::return_icon(
                                     'alert.png',
                                     get_lang('Forum'),
                                     null,
@@ -584,9 +584,9 @@ 
                                     null,
                                     ICON_SIZE_TINY
                                 )
-                                . ' ' .
+                                . ' '.
                                 api_convert_and_format_date($forum['last_post_date'])
-                                . '<br /> ' . get_lang('By') . ' ' .
+                                . '<br /> '.get_lang('By').' '.
                                 display_user_link(
                                     $poster_id,
                                     $name,
@@ -601,13 +601,13 @@ 
                             api_is_allowed_to_edit(false, true) &&
                             !($forum['session_id'] == 0 && intval($sessionId) != 0)
                         ) {
-                            $html .= '<a href="'.api_get_self() . '?' . api_get_cidreq()
-                                . '&action=edit&content=forum&id=' . $forum['forum_id'] . '">'
+                            $html .= '<a href="'.api_get_self().'?'.api_get_cidreq()
+                                . '&action=edit&content=forum&id='.$forum['forum_id'].'">'
                                 . Display::return_icon('edit.png', get_lang('Edit'), array(), ICON_SIZE_SMALL)
                                 . '</a>';
-                            $html .= '<a href="' . api_get_self() . '?' . api_get_cidreq()
-                                . '&action=delete&content=forum&id=' . $forum['forum_id']
-                                . "\" onclick=\"javascript:if(!confirm('" . addslashes(
+                            $html .= '<a href="'.api_get_self().'?'.api_get_cidreq()
+                                . '&action=delete&content=forum&id='.$forum['forum_id']
+                                . "\" onclick=\"javascript:if(!confirm('".addslashes(
                                     api_htmlentities(get_lang('DeleteForum'), ENT_QUOTES)
                                 )
                                 . "')) return false;\">"
@@ -645,8 +645,8 @@ 
                             !api_is_anonymous() &&
                             api_is_allowed_to_session_edit(false, true)
                         ) {
-                            $html .= '<a href="' . api_get_self() . '?' . api_get_cidreq()
-                                . '&action=notify&content=forum&id=' . $forum['forum_id'] . '">'
+                            $html .= '<a href="'.api_get_self().'?'.api_get_cidreq()
+                                . '&action=notify&content=forum&id='.$forum['forum_id'].'">'
                                 . Display::return_icon($iconnotify, get_lang('NotifyMe'), null, ICON_SIZE_SMALL)
                                 . '</a>';
                         }
@@ -663,7 +663,7 @@ 
                 }
             }
         } else {
-            echo '<div class="alert alert-warning">' . get_lang('NoForumInThisCategory') . '</div>'
+            echo '<div class="alert alert-warning">'.get_lang('NoForumInThisCategory').'</div>'
                 . (api_is_allowed_to_edit(false, true) ? '<div>' : '</div>')
                 . '</div>';
         }

main/forum/download.php

Indentation +7 added lines, -7 removed lines

@@ -1,13 +1,13 @@
 <?php
 /* For licensing terms, see /license.txt */
 /**
- * This file is responsible for  passing requested documents to the browser.
- * Html files are parsed to fix a few problems with URLs,
- * but this code will hopefully be replaced soon by an Apache URL
- * rewrite mechanism.
- *
- * @package chamilo.document
- */
+     * This file is responsible for  passing requested documents to the browser.
+     * Html files are parsed to fix a few problems with URLs,
+     * but this code will hopefully be replaced soon by an Apache URL
+     * rewrite mechanism.
+     *
+     * @package chamilo.document
+     */
 
 session_cache_limiter('public');
 

Spacing +4 added lines, -4 removed lines

@@ -31,7 +31,7 @@ 
 $doc_url = str_replace(' ', '+', $doc_url);
 $doc_url = str_replace('/..', '', $doc_url); //echo $doc_url;
 
-if (! isset($_course)) {
+if (!isset($_course)) {
     api_not_allowed(true);
 }
 
@@ -51,11 +51,11 @@ 
     exit;
 }
 
-$tbl_forum_attachment  = Database::get_course_table(TABLE_FORUM_ATTACHMENT);
+$tbl_forum_attachment = Database::get_course_table(TABLE_FORUM_ATTACHMENT);
 $tbl_forum_post = Database::get_course_table(TABLE_FORUM_POST);
 
 $course_id = api_get_course_int_id();
-$courseInfo =     api_get_course_info_by_id($course_id);
+$courseInfo = api_get_course_info_by_id($course_id);
 
 // launch event
 Event::event_download($doc_url);
@@ -85,7 +85,7 @@ 
     api_get_session_id()
 );
 
-if ($forum_thread_visibility==1 && $forum_forum_visibility==1) {
+if ($forum_thread_visibility == 1 && $forum_forum_visibility == 1) {
     if (Security::check_abs_path(
         $full_file_name,
         api_get_path(SYS_COURSE_PATH).$courseInfo['path'].'/upload/forum/')

main/forum/viewthread_threaded.inc.php

Indentation +2 added lines, -2 removed lines

@@ -457,8 +457,8 @@
 
 // The post has been displayed => it can be removed from the what's new array
 if (isset($whatsnew_post_info[$forumId][$threadId][$row['post_id']])) {
-  unset($whatsnew_post_info[$forumId][$threadId][$row['post_id']]);
-  unset($_SESSION['whatsnew_post_info'][$forumId][$threadId][$row['post_id']]);
+    unset($whatsnew_post_info[$forumId][$threadId][$row['post_id']]);
+    unset($_SESSION['whatsnew_post_info'][$forumId][$threadId][$row['post_id']]);
 }
 echo "</table>";
 

Spacing +95 added lines, -95 removed lines

@@ -21,7 +21,7 @@ 
  * @package chamilo.forum
  */
 
-$forumUrl = api_get_path(WEB_CODE_PATH) . 'forum/';
+$forumUrl = api_get_path(WEB_CODE_PATH).'forum/';
 $_user = api_get_user_info();
 $sortDirection = isset($_GET['posts_order']) && $_GET['posts_order'] === 'desc' ? 'DESC' : 'ASC';
 $rows = getPosts($_GET['thread'], $sortDirection, true);
@@ -42,13 +42,13 @@ 
 // Are we in a lp ?
 $origin = '';
 if (isset($_GET['origin'])) {
-    $origin =  Security::remove_XSS($_GET['origin']);
+    $origin = Security::remove_XSS($_GET['origin']);
 }
 
 // Delete attachment file.
 if (
     isset($_GET['action']) &&
-    $_GET['action']=='delete_attach' &&
+    $_GET['action'] == 'delete_attach' &&
     isset($_GET['id_attach'])
 ) {
     delete_attachment(0, $_GET['id_attach']);
@@ -59,7 +59,7 @@ 
 
 // Displaying the thread (structure)
 
-$thread_structure = "<div class=\"structure\">" .get_lang('Structure')."</div>";
+$thread_structure = "<div class=\"structure\">".get_lang('Structure')."</div>";
 $counter = 0;
 $count = 0;
 $prev_next_array = array();
@@ -71,7 +71,7 @@ 
 foreach ($rows as $post) {
     $counter++;
     $indent = $post['indent_cnt'] * '20';
-    $thread_structure .= "<div style=\"margin-left: " . $indent . "px;\">";
+    $thread_structure .= "<div style=\"margin-left: ".$indent."px;\">";
 
     if (
         !empty($whatsnew_post_info[$forumId][$post['thread_id']]) &&
@@ -89,7 +89,7 @@ 
             $counter == 1 AND !isset($_GET['post'])
         )
     ) {
-        $thread_structure .= '<strong>' .prepare4display($post['post_title']) . '</strong></div>';
+        $thread_structure .= '<strong>'.prepare4display($post['post_title']).'</strong></div>';
         $prev_next_array[] = $post['post_id'];
     } else {
         if ($post['visible'] == '0') {
@@ -98,10 +98,10 @@ 
             $class = '';
         }
         $count_loop = ($count == 0) ? '&id=1' : '';
-        $thread_structure .= "<a href=\"viewthread.php?" . api_get_cidreq() .
-            "&forum=" . $forumId . "&thread=" . $threadId .
-            "&post=" . $post['post_id'] . "&origin=$origin$count_loop\"" .
-            "$class>" . prepare4display($post['post_title']) . "</a></div>";
+        $thread_structure .= "<a href=\"viewthread.php?".api_get_cidreq().
+            "&forum=".$forumId."&thread=".$threadId.
+            "&post=".$post['post_id']."&origin=$origin$count_loop\"".
+            "$class>".prepare4display($post['post_title'])."</a></div>";
         $prev_next_array[] = $post['post_id'];
     }
     $count++;
@@ -148,43 +148,43 @@ 
 $class_next = '';
 
 // Links
-$first_href = $forumUrl . 'viewthread.php?' . api_get_cidreq() .
-    '&forum=' . $forumId . '&thread=' . $threadId .
-    '&gradebook=' . $gradebook . '&id=1&post=' . $prev_next_array[0];
-$last_href 	= $forumUrl . 'viewthread.php?' . api_get_cidreq() .
-    '&forum=' . $forumId . '&thread=' . $threadId .
-    '&gradebook=' . $gradebook . '&post=' . $prev_next_array[$max-1];
-$prev_href	= $forumUrl . 'viewthread.php?' . api_get_cidreq() .
-    '&forum=' . $forumId . '&thread=' . $threadId .
-    '&gradebook=' . $gradebook . '&post=' . $prev_next_array[$prev_id];
-$next_href	= $forumUrl . 'viewthread.php?' . api_get_cidreq() .
-    '&forum=' . $forumId . '&thread=' . $threadId .
-    '&post=' . $prev_next_array[$next_id];
+$first_href = $forumUrl.'viewthread.php?'.api_get_cidreq().
+    '&forum='.$forumId.'&thread='.$threadId.
+    '&gradebook='.$gradebook.'&id=1&post='.$prev_next_array[0];
+$last_href 	= $forumUrl.'viewthread.php?'.api_get_cidreq().
+    '&forum='.$forumId.'&thread='.$threadId.
+    '&gradebook='.$gradebook.'&post='.$prev_next_array[$max - 1];
+$prev_href	= $forumUrl.'viewthread.php?'.api_get_cidreq().
+    '&forum='.$forumId.'&thread='.$threadId.
+    '&gradebook='.$gradebook.'&post='.$prev_next_array[$prev_id];
+$next_href	= $forumUrl.'viewthread.php?'.api_get_cidreq().
+    '&forum='.$forumId.'&thread='.$threadId.
+    '&post='.$prev_next_array[$next_id];
 
 echo '<center style="margin-top: 10px; margin-bottom: 10px;">';
 // Go to: first and previous
 if (((int) $current_id) > 0) {
-    echo '<a href="' . $first_href . '" ' . $class . ' title=' .
-        $first_message . '>' . $first_img . ' ' . $first_message .'</a>';
-    echo '<a href="' . $prev_href . '" ' . $class_prev . ' title=' .
-        $prev_message . '>' . $prev_img . ' ' . $prev_message . '</a>';
+    echo '<a href="'.$first_href.'" '.$class.' title='.
+        $first_message.'>'.$first_img.' '.$first_message.'</a>';
+    echo '<a href="'.$prev_href.'" '.$class_prev.' title='.
+        $prev_message.'>'.$prev_img.' '.$prev_message.'</a>';
 } else {
-    echo '<b><span class="invisible">' .
-        $first_img . ' ' . $first_message . '</b></span>';
-    echo '<b><span class="invisible">' .
-        $prev_img . ' ' . $prev_message . '</b></span>';
+    echo '<b><span class="invisible">'.
+        $first_img.' '.$first_message.'</b></span>';
+    echo '<b><span class="invisible">'.
+        $prev_img.' '.$prev_message.'</b></span>';
 }
 
 // Current counter
-echo ' [ ' . ($current_id + 1) . ' / ' . $max . ' ] ';
+echo ' [ '.($current_id + 1).' / '.$max.' ] ';
 
 // Go to: next and last
 if (($current_id + 1) < $max) {
-    echo '<a href="' . $next_href . '" ' . $class_next . ' title=' . $next_message . '>' . $next_message . ' ' . $next_img . '</a>';
-    echo '<a href="' . $last_href . '" ' . $class . ' title=' . $last_message . '>' . $last_message . ' ' . $last_img . '</a>';
+    echo '<a href="'.$next_href.'" '.$class_next.' title='.$next_message.'>'.$next_message.' '.$next_img.'</a>';
+    echo '<a href="'.$last_href.'" '.$class.' title='.$last_message.'>'.$last_message.' '.$last_img.'</a>';
 } else {
-    echo '<b><span class="invisible">' . $next_message . ' ' . $next_img . '</b></span>';
-    echo '<b><span class="invisible">' . $last_message . ' ' . $last_img . '</b></span>';
+    echo '<b><span class="invisible">'.$next_message.' '.$next_img.'</b></span>';
+    echo '<b><span class="invisible">'.$last_message.' '.$last_img.'</b></span>';
 }
 echo '</center>';
 
@@ -218,18 +218,18 @@ 
 }
 
 if (api_get_course_setting('allow_user_image_forum')) {
-    echo '<br />' . display_user_image($rows[$display_post_id]['user_id'], $name, $origin) . '<br />';
+    echo '<br />'.display_user_image($rows[$display_post_id]['user_id'], $name, $origin).'<br />';
 }
 echo display_user_link(
     $rows[$display_post_id]['user_id'],
     $name,
     $origin,
     $username
-) . "<br />";
+)."<br />";
 
 echo api_convert_and_format_date(
     $rows[$display_post_id]['post_date']
-) . '<br /><br />';
+).'<br /><br />';
 // Get attach id
 $attachment_list = get_attachment($display_post_id);
 $id_attach = !empty($attachment_list) ? $attachment_list['id'] : '';
@@ -248,16 +248,16 @@ 
     )
 ) {
     if ($locked == false) {
-        echo "<a href=\"editpost.php?" . api_get_cidreq() .
-            "&forum=" . $forumId . "&thread=" . $threadId .
-            "&post=" . $rows[$display_post_id]['post_id'] .
-            "&id_attach=" . $id_attach . "\">" .
+        echo "<a href=\"editpost.php?".api_get_cidreq().
+            "&forum=".$forumId."&thread=".$threadId.
+            "&post=".$rows[$display_post_id]['post_id'].
+            "&id_attach=".$id_attach."\">".
             Display::return_icon(
                 'edit.png',
                 get_lang('Edit'),
                 array(),
                 ICON_SIZE_SMALL
-            ) . '</a>';
+            ).'</a>';
     }
 }
 
@@ -273,22 +273,22 @@ 
     sort($id_posts, SORT_NUMERIC);
     reset($id_posts);
     // The post minor
-    $post_minor = (int)$id_posts[0];
+    $post_minor = (int) $id_posts[0];
 }
 
 if (
     GroupManager::is_tutor_of_group(api_get_user_id(), $groupId) ||
     api_is_allowed_to_edit(false, true) &&
-    !(api_is_course_coach() &&$current_forum['session_id'] != $sessionId)
+    !(api_is_course_coach() && $current_forum['session_id'] != $sessionId)
 ) {
     if ($locked == false) {
-        echo "<a href=\"" . api_get_self() . "?" . api_get_cidreq() .
-            "&forum=" . $forumId . "&thread=" . $threadId .
-            "&action=delete&content=post&id=" .
-            $rows[$display_post_id]['post_id'] .
-            "\" onclick=\"javascript:if(!confirm('" .
-            addslashes(api_htmlentities(get_lang('DeletePost'), ENT_QUOTES)) .
-            "')) return false;\">" . Display::return_icon(
+        echo "<a href=\"".api_get_self()."?".api_get_cidreq().
+            "&forum=".$forumId."&thread=".$threadId.
+            "&action=delete&content=post&id=".
+            $rows[$display_post_id]['post_id'].
+            "\" onclick=\"javascript:if(!confirm('".
+            addslashes(api_htmlentities(get_lang('DeletePost'), ENT_QUOTES)).
+            "')) return false;\">".Display::return_icon(
                 'delete.png',
                 get_lang('Delete'),
                 array(),
@@ -308,16 +308,16 @@ 
     );
 
     if (!isset($_GET['id']) && $post_id > $post_minor) {
-        echo "<a href=\"viewthread.php?" . api_get_cidreq() .
-            "&forum=" . $forumId . "&thread=" . $threadId .
-            "&origin=" . $origin . "&action=move&post=" .
-            $rows[$display_post_id]['post_id'] . "\">" .
+        echo "<a href=\"viewthread.php?".api_get_cidreq().
+            "&forum=".$forumId."&thread=".$threadId.
+            "&origin=".$origin."&action=move&post=".
+            $rows[$display_post_id]['post_id']."\">".
             Display::return_icon(
                 'move.png',
                 get_lang('MovePost'),
                 array(),
                 ICON_SIZE_SMALL
-            ) . "</a>";
+            )."</a>";
 
     }
 }
@@ -340,16 +340,16 @@ 
         );
 
         if ($locked == false) {
-            echo "<a href=\"forumqualify.php?" . api_get_cidreq() .
-                "&forum=" . $forumId . "&thread=" . $threadId .
-                "&action=list&post=" . $rows[$display_post_id]['post_id'] .
-                "&user=" . $rows[$display_post_id]['user_id'] . "&user_id=" .
-                $rows[$display_post_id]['user_id'] . "&origin=" . $origin .
-                "&idtextqualify=" . $current_qualify_thread .
-                "\" >" . Display::return_icon(
+            echo "<a href=\"forumqualify.php?".api_get_cidreq().
+                "&forum=".$forumId."&thread=".$threadId.
+                "&action=list&post=".$rows[$display_post_id]['post_id'].
+                "&user=".$rows[$display_post_id]['user_id']."&user_id=".
+                $rows[$display_post_id]['user_id']."&origin=".$origin.
+                "&idtextqualify=".$current_qualify_thread.
+                "\" >".Display::return_icon(
                     'quiz.gif',
                     get_lang('Qualify')
-                ) . "</a>";
+                )."</a>";
         }
     }
 }
@@ -361,33 +361,33 @@ 
         ($current_forum['allow_anonymous'] == 1 && !$_user['user_id'])
     ) {
         if (!api_is_anonymous() && api_is_allowed_to_session_edit(false, true)) {
-            echo '<a href="reply.php?' . api_get_cidreq() .
-                '&forum=' . $forumId . '&thread=' . $threadId .
-                '&post=' . $rows[$display_post_id]['post_id'] .
-                '&action=replymessage&origin=' . $origin . '">' .
+            echo '<a href="reply.php?'.api_get_cidreq().
+                '&forum='.$forumId.'&thread='.$threadId.
+                '&post='.$rows[$display_post_id]['post_id'].
+                '&action=replymessage&origin='.$origin.'">'.
                 Display::return_icon(
                     'message_reply_forum.png',
                     get_lang('ReplyToMessage')
-                ) . "</a>";
-            echo '<a href="reply.php?' . api_get_cidreq() .
-                '&forum=' . $forumId . '&thread=' . $threadId .
-                '&post=' . $rows[$display_post_id]['post_id'] .
-                '&action=quote&origin=' . $origin . '">' .
+                )."</a>";
+            echo '<a href="reply.php?'.api_get_cidreq().
+                '&forum='.$forumId.'&thread='.$threadId.
+                '&post='.$rows[$display_post_id]['post_id'].
+                '&action=quote&origin='.$origin.'">'.
                 Display::return_icon(
                     'quote.gif',
                     get_lang('QuoteMessage')
-                ) . "</a>";
+                )."</a>";
         }
     }
 } else {
     if ($current_forum_category && $current_forum_category['locked'] == 1) {
-        echo get_lang('ForumcategoryLocked') . '<br />';
+        echo get_lang('ForumcategoryLocked').'<br />';
     }
     if ($current_forum['locked'] == 1) {
-        echo get_lang('ForumLocked') . '<br />';
+        echo get_lang('ForumLocked').'<br />';
     }
     if ($current_thread['locked'] == 1) {
-        echo get_lang('ThreadLocked') . '<br />';
+        echo get_lang('ThreadLocked').'<br />';
     }
 }
 
@@ -406,17 +406,17 @@ 
     $rows[$display_post_id]['post_notification'] == '1' AND
     $rows[$display_post_id]['poster_id'] == $_user['user_id']
 ) {
-    $post_image .= Display::return_icon('forumnotification.gif',get_lang('YouWillBeNotified'));
+    $post_image .= Display::return_icon('forumnotification.gif', get_lang('YouWillBeNotified'));
 }
 // The post title
-echo "<td class=\"$titleclass\">" .
-    prepare4display($rows[$display_post_id]['post_title']) . "</td>";
+echo "<td class=\"$titleclass\">".
+    prepare4display($rows[$display_post_id]['post_title'])."</td>";
 echo "</tr>";
 
 // The post message
 echo "<tr>";
-echo "<td class=\"$messageclass\">" .
-    prepare4display($rows[$display_post_id]['post_text']) . "</td>";
+echo "<td class=\"$messageclass\">".
+    prepare4display($rows[$display_post_id]['post_text'])."</td>";
 echo "</tr>";
 
 // The check if there is an attachment
@@ -429,24 +429,24 @@ 
         echo Display::return_icon('attachment.gif', get_lang('Attachment'));
         echo '<a href="download.php?file=';
         echo $realname;
-        echo ' "> ' . $user_filename . ' </a>';
-        echo '<span class="forum_attach_comment">' .
-            Security::remove_XSS($attachment['comment'], STUDENT) . '</span>';
+        echo ' "> '.$user_filename.' </a>';
+        echo '<span class="forum_attach_comment">'.
+            Security::remove_XSS($attachment['comment'], STUDENT).'</span>';
 
         if (
-            ($current_forum['allow_edit'] == 1 &&$rows[$display_post_id]['user_id'] == $_user['user_id']) ||
+            ($current_forum['allow_edit'] == 1 && $rows[$display_post_id]['user_id'] == $_user['user_id']) ||
             (api_is_allowed_to_edit(false, true) && !(api_is_course_coach() && $current_forum['session_id'] != $sessionId))
         ) {
-            echo '&nbsp;&nbsp;<a href="' . api_get_self() . '?' .
-                api_get_cidreq() . '&origin=' .
-                Security::remove_XSS($_GET['origin']) .
-                '&action=delete_attach&id_attach=' .
-                $attachment['id'] . '&forum=' . $forumId .
-                '&thread=' . $threadId .
-                '" onclick="javascript:if(!confirm(\'' .
+            echo '&nbsp;&nbsp;<a href="'.api_get_self().'?'.
+                api_get_cidreq().'&origin='.
+                Security::remove_XSS($_GET['origin']).
+                '&action=delete_attach&id_attach='.
+                $attachment['id'].'&forum='.$forumId.
+                '&thread='.$threadId.
+                '" onclick="javascript:if(!confirm(\''.
                 addslashes(api_htmlentities(
                     get_lang('ConfirmYourChoice'), ENT_QUOTES)
-                ) . '\')) return false;">' . Display::return_icon(
+                ).'\')) return false;">'.Display::return_icon(
                     'delete.gif',
                     get_lang('Delete')
                 ).'</a><br />';