chamilo / chamilo-lms

main/ticket/course_user_list.php

Spacing +2 added lines, -2 removed lines

@@ -38,8 +38,8 @@
 $courseSelect = Display::select('course_id', $arrCourseList, 0, array(), false);
 $courseControl = Display::div($courseSelect, array('class' => 'controls'));
 
-$userDiv = Display::div($userLabel . " " . $userControl, array('class' => 'control-group'));
-$courseDiv = Display::div($courseLabel . " " . $courseControl, array('class' => 'control-group'));
+$userDiv = Display::div($userLabel." ".$userControl, array('class' => 'control-group'));
+$courseDiv = Display::div($courseLabel." ".$courseControl, array('class' => 'control-group'));
 echo $userDiv;
 echo $courseDiv;
 

main/forum/index.php

Spacing +37 added lines, -37 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>';
@@ -408,7 +408,7 @@ 
                         // Showing the image
                         if (!empty($forum['forum_image'])) {
 
-                            $image_path = api_get_path(WEB_COURSE_PATH). api_get_course_path(). '/upload/forum/images/'. $forum['forum_image'];
+                            $image_path = api_get_path(WEB_COURSE_PATH).api_get_course_path().'/upload/forum/images/'.$forum['forum_image'];
                             $image_size = api_getimagesize($image_path);
                             $img_attributes = '';
                             if (!empty($image_size)) {
@@ -455,15 +455,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 = '';
                         }
@@ -475,7 +475,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>';
 
@@ -491,9 +491,9 @@ 
                             'a',
                             $forum['forum_title'],
                             [
-                                'href' => 'viewforum.php?' . api_get_cidreq()
-                                    . '&gidReq=' . intval($groupid)
-                                    . '&forum=' . intval($forum['forum_id']),
+                                'href' => 'viewforum.php?'.api_get_cidreq()
+                                    . '&gidReq='.intval($groupid)
+                                    . '&forum='.intval($forum['forum_id']),
                                 'class' => return_visible_invisible(strval(intval($forum['visibility'])))
                             ]
                         );
@@ -505,7 +505,7 @@ 
                             }
                         }
 
-                        $html .= '<h3 class="title">' . $iconForum . $linkForum . '</h3>';
+                        $html .= '<h3 class="title">'.$iconForum.$linkForum.'</h3>';
                         $html .= Display::tag(
                             'p',
                             Security::remove_XSS($forum['forum_comment']),
@@ -520,9 +520,9 @@ 
 
                         // 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,
@@ -533,7 +533,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,
@@ -575,9 +575,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,
@@ -591,13 +591,13 @@ 
                         if (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;\">"
@@ -638,8 +638,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>';
                         }
@@ -656,7 +656,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>';
         }

plugin/azure_active_directory/src/AzureActiveDirectory.php

Doc Comments +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     /**
      * Instance the plugin
      * @staticvar null $result
-     * @return Tour
+     * @return AzureActiveDirectory
      */
     static function create()
     {

Spacing +3 added lines, -3 removed lines

@@ -63,13 +63,13 @@ 
         $settings = [];
 
         foreach ($settingsInfo as $settingInfo) {
-            $variable = str_replace($this->get_name() . '_', '', $settingInfo['variable']);
+            $variable = str_replace($this->get_name().'_', '', $settingInfo['variable']);
 
             $settings[$variable] = $settingInfo['selected_value'];
         }
 
         $url = "https://login.microsoftonline.com/{$settings[self::SETTING_TENANT]}/oauth2/v2.0/";
-        $callback = api_get_path(WEB_PLUGIN_PATH) . $this->get_name() . '/src/callback.php';
+        $callback = api_get_path(WEB_PLUGIN_PATH).$this->get_name().'/src/callback.php';
 
         if ($urlType === self::URL_TYPE_SIGNOUT) {
             $action = 'logout';
@@ -97,6 +97,6 @@ 
             ];
         }
 
-        return $url . $action . '?' . http_build_query($urlParams);
+        return $url.$action.'?'.http_build_query($urlParams);
     }
 }
\ No newline at end of file

plugin/azure_active_directory/src/callback.php

Spacing +5 added lines, -5 removed lines

@@ -1,9 +1,9 @@ 
 <?php
-require dirname(__FILE__) . '/../../../main/inc/global.inc.php';
-require_once dirname(__FILE__) . '/../../../main/auth/external_login/functions.inc.php';
+require dirname(__FILE__).'/../../../main/inc/global.inc.php';
+require_once dirname(__FILE__).'/../../../main/auth/external_login/functions.inc.php';
 
 if (isset($_POST['error']) || empty($_REQUEST)) {
-    header('Location: ' . api_get_path(WEB_PATH) . 'index.php?logout=logout');
+    header('Location: '.api_get_path(WEB_PATH).'index.php?logout=logout');
     exit;
 }
 
