chamilo / chamilo-lms

main/user/user_export.php

Switch Indentation +11 added lines, -11 removed lines

@@ -149,15 +149,15 @@
 }
 
 switch ($file_type) {
-    case 'xml':
-        Export::arrayToXml($data, $filename, 'Contact', 'Contacts');
-        exit;
-        break;
-    case 'csv':
-        Export::arrayToCsv($data, $filename);
-        exit;
-    case 'xls':
-        Export::arrayToXls($data, $filename);
-        exit;
-        break;
+        case 'xml':
+            Export::arrayToXml($data, $filename, 'Contact', 'Contacts');
+            exit;
+            break;
+        case 'csv':
+            Export::arrayToCsv($data, $filename);
+            exit;
+        case 'xls':
+            Export::arrayToXls($data, $filename);
+            exit;
+            break;
 }
\ No newline at end of file

main/inc/lib/formvalidator/Element/FloatNumber.php

Indentation +6 added lines, -6 removed lines

@@ -29,9 +29,9 @@ 
         $this->setType('float');
     }
 
-     /**
-     * @param string $value
-     */
+        /**
+         * @param string $value
+         */
     public function setValue($value)
     {
         $value = api_float_val($value);
@@ -47,10 +47,10 @@ 
      */
     public function getValue()
     {
-         $value = $this->getAttribute('value');
-         $value = api_float_val($value);
+            $value = $this->getAttribute('value');
+            $value = api_float_val($value);
 
-         return $value;
+            return $value;
     }
 
     /**

main/admin/legal_add.php

Indentation +3 added lines, -3 removed lines

@@ -105,8 +105,8 @@ 
 
 if (isset($_POST['language'])) {
 
-	$form->addElement('static', Security::remove_XSS($_POST['language']));
-	$form->addElement('hidden', 'language', Security::remove_XSS($_POST['language']));
+    $form->addElement('static', Security::remove_XSS($_POST['language']));
+    $form->addElement('hidden', 'language', Security::remove_XSS($_POST['language']));
     $form->addHtmlEditor(
         'content',
         get_lang('Content'),
@@ -138,7 +138,7 @@ 
 
 } else {
     $form->addSelectLanguage('language', get_lang('Language'), null, array());
-	$form->addButtonSearch(get_lang('Load'), 'send');
+    $form->addButtonSearch(get_lang('Load'), 'send');
 
 }
 

main/inc/lib/course_request.lib.php

Doc Comments +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
      * @param string $objectives
      * @param string $target_audience
      * @param int $user_id
-     * @return mixed The database id of the newly created course request or FALSE on failure.
+     * @return false|string The database id of the newly created course request or FALSE on failure.
      */
     public static function create_course_request(
         $wanted_code,

main/inc/lib/TicketManager.php

Indentation +5 added lines, -5 removed lines

@@ -1862,9 +1862,9 @@ 
         return 0;
     }
 
-     /**
-     * @return array
-     */
+        /**
+         * @return array
+         */
     public static function getPriorityList()
     {
         $projects = Database::getManager()->getRepository('ChamiloTicketBundle:Priority')->findAll();
@@ -2279,8 +2279,8 @@ 
     }
 
         /**
-     * @return array
-     */
+         * @return array
+         */
     public static function getDefaultPriorityList()
     {
         return [

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

@@ -168,7 +168,7 @@ 
         }
 
         $table = Database::get_main_table(TABLE_TICKET_TICKET);
-        $sql = "UPDATE $table SET category_id = NULL WHERE category_id = $id";
+        $sql = "update $table SET category_id = NULL WHERE category_id = $id";
         Database::query($sql);
 
         $table = Database::get_main_table(TABLE_TICKET_CATEGORY);
@@ -417,13 +417,13 @@ 
             }
 
             // Update code
-            $sql = "UPDATE $table_support_tickets
+            $sql = "update $table_support_tickets
                     SET code = '$ticket_code'
                     WHERE id = '$ticketId'";
             Database::query($sql);
 
             // Update total
-            $sql = "UPDATE $table_support_category
+            $sql = "update $table_support_category
                     SET total_tickets = total_tickets + 1
                     WHERE id = $category_id";
             Database::query($sql);
@@ -589,7 +589,7 @@ 
         $ticket = self::get_ticket_detail_by_id($ticketId);
 
         if ($ticket) {
-            $sql = "UPDATE $table_support_tickets
+            $sql = "update $table_support_tickets
                     SET assigned_last_user = $userId
                     WHERE id = $ticketId";
             Database::query($sql);
@@ -659,7 +659,7 @@ 
         $messageId = Database::insert($table_support_messages, $params);
         if ($messageId) {
             // update_total_message
-            $sql = "UPDATE $table_support_tickets
+            $sql = "update $table_support_tickets
                     SET 
                         sys_lastedit_user_id ='$userId',
                         sys_lastedit_datetime ='$now',
@@ -1282,7 +1282,7 @@ 
         );
         $table_support_tickets = Database::get_main_table(TABLE_TICKET_TICKET);
         $now = api_get_utc_datetime();
-        $sql = "UPDATE $table_support_messages
+        $sql = "update $table_support_messages
                 SET
                     status = 'LEI',
                     sys_lastedit_user_id ='".api_get_user_id()."',
@@ -1298,7 +1298,7 @@ 
         $result = Database::query($sql);
         if (Database::affected_rows($result) > 0) {
             Database::query(
-                "UPDATE $table_support_tickets SET
+                "update $table_support_tickets SET
                     status_id = '".self::STATUS_PENDING."'
                  WHERE id ='$ticketId' AND status_id = '".self::STATUS_NEW."'"
             );
@@ -1420,7 +1420,7 @@ 
         $userId = intval($userId);
 
         $now = api_get_utc_datetime();
-        $sql = "UPDATE $table_support_tickets
+        $sql = "update $table_support_tickets
                 SET
                     status_id = '$status_id',
                     sys_lastedit_user_id ='$userId',
@@ -1487,7 +1487,7 @@ 
         $ticketId = intval($ticketId);
         $userId = intval($userId);
 
-        $sql = "UPDATE $table_support_tickets SET
+        $sql = "update $table_support_tickets SET
                   priority_id = '".self::PRIORITY_HIGH."',
                   sys_lastedit_user_id ='$userId',
                   sys_lastedit_datetime ='$now'
@@ -1506,7 +1506,7 @@ 
 
         $table_support_tickets = Database::get_main_table(TABLE_TICKET_TICKET);
         $now = api_get_utc_datetime();
-        $sql = "UPDATE $table_support_tickets SET
+        $sql = "update $table_support_tickets SET
                     status_id = '".self::STATUS_CLOSE."',
                     sys_lastedit_user_id ='$userId',
                     sys_lastedit_datetime ='".$now."',
@@ -1529,7 +1529,7 @@ 
         $table_support_tickets = Database::get_main_table(TABLE_TICKET_TICKET);
         $now = api_get_utc_datetime();
         $userId = api_get_user_id();
-        $sql = "UPDATE $table_support_tickets
+        $sql = "update $table_support_tickets
                 SET
                     status_id = '".self::STATUS_CLOSE."',
                     sys_lastedit_user_id ='$userId',

Doc Comments +8 added lines, -8 removed lines

@@ -694,8 +694,8 @@ 
     /**
      * Attachment files when a message is sent
      * @param $file_attach
-     * @param $ticketId
-     * @param $message_id
+     * @param integer $ticketId
+     * @param string $message_id
      * @return array
      */
     public static function save_message_attachment_file(
@@ -1315,7 +1315,7 @@ 
      * @param string $message
      * @param int $onlyToUserId
      *
-     * @return bool
+     * @return false|null
      */
     public static function sendNotification($ticketId, $title, $message, $onlyToUserId = 0)
     {
@@ -1941,7 +1941,7 @@ 
     }
 
     /**
-     * @param $id
+     * @param integer $id
      * @return Project
      */
     public static function getProject($id)
@@ -2063,7 +2063,7 @@ 
     }
 
     /**
-     * @param $id
+     * @param integer $id
      * @return Project
      */
     public static function getStatus($id)
@@ -2188,7 +2188,7 @@ 
     }
 
     /**
-     * @param $id
+     * @param integer $id
      * @return Priority
      */
     public static function getPriority($id)
@@ -2260,7 +2260,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function getDefaultStatusList()
     {
@@ -2274,7 +2274,7 @@ 
     }
 
         /**
-     * @return array
+     * @return string[]
      */
     public static function getDefaultPriorityList()
     {

Spacing +1 added lines, -1 removed lines

@@ -1085,7 +1085,7 @@
         $res = Database::query($sql);
         $obj = Database::fetch_object($res);
 
-        return (int)$obj->total;
+        return (int) $obj->total;
     }
 
     /**

plugin/sepe/src/sepe_plugin.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
     public $isAdminPlugin = true;

     /**

      *

-     * @return StaticPlugin

+     * @return SepePlugin

      */

     public static function create()

     {

Braces +14 added lines, -14 removed lines

@@ -6,8 +6,8 @@ 
  * @author Jose Angel Ruiz    <jaruiz@nosolored.com>

  * @author Julio Montoya <gugli100@gmail.com>

  */

-class SepePlugin extends Plugin

-{

+class SepePlugin extends Plugin
+{
     const TABLE_SEPE_CENTER = 'plugin_sepe_center';

     const TABLE_SEPE_ACTIONS = 'plugin_sepe_actions';

     const TABLE_SEPE_SPECIALTY = 'plugin_sepe_specialty';

@@ -30,14 +30,14 @@ 
      *

      * @return StaticPlugin

      */

-    public static function create()

-    {

+    public static function create()
+    {
         static $result = null;

         return $result ? $result : $result = new self();

     }

 

-    protected function __construct()

-    {

+    protected function __construct()
+    {
         parent::__construct(

             '2.0', 

             '

@@ -51,8 +51,8 @@ 
     /**

      * This method creates the tables required to this plugin

      */

-    function install()

-    {

+    function install()
+    {
         $tablesToBeCompared = array(

             self::TABLE_SEPE_CENTER,

             self::TABLE_SEPE_ACTIONS,

@@ -76,7 +76,7 @@ 
         $sm = $cn->getSchemaManager();

         $tables = $sm->tablesExist($tablesToBeCompared);

 

-        if ($tables) {

+        if ($tables) {
             return false;

         }

 

@@ -86,8 +86,8 @@ 
     /**

      * This method drops the plugin tables

      */

-    function uninstall()

-    {

+    function uninstall()
+    {
         $tablesToBeDeleted = array(

             self::TABLE_SEPE_CENTER,

             self::TABLE_SEPE_SPECIALTY_CLASSROOM,

@@ -107,7 +107,7 @@ 
             self::TABLE_SEPE_LOG

         );

 

-        foreach ($tablesToBeDeleted as $tableToBeDeleted) {

+        foreach ($tablesToBeDeleted as $tableToBeDeleted) {
             $table = Database::get_main_table($tableToBeDeleted);

             $sql = "DROP TABLE IF EXISTS $table";

             Database::query($sql);

@@ -115,8 +115,8 @@ 
         $this->manageTab(false);

     }

 

-    function update()

-    {

+    function update()
+    {
         $oldTableCenters = 'plugin_sepe_centros';

         $oldTableTutorsCompany = 'plugin_sepe_tutors_empresa';

         $oldTableCompetence = 'plugin_sepe_competencia_docente';

Indentation +3 added lines, -3 removed lines

@@ -121,9 +121,9 @@
         $oldTableTutorsCompany = 'plugin_sepe_tutors_empresa';

         $oldTableCompetence = 'plugin_sepe_competencia_docente';

         $sql = "RENAME TABLE "

-                   . $oldTableCenters." TO ".self::TABLE_SEPE_CENTERS.", "

-                   . $oldTableTutorsCompany." TO ".self::TABLE_SEPE_TUTORS_COMPANY.", "

-                   . $oldTableCompetence." TO ".self::TABLE_SEPE_TEACHING_COMPETENCE.";";

+                    . $oldTableCenters." TO ".self::TABLE_SEPE_CENTERS.", "

+                    . $oldTableTutorsCompany." TO ".self::TABLE_SEPE_TUTORS_COMPANY.", "

+                    . $oldTableCompetence." TO ".self::TABLE_SEPE_TEACHING_COMPETENCE.";";

         Database::query($sql);

         

         $sepeCourseActionsTable = self::TABLE_SEPE_COURSE_ACTIONS;

main/exercise/fill_blanks.class.php

Indentation +23 added lines, -23 removed lines

@@ -783,29 +783,29 @@
     }
 
     /**
-    * Return an array of student state answers for fill the blank questions
-    * for each students that answered the question
-    * -2  : didn't answer
-    * -1  : student answer is wrong
-    *  0  : student answer is correct
-    * >0  : for fill the blank question with choice menu, is the index of the student answer (right answer indice is 0)
-    *
-    * @param integer $testId
-    * @param integer $questionId
-    * @param $studentsIdList
-    * @param string $startDate
-    * @param string $endDate
-    * @param bool $useLastAnsweredAttempt
-    * @return array
-    * (
-    *     [student_id] => Array
-    *         (
-    *             [first fill the blank for question] => -1
-    *             [second fill the blank for question] => 2
-    *             [third fill the blank for question] => -1
-    *         )
-    * )
-    */
+     * Return an array of student state answers for fill the blank questions
+     * for each students that answered the question
+     * -2  : didn't answer
+     * -1  : student answer is wrong
+     *  0  : student answer is correct
+     * >0  : for fill the blank question with choice menu, is the index of the student answer (right answer indice is 0)
+     *
+     * @param integer $testId
+     * @param integer $questionId
+     * @param $studentsIdList
+     * @param string $startDate
+     * @param string $endDate
+     * @param bool $useLastAnsweredAttempt
+     * @return array
+     * (
+     *     [student_id] => Array
+     *         (
+     *             [first fill the blank for question] => -1
+     *             [second fill the blank for question] => 2
+     *             [third fill the blank for question] => -1
+     *         )
+     * )
+     */
     public static function getFillTheBlankTabResult(
         $testId,
         $questionId,

Doc Comments +2 added lines

@@ -1230,6 +1230,7 @@ 
      * @param string $answer
      * @param string $correct
      * @param bool   $resultsDisabled
+     * @param integer $feedbackType
      *
      * @return string
      */
@@ -1243,6 +1244,7 @@ 
      * @param string $answer
      * @param string $correct
      * @param bool   $resultsDisabled
+     * @param integer $feedbackType
      *
      * @return string
      */

Spacing +2 added lines, -2 removed lines

@@ -371,7 +371,7 @@ 
         // remove spaces at the beginning and the end of text in square brackets
         $answer = preg_replace_callback(
             "/".$blankStartSeparatorRegexp."[^]]+".$blankEndSeparatorRegexp."/",
-            function ($matches) use ($blankStartSeparator, $blankEndSeparator) {
+            function($matches) use ($blankStartSeparator, $blankEndSeparator) {
                 $matchingResult = $matches[0];
                 $matchingResult = trim($matchingResult, $blankStartSeparator);
                 $matchingResult = trim($matchingResult, $blankEndSeparator);
@@ -750,7 +750,7 @@ 
             // remove [ and ] in string
             array_walk(
                 $listWords[0],
-                function (&$value, $key, $tabBlankChar) {
+                function(&$value, $key, $tabBlankChar) {
                     $trimChars = '';
                     for ($i = 0; $i < count($tabBlankChar); $i++) {
                         $trimChars .= $tabBlankChar[$i];

src/Chamilo/CoreBundle/Component/Filesystem/Data.php

Doc Comments +2 added lines, -2 removed lines

@@ -116,7 +116,7 @@ 
 
     /**
      * Create folders
-     * @param array $folderList
+     * @param string[] $folderList
      * @param OutputInterface $output
      * @param string $folderPermissions
      */
@@ -200,7 +200,7 @@ 
      * http://chamilo/courses/ABC/document/file.jpg
      * @param string $content
      *
-     * @return string
+     * @return \simplehtmldom_1_5\simple_html_dom
      */
     public function convertRelativeToAbsoluteUrl($content)
     {

plugin/buycourses/src/configuration.php

Braces +2 added lines, -2 removed lines

@@ -40,12 +40,12 @@
 $tpl->assign('sessions_are_included', $includeSession);

 $tpl->assign('services_are_included', $includeServices);

 

-if ($includeSession) {

+if ($includeSession) {
     $sessions = $plugin->getSessionsForConfiguration();

     $tpl->assign('sessions', $sessions);

 }

 

-if ($includeServices) {

+if ($includeServices) {
     $services = $plugin->getServices();

     $tpl->assign('services', $services);

 }