chamilo / chamilo-lms

main/inc/lib/course_home.lib.php

Doc Comments +1 added lines, -1 removed lines

@@ -218,7 +218,7 @@
     /**
      * Displays the tools of a certain category.
      *
-     * @return void
+     * @return string
      * @param string $course_tool_category	contains the category of tools to display:
      * "Public", "PublicButHide", "courseAdmin", "claroAdmin"
      */

main/lp/learnpath.class.php

Indentation +3 added lines, -3 removed lines

@@ -6038,8 +6038,8 @@ 
             $course_id = api_get_course_int_id();
         }
 
-       $creatorId = empty($creatorId) ? api_get_user_id() : $creatorId;
-       $sessionId = api_get_session_id();
+        $creatorId = empty($creatorId) ? api_get_user_id() : $creatorId;
+        $sessionId = api_get_session_id();
 
         // Generates folder
         $result = $this->generate_lp_folder($courseInfo);
@@ -11114,7 +11114,7 @@ 
         $lpItemId = [];
         $typeListNotToVerify = self::getChapterTypes();
 
-	    // Using get_toc() function instead $this->items because returns the correct order of the items
+        // Using get_toc() function instead $this->items because returns the correct order of the items
         foreach ($this->get_toc() as $item) {
             if (!in_array($item['type'], $typeListNotToVerify)) {
                 $lpItemId[] = $item['id'];

Doc Comments +23 added lines, -19 removed lines

@@ -447,7 +447,7 @@ 
     }
 
     /**
-     * @param $course_id
+     * @param integer $course_id
      * @return int
      */
     public function set_course_int_id($course_id)
@@ -487,7 +487,7 @@ 
      * @param int $max_time_allowed
      * @param int $userId
      *
-     * @return int
+     * @return false|string
      */
     public function add_item(
         $parent,
@@ -1034,7 +1034,7 @@ 
      * @param array $courseInfo
      * @param integer $id    Learnpath ID
      * @param string $delete Whether to delete data or keep it (default: 'keep', others: 'remove')
-     * @return boolean    True on success, false on failure (might change that to return number of elements deleted)
+     * @return false|null    True on success, false on failure (might change that to return number of elements deleted)
      */
     public function delete($courseInfo = null, $id = null, $delete = 'keep')
     {
@@ -1181,7 +1181,7 @@ 
      * @param    integer $id Elem ID (0 if first)
      * @param    integer $remove Whether to remove the resource/data from the
      * system or leave it (default: 'keep', others 'remove')
-     * @return    integer    Number of elements moved
+     * @return    false|null    Number of elements moved
      * @todo implement resource removal
      */
     public function delete_item($id, $remove = 'keep')
@@ -1263,7 +1263,7 @@ 
      * @param   array   $audio The array resulting of the $_FILES[mp3] element
      * @param   int     $max_time_allowed
      * @param   string  $url
-     * @return  boolean True on success, false on error
+     * @return  false|null True on success, false on error
      */
     public function edit_item(
         $id,
@@ -1788,7 +1788,7 @@ 
 
     /**
      * Gets the first element URL.
-     * @return    string    URL to load into the viewer
+     * @return    false|null    URL to load into the viewer
      */
     public function first()
     {
@@ -2334,6 +2334,7 @@ 
      * @param int $student_id
      * @param string Course code (optional)
      * @param int $sessionId
+     * @param string $courseCode
      * @return	bool
      */
     public static function is_lp_visible_for_student(
@@ -2662,7 +2663,7 @@ 
     /**
      * @param string $size
      * @param string $path_type
-     * @return bool|string
+     * @return string|false
      */
     public function get_preview_image_path($size = null, $path_type = 'web')
     {
@@ -2809,6 +2810,8 @@ 
      * Returns the XML DOM document's node
      * @param	resource	Reference to a list of objects to search for the given ITEM_*
      * @param	string		The identifier to look for
+     * @param DOMNodeList $children
+     * @param string $id
      * @return	mixed		The reference to the element found with that identifier. False if not found
      */
     public function get_scorm_xml_node(& $children, $id)
@@ -2994,7 +2997,7 @@ 
     /**
      * Generate and return the table of contents for this learnpath. The (flat) table returned can be
      * used by get_html_toc() to be ready to display
-     * @return	array	TOC as a table with 4 elements per row: title, link, status and level
+     * @return	boolean	TOC as a table with 4 elements per row: title, link, status and level
      */
     public function get_toc()
     {
@@ -3126,7 +3129,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function getChapterTypes()
     {
@@ -4030,7 +4033,7 @@ 
      * Open a resource = initialise all local variables relative to this resource. Depending on the child
      * class, this might be redefined to allow several behaviours depending on the document type.
      * @param integer Resource ID
-     * @return boolean True on success, false otherwise
+     * @return boolean|null True on success, false otherwise
      */
     public function open($id)
     {
@@ -4199,7 +4202,7 @@ 
      * Can be used as abstract
      * @param	integer	$lp_id Learnpath id
      * @param	string	$set_visibility New visibility (v/i - visible/invisible)
-     * @return bool
+     * @return null|false
      */
     public static function toggle_publish($lp_id, $set_visibility = 'v')
     {
@@ -4464,7 +4467,7 @@ 
      * Make sure the results are saved with anoter method. This method should probably be
      * redefined in children classes.
      * To use a similar method  statically, use the create_new_attempt() method
-     * @return string URL to load in the viewer
+     * @return boolean URL to load in the viewer
      */
     public function restart()
     {
@@ -5041,7 +5044,7 @@ 
 
     /**
      * Sets use_max_score
-     * @param   string  $use_max_score Optional string giving the new location of this learnpath
+     * @param   integer  $use_max_score Optional string giving the new location of this learnpath
      * @return  boolean True on success / False on error
      */
     public function set_use_max_score($use_max_score = 1)
@@ -5462,6 +5465,7 @@ 
      * Register the attempt mode into db thanks to flags prevent_reinit and seriousgame_mode flags
      *
      * @param string 'seriousgame', 'single' or 'multiple'
+     * @param string $mode
      * @return boolean
      * @author ndiechburg <noel@cblue.be>
      **/
@@ -5504,7 +5508,7 @@ 
     /**
      * Switch between multiple attempt, single attempt or serious_game mode (only for scorm)
      *
-     * @return boolean
+     * @return boolean|null
      * @author ndiechburg <noel@cblue.be>
      **/
     public function switch_attempt_mode()
@@ -6829,7 +6833,7 @@ 
     /**
      * Function that displays a list with al the resources that
      * could be added to the learning path
-     * @return string
+     * @return boolean
      */
     public function display_resources()
     {
@@ -10588,7 +10592,7 @@ 
 
     /**
      * @param int $lp_id
-     * @return bool
+     * @return false|null
      */
     public function scorm_export_to_pdf($lp_id)
     {
@@ -11138,7 +11142,7 @@ 
 
     /**
      * @param int $itemId
-     * @return learnpathItem|false
+     * @return string
      */
     public function getItem($itemId)
     {
@@ -11313,7 +11317,7 @@ 
 
     /**
      * Get the item of exercise type (evaluation type)
-     * @return array The final evaluation. Otherwise return false
+     * @return integer The final evaluation. Otherwise return false
      */
     public function getFinalEvaluationItem()
     {
@@ -11530,7 +11534,7 @@ 
     /**
      * Create a forum for this learning path
      * @param type $forumCategoryId
-     * @return int The forum ID if was created. Otherwise return false
+     * @return false|string The forum ID if was created. Otherwise return false
      */
     public function createForum($forumCategoryId)
     {

Braces +63 added lines, -40 removed lines

@@ -3964,7 +3964,8 @@ 
         if ($num > 0) {
             $i = 1;
             while ($row = Database::fetch_array($res)) {
-                if ($row['display_order'] != $i) { // If we find a gap in the order, we need to fix it.
+                if ($row['display_order'] != $i) {
+// If we find a gap in the order, we need to fix it.
                     $sql = "UPDATE $lp_table SET display_order = $i
                             WHERE c_id = $courseId AND id = ".$row['id'];
                     Database::query($sql);
@@ -3975,9 +3976,11 @@ 
                 $i++;
             }
         }
-        if ($num > 1) { // If there's only one element, no need to sort.
+        if ($num > 1) {
+// If there's only one element, no need to sort.
             $order = $lps[$lp_id]['display_order'];
-            if ($order > 1) { // If it's the first element, no need to move up.
+            if ($order > 1) {
+// If it's the first element, no need to move up.
                 $sql = "UPDATE $lp_table SET display_order = $order
                         WHERE c_id = $courseId AND id = ".$lp_order[$order - 1];
                 Database::query($sql);
@@ -4037,9 +4040,11 @@ 
                 $i++;
             }
         }
-        if ($num > 1) { // If there's only one element, no need to sort.
+        if ($num > 1) {
+// If there's only one element, no need to sort.
             $order = $lps[$lp_id]['display_order'];
-            if ($order < $max) { // If it's the first element, no need to move up.
+            if ($order < $max) {
+// If it's the first element, no need to move up.
                 $sql_u1 = "UPDATE $lp_table SET display_order = $order
                            WHERE c_id = ".$courseId." AND id = ".$lp_order[$order + 1];
                 Database::query($sql_u1);
@@ -4842,8 +4847,9 @@ 
         if ($this->debug > 0) {
             error_log('New LP - In learnpath::set_maker()', 0);
         }
-        if (empty ($name))
-            return false;
+        if (empty ($name)) {
+                    return false;
+        }
         $this->maker = $name;
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
         $course_id = api_get_course_int_id();
@@ -4908,8 +4914,9 @@ 
     public function set_terms_by_prefix($terms_string, $prefix)
     {
         $course_id = api_get_course_int_id();
-        if (api_get_setting('search_enabled') !== 'true')
-            return false;
+        if (api_get_setting('search_enabled') !== 'true') {
+                    return false;
+        }
 
         if (!extension_loaded('xapian')) {
             return false;
@@ -4922,8 +4929,9 @@ 
         $stored_terms = $this->get_common_index_terms_by_prefix($prefix);
 
         // Don't do anything if no change, verify only at DB, not the search engine.
-        if ((count(array_diff($terms, $stored_terms)) == 0) && (count(array_diff($stored_terms, $terms)) == 0))
-            return false;
+        if ((count(array_diff($terms, $stored_terms)) == 0) && (count(array_diff($stored_terms, $terms)) == 0)) {
+                    return false;
+        }
 
         require_once 'xapian.php'; // TODO: Try catch every xapian use or make wrappers on API.
         require_once api_get_path(LIBRARY_PATH).'search/ChamiloIndexer.class.php';
@@ -5112,8 +5120,9 @@ 
         if ($this->debug > 0) {
             error_log('New LP - In learnpath::set_proximity()', 0);
         }
-        if (empty ($name))
-            return false;
+        if (empty ($name)) {
+                    return false;
+        }
 
         $this->proximity = $name;
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
@@ -6708,10 +6717,12 @@ 
                     if ($new_comment || $new_title) {
                         $tbl_doc = Database::get_course_table(TABLE_DOCUMENT);
                         $ct = '';
-                        if ($new_comment)
-                            $ct .= ", comment='".Database::escape_string($new_comment)."'";
-                        if ($new_title)
-                            $ct .= ", title='".Database::escape_string($new_title)."' ";
+                        if ($new_comment) {
+                                                    $ct .= ", comment='".Database::escape_string($new_comment)."'";
+                        }
+                        if ($new_title) {
+                                                    $ct .= ", title='".Database::escape_string($new_title)."' ";
+                        }
 
                         $sql = "UPDATE ".$tbl_doc." SET ".substr($ct, 1)."
                                WHERE c_id = ".$course_id." AND id = ".$document_id;
@@ -7443,8 +7454,9 @@ 
                         $arrHide[] = $arrLP[$i]['id'];
                     }
                 } else {
-                    if ($arrLP[$i]['item_type'] == 'dir')
-                        $return .= '<option '.(($parent == $arrLP[$i]['id']) ? 'selected="selected" ' : '').'style="padding-left:'.($arrLP[$i]['depth'] * 10).'px;" value="'.$arrLP[$i]['id'].'">'.$arrLP[$i]['title'].'</option>';
+                    if ($arrLP[$i]['item_type'] == 'dir') {
+                                            $return .= '<option '.(($parent == $arrLP[$i]['id']) ? 'selected="selected" ' : '').'style="padding-left:'.($arrLP[$i]['depth'] * 10).'px;" value="'.$arrLP[$i]['id'].'">'.$arrLP[$i]['title'].'</option>';
+                    }
                 }
             }
             reset($arrLP);
@@ -7461,11 +7473,13 @@ 
 
         for ($i = 0; $i < count($arrLP); $i++) {
             if ($arrLP[$i]['parent_item_id'] == $parent && $arrLP[$i]['id'] != $id) {
-                if ($extra_info['previous_item_id'] == $arrLP[$i]['id'])
+                if ($extra_info['previous_item_id'] == $arrLP[$i]['id']) {
+                                    $selected = 'selected="selected" ';
+                } elseif ($action == 'add') {
                     $selected = 'selected="selected" ';
-                elseif ($action == 'add') $selected = 'selected="selected" ';
-                else
-                    $selected = '';
+                } else {
+                                    $selected = '';
+                }
 
                 $return .= '<option '.$selected.'value="'.$arrLP[$i]['id'].'">'.get_lang('After').' "'.$arrLP[$i]['title'].'"</option>';
             }
@@ -7492,9 +7506,11 @@ 
                 $arrHide = array();
                 for ($i = 0; $i < count($arrLP); $i++) {
                     if ($arrLP[$i]['id'] != $id && $arrLP[$i]['item_type'] != 'dir') {
-                        if ($extra_info['previous_item_id'] == $arrLP[$i]['id'])
-                            $s_selected_position = $arrLP[$i]['id'];
-                        elseif ($action == 'add') $s_selected_position = 0;
+                        if ($extra_info['previous_item_id'] == $arrLP[$i]['id']) {
+                                                    $s_selected_position = $arrLP[$i]['id'];
+                        } elseif ($action == 'add') {
+                            $s_selected_position = 0;
+                        }
                         $arrHide[$arrLP[$i]['id']]['value'] = $arrLP[$i]['title'];
                     }
                 }
@@ -7917,9 +7933,11 @@ 
             $s_selected_position = 0;
             for ($i = 0; $i < count($arrLP); $i++) {
                 if ($arrLP[$i]['id'] != $id && $arrLP[$i]['item_type'] != 'dir') {
-                    if ($extra_info['previous_item_id'] == $arrLP[$i]['id'])
-                        $s_selected_position = $arrLP[$i]['id'];
-                    elseif ($action == 'add') $s_selected_position = 0;
+                    if ($extra_info['previous_item_id'] == $arrLP[$i]['id']) {
+                                            $s_selected_position = $arrLP[$i]['id'];
+                    } elseif ($action == 'add') {
+                        $s_selected_position = 0;
+                    }
                     $arrHide[$arrLP[$i]['id']]['value'] = $arrLP[$i]['title'];
 
                 }
@@ -7953,8 +7971,7 @@ 
 
         if (is_numeric($extra_info)) {
             $form->addHidden('path', $extra_info);
-        }
-        elseif (is_array($extra_info)) {
+        } elseif (is_array($extra_info)) {
             $form->addHidden('path', $extra_info['path']);
         }
 
@@ -8818,9 +8835,11 @@ 
             $arrHide = array();
             for ($i = 0; $i < count($arrLP); $i++) {
                 if ($arrLP[$i]['id'] != $id && $arrLP[$i]['item_type'] != 'dir') {
-                    if ($extra_info['previous_item_id'] == $arrLP[$i]['id'])
-                        $s_selected_position = $arrLP[$i]['id'];
-                    elseif ($action == 'add') $s_selected_position = 0;
+                    if ($extra_info['previous_item_id'] == $arrLP[$i]['id']) {
+                                            $s_selected_position = $arrLP[$i]['id'];
+                    } elseif ($action == 'add') {
+                        $s_selected_position = 0;
+                    }
                     $arrHide[$arrLP[$i]['id']]['value'] = $arrLP[$i]['title'];
 
                 }
@@ -9020,9 +9039,11 @@ 
             $arrHide = array();
             for ($i = 0; $i < count($arrLP); $i++) {
                 if ($arrLP[$i]['id'] != $id && $arrLP[$i]['item_type'] != 'dir') {
-                    if ($extra_info['previous_item_id'] == $arrLP[$i]['id'])
-                        $s_selected_position = $arrLP[$i]['id'];
-                    elseif ($action == 'add') $s_selected_position = 0;
+                    if ($extra_info['previous_item_id'] == $arrLP[$i]['id']) {
+                                            $s_selected_position = $arrLP[$i]['id'];
+                    } elseif ($action == 'add') {
+                        $s_selected_position = 0;
+                    }
                     $arrHide[$arrLP[$i]['id']]['value'] = $arrLP[$i]['title'];
 
                 }
@@ -10232,10 +10253,11 @@ 
 
                 // Dependency to other files - not yet supported.
                 $i = 1;
-                if ($inc_docs)
-                foreach ($inc_docs as $doc_info) {
+                if ($inc_docs) {
+                                foreach ($inc_docs as $doc_info) {
                     if (count($doc_info) < 1 || empty($doc_info[0])) {
                         continue;
+                }
                     }
                     $my_dep = $xmldoc->createElement('resource');
                     $res_id = 'RESOURCE_'.$item->get_id().'_'.$i;
@@ -10686,7 +10708,8 @@ 
 
                                                 // File path is courses/CHAMILO/document/....
                                                 $info_file_path = explode('/', $file_path);
-                                                if ($info_file_path[0] == 'courses') { // Add character "/" in file path.
+                                                if ($info_file_path[0] == 'courses') {
+// Add character "/" in file path.
                                                     $file_path_dest = 'document/'.$file_path;
                                                 }
 

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

@@ -228,7 +228,7 @@ 
                     ];
                     $this->lp_view_id = Database::insert($lp_table, $params);
                     if (!empty($this->lp_view_id)) {
-                        $sql = "UPDATE $lp_table SET id = iid
+                        $sql = "update $lp_table SET id = iid
                                 WHERE iid = ".$this->lp_view_id;
                         Database::query($sql);
                     }
@@ -395,7 +395,7 @@ 
                                         $insertId = Database::insert($itemViewTable, $params);
 
                                         if ($insertId) {
-                                            $sql = "UPDATE $itemViewTable SET id = iid
+                                            $sql = "update $itemViewTable SET id = iid
                                                     WHERE iid = $insertId";
                                             Database::query($sql);
                                         }
@@ -626,16 +626,16 @@ 
         }
 
         if ($new_item_id) {
-            $sql = "UPDATE $tbl_lp_item SET id = iid WHERE iid = $new_item_id";
+            $sql = "update $tbl_lp_item SET id = iid WHERE iid = $new_item_id";
             Database::query($sql);
 
-            $sql = "UPDATE $tbl_lp_item
+            $sql = "update $tbl_lp_item
                     SET previous_item_id = $new_item_id 
                     WHERE c_id = $course_id AND id = $next";
             Database::query($sql);
 
             // Update the item that should be before the new item.
-            $sql = "UPDATE $tbl_lp_item
+            $sql = "update $tbl_lp_item
                     SET next_item_id = $new_item_id
                     WHERE c_id = $course_id AND id = $tmp_previous";
             Database::query($sql);
@@ -723,7 +723,7 @@ 
                 $file = $file_components[count($file_components) - 1];
 
                 // Store the mp3 file in the lp_item table.
-                $sql = "UPDATE $tbl_lp_item SET
+                $sql = "update $tbl_lp_item SET
                             audio = '".Database::escape_string($file)."'
                         WHERE id = '" . intval($new_item_id)."'";
                 Database::query($sql);
@@ -871,7 +871,7 @@ 
                 $id = Database::insert($tbl_lp, $params);
 
                 if ($id > 0) {
-                    $sql = "UPDATE $tbl_lp SET id = iid WHERE iid = $id";
+                    $sql = "update $tbl_lp SET id = iid WHERE iid = $id";
                     Database::query($sql);
 
                     // Insert into item_property.
@@ -1236,14 +1236,14 @@ 
         }
         Database::query($sql_del);
         // Now update surrounding items.
-        $sql_upd = "UPDATE $lp_item SET next_item_id = $next
+        $sql_upd = "update $lp_item SET next_item_id = $next
                     WHERE c_id = ".$course_id." AND id = $previous";
         Database::query($sql_upd);
-        $sql_upd = "UPDATE $lp_item SET previous_item_id = $previous
+        $sql_upd = "update $lp_item SET previous_item_id = $previous
                     WHERE c_id = ".$course_id." AND id = $next";
         Database::query($sql_upd);
         // Now update all following items with new display order.
-        $sql_all = "UPDATE $lp_item SET display_order = display_order-1
+        $sql_all = "update $lp_item SET display_order = display_order-1
                     WHERE 
                         c_id = ".$course_id." AND 
                         lp_id = $lp AND 
@@ -1252,7 +1252,7 @@ 
         Database::query($sql_all);
 
         //Removing prerequisites since the item will not longer exist
-        $sql_all = "UPDATE $lp_item SET prerequisite = '' WHERE c_id = ".$course_id." AND prerequisite = $id";
+        $sql_all = "update $lp_item SET prerequisite = '' WHERE c_id = ".$course_id." AND prerequisite = $id";
         Database::query($sql_all);
 
         // Remove from search engine if enabled.
@@ -1468,7 +1468,7 @@ 
 
             // TODO: htmlspecialchars to be checked for encoding related problems.
             // Update the current item with the new data.
-            $sql = "UPDATE $tbl_lp_item
+            $sql = "update $tbl_lp_item
                     SET
                         title = '".Database::escape_string($title)."',
                         description = '".Database::escape_string($description)."',
@@ -3685,7 +3685,7 @@ 
             $id = Database::insert_id();
             $this->lp_view_id = $id;
 
-            $sql = "UPDATE $lp_view_table SET id = iid WHERE iid = $id";
+            $sql = "update $lp_view_table SET id = iid WHERE iid = $id";
             Database::query($sql);
         }
 
@@ -3826,7 +3826,7 @@ 
                     $previous_previous = $row2['previous_item_id'];
                     // Update previous_previous item (switch "next" with current).
                     if ($previous_previous != 0) {
-                        $sql_upd2 = "UPDATE $tbl_lp_item SET
+                        $sql_upd2 = "update $tbl_lp_item SET
                                         next_item_id = $id
                                     WHERE c_id = ".$course_id." AND id = $previous_previous";
                         if ($this->debug > 2) {
@@ -3836,7 +3836,7 @@ 
                     }
                     // Update previous item (switch with current).
                     if ($previous != 0) {
-                        $sql_upd2 = "UPDATE $tbl_lp_item SET
+                        $sql_upd2 = "update $tbl_lp_item SET
                                     next_item_id = $next,
                                     previous_item_id = $id,
                                     display_order = display_order +1
@@ -3849,7 +3849,7 @@ 
 
                     // Update current item (switch with previous).
                     if ($id != 0) {
-                        $sql_upd2 = "UPDATE $tbl_lp_item SET
+                        $sql_upd2 = "update $tbl_lp_item SET
                                         next_item_id = $previous,
                                         previous_item_id = $previous_previous,
                                         display_order = display_order-1
@@ -3861,7 +3861,7 @@ 
                     }
                     // Update next item (new previous item).
                     if ($next != 0) {
-                        $sql_upd2 = "UPDATE $tbl_lp_item SET previous_item_id = $previous
+                        $sql_upd2 = "update $tbl_lp_item SET previous_item_id = $previous
                                      WHERE c_id = ".$course_id." AND id = $next";
                         if ($this->debug > 2) {
                             error_log($sql_upd2, 0);
@@ -3890,14 +3890,14 @@ 
                     $next_next = $row2['next_item_id'];
                     // Update previous item (switch with current).
                     if ($previous != 0) {
-                        $sql_upd2 = "UPDATE $tbl_lp_item 
+                        $sql_upd2 = "update $tbl_lp_item 
                                      SET next_item_id = $next
                                      WHERE c_id = ".$course_id." AND id = $previous";
                         Database::query($sql_upd2);
                     }
                     // Update current item (switch with previous).
                     if ($id != 0) {
-                        $sql_upd2 = "UPDATE $tbl_lp_item SET
+                        $sql_upd2 = "update $tbl_lp_item SET
                                      previous_item_id = $next, 
                                      next_item_id = $next_next, 
                                      display_order = display_order + 1
@@ -3907,7 +3907,7 @@ 
 
                     // Update next item (new previous item).
                     if ($next != 0) {
-                        $sql_upd2 = "UPDATE $tbl_lp_item SET
+                        $sql_upd2 = "update $tbl_lp_item SET
                                      previous_item_id = $previous, 
                                      next_item_id = $id, 
                                      display_order = display_order-1
@@ -3917,7 +3917,7 @@ 
 
                     // Update next_next item (switch "previous" with current).
                     if ($next_next != 0) {
-                        $sql_upd2 = "UPDATE $tbl_lp_item SET
+                        $sql_upd2 = "update $tbl_lp_item SET
                                      previous_item_id = $id
                                      WHERE c_id = ".$course_id." AND id = $next_next";
                         Database::query($sql_upd2);
@@ -3965,7 +3965,7 @@ 
             $i = 1;
             while ($row = Database::fetch_array($res)) {
                 if ($row['display_order'] != $i) { // If we find a gap in the order, we need to fix it.
-                    $sql = "UPDATE $lp_table SET display_order = $i
+                    $sql = "update $lp_table SET display_order = $i
                             WHERE c_id = $courseId AND id = ".$row['id'];
                     Database::query($sql);
                 }
@@ -3978,10 +3978,10 @@ 
         if ($num > 1) { // If there's only one element, no need to sort.
             $order = $lps[$lp_id]['display_order'];
             if ($order > 1) { // If it's the first element, no need to move up.
-                $sql = "UPDATE $lp_table SET display_order = $order
+                $sql = "update $lp_table SET display_order = $order
                         WHERE c_id = $courseId AND id = ".$lp_order[$order - 1];
                 Database::query($sql);
-                $sql = "UPDATE $lp_table SET display_order = ".($order - 1)."
+                $sql = "update $lp_table SET display_order = ".($order - 1)."
                         WHERE c_id = $courseId AND id = ".$lp_id;
                 Database::query($sql);
             }
@@ -4027,7 +4027,7 @@ 
                 $max = $i;
                 if ($row['display_order'] != $i) {
                     // If we find a gap in the order, we need to fix it.
-                    $sql_u = "UPDATE $lp_table SET display_order = $i
+                    $sql_u = "update $lp_table SET display_order = $i
                               WHERE c_id = ".$courseId." AND id = ".$row['id'];
                     Database::query($sql_u);
                 }
@@ -4040,10 +4040,10 @@ 
         if ($num > 1) { // If there's only one element, no need to sort.
             $order = $lps[$lp_id]['display_order'];
             if ($order < $max) { // If it's the first element, no need to move up.
-                $sql_u1 = "UPDATE $lp_table SET display_order = $order
+                $sql_u1 = "update $lp_table SET display_order = $order
                            WHERE c_id = ".$courseId." AND id = ".$lp_order[$order + 1];
                 Database::query($sql_u1);
-                $sql_u2 = "UPDATE $lp_table SET display_order = ".($order + 1)."
+                $sql_u2 = "update $lp_table SET display_order = ".($order + 1)."
                            WHERE c_id = ".$courseId." AND id = ".$lp_id;
                 Database::query($sql_u2);
             }
@@ -4312,11 +4312,11 @@ 
                 Database::query($sql);
                 $insertId = Database::insert_id();
                 if ($insertId) {
-                    $sql = "UPDATE $tbl_tool SET id = iid WHERE iid = $insertId";
+                    $sql = "update $tbl_tool SET id = iid WHERE iid = $insertId";
                     Database::query($sql);
                 }
             } elseif ($set_visibility == 'v' && $num > 0) {
-                $sql = "UPDATE $tbl_tool SET
+                $sql = "update $tbl_tool SET
                             c_id = $course_id,
                             name = '$name',
                             link = '$link',
@@ -4585,7 +4585,7 @@ 
         $view_id = Database::insert_id();
 
         if ($view_id) {
-            $sql = "UPDATE $lp_view_table SET id = iid WHERE iid = $view_id";
+            $sql = "update $lp_view_table SET id = iid WHERE iid = $view_id";
             Database::query($sql);
             $this->lp_view_id = $view_id;
             $this->attempt = $this->attempt + 1;
@@ -4707,7 +4707,7 @@ 
             if ($this->debug > 2) {
                 error_log('New LP - Saving current item ('.$this->current.') for later review', 0);
             }
-            $sql = "UPDATE $table SET
+            $sql = "update $table SET
                         last_item = ".intval($this->get_current_item_id())."
                     WHERE
                         c_id = $course_id AND
@@ -4725,7 +4725,7 @@ 
             list($progress,) = $this->get_progress_bar_text('%');
             if ($progress >= 0 && $progress <= 100) {
                 $progress = (int) $progress;
-                $sql = "UPDATE $table SET
+                $sql = "update $table SET
                             progress = $progress
                         WHERE
                             c_id = ".$course_id." AND
@@ -4798,7 +4798,7 @@ 
             $lp = $this->get_id();
             if ($lp != 0) {
                 $tbl_lp = Database::get_course_table(TABLE_LP_MAIN);
-                $sql = "UPDATE $tbl_lp SET default_encoding = '$enc' 
+                $sql = "update $tbl_lp SET default_encoding = '$enc' 
                         WHERE c_id = ".$course_id." AND id = ".$lp;
                 $res = Database::query($sql);
                 return $res;
@@ -4823,7 +4823,7 @@ 
 
         if ($lp != 0) {
             $tbl_lp = Database::get_course_table(TABLE_LP_MAIN);
-            $sql = "UPDATE $tbl_lp SET js_lib = '$lib' 
+            $sql = "update $tbl_lp SET js_lib = '$lib' 
                     WHERE c_id = ".$course_id." AND id = ".$lp;
             $res = Database::query($sql);
             return $res;
@@ -4848,7 +4848,7 @@ 
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
         $course_id = api_get_course_int_id();
         $lp_id = $this->get_id();
-        $sql = "UPDATE $lp_table SET
+        $sql = "update $lp_table SET
                 content_maker = '".Database::escape_string($this->maker)."'
                 WHERE c_id = ".$course_id." AND id = '$lp_id'";
         if ($this->debug > 2) {
@@ -4875,7 +4875,7 @@ 
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
         $lp_id = $this->get_id();
         $course_id = $this->course_info['real_id'];
-        $sql = "UPDATE $lp_table SET
+        $sql = "update $lp_table SET
                 name = '".Database::escape_string($this->name)."'
                 WHERE c_id = ".$course_id." AND id = '$lp_id'";
         if ($this->debug > 2) {
@@ -4888,7 +4888,7 @@ 
             $session_condition = api_get_session_condition($session_id);
             $tbl_tool = Database::get_course_table(TABLE_TOOL_LIST);
             $link = 'lp/lp_controller.php?action=view&lp_id='.$lp_id.'&id_session='.$session_id;
-            $sql = "UPDATE $tbl_tool SET name = '$this->name'
+            $sql = "update $tbl_tool SET name = '$this->name'
             	    WHERE
             	        c_id = $course_id AND
             	        (link='$link' AND image='scormbuilder.gif' $session_condition)";
@@ -4990,7 +4990,7 @@ 
         $this->theme = $name;
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
         $lp_id = $this->get_id();
-        $sql = "UPDATE $lp_table SET theme = '".Database::escape_string($this->theme)."'
+        $sql = "update $lp_table SET theme = '".Database::escape_string($this->theme)."'
                 WHERE c_id = ".$course_id." AND id = '$lp_id'";
         if ($this->debug > 2) {
             error_log('New LP - lp updated with new theme : '.$this->theme, 0);
@@ -5015,7 +5015,7 @@ 
         $this->preview_image = $name;
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
         $lp_id = $this->get_id();
-        $sql = "UPDATE $lp_table SET
+        $sql = "update $lp_table SET
                 preview_image = '".Database::escape_string($this->preview_image)."'
                 WHERE c_id = ".$course_id." AND id = '$lp_id'";
         if ($this->debug > 2) {
@@ -5039,7 +5039,7 @@ 
         $this->author = $name;
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
         $lp_id = $this->get_id();
-        $sql = "UPDATE $lp_table SET author = '".Database::escape_string($name)."'
+        $sql = "update $lp_table SET author = '".Database::escape_string($name)."'
                 WHERE c_id = ".$course_id." AND id = '$lp_id'";
         if ($this->debug > 2) {
             error_log('New LP - lp updated with new preview author : '.$this->author, 0);
@@ -5064,7 +5064,7 @@ 
             $this->hide_toc_frame = $hide;
             $lp_table = Database::get_course_table(TABLE_LP_MAIN);
             $lp_id = $this->get_id();
-            $sql = "UPDATE $lp_table SET
+            $sql = "update $lp_table SET
                     hide_toc_frame = '".(int) $this->hide_toc_frame."'
                     WHERE c_id = ".$course_id." AND id = '$lp_id'";
             if ($this->debug > 2) {
@@ -5092,7 +5092,7 @@ 
         $this->prerequisite = intval($prerequisite);
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
         $lp_id = $this->get_id();
-        $sql = "UPDATE $lp_table SET prerequisite = '".$this->prerequisite."'
+        $sql = "update $lp_table SET prerequisite = '".$this->prerequisite."'
                 WHERE c_id = ".$course_id." AND id = '$lp_id'";
         if ($this->debug > 2) {
             error_log('New LP - lp updated with new preview requisite : '.$this->requisite, 0);
@@ -5118,7 +5118,7 @@ 
         $this->proximity = $name;
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
         $lp_id = $this->get_id();
-        $sql = "UPDATE $lp_table SET
+        $sql = "update $lp_table SET
                     content_local = '".Database::escape_string($name)."'
                 WHERE c_id = ".$course_id." AND id = '$lp_id'";
         if ($this->debug > 2) {
@@ -5155,7 +5155,7 @@ 
         $this->use_max_score = $use_max_score;
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
         $lp_id = $this->get_id();
-        $sql = "UPDATE $lp_table SET
+        $sql = "update $lp_table SET
                     use_max_score = '".$this->use_max_score."'
                 WHERE c_id = ".$course_id." AND id = '$lp_id'";
 
@@ -5257,7 +5257,7 @@ 
         $this->modified_on = api_get_utc_datetime();
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
         $lp_id = $this->get_id();
-        $sql = "UPDATE $lp_table SET modified_on = '".$this->modified_on."'
+        $sql = "update $lp_table SET modified_on = '".$this->modified_on."'
                 WHERE c_id = ".$course_id." AND id = '$lp_id'";
         if ($this->debug > 2) {
             error_log('New LP - lp updated with new expired_on : '.$this->modified_on, 0);
@@ -5411,7 +5411,7 @@ 
                     $view_mode = 'fullscreen';
                     break;
             }
-            $sql = "UPDATE $lp_table SET default_view_mod = '$view_mode'
+            $sql = "update $lp_table SET default_view_mod = '$view_mode'
                     WHERE c_id = ".$course_id." AND id = ".$this->get_id();
             Database::query($sql);
             $this->mode = $view_mode;
@@ -5449,7 +5449,7 @@ 
                 $force = 1;
                 $force_return = true;
             }
-            $sql = "UPDATE $lp_table SET force_commit = $force
+            $sql = "update $lp_table SET force_commit = $force
                     WHERE c_id = ".$course_id." AND id = ".$this->get_id();
             Database::query($sql);
             $this->force_commit = $force_return;
@@ -5486,7 +5486,7 @@ 
             while ($row = Database::fetch_array($res)) {
                 if ($row['display_order'] != $i) {
                     // If we find a gap in the order, we need to fix it.
-                    $sql = "UPDATE $lp_table SET display_order = $i
+                    $sql = "update $lp_table SET display_order = $i
                             WHERE c_id = ".$course_id." AND id = ".$row['id'];
                     Database::query($sql);
                 }
@@ -5518,7 +5518,7 @@ 
             } elseif ($force == 0) {
                 $force = 1;
             }
-            $sql = "UPDATE $lp_table SET prevent_reinit = $force
+            $sql = "update $lp_table SET prevent_reinit = $force
                     WHERE c_id = ".$course_id." AND id = ".$this->get_id();
             Database::query($sql);
             $this->prevent_reinit = $force;
@@ -5590,7 +5590,7 @@ 
         $this->prevent_reinit = $prevent_reinit;
         $this->seriousgame_mode = $sg_mode;
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
-        $sql = "UPDATE $lp_table SET
+        $sql = "update $lp_table SET
                 prevent_reinit = $prevent_reinit ,
                 seriousgame_mode = $sg_mode
                 WHERE c_id = ".$course_id." AND id = ".$this->get_id();
@@ -5656,7 +5656,7 @@ 
             } elseif ($force == 0) {
                 $force = 1;
             }
-            $sql = "UPDATE $lp_table SET seriousgame_mode = $force
+            $sql = "update $lp_table SET seriousgame_mode = $force
 			        WHERE c_id = ".$course_id." AND id = ".$this->get_id();
             Database::query($sql);
             $this->seriousgame_mode = $force;
@@ -5691,7 +5691,7 @@ 
             } elseif ($force == 0) {
                 $force = 1;
             }
-            $sql = "UPDATE $lp_table SET debug = $force
+            $sql = "update $lp_table SET debug = $force
                     WHERE c_id = ".$course_id." AND id = ".$this->get_id();
             Database::query($sql);
             $this->scorm_debug = $force;
@@ -10908,7 +10908,7 @@ 
         </div>
     </body>
 </html>
-EOD;
+eod;
         if (!is_dir($archive_path.$temp_dir_short.'/document')) {
             @mkdir($archive_path.$temp_dir_short.'/document', api_get_permissions_for_new_directories());
         }
@@ -11244,12 +11244,12 @@ 
         $tbl_lp_item = Database::get_course_table(TABLE_LP_ITEM);
         $lp_id = $this->get_id();
         //Cleaning prerequisites
-        $sql = "UPDATE $tbl_lp_item SET prerequisite = ''
+        $sql = "update $tbl_lp_item SET prerequisite = ''
                 WHERE c_id = ".$course_id." AND lp_id = '$lp_id'";
         Database::query($sql);
 
         //Cleaning mastery score for exercises
-        $sql = "UPDATE $tbl_lp_item SET mastery_score = ''
+        $sql = "update $tbl_lp_item SET mastery_score = ''
                 WHERE c_id = ".$course_id." AND lp_id = '$lp_id' AND item_type = 'quiz'";
         Database::query($sql);
     }
@@ -11277,12 +11277,12 @@ 
                         // Current item is not a folder, so it qualifies to get a prerequisites
                         if ($last_item_not_dir_type == 'quiz') {
                             // if previous is quiz, mark its max score as default score to be achieved
-                            $sql = "UPDATE $tbl_lp_item SET mastery_score = '$last_item_not_dir_max'
+                            $sql = "update $tbl_lp_item SET mastery_score = '$last_item_not_dir_max'
                                     WHERE c_id = ".$course_id." AND lp_id = '$lp_id' AND id = '$last_item_not_dir'";
                             Database::query($sql);
                         }
                         // now simply update the prerequisite to set it to the last non-chapter item
-                        $sql = "UPDATE $tbl_lp_item SET prerequisite = '$last_item_not_dir'
+                        $sql = "update $tbl_lp_item SET prerequisite = '$last_item_not_dir'
                                 WHERE c_id = ".$course_id." AND lp_id = '$lp_id' AND id = '$current_item_id'";
                         Database::query($sql);
                         // record item as 'non-chapter' reference
@@ -11548,7 +11548,7 @@ 
         $courseId = api_get_course_int_id();
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
         $lp_id = $this->get_id();
-        $sql = "UPDATE $lp_table SET category_id = ".$this->categoryId."
+        $sql = "update $lp_table SET category_id = ".$this->categoryId."
                 WHERE c_id = $courseId AND id = $lp_id";
         Database::query($sql);
 
@@ -11579,7 +11579,7 @@ 
         $this->subscribeUsers = (int) $value;
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
         $lp_id = $this->get_id();
-        $sql = "UPDATE $lp_table SET subscribe_users = ".$this->subscribeUsers."
+        $sql = "update $lp_table SET subscribe_users = ".$this->subscribeUsers."
                 WHERE c_id = ".$this->course_int_id." AND id = $lp_id";
         Database::query($sql);
 
@@ -12154,7 +12154,7 @@ 
         $this->accumulateScormTime = intval($value);
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
         $lp_id = $this->get_id();
-        $sql = "UPDATE $lp_table 
+        $sql = "update $lp_table 
                 SET accumulate_scorm_time = ".$this->accumulateScormTime."
                 WHERE c_id = ".$this->course_int_id." AND id = $lp_id";
         Database::query($sql);

main/gradebook/lib/user_data_generator.class.php

Doc Comments +4 added lines, -4 removed lines

@@ -285,7 +285,7 @@ 
 
     /**
      * @param $item
-     * @param $ignore_score_color
+     * @param boolean $ignore_score_color
      * @return string
      */
     private function build_average_column($item, $ignore_score_color)
@@ -306,7 +306,7 @@ 
 
     /**
      * @param $item
-     * @param $ignore_score_color
+     * @param boolean $ignore_score_color
      * @return string
      */
     private function build_result_column($item, $ignore_score_color)
@@ -323,7 +323,7 @@ 
 
     /**
      * @param $item
-     * @param $ignore_score_color
+     * @param boolean $ignore_score_color
      * @return string
      */
     private function build_mask_column($item, $ignore_score_color)
@@ -339,7 +339,7 @@ 
 
     /**
      * @param $coursecode
-     * @return mixed
+     * @return string
      */
     private function get_course_name_from_code_cached($coursecode)
     {

src/Chamilo/UserBundle/Entity/Manager/UserManager.php

Doc Comments +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
      *
      * @param string $token
      *
-     * @return UserInterface
+     * @return \FOS\UserBundle\Model\UserInterface|null
      */
     public function findUserByConfirmationToken($token)
     {

plugin/vchamilo/lib/Virtual.php

Doc Comments +9 added lines, -7 removed lines

@@ -184,7 +184,7 @@ 
     * provides a side connection to a vchamilo database
     * @param array $_configuration
      *
-    * @return \Doctrine\DBAL\Driver\Connection
+    * @return string
     */
     public static function bootConnection(&$_configuration)
     {
@@ -372,7 +372,7 @@ 
 
     /**
     * get a proper SQLdump command
-    * @param object $vchamilodata the complete new host information
+    * @param stdClass $vchamilodata the complete new host information
     * @return string the shell command
     */
     public static function getDatabaseDumpCmd($vchamilodata)
@@ -456,8 +456,8 @@ 
 
     /**
      * Backups a database for having a snapshot.
-     * @param        $vchamilo    object        The Vchamilo object.
-     * @param        $outputfilerad    string        The output SQL file radical.
+     * @param        stdClass $vchamilo    object        The Vchamilo object.
+     * @param        string $outputfilerad    string        The output SQL file radical.
      * @return        bool    If TRUE, dumping database was a success, otherwise FALSE.
      */
     public static function backupDatabase($vchamilo, $outputfilerad)
@@ -619,6 +619,8 @@ 
     * this function set will map standard moodle API calls to chamilo
     * internal primitives. This avoids too many changes to do in imported
     * code
+    * @param string $module
+    * @param string $key
     */
     public static function getConfig($module, $key, $isplugin = true)
     {
@@ -698,7 +700,7 @@ 
     /**
      * @param string $path
      *
-     * @return mixed
+     * @return string
      */
     public static function chopLastSlash($path)
     {
@@ -852,7 +854,7 @@ 
 
     /**
      * @param object $instance
-     * @return bool|\Doctrine\DBAL\Connection
+     * @return Doctrine\ORM\EntityManagerInterface
      */
     public static function getConnectionFromInstance($instance, $getManager = false)
     {
@@ -1297,7 +1299,7 @@ 
     /**
      * @param stdClass $instance
      *
-     * @return bool|string returns the original version of the app
+     * @return string|false returns the original version of the app
      */
     public static function canBeUpgraded($instance)
     {

Indentation +20 added lines, -20 removed lines

@@ -186,11 +186,11 @@ 
     }
 
     /**
-    * provides a side connection to a vchamilo database
-    * @param array $_configuration
+     * provides a side connection to a vchamilo database
+     * @param array $_configuration
      *
-    * @return \Doctrine\DBAL\Driver\Connection
-    */
+     * @return \Doctrine\DBAL\Driver\Connection
+     */
     public static function bootConnection(&$_configuration)
     {
         $dbParams = array(
@@ -298,10 +298,10 @@ 
     }
 
     /**
-    * drop a vchamilo instance databases using the physical connection
-    * @param stdClass $params
-    * return an array of errors or false if ok
-    */
+     * drop a vchamilo instance databases using the physical connection
+     * @param stdClass $params
+     * return an array of errors or false if ok
+     */
     public static function dropDatabase($params)
     {
         $params = clone $params;
@@ -376,10 +376,10 @@ 
     }
 
     /**
-    * get a proper SQLdump command
-    * @param object $vchamilodata the complete new host information
-    * @return string the shell command
-    */
+     * get a proper SQLdump command
+     * @param object $vchamilodata the complete new host information
+     * @return string the shell command
+     */
     public static function getDatabaseDumpCmd($vchamilodata)
     {
         $pgm = self::getConfig('vchamilo', 'mysql_cmd');
@@ -536,8 +536,8 @@ 
     }
 
     /**
-    * read manifest values in vchamilo template.
-    */
+     * read manifest values in vchamilo template.
+     */
     public static function getVmanifest($version)
     {
         $templatewwwroot = '';
@@ -558,8 +558,8 @@ 
     }
 
     /**
-    * make a fake vchamilo that represents the current host
-    */
+     * make a fake vchamilo that represents the current host
+     */
     public static function makeThis()
     {
         global $_configuration;
@@ -623,10 +623,10 @@ 
     }
 
     /**
-    * this function set will map standard moodle API calls to chamilo
-    * internal primitives. This avoids too many changes to do in imported
-    * code
-    */
+     * this function set will map standard moodle API calls to chamilo
+     * internal primitives. This avoids too many changes to do in imported
+     * code
+     */
     public static function getConfig($module, $key, $isplugin = true)
     {
         if ($isplugin) {

Braces +2 added lines, -1 removed lines

@@ -173,7 +173,8 @@
         $_configuration['vchamilo_web_root'] = "{$protocol}://".@$_SERVER['HTTP_HOST'].$contentPrefix;
 
         $_configuration['vchamilo_name'] = @$_SERVER['HTTP_HOST'];
-        if (empty($_configuration['vchamilo_name'])) { // try again with another source if has failed
+        if (empty($_configuration['vchamilo_name'])) {
+// try again with another source if has failed
             $_configuration['vchamilo_name'] = "{$protocol}://".$_SERVER['SERVER_NAME'];
             if ($_SERVER['SERVER_PORT'] != 80) {
                 $_configuration['vchamilo_name'] .= ':'.$_SERVER['SERVER_PORT'];

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

@@ -667,7 +667,7 @@ 
 
         // get the protocol free hostname
         Display::addFlash(
-            Display::return_message("Copying {$templateDir}/data/courses => $coursePath")
+            Display::return_message("copying {$templateDir}/data/courses => $coursePath")
         );
 
         copyDirTo(
@@ -677,7 +677,7 @@ 
         );
 
         Display::addFlash(
-            Display::return_message("Copying {$templateDir}/data/archive => $archivePath")
+            Display::return_message("copying {$templateDir}/data/archive => $archivePath")
         );
 
         copyDirTo(
@@ -687,7 +687,7 @@ 
         );
 
         Display::addFlash(
-            Display::return_message("Copying {$templateDir}/data/home => $homePath")
+            Display::return_message("copying {$templateDir}/data/home => $homePath")
         );
 
         copyDirTo(
@@ -698,7 +698,7 @@ 
 
         // Upload
         Display::addFlash(
-            Display::return_message("Copying {$templateDir}/data/upload => $uploadPath")
+            Display::return_message("copying {$templateDir}/data/upload => $uploadPath")
         );
 
         copyDirTo(
@@ -1029,13 +1029,13 @@ 
         $sitename = Database::escape_string($data->sitename);
         $institution = Database::escape_string($data->institution);
 
-        $sqls[] = "UPDATE {$settingstable} SET selected_value = '{$sitename}' 
+        $sqls[] = "update {$settingstable} SET selected_value = '{$sitename}' 
                    WHERE variable = 'siteName' AND category = 'Platform' ";
 
-        $sqls[] = "UPDATE {$settingstable} SET selected_value = '{$institution}' 
+        $sqls[] = "update {$settingstable} SET selected_value = '{$institution}' 
                    WHERE variable = 'institution' AND category = 'Platform' ";
 
-        $sqls[] = "UPDATE {$accessurltable} SET url = '{$data->root_web}' WHERE id = '1' ";
+        $sqls[] = "update {$accessurltable} SET url = '{$data->root_web}' WHERE id = '1' ";
 
         foreach ($sqls as $sql) {
             Database::query($sql);

plugin/google_maps/src/GoogleMapsPlugin.php

Doc Comments +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
 
     /**
      * Install the plugin
-     * @return void
+     * @return boolean
      */
     public function install()
     {
@@ -52,7 +52,7 @@ 
 
     /**
      * Uninstall the plugin
-     * @return void
+     * @return boolean
      */
     public function uninstall()
     {

src/Chamilo/CoreBundle/Entity/Repository/SequenceRepository.php

Doc Comments +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
      *
      * @param int $resourceId
      * @param int $type
-     * @return boolean
+     * @return boolean|null
      */
     public function deleteResource($resourceId, $type)
     {

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

Doc Comments +3 added lines, -1 removed lines

@@ -71,6 +71,8 @@ 
 
     /**
      * @inheritdoc
+     * @param string $toolbar
+     * @param string $prefix
      */
     public function __construct(
         $toolbar = null,
@@ -253,7 +255,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public function getNewPageBlock()
     {

plugin/courseblock/CourseBlockPlugin.php

Indentation +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 
     // When creating a new course this settings are added to the course
     public $course_settings = array(
-         array(
+            array(
             'name' => 'course_block_pre_footer',
             'type' => 'textarea'
         ),