chamilo / chamilo-lms

main/admin/filler.php

Indentation +2 added lines, -2 removed lines

@@ -53,8 +53,8 @@
     $result .= '<table>';
     foreach ($output as $line) {
         $result .= '<tr>';
-	    $result .= '<td class="filler-report-data-init">'.$line['line-init'].' </td><td class="filler-report-data">'.$line['line-info'].'</td>';
-	    $result .= '</tr>';
+        $result .= '<td class="filler-report-data-init">'.$line['line-init'].' </td><td class="filler-report-data">'.$line['line-info'].'</td>';
+        $result .= '</tr>';
     }
     $result .= '</table>';
     $result .= '</div>';

main/inc/lib/webservices/Rest.php

Doc Comments -1 removed lines

@@ -318,7 +318,6 @@
     }
 
     /**
-     * @param int $courseId
      * @return array
      * @throws Exception
      */

Braces +2 added lines, -1 removed lines

@@ -809,7 +809,8 @@
      * @param string $encoded
      * @return array
      */
-    public static function decodeParams($encoded) {
+    public static function decodeParams($encoded)
+    {
         $decoded = str_replace(['-', '_', '.'], ['+', '/', '='], $encoded);
         $mod4 = strlen($decoded) % 4;
 

src/Chamilo/CoreBundle/Composer/ScriptHandler.php

Doc Comments +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function getFoldersToDelete()
     {
@@ -95,7 +95,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function getFilesToDelete()
     {

plugin/ims_lti/ImsLtiPlugin.php

Doc Comments +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
     /**
      * Get the class instance
      * @staticvar MsiLtiPlugin $result
-     * @return MsiLtiPlugin
+     * @return ImsLtiPlugin
      */
     public static function create()
     {

plugin/ims_lti/ImsLtiTool.php

Doc Comments +3 added lines

@@ -20,6 +20,9 @@
         return $this->id;
     }
 
+    /**
+     * @return string
+     */
     public function getName()
     {
         return $this->name;

main/gradebook/lib/be/studentpublicationlink.class.php

Indentation +1 added lines, -1 removed lines

@@ -195,7 +195,7 @@
         $parentId = !$assignment ? 0 : $assignment->getId();
 
         if (empty($session)) {
-           $dql = 'SELECT a FROM ChamiloCourseBundle:CStudentPublication a
+            $dql = 'SELECT a FROM ChamiloCourseBundle:CStudentPublication a
                    WHERE
                         a.cId = :course AND
                         a.active = :active AND

main/notebook/index.php

Indentation +5 added lines, -5 removed lines

@@ -38,12 +38,12 @@ 
 
 // Tool name
 if ($action === 'addnote') {
-	$tool = 'NoteAddNew';
-	$interbreadcrumb[] = array('url' => 'index.php?'.api_get_cidreq(), 'name' => get_lang('ToolNotebook'));
+    $tool = 'NoteAddNew';
+    $interbreadcrumb[] = array('url' => 'index.php?'.api_get_cidreq(), 'name' => get_lang('ToolNotebook'));
 }
 if ($action === 'editnote') {
-	$tool = 'ModifyNote';
-	$interbreadcrumb[] = array('url' => 'index.php?'.api_get_cidreq(), 'name' => get_lang('ToolNotebook'));
+    $tool = 'ModifyNote';
+    $interbreadcrumb[] = array('url' => 'index.php?'.api_get_cidreq(), 'name' => get_lang('ToolNotebook'));
 }
 
 // Displaying the header
@@ -63,7 +63,7 @@ 
         exit;
     }
 
-	$_SESSION['notebook_view'] = 'creation_date';
+    $_SESSION['notebook_view'] = 'creation_date';
 
     $form = new FormValidator(
         'note',

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'];

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

@@ -216,7 +216,7 @@ 
                     ];
                     $this->lp_view_id = Database::insert($lp_table, $params);
                     if (!empty($this->lp_view_id)) {
-                        $sql = "UPDATE $lp_table SET id = iid WHERE iid = ".$this->lp_view_id;
+                        $sql = "update $lp_table SET id = iid WHERE iid = ".$this->lp_view_id;
                         Database::query($sql);
                     }
                 }
@@ -379,7 +379,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);
                                         }
@@ -611,16 +611,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);
@@ -705,7 +705,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);
@@ -852,7 +852,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.
@@ -1201,19 +1201,19 @@ 
         }
         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 parent_item_id = $parent AND display_order > $display";
         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.
