chamilo / chamilo-lms

plugin/jcapture/plugin_applet.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 $imageFormat = "PNG";
 $cookies = null;
 foreach (array_keys($_COOKIE) as $cookieName) {
-    $cookies.=bin2hex($cookieName)."=".bin2hex($_COOKIE[$cookieName]).";";
+    $cookies .= bin2hex($cookieName)."=".bin2hex($_COOKIE[$cookieName]).";";
 }
 
 $pageName = 'file';

plugin/ticket/src/myticket.php

Spacing +50 added lines, -50 removed lines

@@ -23,7 +23,7 @@ 
     $.ajax({
         contentType: "application/x-www-form-urlencoded",
         beforeSend: function(object) {
-        $("div#"+div_course).html("<img src=\'' . $webLibPath . 'javascript/indicator.gif\' />"); },
+        $("div#"+div_course).html("<img src=\'' . $webLibPath.'javascript/indicator.gif\' />"); },
         type: "POST",
         url: "ticket_assign_log.php",
         data: "ticket_id="+ticket_id,
@@ -40,8 +40,8 @@ 
 }
 
 $(function() {
-    $( "#keyword_start_date_start" ).datepicker({ dateFormat: ' . "'dd/mm/yy'" . ' });
-    $( "#keyword_start_date_end" ).datepicker({ dateFormat: ' . "'dd/mm/yy'" . ' });
+    $( "#keyword_start_date_start" ).datepicker({ dateFormat: ' . "'dd/mm/yy'".' });
+    $( "#keyword_start_date_end" ).datepicker({ dateFormat: ' . "'dd/mm/yy'".' });
 });
 
 $(document).ready(function() {
@@ -51,10 +51,10 @@ 
 function display_advanced_search_form () {
     if ($("#advanced_search_form").css("display") == "none") {
         $("#advanced_search_form").css("display","block");
-        $("#img_plus_and_minus").html(\'&nbsp;' . Display::return_icon('div_hide.gif', get_lang('Hide'), array('style' => 'vertical-align:middle')) . '&nbsp;' . get_lang('AdvancedSearch') . '\');
+        $("#img_plus_and_minus").html(\'&nbsp;' . Display::return_icon('div_hide.gif', get_lang('Hide'), array('style' => 'vertical-align:middle')).'&nbsp;'.get_lang('AdvancedSearch').'\');
     } else {
         $("#advanced_search_form").css("display","none");
-        $("#img_plus_and_minus").html(\'&nbsp;' . Display::return_icon('div_show.gif', get_lang('Show'), array('style' => 'vertical-align:middle')) . '&nbsp;' . get_lang('AdvancedSearch') . '\');
+        $("#img_plus_and_minus").html(\'&nbsp;' . Display::return_icon('div_show.gif', get_lang('Show'), array('style' => 'vertical-align:middle')).'&nbsp;'.get_lang('AdvancedSearch').'\');
     }
 }
 </script>
@@ -237,42 +237,42 @@ 
     }
     //select categories
     $select_types = '<select class="chzn-select" name = "keyword_category" id="keyword_category" ">';
-    $select_types .= '<option value="">---' . get_lang('Select') . '---</option>';
+    $select_types .= '<option value="">---'.get_lang('Select').'---</option>';
     $types = TicketManager::get_all_tickets_categories();
     foreach ($types as $type) {
-        $select_types.= "<option value = '" . $type['category_id'] . "'>" . $type['name'] . "</option>";
+        $select_types .= "<option value = '".$type['category_id']."'>".$type['name']."</option>";
     }
     $select_types .= "</select>";
     //select admins
     $select_admins = '<select  class ="chzn-select" name = "keyword_admin" id="keyword_admin" ">';
-    $select_admins .= '<option value="">---' . get_lang('Select') . '---</option>';
-    $select_admins .= '<option value = "0">' . $plugin->get_lang('Unassigned') . '</option>';
+    $select_admins .= '<option value="">---'.get_lang('Select').'---</option>';
+    $select_admins .= '<option value = "0">'.$plugin->get_lang('Unassigned').'</option>';
     $admins = UserManager::get_user_list_like(array("status" => "1"), array("username"), true);
     foreach ($admins as $admin) {
-        $select_admins.= "<option value = '" . $admin['user_id'] . "'>" . $admin['lastname'] . " ," . $admin['firstname'] . "</option>";
+        $select_admins .= "<option value = '".$admin['user_id']."'>".$admin['lastname']." ,".$admin['firstname']."</option>";
     }
     $select_admins .= "</select>";
     //select status
     $select_status = '<select  class ="chzn-select" name = "keyword_status" id="keyword_status" >';
-    $select_status .= '<option value="">---' . get_lang('Select') . '---</option>';
+    $select_status .= '<option value="">---'.get_lang('Select').'---</option>';
     $status = TicketManager::get_all_tickets_status();
     foreach ($status as $stat) {
-        $select_status.= "<option value = '" . $stat['status_id'] . "'>" . $stat['name'] . "</option>";
+        $select_status .= "<option value = '".$stat['status_id']."'>".$stat['name']."</option>";
     }
     $select_status .= "</select>";
     //select priority
     $select_priority = '<select  name = "keyword_priority" id="keyword_priority" >';
-    $select_priority .= '<option value="">' . get_lang('All') . '</option>';
-    $select_priority .= '<option value="NRM">' . $plugin->get_lang('PriorityNormal') . '</option>';
-    $select_priority .= '<option value="HGH">' . $plugin->get_lang('PriorityHigh') . '</option>';
-    $select_priority .= '<option value="LOW">' . $plugin->get_lang('PriorityLow') . '</option>';
+    $select_priority .= '<option value="">'.get_lang('All').'</option>';
+    $select_priority .= '<option value="NRM">'.$plugin->get_lang('PriorityNormal').'</option>';
+    $select_priority .= '<option value="HGH">'.$plugin->get_lang('PriorityHigh').'</option>';
+    $select_priority .= '<option value="LOW">'.$plugin->get_lang('PriorityLow').'</option>';
     $select_priority .= "</select>";
 
     //select unread
     $select_unread = '<select  name = "keyword_unread" id="keyword_unread" >';
-    $select_unread .= '<option value="">' . get_lang('All') . '</option>';
-    $select_unread .= '<option value="yes">' . $plugin->get_lang('Unread') . '</option>';
-    $select_unread .= '<option value="no">' . $plugin->get_lang('Read')  . '</option>';
+    $select_unread .= '<option value="">'.get_lang('All').'</option>';
+    $select_unread .= '<option value="yes">'.$plugin->get_lang('Unread').'</option>';
+    $select_unread .= '<option value="no">'.$plugin->get_lang('Read').'</option>';
     $select_unread .= "</select>";
     // Create a search-box
     $form = new FormValidator('search_simple', 'get', '', '', null, false);
@@ -283,49 +283,49 @@ 
     $form->addElement('static', 'search_advanced_link', null,
             '<a href="javascript://" class = "advanced-parameters" onclick="display_advanced_search_form();">'
             . '<span id="img_plus_and_minus">&nbsp;'
-            . Display::return_icon('div_show.gif', get_lang('Show')) . ' '
-            . get_lang('AdvancedSearch') . '</span></a>');
+            . Display::return_icon('div_show.gif', get_lang('Show')).' '
+            . get_lang('AdvancedSearch').'</span></a>');
 
     echo '<div class="actions" >';
     if (api_is_platform_admin()) {
-        echo '<span class="fleft">' .
-                '<a href="' . api_get_path(WEB_PLUGIN_PATH) . 'ticket/src/new_ticket.php">' .
-                    Display::return_icon('add.png', $plugin->get_lang('TckNew'), '', '32') . '</a>' .
-                '<a href="' . api_get_self() . '?action=export' . $get_parameter . $get_parameter2 . '">' .
-                    Display::return_icon('export_excel.png', get_lang('Export'), '', '32') . '</a>' .
+        echo '<span class="fleft">'.
+                '<a href="'.api_get_path(WEB_PLUGIN_PATH).'ticket/src/new_ticket.php">'.
+                    Display::return_icon('add.png', $plugin->get_lang('TckNew'), '', '32').'</a>'.
+                '<a href="'.api_get_self().'?action=export'.$get_parameter.$get_parameter2.'">'.
+                    Display::return_icon('export_excel.png', get_lang('Export'), '', '32').'</a>'.
              '</span>';
     }
     $form->display();
     echo '</div>';
-    echo '<form action="' . api_get_self() . '" method="get" name="advanced_search" id="advanced_search" display:"none">
+    echo '<form action="'.api_get_self().'" method="get" name="advanced_search" id="advanced_search" display:"none">
             <div id="advanced_search_form" style="display: block;">
             <div>
-               <div class="form_header">' . get_lang('AdvancedSearch') . '</div>
+               <div class="form_header">' . get_lang('AdvancedSearch').'</div>
             </div>
             <table >
                <tbody>
                   <tr>
                      <td>
                         <div>
-                           <div class="label2">' . get_lang('Category') . ': </div>
-                           <div class="formw2" style="margin-top: -5px;">' . $select_types . '</div>
+                           <div class="label2">' . get_lang('Category').': </div>
+                           <div class="formw2" style="margin-top: -5px;">' . $select_types.'</div>
                         </div>
                      </td>
                      <td>
                         <div>
-                           <div class="label3">' . get_lang('User') . ': </div>
+                           <div class="label3">' . get_lang('User').': </div>
                            <div class="formw2"><input class="input-width" id="keyword_request_user" name="keyword_request_user" type="text"></div>
                         </div>
                      </td>
                      <td>
                         <div>
-                           <div class="label3">' . $plugin->get_lang('RegisterDate') . ': </div>
+                           <div class="label3">' . $plugin->get_lang('RegisterDate').': </div>
                            <div class="formw2"><input class="input-width" id="keyword_start_date_start" name="keyword_start_date_start" type="text"></div>
                         </div>
                      </td>
                      <td>
                         <div>
-                           <div class="label3"><input type="checkbox" name="keyword_dates" value="1">' . $plugin->get_lang('Untill') . '</div>
+                           <div class="label3"><input type="checkbox" name="keyword_dates" value="1">' . $plugin->get_lang('Untill').'</div>
                            <div class="formw2"><input class="input-width" id="keyword_start_date_end" name="keyword_start_date_end" type="text"></div>
                         </div>
                      </td>
@@ -333,42 +333,42 @@ 
                   <tr >
                      <td>
                         <div>
-                           <div class="label2">' . $plugin->get_lang('AssignedTo') . ': </div>
-                           <div class="formw2 select-margin-top">' . $select_admins . '</div>
+                           <div class="label2">' . $plugin->get_lang('AssignedTo').': </div>
+                           <div class="formw2 select-margin-top">' . $select_admins.'</div>
                         </div>
                      </td>
                      <td>
                         <div>
-                           <div class="label3 remove-margin-top">' . get_lang('Status') . ':</div>
-                           <div class="formw2 select-margin-top">' . $select_status . '</div>
+                           <div class="label3 remove-margin-top">' . get_lang('Status').':</div>
+                           <div class="formw2 select-margin-top">' . $select_status.'</div>
                         </div>
                      </td>
                      <td>
                         <div>
                         <div>
-                           <div class="label3">' . $plugin->get_lang('Priority') . ': </div>
-                           <div class="formw2">' . $select_priority . '</div>
+                           <div class="label3">' . $plugin->get_lang('Priority').': </div>
+                           <div class="formw2">' . $select_priority.'</div>
                         </div>
                      </td>
                      <td>
                         <div>
                            <div>
-                              <div class="label3">' . get_lang('Status') . ': </div>
-                              <div class="formw2">' . $select_unread . '</div>
+                              <div class="label3">' . get_lang('Status').': </div>
+                              <div class="formw2">' . $select_unread.'</div>
                            </div>
                      </td>
                   </tr>
                   <tr>
                   <td>
                   <div >
-                  <div class="label4">' . get_lang('Course') . ': </div>
+                  <div class="label4">' . get_lang('Course').': </div>
                   <div class="formw2">
                   <input id="keyword_course" style="width: 170px;" name="keyword_course" type="text"></div>
                   </div>
                   </td>
                   <td colspan= "3">
                   <div>
-                  <button  name="submit_advanced" type="submit">' . get_lang('AdvancedSearch') . '</button>
+                  <button  name="submit_advanced" type="submit">' . get_lang('AdvancedSearch').'</button>
                   </div>
                   </td>
                   </tr>
@@ -381,12 +381,12 @@ 
 } else {
     if ($plugin->get('allow_student_add') == 'true') {
         echo '<div class="actions" >';
-        echo '<span style="float:right;">' .
-                '<a href="' . api_get_path(WEB_PLUGIN_PATH) . 'ticket/src/new_ticket.php">' .
-                    Display::return_icon('add.png', $plugin->get_lang('TckNew'), '', '32') .
-                '</a>' .
+        echo '<span style="float:right;">'.
+                '<a href="'.api_get_path(WEB_PLUGIN_PATH).'ticket/src/new_ticket.php">'.
+                    Display::return_icon('add.png', $plugin->get_lang('TckNew'), '', '32').
+                '</a>'.
               '</span>';
-        echo '<span style="float:right;">' .
+        echo '<span style="float:right;">'.
         '</span>';
         echo '</div>';
     }
@@ -405,8 +405,8 @@ 
     $table->set_header(8, get_lang('Actions'), true);
     $table->set_header(9, get_lang('Description'), true, array("style" => "width:200px"));
 } else {
-    echo '<center><h1>' . $plugin->get_lang('MyTickets') . '</h1></center>';
-    echo '<center><p>' . $plugin->get_lang('MsgWelcome') . '</p></center>';
+    echo '<center><h1>'.$plugin->get_lang('MyTickets').'</h1></center>';
+    echo '<center><p>'.$plugin->get_lang('MsgWelcome').'</p></center>';
     if (isset($_GET['message'])) {
         Display::display_confirmation_message($plugin->get_lang('TckSuccessSave'));
     }

plugin/ticket/src/index.php

Spacing +1 added lines, -1 removed lines

@@ -7,5 +7,5 @@
  */
 
 require_once '../config.php';
-header('location:' . api_get_path(WEB_PLUGIN_PATH) . PLUGIN_NAME . '/src/myticket.php?message=success');
+header('location:'.api_get_path(WEB_PLUGIN_PATH).PLUGIN_NAME.'/src/myticket.php?message=success');
 exit;

plugin/ticket/src/download.php

Spacing +2 added lines, -2 removed lines

@@ -42,8 +42,8 @@
 $file_url = Database::escape_string($file_url);
 $title = $_GET['title'];
 $path_attachment = api_get_path(SYS_ARCHIVE_PATH);
-$path_message_attach = $path_attachment . 'plugin_ticket_messageattch/';
-$full_file_name = $path_message_attach . $file_url;
+$path_message_attach = $path_attachment.'plugin_ticket_messageattch/';
+$full_file_name = $path_message_attach.$file_url;
 if (Security::check_abs_path($full_file_name, $path_message_attach)) {
     DocumentManager::file_send_for_download($full_file_name, true, $title);
 }

plugin/ticket/src/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'   ";
         }
     }

plugin/ticket/src/ticket.class.php

Spacing +61 added lines, -61 removed lines

@@ -42,7 +42,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)) {
@@ -157,7 +157,7 @@ 
         }
 
         if ($ticket_id != 0) {
-            $ticket_code = "A" . str_pad(
+            $ticket_code = "A".str_pad(
                 (int) $ticket_id, 11, "0", STR_PAD_LEFT
             );
             $sql_update_code = "UPDATE $table_support_tickets
@@ -175,24 +175,24 @@ 
                     $user = api_get_user_info($request_user);
                     $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('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('Topic') . '</b></td>
-                                                <td width="400px">' . $subject . '</td>
+                                                <td width="100px"><b>' . get_lang('Topic').'</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>';
 
@@ -274,7 +274,7 @@ 
             if ($insert_id != $user_id) {
                 $info = api_get_user_info($user_id);
                 $sender = api_get_user_info($insert_id);
-                $href = api_get_path(WEB_PLUGIN_PATH) . PLUGIN_NAME . '/src/ticket_details.php?ticket_id=' . $ticket_id;
+                $href = api_get_path(WEB_PLUGIN_PATH).PLUGIN_NAME.'/src/ticket_details.php?ticket_id='.$ticket_id;
                 $message = sprintf($plugin->get_lang('TicketAssignedMsg'), $info['complete_name'], $href, $ticket_id);
                 $mailTitle = sprintf($plugin->get_lang('TicketAssignX'), $ticket_id);
                 api_mail_html(
@@ -319,10 +319,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>' . $plugin->get_lang('TicketWasThisAnswerSatisfying') . '</p>
-                         <button name="response" id="responseyes" value="1">' . get_lang('Yes') . '</button>
-                         <button 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>' . $plugin->get_lang('TicketWasThisAnswerSatisfying').'</p>
+                         <button name="response" id="responseyes" value="1">' . get_lang('Yes').'</button>
+                         <button name="response" id="responseno" value="0">' . get_lang('No').'</button>
                      </form>';
             $content .= $form;
             Database::query(
@@ -353,11 +353,11 @@ 
             '$message_id',
             '$subject',
             '$content',
-            '" . Database::escape_string($_SERVER['REMOTE_ADDR']) . "',
+            '".Database::escape_string($_SERVER['REMOTE_ADDR'])."',
             '$user_id',
-            '" . $now . "',
+            '".$now."',
             '$user_id',
-            '" . $now . "',
+            '".$now."',
             '$status'
         )";
         Database::query($sql_insert_message);
@@ -428,11 +428,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);
             }
@@ -455,7 +455,7 @@ 
                     '$ticket_id',
                     '$message_id',
                     '$message_attch_id',
-                    '" . $file_attach['size'] . "',
+                    '".$file_attach['size']."',
                     '$user_id',
                     '$now',
                     '$user_id',
@@ -464,7 +464,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
             );
         }
@@ -689,14 +689,14 @@ 
             $result_unread = Database::query($sql_unread);
             $unread = Database::fetch_object($result_unread)->unread;
             $userInfo = api_get_user_info($row['user_id']);
-            $hrefUser = $webPath . 'main/admin/user_information.php?user_id=' . $row['user_id'];
+            $hrefUser = $webPath.'main/admin/user_information.php?user_id='.$row['user_id'];
             $name = "<a href='$hrefUser'> {$userInfo['username']} </a>";
             $actions = "";
 
             if ($row['responsible'] != 0) {
                 $row['responsible'] = api_get_user_info($row['responsible']);
                 if (!empty($row['responsible'])) {
-                    $hrefResp = $webPath . 'main/admin/user_information.php?user_id=' . $row['responsible']['user_id'];
+                    $hrefResp = $webPath.'main/admin/user_information.php?user_id='.$row['responsible']['user_id'];
                     $row['responsible'] = "<a href='$hrefResp'> {$row['responsible']['username']} </a>";
                 } else {
                     $row['responsible'] = get_lang('UnknownUser');
@@ -704,9 +704,9 @@ 
 
             } else {
                 if ($row['status_id'] != 'REE') {
-                    $row['responsible'] = '<span style="color:#ff0000;">' . $plugin->get_lang('ToBeAssigned') . '</span>';
+                    $row['responsible'] = '<span style="color:#ff0000;">'.$plugin->get_lang('ToBeAssigned').'</span>';
                 } else {
-                    $row['responsible'] = '<span style="color:#00ff00;">' . get_lang('MessageResent') . '</span>';
+                    $row['responsible'] = '<span style="color:#00ff00;">'.get_lang('MessageResent').'</span>';
                 }
             }
 
@@ -727,17 +727,17 @@ 
             $row['col1'] = api_get_local_time($row['col1']);
             $row['col2'] = api_get_local_time($row['col2']);
             if ($isAdmin) {
-                $actions .= '<a href="ticket_details.php?ticket_id=' . $row['col0'] . '">' . Display::return_icon(
+                $actions .= '<a href="ticket_details.php?ticket_id='.$row['col0'].'">'.Display::return_icon(
                                 'synthese_view.gif', get_lang('Info')
-                        ) . '</a>&nbsp;&nbsp;';
+                        ).'</a>&nbsp;&nbsp;';
                 if ($row['priority_id'] == 'HGH' && $row['status_id'] != 'CLS') {
-                    $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['ticket_code'] . '</a>';
+                        ).'<a href="ticket_details.php?ticket_id='.$row['col0'].'">'.$row['ticket_code'].'</a>';
                 if ($row['col7'] == 'PENDIENTE') {
-                    $row['col7'] = '<span style="color: #f00; font-weight:bold;">' . $row['col7'] . '</span>';
+                    $row['col7'] = '<span style="color: #f00; font-weight:bold;">'.$row['col7'].'</span>';
                 }
 
                 $ticket = array(
@@ -754,22 +754,22 @@ 
                 );
             } else {
                 $actions = "";
-                $actions .= '<a href="ticket_details.php?ticket_id=' . $row['col0'] . '">' . Display::return_icon(
+                $actions .= '<a href="ticket_details.php?ticket_id='.$row['col0'].'">'.Display::return_icon(
                                 'synthese_view.gif', get_lang('Info')
-                        ) . '</a>&nbsp;&nbsp;';
+                        ).'</a>&nbsp;&nbsp;';
                 $row['col0'] = Display::return_icon(
                                 $img_source, get_lang('Info')
-                        ) . '<a href="ticket_details.php?ticket_id=' . $row['col0'] . '">' . $row['ticket_code'] . '</a>';
+                        ).'<a href="ticket_details.php?ticket_id='.$row['col0'].'">'.$row['ticket_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'] == 'NRM' && $row['status_id'] != 'CLS') {
-                    $actions .= '<a href="myticket.php?ticket_id=' . $row['ticket_id'] . '&amp;action=alert">
-                                 <img src="' . $webPath . 'main/img/exclamation.png" border="0" /></a>';
+                    $actions .= '<a href="myticket.php?ticket_id='.$row['ticket_id'].'&amp;action=alert">
+                                 <img src="' . $webPath.'main/img/exclamation.png" border="0" /></a>';
                 }
                 if ($row['priority_id'] == 'HGH') {
-                    $actions .= '<img src="' . $webCodePath . 'img/admin_star.png" border="0" />';
+                    $actions .= '<img src="'.$webCodePath.'img/admin_star.png" border="0" />';
                 }
                 $ticket = array(
                     $row['col0'],
@@ -781,16 +781,16 @@ 
                 );
             }
             if ($unread > 0) {
-                $ticket['0'] = $ticket['0'] . '&nbsp;&nbsp;(' . $unread . ')<a href="ticket_details.php?ticket_id=' . $row['ticket_id'] . '">
-                                <img src="' . $webPath . 'main/img/message_new.png" border="0" title="' . $unread . ' ' . get_lang('Messages') . '"/>
+                $ticket['0'] = $ticket['0'].'&nbsp;&nbsp;('.$unread.')<a href="ticket_details.php?ticket_id='.$row['ticket_id'].'">
+                                <img src="' . $webPath.'main/img/message_new.png" border="0" title="'.$unread.' '.get_lang('Messages').'"/>
                                 </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="' . $webPath . 'main/img/history.gif" title="' . get_lang(
+                $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="'.$webPath.'main/img/history.gif" title="'.get_lang(
                                 'Historial'
-                        ) . '" alt="' . get_lang('Historial') . '"/>
-					<div class="blackboard_hide" id="div_' . $row['ticket_id'] . '">&nbsp;&nbsp;</div>
+                        ).'" alt="'.get_lang('Historial').'"/>
+					<div class="blackboard_hide" id="div_' . $row['ticket_id'].'">&nbsp;&nbsp;</div>
 					</a>&nbsp;&nbsp;';
             }
             $tickets[] = $ticket;
