chamilo / chamilo-lms

main/permissions/roles.php

Braces +18 added lines, -27 removed lines

@@ -14,27 +14,21 @@ 
 
 
 // storing all the permission for a given role when the checkbox approach is used
-if ($_POST['StoreRolePermissions'])
-{
-    if (!empty($_POST['role_name']))
-    {
+if ($_POST['StoreRolePermissions']) {
+    if (!empty($_POST['role_name'])) {
         $table_role = Database::get_course_table(TABLE_ROLE);
         $sql = "INSERT INTO $table_role (role_name, role_comment, default_role)
                     VALUES ('".Database::escape_string($_POST['role_name'])."','".Database::escape_string($_POST['role_comment'])."','".Database::escape_string($_POST['default_role'])."')";
         $result = Database::query($sql);
         $role_id = Database::insert_id();
         $result_message = store_permissions('role', $role_id);
-    }
-    else
-    {
+    } else {
         $result_message = get_lang('ErrorPleaseGiveRoleName');
     }
 }
 // storing a permission for a given role when the image approach is used
-if (isset($_GET['action']) AND isset($_GET['permission']) AND isset($_GET['tool']))
-{
-    if ($_GET['action'] == 'grant' OR $_GET['action'] == 'revoke')
-    {
+if (isset($_GET['action']) AND isset($_GET['permission']) AND isset($_GET['tool'])) {
+    if ($_GET['action'] == 'grant' OR $_GET['action'] == 'revoke') {
         $result_message = store_one_permission('role', $_GET['action'], $role_id, $_GET['tool'], $_GET['permission']);
     }
 }
@@ -71,8 +65,7 @@ 
 // 		ADDING A NEW ROLE (FORM AND LINK)
 echo '<img src="../img/add.png" /> <a href="roles.php?action=add">'.get_lang('AddRole').'</a>';
 
-if ($_GET['action'] == 'add')
-{
+if ($_GET['action'] == 'add') {
     echo "<form method=\"post\" action=\"".api_get_self()."\">";
     echo "\n<table>";
     echo "\n\t<tr>";
@@ -117,9 +110,11 @@ 
     echo "\t</tr>\n";
 
     // the main area with the checkboxes or images
-    foreach ($tool_rights as $tool=>$rights) // $tool_rights contains all the possible tools and their rights
+    foreach ($tool_rights as $tool=>$rights) {
+        // $tool_rights contains all the possible tools and their rights
     {
         echo "\t<tr>\n";
+    }
         echo "\t\t<td>\n";
         echo get_lang($tool);
         echo "\t\t</td>\n";
@@ -140,12 +135,10 @@ 
 // 		DISPLAYING THE EXISTING ROLES
 // platform roles
 $all_roles = get_all_roles('platform');
-foreach ($all_roles as $role)
-{
+foreach ($all_roles as $role) {
     echo '<div><a href="roles.php?action=view&amp;role_id='.$role['role_id'].'&amp;scope=platform">'.$role['role_name'].'</a></div>';
     echo '<div>'.$role['role_comment'].'</div><br />';
-    if ($role['role_id'] == $_GET['role_id'])
-    {
+    if ($role['role_id'] == $_GET['role_id']) {
         $current_role_info = $role;
     }
 }
@@ -154,8 +147,7 @@ 
 foreach ($all_roles as $role) {
     echo '<div><a href="roles.php?action=view&amp;role_id='.$role['role_id'].'">'.$role['role_name'].'</a><a href="roles.php?action=delete&amp;role_id='.$role['role_id'].'"><img src="../img/delete.gif" /></a></div>';
     echo '<div>'.$role['role_comment'].'</div><br />';
-    if ($role['role_id'] == $_GET['role_id'])
-    {
+    if ($role['role_id'] == $_GET['role_id']) {
         $current_role_info = $role;
     }
 }
@@ -165,12 +157,10 @@ 
     $current_role_permissions = get_permissions('role', $_GET['role_id']);
     // 			LIMITED OR FULL
     $current_role_permissions = limited_or_full($current_role_permissions);
-    if (api_get_setting('permissions') == 'limited')
-    {
+    if (api_get_setting('permissions') == 'limited') {
         $header_array = $rights_limited;
     }
-    if (api_get_setting('permissions') == 'full')
-    {
+    if (api_get_setting('permissions') == 'full') {
         $header_array = $rights_full;
     }
     // ---------------------------------------------------
@@ -180,8 +170,7 @@ 
 
     // the list of the roles for the user
     echo get_lang('PermissionsOfRole').':'.$current_role_info['role_name'].'<br />';
-    if ($_GET['scope'] == 'platform')
-    {
+    if ($_GET['scope'] == 'platform') {
         echo get_lang('IsPlatformRoleNotEditable').'<br />';
     }
 
@@ -196,9 +185,11 @@ 
     echo "\t</tr>\n";
 
     // the main area with the checkboxes or images
-    foreach ($tool_rights as $tool=>$rights) // $tool_rights contains all the possible tools and their rights
+    foreach ($tool_rights as $tool=>$rights) {
+        // $tool_rights contains all the possible tools and their rights
     {
         echo "\t<tr>\n";
+    }
         echo "\t\t<td>\n";
         echo get_lang($tool);
         echo "\t\t</td>\n";

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

@@ -31,16 +31,16 @@
     }
 }
 // storing a permission for a given role when the image approach is used
-if (isset($_GET['action']) AND isset($_GET['permission']) AND isset($_GET['tool']))
+if (isset($_GET['action']) and isset($_GET['permission']) and isset($_GET['tool']))
 {
-    if ($_GET['action'] == 'grant' OR $_GET['action'] == 'revoke')
+    if ($_GET['action'] == 'grant' or $_GET['action'] == 'revoke')
     {
         $result_message = store_one_permission('role', $_GET['action'], $role_id, $_GET['tool'], $_GET['permission']);
     }
 }
 
 // deleting a role
-if (isset($_GET['action']) AND isset($_GET['role_id']) AND $_GET['action'] == 'delete') {
+if (isset($_GET['action']) and isset($_GET['role_id']) and $_GET['action'] == 'delete') {
     //deleting the assignments fo this role: users
     $table = Database::get_course_table(TABLE_ROLE_USER);
     $sql = "DELETE FROM $table WHERE role_id='".intval($_GET['role_id'])."'";

main/exercise/hotspot.inc.php

Indentation +6 added lines, -6 removed lines

@@ -18,10 +18,10 @@
 $file = file(api_get_path(SYS_LANG_PATH).'english/hotspot.inc.php');
 
 foreach ($file as &$value) {
-	$variable = explode('=', $value, 2);
-	if (count($variable) > 1) {
-		$variable = substr(trim($variable[0]), 1);
-		$variable = '&'.$variable.'='.api_utf8_encode(get_lang($variable)).' ';
-		echo $variable;
-	}
+    $variable = explode('=', $value, 2);
+    if (count($variable) > 1) {
+        $variable = substr(trim($variable[0]), 1);
+        $variable = '&'.$variable.'='.api_utf8_encode(get_lang($variable)).' ';
+        echo $variable;
+    }
 }

main/inc/lib/sortable_table.class.php

Indentation +5 added lines, -5 removed lines

@@ -340,9 +340,9 @@ 
     }
 
     /**
-    * This function shows the content of a table in a grid.
-    * Should not be use to edit information (edit/delete rows) only.
-    **/
+     * This function shows the content of a table in a grid.
+     * Should not be use to edit information (edit/delete rows) only.
+     **/
     public function display_grid()
     {
         $empty_table = false;
@@ -800,7 +800,7 @@ 
                 //$param[$tablename.'_direction'] = $_GET[$tablename.'_direction'];
                 $my_get_direction = $_GET[$tablename.'_direction'];
                 if (!in_array($my_get_direction, array('ASC', 'DESC'))) {
-                     $param[$tablename.'_direction'] = 'ASC';
+                        $param[$tablename.'_direction'] = 'ASC';
                 } else {
                     $param[$tablename.'_direction'] = $my_get_direction;
                 }
@@ -929,7 +929,7 @@ 
         if (is_array($row)) {
             foreach ($row as & $value) {
                 if (empty($value)) {
-                     $value = '-';
+                        $value = '-';
                 }
             }
         }

Doc Comments +1 added lines

@@ -443,6 +443,7 @@
      * @param bool       hide navigation optionally
      * @param int        content per page when show navigation (optional)
      * @param bool       sort data optionally
+     * @param boolean $visibility_options
      * @return string    grid html
      */
     public function display_simple_grid(

main/cron/user_import/get_data_from_mail.php

Indentation +8 added lines, -8 removed lines

@@ -20,16 +20,16 @@
 $users = Database::get_main_table(TABLE_MAIN_USER);
 $string = '';
 foreach ($list as $mail) {
-  $mail = trim($mail);
-  $sql = "SELECT user_id, official_code, firstname, lastname, email FROM $users WHERE email = '$mail'\n";
-  $res = Database::query($sql);
-  if ($res === false) { die(mysql_error()); }
-  if (Database::num_rows($res) == 0) {
+    $mail = trim($mail);
+    $sql = "SELECT user_id, official_code, firstname, lastname, email FROM $users WHERE email = '$mail'\n";
+    $res = Database::query($sql);
+    if ($res === false) { die(mysql_error()); }
+    if (Database::num_rows($res) == 0) {
     $string .= 'No encontrado;'.$row['email'];
-  } else {
+    } else {
     $row = Database::fetch_assoc($res);
-   $string .= $row['user_id'].';'.$row['email'].';'.$row['firstname'].';'.$row['lastname'].';'.$row['official_code']."\r\n";
-  }
+    $string .= $row['user_id'].';'.$row['email'].';'.$row['firstname'].';'.$row['lastname'].';'.$row['official_code']."\r\n";
+    }
 }
 echo $string;
 file_put_contents('/tmp/list.txt', $string);

main/exercise/hotpotatoes.lib.php

Doc Comments +2 added lines, -2 removed lines

@@ -261,7 +261,7 @@ 
  * Searches for a node in the given array.
  * @param    array    $array Reference to the array to search
  * @param    string   $node  Node we are looking for in the array
- * @return   mixed    Node name or false if not found
+ * @return   false|string    Node name or false if not found
  */
 function myarraysearch(&$array, $node)
 {
@@ -282,7 +282,7 @@ 
  * Searches an image name into an array.
  * @param    array        $imgparams Reference to an array to search
  * @param    string       $string String to look for
- * @return   mixed        String given if found, false otherwise
+ * @return   false|string        String given if found, false otherwise
  * @uses     myarraysearch    This function is just an additional layer on the myarraysearch() function
  */
 function CheckImageName(&$imgparams, $string)

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

@@ -103,7 +103,7 @@
     $path = Database::escape_string($path);
     $comment = Database::escape_string($comment);
     $course_id = api_get_course_int_id();
-    $query = "UPDATE $dbTable SET comment = '$comment'
+    $query = "update $dbTable SET comment = '$comment'
               WHERE $course_id AND path = '$path'";
     $result = Database::query($query);
 

main/exercise/export/scorm/scorm_classes.php

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

@@ -663,7 +663,7 @@
                     $answer_list
                 </td>
             <tr>
-HTML;
+html;
         $html .= '</table></td></tr>';
 
         // currently the free answers cannot be displayed, so ignore the textarea

main/inc/lib/internationalization.lib.php

Indentation +4 added lines, -5 removed lines

@@ -1743,7 +1743,6 @@ 
 
 /**
  * Return true a date is valid
-
  * @param string $date example: 2014-06-30 13:05:05
  * @param string $format example: "Y-m-d H:i:s"
  *
@@ -2016,15 +2015,15 @@ 
 {
     if ($showTime) {
         if ($humanForm) {
-           return $date->format('j M Y H:i:s');
+            return $date->format('j M Y H:i:s');
         } else {
-           return $date->format('Y-m-d H:i:s');
+            return $date->format('Y-m-d H:i:s');
         }
     } else {
         if ($humanForm) {
-           return $date->format('j M Y');
+            return $date->format('j M Y');
         } else {
-           return $date->format('Y-m-d');
+            return $date->format('Y-m-d');
         }
     }
 }

Doc Comments +19 added lines, -13 removed lines

@@ -448,6 +448,7 @@ 
  * If null, the timezone will be determined based on user preference,
  * or timezone chosen by the admin for the platform.
  * @param string The timezone to be converted from. If null, UTC will be assumed.
+ * @param string $to_timezone
  * @return string The converted time formatted as Y-m-d H:i:s
  *
  * @author Guillaume Viguier <guillaume.viguier@beeznest.com>
@@ -719,6 +720,8 @@ 
  * @param mixed The time to be converted
  * @param mixed Format to be used (TIME_NO_SEC_FORMAT, DATE_FORMAT_SHORT, DATE_FORMAT_LONG, DATE_TIME_FORMAT_LONG)
  * @param string Timezone to be converted from. If null, UTC will be assumed.
+ * @param integer $format
+ * @param string $from_timezone
  * @return string Converted and localized date
  *
  * @author Guillaume Viguier <guillaume.viguier@beeznest.com>
@@ -805,7 +808,7 @@ 
  * This parameter has meaning with the format PERSON_NAME_COMMON_CONVENTION only.
  * @param string $encoding (optional)    The used internally by this function
  * character encoding. If it is omitted, the platform character set will be used by default.
- * @return bool The result is sort of full name of the person.
+ * @return string The result is sort of full name of the person.
  * Sample results:
  * Peter Ustinoff or Dr. Peter Ustinoff     - the Western order
  * Ustinoff Peter or Dr. Ustinoff Peter     - the Eastern order
@@ -1148,8 +1151,8 @@ 
 /**
  * This function returns a string or an array with all occurrences of search
  * in subject (ignoring case) replaced with the given replace value.
- * @param mixed $search String or array of strings to be found.
- * @param mixed $replace String or array of strings used for replacement.
+ * @param string $search String or array of strings to be found.
+ * @param string $replace String or array of strings used for replacement.
  * @param mixed $subject String or array of strings being searched.
  * @param int $count (optional) The number of matched and replaced needles
  * will be returned in count, which is passed by reference.
@@ -1219,12 +1222,12 @@ 
 /**
  * Finds first occurrence of a string within another, case insensitive.
  * @param string $haystack The string from which to get the first occurrence.
- * @param mixed $needle The string to be found.
+ * @param string $needle The string to be found.
  * @param bool $before_needle (optional) Determines which portion of $haystack
  * this function returns. The default value is FALSE.
  * @param string $encoding (optional) The used internally by this function
  * character encoding. If it is omitted, the platform character set will be used by default.
- * @return mixed Returns the portion of $haystack, or FALSE if $needle is not found.
+ * @return false|string Returns the portion of $haystack, or FALSE if $needle is not found.
  * Notes:
  * If $needle is not a string, it is converted to an integer and applied as the
  * ordinal value (codepoint if the encoding is UTF-8) of a character.
@@ -1283,7 +1286,7 @@ 
  * @param mixed $needle The string which first character is to be found.
  * @param bool $before_needle (optional) Determines which portion of $haystack this function returns. The default value is FALSE.
  * @param string $encoding (optional) The used internally by this function character encoding. If it is omitted, the platform character set will be used by default.
- * @return mixed Returns the portion of $haystack, or FALSE if the first character from $needle is not found.
+ * @return false|string Returns the portion of $haystack, or FALSE if the first character from $needle is not found.
  * Notes:
  * If $needle is not a string, it is converted to an integer and applied as the ordinal value (codepoint if the encoding is UTF-8) of a character.
  * If $before_needle is set to TRUE, the function returns all of $haystack from the beginning to the first occurrence.
@@ -1350,7 +1353,7 @@ 
  * @param mixed $needle The string to be found.
  * @param bool $before_needle (optional) Determines which portion of $haystack this function returns. The default value is FALSE.
  * @param string $encoding (optional) The used internally by this function character encoding. If it is omitted, the platform character set will be used by default.
- * @return mixed Returns the portion of $haystack, or FALSE if $needle is not found.
+ * @return false|string Returns the portion of $haystack, or FALSE if $needle is not found.
  * Notes:
  * If $needle is not a string, it is converted to an integer and applied as the ordinal value (codepoint if the encoding is UTF-8) of a character.
  * If $before_needle is set to TRUE, the function returns all of $haystack from the beginning to the first occurrence of $needle.
@@ -1540,8 +1543,8 @@ 
 
 /**
  * Performs a regular expression search and replace, UTF-8 aware when it is applicable.
- * @param string|array $pattern			The pattern to search for. It can be either a string or an array with strings.
- * @param string|array $replacement		The string or an array with strings to replace.
+ * @param string $pattern			The pattern to search for. It can be either a string or an array with strings.
+ * @param string $replacement		The string or an array with strings to replace.
  * @param string|array $subject			The string or an array with strings to search and replace.
  * @param int $limit					The maximum possible replacements for each pattern in each subject string. Defaults to -1 (no limit).
  * @param int &$count					If specified, this variable will be filled with the number of replacements done.
@@ -1692,7 +1695,7 @@ 
 
 /**
  * This function checks whether two $encoding are equal (same, equvalent).
- * @param string|array $encoding1		The first encoding
+ * @param string $encoding1		The first encoding
  * @param string|array $encoding2		The second encoding
  * @param bool $strict					When this parameter is TRUE the comparison ignores aliases of encodings.
  * When the parameter is FALSE, aliases are taken into account.
@@ -1800,7 +1803,7 @@ 
  * Checks a string for UTF-8 validity.
  * @param string $string
  *
- * @return string
+ * @return boolean
  */
 function api_is_valid_utf8($string)
 {
@@ -1974,7 +1977,7 @@ 
 /**
  * Replaces non-valid formats for person names with the default (English) format.
  * @param string $format The input format to be verified.
- * @return bool Returns the same format if is is valid, otherwise returns a valid English format.
+ * @return string Returns the same format if is is valid, otherwise returns a valid English format.
  */
 function _api_validate_person_name_format($format)
 {
@@ -2106,6 +2109,9 @@ 
 
 // This function checks whether the function _api_convert_encoding() (the php-
 // implementation) is able to convert from/to a given encoding.
+/**
+ * @param string $encoding
+ */
 function _api_convert_encoding_supports($encoding)
 {
     static $supports = array();
@@ -2117,7 +2123,7 @@ 
 
 /**
  * Given a date object, return a human or ISO format, with or without h:m:s
- * @param object $date The Date object
+ * @param DateTime $date The Date object
  * @param bool $showTime Whether to show the time and date (true) or only the date (false)
  * @param bool $humanForm Whether to show day-month-year (true) or year-month-day (false)
  * @return string Formatted date

Braces +17 added lines, -10 removed lines

@@ -166,7 +166,8 @@ 
     $purified = false,
     $check_sub_language = false,
     $setParentLanguageName = true
-) {
+)
+{
     global $language_interface;
 
     if (empty($language_interface)) {
@@ -408,7 +409,8 @@ 
     $time = null,
     $return_null_if_invalid_date = false,
     $returnObj = false
-) {
+)
+{
     $from_timezone = api_get_timezone();
     $to_timezone = 'UTC';
     if (is_null($time) || empty($time) || $time === '0000-00-00 00:00:00') {
@@ -459,7 +461,8 @@ 
     $return_null_if_invalid_date = false,
     $showTime = true,
     $humanForm = false
-) {
+)
+{
     // Determining the timezone to be converted from
     if (is_null($from_timezone)) {
         $from_timezone = 'UTC';
@@ -823,7 +826,8 @@ 
     $language = null,
     $encoding = null,
     $username = null
-) {
+)
+{
     static $valid = array();
     if (empty($format)) {
         $format = PERSON_NAME_COMMON_CONVENTION;
@@ -1505,7 +1509,8 @@ 
     $flags = 0,
     $offset = 0,
     $encoding = null
-) {
+)
+{
     if (empty($encoding)) {
         $encoding = _api_mb_internal_encoding();
     }
@@ -1528,7 +1533,8 @@ 
  * @return int|boolean					Returns the number of full pattern matches (which might be zero), or FALSE if an error occurred.
  * @link http://php.net/preg_match_all
  */
-function api_preg_match_all($pattern, $subject, &$matches, $flags = PREG_PATTERN_ORDER, $offset = 0, $encoding = null) {
+function api_preg_match_all($pattern, $subject, &$matches, $flags = PREG_PATTERN_ORDER, $offset = 0, $encoding = null)
+{
     if (empty($encoding)) {
         $encoding = _api_mb_internal_encoding();
     }
@@ -1581,7 +1587,8 @@ 
  * @return array						Returns an array containing substrings of $subject split along boundaries matched by $pattern.
  * @link http://php.net/preg_split
  */
-function api_preg_split($pattern, $subject, $limit = -1, $flags = 0, $encoding = null) {
+function api_preg_split($pattern, $subject, $limit = -1, $flags = 0, $encoding = null)
+{
     if (empty($encoding)) {
         $encoding = _api_mb_internal_encoding();
     }
@@ -1708,8 +1715,7 @@ 
             }
         }
         return false;
-    }
-    elseif (is_array($encoding2)) {
+    } elseif (is_array($encoding2)) {
         foreach ($encoding2 as $encoding) {
             if (api_equal_encodings($encoding1, $encoding, $strict)) {
                 return true;
@@ -1838,7 +1844,8 @@ 
  * @param string $pluginName the Plugin name
  * @return string the variable translated
  */
-function get_plugin_lang($variable, $pluginName) {
+function get_plugin_lang($variable, $pluginName)
+{
     $plugin = $pluginName::create();
     return $plugin->get_lang($variable);
 }

main/wiki/wiki.inc.php

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

@@ -372,7 +372,7 @@ 
         $id = Database::insert($tbl_wiki, $params);
 
         if ($id > 0) {
-            $sql = "UPDATE $tbl_wiki SET id = iid WHERE iid = $id";
+            $sql = "update $tbl_wiki SET id = iid WHERE iid = $id";
             Database::query($sql);
 
             // insert into item_property
@@ -493,7 +493,7 @@ 
         $id = Database::insert($tbl_wiki, $params);
 
         if ($id) {
-            $sql = "UPDATE $tbl_wiki SET id = iid WHERE iid = $id";
+            $sql = "update $tbl_wiki SET id = iid WHERE iid = $id";
             Database::query($sql);
 
             api_item_property_update(
@@ -682,7 +682,7 @@ 
                 ];
                 $id = Database::insert($tbl_wiki, $params);
                 if ($id > 0) {
-                    $sql = "UPDATE $tbl_wiki SET id = iid WHERE iid = $id";
+                    $sql = "update $tbl_wiki SET id = iid WHERE iid = $id";
                     Database::query($sql);
 
                     //insert into item_property
@@ -3386,7 +3386,7 @@ 
             ];
             $discussId = Database::insert($tbl_wiki_discuss, $params);
             if ($discussId) {
-                $sql = "UPDATE $tbl_wiki_discuss SET id = iid WHERE iid = $discussId";
+                $sql = "update $tbl_wiki_discuss SET id = iid WHERE iid = $discussId";
                 Database::query($sql);
             }
 

Doc Comments +14 added lines, -6 removed lines

@@ -292,7 +292,7 @@ 
      * This function saves a change in a wiki page
      * @author Patrick Cool <patrick.cool@ugent.be>, Ghent University
      * @param array $values
-     * @return language string saying that the changes are stored
+     * @return string string saying that the changes are stored
      **/
     public function save_wiki($values)
     {
@@ -1503,6 +1503,7 @@ 
      * Checks if this navigation tab has to be set to active
      * @author Patrick Cool <patrick.cool@ugent.be>, Ghent University
      *
+     * @param string $paramwk
      * @return string html code
      */
     public function is_active_navigation_tab($paramwk)
@@ -2034,6 +2035,7 @@ 
 
     /**
      * Sends pending e-mails
+     * @param string $type
      */
     public function check_emailcue(
         $id_or_ref,
@@ -2258,7 +2260,7 @@ 
     /**
      * Function export last wiki page version to document area
      * @param int $doc_id wiki page id
-     * @return mixed
+     * @return false|string
      * @author Juan Carlos Raña <herodoto@telefonica.net>
      */
     public function export2doc($doc_id)
@@ -2411,6 +2413,7 @@ 
 
     /**
      * Exports the wiki page to PDF
+     * @param string $course_code
      */
     public function export_to_pdf($id, $course_code)
     {
@@ -3084,6 +3087,7 @@ 
     /**
      * Draws an HTML form select with the given options
      *
+     * @param string $name
      */
     public function make_select($name, $values, $checked = '')
     {
@@ -3122,7 +3126,7 @@ 
     /**
      * Get wiki information
      * @param   int|bool wiki id
-     * @return  array   wiki data
+     * @return  string   wiki data
      */
     public function getWikiDataFromDb($id)
     {
@@ -3170,6 +3174,7 @@ 
      * Get wiki information
      * @param   string     wiki id
      * @param int $courseId
+     * @param string $title
      * @return  array   wiki data
      */
     public function getPageByTitle($title, $courseId = null)
@@ -3202,6 +3207,8 @@ 
      * @param int $courseId
      * @param string
      * @param string
+     * @param string $groupfilter
+     * @param string $condition_session
      * @return bool
      */
     public function deletePage(
@@ -4553,6 +4560,7 @@ 
 
     /**
      * Show all pages
+     * @param string $action
      */
     public function allPages($action)
     {
@@ -5446,7 +5454,7 @@ 
 
     /**
      * Restore page
-     * @return bool
+     * @return false|null
      */
     public function restorePage()
     {
@@ -5600,7 +5608,7 @@ 
     }
 
     /**
-     * @param int|bool $wikiId
+     * @param false|string $wikiId
      */
     public function setWikiData($wikiId)
     {
@@ -5618,7 +5626,7 @@ 
     /**
      * Check last version
      * @param int $view
-     * @return bool
+     * @return false|null
      */
     public function checkLastVersion($view)
     {

Spacing +1 added lines, -1 removed lines

@@ -3114,7 +3114,7 @@
      */
     public function two_digits($number)
     {
-        $number = (int)$number;
+        $number = (int) $number;
 
         return ($number < 10) ? '0'.$number : $number;
     }

Braces +8 added lines, -4 removed lines

@@ -4577,7 +4577,8 @@ 
         echo '</div>';
 
         if (api_is_allowed_to_edit(false, true) || api_is_platform_admin(
-            )) { //only by professors if page is hidden
+            )) {
+//only by professors if page is hidden
             $sql = 'SELECT  *
                     FROM  '.$tbl_wiki.' s1
         		    WHERE s1.c_id = '.$course_id.' AND id=(
@@ -5364,7 +5365,8 @@ 
 
         if (api_is_allowed_to_edit(false, true) ||
             api_is_platform_admin()
-        ) { //only by professors if page is hidden
+        ) {
+//only by professors if page is hidden
             $sql = 'SELECT *, MAX(version) AS MAX FROM '.$tbl_wiki.'
                     WHERE c_id = '.$course_id.' AND '.$groupfilter.$condition_session.'
                     GROUP BY reflink';//TODO:check MAX and group by return last version
@@ -6029,7 +6031,8 @@ 
         echo '<div class="actions">'.get_lang('MostVisitedPages').'</div>';
 
         if (api_is_allowed_to_edit(false, true) || api_is_platform_admin(
-            )) { //only by professors if page is hidden
+            )) {
+//only by professors if page is hidden
             $sql = 'SELECT *, SUM(hits) AS tsum FROM '.$tbl_wiki.'
                     WHERE c_id = '.$course_id.' AND '.$groupfilter.$condition_session.'
                     GROUP BY reflink';
@@ -6877,7 +6880,8 @@ 
                         'ShowDifferences'
                     ).' '.get_lang('WordsDiff').'</button>';
                 echo '</ul></form>';
-            } else { // We show the differences between two versions
+            } else {
+// We show the differences between two versions
                 $version_old = array();
                 if (isset($_POST['old'])) {
                     $sql_old = "SELECT * FROM $tbl_wiki

plugin/maintenancemode/MaintenanceModePlugin.php

Doc Comments +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
 class MaintenanceModePlugin extends Plugin
 {
     /**
-     * @return EditHtaccessPlugin
+     * @return MaintenanceModePlugin
      */
     public static function create()
     {