chamilo / chamilo-lms

plugin/ticket/src/ticket_details.php

Spacing +44 added lines, -44 removed lines

@@ -30,10 +30,10 @@ 
 		width: 600,
 		modal: true,
 		buttons: {
-                    ' . get_lang('Accept') . ': function(){
+                    ' . get_lang('Accept').': function(){
                         $("#frmResponsable").submit()
                     },
-                    ' . ucfirst(get_lang('Close')) . ': function() {
+                    ' . ucfirst(get_lang('Close')).': function() {
                         $(this).dialog("close");
                     }
                 }
@@ -44,37 +44,37 @@ 
         });
 
         $("input#responseyes").click(function () {
-            if(!confirm("' . $plugin->get_lang('AreYouSure') . ' : ' . strtoupper(get_lang('Yes')) . '. ' . $plugin->get_lang('IfYouAreSureTheTicketWillBeClosed') . '")){
+            if(!confirm("' . $plugin->get_lang('AreYouSure').' : '.strtoupper(get_lang('Yes')).'. '.$plugin->get_lang('IfYouAreSureTheTicketWillBeClosed').'")){
                 return false;
             }
         });
 
         $("input#responseno").click(function () {
-            if(!confirm("' . $plugin->get_lang('AreYouSure') . ' : ' . strtoupper(get_lang('No')) . '")){
+            if(!confirm("' . $plugin->get_lang('AreYouSure').' : '.strtoupper(get_lang('No')).'")){
                 return false;
             }
         });
 
         $("#unassign").click(function () {
-            if (!confirm("' . $plugin->get_lang('AreYouSureYouWantToUnassignTheTicket') . '")) {
+            if (!confirm("' . $plugin->get_lang('AreYouSureYouWantToUnassignTheTicket').'")) {
                 return false;
             }
         });
 
         $("#close").click(function () {
-            if (!confirm("' . $plugin->get_lang('AreYouSureYouWantToCloseTheTicket') . '")) {
+            if (!confirm("' . $plugin->get_lang('AreYouSureYouWantToCloseTheTicket').'")) {
                 return false;
             }
         });
 
-    ' . $disableReponseButtons . '
+    ' . $disableReponseButtons.'
 });
 
 function validate() {
     fckEditor1val = CKEDITOR.instances["content"].getData();
     document.getElementById("content").value= fckEditor1val;
     if(fckEditor1val == ""){
-        alert("' . $plugin->get_lang('YouMustWriteAMessage') . '");
+        alert("' . $plugin->get_lang('YouMustWriteAMessage').'");
         return false;
     }
 }
@@ -117,7 +117,7 @@ 
     });
 
     img_remove = $("<img/>", {
-        src: "' . api_get_path(WEB_CODE_PATH) . 'img/delete.gif"
+        src: "' . api_get_path(WEB_CODE_PATH).'img/delete.gif"
     });
 
     new_filepath_id = $("#filepath_" + counter_image);
@@ -239,22 +239,22 @@ 
     if ($ticket['ticket']['status_id'] != 'REE' && $ticket['ticket']['status_id'] != 'CLS' && $isAdmin) {
         if (intval($ticket['ticket']['assigned_last_user']) == $user_id) {
             if ($ticket['ticket']['status_id'] != 'CLS') {
-                $form_close_ticket.= '<form enctype="multipart/form-data" action="' . api_get_self() . '?ticket_id=' . $ticket['ticket']['ticket_id'] . '" method="post" name="close_ticket" id="close_ticket" >';
-                $form_close_ticket.= '<input type="hidden" name="ticket_id" value="' . $ticket['ticket']['ticket_id'] . '"/>
-                                        <button class="minus" name="close" type="submit" id="close" >' . get_lang('Close') . '</button>';
-                $form_close_ticket.= '</form>';
+                $form_close_ticket .= '<form enctype="multipart/form-data" action="'.api_get_self().'?ticket_id='.$ticket['ticket']['ticket_id'].'" method="post" name="close_ticket" id="close_ticket" >';
+                $form_close_ticket .= '<input type="hidden" name="ticket_id" value="'.$ticket['ticket']['ticket_id'].'"/>
+                                        <button class="minus" name="close" type="submit" id="close" >' . get_lang('Close').'</button>';
+                $form_close_ticket .= '</form>';
             }
         }
     }