@@ -1028,8 +1028,8 @@ 
                     }
                 }
                 $userInfo = api_get_user_info($row['request_user']);
-                $row['user_url'] = '<a href="' . api_get_path(WEB_PATH) . 'main/admin/user_information.php?user_id=' . $row['request_user'] . '">
-                ' . api_get_person_name($userInfo['firstname'], $userInfo['lastname']) . '</a>';
+                $row['user_url'] = '<a href="'.api_get_path(WEB_PATH).'main/admin/user_information.php?user_id='.$row['request_user'].'">
+                ' . api_get_person_name($userInfo['firstname'], $userInfo['lastname']).'</a>';
                 $ticket['usuario'] = $userInfo;
                 $ticket['ticket'] = $row;
             }
@@ -1046,10 +1046,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;
@@ -1061,12 +1061,12 @@ 
                 $sql = "SELECT *
                         FROM $table_support_message_attachments
                         WHERE
-                            message_id = " . $row['message_id'] . " AND
+                            message_id = ".$row['message_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['atachments'][] = $row2;
                 }
                 $ticket['messages'][] = $message;
@@ -1090,9 +1090,9 @@ 
         );
         $table_support_tickets = Database::get_main_table(TABLE_TICKET_TICKET);
         $now = api_get_utc_datetime();