@@ -1424,7 +1424,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)."',
@@ -1821,12 +1821,12 @@ 
             $index = 0;
             // Loop through all ordered items and stop at the first item that is
             // not a directory *and* that has not been completed yet.
-            while (!empty($this->ordered_items[$index]) AND
-                is_a($this->items[$this->ordered_items[$index]], 'learnpathItem') AND
+            while (!empty($this->ordered_items[$index]) and
+                is_a($this->items[$this->ordered_items[$index]], 'learnpathItem') and
                 (
-                    $this->items[$this->ordered_items[$index]]->get_type() == 'dir' OR
+                    $this->items[$this->ordered_items[$index]]->get_type() == 'dir' or
                     $this->items[$this->ordered_items[$index]]->is_done() === true
-                ) AND $index < $this->max_ordered_items) {
+                ) and $index < $this->max_ordered_items) {
                 $index++;
             }
             $this->last = $this->current;
@@ -2012,7 +2012,7 @@ 
         if ($this->debug > 2) {
             error_log('New LP - Now looking at ordered_items['.($index).'] - type is '.$this->items[$this->ordered_items[$index]]->type, 0);
         }
-        while (!empty ($this->ordered_items[$index]) AND ($this->items[$this->ordered_items[$index]]->get_type() == 'dir') AND $index < $this->max_ordered_items) {
+        while (!empty ($this->ordered_items[$index]) and ($this->items[$this->ordered_items[$index]]->get_type() == 'dir') and $index < $this->max_ordered_items) {
             $index++;
             if ($index == $this->max_ordered_items) {
                 if ($this->items[$this->ordered_items[$index]]->get_type() == 'dir') {
@@ -3584,7 +3584,7 @@ 
         }
         $search = '';
         // Use $attempt_num to enable multi-views management (disabled so far).
-        if ($attempt_num != 0 AND intval(strval($attempt_num)) == $attempt_num) {
+        if ($attempt_num != 0 and intval(strval($attempt_num)) == $attempt_num) {
             $search = 'AND view_count = '.$attempt_num;
         }
         // When missing $attempt_num, search for a unique lp_view record for this lp and user.
@@ -3613,7 +3613,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);
         }
 
@@ -3751,7 +3751,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) {
@@ -3761,7 +3761,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
@@ -3774,7 +3774,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
@@ -3786,7 +3786,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);
@@ -3814,13 +3814,13 @@ 
                     $next_next = $row2['next_item_id'];
                     // Update previous item (switch with current).
                     if ($previous != 0) {
-                        $sql_upd2 = "UPDATE $tbl_lp_item SET next_item_id = $next
+                        $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
                                      WHERE c_id = ".$course_id." AND id = $id";
                         Database::query($sql_upd2);
@@ -3828,7 +3828,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
                                      WHERE c_id = ".$course_id." AND id = $next";
                         Database::query($sql_upd2);
@@ -3836,7 +3836,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);
@@ -3884,7 +3884,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);
                 }
@@ -3897,10 +3897,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);
             }
@@ -3946,7 +3946,7 @@ 
                 $max = $i;
                 if ($row['display_order'] != $i) { // If we find a gap in the order, we need to fix it.
                     $need_fix = true;
-                    $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);
                 }
@@ -3959,10 +3959,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);
             }
@@ -4185,11 +4185,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',
@@ -4245,7 +4245,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;
@@ -4358,7 +4358,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
@@ -4376,7 +4376,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
@@ -4449,7 +4449,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;
@@ -4474,7 +4474,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;
@@ -4499,7 +4499,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) {
@@ -4526,7 +4526,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) {
@@ -4539,7 +4539,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)";
@@ -4642,7 +4642,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);
@@ -4667,7 +4667,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) {
@@ -4691,7 +4691,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);
@@ -4716,7 +4716,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) {
@@ -4744,7 +4744,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);
@@ -4770,7 +4770,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) {
@@ -4807,7 +4807,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'";
 
@@ -4900,7 +4900,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);
@@ -5054,7 +5054,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;
@@ -5092,7 +5092,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;
@@ -5129,7 +5129,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);
                 }