@@ -25,7 +25,7 @@ 
     'username' => $jwtPayload->emails[0],
     'language' => 'en',
     'password' => 'azure_active_directory',
-    'auth_source' => 'azure_active_directory ' . $jwtPayload->idp,
+    'auth_source' => 'azure_active_directory '.$jwtPayload->idp,
     'extra' => array()
 );
 
@@ -51,5 +51,5 @@ 
     $_SESSION['_user'] = $_user;
 }
 
-header('Location: ' . api_get_path(WEB_PATH));
+header('Location: '.api_get_path(WEB_PATH));
 exit;

main/ticket/index.php

Spacing +1 added lines, -1 removed lines

@@ -7,5 +7,5 @@
  */
 
 require_once __DIR__.'/../inc/global.inc.php';
-header('Location:' . api_get_path(WEB_CODE_PATH) . 'ticket/tickets.php');
+header('Location:'.api_get_path(WEB_CODE_PATH).'ticket/tickets.php');
 exit;

main/inc/lib/TicketManager.php

Indentation +4 added lines, -4 removed lines

@@ -1480,7 +1480,7 @@ 
         ];
         Database::update($table, $newParams, ['id = ? ' => $ticketId]);
 
-         self::sendNotification(
+            self::sendNotification(
             $ticketId,
             $userId,
             get_lang('TicketUpdated'),
@@ -1901,9 +1901,9 @@ 
         return $list;
     }
 
-     /**
-     * @return array
-     */
+        /**
+         * @return array
+         */
     public static function getPriorityList()
     {
         $projects = Database::getManager()->getRepository('ChamiloTicketBundle:Priority')->findAll();

Doc Comments +9 added lines, -9 removed lines

@@ -642,10 +642,10 @@ 
     /**
      * Insert message between Users and Admins
      * @param $ticket_id
-     * @param $subject
-     * @param $content
+     * @param string $subject
+     * @param string $content
      * @param $file_attachments
-     * @param $user_id
+     * @param integer $user_id
      * @param string $status
      * @param bool $sendConfirmation
      * @return bool
@@ -744,9 +744,9 @@ 
     /**
      * Attachment files when a message is sent
      * @param $file_attach
-     * @param $ticket_id
-     * @param $message_id
-     * @param $message_attch_id
+     * @param integer $ticket_id
+     * @param integer $message_id
+     * @param integer $message_attch_id
      * @return array
      */
     public static function save_message_attachment_file(
@@ -2006,7 +2006,7 @@ 
     }
 
     /**
-     * @param $id
+     * @param integer $id
      * @return Project
      */
     public static function getProject($id)
@@ -2125,7 +2125,7 @@ 
     }
 
     /**
-     * @param $id
+     * @param integer $id
      * @return Project
      */
     public static function getStatus($id)
@@ -2249,7 +2249,7 @@ 
     }
 
     /**
-     * @param $id
+     * @param integer $id
      * @return Priority
      */
     public static function getPriority($id)

Spacing +57 added lines, -57 removed lines

@@ -78,11 +78,11 @@ 
         $sql = "SELECT id, name, description, total_tickets
                 FROM $table";
 