-        $sql = "UPDATE " . $table_support_messages . "
-                SET status = 'LEI', sys_lastedit_user_id ='" . api_get_user_id() . "',
-                sys_lastedit_datetime ='" . $now . "'
+        $sql = "UPDATE ".$table_support_messages."
+                SET status = 'LEI', sys_lastedit_user_id ='" . api_get_user_id()."',
+                sys_lastedit_datetime ='" . $now."'
                 WHERE ticket_id ='$ticket_id' ";
 
         if (api_is_platform_admin()) {
@@ -1191,10 +1191,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);
 
@@ -1279,7 +1279,7 @@ 
         $sql = "UPDATE $table_support_tickets SET
                     status_id = 'CLS',
                     sys_lastedit_user_id ='$user_id',
-                    sys_lastedit_datetime ='" . $now . "',
+                    sys_lastedit_datetime ='".$now."',
                     end_date ='$now'
                 WHERE ticket_id ='$ticket_id'";
         Database::query($sql);
@@ -1336,8 +1336,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;
         }
         return $history;
@@ -1533,7 +1533,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'

plugin/ticket/src/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;
 

plugin/ticket/src/ticket_plugin.class.php

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
     public function install()
     {
         // Create database tables and insert a Tab
-        require_once api_get_path(SYS_PLUGIN_PATH) . PLUGIN_NAME . '/database.php';
+        require_once api_get_path(SYS_PLUGIN_PATH).PLUGIN_NAME.'/database.php';
 
     }
 