@@ -5161,7 +5161,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;
@@ -5233,7 +5233,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();
@@ -5298,7 +5298,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;
@@ -5333,7 +5333,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;
@@ -10294,7 +10294,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());
         }
@@ -10602,12 +10602,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);
     }
@@ -10633,12 +10633,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
@@ -10904,7 +10904,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);
 
@@ -10935,7 +10935,7 @@ 
         $this->subscribeUsers = intval($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);
 
@@ -11495,7 +11495,7 @@ 
         $this->accumulateScormTime = intval($value);
         $lp_table = Database::get_course_table(TABLE_LP_MAIN);
         $lp_id = $this->get_id();
-        $sql = "UPDATE $lp_table SET accumulate_scorm_time = ".$this->accumulateScormTime."
+        $sql = "update $lp_table SET accumulate_scorm_time = ".$this->accumulateScormTime."
                 WHERE c_id = ".$this->course_int_id." AND id = $lp_id";
         Database::query($sql);
 

Braces +78 added lines, -51 removed lines

@@ -3885,7 +3885,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);
@@ -3896,9 +3897,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);
@@ -3946,7 +3949,8 @@ 
             $i = 1;
             while ($row = Database::fetch_array($res)) {
                 $max = $i;
-                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.
                     $need_fix = true;
                     $sql_u = "UPDATE $lp_table SET display_order = $i
                               WHERE c_id = ".$courseId." AND id = ".$row['id'];
@@ -3958,9 +3962,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);
@@ -4367,8 +4373,7 @@ 
     public static function categoryIsVisibleForStudent(
         CLpCategory $category,
         User $user
-    )
-    {
+    ) {
         $isAllowedToEdit = api_is_allowed_to_edit(null, true);
 
         if ($isAllowedToEdit) {
@@ -4397,8 +4402,7 @@ 
     public static function categoryIsPusblished(
         CLpCategory $category,
         $courseId
-    )
-    {
+    ) {
         $link = self::getCategoryLinkForTool($category->getId());
         $em = Database::getManager();
 
@@ -4701,8 +4705,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();
@@ -4767,8 +4772,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;
@@ -4781,8 +4787,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';
@@ -4972,8 +4979,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);
@@ -6482,10 +6490,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;
@@ -7183,8 +7193,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);
@@ -7201,11 +7212,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>';
             }
@@ -7232,9 +7245,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'];
                     }
                 }
@@ -7438,9 +7453,11 @@ 
             $arrHide = array();
             for ($i = 0; $i < count($arrLP); $i++) {
                 if ($arrLP[$i]['id'] != $id && $arrLP[$i]['item_type'] != 'dir') {
-                    if (isset($extra_info['previous_item_id']) && $extra_info['previous_item_id'] == $arrLP[$i]['id'])
-                        $s_selected_position = $arrLP[$i]['id'];
-                    elseif ($action == 'add') $s_selected_position = 0;
+                    if (isset($extra_info['previous_item_id']) && $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'];
                 }
             }
@@ -7637,9 +7654,11 @@ 
 
             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'];
 
                 }
@@ -7673,8 +7692,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']);
         }
 
@@ -8195,9 +8213,11 @@ 
             $arrHide = array();
             for ($i = 0; $i < count($arrLP); $i++) {
                 if ($arrLP[$i]['id'] != $id && $arrLP[$i]['item_type'] != 'dir' && $arrLP[$i]['item_type'] !== TOOL_LP_FINAL_ITEM) {
-                    if (isset($extra_info['previous_item_id']) && $extra_info['previous_item_id'] == $arrLP[$i]['id'])
+                    if (isset($extra_info['previous_item_id']) && $extra_info['previous_item_id'] == $arrLP[$i]['id']) {
+                                            $s_selected_position = $arrLP[$i]['id'];
+                    } elseif ($action == 'add') {
                         $s_selected_position = $arrLP[$i]['id'];
-                    elseif ($action == 'add') $s_selected_position = $arrLP[$i]['id'];
+                    }
 
                     $arrHide[$arrLP[$i]['id']]['value'] = $arrLP[$i]['title'];
 
@@ -8496,9 +8516,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'];
 
                 }
@@ -8692,9 +8714,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'];
 
                 }
