chamilo / chamilo-lms

main/ticket/report.php

Spacing +20 added lines, -20 removed lines

@@ -21,8 +21,8 @@ 
 $htmlHeadXtra[] = '
 <script language="javascript">
 $(document).ready(function(){
-    $( "#keyword_start_date_start" ).datepicker({ dateFormat: ' . "'yy-mm-dd'" . ' });
-    $( "#keyword_start_date_end" ).datepicker({ dateFormat: ' . "'yy-mm-dd'" . ' });
+    $( "#keyword_start_date_start" ).datepicker({ dateFormat: ' . "'yy-mm-dd'".' });
+    $( "#keyword_start_date_end" ).datepicker({ dateFormat: ' . "'yy-mm-dd'".' });
 });
 function validate(){
     if( $("#keyword_start_date_start").val() != "" &&  $("#keyword_start_date_end").val() != ""){
@@ -91,7 +91,7 @@ 
  */
 function js_str($s)
 {
-    return '"' . addcslashes($s, "\0..\37\"\\") . '"';
+    return '"'.addcslashes($s, "\0..\37\"\\").'"';
 }
 /**
  * This function is to show the ticket form
@@ -101,29 +101,29 @@ 
 {
     global $tools;
     echo '<div class="ticket-form">';
-    echo '<form enctype="multipart/form-data" action="' . api_get_self() . '" method="post" name="send_ticket" id="send_ticket"
+    echo '<form enctype="multipart/form-data" action="'.api_get_self().'" method="post" name="send_ticket" id="send_ticket"
  	onsubmit="return validate()" style="width:100%">';
     $select_course = '<div id="user_request" >
 	 </div>';
     echo $select_course;
     //select status
     $select_tool = '<div class="row"  >
-	<div class="label2"  >' . get_lang('Tool') .':</div>
+	<div class="label2"  >' . get_lang('Tool').':</div>
 	<div class="formw2">';
     $select_tool .= '<select style="width: 95%; " name = "tool" id="tool" >';
 
     foreach ($tools as $tool) {
-        $select_tool .= "<option value = '" . $tool['id'] . "' selected >" . $tool['name'] . "</option>";
+        $select_tool .= "<option value = '".$tool['id']."' selected >".$tool['name']."</option>";
     }
     $select_tool .= "</select>";
     $select_tool .= '</div></div>';
     echo $select_tool;
     echo '<div class="row">
-	      <div class="label2">' . get_lang('From') . ':</div>
+	      <div class="label2">' . get_lang('From').':</div>
               <div class="formw2"><input id="keyword_start_date_start" name="keyword_start_date_start" type="text"></div>
           </div>
 	  <div class="row">
-	      <div class="label2"> ' . get_lang('To') . '</div>
+	      <div class="label2"> ' . get_lang('To').'</div>
 	      <div class="formw2"><input id="keyword_start_date_end" name="keyword_start_date_end" type="text"></div>
 	  </div>';
     echo '</div>';
@@ -131,7 +131,7 @@ 
 		<div class="label2">
 		</div>
 		<div class="formw2">
-			<button class="save" name="report" type="submit" id="btnsubmit" disabled="disabled">' . get_lang('CompleteReport') .'</button>
+			<button class="save" name="report" type="submit" id="btnsubmit" disabled="disabled">' . get_lang('CompleteReport').'</button>
 		</div>
 	</div>';
 }
@@ -146,7 +146,7 @@ 
     $sql = "SELECT COUNT(u.user_id) AS total_number_of_items FROM $user_table u";
     if ((api_is_platform_admin() || api_is_session_admin()) && api_get_multiple_access_url()) {
         $access_url_rel_user_table = Database :: get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
-        $sql.= " INNER JOIN $access_url_rel_user_table url_rel_user ON (u.user_id=url_rel_user.user_id)";
+        $sql .= " INNER JOIN $access_url_rel_user_table url_rel_user ON (u.user_id=url_rel_user.user_id)";
     }
     if (isset($_GET['keyword'])) {
         $keyword = Database::escape_string(trim($_GET['keyword']));
@@ -221,10 +221,10 @@ 
     $webPath = api_get_path(WEB_PATH);
     while ($user = Database::fetch_row($res)) {
         $userPicture = UserManager::getUserPicture($user[0]);
-        $photo = '<img src="' . $userPicture . '" alt="' . api_get_person_name($user[2], $user[3]) . '" title="' . api_get_person_name($user[2], $user[3]) . '" />';
+        $photo = '<img src="'.$userPicture.'" alt="'.api_get_person_name($user[2], $user[3]).'" title="'.api_get_person_name($user[2], $user[3]).'" />';
         $user_id = $user[0];
-        $button = '<a  href="javascript:void(0)" onclick="load_course_list(\'div_' . $user_id . '\',' . $user_id . ')">
-					<img onclick="load_course_list(\'div_' . $user_id . '\',' . $user_id . ')"  src="' . $webPath . 'img/view_more_stats.gif" title="' . get_lang('Courses') . '" alt="' . get_lang('Courses') . '"/>
+        $button = '<a  href="javascript:void(0)" onclick="load_course_list(\'div_'.$user_id.'\','.$user_id.')">
+					<img onclick="load_course_list(\'div_' . $user_id.'\','.$user_id.')"  src="'.$webPath.'img/view_more_stats.gif" title="'.get_lang('Courses').'" alt="'.get_lang('Courses').'"/>
 					</a>&nbsp;&nbsp;';
         $users[] = array(
             $photo,
@@ -242,7 +242,7 @@ 
 
 Display::display_header('Reports');
 echo '<div class="actions">
-    <form action="' . api_get_self() . '" method="get" name="search_simple" id="search_simple">
+    <form action="' . api_get_self().'" method="get" name="search_simple" id="search_simple">
         <input name="user_id_request" id="user_id_request" type="hidden" value="">
         <span><label for="keyword">B&uacute;squeda del usuario: </label><input size="25" name="keyword" type="text" id="keyword"></span>
         <span><button class="search" name="submit" type="submit">Buscar</button></span>
@@ -276,12 +276,12 @@ 
                 u.username , CONCAT(u.lastname, ' ', u.firstname) AS fullname,
                 DATE_SUB(access.access_date,INTERVAL 5 HOUR) AS  access_date,
                 c.title AS course, access_tool AS tool
-            FROM  " . Database::get_main_table(TABLE_STATISTIC_TRACK_E_ACCESS) . " access
-            LEFT JOIN  " . Database::get_main_table(TABLE_MAIN_USER) . " u ON access.access_user_id = u.user_id
-            LEFT JOIN  " . Database::get_main_table(TABLE_MAIN_COURSE) . " c ON access.c_id = c.id
-            WHERE access.c_id = " . $course_info['real_id'] . " AND u.user_id = $user_id ";
+            FROM  " . Database::get_main_table(TABLE_STATISTIC_TRACK_E_ACCESS)." access
+            LEFT JOIN  " . Database::get_main_table(TABLE_MAIN_USER)." u ON access.access_user_id = u.user_id
+            LEFT JOIN  " . Database::get_main_table(TABLE_MAIN_COURSE)." c ON access.c_id = c.id
+            WHERE access.c_id = " . $course_info['real_id']." AND u.user_id = $user_id ";
     if ($tool != '') {
-        $sql.="AND access.access_tool = '$tool' ";
+        $sql .= "AND access.access_tool = '$tool' ";
     }
 
     $start_date = Database::escape_string($_POST['keyword_start_date_start']);
@@ -292,7 +292,7 @@ 
         if ($start_date != '')
             $sql .= "  access_date >= '$start_date'   ";
         if ($end_date != '') {
-            $sql = ($start_date == '') ? $sql : ($sql . " AND ");
+            $sql = ($start_date == '') ? $sql : ($sql." AND ");
             $sql .= "  access_date <= '$end_date'   ";
         }
     }

main/ticket/update_report.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 if (!api_is_allowed_to_edit()) {
     Display::display_error_message($plugin->get_lang("DeniedAccess"));
 } else {
-    $sql = "UPDATE " . Database::get_main_table('rp_reporte_semanas') . "
+    $sql = "UPDATE ".Database::get_main_table('rp_reporte_semanas')."
         SET work_id = '$work_id', forum_id = '$forum_id'
         WHERE  id ='$rs_id'";
     Database::query($sql);

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

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/sortable_table.class.php

Spacing +15 added lines, -15 removed lines

@@ -215,7 +215,7 @@ 
             $params['spacesBeforeSeparator'] = '';
             $params['spacesAfterSeparator'] = '';
             $query_vars = array_keys($_GET);
-            $query_vars_needed = array ($this->param_prefix.'column', $this->param_prefix.'direction', $this->param_prefix.'per_page');
+            $query_vars_needed = array($this->param_prefix.'column', $this->param_prefix.'direction', $this->param_prefix.'per_page');
             if (count($this->additional_parameters) > 0) {
                 $query_vars_needed = array_merge($query_vars_needed, array_keys($this->additional_parameters));
             }
@@ -285,7 +285,7 @@ 
 
         if (!$empty_table) {
             if (!empty($this->additional_parameters)) {
-                foreach($this->additional_parameters as $key => $value) {
+                foreach ($this->additional_parameters as $key => $value) {
                     $html .= '<input type="hidden" name ="'.Security::remove_XSS($key).'" value ="'.Security::remove_XSS($value).'" />';
                 }
             }
@@ -310,7 +310,7 @@ 
                     $html .= '<li><a data-action ="'.$action.'" href="#" onclick="javascript:action_click(this, \''.$table_id.'\');">'.$label.'</a></li>';
                 }
                 $html .= '</ul>';
-                $html .= '</div>';//btn-group
+                $html .= '</div>'; //btn-group
                 $html .= '</div>'; //toolbar
             } else {
                 $html .= $form;
@@ -334,7 +334,7 @@ 
             }
         }
 
-        return '<div class="table-responsive">' . $html . '</div>';
+        return '<div class="table-responsive">'.$html.'</div>';
     }
 
     /**
@@ -388,7 +388,7 @@ 
 
             $html .= '<div class="clear"></div>';
             if (count($this->form_actions) > 0) {
-                $script= '<script>
+                $script = '<script>
                             /*<![CDATA[*/
                             function setCheckbox(value) {
                                  d = document.form_'.$this->table_name.';
@@ -410,7 +410,7 @@ 
             }
         }
         // Getting the items of the table
-        $items = $this->get_clean_html(false);    //no sort
+        $items = $this->get_clean_html(false); //no sort
 
         // Generation of style classes must be improved. Maybe we need a a table name to create style on the fly:
         // i.e: .whoisonline_table_grid_container instead of  .grid_container
@@ -478,7 +478,7 @@ 
 
             $html .= '<div class="clear"></div>';
             if (count($this->form_actions) > 0) {
-                $script= '<script>
+                $script = '<script>
                             /*<![CDATA[*/
                             function setCheckbox(value) {
                                  d = document.form_'.$this->table_name.';
@@ -586,7 +586,7 @@ 
                     $counter = 0;
                     foreach ($row as $index => $rowInfo) {
                         if (!isset($this->columnsToHide[$index])) {
-                            $newRow[$counter] = $rowInfo ;
+                            $newRow[$counter] = $rowInfo;
                             $counter++;
                         }
                     }
@@ -601,7 +601,7 @@ 
         }
 
         if ($this->odd_even_rows_enabled == true) {
-            $this->altRowAttributes(0, array ('class' => 'row_odd'), array ('class' => 'row_even'), true);
+            $this->altRowAttributes(0, array('class' => 'row_odd'), array('class' => 'row_even'), true);
         }
 
         foreach ($this->th_attributes as $column => $attributes) {
@@ -663,11 +663,11 @@ 
         }
         $result[] = '<select name="'.$this->param_prefix.'per_page" onchange="javascript: this.form.submit();">';
         for ($nr = 10; $nr <= min(50, $total_number_of_items); $nr += 10) {
-            $result[] = '<option value="'.$nr.'" '. ($nr == $this->per_page ? 'selected="selected"' : '').'>'.$nr.'</option>';
+            $result[] = '<option value="'.$nr.'" '.($nr == $this->per_page ? 'selected="selected"' : '').'>'.$nr.'</option>';
         }
         // @todo no limits
         //if ($total_number_of_items < 500) {
-            $result[] = '<option value="'.$total_number_of_items.'" '. ($total_number_of_items == $this->per_page ? 'selected="selected"' : '').'>'.api_ucfirst(get_lang('All')).'</option>';
+            $result[] = '<option value="'.$total_number_of_items.'" '.($total_number_of_items == $this->per_page ? 'selected="selected"' : '').'>'.api_ucfirst(get_lang('All')).'</option>';
         //}
         $result[] = '</select>';
         $result[] = '<noscript>';
@@ -785,7 +785,7 @@ 
      */
     public function get_additional_url_paramstring()
     {
-        $param_string_parts = array ();
+        $param_string_parts = array();
         if (is_array($this->additional_parameters) && count($this->additional_parameters) > 0) {
             foreach ($this->additional_parameters as $key => & $value) {
                 $param_string_parts[] = urlencode($key).'='.urlencode($value);
@@ -798,7 +798,7 @@ 
                 //$param[$tablename.'_direction'] = $_GET[$tablename.'_direction'];
                 $my_get_direction = $_GET[$tablename.'_direction'];
                 if (!in_array($my_get_direction, array('ASC', 'DESC'))) {
-                     $param[$tablename.'_direction'] =  'ASC';
+                     $param[$tablename.'_direction'] = 'ASC';
                 } else {
                     $param[$tablename.'_direction'] = $my_get_direction;
                 }
@@ -812,7 +812,7 @@ 
             if (isset($_GET[$tablename.'_column'])) {
                 $param[$tablename.'_column'] = intval($_GET[$tablename.'_column']);
             }
-            $param_string_parts = array ();
+            $param_string_parts = array();
             foreach ($param as $key => & $value) {
                 $param_string_parts[] = urlencode($key).'='.urlencode($value);
             }
@@ -833,7 +833,7 @@ 
         $param[$this->param_prefix.'page_nr'] = $this->page_nr;
         $param[$this->param_prefix.'per_page'] = $this->per_page;
         $param[$this->param_prefix.'column'] = $this->column;
-        $param_string_parts = array ();
+        $param_string_parts = array();
         foreach ($param as $key => & $value) {
             $param_string_parts[] = urlencode($key).'='.urlencode($value);
         }

main/inc/lib/TicketManager.php

Spacing +55 added lines, -55 removed lines

@@ -75,11 +75,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 ";
@@ -236,7 +236,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)) {
@@ -355,7 +355,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(
@@ -396,32 +396,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>';
 
@@ -656,10 +656,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;
 
@@ -760,11 +760,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);
             }
@@ -787,7 +787,7 @@ 
                     '$ticket_id',
                     '$message_id',
                     '$message_attch_id',
-                    '" . $file_attach['size'] . "',
+                    '".$file_attach['size']."',
                     '$user_id',
                     '$now',
                     '$user_id',
@@ -796,7 +796,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,
             );
         }
@@ -999,23 +999,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>';
                 }
             }
 
