chamilo / chamilo-lms

plugin/ticket/src/ticket.class.php

Doc Comments +9 added lines, -9 removed lines

@@ -57,7 +57,7 @@ 
      * @param $category_id
      * @param $course_id
      * @param $project_id
-     * @param $other_area
+     * @param integer $other_area
      * @param $email
      * @param $subject
      * @param $content
@@ -240,8 +240,8 @@ 
 
     /**
      * Assign ticket to admin
-     * @param $ticket_id
-     * @param $user_id
+     * @param string $ticket_id
+     * @param integer $user_id
      */
     public static function assign_ticket_user($ticket_id, $user_id)
     {
@@ -294,9 +294,9 @@ 
 
     /**
      * Insert message between Users and Admins
-     * @param $ticket_id
-     * @param $subject
-     * @param $content
+     * @param string $ticket_id
+     * @param string $subject
+     * @param string $content
      * @param $file_attachments
      * @param $user_id
      * @param string $status
@@ -402,9 +402,9 @@ 
     /**
      * Attachment files when a message is sent
      * @param $file_attach
-     * @param $ticket_id
-     * @param $message_id
-     * @param $message_attch_id
+     * @param integer $ticket_id
+     * @param integer $message_id
+     * @param integer $message_attch_id
      * @return array
      */
     public static function save_message_attachment_file(

Spacing +60 added lines, -60 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);
@@ -431,11 +431,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);
             }
@@ -458,7 +458,7 @@ 
                     '$ticket_id',
                     '$message_id',
                     '$message_attch_id',
-                    '" . $file_attach['size'] . "',
+                    '".$file_attach['size']."',
                     '$user_id',
                     '$now',
                     '$user_id',
@@ -467,7 +467,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
             );
         }
@@ -692,14 +692,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');
@@ -707,9 +707,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>';
                 }
             }
 
@@ -730,17 +730,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(
@@ -757,22 +757,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="' . Display::returnIconPath('exclamation.png') . '" border="0" /></a>';
+                    $actions .= '<a href="myticket.php?ticket_id='.$row['ticket_id'].'&amp;action=alert">
+                                 <img src="' . Display::returnIconPath('exclamation.png').'" border="0" /></a>';
                 }
                 if ($row['priority_id'] == 'HGH') {
-                    $actions .= '<img src="' . Display::returnIconPath('admin_star.png') . '" border="0" />';
+                    $actions .= '<img src="'.Display::returnIconPath('admin_star.png').'" border="0" />';
                 }
                 $ticket = array(
                     $row['col0'],
@@ -784,14 +784,14 @@ 
                 );
             }
             if ($unread > 0) {
-                $ticket['0'] = $ticket['0'] . '&nbsp;&nbsp;(' . $unread . ')<a href="ticket_details.php?ticket_id=' . $row['ticket_id'] . '">
-                                <img src="' . Display::returnIconPath('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="' . Display::returnIconPath('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="' . Display::returnIconPath('history.gif') . '" title="' . get_lang('Historial') . '" alt="' . get_lang('Historial') . '"/>
-					<div class="blackboard_hide" id="div_' . $row['ticket_id'] . '">&nbsp;&nbsp;</div>
+                $ticket['0'] .= '&nbsp;&nbsp;<a  href="javascript:void(0)" onclick="load_history_ticket(\'div_'.$row['ticket_id'].'\','.$row['ticket_id'].')">
+					<img onclick="load_course_list(\'div_' . $row['ticket_id'].'\','.$row['ticket_id'].')" onmouseover="clear_course_list (\'div_'.$row['ticket_id'].'\')" src="'.Display::returnIconPath('history.gif').'" title="'.get_lang('Historial').'" alt="'.get_lang('Historial').'"/>
+					<div class="blackboard_hide" id="div_' . $row['ticket_id'].'">&nbsp;&nbsp;</div>
 					</a>&nbsp;&nbsp;';
             }
             $tickets[] = $ticket;
@@ -1029,8 +1029,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;
             }
@@ -1047,10 +1047,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;
@@ -1062,12 +1062,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;
@@ -1091,9 +1091,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()) {
@@ -1192,10 +1192,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);
 