-    $titulo = '<center><h1>Ticket #' . $ticket['ticket']['ticket_code'] . '</h1></center>';
+    $titulo = '<center><h1>Ticket #'.$ticket['ticket']['ticket_code'].'</h1></center>';
     $img_assing = '';
     if ($isAdmin && $ticket['ticket']['status_id'] != 'CLS' && $ticket['ticket']['status_id'] != 'REE') {
         if ($ticket['ticket']['assigned_last_user'] != 0 && $ticket['ticket']['assigned_last_user'] == $user_id) {
-            $img_assing = '<a href="' . api_get_self() . '?ticket_id=' . $ticket['ticket']['ticket_id'] . '&amp;action=unassign" id="unassign">
-                            <img src="' . api_get_path(WEB_CODE_PATH) . 'img/admin_star.png"  style="height: 32px; width: 32px;" border="0" title="Unassign" align="center"/>
+            $img_assing = '<a href="'.api_get_self().'?ticket_id='.$ticket['ticket']['ticket_id'].'&amp;action=unassign" id="unassign">
+                            <img src="' . api_get_path(WEB_CODE_PATH).'img/admin_star.png"  style="height: 32px; width: 32px;" border="0" title="Unassign" align="center"/>
                            </a>';
         } else {
-            $img_assing = '<a href="#" id="assign"><img src="' . api_get_path(WEB_CODE_PATH) . 'img/admin_star_na.png" style="height: 32px; width: 32px;" title="Assign" align="center"/></a>';
+            $img_assing = '<a href="#" id="assign"><img src="'.api_get_path(WEB_CODE_PATH).'img/admin_star_na.png" style="height: 32px; width: 32px;" title="Assign" align="center"/></a>';
         }
     }
     $bold = '';
@@ -271,39 +271,39 @@ 
     echo '<div style="margin-left:20%;margin-right:20%;">
 			<table width="100%" >
 				<tr>
-	              <td colspan="3" style="width:65%">' . $titulo . '</td>
-	              <td style="width: 15%">' . $img_assing . '</td>
-	              <td style="width: 15%">' . $form_close_ticket . '</td>
+	              <td colspan="3" style="width:65%">' . $titulo.'</td>
+	              <td style="width: 15%">' . $img_assing.'</td>
+	              <td style="width: 15%">' . $form_close_ticket.'</td>
 	            </tr>
 	         	<tr>
-	              <td style="width:45%;" ><p>' . get_lang('Sent') . ': ' . $ticket['ticket']['start_date'] . '</p></td>
+	              <td style="width:45%;" ><p>' . get_lang('Sent').': '.$ticket['ticket']['start_date'].'</p></td>
 	              <td style="width:50px;"></td>
-	              <td style="width:45%;" ><p>' . $plugin->get_lang('LastResponse') . ': ' . $ticket['ticket']['sys_lastedit_datetime'] . '</p></td>
+	              <td style="width:45%;" ><p>' . $plugin->get_lang('LastResponse').': '.$ticket['ticket']['sys_lastedit_datetime'].'</p></td>
 	              <td colspan="2"></td>
 	            </tr>
 	            <tr>
-	               <td><p>' . get_lang('Subject') . ': ' . $ticket['messages'][0]['subject'] . '</p></td>
+	               <td><p>' . get_lang('Subject').': '.$ticket['messages'][0]['subject'].'</p></td>
 	               <td></td>
-	               <td><p ' . $bold . '>' . get_lang('Status') . ': ' . $ticket['ticket']['status'] . '</p></td>
+	               <td><p ' . $bold.'>'.get_lang('Status').': '.$ticket['ticket']['status'].'</p></td>
 	               <td colspan="2"></td>
 	            </tr>
 	            <tr>
-	                <td><p>' . get_lang('Category') . ': ' . $ticket['ticket']['name'] . '</p></td>
+	                <td><p>' . get_lang('Category').': '.$ticket['ticket']['name'].'</p></td>
 	                <td></td>