@@ -84,7 +84,7 @@ 
         $rsTab = $this->deleteTab($plugSetting['comment']);
 
         if ($rsTab) {
-            echo "<script>location.href = '" . $_SERVER['REQUEST_URI'] . "';</script>";
+            echo "<script>location.href = '".$_SERVER['REQUEST_URI']."';</script>";
         }
     }
 }

plugin/ticket/src/assign_tickets.php

Spacing +9 added lines, -9 removed lines

@@ -19,7 +19,7 @@ 
 echo '<div id="confirmation"></div>';
 $id = intval($_GET['id']);
 $tblWeeklyReport = Database::get_main_table('rp_reporte_semanas');
-$sql ="SELECT * FROM $tblWeeklyReport WHERE id = '$id'";
+$sql = "SELECT * FROM $tblWeeklyReport WHERE id = '$id'";
 $sql_tasks = "SELECT id AS colid, title as coltitle
     FROM ".Database::get_course_table(TABLE_STUDENT_PUBLICATION)."
     WHERE parent_id = 0
@@ -44,27 +44,27 @@ 
 $result_forum = Database::query($sql_forum);
 
 echo '<div class="row">
-        <input type="hidden" id="rs_id" name ="rs_id" value="' . $id . '">
-        <div class="formw">' . get_lang('PleaseSelectTasks') . '</div>
+        <input type="hidden" id="rs_id" name ="rs_id" value="' . $id.'">
+        <div class="formw">' . get_lang('PleaseSelectTasks').'</div>
     </div>';
 echo '<div class="row"><div class="formw"><select name ="work_id" id="work_id">';
