chamilo / chamilo-lms

main/tracking/lp_results_by_user.php

Indentation +1 added lines, -1 removed lines

@@ -100,7 +100,7 @@
 
         $menu_items[] = '<a href="'.api_get_path(WEB_CODE_PATH).'mySpace/?view=teacher">'.get_lang('TeacherInterface').'</a>';
         if (api_is_platform_admin()) {
-          $menu_items[] = '<a href="'.api_get_path(WEB_CODE_PATH).'mySpace/?view=admin">'.get_lang('AdminInterface').'</a>';
+            $menu_items[] = '<a href="'.api_get_path(WEB_CODE_PATH).'mySpace/?view=admin">'.get_lang('AdminInterface').'</a>';
         } else {
             $menu_items[] = '<a href="'.api_get_path(WEB_CODE_PATH).'mySpace/?view=coach">'.get_lang('AdminInterface').'</a>';
         }

main/webservices/cm_webservice.php

Doc Comments +5 added lines, -2 removed lines

@@ -42,6 +42,8 @@ 
      *
      * @param int Error code
      * @param string Error message
+     * @param integer $code
+     * @param string $message
      */
     public function __construct($code, $message)
     {
@@ -91,6 +93,7 @@ 
      * Handle method
      *
      * @param WSError $error Error
+     * @return void
      */
     public function handle($error);
 }
@@ -119,7 +122,7 @@ 
      * Verifies the API key
      *
      * @param string $secret_key Secret key