-	                <td ><p>' . $plugin->get_lang('Priority') . ':' . $ticket['ticket']['priority'] . '<p></td>
+	                <td ><p>' . $plugin->get_lang('Priority').':'.$ticket['ticket']['priority'].'<p></td>
 	                <td colspan="2"></td>
 	            </tr>';
     if ($ticket['ticket']['course_url'] != null) {
         echo '<tr>
-				<td><p>' . get_lang('Course') . ':</p></td>
+				<td><p>' . get_lang('Course').':</p></td>
 	            <td></td>
-			    <td>' . $ticket['ticket']['course_url'] . '</td>
+			    <td>' . $ticket['ticket']['course_url'].'</td>
 	            <td colspan="2"></td>
 	          </tr>';
     }
     if ($isAdmin) {
         echo '<tr>
-		<td><p>' . get_lang('User') . ': &nbsp;' . $user_info = $ticket['ticket']['user_url'] . ' (' . $ticket['usuario']['username'] . ')</p></td>
+		<td><p>' . get_lang('User').': &nbsp;'.$user_info = $ticket['ticket']['user_url'].' ('.$ticket['usuario']['username'].')</p></td>
  	      </tr>';
     }
     //select admins
@@ -311,15 +311,15 @@ 
 
     $admins = UserManager::get_user_list_like(array("status" => "1"), array("username"), true);
     foreach ($admins as $admin) {
-        $select_admins.= "<option value = '" . $admin['user_id'] . "' " . (($user_id == $admin['user_id']) ? ("selected='selected'") : "") . ">" . $admin['lastname'] . " ," . $admin['firstname'] . "</option>";
+        $select_admins .= "<option value = '".$admin['user_id']."' ".(($user_id == $admin['user_id']) ? ("selected='selected'") : "").">".$admin['lastname']." ,".$admin['firstname']."</option>";
     }
     $select_admins .= "</select>";
-    echo '<div id="dialog-form" title="' . $plugin->get_lang('AssignTicket') . '" >';
-    echo '<form id="frmResponsable" method="POST" action="ticket_details.php?ticket_id=' . $ticket['ticket']['ticket_id'] . '">
+    echo '<div id="dialog-form" title="'.$plugin->get_lang('AssignTicket').'" >';
+    echo '<form id="frmResponsable" method="POST" action="ticket_details.php?ticket_id='.$ticket['ticket']['ticket_id'].'">
 			<input type="hidden" name ="action" id="action" value="assign"/>
 			<div>
-				<div class="label">' . get_lang('Responsable') . ':</div>
-				<div class="formw">' . $select_admins . '</div>
+				<div class="label">' . get_lang('Responsable').':</div>
+				<div class="formw">' . $select_admins.'</div>
 			</div>
 		  </form>';
     echo '</div>';
@@ -333,13 +333,13 @@ 
         if ($message['admin']) {
             $type = "normal";
             if ($isAdmin) {
-                $message['message'].='<br/><b>' . $plugin->get_lang('AttendedBy') . ': ' . $message['user_created'] . " - " . api_convert_and_format_date(api_get_local_time($message['sys_insert_datetime']), DATE_TIME_FORMAT_LONG, _api_get_timezone()) . "</b>";
+                $message['message'] .= '<br/><b>'.$plugin->get_lang('AttendedBy').': '.$message['user_created']." - ".api_convert_and_format_date(api_get_local_time($message['sys_insert_datetime']), DATE_TIME_FORMAT_LONG, _api_get_timezone())."</b>";
             }
-        }else {
-            $message['message'].='<b>' . get_lang('Sent') . ': ' . api_convert_and_format_date(api_get_local_time($message['sys_insert_datetime']), DATE_TIME_FORMAT_LONG, _api_get_timezone()) . "</b>";
+        } else {
+            $message['message'] .= '<b>'.get_lang('Sent').': '.api_convert_and_format_date(api_get_local_time($message['sys_insert_datetime']), DATE_TIME_FORMAT_LONG, _api_get_timezone())."</b>";
         }
 