-echo '<option value="0"' . (($row['colid'] == $rs->work_id) ? "selected" : "") . '>' . get_lang('PleaseSelect') . '</option>';
+echo '<option value="0"'.(($row['colid'] == $rs->work_id) ? "selected" : "").'>'.get_lang('PleaseSelect').'</option>';
 while ($row = Database::fetch_assoc($result_tasks)) {
-    echo '<option value="' . $row['colid'] . '"' . (($row['colid'] == $rs->work_id) ? "selected" : "") . '>' . $row['coltitle'] . '</option>';
+    echo '<option value="'.$row['colid'].'"'.(($row['colid'] == $rs->work_id) ? "selected" : "").'>'.$row['coltitle'].'</option>';
 }
 echo '</select></div><div>';
 echo '<div class="row">
-        <div class="formw">' . get_lang('PleaseSelectThread') . '</div>
+        <div class="formw">' . get_lang('PleaseSelectThread').'</div>
     </div>';
 echo '<div class="row"><div class="formw"><select name ="forum_id" id="forum_id">';
-echo '<option value="0"' . (($row['colid'] == $rs->work_id) ? "forum_id" : "") . '>' . get_lang('PleaseSelect') . '</option>';
+echo '<option value="0"'.(($row['colid'] == $rs->work_id) ? "forum_id" : "").'>'.get_lang('PleaseSelect').'</option>';
 while ($row = Database::fetch_assoc($result_forum)) {
-    echo '<option value="' . $row['colid'] . '"' . (($row['colid'] == $rs->forum_id) ? "selected" : "") . '>' . $row['coltitle'] . '</option>';
+    echo '<option value="'.$row['colid'].'"'.(($row['colid'] == $rs->forum_id) ? "selected" : "").'>'.$row['coltitle'].'</option>';
 }
 echo '</select></div><div>';
 echo '<div class="row">
         <div class="formw">
-        <button class="save" name="edit" type="button" value="' . get_lang('Edit') . '" onClick="save(' . "$id" . ');">' . get_lang('Edit') . '</button>
+        <button class="save" name="edit" type="button" value="' . get_lang('Edit').'" onClick="save('."$id".');">'.get_lang('Edit').'</button>
         </div>
     </div>';
 echo '</form>';