-        if (!in_array($direction, array('ASC','DESC'))) {
+        if (!in_array($direction, array('ASC', 'DESC'))) {
             $direction = 'ASC';
         }
         $column = intval($column);
-        $from 	= intval($from);
+        $from = intval($from);
         $numberItems = intval($numberItems);
 
         //$sql .= " ORDER BY col$column $direction ";
@@ -239,7 +239,7 @@ 
     public static function get_all_tickets_status()
     {
         $table_support_status = Database::get_main_table(TABLE_TICKET_STATUS);
-        $sql = "SELECT * FROM " . $table_support_status;
+        $sql = "SELECT * FROM ".$table_support_status;
         $result = Database::query($sql);
         $types = array();
         while ($row = Database::fetch_assoc($result)) {
@@ -363,7 +363,7 @@ 
         $ticket_id = Database::insert($table_support_tickets, $params);
 
         if ($ticket_id) {
-            $ticket_code = "A" . str_pad($ticket_id, 11, '0', STR_PAD_LEFT);
+            $ticket_code = "A".str_pad($ticket_id, 11, '0', STR_PAD_LEFT);
 
             Display::addFlash(Display::return_message(
                 sprintf(
@@ -404,32 +404,32 @@ 
             $helpDeskMessage =
                 '<table>
                         <tr>
-                            <td width="100px"><b>' . get_lang('User') . '</b></td>
-                            <td width="400px">' . $user['firstname']. ' ' . $user['lastname'] . '</td>
+                            <td width="100px"><b>' . get_lang('User').'</b></td>
+                            <td width="400px">' . $user['firstname'].' '.$user['lastname'].'</td>
                         </tr>
                         <tr>
-                            <td width="100px"><b>' . get_lang('Username') . '</b></td>
-                            <td width="400px">' . $user['username'] . '</td>
+                            <td width="100px"><b>' . get_lang('Username').'</b></td>
+                            <td width="400px">' . $user['username'].'</td>
                         </tr>
                         <tr>
-                            <td width="100px"><b>' . get_lang('Email') . '</b></td>
-                            <td width="400px">' . $user['email'] . '</td>
+                            <td width="100px"><b>' . get_lang('Email').'</b></td>
+                            <td width="400px">' . $user['email'].'</td>
                         </tr>
                         <tr>
-                            <td width="100px"><b>' . get_lang('Phone') . '</b></td>
-                            <td width="400px">' . $user['phone'] . '</td>
+                            <td width="100px"><b>' . get_lang('Phone').'</b></td>
+                            <td width="400px">' . $user['phone'].'</td>
                         </tr>
                         <tr>
-                            <td width="100px"><b>' . get_lang('Date') . '</b></td>
-                            <td width="400px">' . api_convert_and_format_date($now, DATE_TIME_FORMAT_LONG) . '</td>
+                            <td width="100px"><b>' . get_lang('Date').'</b></td>
+                            <td width="400px">' . api_convert_and_format_date($now, DATE_TIME_FORMAT_LONG).'</td>
                         </tr>
                         <tr>
-                            <td width="100px"><b>' . get_lang('Title') . '</b></td>
-                            <td width="400px">' . $subject . '</td>
+                            <td width="100px"><b>' . get_lang('Title').'</b></td>
+                            <td width="400px">' . $subject.'</td>
                         </tr>
                         <tr>
-                            <td width="100px"><b>' . get_lang('Description') . '</b></td>
-                            <td width="400px">' . $content . '</td>
+                            <td width="100px"><b>' . get_lang('Description').'</b></td>
+                            <td width="400px">' . $content.'</td>
                         </tr>
                     </table>';
 
@@ -666,10 +666,10 @@ 
         $table_support_tickets = Database::get_main_table(TABLE_TICKET_TICKET);
         $table_support_message_attachments = Database::get_main_table(TABLE_TICKET_MESSAGE_ATTACHMENTS);
         if ($sendConfirmation) {
-            $form = '<form action="ticket_details.php?ticket_id=' . $ticket_id . '" id="confirmticket" method="POST" >
-                         <p>' . get_lang('TicketWasThisAnswerSatisfying') . '</p>
-                         <button class="btn btn-primary responseyes" name="response" id="responseyes" value="1">' . get_lang('Yes') . '</button>
-                         <button class="btn btn-danger responseno" name="response" id="responseno" value="0">' . get_lang('No') . '</button>
+            $form = '<form action="ticket_details.php?ticket_id='.$ticket_id.'" id="confirmticket" method="POST" >
+                         <p>' . get_lang('TicketWasThisAnswerSatisfying').'</p>
+                         <button class="btn btn-primary responseyes" name="response" id="responseyes" value="1">' . get_lang('Yes').'</button>
+                         <button class="btn btn-danger responseno" name="response" id="responseno" value="0">' . get_lang('No').'</button>
                      </form>';
             $content .= $form;
 
@@ -770,11 +770,11 @@ 
         } else {
             $new_file_name = uniqid('');
             $path_attachment = api_get_path(SYS_ARCHIVE_PATH);
-            $path_message_attach = $path_attachment . 'plugin_ticket_messageattch/';
+            $path_message_attach = $path_attachment.'plugin_ticket_messageattch/';
             if (!file_exists($path_message_attach)) {
                 @mkdir($path_message_attach, api_get_permissions_for_new_directories(), true);
             }
-            $new_path = $path_message_attach . $new_file_name;
+            $new_path = $path_message_attach.$new_file_name;
             if (is_uploaded_file($file_attach['tmp_name'])) {
                 @copy($file_attach['tmp_name'], $new_path);
             }
@@ -797,7 +797,7 @@ 
                     '$ticket_id',
                     '$message_id',
                     '$message_attch_id',
-                    '" . $file_attach['size'] . "',
+                    '".$file_attach['size']."',
                     '$user_id',
                     '$now',
                     '$user_id',
@@ -806,7 +806,7 @@ 
             Database::query($sql);
 
             return array(
-                'path' => $path_message_attach . $safe_new_file_name,
+                'path' => $path_message_attach.$safe_new_file_name,
                 'filename' => $safe_file_name,
             );
         }
@@ -979,23 +979,23 @@ 
             $unread = Database::fetch_object($result_unread)->unread;*/
 
             $userInfo = api_get_user_info($row['sys_insert_user_id']);
-            $hrefUser = $webPath . 'main/admin/user_information.php?user_id=' . $userInfo['user_id'];
+            $hrefUser = $webPath.'main/admin/user_information.php?user_id='.$userInfo['user_id'];
             $name = "<a href='$hrefUser'> {$userInfo['username']} </a>";
             $actions = '';
 
             if ($row['assigned_last_user'] != 0) {
                 $assignedUserInfo = api_get_user_info($row['assigned_last_user']);
                 if (!empty($assignedUserInfo)) {
-                    $hrefResp = $webPath . 'main/admin/user_information.php?user_id=' . $assignedUserInfo['user_id'];
+                    $hrefResp = $webPath.'main/admin/user_information.php?user_id='.$assignedUserInfo['user_id'];
                     $row['assigned_last_user'] = "<a href='$hrefResp'> {$assignedUserInfo['username']} </a>";
                 } else {
                     $row['assigned_last_user'] = get_lang('UnknownUser');
                 }
             } else {
                 if ($row['status_id'] !== self::STATUS_FORWARDED) {
-                    $row['assigned_last_user'] = '<span style="color:#ff0000;">' . get_lang('ToBeAssigned') . '</span>';
+                    $row['assigned_last_user'] = '<span style="color:#ff0000;">'.get_lang('ToBeAssigned').'</span>';
                 } else {
-                    $row['assigned_last_user'] = '<span style="color:#00ff00;">' . get_lang('MessageResent') . '</span>';
+                    $row['assigned_last_user'] = '<span style="color:#00ff00;">'.get_lang('MessageResent').'</span>';
                 }
             }
 
@@ -1018,13 +1018,13 @@ 
             $row['col2'] = Display::tip(date_to_str_ago($row['col2']), api_get_local_time($row['col2']));
             if ($isAdmin) {
                 if ($row['priority_id'] === self::PRIORITY_HIGH && $row['status_id'] != self::STATUS_CLOSE) {
-                    $actions .= '<img src="' . $webCodePath . 'img/exclamation.png" border="0" />';
+                    $actions .= '<img src="'.$webCodePath.'img/exclamation.png" border="0" />';
                 }
                 $row['col0'] = Display::return_icon(
                         $img_source,
                         get_lang('Info')
                         ).
-                    '<a href="ticket_details.php?ticket_id=' . $row['col0'] . '">' . $row['code'] . '</a>';
+                    '<a href="ticket_details.php?ticket_id='.$row['col0'].'">'.$row['code'].'</a>';
                 // @todo fix
                 /*if ($row['col7'] == 'PENDIENTE') {
                     $row['col7'] = '<span style="color: #f00; font-weight:bold;">' . $row['col7'] . '</span>';
@@ -1045,17 +1045,17 @@ 
                 $row['col0'] = Display::return_icon(
                     $img_source,
                     get_lang('Info')
-                ) . '<a href="ticket_details.php?ticket_id=' . $row['col0'] . '">' . $row['code'] . '</a>';
+                ).'<a href="ticket_details.php?ticket_id='.$row['col0'].'">'.$row['code'].'</a>';
                 $now = api_strtotime(api_get_utc_datetime());
                 $last_edit_date = api_strtotime($row['sys_lastedit_datetime']);
                 $dif = $now - $last_edit_date;
 
                 if ($dif > 172800 && $row['priority_id'] === self::PRIORITY_NORMAL && $row['status_id'] != self::STATUS_CLOSE) {
-                    $actions .= '<a href="'.api_get_path(WEB_CODE_PATH).'ticket/tickets.php?ticket_id=' . $row['ticket_id'] . '&amp;action=alert">
-                                 <img src="' . Display::returnIconPath('exclamation.png') . '" border="0" /></a>';
+                    $actions .= '<a href="'.api_get_path(WEB_CODE_PATH).'ticket/tickets.php?ticket_id='.$row['ticket_id'].'&amp;action=alert">
+                                 <img src="' . Display::returnIconPath('exclamation.png').'" border="0" /></a>';
                 }
                 if ($row['priority_id'] === self::PRIORITY_HIGH) {
-                    $actions .= '<img src="' . Display::returnIconPath('admin_star.png') . '" border="0" />';
+                    $actions .= '<img src="'.Display::returnIconPath('admin_star.png').'" border="0" />';
                 }
                 $ticket = array(
                     $row['col0'],
@@ -1071,9 +1071,9 @@ 
                                 </a>';
             }*/
             if ($isAdmin) {
-                $ticket['0'] .= '&nbsp;&nbsp;<a  href="javascript:void(0)" onclick="load_history_ticket(\'div_' . $row['ticket_id'] . '\',' . $row['ticket_id'] . ')">
-					<img onclick="load_course_list(\'div_' . $row['ticket_id'] . '\',' . $row['ticket_id'] . ')" onmouseover="clear_course_list (\'div_' . $row['ticket_id'] . '\')" src="' . Display::returnIconPath('history.gif') . '" title="' . get_lang('Historial') . '" alt="' . get_lang('Historial') . '"/>
-					<div class="blackboard_hide" id="div_' . $row['ticket_id'] . '">&nbsp;&nbsp;</div>
+                $ticket['0'] .= '&nbsp;&nbsp;<a  href="javascript:void(0)" onclick="load_history_ticket(\'div_'.$row['ticket_id'].'\','.$row['ticket_id'].')">
+					<img onclick="load_course_list(\'div_' . $row['ticket_id'].'\','.$row['ticket_id'].')" onmouseover="clear_course_list (\'div_'.$row['ticket_id'].'\')" src="'.Display::returnIconPath('history.gif').'" title="'.get_lang('Historial').'" alt="'.get_lang('Historial').'"/>
+					<div class="blackboard_hide" id="div_' . $row['ticket_id'].'">&nbsp;&nbsp;</div>
 					</a>&nbsp;&nbsp;';
             }
             $tickets[] = $ticket;
@@ -1308,8 +1308,8 @@ 
                 }
 
                 $userInfo = api_get_user_info($row['sys_insert_user_id']);
-                $row['user_url'] = '<a href="' . api_get_path(WEB_PATH) . 'main/admin/user_information.php?user_id=' . $userInfo['user_id'] . '">
-                ' . $userInfo['complete_name']. '</a>';
+                $row['user_url'] = '<a href="'.api_get_path(WEB_PATH).'main/admin/user_information.php?user_id='.$userInfo['user_id'].'">
+                ' . $userInfo['complete_name'].'</a>';
                 $ticket['usuario'] = $userInfo;
                 $ticket['ticket'] = $row;
             }
@@ -1326,10 +1326,10 @@ 
             while ($row = Database::fetch_assoc($result)) {
                 $message = $row;
                 $completeName = api_get_person_name($row['firstname'], $row['lastname']);
-                $href = $webPath . 'main/admin/user_information.php?user_id=' . $row['user_id'];
+                $href = $webPath.'main/admin/user_information.php?user_id='.$row['user_id'];
                 // Check if user is an admin
                 $sql_admin = "SELECT user_id FROM $admin_table
-		                      WHERE user_id = '" . intval($message['user_id']) . "'
+		                      WHERE user_id = '".intval($message['user_id'])."'
                               LIMIT 1";
                 $result_admin = Database::query($sql_admin);
                 $message['admin'] = false;
@@ -1341,12 +1341,12 @@ 
                 $sql = "SELECT *
                         FROM $table_support_message_attachments
                         WHERE
-                            message_id = " . $row['id'] . " AND
+                            message_id = ".$row['id']." AND
                             ticket_id = '$ticket_id'  ";
                 $result_attach = Database::query($sql);
                 while ($row2 = Database::fetch_assoc($result_attach)) {
-                    $archiveURL = $archiveURL = $webPath . "plugin/" . PLUGIN_NAME . '/src/download.php?ticket_id=' . $ticket_id . '&file=';
-                    $row2['attachment_link'] = $attach_icon . '&nbsp;<a href="' . $archiveURL . $row2['path'] . '&title=' . $row2['filename'] . '">' . $row2['filename'] . '</a>&nbsp;(' . $row2['size'] . ')';
+                    $archiveURL = $archiveURL = $webPath."plugin/".PLUGIN_NAME.'/src/download.php?ticket_id='.$ticket_id.'&file=';
+                    $row2['attachment_link'] = $attach_icon.'&nbsp;<a href="'.$archiveURL.$row2['path'].'&title='.$row2['filename'].'">'.$row2['filename'].'</a>&nbsp;('.$row2['size'].')';
                     $message['attachments'][] = $row2;
                 }
                 $ticket['messages'][] = $message;
@@ -1370,10 +1370,10 @@ 
         );
         $table_support_tickets = Database::get_main_table(TABLE_TICKET_TICKET);
         $now = api_get_utc_datetime();
-        $sql = "UPDATE " . $table_support_messages . "
+        $sql = "UPDATE ".$table_support_messages."
                 SET status = 'LEI', 
-                sys_lastedit_user_id ='" . api_get_user_id() . "',
-                sys_lastedit_datetime ='" . $now . "'
+                sys_lastedit_user_id ='" . api_get_user_id()."',
+                sys_lastedit_datetime ='" . $now."'
                 WHERE ticket_id ='$ticket_id' ";
 
         if (api_is_platform_admin()) {
@@ -1492,10 +1492,10 @@ 
         $user_id = intval($user_id);
 
         $now = api_get_utc_datetime();
-        $sql = "UPDATE " . $table_support_tickets . " SET
+        $sql = "UPDATE ".$table_support_tickets." SET
                 status_id = '$status_id',
                 sys_lastedit_user_id ='$user_id',
-                sys_lastedit_datetime ='" . $now . "'
+                sys_lastedit_datetime ='".$now."'
                 WHERE ticket_id ='$ticket_id'";
         $result = Database::query($sql);
 
@@ -1580,7 +1580,7 @@ 
         $sql = "UPDATE $table_support_tickets SET
                     status_id = '".self::STATUS_CLOSE."',
                     sys_lastedit_user_id ='$user_id',
-                    sys_lastedit_datetime ='" . $now . "',
+                    sys_lastedit_datetime ='".$now."',
                     end_date ='$now'
                 WHERE id ='$ticket_id'";
         Database::query($sql);
@@ -1634,15 +1634,15 @@ 
 
             if ($row['user_id'] != 0) {
                 $assignuser = api_get_user_info($row['user_id']);
-                $row['assignuser'] = '<a href="' . $webpath . 'main/admin/user_information.php?user_id=' . $row['user_id'] . '"  target="_blank">' .
-                $assignuser['username'] . '</a>';
+                $row['assignuser'] = '<a href="'.$webpath.'main/admin/user_information.php?user_id='.$row['user_id'].'"  target="_blank">'.
+                $assignuser['username'].'</a>';
             } else {
                 $row['assignuser'] = get_lang('Unassign');
             }
             $row['assigned_date'] = date_to_str_ago($row['assigned_date']);
             $insertuser = api_get_user_info($row['sys_insert_user_id']);
-            $row['insertuser'] = '<a href="' . $webpath . 'main/admin/user_information.php?user_id=' . $row['sys_insert_user_id'] . '"  target="_blank">' .
-                $insertuser['username'] . '</a>';
+            $row['insertuser'] = '<a href="'.$webpath.'main/admin/user_information.php?user_id='.$row['sys_insert_user_id'].'"  target="_blank">'.
+                $insertuser['username'].'</a>';
             $history[] = $row;
         }
 
@@ -1838,7 +1838,7 @@ 
         while ($row = Database::fetch_assoc($result)) {
             if ($row['responsable'] != 0) {
                 $row['responsable'] = api_get_user_info($row['responsable']);
-                $row['responsable'] = $row['responsable']['firstname'] . ' ' . $row['responsable']['lastname'];
+                $row['responsable'] = $row['responsable']['firstname'].' '.$row['responsable']['lastname'];
             }
             $row['sys_insert_datetime'] = api_format_date(
                     $row['sys_insert_datetime'], '%d/%m/%y - %I:%M:%S %p'

main/admin/legal_add.php

Indentation +4 added lines, -4 removed lines

@@ -105,8 +105,8 @@ 
 
 if (isset($_POST['language'])) {
 
-	$form->addElement('static', Security::remove_XSS($_POST['language']));
-	$form->addElement('hidden', 'language', Security::remove_XSS($_POST['language']));
+    $form->addElement('static', Security::remove_XSS($_POST['language']));
+    $form->addElement('hidden', 'language', Security::remove_XSS($_POST['language']));
     $form->addHtmlEditor(
         'content',
         get_lang('Content'),
@@ -136,8 +136,8 @@ 
     $form->addElement('html', $buttons);
 
 } else {
-	$form->addElement('select_language', 'language', get_lang('Language'),null,array());
-	$form->addButtonSearch(get_lang('Load'), 'send');
+    $form->addElement('select_language', 'language', get_lang('Language'),null,array());
+    $form->addButtonSearch(get_lang('Load'), 'send');
 
 }
 

Spacing +4 added lines, -4 removed lines

@@ -47,7 +47,7 @@ 
         } else {
             $changes = '';
         }
-        $submit  = $values['send'];
+        $submit = $values['send'];
 
         $default['content'] = $content;
         if (isset($values['language'])) {
@@ -66,7 +66,7 @@ 
                 Display::addFlash(Display::return_message($message));
                 header('Location: legal_list.php?sec_token='.$tok);
                 exit();
-            } elseif ($submit=='preview') {
+            } elseif ($submit == 'preview') {
                 $defaults['type'] = $type;
                 $defaults['content'] = $content;
                 $defaults['changes'] = $changes;
@@ -137,13 +137,13 @@ 
     $form->addElement('html', $buttons);
 
 } else {
-	$form->addElement('select_language', 'language', get_lang('Language'),null,array());
+	$form->addElement('select_language', 'language', get_lang('Language'), null, array());
 	$form->addButtonSearch(get_lang('Load'), 'send');
 
 }
 
 $tool_name = get_lang('AddTermsAndConditions');
-$interbreadcrumb[] = array ("url" => 'index.php', "name" => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array("url" => 'index.php', "name" => get_lang('PlatformAdmin'));
 Display :: display_header($tool_name);
 
 echo '<script>

main/admin/configure_inscription.php

Spacing +17 added lines, -17 removed lines

@@ -46,7 +46,7 @@ 
 $tool_name = get_lang('ConfigureInscription');
 if (!empty($action)) {
     $interbreadcrumb[] = array('url' => 'configure_inscription.php', 'name' => get_lang('ConfigureInscription'));
-    switch($action) {
+    switch ($action) {
         case 'edit_top':
             $tool_name = get_lang('EditTopRegister');
             break;
@@ -141,9 +141,9 @@ 
                 // This request is only the preparation for the update of the home_top
                 $home_top = '';
                 if (is_file($homep.$topf.'_'.$lang.$ext) && is_readable($homep.$topf.'_'.$lang.$ext)) {
-                    $home_top = @(string)file_get_contents($homep.$topf.'_'.$lang.$ext);
+                    $home_top = @(string) file_get_contents($homep.$topf.'_'.$lang.$ext);
                 } elseif (is_file($homep.$topf.$lang.$ext) && is_readable($homep.$topf.$lang.$ext)) {
-                    $home_top = @(string)file_get_contents($homep.$topf.$lang.$ext);
+                    $home_top = @(string) file_get_contents($homep.$topf.$lang.$ext);
                 } else {
                     $errorMsg = get_lang('HomePageFilesNotReadable');
                 }
@@ -184,15 +184,15 @@ 
     if (api_is_western_name_order()) {
         //	FIRST NAME and LAST NAME
         $form->addElement('text', 'firstname', get_lang('FirstName'), array('size' => 40, 'disabled' => 'disabled'));
-        $form->addElement('text', 'lastname',  get_lang('LastName'),  array('size' => 40, 'disabled' => 'disabled'));
+        $form->addElement('text', 'lastname', get_lang('LastName'), array('size' => 40, 'disabled' => 'disabled'));
     } else {
         //	LAST NAME and FIRST NAME
-        $form->addElement('text', 'lastname',  get_lang('LastName'),  array('size' => 40, 'disabled' => 'disabled'));
+        $form->addElement('text', 'lastname', get_lang('LastName'), array('size' => 40, 'disabled' => 'disabled'));
         $form->addElement('text', 'firstname', get_lang('FirstName'), array('size' => 40, 'disabled' => 'disabled'));
     }
     $form->applyFilter('firstname', 'trim');
     $form->applyFilter('lastname', 'trim');
-    $form->addRule('lastname',  get_lang('ThisFieldIsRequired'), 'required');
+    $form->addRule('lastname', get_lang('ThisFieldIsRequired'), 'required');
     $form->addRule('firstname', get_lang('ThisFieldIsRequired'), 'required');
 
     //	EMAIL
@@ -210,10 +210,10 @@ 
     $form->addRule('username', get_lang('ThisFieldIsRequired'), 'required');
     $form->addRule('username', get_lang('UsernameWrong'), 'username');
     $form->addRule('username', get_lang('UserTaken'), 'username_available');
-    $form->addRule('username', sprintf(get_lang('UsernameMaxXCharacters'), (string)USERNAME_MAX_LENGTH), 'maxlength', USERNAME_MAX_LENGTH);
+    $form->addRule('username', sprintf(get_lang('UsernameMaxXCharacters'), (string) USERNAME_MAX_LENGTH), 'maxlength', USERNAME_MAX_LENGTH);
 
     //	PASSWORD
-    $form->addElement('password', 'pass1', get_lang('Pass'),         array('size' => 40, 'disabled' => 'disabled'));
+    $form->addElement('password', 'pass1', get_lang('Pass'), array('size' => 40, 'disabled' => 'disabled'));
     $form->addElement('password', 'pass2', get_lang('Confirmation'), array('size' => 40, 'disabled' => 'disabled'));
     $form->addRule('pass1', get_lang('ThisFieldIsRequired'), 'required');
     $form->addRule('pass2', get_lang('ThisFieldIsRequired'), 'required');
@@ -241,25 +241,25 @@ 
 
     //	EXTENDED FIELDS
     if (api_get_setting('extended_profile') == 'true' &&
-        api_get_setting('extendedprofile_registration','mycomptetences') == 'true'
+        api_get_setting('extendedprofile_registration', 'mycomptetences') == 'true'
     ) {
         $form->addHtmlEditor('competences', get_lang('MyCompetences'), false, false, array('ToolbarSet' => 'Profile', 'Width' => '100%', 'Height' => '130'));
     }
 
     if (api_get_setting('extended_profile') == 'true' &&
-        api_get_setting('extendedprofile_registration','mydiplomas') == 'true'
+        api_get_setting('extendedprofile_registration', 'mydiplomas') == 'true'
     ) {
         $form->addHtmlEditor('diplomas', get_lang('MyDiplomas'), false, false, array('ToolbarSet' => 'Profile', 'Width' => '100%', 'Height' => '130'));
     }
 
     if (api_get_setting('extended_profile') == 'true' &&
-        api_get_setting('extendedprofile_registration','myteach') == 'true'
+        api_get_setting('extendedprofile_registration', 'myteach') == 'true'
     ) {
         $form->addHtmlEditor('teach', get_lang('MyTeach'), false, false, array('ToolbarSet' => 'Profile', 'Width' => '100%', 'Height' => '130'));
     }
 
     if (api_get_setting('extended_profile') == 'true' &&
-        api_get_setting('extendedprofile_registration','mypersonalopenarea') == 'true'
+        api_get_setting('extendedprofile_registration', 'mypersonalopenarea') == 'true'
     ) {
         $form->addHtmlEditor('openarea', get_lang('MyPersonalOpenArea'), false, false, array('ToolbarSet' => 'Profile', 'Width' => '100%', 'Height' => '130'));
     }
@@ -349,14 +349,14 @@ 
 
 $form->setDefaults($defaults);
 
-switch ($action){
+switch ($action) {
     case 'edit_top':
         if ($action == 'edit_top') {
             $name = $topf;
             $open = $home_top;
         } else {
             $name = $newsf;
-            $open = @(string)file_get_contents($homep.$newsf.'_'.$lang.$ext);
+            $open = @(string) file_get_contents($homep.$newsf.'_'.$lang.$ext);
             $open = api_to_system_encoding($open, api_detect_encoding(strip_tags($open)));
         }
 
@@ -366,7 +366,7 @@ 
 
         $default = array();
         $form = new FormValidator('configure_inscription_'.$action, 'post', api_get_self().'?action='.$action, '', array('style' => 'margin: 0px;'));
-        $renderer =& $form->defaultRenderer();
+        $renderer = & $form->defaultRenderer();
         $renderer->setHeaderTemplate('');
         $renderer->setFormTemplate('<form{attributes}><table border="0" cellpadding="5" cellspacing="0" width="100%">{content}</table></form>');
         $renderer->setCustomElementTemplate('<tr><td>{element}</td></tr>');
@@ -385,9 +385,9 @@ 
 
         $open = '';
         if (file_exists($homep.$topf.'_'.$lang.$ext)) {
-            $open = @(string)file_get_contents($homep.$topf.'_'.$lang.$ext);
+            $open = @(string) file_get_contents($homep.$topf.'_'.$lang.$ext);
         } else {
-            $open = @(string)file_get_contents($homep.$topf.$ext);
+            $open = @(string) file_get_contents($homep.$topf.$ext);
         }
         $open = api_to_system_encoding($open, api_detect_encoding(strip_tags($open)));
         if (!empty($open)) {

main/admin/legal_list.php

Indentation +2 added lines, -2 removed lines

@@ -42,10 +42,10 @@
 // this 2 "mask" function are here just because the SortableTable
 function get_legal_data_mask($id, $params = null, $row = null)
 {
-	return LegalManager::get_legal_data($id, $params, $row);
+    return LegalManager::get_legal_data($id, $params, $row);
 }
 function count_mask()
 {
-	return LegalManager::count();
+    return LegalManager::count();
 }
 Display :: display_footer();

Spacing +2 added lines, -2 removed lines

@@ -10,7 +10,7 @@ 
 require_once '../inc/global.inc.php';
 $this_section = SECTION_PLATFORM_ADMIN;
 api_protect_admin_script();
-$interbreadcrumb[] = array ("url" => 'index.php', "name" => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array("url" => 'index.php', "name" => get_lang('PlatformAdmin'));
 $tool_name = get_lang('TermsAndConditions');
 Display :: display_header($tool_name);
 
@@ -19,7 +19,7 @@ 
 // action menu
 echo '<div class="actions">';
 echo '<a href="'.api_get_path(WEB_CODE_PATH).'admin/legal_add.php">'.
-    Display::return_icon('edit.gif',get_lang('EditTermsAndConditions'), '').get_lang('EditTermsAndConditions').'</a>&nbsp;&nbsp;';
+    Display::return_icon('edit.gif', get_lang('EditTermsAndConditions'), '').get_lang('EditTermsAndConditions').'</a>&nbsp;&nbsp;';
 echo '</div>';
 
 $legal_count = LegalManager::count();