chamilo / chamilo-lms

main/auth/sso/sso.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
 
     /**
      * Validates the received active connection data with the database
-     * @return	bool	Return the loginFailed variable value to local.inc.php
+     * @return	null|false	Return the loginFailed variable value to local.inc.php
      */
     public function check_user()
     {

main/document/downloadfolder.inc.php

Doc Comments +1 added lines, -1 removed lines

@@ -327,7 +327,7 @@
  * @param array $arr1 first array
  * @param array $arr2 second array
  *
- * @return array difference between the two arrays
+ * @return string difference between the two arrays
  */
 function diff($arr1, $arr2)
 {

main/inc/lib/export.lib.inc.php

Doc Comments +2 added lines, -2 removed lines

@@ -157,7 +157,7 @@ 
      * @param string Name of common tag to place each line in
      * @param string Name of the root element. A root element should always be given.
      * @param string Encoding in which the data is provided
-     * @return void  Prompts the user for a file download
+     * @return boolean  Prompts the user for a file download
      */
     public static function export_complex_table_xml(
         $data,
@@ -283,7 +283,7 @@ 
      * @param string $name
      * @param string $format
      *
-     * @return bool
+     * @return false|null
      */
     public static function htmlToOdt($html, $name, $format = 'odt')
     {

Indentation +8 added lines, -8 removed lines

@@ -115,13 +115,13 @@ 
     }
 
     /**
-    * Export tabular data to XML-file
-    * @param array  Simple array of data to put in XML
-    * @param string Name of file to be given to the user
-    * @param string Name of common tag to place each line in
-    * @param string Name of the root element. A root element should always be given.
-    * @param string Encoding in which the data is provided
-    */
+     * Export tabular data to XML-file
+     * @param array  Simple array of data to put in XML
+     * @param string Name of file to be given to the user
+     * @param string Name of common tag to place each line in
+     * @param string Name of the root element. A root element should always be given.
+     * @param string Encoding in which the data is provided
+     */
     public static function arrayToXml(
         $data,
         $filename = 'export',
@@ -200,7 +200,7 @@ 
         foreach ($data as $row) {
             $string .= "\n".str_repeat("\t", $level).'<'.$row['name'].'>';
             if (is_array($row['value'])) {
-            	$string .= self::_export_complex_table_xml_helper($row['value'], $level + 1)."\n";
+                $string .= self::_export_complex_table_xml_helper($row['value'], $level + 1)."\n";
                 $string .= str_repeat("\t", $level).'</'.$row['name'].'>';
             } else {
                 $string .= $row['value'];

main/inc/lib/formvalidator/Rule/MobilePhoneNumber.php

Doc Comments +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
      *
      * @param string Mobile phone number to be validated
      * @param string Not using it. Just to respect the declaration
-     * @return boolean Returns true if valid, false otherwise.
+     * @return integer Returns true if valid, false otherwise.
      */
     function validate($mobilePhoneNumber, $options = null)
     {

main/inc/lib/plugin.lib.php

Doc Comments +1 added lines, -1 removed lines

@@ -626,7 +626,7 @@
 
     /**
      * Get first SMS plugin name
-     * @return string|boolean
+     * @return string|false
      */
     public function getSMSPluginName()
     {

main/inc/lib/promotion.lib.php

Doc Comments +2 added lines, -2 removed lines

@@ -129,7 +129,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public function get_status_list()
     {
@@ -272,7 +272,7 @@ 
     /**
      * @param int $id
      *
-     * @return bool
+     * @return null|false
      */
     public function delete($id)
     {

main/inc/lib/VideoChat.php

Doc Comments +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
      * Create a video chat
      * @param int $fromUser The sender user
      * @param int $toUser The receiver user
-     * @return int The created video chat id. Otherwise return false
+     * @return false|string The created video chat id. Otherwise return false
      */
     public static function createRoom($fromUser, $toUser)
     {

main/inc/lib/zombie/zombie_report.class.php

Doc Comments +3 added lines

@@ -121,6 +121,9 @@
         return $form->isSubmitted() == false || $form->validate();
     }
 
+    /**
+     * @param string $format
+     */
     function get_ceiling($format = null)
     {
         $result = Request::get('ceiling');

main/permissions/permissions_functions.inc.php

Doc Comments +2 added lines, -2 removed lines

@@ -118,7 +118,7 @@ 
 
 /**
 * This function retrieves the existing permissions of a user, group or role.
-* @param $content are we retrieving the rights of a user, a group or a role (the database depends on it)
+* @param string $content are we retrieving the rights of a user, a group or a role (the database depends on it)
 * @param $id the id of the user, group or role
 * @author Patrick Cool <patrick.cool@ugent.be>, Ghent University
 * @version 1.0
@@ -526,7 +526,7 @@ 
 
 /**
 * This function gets all the roles that are defined
-* @param $content are we finding the roles for a user or a group (the database depends on it)
+* @param string $content are we finding the roles for a user or a group (the database depends on it)
 * @param $id the id of the user or group
 * @param string	Deprecated parameter allowing use of 'platform' scope - the corresponding tables don't exist anymore so the scope is always set to 'course'
 * @return array that contains the name of the roles the user has

Braces +10 added lines, -17 removed lines

@@ -206,8 +206,7 @@ 
 function display_checkbox_matrix($permission_array, $tool, $permission, $inherited_permissions = array())
 {
     $checked = "";
-    if (is_array($permission_array[$tool]) AND in_array($permission, $permission_array[$tool]))
-    {
+    if (is_array($permission_array[$tool]) AND in_array($permission, $permission_array[$tool])) {
         $checked = "checked";
     }
     echo "\t\t\t<input type=\"checkbox\" name=\"permission*$tool*$permission\" $checked>\n";
@@ -254,8 +253,7 @@ 
                     echo "</a>";
                 }
             } else {
-                if ($editable)
-                {
+                if ($editable) {
                     $url = api_get_self();
                     $urlparameters = '';
                     foreach ($_GET as $key => $value) {
@@ -273,8 +271,7 @@ 
                     echo "\t\t\t <a href=\"".$url."\">";
                 }
                 echo "<img src=\"../img/wrong.gif\" border=\"0\"/>";
-                if ($editable)
-                {
+                if ($editable) {
                     echo "</a>";
                 }
             }
@@ -306,8 +303,7 @@ 
                 if ($editable) {
                     $url = api_get_self();
                     $urlparameters = '';
-                    foreach ($_GET as $key => $value)
-                    {
+                    foreach ($_GET as $key => $value) {
                         $parameter[$key] = $value;
                     }
                     $parameter['action'] = 'manage_rights';
@@ -315,8 +311,7 @@ 
                     $parameter['permission'] = $permission;
                     $parameter['tool'] = $tool;
                     $parameter['user_id'] = $user_id;
-                    foreach ($parameter as $key=>$value)
-                    {
+                    foreach ($parameter as $key=>$value) {
                         $urlparameters .= $key.'='.$value.'&amp;';
                     }
                     $url = $url.'?'.$urlparameters;
@@ -433,16 +428,15 @@ 
 * @author Patrick Cool <patrick.cool@ugent.be>, Ghent University
 * @version 1.0
 */
-function get_all_roles($content = 'course') {
+function get_all_roles($content = 'course')
+{
     $course_id = api_get_course_int_id();
     $course_id_condition = " WHERE c_id = $course_id ";
 
-    if ($content == 'course')
-    {
+    if ($content == 'course') {
         $table_role = Database::get_course_table(TABLE_ROLE);
     }
-    if ($content == 'platform')
-    {
+    if ($content == 'platform') {
         $table_role = Database::get_main_table(TABLE_ROLE);
         $course_id_condition = '';
     }
@@ -450,8 +444,7 @@ 
     $current_roles = array();
     $sql = "SELECT * FROM $table_role $course_id_condition ";
     $result = Database::query($sql);
-    while ($row = Database::fetch_array($result))
-    {
+    while ($row = Database::fetch_array($result)) {
         $roles[] = $row;
     }
 

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

@@ -175,7 +175,7 @@ 
                 if ($value == 'View') {
                     unset($current_permissions[$tool][$key]);
                 }
-                if ($value == 'Visibility' OR $value == 'Move') {
+                if ($value == 'Visibility' or $value == 'Move') {
                     if (!in_array('Edit', $current_permissions[$tool])) {
                         $current_permissions[$tool][] = 'Edit';
                     }
@@ -206,7 +206,7 @@ 
 function display_checkbox_matrix($permission_array, $tool, $permission, $inherited_permissions = array())
 {
     $checked = "";
-    if (is_array($permission_array[$tool]) AND in_array($permission, $permission_array[$tool]))
+    if (is_array($permission_array[$tool]) and in_array($permission, $permission_array[$tool]))
     {
         $checked = "checked";
     }
@@ -232,7 +232,7 @@ 
         if (in_array($permission, $inherited_permissions[$tool])) {
             echo "\t\t\t<img src=\"../img/checkbox_on3.gif\" border=\"0\"/ title=\"".get_lang('PermissionGrantedByGroupOrRole')."\">";
         } else {
-            if (is_array($permission_array[$tool]) AND in_array($permission, $permission_array[$tool])) {
+            if (is_array($permission_array[$tool]) and in_array($permission, $permission_array[$tool])) {
                 if ($editable) {
                     $url = api_get_self();
                     $urlparameters = '';
@@ -302,7 +302,7 @@ 
         if (!empty($inherited_permissions) and in_array($permission, $inherited_permissions[$tool])) {
             echo "\t\t\t<img src=\"../img/checkbox_on3.gif\" border=\"0\"/ title=\"".get_lang('PermissionGrantedByGroupOrRole')."\">";
         } else {
-            if (is_array($permission_array[$tool]) AND in_array($permission, $permission_array[$tool])) {
+            if (is_array($permission_array[$tool]) and in_array($permission, $permission_array[$tool])) {
                 if ($editable) {
                     $url = api_get_self();
                     $urlparameters = '';