-        $receivedMessage = '<b>' . get_lang('Subject') . ': </b> ' . $message['subject'] . '<br/> <b>' . get_lang('Message') . ':</b>' . $message['message'] . '<br/>';
+        $receivedMessage = '<b>'.get_lang('Subject').': </b> '.$message['subject'].'<br/> <b>'.get_lang('Message').':</b>'.$message['message'].'<br/>';
         $attachementLinks = "";
 
         if (isset($message['atachments'])) {
@@ -351,12 +351,12 @@ 
             }
         }
 
-        $entireMessage = $receivedMessage . $attachementLinks;
+        $entireMessage = $receivedMessage.$attachementLinks;
         echo Display::return_message($entireMessage, $type, false);
     }
     echo "</div>";
     echo "</div>";
-    $subject = get_lang('ReplyShort') .": " . $message['subject'];
+    $subject = get_lang('ReplyShort').": ".$message['subject'];
     $user_admin = api_is_platform_admin();
     if ($ticket['ticket']['status_id'] != 'REE' AND $ticket['ticket']['status_id'] != 'CLS') {
         if (!$isAdmin && $ticket['ticket']['status_id'] != 'XCF') {
@@ -379,7 +379,7 @@ 
     $file_attachments = $_FILES;
     $user_id = api_get_user_id();
     TicketManager::insert_message($ticket_id, $subject, $content, $file_attachments, $user_id, 'NOL', $mensajeconfirmacion);
-    header("location:" . api_get_self() . "?ticket_id=" . $ticket_id);
+    header("location:".api_get_self()."?ticket_id=".$ticket_id);
     exit;
 }
 
@@ -395,7 +395,7 @@ 
     $form = new FormValidator(
         'send_ticket',
         'POST',
-        api_get_self() . '?ticket_id=' . $ticket['ticket']['ticket_id'],
+        api_get_self().'?ticket_id='.$ticket['ticket']['ticket_id'],
         '',
         array(
             'enctype' => 'multipart/form-data',
@@ -446,9 +446,9 @@ 
 
     $form->addElement('html', '<div class="controls">');
     $form->addElement('html', '<span id="link-more-attach" >');
-    $form->addElement('html', '<span class="label label-info" onclick="return add_image_form()">' . get_lang('AddOneMoreFile') . '</span>');
+    $form->addElement('html', '<span class="label label-info" onclick="return add_image_form()">'.get_lang('AddOneMoreFile').'</span>');
     $form->addElement('html', '</span>');
-    $form->addElement('html', '(' . sprintf(get_lang('MaximunFileSizeX'), format_file_size(api_get_setting('message_max_upload_filesize'))) . ')');
+    $form->addElement('html', '('.sprintf(get_lang('MaximunFileSizeX'), format_file_size(api_get_setting('message_max_upload_filesize'))).')');
 
     $form->addElement('html', '<br/>');
     $form->addElement(

plugin/ticket/src/new_ticket.php

Spacing +28 added lines, -28 removed lines

@@ -12,7 +12,7 @@ 
 if (!api_is_platform_admin() &&
     $plugin->get('allow_student_add') != 'true'
 ) {
-    header('location:' . api_get_path(WEB_PLUGIN_PATH) . PLUGIN_NAME . '/src/myticket.php');
+    header('location:'.api_get_path(WEB_PLUGIN_PATH).PLUGIN_NAME.'/src/myticket.php');
     exit;
 }
 
@@ -38,7 +38,7 @@ 
             $("#user_id_request").val(my_user_id);
             $("#personal_email").val(user_email);
             $("#btnsubmit").attr("disabled", false);
-            ' . $scrollTol . '
+            ' . $scrollTol.'
         }
     });
 }