@@ -1281,7 +1281,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);
@@ -1338,8 +1338,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;
@@ -1534,7 +1534,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/ticket_plugin.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
     /**
      * Set the result
      * @staticvar null $result
-     * @return type
+     * @return TicketPlugin
      */
     static function create()
     {

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,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';
     }
 
     /**
@@ -92,7 +92,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_report.lib.php

Doc Comments +4 added lines, -4 removed lines

@@ -73,7 +73,7 @@ 
 
 /**
  * @param $courseInfo
- * @param $weeksCount
+ * @param string $weeksCount
  * @param $page
  * @return array
  */
@@ -182,7 +182,7 @@ 
 
 /**
  * @param $datos
- * @param $pagina
+ * @param integer $pagina
  * @return string
  */
 function showStudentResult($datos, $pagina)
@@ -201,8 +201,8 @@ 
 
 /**
  * @param $data
- * @param $numero_semanas
- * @return array
+ * @param integer $numero_semanas
+ * @return string[]
  */
 function showStudentResultExport($data, $numero_semanas)
 {

Spacing +22 added lines, -22 removed lines

@@ -21,7 +21,7 @@ 
     $table_post = Database::get_course_table(TABLE_FORUM_POST);
     $table_work = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
     $course_code = Database::escape_string($course_code);
-    $res = Database::query("SELECT COUNT(*) as cant FROM $table_reporte_semanas WHERE course_code = '" . $course_code . "'");
+    $res = Database::query("SELECT COUNT(*) as cant FROM $table_reporte_semanas WHERE course_code = '".$course_code."'");
     $sqlWeeks = "SELECT semanas FROM $table_semanas_curso WHERE course_code = '$course_code'";
     $resWeeks = Database::query($sqlWeeks);
     $weeks = Database::fetch_object($resWeeks);
@@ -105,52 +105,52 @@ 
     $lineHeaderExport = array(null, null);
     $lineHeaderExport2 = array(null, ull);
     while ($rowe = Database::fetch_assoc($resultHeader)) {
-        $lineHeaderExport[] = utf8_decode('Work' . $rowe['week_id']);
-        $lineHeaderExport[] = utf8_decode('Forum' . $rowe['week_id']);
+        $lineHeaderExport[] = utf8_decode('Work'.$rowe['week_id']);
+        $lineHeaderExport[] = utf8_decode('Forum'.$rowe['week_id']);
         //$fila_export_encabezado[] =  utf8_decode('Eval'.$rowe['week_id']);
         //$fila_export_encabezado[] =  utf8_decode('PC'.$rowe['week_id']);
         $lineHeaderExport2[] = utf8_decode($rowe['work_title']);
         $lineHeaderExport2[] = utf8_decode($rowe['thread_title']);
         //$fila_export_encabezado2[] = utf8_decode($rowe['eval_title']);
         //$fila_export_encabezado2[] = utf8_decode($rowe['pc_title']);
-        $fila_export = array('Work' . $rowe['week_id'], 'Forum' . $rowe['week_id'], 'Eval' . $rowe['week_id'], 'PC' . $rowe['week_id']);
+        $fila_export = array('Work'.$rowe['week_id'], 'Forum'.$rowe['week_id'], 'Eval'.$rowe['week_id'], 'PC'.$rowe['week_id']);
         if ($rowe['week_id'] > (($page - 1) * 7) && $rowe['week_id'] <= (7 * $page)) {
             $ids[$rowe['week_id']] = $rowe['id'];
-            $line.='<th>
-                <a href="#" onClick="showContent(' . "'tarea" . $rowe['week_id'] . "'" . ');">Work' . $rowe['week_id'] . '
-                        <div class="blackboard_hide" id="tarea' . $rowe['week_id'] . '">' . $rowe['work_title'] . '</div>
+            $line .= '<th>
+                <a href="#" onClick="showContent(' . "'tarea".$rowe['week_id']."'".');">Work'.$rowe['week_id'].'
+                        <div class="blackboard_hide" id="tarea' . $rowe['week_id'].'">'.$rowe['work_title'].'</div>
                 </a></th>';
-            $line.= '<th>
-                <a href="#" onClick="showContent(' . "'foro" . $rowe['week_id'] . "'" . ');">Forum' . $rowe['week_id'] . '
-                        <div class="blackboard_hide" id="foro' . $rowe['week_id'] . '">' . $rowe['thread_title'] . '</div>
+            $line .= '<th>
+                <a href="#" onClick="showContent(' . "'foro".$rowe['week_id']."'".');">Forum'.$rowe['week_id'].'
+                        <div class="blackboard_hide" id="foro' . $rowe['week_id'].'">'.$rowe['thread_title'].'</div>
                 </a>
                 </th>';
         }
     }
     $tableExport[] = $lineHeaderExport;
     $tableExport[] = $lineHeaderExport2;
-    $line.= '</tr>';
+    $line .= '</tr>';
 
     $html = '<form action="tutor.php" name="semanas" id="semanas" method="POST">
             <div class="row">
-            ' . get_lang('SelectWeeksSpan') . '
+            ' . get_lang('SelectWeeksSpan').'
             <select name="weeksNumber" id="weeksNumber" onChange="submit();">
-            <option value="7" ' . (($weeksCount == 7) ? 'selected="selected"' : "") . '>7 weeks</option>
-            <option value="14" ' . (($weeksCount == 14) ? 'selected="selected"' : "") . '>14 weeks</option>
+            <option value="7" ' . (($weeksCount == 7) ? 'selected="selected"' : "").'>7 weeks</option>
+            <option value="14" ' . (($weeksCount == 14) ? 'selected="selected"' : "").'>14 weeks</option>
             </select>';
 
 
     if ($weeksCount == 14) {
-        $html .= '<span style="float:right;"><a href="tutor.php?page=' . (($page == 1) ? 2 : 1) . '">' . (($page == 1) ? "Siguiente" : "Anterior") . '</a></span>';
+        $html .= '<span style="float:right;"><a href="tutor.php?page='.(($page == 1) ? 2 : 1).'">'.(($page == 1) ? "Siguiente" : "Anterior").'</a></span>';
     }
-    $html .= '<span style="float:right;"><a href="' . api_get_self() . '?action=export' . $get_parameter . $get_parameter2 . '">' . Display::return_icon('export_excel.png', get_lang('Export'), '', '32') . '</a></span>';
+    $html .= '<span style="float:right;"><a href="'.api_get_self().'?action=export'.$get_parameter.$get_parameter2.'">'.Display::return_icon('export_excel.png', get_lang('Export'), '', '32').'</a></span>';
 
     $html .= '</form>';
     $html .= '<table class="reports">';
     $html .= '<tr>
             <th ></th>';
     for ($i = (7 * $page - 6); $i <= $page * 7; $i++) {
-        $html .= '<th colspan="2">Week ' . $i . '<a href="assign_tickets.php?id=' . $ids[$i] . '" class="ajax">' . Display::return_icon('edit.png', get_lang('Edit'), array('width' => '16', 'height' => '16'), 22) . '</a></th>';
+        $html .= '<th colspan="2">Week '.$i.'<a href="assign_tickets.php?id='.$ids[$i].'" class="ajax">'.Display::return_icon('edit.png', get_lang('Edit'), array('width' => '16', 'height' => '16'), 22).'</a></th>';
     }
     $html .= '</tr>';
     $html .= $line;
@@ -167,7 +167,7 @@ 
         if ($row['week_id'] > (($page - 1) * 7) && $row['week_id'] <= (7 * $page)) {
             $results[$row['username']][$row['week_id']] = $row;
             if (count($results[$row['username']]) == 7) {
-                $html.= showStudentResult($results[$row['username']], $page);
+                $html .= showStudentResult($results[$row['username']], $page);
             }
         }
         if (count($resultadose[$row['username']]) == $weeksCount) {
@@ -190,12 +190,12 @@ 
     $inicio = (7 * $pagina - 6);
     $fila = '<tr>';
 
-    $fila.= '<td><a href="' . api_get_path(WEB_CODE_PATH) . 'user/userInfo.php?' . api_get_cidreq() . '&uInfo=' . $datos[$inicio]['user_id'] . '">' . $datos[$inicio]['username'] . '</a></td>';
+    $fila .= '<td><a href="'.api_get_path(WEB_CODE_PATH).'user/userInfo.php?'.api_get_cidreq().'&uInfo='.$datos[$inicio]['user_id'].'">'.$datos[$inicio]['username'].'</a></td>';
     foreach ($datos as $dato) {
-        $fila.= '<td align="center">' . (($dato['work_ok'] == 1) ? Display::return_icon('check.png') : Display::return_icon('aspa.png')) . '</td>';
-        $fila.= '<td align="center">' . (($dato['thread_ok'] == 1) ? Display::return_icon('check.png') : Display::return_icon('aspa.png')) . '</td>';
+        $fila .= '<td align="center">'.(($dato['work_ok'] == 1) ? Display::return_icon('check.png') : Display::return_icon('aspa.png')).'</td>';
+        $fila .= '<td align="center">'.(($dato['thread_ok'] == 1) ? Display::return_icon('check.png') : Display::return_icon('aspa.png')).'</td>';
     }
-    $fila.= '</tr>';
+    $fila .= '</tr>';
     return $fila;
 }
 

src/Chamilo/CoreBundle/Component/Editor/CkEditor/CkEditor.php

Doc Comments +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
     /**
      * @param array $templates
      *
-     * @return null
+     * @return null|string
      */
     public function formatTemplates($templates)
     {

Spacing +7 added lines, -7 removed lines

@@ -145,14 +145,14 @@ 
         return [[
             'title' => get_lang('EmptyTemplate'),
             'description' => null,
-            'image' => api_get_path(WEB_APP_PATH) . 'home/default_platform_document/template_thumb/empty.gif',
+            'image' => api_get_path(WEB_APP_PATH).'home/default_platform_document/template_thumb/empty.gif',
             'html' => '
                 <!DOCYTPE html>
                 <html>
                     <head>
-                        <meta charset="' . api_get_system_encoding() . '" />
+                        <meta charset="' . api_get_system_encoding().'" />
                     </head>
-                    <body  dir="' . api_get_text_direction() . '">
+                    <body  dir="' . api_get_text_direction().'">
                         <p>
                             <br/>
                         </p>
@@ -175,7 +175,7 @@ 
         $search = array('{CSS}', '{IMG_DIR}', '{REL_PATH}', '{COURSE_DIR}');
         $replace = array(
             '',
-            api_get_path(REL_CODE_PATH) . 'img/',
+            api_get_path(REL_CODE_PATH).'img/',
             api_get_path(REL_PATH),
             api_get_path(REL_DEFAULT_COURSE_DOCUMENT_PATH),
             api_get_path(REL_DEFAULT_COURSE_DOCUMENT_PATH)
@@ -186,7 +186,7 @@ 
         foreach ($systemTemplates as $template) {
             $image = $template->getImage();
             $image = !empty($image) ? $image : 'empty.gif';
-            $image = api_get_path(WEB_APP_PATH) . 'home/default_platform_document/template_thumb/' . $image;
+            $image = api_get_path(WEB_APP_PATH).'home/default_platform_document/template_thumb/'.$image;
 
             /*$image = $this->urlGenerator->generate(
                 'get_document_template_action',
@@ -238,12 +238,12 @@ 
             $templateItem['image'] = api_get_path(WEB_APP_PATH)
                 . 'home/default_platform_document/template_thumb/noimage.gif';
             $templateItem['html'] = file_get_contents(api_get_path(SYS_COURSE_PATH)
-                . $courseDirectory . '/document' . $templateData['path']);
+                . $courseDirectory.'/document'.$templateData['path']);
 
             $image = $template->getImage();
             if (!empty($image)) {
                 $templateItem['image'] = api_get_path(WEB_COURSE_PATH)
-                    . $courseDirectory . '/upload/template_thumbnails/' . $template->getImage();
+                    . $courseDirectory.'/upload/template_thumbnails/'.$template->getImage();
             }
 
             $templateList[] = $templateItem;

src/Chamilo/CoreBundle/Component/Editor/Connector.php

Doc Comments +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
 
     /**
      * Available driver list.
-     * @return array
+     * @return string[]
      */
     private function getDefaultDriverList()
     {

Indentation +4 added lines, -4 removed lines

@@ -51,7 +51,7 @@ 
             'root_sys' => api_get_path(SYS_PATH),
             'sys_root' => api_get_path(SYS_PATH), // just an alias
             'sys_course_path' => api_get_path(SYS_COURSE_PATH),
-         //   'sys_config_path' => $app['path.config'],
+            //   'sys_config_path' => $app['path.config'],
             'path.temp' => api_get_path(SYS_ARCHIVE_PATH),
             //'sys_log_path' => $app['path.logs']
         );
@@ -325,9 +325,9 @@ 
      **/
     public function access($attr, $path, $data, $volume)
     {
-    	return strpos(basename($path), '.') === 0       // if file/folder begins with '.' (dot)
-    		? !($attr == 'read' || $attr == 'write')    // set read+write to false, other (locked+hidden) set to true
-    		:  null;                                    // else elFinder decide it itself
+        return strpos(basename($path), '.') === 0       // if file/folder begins with '.' (dot)
+            ? !($attr == 'read' || $attr == 'write')    // set read+write to false, other (locked+hidden) set to true
+            :  null;                                    // else elFinder decide it itself
     }
 
     /**

Spacing +1 added lines, -1 removed lines

@@ -327,7 +327,7 @@
     {
     	return strpos(basename($path), '.') === 0       // if file/folder begins with '.' (dot)
     		? !($attr == 'read' || $attr == 'write')    // set read+write to false, other (locked+hidden) set to true
-    		:  null;                                    // else elFinder decide it itself
+    		:  null; // else elFinder decide it itself
     }
 
     /**

src/Chamilo/CoreBundle/Component/Editor/Driver/DropBoxDriver.php

Doc Comments +3 added lines, -3 removed lines

@@ -133,7 +133,7 @@ 
     /**
      * Close connection
      *
-     * @return void
+     * @return boolean
      * @author Dmitry (dio) Levashov
      **/
     public function umount() {
@@ -488,7 +488,7 @@ 
      * Close opened file
      *
      * @param  resource  $fp  file pointer
-     * @return bool
+     * @return boolean|null
      * @author Dmitry (dio) Levashov
      **/
     protected function _fclose($fp, $path='') {
@@ -505,7 +505,7 @@ 
      *
      * @param  string  $path  parent dir path
      * @param string  $name  new directory name
-     * @return string|bool
+     * @return string|false
      * @author Dmitry (dio) Levashov
      **/
     protected function _mkdir($path, $name) {

Indentation +2 added lines, -2 removed lines

@@ -633,8 +633,8 @@
     }
 
     /**
-    * {@inheritdoc}
-    */
+     * {@inheritdoc}
+     */
     protected function _archive($dir, $files, $name, $arc) {
         return false;
     }

Spacing +2 added lines, -2 removed lines

@@ -464,7 +464,7 @@ 
      * @return resource|false
      * @author Dmitry (dio) Levashov
      **/
-    protected function _fopen($path, $mode='rb') {
+    protected function _fopen($path, $mode = 'rb') {
         $fp = $this->tmbPath
             ? @fopen($this->tmpname($path), 'w+')
             : @tmpfile();
@@ -491,7 +491,7 @@ 
      * @return bool
      * @author Dmitry (dio) Levashov
      **/
-    protected function _fclose($fp, $path='') {
+    protected function _fclose($fp, $path = '') {
         @fclose($fp);
         if ($path) {
             @unlink($this->tmpname($path));

src/Chamilo/CoreBundle/Entity/BranchSync.php

Doc Comments +1 added lines, -2 removed lines

@@ -532,7 +532,6 @@ 
      /**
      * Set sslPubKey
      *
-     * @param string $sslPubKey
      * @return BranchSync
      */
     public function setBranchType($branchType)
@@ -714,7 +713,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return null|BranchSync
      */
     public function getParent()
     {

Indentation +11 added lines, -11 removed lines

@@ -136,11 +136,11 @@ 
      */
     private $lastSyncType;
 
-     /**
-     * @var string
-     *
-     * @ORM\Column(name="ssl_pub_key", type="string", length=250, nullable=true, unique=false)
-     */
+        /**
+         * @var string
+         *
+         * @ORM\Column(name="ssl_pub_key", type="string", length=250, nullable=true, unique=false)
+         */
     private $sslPubKey;
 
     /**
@@ -529,12 +529,12 @@ 
         return $this->sslPubKey;
     }
 
-     /**
-     * Set sslPubKey
-     *
-     * @param string $sslPubKey
-     * @return BranchSync
-     */
+        /**
+         * Set sslPubKey
+         *
+         * @param string $sslPubKey
+         * @return BranchSync
+         */
     public function setBranchType($branchType)
     {
         $this->branchType = $branchType;

src/Chamilo/CoreBundle/Entity/Course.php

Doc Comments +2 added lines, -2 removed lines

@@ -318,7 +318,7 @@ 
     }
 
     /**
-     * @return ArrayCollection
+     * @return CTool[]
      */
     public function getTools()
     {
@@ -1146,7 +1146,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function getStatusList()
     {

src/Chamilo/CoreBundle/Entity/CourseRelUser.php

Doc Comments +1 added lines, -1 removed lines

@@ -139,7 +139,7 @@
     }
 
     /**
-     * @param $user
+     * @param User $user
      * @return $this
      */
     public function setUser($user)