@@ -1037,11 +1037,11 @@ 
             $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>';
@@ -1062,17 +1062,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'],
@@ -1088,9 +1088,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;
@@ -1324,8 +1324,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;
             }
@@ -1342,10 +1342,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;
@@ -1357,12 +1357,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;
@@ -1386,10 +1386,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()) {
@@ -1508,10 +1508,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);
 
@@ -1596,7 +1596,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);
@@ -1656,8 +1656,8 @@ 
             $row['assigned_date'] = api_convert_and_format_date(
                 api_get_local_time($row['assigned_date']), '%d/%m/%y-%H:%M:%S', _api_get_timezone()
             );
-            $row['assignuser'] = $row['user_id'] != 0 ? ('<a href="' . $webpath . 'main/admin/user_information.php?user_id=' . $row['user_id'] . '"  target="_blank">' . $assignuser['username'] . '</a>') : get_lang('Unassign');
-            $row['insertuser'] = '<a href="' . $webpath . 'main/admin/user_information.php?user_id=' . $row['sys_insert_user_id'] . '"  target="_blank">' . $insertuser['username'] . '</a>';
+            $row['assignuser'] = $row['user_id'] != 0 ? ('<a href="'.$webpath.'main/admin/user_information.php?user_id='.$row['user_id'].'"  target="_blank">'.$assignuser['username'].'</a>') : get_lang('Unassign');
+            $row['insertuser'] = '<a href="'.$webpath.'main/admin/user_information.php?user_id='.$row['sys_insert_user_id'].'"  target="_blank">'.$insertuser['username'].'</a>';
             $history[] = $row;
         }
 
@@ -1853,7 +1853,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'