@@ -74,22 +74,22 @@ 
     var selected = document.getElementById("category_id").selectedIndex;
     var id = document.getElementById("category_id").options[selected].value;
     if (document.getElementById("user_id_request").value == "") {
-        alert("' . $plugin->get_lang("ValidUser") . '");
+        alert("' . $plugin->get_lang("ValidUser").'");
         return false;
     } else if(id == 0) {
-        alert("' . $plugin->get_lang("ValidType") . '");
+        alert("' . $plugin->get_lang("ValidType").'");
         return false;
     } else if(document.getElementById("subject").value == "") {
-        alert("' . $plugin->get_lang("ValidSubject") . '");
+        alert("' . $plugin->get_lang("ValidSubject").'");
         return false;
     } else if(parseInt(course_required[id]) == 1 && document.getElementById("course_id").value == 0) {
-        alert("' . $plugin->get_lang("ValidCourse") . '");
+        alert("' . $plugin->get_lang("ValidCourse").'");
         return false;
     } else if(id != "CUR" && parseInt(course_required[id]) != 1  && !re.test(document.getElementById("personal_email").value)) {
-        alert("' . $plugin->get_lang("ValidEmail") . '");
+        alert("' . $plugin->get_lang("ValidEmail").'");
         return false;
     } else if(fckEditor1val == "") {
-        alert("' . $plugin->get_lang("ValidMessage") . '");
+        alert("' . $plugin->get_lang("ValidMessage").'");
         return false;
     }
 }
@@ -132,7 +132,7 @@ 
     });
 
     img_remove = $("<img/>", {
-        src: "' . api_get_path(WEB_CODE_PATH) . 'img/delete.gif"
+        src: "' . api_get_path(WEB_CODE_PATH).'img/delete.gif"
     });
 
     new_filepath_id = $("#filepath_" + counter_image);
@@ -162,10 +162,10 @@ 
 </style>';
 $types = TicketManager::get_all_tickets_categories();
 $htmlHeadXtra[] = '<script language="javascript">
-    var projects = ' . js_array($types, 'projects', 'project_id') . '
-    var course_required = ' . js_array($types, 'course_required', 'course_required') . '
-    var other_area = ' . js_array($types, 'other_area', 'other_area') . '
-    var email = ' . js_array($types, 'email', 'email') .
+    var projects = ' . js_array($types, 'projects', 'project_id').'
+    var course_required = ' . js_array($types, 'course_required', 'course_required').'
+    var other_area = ' . js_array($types, 'other_area', 'other_area').'
+    var email = ' . js_array($types, 'email', 'email').
 '</script>';
 
 
