chamilo / chamilo-lms

main/forum/iframe_thread.php

Spacing +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
     echo "<tr>";
     echo "<td rowspan=\"2\" class=\"forum_message_left\">";
     $username = api_htmlentities(sprintf(get_lang('LoginX'), $row['username']), ENT_QUOTES);
-    if ($row['user_id']=='0') {
+    if ($row['user_id'] == '0') {
         $name = $row['poster_name'];
     } else {
         $name = api_get_person_name($row['firstname'], $row['lastname']);

main/forum/viewforumcategory.php

Indentation +1 added lines, -1 removed lines

@@ -500,7 +500,7 @@
                 $html .= '</div>';
                 $html .= '</div></div>';
             }
-           echo $html;
+            echo $html;
         }
     }
     if (count($forum_list) == 0) {

Spacing +43 added lines, -43 removed lines

@@ -52,17 +52,17 @@ 
 $origin = '';
 
 if (isset($_GET['origin'])) {
-    $origin =  Security::remove_XSS($_GET['origin']);
+    $origin = Security::remove_XSS($_GET['origin']);
 }
 
 /* Header and Breadcrumbs */
 $gradebook = null;
 if (isset($_SESSION['gradebook'])) {
-    $gradebook=	$_SESSION['gradebook'];
+    $gradebook = $_SESSION['gradebook'];
 }
 
 if (!empty($gradebook) && $gradebook == 'view') {
-    $interbreadcrumb[] = array (
+    $interbreadcrumb[] = array(
         'url' => '../gradebook/'.$_SESSION['gradebook_dest'],
         'name' => get_lang('ToolGradebook')
     );
@@ -72,16 +72,16 @@ 
 
 $current_forum_category = get_forum_categories($_GET['forumcategory']);
 $interbreadcrumb[] = array(
-    'url' => 'index.php?gradebook=' . $gradebook . '&search='
+    'url' => 'index.php?gradebook='.$gradebook.'&search='
         . Security::remove_XSS(urlencode(isset($_GET['search']) ? $_GET['search'] : '')),
     'name' => get_lang('Forum')
 );
 
 if (!empty($_GET['action']) && !empty($_GET['content'])) {
-    if ($_GET['action']=='add' && $_GET['content']=='forum' ) {
+    if ($_GET['action'] == 'add' && $_GET['content'] == 'forum') {
         $interbreadcrumb[] = array(
             'url' => 'viewforumcategory.php?forumcategory='
-                . $current_forum_category['cat_id'] . '&origin=' . $origin,
+                . $current_forum_category['cat_id'].'&origin='.$origin,
             'name' => $current_forum_category['cat_title']
         );
         $interbreadcrumb[] = array(
@@ -96,7 +96,7 @@ 
     );
 }
 
-if ($origin=='learnpath') {
+if ($origin == 'learnpath') {
     Display::display_reduced_header();
 } else {
     Display::display_header(null);
@@ -111,7 +111,7 @@ 
 // then the user is not allowed here.
 if (
     !api_is_allowed_to_edit(false, true) AND
-    ( $current_forum_category && $current_forum_category['visibility'] == 0)
+    ($current_forum_category && $current_forum_category['visibility'] == 0)
 ) {
     api_not_allowed();
 }
@@ -121,10 +121,10 @@ 
 $html .= '<div class="actions">';
 $html .= '<a href="index.php?gradebook='.$gradebook.'&'.api_get_cidreq().'">'.
     Display::return_icon('back.png', get_lang('BackToForumOverview'), '', ICON_SIZE_MEDIUM).'</a>';
-if (api_is_allowed_to_edit(false,true)) {
-    $html .= '<a href="' . api_get_self() . '?' . api_get_cidreq() . '&forumcategory='
-        . $current_forum_category['cat_id'] . '&action=add&content=forum"> '
-        . Display::return_icon('new_forum.png', get_lang('AddForum'), '', ICON_SIZE_MEDIUM) . '</a>';
+if (api_is_allowed_to_edit(false, true)) {
+    $html .= '<a href="'.api_get_self().'?'.api_get_cidreq().'&forumcategory='
+        . $current_forum_category['cat_id'].'&action=add&content=forum"> '
+        . Display::return_icon('new_forum.png', get_lang('AddForum'), '', ICON_SIZE_MEDIUM).'</a>';
 }
 $html .= search_link();
 $html .= '</div>';
@@ -184,7 +184,7 @@ 
     $forum_categories_list = '';
     $forumId = $forum_category['cat_id'];
     $forumTitle = $forum_category['cat_title'];
-    $linkForumCategory = 'viewforumcategory.php?' . api_get_cidreq() . '&forumcategory=' . strval(intval($forumId));
+    $linkForumCategory = 'viewforumcategory.php?'.api_get_cidreq().'&forumcategory='.strval(intval($forumId));
     $descriptionCategory = $forum_category['cat_comment'];
     $icoCategory = Display::return_icon(
         'forum_blue.png',
@@ -195,16 +195,16 @@ 
 
     if (api_is_allowed_to_edit(false, true) && !($forum_category['session_id'] == 0 && $sessionId != 0)) {
 
-        $iconsEdit = '<a href="' . api_get_self() . '?' . api_get_cidreq() . '&forumcategory='
-            . Security::remove_XSS($_GET['forumcategory']) . '&action=edit&content=forumcategory&id='
-            . '' . $forumId . '">'
-            . Display::return_icon('edit.png', get_lang('Edit'), array(), ICON_SIZE_SMALL) . '</a>';
-        $iconsEdit .= '<a href="' . api_get_self() . '?' . api_get_cidreq() . '&forumcategory='
+        $iconsEdit = '<a href="'.api_get_self().'?'.api_get_cidreq().'&forumcategory='
+            . Security::remove_XSS($_GET['forumcategory']).'&action=edit&content=forumcategory&id='
+            . ''.$forumId.'">'
+            . Display::return_icon('edit.png', get_lang('Edit'), array(), ICON_SIZE_SMALL).'</a>';
+        $iconsEdit .= '<a href="'.api_get_self().'?'.api_get_cidreq().'&forumcategory='
             . Security::remove_XSS($_GET['forumcategory'])
-            . '&action=delete&content=forumcategory&id=' . $forumId
+            . '&action=delete&content=forumcategory&id='.$forumId
             . "\" onclick=\"javascript:if(!confirm('"
             . addslashes(api_htmlentities(get_lang('DeleteForumCategory'), ENT_QUOTES))
-            . "')) return false;\">" . Display::return_icon('delete.png', get_lang('Delete'), array(), ICON_SIZE_SMALL)
+            . "')) return false;\">".Display::return_icon('delete.png', get_lang('Delete'), array(), ICON_SIZE_SMALL)
             . '</a>';
         $iconsEdit .= return_visible_invisible_icon(
             'forumcategory', $forum_category['cat_id'],
@@ -244,7 +244,7 @@ 
     );
 
 
-    if ($descriptionCategory != '' && trim($descriptionCategory)!= '&nbsp;')
+    if ($descriptionCategory != '' && trim($descriptionCategory) != '&nbsp;')
     {
         $html .= '<div class="forum-description">'.$descriptionCategory.'</div>';
     }
@@ -284,7 +284,7 @@ 
             // SHOULD WE SHOW THIS PARTICULAR FORUM
             // you are teacher => show forum
 
-            if (api_is_allowed_to_edit(false,true)) {
+            if (api_is_allowed_to_edit(false, true)) {
                 //echo 'teacher';
                 $show_forum = true;
             } else {
@@ -351,9 +351,9 @@ 
                         ? $all_groups[$forum['forum_of_group']]['id']
                         : null;
                     $group_title = api_substr($my_all_groups_forum_name, 0, 30);
-                    $forum_title_group_addition = ' (<a href="../group/group_space.php?' . api_get_cidreq()
-                        . '&gidReq=' . $my_all_groups_forum_id . '" class="forum_group_link">'
-                        . get_lang('GoTo') . ' ' . $group_title . '</a>)';
+                    $forum_title_group_addition = ' (<a href="../group/group_space.php?'.api_get_cidreq()
+                        . '&gidReq='.$my_all_groups_forum_id.'" class="forum_group_link">'
+                        . get_lang('GoTo').' '.$group_title.'</a>)';
                 } else {
                     $forum_title_group_addition = '';
                 }
@@ -371,7 +371,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>' . $my_number_threads . ' ' . get_lang('ForumThreads') . '</p></div>';
+                $html .= '<div class="number-post">'.$forum_image.'<p>'.$my_number_threads.' '.get_lang('ForumThreads').'</p></div>';
                 $html .= '</div>';
 
                 $html .= '<div class="col-md-9">';
@@ -386,7 +386,7 @@ 
                     'a',
                     $forum['forum_title'].$session_displayed,
                     array(
-                        'href' => 'viewforum.php?' . api_get_cidreq()
+                        'href' => 'viewforum.php?'.api_get_cidreq()
                             . "&gidReq={$forum['forum_of_group']}&forum={$forum['forum_id']}&origin=$origin&search="
                             . Security::remove_XSS(urlencode(isset($_GET['search']) ? $_GET['search'] : '')),
                         'class' => return_visible_invisible($forum['visibility'])
@@ -394,7 +394,7 @@ 
                 );
                 $html .= Display::tag(
                     'h3',
-                    $linkForum . ' ' . $forum_title_group_addition,
+                    $linkForum.' '.$forum_title_group_addition,
                     array(
                         'class' => 'title'
                     )
@@ -410,19 +410,19 @@ 
                 $html .= '</div>';
                 $html .= '<div class="col-md-6">';
 
-                $iconEmpty='';
+                $iconEmpty = '';
 
                 // The number of topics and posts.
                 if ($forum['forum_of_group'] !== '0') {
                     $newPost = '';
                     if (is_array($my_whatsnew_post_info) && !empty($my_whatsnew_post_info)) {
-                        $newPost = ' ' . Display::return_icon('alert.png', get_lang('Forum'), null, ICON_SIZE_SMALL);
+                        $newPost = ' '.Display::return_icon('alert.png', get_lang('Forum'), null, ICON_SIZE_SMALL);
                     } else {
                         $newPost = $iconEmpty;
                     }
                 } else {
                     if (is_array($my_whatsnew_post_info) && !empty($my_whatsnew_post_info)) {
-                        $newPost = ' ' . Display::return_icon('alert.png', get_lang('Forum'), null, ICON_SIZE_SMALL);
+                        $newPost = ' '.Display::return_icon('alert.png', get_lang('Forum'), null, ICON_SIZE_SMALL);
                     } else {
                         $newPost = $iconEmpty;
                     }
@@ -430,7 +430,7 @@ 
 
                 $html .= '<div class="row">';
                 $html .= '<div class="col-md-2">';
-                $html .= $newPost . '</div>';
+                $html .= $newPost.'</div>';
 
                 // the last post in the forum
                 if ($forum['last_poster_name'] != '') {
@@ -442,9 +442,9 @@ 
                 }
                 $html .= '<div class="col-md-6">';
                 if (!empty($forum['last_post_id'])) {
-                    $html .= Display::return_icon('post-item.png', null, null, ICON_SIZE_TINY) . ' ';
+                    $html .= Display::return_icon('post-item.png', null, null, ICON_SIZE_TINY).' ';
                     $html .= api_convert_and_format_date($forum['last_post_date'])
-                        . ' ' . get_lang('By') . ' '
+                        . ' '.get_lang('By').' '
                         . display_user_link($poster_id, $name);
                 }
                 $html .= '</div>';
@@ -454,13 +454,13 @@ 
                     api_is_allowed_to_edit(false, true) &&
                     !($forum['session_id'] == 0 && $sessionId != 0)
                 ) {
-                    $html .= '<a href="' . api_get_self() . '?' . api_get_cidreq() . '&forumcategory='
+                    $html .= '<a href="'.api_get_self().'?'.api_get_cidreq().'&forumcategory='
                         . Security::remove_XSS($_GET['forumcategory'])
-                        . '&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() . '&forumcategory='
+                        . '&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().'&forumcategory='
                         . Security::remove_XSS($_GET['forumcategory'])
-                        . '&action=delete&content=forum&id=' . $forum['forum_id']
+                        . '&action=delete&content=forum&id='.$forum['forum_id']
                         . "\" onclick=\"javascript:if(!confirm('"
                         . addslashes(api_htmlentities(get_lang('DeleteForum'), ENT_QUOTES))
                         . "')) return false;\">"
@@ -484,15 +484,15 @@ 
                 $iconnotify = 'notification_mail_na.png';
 
                 if (is_array(isset($_SESSION['forum_notification']['forum']) ? $_SESSION['forum_notification']['forum'] : null)) {
-                    if (in_array($forum['forum_id'],$_SESSION['forum_notification']['forum'])) {
+                    if (in_array($forum['forum_id'], $_SESSION['forum_notification']['forum'])) {
                         $iconnotify = 'notification_mail.png';
                     }
                 }
 
                 if (!api_is_anonymous()) {
-                    $html .= '<a href="' . api_get_self() . '?' . api_get_cidreq() . '&forumcategory='
-                        . Security::remove_XSS($_GET['forumcategory']) . '&action=notify&content=forum&id='
-                        . $forum['forum_id'] . '">' . Display::return_icon($iconnotify, get_lang('NotifyMe')) . '</a>';
+                    $html .= '<a href="'.api_get_self().'?'.api_get_cidreq().'&forumcategory='
+                        . Security::remove_XSS($_GET['forumcategory']).'&action=notify&content=forum&id='
+                        . $forum['forum_id'].'">'.Display::return_icon($iconnotify, get_lang('NotifyMe')).'</a>';
                 }
                 $html .= '</div>';
                 $html .= '</div>';

main/forum/reply.php

Spacing +8 added lines, -8 removed lines

@@ -30,7 +30,7 @@ 
 
 $origin = '';
 if (isset($_GET['origin'])) {
-    $origin =  Security::remove_XSS($_GET['origin']);
+    $origin = Security::remove_XSS($_GET['origin']);
     $origin_string = '&origin='.$origin;
 }
 
@@ -44,8 +44,8 @@ 
 // We are getting all the information about the current forum and forum category.
 // Note pcool: I tried to use only one sql statement (and function) for this,
 // but the problem is that the visibility of the forum AND forum cateogory are stored in the item_property table.
-$current_thread	= get_thread_information($_GET['thread']); // Note: This has to be validated that it is an existing thread.
-$current_forum	= get_forum_information($current_thread['forum_id']); // Note: This has to be validated that it is an existing forum.
+$current_thread = get_thread_information($_GET['thread']); // Note: This has to be validated that it is an existing thread.
+$current_forum = get_forum_information($current_thread['forum_id']); // Note: This has to be validated that it is an existing forum.
 $current_forum_category = get_forumcategory_information(Security::remove_XSS($current_forum['forum_category']));
 
 /* Is the user allowed here? */
@@ -80,19 +80,19 @@ 
 /* Breadcrumbs */
 
 $gradebook = null;
-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')
     );
 }
 
 if ($origin == 'group') {
-    $_clean['toolgroup'] = (int)$_SESSION['toolgroup'];
+    $_clean['toolgroup'] = (int) $_SESSION['toolgroup'];
     $group_properties  = GroupManager :: get_group_properties($_clean['toolgroup']);
     $interbreadcrumb[] = array(
         'url' => '../group/group.php?'.api_get_cidreq(),
@@ -188,7 +188,7 @@ 
 /* End new display forum */
 // The form for the reply
 $my_action   = isset($_GET['action']) ? Security::remove_XSS($_GET['action']) : '';
-$my_post     = isset($_GET['post']) ?   Security::remove_XSS($_GET['post']) : '';
+$my_post     = isset($_GET['post']) ? Security::remove_XSS($_GET['post']) : '';
 $my_elements = isset($_SESSION['formelements']) ? $_SESSION['formelements'] : '';
 $values = show_add_post_form(
     $current_forum,
@@ -202,7 +202,7 @@ 
     //@todo split the show_add_post_form function
     $origin = isset($_GET['origin']) && $_GET['origin'] === 'learnpath' ? 'learnpath' : null;
 
-    $url = 'viewthread.php?' . http_build_query([
+    $url = 'viewthread.php?'.http_build_query([
         'forum' => $current_thread['forum_id'],
         'gradebook' => $gradebook,
         'thread' => intval($_GET['thread']),

main/forum/editpost.php

Spacing +1 added lines, -1 removed lines

@@ -142,7 +142,7 @@
 
 if (!api_is_allowed_to_edit(null, true) &&
     (
-        ($current_forum_category && $current_forum_category['locked'] <> 0 ) ||
+        ($current_forum_category && $current_forum_category['locked'] <> 0) ||
         $current_forum['locked'] <> 0 ||
         $current_thread['locked'] <> 0
     )

main/forum/viewpost.php

Indentation +2 added lines, -2 removed lines

@@ -1,8 +1,8 @@
 <?php
 /* For licensing terms, see /license.txt */
 /**
- * @package chamilo.forum
- */
+     * @package chamilo.forum
+     */
 
 require_once '../inc/global.inc.php';
 

Spacing +4 added lines, -4 removed lines

@@ -28,7 +28,7 @@ 
 // Are we in a lp ?
 $origin = '';
 if (isset($_GET['origin'])) {
-    $origin =  Security::remove_XSS($_GET['origin']);
+    $origin = Security::remove_XSS($_GET['origin']);
 }
 
 /* MAIN DISPLAY SECTION */
@@ -45,12 +45,12 @@ 
 
 /* Header and Breadcrumbs */
 
-if (isset($_SESSION['gradebook'])){
+if (isset($_SESSION['gradebook'])) {
     $gradebook = $_SESSION['gradebook'];
 }
 
 if (!empty($gradebook) && $gradebook == 'view') {
-    $interbreadcrumb[] = array (
+    $interbreadcrumb[] = array(
         'url' => '../gradebook/'.$_SESSION['gradebook_dest'],
         'name' => get_lang('ToolGradebook')
     );
@@ -183,7 +183,7 @@ 
     echo "\t<tr>\n\t\t<th style=\"padding-left:5px;\" align=\"left\" colspan=\"6\">";
     echo '<span class="forum_title">'.prepare4display($current_thread['thread_title']).'</span><br />';
 
-    if ($origin!='learnpath') {
+    if ($origin != 'learnpath') {
         echo '<span class="forum_low_description">'.prepare4display($current_forum_category['cat_title']).' - ';
     }
 

main/forum/forumqualify.php

Spacing +24 added lines, -24 removed lines

@@ -59,12 +59,12 @@ 
 /*
     Header and Breadcrumbs
 */
-if (isset($_SESSION['gradebook'])){
-    $gradebook=	$_SESSION['gradebook'];
+if (isset($_SESSION['gradebook'])) {
+    $gradebook = $_SESSION['gradebook'];
 }
 
-if (!empty($gradebook) && $gradebook=='view') {
-    $interbreadcrumb[]= array (
+if (!empty($gradebook) && $gradebook == 'view') {
+    $interbreadcrumb[] = array(
         'url' => '../gradebook/'.$_SESSION['gradebook_dest'],
         'name' => get_lang('ToolGradebook')
     );
@@ -88,7 +88,7 @@ 
             "name" => prepare4display($currentForum['forum_title'])
         );
         if ($message <> 'PostDeletedSpecial') {
-            $interbreadcrumb[]= array(
+            $interbreadcrumb[] = array(
                 "url" => "viewthread.php?forum=".Security::remove_XSS($_GET['forum'])."&gradebook=".$gradebook."&thread=".Security::remove_XSS($_GET['thread']),
                 "name" => prepare4display($currentThread['thread_title'])
             );
@@ -119,8 +119,8 @@ 
         );
 
         if ($message <> 'PostDeletedSpecial') {
-            if (isset($_GET['gradebook']) and $_GET['gradebook']=='view') {
-                $info_thread=get_thread_information(Security::remove_XSS($_GET['thread']));
+            if (isset($_GET['gradebook']) and $_GET['gradebook'] == 'view') {
+                $info_thread = get_thread_information(Security::remove_XSS($_GET['thread']));
                 $interbreadcrumb[] = array(
                     "url" => "viewthread.php?".api_get_cidreq()."&forum=".$info_thread['forum_id']."&thread=".Security::remove_XSS($_GET['thread']),
                     "name" => prepare4display($currentThread['thread_title'])
@@ -133,7 +133,7 @@ 
             }
         }
         // the last element of the breadcrumb navigation is already set in interbreadcrumb, so give empty string
-        $interbreadcrumb[]=array("url" => "#","name" => get_lang('QualifyThread'));
+        $interbreadcrumb[] = array("url" => "#", "name" => get_lang('QualifyThread'));
         Display :: display_header('');
     }
 }
@@ -143,7 +143,7 @@ 
 */
 $action = isset($_GET['action']) ? $_GET['action'] : '';
 
-if ($action =='delete' &&
+if ($action == 'delete' &&
     isset($_GET['content']) &&
     isset($_GET['id']) && api_is_allowed_to_edit(false, true)
 ) {
@@ -240,8 +240,8 @@ 
     if (isset($rows)) {
         $counter = 1;
         foreach ($rows as $row) {
-            if ($row['status']=='0') {
-                $style =" id = 'post".$post_en."' class=\"hide-me\" style=\"border:1px solid red; display:none; background-color:#F7F7F7; width:95%; margin: 0px 0px 4px 40px; \" ";
+            if ($row['status'] == '0') {
+                $style = " id = 'post".$post_en."' class=\"hide-me\" style=\"border:1px solid red; display:none; background-color:#F7F7F7; width:95%; margin: 0px 0px 4px 40px; \" ";
             } else {
                 $style = "";
                 $post_en = $row['post_parent_id'];
@@ -258,20 +258,20 @@ 
 
             echo "<div ".$style."><table class=\"data_table\">";
 
-            if ($row['visible']=='0') {
-                $titleclass='forum_message_post_title_2_be_approved';
-                $messageclass='forum_message_post_text_2_be_approved';
-                $leftclass='forum_message_left_2_be_approved';
+            if ($row['visible'] == '0') {
+                $titleclass = 'forum_message_post_title_2_be_approved';
+                $messageclass = 'forum_message_post_text_2_be_approved';
+                $leftclass = 'forum_message_left_2_be_approved';
             } else {
-                $titleclass='forum_message_post_title';
-                $messageclass='forum_message_post_text';
-                $leftclass='forum_message_left';
+                $titleclass = 'forum_message_post_title';
+                $messageclass = 'forum_message_post_text';
+                $leftclass = 'forum_message_left';
             }
 
             echo "<tr>";
             echo "<td rowspan=\"3\" class=\"$leftclass\">";
 
-            echo '<br /><b>'.  api_convert_and_format_date($row['post_date'], DATE_TIME_FORMAT_LONG).'</b><br />';
+            echo '<br /><b>'.api_convert_and_format_date($row['post_date'], DATE_TIME_FORMAT_LONG).'</b><br />';
 
             echo "</td>";
 
@@ -292,7 +292,7 @@ 
                 $realname = $attachment_list['path'];
                 $user_filename = $attachment_list['filename'];
 
-                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 $realname;
                 echo ' "> '.$user_filename.' </a>';
@@ -311,8 +311,8 @@ 
 
     // Show past data
     if (api_is_allowed_to_edit() && $counter > 0) {
-        if (isset($_GET['gradebook'])){
-            $view_gradebook='&gradebook=view';
+        if (isset($_GET['gradebook'])) {
+            $view_gradebook = '&gradebook=view';
         }
         echo '<h4>'.get_lang('QualificationChangesHistory').'</h4>';
         if (isset($_GET['type']) && $_GET['type'] == 'false') {
@@ -342,12 +342,12 @@ 
             );
             $table_list .= '</td></tr>';
         }
-        $table_list.= '</table>';
+        $table_list .= '</table>';
 
         echo $table_list;
     }
 }
 
-if ($origin!='learnpath') {
+if ($origin != 'learnpath') {
     Display :: display_footer();
 }

main/forum/editthread.php

Spacing +4 added lines, -4 removed lines

@@ -47,7 +47,7 @@ 
 }
 
 if (!empty($gradebook) && $gradebook == 'view') {
-    $interbreadcrumb[] = array (
+    $interbreadcrumb[] = array(
         'url' => '../gradebook/'.Security::remove_XSS($_SESSION['gradebook_dest']),
         'name' => get_lang('ToolGradebook')
     );
@@ -55,7 +55,7 @@ 
 
 if (!empty($_GET['gidReq'])) {
     $toolgroup = intval($_GET['gidReq']);
-    Session::write('toolgroup',$toolgroup);
+    Session::write('toolgroup', $toolgroup);
 }
 
 $threadId = isset($_GET['thread']) ? intval($_GET['thread']) : 0;
@@ -114,7 +114,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' => $currentForum['forum_title']);
-    $interbreadcrumb[] = array('url' => 'newthread.php?'.$cidreq.'&forum='.Security::remove_XSS($_GET['forum']),'name' => get_lang('EditThread'));
+    $interbreadcrumb[] = array('url' => 'newthread.php?'.$cidreq.'&forum='.Security::remove_XSS($_GET['forum']), 'name' => get_lang('EditThread'));
 } else {
     $interbreadcrumb[] = array('url' => 'index.php?'.$cidreq, 'name' => $nameTools);
     $interbreadcrumb[] = array('url' => 'viewforumcategory.php?'.$cidreq.'&forumcategory='.$currentForumCategory['cat_id'], 'name' => $currentForumCategory['cat_title']);
@@ -163,7 +163,7 @@ 
 echo '<div class="actions">';
 echo '<span style="float:right;">'.search_link().'</span>';
 echo '<a href="viewforum.php?forum='.intval($_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>';
 
 $threadData = getThreadInfo($threadId, $cId);

main/forum/viewthread_flat.inc.php

Spacing +31 added lines, -31 removed lines

@@ -66,9 +66,9 @@ 
 
                         $buttonReply = Display::tag(
                             'a',
-                            '<em class="fa fa-reply"></em> ' . get_lang('ReplyToMessage'),
+                            '<em class="fa fa-reply"></em> '.get_lang('ReplyToMessage'),
                             array(
-                                'href' => 'reply.php?' . api_get_cidreq()
+                                'href' => 'reply.php?'.api_get_cidreq()
                                     . "&forum=$clean_forum_id&thread=$clean_thread_id&post="
                                     . "{$row['post_id']}&action=replymessage&origin=$origin",
                                 'class' => 'btn btn-primary'
@@ -77,9 +77,9 @@ 
 
                         $buttonQuote = Display::tag(
                             'a',
-                            '<em class="fa fa-quote-left"></em> ' . get_lang('QuoteMessage'),
+                            '<em class="fa fa-quote-left"></em> '.get_lang('QuoteMessage'),
                             array(
-                                'href' => 'reply.php?' . api_get_cidreq()
+                                'href' => 'reply.php?'.api_get_cidreq()
                                     . "&forum=$clean_forum_id&thread=$clean_thread_id"
                                     . "&post={$row['post_id']}&action=quote&origin=$origin",
                                 'class' => 'btn btn-success'
@@ -92,21 +92,21 @@ 
                 if (($current_forum_category && $current_forum_category['locked'] == 1)) {
                     $closedPost = Display::tag(
                         'div',
-                        '<em class="fa fa-exclamation-triangle"></em> ' . get_lang('ForumcategoryLocked'),
+                        '<em class="fa fa-exclamation-triangle"></em> '.get_lang('ForumcategoryLocked'),
                         array('class' => 'alert alert-warning post-closed')
                     );
                 }
                 if ($current_forum['locked'] == 1) {
                     $closedPost = Display::tag(
                         'div',
-                        '<em class="fa fa-exclamation-triangle"></em> ' . get_lang('ForumLocked'),
+                        '<em class="fa fa-exclamation-triangle"></em> '.get_lang('ForumLocked'),
                         array('class' => 'alert alert-warning post-closed')
                     );
                 }
                 if ($current_thread['locked'] == 1) {
                     $closedPost = Display::tag(
                         'div',
-                        '<em class="fa fa-exclamation-triangle"></em> ' . get_lang('ThreadLocked'),
+                        '<em class="fa fa-exclamation-triangle"></em> '.get_lang('ThreadLocked'),
                         array('class' => 'alert alert-warning post-closed')
                     );
                 }
@@ -121,7 +121,7 @@ 
 
             if ($origin != 'learnpath') {
                 if (api_get_course_setting('allow_user_image_forum')) {
-                    $html .= '<div class="thumbnail">' . display_user_image($row['user_id'], $name) . '</div>';
+                    $html .= '<div class="thumbnail">'.display_user_image($row['user_id'], $name).'</div>';
                 }
                 $html .= Display::tag(
                     'h4',
@@ -129,7 +129,7 @@ 
                     array('class' => 'title-username')
                 );
             } else {
-                $name = Display::tag('strong', "#" . $postCount--, ['class' => 'text-info']) . " | $name";
+                $name = Display::tag('strong', "#".$postCount--, ['class' => 'text-info'])." | $name";
 
                 $html .= Display::tag(
                     'p',
@@ -172,10 +172,10 @@ 
             ) {
                 if (api_is_allowed_to_session_edit(false, true)) {
                     if ($locked == false) {
-                        $iconEdit .= "<a href=\"editpost.php?" . api_get_cidreq() . "&forum=" . $clean_forum_id
-                            . "&thread=" . $clean_thread_id . "&post=" . $row['post_id'] . "&origin="
-                            . $origin . "&edit=edition&id_attach=" . $id_attach . "\">"
-                            . Display::return_icon('edit.png', get_lang('Edit'), array(), ICON_SIZE_SMALL) . "</a>";
+                        $iconEdit .= "<a href=\"editpost.php?".api_get_cidreq()."&forum=".$clean_forum_id
+                            . "&thread=".$clean_thread_id."&post=".$row['post_id']."&origin="
+                            . $origin."&edit=edition&id_attach=".$id_attach."\">"
+                            . Display::return_icon('edit.png', get_lang('Edit'), array(), ICON_SIZE_SMALL)."</a>";
                     }
                 }
             }
@@ -187,10 +187,10 @@ 
                 ) {
 
                     if ($locked == false) {
-                        $iconEdit .= "<a href=\"" . api_get_self() . "?" . api_get_cidreq() . "&forum="
-                            . $clean_forum_id . "&thread=" . $clean_thread_id
-                            . "&action=delete&content=post&id=" . $row['post_id'] . "&origin="
-                            . $origin . "\" onclick=\"javascript:if(!confirm('"
+                        $iconEdit .= "<a href=\"".api_get_self()."?".api_get_cidreq()."&forum="
+                            . $clean_forum_id."&thread=".$clean_thread_id
+                            . "&action=delete&content=post&id=".$row['post_id']."&origin="
+                            . $origin."\" onclick=\"javascript:if(!confirm('"
                             . addslashes(api_htmlentities(get_lang('DeletePost'), ENT_QUOTES))
                             . "')) return false;\">"
                             . Display::return_icon('delete.png', get_lang('Delete'), array(), ICON_SIZE_SMALL)
@@ -210,9 +210,9 @@ 
                     );
                     $iconEdit .= "";
                     if ($increment > 0) {
-                        $iconEdit .= "<a href=\"viewthread.php?" . api_get_cidreq() . "&forum=" . $clean_forum_id
-                            . "&thread=" . $clean_thread_id . "&action=move&post=" . $row['post_id']
-                            . "&origin=" . $origin . "\">"
+                        $iconEdit .= "<a href=\"viewthread.php?".api_get_cidreq()."&forum=".$clean_forum_id
+                            . "&thread=".$clean_thread_id."&action=move&post=".$row['post_id']
+                            . "&origin=".$origin."\">"
                             . Display::return_icon('move.png', get_lang('MovePost'), array(), ICON_SIZE_SMALL)
                             . "</a>";
                     }
@@ -245,16 +245,16 @@ 
                     $userCanEdit = true;
                 }
                 if ($increment > 0 && $locked == false && $userCanEdit) {
-                    $iconEdit .= "<a href=\"forumqualify.php?" . api_get_cidreq() . "&forum=" . $my_forum_id
-                        . "&thread=" . $clean_thread_id . "&action=list&post=" . $row['post_id']
-                        . "&user=" . $row['poster_id'] . "&user_id=" . $row['poster_id']
-                        . "&origin=" . $origin . "&idtextqualify=" . $current_qualify_thread . "\" >"
+                    $iconEdit .= "<a href=\"forumqualify.php?".api_get_cidreq()."&forum=".$my_forum_id
+                        . "&thread=".$clean_thread_id."&action=list&post=".$row['post_id']
+                        . "&user=".$row['poster_id']."&user_id=".$row['poster_id']
+                        . "&origin=".$origin."&idtextqualify=".$current_qualify_thread."\" >"
                         . Display::return_icon('quiz.gif', get_lang('Qualify'))
                         . "</a> ";
                 }
             }
             if ($iconEdit != '') {
-                $html .= '<div class="tools-icons">' . $iconEdit . '</div>';
+                $html .= '<div class="tools-icons">'.$iconEdit.'</div>';
             }
 
             $html .= $closedPost;
@@ -312,27 +312,27 @@ 
                     $user_filename = $attachment['filename'];
 
                     $html .= Display::return_icon('attachment.gif', get_lang('Attachment'));
-                    $html .= '<a href="download.php?file=' . $realname . '"> ' . $user_filename . ' </a>';
+                    $html .= '<a href="download.php?file='.$realname.'"> '.$user_filename.' </a>';
 
                     if (($current_forum['allow_edit'] == 1 && $row['user_id'] == $_user['user_id']) ||
                         (api_is_allowed_to_edit(false, true) && !(api_is_course_coach() && $current_forum['session_id'] != $sessionId))
                     ) {
-                        $html .= '&nbsp;&nbsp;<a href="' . api_get_self() . '?' . api_get_cidreq() . '&origin='
-                            . Security::remove_XSS($origin) . '&action=delete_attach&id_attach='
-                            . $attachment['iid'] . '&forum=' . $clean_forum_id . '&thread=' . $clean_thread_id
+                        $html .= '&nbsp;&nbsp;<a href="'.api_get_self().'?'.api_get_cidreq().'&origin='
+                            . Security::remove_XSS($origin).'&action=delete_attach&id_attach='
+                            . $attachment['iid'].'&forum='.$clean_forum_id.'&thread='.$clean_thread_id
                             . '" onclick="javascript:if(!confirm(\''
                             . addslashes(api_htmlentities(get_lang('ConfirmYourChoice'), ENT_QUOTES))
                             . '\')) return false;">'
                             . Display::return_icon('delete.png', get_lang('Delete'), array(), ICON_SIZE_SMALL)
                             . '</a><br />';
                     }
-                    $html .= '<span class="forum_attach_comment" >' . $attachment['comment'] . '</span>';
+                    $html .= '<span class="forum_attach_comment" >'.$attachment['comment'].'</span>';
                 }
             }
 
             $html .= '</div>';
             $html .= '<div class="col-md-5 text-right">';
-            $html .= $buttonReply . ' ' . $buttonQuote;
+            $html .= $buttonReply.' '.$buttonQuote;
             $html .= '</div>';
             $html .= '</div>';
 

main/forum/viewthread_nested.inc.php

Spacing +25 added lines, -25 removed lines

@@ -56,7 +56,7 @@ 
     $indent = $post['indent_cnt'];
 
     $html = '';
-    $html .= '<div class="col-md-offset-' . $indent . '" >';
+    $html .= '<div class="col-md-offset-'.$indent.'" >';
     $html .= '<div class="panel panel-default forum-post">';
     $html .= '<div class="panel-body">';
 
@@ -73,7 +73,7 @@ 
 
     if ($origin != 'learnpath') {
         if (api_get_course_setting('allow_user_image_forum')) {
-            $html .= '<div class="thumbnail">' . display_user_image($post['user_id'], $name, $origin) . '</div>';
+            $html .= '<div class="thumbnail">'.display_user_image($post['user_id'], $name, $origin).'</div>';
         }
 
         $html .= Display::tag(
@@ -120,7 +120,7 @@ 
         (api_is_allowed_to_edit(false, true) && !(api_is_course_coach() && $current_forum['session_id'] != $sessionId))
     ) {
         if ($locked == false) {
-            $iconEdit .= "<a href=\"editpost.php?" . api_get_cidreq()
+            $iconEdit .= "<a href=\"editpost.php?".api_get_cidreq()
                 . "&forum=$clean_forum_id&thread=$clean_thread_id&post={$post['post_id']}&id_attach=$id_attach"
                 . "\">"
                 . Display::return_icon('edit.png', get_lang('Edit'), array(), ICON_SIZE_SMALL)
@@ -134,10 +134,10 @@ 
         !(api_is_course_coach() && $current_forum['session_id'] != $sessionId)
     ) {
         if ($locked == false) {
-            $iconEdit .="<a href=\"" . api_get_self() . "?" . api_get_cidreq()
+            $iconEdit .= "<a href=\"".api_get_self()."?".api_get_cidreq()
                 . "&forum=$clean_forum_id&thread=$clean_thread_id&action=delete&content=post&id={$post['post_id']}"
                 . "\" onclick=\"javascript:if(!confirm('"
-                . addslashes(api_htmlentities(get_lang('DeletePost'), ENT_QUOTES)) . "')) return false;\">"
+                . addslashes(api_htmlentities(get_lang('DeletePost'), ENT_QUOTES))."')) return false;\">"
                 . Display::return_icon('delete.png', get_lang('Delete'), array(), ICON_SIZE_SMALL)
                 . "</a>";
         }
@@ -161,9 +161,9 @@ 
         );
 
         if ($count > 0) {
-            $iconEdit .= "<a href=\"viewthread.php?" . api_get_cidreq()
+            $iconEdit .= "<a href=\"viewthread.php?".api_get_cidreq()
                 . "&forum=$clean_forum_id&thread=$clean_thread_id&action=move&origin=$origin&post={$post['post_id']}"
-                . "\">" . Display::return_icon('move.png', get_lang('MovePost'), array(), ICON_SIZE_SMALL) . "</a>";
+                . "\">".Display::return_icon('move.png', get_lang('MovePost'), array(), ICON_SIZE_SMALL)."</a>";
         }
     }
 
@@ -182,17 +182,17 @@ 
                 '1', $post['user_id'], $_GET['thread']
             );
             if ($locked == false) {
-                $iconEdit .= "<a href=\"forumqualify.php?" . api_get_cidreq()
+                $iconEdit .= "<a href=\"forumqualify.php?".api_get_cidreq()
                     . "&forum=$clean_forum_id&thread=$clean_thread_id&action=list&post={$post['post_id']}"
                     . "&user={$post['user_id']}&user_id={$post['user_id']}&origin=$origin"
                     . "&idtextqualify=$current_qualify_thread"
-                    . "\" >" . Display::return_icon('quiz.gif', get_lang('Qualify')) . "</a>";
+                    . "\" >".Display::return_icon('quiz.gif', get_lang('Qualify'))."</a>";
             }
         }
     }
 
     if ($iconEdit != '') {
-        $html .= '<div class="tools-icons">' . $iconEdit . '</div>';
+        $html .= '<div class="tools-icons">'.$iconEdit.'</div>';
     }
 
     if (($current_forum_category && $current_forum_category['locked'] == 0) &&
@@ -203,9 +203,9 @@ 
 
                 $buttonReply = Display::tag(
                     'a',
-                    '<em class="fa fa-reply"></em> ' . get_lang('ReplyToMessage'),
+                    '<em class="fa fa-reply"></em> '.get_lang('ReplyToMessage'),
                     array(
-                        'href' => 'reply.php?' . api_get_cidreq()
+                        'href' => 'reply.php?'.api_get_cidreq()
                             . "&forum=$clean_forum_id'&thread=$clean_thread_id"
                             . "&post={$post['post_id']}&action=replymessage&origin=$origin",
                         'class' => 'btn btn-primary'
@@ -214,9 +214,9 @@ 
 
                 $buttonQuote = Display::tag(
                     'a',
-                    '<em class="fa fa-quote-left"></em> ' . get_lang('QuoteMessage'),
+                    '<em class="fa fa-quote-left"></em> '.get_lang('QuoteMessage'),
                     array(
-                        'href' => 'reply.php?' . api_get_cidreq()
+                        'href' => 'reply.php?'.api_get_cidreq()
                             . "&forum=$clean_forum_id&thread=$clean_thread_id"
                             . "&post={$post['post_id']}&action=quote&origin=$origin",
                         'class' => 'btn btn-success'
@@ -228,21 +228,21 @@ 
         if ($current_forum_category && $current_forum_category['locked'] == 1) {
             $closedPost = Display::tag(
                 'div',
-                '<em class="fa fa-exclamation-triangle"></em> ' . get_lang('ForumcategoryLocked'),
+                '<em class="fa fa-exclamation-triangle"></em> '.get_lang('ForumcategoryLocked'),
                 array('class' => 'alert alert-warning post-closed')
             );
         }
         if ($current_forum['locked'] == 1) {
             $closedPost = Display::tag(
                 'div',
-                '<em class="fa fa-exclamation-triangle"></em> ' . get_lang('ForumLocked'),
+                '<em class="fa fa-exclamation-triangle"></em> '.get_lang('ForumLocked'),
                 array('class' => 'alert alert-warning post-closed')
             );
         }
         if ($current_thread['locked'] == 1) {
             $closedPost = Display::tag(
                 'div',
-                '<em class="fa fa-exclamation-triangle"></em> ' . get_lang('ThreadLocked'),
+                '<em class="fa fa-exclamation-triangle"></em> '.get_lang('ThreadLocked'),
                 array('class' => 'alert alert-warning post-closed')
             );
         }
@@ -294,24 +294,24 @@ 
             $html .= Display::return_icon('attachment.gif', get_lang('Attachment'));
             $html .= '<a href="download.php?file=';
             $html .= $realname;
-            $html .= ' "> ' . $user_filename . ' </a>';
-            $html .= '<span class="forum_attach_comment" >' . $attachment['comment'] . '</span>';
+            $html .= ' "> '.$user_filename.' </a>';
+            $html .= '<span class="forum_attach_comment" >'.$attachment['comment'].'</span>';
             if (($current_forum['allow_edit'] == 1 && $post['user_id'] == $userId) ||
                 (api_is_allowed_to_edit(false, true) && !(api_is_course_coach() && $current_forum['session_id'] != $sessionId))
             ) {
-                $html .= '&nbsp;&nbsp;<a href="' . api_get_self() . '?' . api_get_cidreq() . '&origin='
-                    . Security::remove_XSS($_GET['origin']) . '&action=delete_attach&id_attach='
-                    . $attachment['iid'] . '&forum=' . $clean_forum_id . '&thread=' . $clean_thread_id
+                $html .= '&nbsp;&nbsp;<a href="'.api_get_self().'?'.api_get_cidreq().'&origin='
+                    . Security::remove_XSS($_GET['origin']).'&action=delete_attach&id_attach='
+                    . $attachment['iid'].'&forum='.$clean_forum_id.'&thread='.$clean_thread_id
                     . '" onclick="javascript:if(!confirm(\''
-                    . addslashes(api_htmlentities(get_lang('ConfirmYourChoice'), ENT_QUOTES)) . '\')) return false;">'
-                    . Display::return_icon('delete.gif', get_lang('Delete')) . '</a><br />';
+                    . addslashes(api_htmlentities(get_lang('ConfirmYourChoice'), ENT_QUOTES)).'\')) return false;">'
+                    . Display::return_icon('delete.gif', get_lang('Delete')).'</a><br />';
             }
         }
     }
 
     $html .= '</div>';
     $html .= '<div class="col-md-6 text-right">';
-    $html .= $buttonReply . ' ' . $buttonQuote;
+    $html .= $buttonReply.' '.$buttonQuote;
     $html .= '</div>';
     $html .= '</div>';
     // The post has been displayed => it can be removed from the what's new array