-     * @return mixed WSError in case of failure, null in case of success
+     * @return WSCMError|null WSError in case of failure, null in case of success
      */
     protected function verifyKey($secret_key)
     {
@@ -145,7 +148,7 @@ 
      * @param string $username of the user in chamilo
      * @param string $pass of the same user (in MD5 of SHA)
      *
-     * @return mixed "valid" if username e password are correct! Else, return a message error
+     * @return string "valid" if username e password are correct! Else, return a message error
      */
     public function verifyUserPass($username, $pass)
     {

main/inc/lib/career.lib.php

Spacing +5 added lines, -5 removed lines

@@ -347,7 +347,7 @@ 
                                     $pos = strpos($explode[0], 'G');
                                     if (is_numeric($pos)) {
                                         // group_123 id
-                                        $groupValueId = (int)str_replace(
+                                        $groupValueId = (int) str_replace(
                                             'G',
                                             '',
                                             $explode[0]
@@ -362,14 +362,14 @@ 
                                     }
                                 } else {
                                     // subgroup__123 id
-                                    $firstConnection = 'subgroup_'.(int)str_replace('SG', '', $explode[0]);
+                                    $firstConnection = 'subgroup_'.(int) str_replace('SG', '', $explode[0]);
                                 }
 
                                 $pos = strpos($explode[1], 'SG');
                                 if ($pos === false) {
                                     $pos = strpos($explode[1], 'G');
                                     if (is_numeric($pos)) {
-                                        $groupValueId = (int)str_replace(
+                                        $groupValueId = (int) str_replace(
                                             'G',
                                             '',
                                             $explode[1]
@@ -383,7 +383,7 @@ 
                                         }
                                     }
                                 } else {
-                                    $secondConnection = 'subgroup_'.(int)str_replace('SG', '', $explode[1]);
+                                    $secondConnection = 'subgroup_'.(int) str_replace('SG', '', $explode[1]);
                                 }
 
                                 if (!empty($firstConnection) && !empty($firstConnection)) {
@@ -487,7 +487,7 @@ 
                         if ($subGroup == '' || empty($subGroup)) {
                             $defaultSubGroup = 0;
                         } else {
-                            $defaultSubGroup = (int)$subGroup;
+                            $defaultSubGroup = (int) $subGroup;
                         }
                     }
                     $newRowList[$i + 1][$defaultSubGroup][] = $vertex;

Doc Comments +3 added lines, -3 removed lines

@@ -89,7 +89,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public function get_status_list()
     {
@@ -427,7 +427,7 @@ 
      * @param int $group
      * @param bool $showGroupLine
      * @param array $subGroupList
-     * @param $widthGroup
+     * @param integer $widthGroup
      * @return string
      */
     public static function parseSubGroups(
@@ -590,7 +590,7 @@ 
     /**
      * @param string $source
      * @param string $target
-     * @param array $anchor
+     * @param string[] $anchor
      * @return string
      */
     public static function createConnection($source, $target, $anchor = [])

main/inc/lib/statistics.lib.php

Indentation +2 added lines, -2 removed lines

@@ -565,7 +565,7 @@ 
                     WHERE 
                         login_date BETWEEN '$startDate' AND '$endDate'
                         $where_url";
-             $sqlList[$label] = $sql;
+                $sqlList[$label] = $sql;
         }
         $sqlList[get_lang('Total')] = "SELECT count($field) AS number FROM $table $table_url WHERE 1=1 $where_url";
 
@@ -724,7 +724,7 @@ 
         $res = Database::query($sql);
         $count1 = Database::fetch_object($res);
         $sql = "SELECT COUNT(*) AS n FROM $user_table as u $table ".
-               "WHERE LENGTH(picture_uri) > 0 $url_condition2";
+                "WHERE LENGTH(picture_uri) > 0 $url_condition2";
         $res = Database::query($sql);
         $count2 = Database::fetch_object($res);
         // #users without picture

main/inc/lib/exercise_show_functions.lib.php

Doc Comments +30 added lines, -6 removed lines

@@ -72,6 +72,11 @@ 
      * @param string    Answer text
      * @param int       Exercise ID
      * @param int       Question ID
+     * @param integer $feedback_type
+     * @param string $answer
+     * @param integer $id
+     * @param integer $questionId
+     * @param boolean|string $showTotalScoreAndUserChoices
      * @return void
      */
     public static function display_calculated_answer(
@@ -111,6 +116,9 @@ 
      * @param string    Answer text
      * @param int       Exercise ID
      * @param int       Question ID
+     * @param integer $feedback_type
+     * @param integer $exe_id
+     * @param integer $questionId
      * @return void
      */
     public static function display_free_answer(
@@ -140,10 +148,10 @@ 
     }
 
     /**
-     * @param $feedback_type
+     * @param integer $feedback_type
      * @param $answer
-     * @param $id
-     * @param $questionId
+     * @param integer $id
+     * @param integer $questionId
      * @param null $fileUrl
      * @param int $results_disabled
      * @param int $questionScore
@@ -385,6 +393,14 @@ 
      * @param integer Exercise ID
      * @param integer Question ID
      * @param boolean Whether to show the answer comment or not
+     * @param integer $feedback_type
+     * @param integer $answerType
+     * @param string $answer
+     * @param string $answerComment
+     * @param integer $answerCorrect
+     * @param integer $id
+     * @param integer $questionId
+     * @param boolean $showTotalScoreAndUserChoices
      * @return void
      */
     public static function display_multiple_answer_true_false(
@@ -489,6 +505,14 @@ 
      * @param integer Exercise ID
      * @param integer Question ID
      * @param boolean Whether to show the answer comment or not
+     * @param integer $feedback_type
+     * @param integer $answerType
+     * @param string $answer
+     * @param string $answerComment
+     * @param integer $answerCorrect
+     * @param integer $id
+     * @param integer $questionId
+     * @param boolean $showTotalScoreAndUserChoices
      * @return void
      */
     public static function display_multiple_answer_combination_true_false(
@@ -578,9 +602,9 @@ 
     }
 
     /**
-    * @param $feedback_type
-    * @param $exe_id
-    * @param $questionId
+    * @param integer $feedback_type
+    * @param integer $exe_id
+    * @param integer $questionId
     * @param null $questionScore
     * @param int $results_disabled
     */

Indentation +20 added lines, -20 removed lines

@@ -478,19 +478,19 @@ 
         <?php
     }
 
-     /**
-     * Display the answers to a multiple choice question
-     *
-     * @param integer Answer type
-     * @param integer Student choice
-     * @param string  Textual answer
-     * @param string  Comment on answer
-     * @param string  Correct answer comment
-     * @param integer Exercise ID
-     * @param integer Question ID
-     * @param boolean Whether to show the answer comment or not
-     * @return void
-     */
+        /**
+         * Display the answers to a multiple choice question
+         *
+         * @param integer Answer type
+         * @param integer Student choice
+         * @param string  Textual answer
+         * @param string  Comment on answer
+         * @param string  Correct answer comment
+         * @param integer Exercise ID
+         * @param integer Question ID
+         * @param boolean Whether to show the answer comment or not
+         * @return void
+         */
     public static function display_multiple_answer_combination_true_false(
         $feedback_type,
         $answerType,
@@ -556,7 +556,7 @@ 
             <?php
             //@todo replace this harcoded value
             if ($studentChoice) {
-                 $color = "black";
+                    $color = "black";
                 if ($studentChoice == $answerCorrect) {
                     $color = "green";
                 }
@@ -578,12 +578,12 @@ 
     }
 
     /**
-    * @param $feedback_type
-    * @param $exe_id
-    * @param $questionId
-    * @param null $questionScore
-    * @param int $results_disabled
-    */
+     * @param $feedback_type
+     * @param $exe_id
+     * @param $questionId
+     * @param null $questionScore
+     * @param int $results_disabled
+     */
     public static function displayAnnotationAnswer(
         $feedback_type,
         $exe_id,

Braces +1 added lines, -2 removed lines

@@ -156,8 +156,7 @@
         $fileUrl = null,
         $results_disabled = 0,
         $questionScore = 0
-    )
-    {
+    ) {
         if (isset($fileUrl)) {
             echo '
                 <tr>

main/session/index.php

Indentation +1 added lines, -1 removed lines

@@ -550,7 +550,7 @@
         }
     }
 <?php
-     //Displays js code to use a jqgrid
+        //Displays js code to use a jqgrid
     echo Display::grid_js(
         'courses',
         '',

main/session/session_category_edit.php

Braces +354 added lines, -88 removed lines

@@ -85,7 +85,12 @@ 
         <div class="form-group">
             <label class="col-sm-3 control-label"><?php echo get_lang('Name') ?></label>
             <div class="col-sm-6">
-                <input class="form-control" type="text" name="name" size="50" maxlength="50" value="<?php if ($formSent) echo api_htmlentities($name, ENT_QUOTES, $charset); else echo api_htmlentities($infos['name'], ENT_QUOTES, $charset); ?>">
+                <input class="form-control" type="text" name="name" size="50" maxlength="50" value="<?php if ($formSent) {
+    echo api_htmlentities($name, ENT_QUOTES, $charset);
+} else {
+    echo api_htmlentities($infos['name'], ENT_QUOTES, $charset);
+}
+?>">
             </div>
             <div class="col-sm-3"></div>
         </div>
@@ -101,58 +106,184 @@ 
                 <div class="col-sm-6">
                     <select name="day_start">
                         <option value="1">01</option>
-                        <option value="2" <?php if ($day_start == 2) echo 'selected="selected"'; ?> >02</option>
-                        <option value="3" <?php if ($day_start == 3) echo 'selected="selected"'; ?> >03</option>
-                        <option value="4" <?php if ($day_start == 4) echo 'selected="selected"'; ?> >04</option>
-                        <option value="5" <?php if ($day_start == 5) echo 'selected="selected"'; ?> >05</option>
-                        <option value="6" <?php if ($day_start == 6) echo 'selected="selected"'; ?> >06</option>
-                        <option value="7" <?php if ($day_start == 7) echo 'selected="selected"'; ?> >07</option>
-                        <option value="8" <?php if ($day_start == 8) echo 'selected="selected"'; ?> >08</option>
-                        <option value="9" <?php if ($day_start == 9) echo 'selected="selected"'; ?> >09</option>
-                        <option value="10" <?php if ($day_start == 10) echo 'selected="selected"'; ?> >10</option>
-                        <option value="11" <?php if ($day_start == 11) echo 'selected="selected"'; ?> >11</option>
-                        <option value="12" <?php if ($day_start == 12) echo 'selected="selected"'; ?> >12</option>
-                        <option value="13" <?php if ($day_start == 13) echo 'selected="selected"'; ?> >13</option>
-                        <option value="14" <?php if ($day_start == 14) echo 'selected="selected"'; ?> >14</option>
-                        <option value="15" <?php if ($day_start == 15) echo 'selected="selected"'; ?> >15</option>
-                        <option value="16" <?php if ($day_start == 16) echo 'selected="selected"'; ?> >16</option>
-                        <option value="17" <?php if ($day_start == 17) echo 'selected="selected"'; ?> >17</option>
-                        <option value="18" <?php if ($day_start == 18) echo 'selected="selected"'; ?> >18</option>
-                        <option value="19" <?php if ($day_start == 19) echo 'selected="selected"'; ?> >19</option>
-                        <option value="20" <?php if ($day_start == 20) echo 'selected="selected"'; ?> >20</option>
-                        <option value="21" <?php if ($day_start == 21) echo 'selected="selected"'; ?> >21</option>
-                        <option value="22" <?php if ($day_start == 22) echo 'selected="selected"'; ?> >22</option>
-                        <option value="23" <?php if ($day_start == 23) echo 'selected="selected"'; ?> >23</option>
-                        <option value="24" <?php if ($day_start == 24) echo 'selected="selected"'; ?> >24</option>
-                        <option value="25" <?php if ($day_start == 25) echo 'selected="selected"'; ?> >25</option>
-                        <option value="26" <?php if ($day_start == 26) echo 'selected="selected"'; ?> >26</option>
-                        <option value="27" <?php if ($day_start == 27) echo 'selected="selected"'; ?> >27</option>
-                        <option value="28" <?php if ($day_start == 28) echo 'selected="selected"'; ?> >28</option>
-                        <option value="29" <?php if ($day_start == 29) echo 'selected="selected"'; ?> >29</option>
-                        <option value="30" <?php if ($day_start == 30) echo 'selected="selected"'; ?> >30</option>
-                        <option value="31" <?php if ($day_start == 31) echo 'selected="selected"'; ?> >31</option>
+                        <option value="2" <?php if ($day_start == 2) {
+    echo 'selected="selected"';
+}
+?> >02</option>
+                        <option value="3" <?php if ($day_start == 3) {
+    echo 'selected="selected"';
+}
+?> >03</option>
+                        <option value="4" <?php if ($day_start == 4) {
+    echo 'selected="selected"';
+}
+?> >04</option>
+                        <option value="5" <?php if ($day_start == 5) {
+    echo 'selected="selected"';
+}
+?> >05</option>
+                        <option value="6" <?php if ($day_start == 6) {
+    echo 'selected="selected"';
+}
+?> >06</option>
+                        <option value="7" <?php if ($day_start == 7) {
+    echo 'selected="selected"';
+}
+?> >07</option>
+                        <option value="8" <?php if ($day_start == 8) {
+    echo 'selected="selected"';
+}
+?> >08</option>
+                        <option value="9" <?php if ($day_start == 9) {
+    echo 'selected="selected"';
+}
+?> >09</option>
+                        <option value="10" <?php if ($day_start == 10) {
+    echo 'selected="selected"';
+}
+?> >10</option>
+                        <option value="11" <?php if ($day_start == 11) {
+    echo 'selected="selected"';
+}
+?> >11</option>
+                        <option value="12" <?php if ($day_start == 12) {
+    echo 'selected="selected"';
+}
+?> >12</option>
+                        <option value="13" <?php if ($day_start == 13) {
+    echo 'selected="selected"';
+}
+?> >13</option>
+                        <option value="14" <?php if ($day_start == 14) {
+    echo 'selected="selected"';
+}
+?> >14</option>
+                        <option value="15" <?php if ($day_start == 15) {
+    echo 'selected="selected"';
+}
+?> >15</option>
+                        <option value="16" <?php if ($day_start == 16) {
+    echo 'selected="selected"';
+}
+?> >16</option>
+                        <option value="17" <?php if ($day_start == 17) {
+    echo 'selected="selected"';
+}
+?> >17</option>
+                        <option value="18" <?php if ($day_start == 18) {
+    echo 'selected="selected"';
+}
+?> >18</option>
+                        <option value="19" <?php if ($day_start == 19) {
+    echo 'selected="selected"';
+}
+?> >19</option>
+                        <option value="20" <?php if ($day_start == 20) {
+    echo 'selected="selected"';
+}
+?> >20</option>
+                        <option value="21" <?php if ($day_start == 21) {
+    echo 'selected="selected"';
+}
+?> >21</option>
+                        <option value="22" <?php if ($day_start == 22) {
+    echo 'selected="selected"';
+}
+?> >22</option>
+                        <option value="23" <?php if ($day_start == 23) {
+    echo 'selected="selected"';
+}
+?> >23</option>
+                        <option value="24" <?php if ($day_start == 24) {
+    echo 'selected="selected"';
+}
+?> >24</option>
+                        <option value="25" <?php if ($day_start == 25) {
+    echo 'selected="selected"';
+}
+?> >25</option>
+                        <option value="26" <?php if ($day_start == 26) {
+    echo 'selected="selected"';
+}
+?> >26</option>
+                        <option value="27" <?php if ($day_start == 27) {
+    echo 'selected="selected"';
+}
+?> >27</option>
+                        <option value="28" <?php if ($day_start == 28) {
+    echo 'selected="selected"';
+}
+?> >28</option>
+                        <option value="29" <?php if ($day_start == 29) {
+    echo 'selected="selected"';
+}
+?> >29</option>
+                        <option value="30" <?php if ($day_start == 30) {
+    echo 'selected="selected"';
+}
+?> >30</option>
+                        <option value="31" <?php if ($day_start == 31) {
+    echo 'selected="selected"';
+}
+?> >31</option>
                   </select>
                   /
                   <select name="month_start">
                         <option value="1">01</option>
-                        <option value="2" <?php if ($month_start == 2) echo 'selected="selected"'; ?> >02</option>
-                        <option value="3" <?php if ($month_start == 3) echo 'selected="selected"'; ?> >03</option>
-                        <option value="4" <?php if ($month_start == 4) echo 'selected="selected"'; ?> >04</option>
-                        <option value="5" <?php if ($month_start == 5) echo 'selected="selected"'; ?> >05</option>
-                        <option value="6" <?php if ($month_start == 6) echo 'selected="selected"'; ?> >06</option>
-                        <option value="7" <?php if ($month_start == 7) echo 'selected="selected"'; ?> >07</option>
-                        <option value="8" <?php if ($month_start == 8) echo 'selected="selected"'; ?> >08</option>
-                        <option value="9" <?php if ($month_start == 9) echo 'selected="selected"'; ?> >09</option>
-                        <option value="10" <?php if ($month_start == 10) echo 'selected="selected"'; ?> >10</option>
-                        <option value="11" <?php if ($month_start == 11) echo 'selected="selected"'; ?> >11</option>
-                        <option value="12" <?php if ($month_start == 12) echo 'selected="selected"'; ?> >12</option>
+                        <option value="2" <?php if ($month_start == 2) {
+    echo 'selected="selected"';
+}
+?> >02</option>
+                        <option value="3" <?php if ($month_start == 3) {
+    echo 'selected="selected"';
+}
+?> >03</option>
+                        <option value="4" <?php if ($month_start == 4) {
+    echo 'selected="selected"';
+}
+?> >04</option>
+                        <option value="5" <?php if ($month_start == 5) {
+    echo 'selected="selected"';
+}
+?> >05</option>
+                        <option value="6" <?php if ($month_start == 6) {
+    echo 'selected="selected"';
+}
+?> >06</option>
+                        <option value="7" <?php if ($month_start == 7) {
+    echo 'selected="selected"';
+}
+?> >07</option>
+                        <option value="8" <?php if ($month_start == 8) {
+    echo 'selected="selected"';
+}
+?> >08</option>
+                        <option value="9" <?php if ($month_start == 9) {
+    echo 'selected="selected"';
+}
+?> >09</option>
+                        <option value="10" <?php if ($month_start == 10) {
+    echo 'selected="selected"';
+}
+?> >10</option>
+                        <option value="11" <?php if ($month_start == 11) {
+    echo 'selected="selected"';
+}
+?> >11</option>
+                        <option value="12" <?php if ($month_start == 12) {
+    echo 'selected="selected"';
+}
+?> >12</option>
                   </select>
                   /
                 <select name="year_start">
                         <?php
                         for ($i = $thisYear - 5; $i <= ($thisYear + 5); $i++)
                         { ?>
-                                <option value="<?php echo $i; ?>" <?php if ($year_start == $i) echo 'selected="selected"'; ?> ><?php echo $i; ?></option>
+                                <option value="<?php echo $i; ?>" <?php if ($year_start == $i) {
+    echo 'selected="selected"';
+}
+?> ><?php echo $i; ?></option>
                         <?php
                         } ?>
                 </select>
@@ -164,53 +295,182 @@ 
                 <div class="col-sm-6">
                     <select name="day_end">
                         <option value="0">--</option>
-                        <option value="1" <?php if ($day_end == 1) echo 'selected="selected"'; ?> >01</option>
-                        <option value="2" <?php if ($day_end == 2) echo 'selected="selected"'; ?> >02</option>
-                        <option value="3" <?php if ($day_end == 3) echo 'selected="selected"'; ?> >03</option>
-                        <option value="4" <?php if ($day_end == 4) echo 'selected="selected"'; ?> >04</option>
-                        <option value="5" <?php if ($day_end == 5) echo 'selected="selected"'; ?> >05</option>
-                        <option value="6" <?php if ($day_end == 6) echo 'selected="selected"'; ?> >06</option>
-                        <option value="7" <?php if ($day_end == 7) echo 'selected="selected"'; ?> >07</option>
-                        <option value="8" <?php if ($day_end == 8) echo 'selected="selected"'; ?> >08</option>
-                        <option value="9" <?php if ($day_end == 9) echo 'selected="selected"'; ?> >09</option>
-                        <option value="10" <?php if ($day_end == 10) echo 'selected="selected"'; ?> >10</option>
-                        <option value="11" <?php if ($day_end == 11) echo 'selected="selected"'; ?> >11</option>
-                        <option value="12" <?php if ($day_end == 12) echo 'selected="selected"'; ?> >12</option>
-                        <option value="13" <?php if ($day_end == 13) echo 'selected="selected"'; ?> >13</option>
-                        <option value="14" <?php if ($day_end == 14) echo 'selected="selected"'; ?> >14</option>
-                        <option value="15" <?php if ($day_end == 15) echo 'selected="selected"'; ?> >15</option>
-                        <option value="16" <?php if ($day_end == 16) echo 'selected="selected"'; ?> >16</option>
-                        <option value="17" <?php if ($day_end == 17) echo 'selected="selected"'; ?> >17</option>
-                        <option value="18" <?php if ($day_end == 18) echo 'selected="selected"'; ?> >18</option>
-                        <option value="19" <?php if ($day_end == 19) echo 'selected="selected"'; ?> >19</option>
-                        <option value="20" <?php if ($day_end == 20) echo 'selected="selected"'; ?> >20</option>
-                        <option value="21" <?php if ($day_end == 21) echo 'selected="selected"'; ?> >21</option>
-                        <option value="22" <?php if ($day_end == 22) echo 'selected="selected"'; ?> >22</option>
-                        <option value="23" <?php if ($day_end == 23) echo 'selected="selected"'; ?> >23</option>
-                        <option value="24" <?php if ($day_end == 24) echo 'selected="selected"'; ?> >24</option>
-                        <option value="25" <?php if ($day_end == 25) echo 'selected="selected"'; ?> >25</option>
-                        <option value="26" <?php if ($day_end == 26) echo 'selected="selected"'; ?> >26</option>
-                        <option value="27" <?php if ($day_end == 27) echo 'selected="selected"'; ?> >27</option>
-                        <option value="28" <?php if ($day_end == 28) echo 'selected="selected"'; ?> >28</option>
-                        <option value="29" <?php if ($day_end == 29) echo 'selected="selected"'; ?> >29</option>
-                        <option value="30" <?php if ($day_end == 30) echo 'selected="selected"'; ?> >30</option>
-                        <option value="31" <?php if ($day_end == 31) echo 'selected="selected"'; ?> >31</option>
+                        <option value="1" <?php if ($day_end == 1) {
+    echo 'selected="selected"';
+}
+?> >01</option>
+                        <option value="2" <?php if ($day_end == 2) {
+    echo 'selected="selected"';
+}
+?> >02</option>
+                        <option value="3" <?php if ($day_end == 3) {
+    echo 'selected="selected"';
+}
+?> >03</option>
+                        <option value="4" <?php if ($day_end == 4) {
+    echo 'selected="selected"';
+}
+?> >04</option>
+                        <option value="5" <?php if ($day_end == 5) {
+    echo 'selected="selected"';
+}
+?> >05</option>
+                        <option value="6" <?php if ($day_end == 6) {
+    echo 'selected="selected"';
+}
+?> >06</option>
+                        <option value="7" <?php if ($day_end == 7) {
+    echo 'selected="selected"';
+}
+?> >07</option>
+                        <option value="8" <?php if ($day_end == 8) {
+    echo 'selected="selected"';
+}
+?> >08</option>
+                        <option value="9" <?php if ($day_end == 9) {
+    echo 'selected="selected"';
+}
+?> >09</option>
+                        <option value="10" <?php if ($day_end == 10) {
+    echo 'selected="selected"';
+}
+?> >10</option>
+                        <option value="11" <?php if ($day_end == 11) {
+    echo 'selected="selected"';
+}
+?> >11</option>
+                        <option value="12" <?php if ($day_end == 12) {
+    echo 'selected="selected"';
+}
+?> >12</option>
+                        <option value="13" <?php if ($day_end == 13) {
+    echo 'selected="selected"';
+}
+?> >13</option>
+                        <option value="14" <?php if ($day_end == 14) {
+    echo 'selected="selected"';
+}
+?> >14</option>
+                        <option value="15" <?php if ($day_end == 15) {
+    echo 'selected="selected"';
+}
+?> >15</option>
+                        <option value="16" <?php if ($day_end == 16) {
+    echo 'selected="selected"';
+}
+?> >16</option>
+                        <option value="17" <?php if ($day_end == 17) {
+    echo 'selected="selected"';
+}
+?> >17</option>
+                        <option value="18" <?php if ($day_end == 18) {
+    echo 'selected="selected"';
+}
+?> >18</option>
+                        <option value="19" <?php if ($day_end == 19) {
+    echo 'selected="selected"';
+}
+?> >19</option>
+                        <option value="20" <?php if ($day_end == 20) {
+    echo 'selected="selected"';
+}
+?> >20</option>
+                        <option value="21" <?php if ($day_end == 21) {
+    echo 'selected="selected"';
+}
+?> >21</option>
+                        <option value="22" <?php if ($day_end == 22) {
+    echo 'selected="selected"';
+}
+?> >22</option>
+                        <option value="23" <?php if ($day_end == 23) {
+    echo 'selected="selected"';
+}
+?> >23</option>
+                        <option value="24" <?php if ($day_end == 24) {
+    echo 'selected="selected"';
+}
+?> >24</option>
+                        <option value="25" <?php if ($day_end == 25) {
+    echo 'selected="selected"';
+}
+?> >25</option>
+                        <option value="26" <?php if ($day_end == 26) {
+    echo 'selected="selected"';
+}
+?> >26</option>
+                        <option value="27" <?php if ($day_end == 27) {
+    echo 'selected="selected"';
+}
+?> >27</option>
+                        <option value="28" <?php if ($day_end == 28) {
+    echo 'selected="selected"';
+}
+?> >28</option>
+                        <option value="29" <?php if ($day_end == 29) {
+    echo 'selected="selected"';
+}
+?> >29</option>
+                        <option value="30" <?php if ($day_end == 30) {
+    echo 'selected="selected"';
+}
+?> >30</option>
+                        <option value="31" <?php if ($day_end == 31) {
+    echo 'selected="selected"';
+}
+?> >31</option>
                   </select>
                   /
                   <select name="month_end">
                         <option value="0">--</option>
-                        <option value="1" <?php if ($month_end == 1) echo 'selected="selected"'; ?> >01</option>
-                        <option value="2" <?php if ($month_end == 2) echo 'selected="selected"'; ?> >02</option>
-                        <option value="3" <?php if ($month_end == 3) echo 'selected="selected"'; ?> >03</option>
-                        <option value="4" <?php if ($month_end == 4) echo 'selected="selected"'; ?> >04</option>
-                        <option value="5" <?php if ($month_end == 5) echo 'selected="selected"'; ?> >05</option>
-                        <option value="6" <?php if ($month_end == 6) echo 'selected="selected"'; ?> >06</option>
-                        <option value="7" <?php if ($month_end == 7) echo 'selected="selected"'; ?> >07</option>
-                        <option value="8" <?php if ($month_end == 8) echo 'selected="selected"'; ?> >08</option>
-                        <option value="9" <?php if ($month_end == 9) echo 'selected="selected"'; ?> >09</option>
-                        <option value="10" <?php if ($month_end == 10) echo 'selected="selected"'; ?> >10</option>
-                        <option value="11" <?php if ($month_end == 11) echo 'selected="selected"'; ?> >11</option>
-                        <option value="12" <?php if ($month_end == 12) echo 'selected="selected"'; ?> >12</option>
+                        <option value="1" <?php if ($month_end == 1) {
+    echo 'selected="selected"';
+}
+?> >01</option>
+                        <option value="2" <?php if ($month_end == 2) {
+    echo 'selected="selected"';
+}
+?> >02</option>
+                        <option value="3" <?php if ($month_end == 3) {
+    echo 'selected="selected"';
+}
+?> >03</option>
+                        <option value="4" <?php if ($month_end == 4) {
+    echo 'selected="selected"';
+}
+?> >04</option>
+                        <option value="5" <?php if ($month_end == 5) {
+    echo 'selected="selected"';
+}
+?> >05</option>
+                        <option value="6" <?php if ($month_end == 6) {
+    echo 'selected="selected"';
+}
+?> >06</option>
+                        <option value="7" <?php if ($month_end == 7) {
+    echo 'selected="selected"';
+}
+?> >07</option>
+                        <option value="8" <?php if ($month_end == 8) {
+    echo 'selected="selected"';
+}
+?> >08</option>
+                        <option value="9" <?php if ($month_end == 9) {
+    echo 'selected="selected"';
+}
+?> >09</option>
+                        <option value="10" <?php if ($month_end == 10) {
+    echo 'selected="selected"';
+}
+?> >10</option>
+                        <option value="11" <?php if ($month_end == 11) {
+    echo 'selected="selected"';
+}
+?> >11</option>
+                        <option value="12" <?php if ($month_end == 12) {
+    echo 'selected="selected"';
+}
+?> >12</option>
                   </select>
                   /
                   <select name="year_end">
@@ -218,7 +478,10 @@ 
                         <?php
                         for ($i = $thisYear - 5; $i <= ($thisYear + 5); $i++)
                         { ?>
-                         <option value="<?php echo $i; ?>" <?php if ($year_end == $i) echo 'selected="selected"'; ?> ><?php echo $i; ?></option>
+                         <option value="<?php echo $i; ?>" <?php if ($year_end == $i) {
+    echo 'selected="selected"';
+}
+?> ><?php echo $i; ?></option>
                         <?php
                         } ?>
                  </select>
@@ -239,7 +502,10 @@ 
 
 
 <script>
-<?php if ($year_start == "0000") echo "setDisable(document.form.nolimit);\r\n"; ?>
+<?php if ($year_start == "0000") {
+    echo "setDisable(document.form.nolimit);\r\n";
+}
+?>
 function setDisable(select){
 	document.form.day_start.disabled = (select.checked) ? true : false;
 	document.form.month_start.disabled = (select.checked) ? true : false;

main/inc/lib/message.lib.php

Indentation +2 added lines, -2 removed lines

@@ -226,8 +226,8 @@
     {
         if (!empty($aboutUserInfo)) {
             $criteria = [
-              'userReceiverId' => $aboutUserInfo['id'],
-              'msgStatus' => MESSAGE_STATUS_CONVERSATION
+                'userReceiverId' => $aboutUserInfo['id'],
+                'msgStatus' => MESSAGE_STATUS_CONVERSATION
             ];
             $repo = Database::getManager()->getRepository('ChamiloCoreBundle:Message');
             $messages = $repo->findBy($criteria, ['sendDate' => 'DESC']);

Doc Comments +5 added lines, -3 removed lines

@@ -700,6 +700,8 @@ 
      * @param  int    message id
      * @param  int    message user id (receiver user id or sender user id)
      * @param  int    group id (optional)
+     * @param integer $message_id
+     * @param integer $message_uid
      */
     public static function delete_message_attachment_file(
         $message_id,
@@ -747,7 +749,7 @@ 
      * update messages by user id and message id
      * @param  int $user_id
      * @param  int $message_id
-     * @return resource
+     * @return false|null
      */
     public static function update_message($user_id, $message_id)
     {
@@ -768,7 +770,7 @@ 
      * @param int $user_id
      * @param int $message_id
      * @param string $type
-     * @return bool
+     * @return false|null
      */
     public static function update_message_status($user_id, $message_id, $type)
     {
@@ -1592,7 +1594,7 @@ 
      * Sort date by desc from a multi-dimensional array
      * @param array $array1 first array to compare
      * @param array $array2 second array to compare
-     * @return bool
+     * @return integer
      */
     public function order_desc_date($array1, $array2)
     {

Spacing +2 added lines, -2 removed lines

@@ -173,13 +173,13 @@
                 Display::url(
                     Display::returnFontAwesomeIcon('reply', 2),
                     $newMessageLink.'?re_id='.$result[0].$link,
-                    ['title' => get_lang('ReplyToMessage') ]
+                    ['title' => get_lang('ReplyToMessage')]
                 ).
                 '  '.
                 Display::url(
                     Display::returnFontAwesomeIcon('share', 2),
                     $newMessageLink.'?forward_id='.$result[0].$link,
-                    ['title' => get_lang('ForwardMessage') ]
+                    ['title' => get_lang('ForwardMessage')]
                 ).
                 '&nbsp;&nbsp;<a title="'.addslashes(get_lang('DeleteMessage')).'" onclick="javascript:if(!confirm('."'".addslashes(api_htmlentities(get_lang('ConfirmDeleteMessage')))."'".')) return false;" href="inbox.php?action=deleteone&id='.$result[0].$link.'">'.
                 Display::returnFontAwesomeIcon('trash', 2).'</a>';

Upper-Lower-Casing +7 added lines, -7 removed lines

@@ -580,7 +580,7 @@ 
         $row = Database::fetch_array($result);
 
         // update parent_id for other user receiver
-        $sql = "UPDATE $table_message SET parent_id = ".$row['id']."
+        $sql = "update $table_message SET parent_id = ".$row['id']."
                 WHERE id = $message_id";
         Database::query($sql);
     }
@@ -606,7 +606,7 @@ 
             // delete attachment file
             self::delete_message_attachment_file($id, $user_receiver_id);
             // delete message
-            $query = "UPDATE $table_message 
+            $query = "update $table_message 
                       SET msg_status = ".MESSAGE_STATUS_DELETED."
                       WHERE 
                         user_receiver_id=".$user_receiver_id." AND 
@@ -644,7 +644,7 @@ 
             // delete attachment file
             self::delete_message_attachment_file($id, $user_sender_id);
             // delete message
-            $sql = "UPDATE $table_message 
+            $sql = "update $table_message 
                     SET msg_status = ".MESSAGE_STATUS_DELETED."
                     WHERE user_sender_id='$user_sender_id' AND id='$id'";
             Database::query($sql);
@@ -764,7 +764,7 @@ 
             $path_message_attach = $path_user_info['dir'].'message_attachments/';
             if (is_file($path_message_attach.$path)) {
                 if (rename($path_message_attach.$path, $path_message_attach.$new_path)) {
-                    $sql = "UPDATE $table_message_attach set path='$new_path'
+                    $sql = "update $table_message_attach set path='$new_path'
                             WHERE id ='$attach_id'";
                     Database::query($sql);
                 }
@@ -785,7 +785,7 @@ 
         }
 
         $table_message = Database::get_main_table(TABLE_MESSAGE);
-        $sql = "UPDATE $table_message SET 
+        $sql = "update $table_message SET 
                     msg_status = '".MESSAGE_STATUS_NEW."'
                 WHERE
                     msg_status <> ".MESSAGE_STATUS_OUTBOX." AND
@@ -807,7 +807,7 @@ 
             return false;
         }
         $table_message = Database::get_main_table(TABLE_MESSAGE);
-        $sql = "UPDATE $table_message SET
+        $sql = "update $table_message SET
                     msg_status = '$type'
                 WHERE
                     user_receiver_id=".intval($user_id)." AND
@@ -1104,7 +1104,7 @@ 
             }
         } else {
             if (is_numeric($message_id) && !empty($message_id)) {
-                $query = "UPDATE $table_message SET
+                $query = "update $table_message SET
                           msg_status = '".MESSAGE_STATUS_NEW."'
                           WHERE
                             user_receiver_id=" . api_get_user_id()." AND

main/inc/lib/CourseChatUtils.php

Doc Comments +3 added lines, -3 removed lines

@@ -217,7 +217,7 @@ 
 
     /**
      * Disconnect a user from course chats
-     * @param $userId
+     * @param integer $userId
      */
     public static function exitChat($userId)
     {
@@ -1388,7 +1388,7 @@ 
 
     /**
      * Get the emoji list to include in chat
-     * @return array
+     * @return string[]
      */
     public static function getEmojisToInclude()
     {
@@ -1693,7 +1693,7 @@ 
 
     /**
      * Get the number of users connected in chat
-     * @return mixed
+     * @return integer
      */
     public function countUsersOnline()
     {