@@ -175,7 +175,7 @@ 
  */
 function js_str($s)
 {
-    return '"' . addcslashes($s, "\0..\37\"\\") . '"';
+    return '"'.addcslashes($s, "\0..\37\"\\").'"';
 }
 
 /**
@@ -188,7 +188,7 @@ 
 {
     $return = "new Array(); ";
     foreach ($array as $value) {
-        $return .= $name . "['" . $value['category_id'] . "'] ='" . $value[$key] . "'; ";
+        $return .= $name."['".$value['category_id']."'] ='".$value[$key]."'; ";
     }
 
     return $return;
@@ -205,7 +205,7 @@ 
     // Category List
     $categoryList = array();
     foreach ($types as $type) {
-        $categoryList[$type['category_id']] = $type['name'] . ": " . $type['description'];
+        $categoryList[$type['category_id']] = $type['name'].": ".$type['description'];
     }
 
     // Status List
@@ -378,7 +378,7 @@ 
     $form->addElement(
         'text',
         'phone',
-        get_lang('Phone') . ' (' . $plugin->get_lang('Optional') . ')',
+        get_lang('Phone').' ('.$plugin->get_lang('Optional').')',
         array(
             'id' => 'phone'
         )
@@ -403,9 +403,9 @@ 
 
     $form->addElement('html', '<div class="controls">');
     $form->addElement('html', '<span id="link-more-attach" >');
-    $form->addElement('html', '<span class="label label-info" onclick="return add_image_form()">' . get_lang('AddOneMoreFile') . '</span>');
+    $form->addElement('html', '<span class="label label-info" onclick="return add_image_form()">'.get_lang('AddOneMoreFile').'</span>');
     $form->addElement('html', '</span>');
-    $form->addElement('html', '(' . sprintf(get_lang('MaximunFileSizeX'), format_file_size(api_get_setting('message_max_upload_filesize'))) . ')');
+    $form->addElement('html', '('.sprintf(get_lang('MaximunFileSizeX'), format_file_size(api_get_setting('message_max_upload_filesize'))).')');
 
     $form->addElement('html', '<br/>');
     $form->addElement(
@@ -433,7 +433,7 @@ 
     $category_id = $_POST['category_id'];
     $content = $_POST['content'];
     if ($_POST['phone'] != "") {
-        $content .= '<p style="color:red">&nbsp;' . get_lang('Phone') . ': ' . Security::remove_XSS($_POST['phone']). '</p>';
+        $content .= '<p style="color:red">&nbsp;'.get_lang('Phone').': '.Security::remove_XSS($_POST['phone']).'</p>';
     }
     $course_id = $_POST['course_id'];
     $project_id = $_POST['project_id'];
@@ -464,7 +464,7 @@ 
         $responsible
     )
     ) {
-        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;
     } else {
         Display::display_header(get_lang('ComposeMessage'));
@@ -483,7 +483,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']));
@@ -560,9 +560,9 @@ 
     while ($user = Database::fetch_row($res)) {
         $user_id = $user[0];
         $userPicture = UserManager::getUserPicture($user_id);
-        $photo = '<img src="' . $userPicture. '" alt="' . api_get_person_name($user[2], $user[3]) . '" title="' . api_get_person_name($user[2], $user[3]) . '" />';
-        $button = '<a  href="javascript:void(0)" onclick="load_course_list(\'div_' . $user_id . '\',' . $user_id . ', \'' . $user[5] . '\')">'
-                    . Display::return_icon('view_more_stats.gif', get_lang('Info')) .
+        $photo = '<img src="'.$userPicture.'" alt="'.api_get_person_name($user[2], $user[3]).'" title="'.api_get_person_name($user[2], $user[3]).'" />';
+        $button = '<a  href="javascript:void(0)" onclick="load_course_list(\'div_'.$user_id.'\','.$user_id.', \''.$user[5].'\')">'
+                    . Display::return_icon('view_more_stats.gif', get_lang('Info')).
                    '</a>&nbsp;&nbsp;';
         $users[] = array(
             $photo,
@@ -586,10 +586,10 @@ 
         echo '
             <div class="actions">
               <span style="float: right;">&nbsp;</span>
-              <form id="search_simple" name="search_simple" method="get" action="' . api_get_self() . '" class="form-search">
+              <form id="search_simple" name="search_simple" method="get" action="' . api_get_self().'" class="form-search">
                 <fieldset>
-                <span><label for="keyword">' . get_lang('SearchAUser') . ': &nbsp;</label><input type="text" name="keyword" size="25"></span>
-                <span><button type="submit" name="submit" class="btn btn">' . get_lang('Search') . '</button></span>
+                <span><label for="keyword">' . get_lang('SearchAUser').': &nbsp;</label><input type="text" name="keyword" size="25"></span>
+                <span><button type="submit" name="submit" class="btn btn">' . get_lang('Search').'</button></span>
                 <div class="clear"></div>
                 </fieldset>
               </form>

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/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/tutor.php

Spacing +3 added lines, -3 removed lines

@@ -17,7 +17,7 @@ 
         var url     = this.href;
         var dialog  = $("#dialog");
         if ($("#dialog").length == 0) {
-                dialog  = $("' . '<div id="dialog" style="display:hidden"></div>' . '").appendTo("body");
+                dialog  = $("' . '<div id="dialog" style="display:hidden"></div>'.'").appendTo("body");
         }
 
         // load remote content
@@ -56,7 +56,7 @@ 
 	 $.ajax({
 		contentType: "application/x-www-form-urlencoded",
 		beforeSend: function(objeto) {
-		$("div#confirmation").html("<img src=\"' . api_get_path(WEB_LIBRARY_PATH) . 'javascript/indicator.gif\" />"); },
+		$("div#confirmation").html("<img src=\"' . api_get_path(WEB_LIBRARY_PATH).'javascript/indicator.gif\" />"); },
 		type: "POST",
 		url: "update_report.php",
 		data: "work_id="+work_id+"&forum_id="+forum_id+"&rs_id="+rs_id,
@@ -95,7 +95,7 @@ 
 $course_code = api_get_course_id();
 $results = initializeReport($course_code);
 if (isset($_GET['action'])) {
-    Export::arrayToXls($results['export'], "COURSE_USER_REPORT" . $course_code);
+    Export::arrayToXls($results['export'], "COURSE_USER_REPORT".$course_code);
 } else {
     Display::display_header();
     api_protect_course_script();