@@ -9844,10 +9868,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;
@@ -10281,7 +10306,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;
                                                 }
 
@@ -11994,7 +12020,8 @@ 
 }
 
 if (!function_exists('trim_value')) {
-    function trim_value(& $value) {
+    function trim_value(& $value)
+    {
         $value = trim($value);
     }
 }

Doc Comments +23 added lines, -19 removed lines

@@ -478,7 +478,7 @@ 
      * @param int $max_time_allowed
      * @param int $userId
      *
-     * @return int
+     * @return false|string
      */
     public function add_item(
         $parent,
@@ -1026,7 +1026,7 @@ 
      * @param	array $courseInfo
      * @param	integer	Learnpath ID
      * @param	string	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')
     {
@@ -1166,7 +1166,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')
@@ -1248,7 +1248,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,
@@ -1769,7 +1769,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()
     {
@@ -2276,6 +2276,7 @@ 
      * @param int $studentId
      * @param int $prerequisite
      * @param array $courseInfo
+     * @param integer $sessionId
      * @return bool
      *
      */
@@ -2310,6 +2311,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(
@@ -2638,7 +2640,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')
     {
@@ -2785,6 +2787,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)
@@ -2970,7 +2974,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()
     {
@@ -3106,7 +3110,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function getChapterTypes()
     {
@@ -4006,7 +4010,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)
     {
@@ -4169,7 +4173,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')
     {
@@ -4431,7 +4435,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()
     {
@@ -5005,7 +5009,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)
@@ -5417,6 +5421,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>
      **/
@@ -5459,7 +5464,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()
@@ -6784,7 +6789,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()
     {
@@ -10540,7 +10545,7 @@ 
 
     /**
      * @param int $lp_id
-     * @return bool
+     * @return false|null
      */
     public function scorm_export_to_pdf($lp_id)
     {
@@ -11088,7 +11093,7 @@ 
 
     /**
      * @param int $itemId
-     * @return learnpathItem|false
+     * @return string
      */
     public function getItem($itemId)
     {
@@ -11137,7 +11142,6 @@ 
     /**
      * Set whether this is a learning path with the possibility to subscribe
      * users or not
-     * @param int $subscribeUsers (0 = false, 1 = true)
      * @return bool
      */
     public function setSubscribeUsers($value)
@@ -11263,7 +11267,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()
     {
@@ -11480,7 +11484,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)
     {

main/dropbox/dropbox_init.inc.php

Indentation +5 added lines, -5 removed lines

@@ -216,7 +216,7 @@ 
             $javascript .= "'".$dropbox_person->sentWork[$i]->title."'";
         }
     }
-	$javascript .= ");
+    $javascript .= ");
 
 		function checkfile(str)
 		{
@@ -326,15 +326,15 @@ 
         'url' => api_get_path(WEB_CODE_PATH).'dropbox/index.php?'.api_get_cidreq(),
         'name' => get_lang('Dropbox', ''),
     );
-	$nameTools = get_lang('ReceivedFiles');
+    $nameTools = get_lang('ReceivedFiles');
 
-	if ($action == 'addreceivedcategory') {
+    if ($action == 'addreceivedcategory') {
         $interbreadcrumb[] = array(
             'url' => api_get_path(WEB_CODE_PATH).'dropbox/index.php?view=received&'.api_get_cidreq(),
             'name' => get_lang('ReceivedFiles'),
         );
-		$nameTools = get_lang('AddNewCategory');
-	}
+        $nameTools = get_lang('AddNewCategory');
+    }
 }
 
 if ($view == 'sent' || empty($view)) {