chamilo / chamilo-lms

main/work/work.lib.php

Doc Comments +14 added lines, -12 removed lines

@@ -729,7 +729,7 @@ 
  * @author Bert Vanderkimpen
  * @author Yannick Warnier <ywarnier@beeznest.org> Adaptation for work tool
  * @param  string $base_work_dir Base work dir (.../work)
- * @param  string $desiredDirName complete path of the desired name
+ * @param  string $desired_dir_name complete path of the desired name
  *
  * @return  string actual directory name if it succeeds, boolean false otherwise
  */
@@ -751,7 +751,7 @@ 
 /**
  * Delete a work-tool directory
  * @param   int  $id work directory id to delete
- * @return  integer -1 on error
+ * @return  boolean|null -1 on error
  */
 function deleteDirWork($id)
 {
@@ -955,7 +955,7 @@ 
  * Update the url of a dir in the student_publication table
  * @param  array $work_data work original data
  * @param  string $newPath Example: "folder1"
- * @return bool
+ * @return boolean|null
  */
 function updateDirName($work_data, $newPath)
 {
@@ -1014,6 +1014,7 @@ 
 /**
  * Transform an all directory structure (only directories) in an array
  * @param   string path of the directory
+ * @param string $directory
  * @return  array the directory structure into an array
  * @author  Julio Montoya
  * @version April 2008
@@ -1087,7 +1088,7 @@ 
  * @param   string the path of the directory
  * @param   boolean true if we want the total quantity of files
  * include in others child directories, false only  files in the directory
- * @return  array the first element is an integer with the number of files
+ * @return  integer[] the first element is an integer with the number of files
  * in the folder, the second element is the number of directories
  * @author  Julio Montoya
  * @version April 2008
@@ -2898,7 +2899,7 @@ 
  * @param int $userId
  * @param int $workId
  * @param int $courseId
- * @return bool
+ * @return boolean|null
  */
 function allowOnlySubscribedUser($userId, $workId, $courseId)
 {
@@ -3175,6 +3176,7 @@ 
  * @param array $parentInfo
  * @param array $courseInfo
  * @param int $sessionId
+ * @param integer $userId
  * @return int
  */
 function getLastWorkStudentFromParentByUser(
@@ -3326,7 +3328,7 @@ 
  * @param array $parentWork
  * @param array $work
  * @param array $data
- * @return int
+ * @return false|null
  */
 function addWorkComment($courseInfo, $userId, $parentWork, $work, $data)
 {
@@ -4616,7 +4618,7 @@ 
 }
 
 /**
- * @return array
+ * @return string[]
  */
 function getUploadDocumentType()
 {
@@ -5004,7 +5006,7 @@ 
  * @param int Session ID
  * @param $correction
  *
- * @return array|bool
+ * @return boolean
  */
 function getFileContents($id, $course_info, $sessionId = 0, $correction = false)
 {
@@ -5133,7 +5135,7 @@ 
  * @param int $userId
  * @param array $courseInfo
  * @param string $format
- * @return bool
+ * @return false|null
  */
 function exportAllWork($userId, $courseInfo, $format = 'pdf')
 {
@@ -5179,7 +5181,7 @@ 
  * @param array $courseInfo
  * @param int $sessionId
  * @param string $format
- * @return bool
+ * @return false|null
  */
 function exportAllStudentWorkFromPublication(
     $workId,
@@ -5313,7 +5315,7 @@ 
  * Downloads all user files per user
  * @param int $userId
  * @param array $courseInfo
- * @return bool
+ * @return false|null
  */
 function downloadAllFilesPerUser($userId, $courseInfo)
 {
@@ -5428,7 +5430,7 @@ 
 /**
  * @param array $courseInfo
  * @param int $workId
- * @return bool
+ * @return boolean|null
  */
 function protectWork($courseInfo, $workId)
 {

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

@@ -830,11 +830,11 @@ 
             }
 
             // Deleting all contents inside the folder
-            $sql = "UPDATE $table SET active = 2
+            $sql = "update $table SET active = 2
                     WHERE c_id = $course_id AND filetype = 'folder' AND id = $id";
             Database::query($sql);
 
-            $sql = "UPDATE $table SET active = 2
+            $sql = "update $table SET active = 2
                     WHERE c_id = $course_id AND parent_id = $id";
             Database::query($sql);
 
@@ -941,7 +941,7 @@ 
         $new_url = $new_path.$filename;
         $new_url = Database::escape_string($new_url);
 
-        $sql = "UPDATE $table SET
+        $sql = "update $table SET
                    url = '$new_url',
                    parent_id = '$parent_id'
                 WHERE c_id = $course_id AND id = $id";
@@ -974,7 +974,7 @@ 
 
     if (!empty($newPath)) {
         $table = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
-        $sql = "UPDATE $table SET
+        $sql = "update $table SET
                     title = '".$originalNewPath."'
                 WHERE
                     c_id = $course_id AND
@@ -3366,7 +3366,7 @@ 
         Display::addFlash(
             Display::return_message(get_lang('CommentAdded'))
         );
-        $sql = "UPDATE $commentTable SET id = iid WHERE iid = $commentId";
+        $sql = "update $commentTable SET id = iid WHERE iid = $commentId";
         Database::query($sql);
     }
 
@@ -3901,12 +3901,12 @@ 
         $workId = Database::insert($work_table, $params);
 
         if ($workId) {
-            $sql = "UPDATE $work_table SET id = iid WHERE iid = $workId ";
+            $sql = "update $work_table SET id = iid WHERE iid = $workId ";
             Database::query($sql);
 
             if (array_key_exists('filename', $workInfo) && !empty($filename)) {
                 $filename = Database::escape_string($filename);
-                $sql = "UPDATE $work_table SET
+                $sql = "update $work_table SET
                             filename = '$filename'
                         WHERE iid = $workId";
                 Database::query($sql);
@@ -3914,7 +3914,7 @@ 
 
             if (array_key_exists('document_id', $workInfo)) {
                 $documentId = isset($values['document_id']) ? intval($values['document_id']) : 0;
-                $sql = "UPDATE $work_table SET
+                $sql = "update $work_table SET
                             document_id = '$documentId'
                         WHERE iid = $workId";
                 Database::query($sql);
@@ -4286,19 +4286,19 @@ 
         $my_last_id = Database::insert_id();
 
         if ($my_last_id) {
-            $sql = "UPDATE $table SET
+            $sql = "update $table SET
                         id = iid
                     WHERE iid = $my_last_id";
             Database::query($sql);
 
-            $sql = "UPDATE $workTable SET
+            $sql = "update $workTable SET
                         has_properties  = $my_last_id,
                         view_properties = 1
                     WHERE c_id = $course_id AND id = $workId";
             Database::query($sql);
         }
     } else {
-        $sql = "UPDATE $table SET
+        $sql = "update $table SET
                     $expiryDateCondition
                     $endOnCondition
                     add_to_calendar  = $agendaId,
@@ -4461,7 +4461,7 @@ 
                 }
             } // end of considered_working_time check section
 
-            $sql = "UPDATE $work_table SET active = 2
+            $sql = "update $work_table SET active = 2
                     WHERE c_id = $course_id AND id = $item_id";
             Database::query($sql);
             $sql = "DELETE FROM $TSTDPUBASG
@@ -4659,7 +4659,7 @@ 
         return false;
     }
 
-    $query = "UPDATE $main_course_table
+    $query = "update $main_course_table
               SET show_score = '$showScore'
               WHERE id = $courseId";
     Database::query($query);
@@ -4708,7 +4708,7 @@ 
     $course_id = $course_info['real_id'];
     $item_id = intval($item_id);
 
-    $sql = "UPDATE $work_table SET accepted = 1
+    $sql = "update $work_table SET accepted = 1
             WHERE c_id = $course_id AND id = $item_id";
     Database::query($sql);
     api_item_property_update($course_info, 'work', $item_id, 'visible', api_get_user_id());
@@ -5497,7 +5497,7 @@ 
     if (isset($work['url_correction']) && !empty($work['url_correction']) && isset($work['iid'])) {
         $id = $work['iid'];
         $table = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
-        $sql = "UPDATE $table SET
+        $sql = "update $table SET
                     url_correction = '',
                     title_correction = ''
                 WHERE iid = $id";

Braces +24 added lines, -12 removed lines

@@ -442,7 +442,8 @@ 
     $sessionId,
     $userId = null,
     $onlyUserList = array()
-) {
+)
+{
     $work_table = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
     $user_table = Database::get_main_table(TABLE_MAIN_USER);
 
@@ -1369,7 +1370,8 @@ 
     $direction,
     $where_condition,
     $getCount = false
-) {
+)
+{
     $workTable = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
     $workTableAssignment = Database::get_course_table(TABLE_STUDENT_PUBLICATION_ASSIGNMENT);
     $courseInfo = api_get_course_info();
@@ -1528,7 +1530,8 @@ 
     $direction,
     $where_condition,
     $getCount = false
-) {
+)
+{
     $workTable = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
     $workTableAssignment = Database::get_course_table(TABLE_STUDENT_PUBLICATION_ASSIGNMENT);
 
@@ -1715,7 +1718,8 @@ 
     $studentId = null,
     $whereCondition = '',
     $getCount = false
-) {
+)
+{
     if ($getCount) {
         $select1 = " SELECT count(u.user_id) as count ";
         $select2 = " SELECT count(u.user_id) as count ";
@@ -1916,7 +1920,8 @@ 
     $getCount = false,
     $courseId = 0,
     $sessionId = 0
-) {
+)
+{
     $work_table = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
     $user_table = Database::get_main_table(TABLE_MAIN_USER);
 
@@ -2870,7 +2875,8 @@ 
     $groupId = null,
     $sessionId = null,
     $getCount = false
-) {
+)
+{
     $usersInWork = null;
     $usersInCourse = null;
 
@@ -3103,7 +3109,8 @@ 
     $parentId,
     $courseInfo = array(),
     $sessionId = 0
-) {
+)
+{
     if (empty($courseInfo)) {
         $courseInfo = api_get_course_info();
     }
@@ -3147,7 +3154,8 @@ 
     $parentId,
     $courseInfo = array(),
     $sessionId = 0
-) {
+)
+{
     if (empty($courseInfo)) {
         $courseInfo = api_get_course_info();
     }
@@ -3196,7 +3204,8 @@ 
     $parentInfo,
     $courseInfo = array(),
     $sessionId = 0
-) {
+)
+{
     if (empty($courseInfo)) {
         $courseInfo = api_get_course_info();
     }
@@ -3813,7 +3822,8 @@ 
     $file = [],
     $checkDuplicated = false,
     $showFlashMessage = true
-) {
+)
+{
     $work_table = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
 
     $courseId = $courseInfo['real_id'];
@@ -4921,7 +4931,8 @@ 
     $sidx,
     $sord,
     $getCount = false
-) {
+)
+{
     $my_folder_data = get_work_data_by_id($workId);
     $workParents = array();
     if (empty($my_folder_data)) {
@@ -5202,7 +5213,8 @@ 
     $courseInfo,
     $sessionId,
     $format = 'pdf'
-) {
+)
+{
     if (empty($courseInfo)) {
         return false;
     }

Spacing +1 added lines, -1 removed lines

@@ -3308,7 +3308,7 @@
         $label = 'warning';
     }
 
-    $scoreBasedInModel = ExerciseLib::convertScoreToModel($relativeScore*100);
+    $scoreBasedInModel = ExerciseLib::convertScoreToModel($relativeScore * 100);
     if (empty($scoreBasedInModel)) {
         $finalScore = api_number_format($score, 1).' / '.$weight;
 

main/user/session_list.php

Indentation +4 added lines, -4 removed lines

@@ -55,13 +55,13 @@ 
     array('name'=>'access_end_date', 'index'=>'access_end_date', 'width'=>'40', 'align'=>'left', 'search' => 'true'),
     array('name'=>'coach_name', 'index'=>'coach_name', 'width'=>'80', 'align'=>'left', 'search' => 'false'),
     array('name'=>'status', 'index'=>'session_active', 'width'=>'40', 'align'=>'left', 'search' => 'true', 'stype'=>'select',
-      //for the bottom bar
+        //for the bottom bar
         'searchoptions' => array(
             'defaultValue' => '1',
             'value' => '1:'.get_lang('Active').';0:'.get_lang('Inactive')
         ),
-      //for the top bar
-      'editoptions' => array('value' => ':'.get_lang('All').';1:'.get_lang('Active').';0:'.get_lang('Inactive'))),
+        //for the top bar
+        'editoptions' => array('value' => ':'.get_lang('All').';1:'.get_lang('Active').';0:'.get_lang('Inactive'))),
     array('name'=>'visibility', 'index'=>'visibility', 'width'=>'40', 'align'=>'left', 'search' => 'false'),
     array('name'=>'actions', 'index'=>'actions', 'width'=>'100', 'align'=>'left', 'formatter'=>'action_formatter', 'sortable'=>'false', 'search' => 'false')
 );
@@ -76,7 +76,7 @@ 
 //With this function we can add actions to the jgrid (edit, delete, etc)
 $action_links = 'function action_formatter(cellvalue, options, rowObject) {
      return \'&nbsp;<a href="add_users_to_session.php?page=session_list.php&id_session=\'+options.rowId+\'">'.Display::return_icon('user_subscribe_session.png', get_lang('SubscribeUsersToSession'), '', ICON_SIZE_SMALL).'</a>'.
-     '\';
+        '\';
 }';
 ?>
 <script>

src/Chamilo/CourseBundle/Component/CourseCopy/CourseRestorer.php

Doc Comments +4 added lines, -3 removed lines

@@ -1152,7 +1152,7 @@ 
     /**
      * Restore a forum-topic
      * @param false|string $forum_id
-     * @return int
+     * @return false|string
      */
     public function restore_topic($thread_id, $forum_id, $sessionId = 0)
     {
@@ -1210,7 +1210,8 @@ 
      * Restore a forum-post
      * @TODO Restore tree-structure of posts. For example: attachments to posts.
      * @param false|string $topic_id
-     * @return int
+     * @param false|string $forum_id
+     * @return false|string
      */
     public function restore_post($id, $topic_id, $forum_id, $sessionId = 0)
     {
@@ -3316,7 +3317,7 @@ 
     /**
      * Restore gradebook
      * @param int $sessionId
-     * @return  bool
+     * @return  false|null
      */
     public function restore_gradebook($sessionId = 0)
     {

Indentation +14 added lines, -14 removed lines

@@ -25,8 +25,8 @@ 
 class CourseRestorer
 {
     /**
-    * The course-object
-    */
+     * The course-object
+     */
     public $course;
     public $destination_course_info;
 
@@ -45,7 +45,7 @@ 
         'events',
         'forum_category',
         'forums',
-       // 'forum_topics',
+        // 'forum_topics',
         'glossary',
         'quizzes',
         'test_category',
@@ -2864,15 +2864,15 @@ 
     }
 
     /**
-    * copy all directory and sub directory
-    * @param string The path origin
-    * @param string The path destination
-    * @param boolean Option Overwrite
-    * @param string $source
-    * @param string $dest
-    * @return void
-    * @deprecated
-    */
+     * copy all directory and sub directory
+     * @param string The path origin
+     * @param string The path destination
+     * @param boolean Option Overwrite
+     * @param string $source
+     * @param string $dest
+     * @return void
+     * @deprecated
+     */
     public function allow_create_all_directory($source, $dest, $overwrite = false)
     {
         if (!is_dir($dest)) {
@@ -2885,7 +2885,7 @@ 
                 if ($file != '.' && $file != '..') {
                     $path = $source.'/'.$file;
                     if (is_file($path)) {
-                       /* if (!is_file($dest . '/' . $file) || $overwrite)
+                        /* if (!is_file($dest . '/' . $file) || $overwrite)
                         if (!@copy($path, $dest . '/' . $file)) {
                             echo '<font color="red">File ('.$path.') '.get_lang('NotHavePermission').'</font>';
                         }*/
@@ -3415,7 +3415,7 @@ 
                                         break;
                                 }
 
-                                 if ($this->course->has_resources($type) &&
+                                    if ($this->course->has_resources($type) &&
                                     isset($this->course->resources[$type][$itemId])
                                 ) {
                                     $item = $this->course->resources[$type][$itemId];

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

@@ -234,7 +234,7 @@ 
 
                                 $id = Database::insert($table, $params);
                                 if ($id) {
-                                    $sql = "UPDATE $table SET id = iid WHERE iid = $id";
+                                    $sql = "update $table SET id = iid WHERE iid = $id";
                                     Database::query($sql);
                                 }
                             }
@@ -465,7 +465,7 @@ 
                                     $document_id = Database::insert($table, $params);
 
                                     if ($document_id) {
-                                        $sql = "UPDATE $table SET id = iid WHERE iid = $document_id";
+                                        $sql = "update $table SET id = iid WHERE iid = $document_id";
                                         Database::query($sql);
                                     }
                                     $this->course->resources[RESOURCE_DOCUMENT][$id]->destination_id = $document_id;
@@ -692,7 +692,7 @@ 
                                     $document_id = Database::insert($table, $params);
 
                                     if ($document_id) {
-                                        $sql = "UPDATE $table SET id = iid WHERE iid = $document_id";
+                                        $sql = "update $table SET id = iid WHERE iid = $document_id";
                                         Database::query($sql);
                                     }
 
@@ -755,7 +755,7 @@ 
                                     $document_id = Database::insert($table, $params);
 
                                     if ($document_id) {
-                                        $sql = "UPDATE $table SET id = iid WHERE iid = $document_id";
+                                        $sql = "update $table SET id = iid WHERE iid = $document_id";
                                         Database::query($sql);
 
                                         $this->course->resources[RESOURCE_DOCUMENT][$id]->destination_id = $document_id;
@@ -821,7 +821,7 @@ 
                                 $document_id = Database::insert($table, $params);
 
                                 if ($document_id) {
-                                    $sql = "UPDATE $table SET id = iid WHERE iid = $document_id";
+                                    $sql = "update $table SET id = iid WHERE iid = $document_id";
                                     Database::query($sql);
                                 }
 
@@ -898,7 +898,7 @@ 
                         $document_id = Database::insert($table, $params);
 
                         if ($document_id) {
-                            $sql = "UPDATE $table SET id = iid WHERE iid = $document_id";
+                            $sql = "update $table SET id = iid WHERE iid = $document_id";
                             Database::query($sql);
                         }
 
@@ -1095,7 +1095,7 @@ 
                 $new_id = Database::insert($table_forum, $params);
 
                 if ($new_id) {
-                    $sql = "UPDATE $table_forum SET forum_id = iid WHERE iid = $new_id";
+                    $sql = "update $table_forum SET forum_id = iid WHERE iid = $new_id";
                     Database::query($sql);
                 }
 
@@ -1151,7 +1151,7 @@ 
                     $new_id = Database::insert($forum_cat_table, $params);
 
                     if ($new_id) {
-                        $sql = "UPDATE $forum_cat_table SET cat_id = iid WHERE iid = $new_id";
+                        $sql = "update $forum_cat_table SET cat_id = iid WHERE iid = $new_id";
                         Database::query($sql);
                     }
 
@@ -1192,7 +1192,7 @@ 
         $new_id = Database::insert($table, $params);
 
         if ($new_id) {
-            $sql = "UPDATE $table SET thread_id = iid WHERE iid = $new_id";
+            $sql = "update $table SET thread_id = iid WHERE iid = $new_id";
             Database::query($sql);
         }
 
@@ -1250,7 +1250,7 @@ 
         $new_id = Database::insert($table_post, $params);
 
         if ($new_id) {
-            $sql = "UPDATE $table_post SET post_id = iid WHERE iid = $new_id";
+            $sql = "update $table_post SET post_id = iid WHERE iid = $new_id";
             Database::query($sql);
         }
 
@@ -1310,7 +1310,7 @@ 
                 $id = Database::insert($link_table, $params);
 
                 if ($id) {
-                    $sql = "UPDATE $link_table SET id = iid WHERE iid = $id";
+                    $sql = "update $link_table SET id = iid WHERE iid = $id";
                     Database::query($sql);
 
                     api_item_property_update(
@@ -1362,7 +1362,7 @@ 
             $new_id = Database::insert($link_cat_table, $params);
 
             if ($new_id) {
-                $sql = "UPDATE $link_cat_table 
+                $sql = "update $link_cat_table 
                         SET id = iid 
                         WHERE iid = $new_id";
                 Database::query($sql);
@@ -1452,7 +1452,7 @@ 
                 $new_event_id = Database::insert($table, $params);
 
                 if ($new_event_id) {
-                    $sql = "UPDATE $table SET id = iid WHERE iid = $new_event_id";
+                    $sql = "update $table SET id = iid WHERE iid = $new_event_id";
                     Database::query($sql);
 
                     if (!isset($this->course->resources[RESOURCE_EVENT][$id])) {
@@ -1495,7 +1495,7 @@ 
                             ];
                             $id = Database::insert($table_attachment, $params);
                             if ($id) {
-                                $sql = "UPDATE $table_attachment SET id = iid WHERE iid = $id";
+                                $sql = "update $table_attachment SET id = iid WHERE iid = $id";
                                 Database::query($sql);
                             }
                         }
@@ -1525,7 +1525,7 @@ 
                             $id = Database::insert($table_attachment, $params);
 
                             if ($id) {
-                                $sql = "UPDATE $table_attachment SET id = iid WHERE iid = $id";
+                                $sql = "update $table_attachment SET id = iid WHERE iid = $id";
                                 Database::query($sql);
                             }
                         }
@@ -1572,7 +1572,7 @@ 
 
                 $id = Database::insert($table, $params);
                 if ($id) {
-                    $sql = "UPDATE $table SET id = iid WHERE iid = $id";
+                    $sql = "update $table SET id = iid WHERE iid = $id";
                     Database::query($sql);
 
                     if (!isset($this->course->resources[RESOURCE_COURSEDESCRIPTION][$id])) {
@@ -1617,7 +1617,7 @@ 
                 $new_announcement_id = Database::insert($table, $params);
 
                 if ($new_announcement_id) {
-                    $sql = "UPDATE $table SET id = iid WHERE iid = $new_announcement_id";
+                    $sql = "update $table SET id = iid WHERE iid = $new_announcement_id";
                     Database::query($sql);
 
                     if (!isset($this->course->resources[RESOURCE_ANNOUNCEMENT][$id])) {
@@ -1665,7 +1665,7 @@ 
                             $attachmentId = Database::insert($table_attachment, $params);
 
                             if ($attachmentId) {
-                                $sql = "UPDATE $table_attachment SET id = iid WHERE iid = $attachmentId";
+                                $sql = "update $table_attachment SET id = iid WHERE iid = $attachmentId";
                                 Database::query($sql);
                             }
                         }
@@ -1694,7 +1694,7 @@ 
                             $attachmentId = Database::insert($table_attachment, $params);
 
                             if ($attachmentId) {
-                                $sql = "UPDATE $table_attachment SET id = iid WHERE iid = $attachmentId";
+                                $sql = "update $table_attachment SET id = iid WHERE iid = $attachmentId";
                                 Database::query($sql);
                             }
                         }
@@ -1804,7 +1804,7 @@ 
                     $new_id = Database::insert($table_qui, $params);
 
                     if ($new_id) {
-                        $sql = "UPDATE $table_qui SET id = iid WHERE iid = $new_id";
+                        $sql = "update $table_qui SET id = iid WHERE iid = $new_id";
                         Database::query($sql);
                     }
 
@@ -1889,7 +1889,7 @@ 
             $new_id = Database::insert($table_que, $params);
 
             if ($new_id) {
-                $sql = "UPDATE $table_que SET id = iid WHERE iid = $new_id";
+                $sql = "update $table_que SET id = iid WHERE iid = $new_id";
                 Database::query($sql);
             }
 
@@ -1991,7 +1991,7 @@ 
                     $answerId = Database::insert($table_ans, $params);
 
                     if ($answerId) {
-                        $sql = "UPDATE $table_ans SET id = iid, id_auto = iid WHERE iid = $answerId";
+                        $sql = "update $table_ans SET id = iid, id_auto = iid WHERE iid = $answerId";
                         Database::query($sql);
                     }
 
@@ -2021,7 +2021,7 @@ 
                         $question_option_id = Database::insert($table_options, $item);
                         if ($question_option_id) {
                             $old_option_ids[$old_id] = $question_option_id;
-                            $sql = "UPDATE $table_options SET id = iid WHERE iid = $question_option_id";
+                            $sql = "update $table_options SET id = iid WHERE iid = $question_option_id";
                             Database::query($sql);
                         }
                     }
@@ -2069,7 +2069,7 @@ 
 
                             if ($question_option_id) {
                                 $new_options[$obj->obj->id] = $question_option_id;
-                                $sql = "UPDATE $table_options SET id = iid WHERE iid = $question_option_id";
+                                $sql = "update $table_options SET id = iid WHERE iid = $question_option_id";
                                 Database::query($sql);
                             }
                         }
@@ -2299,16 +2299,16 @@ 
                             $params['code'] = $survey_code;
                             $new_id = Database::insert($table_sur, $params);
                             if ($new_id) {
-                                $sql = "UPDATE $table_sur SET survey_id = iid WHERE iid = $new_id";
+                                $sql = "update $table_sur SET survey_id = iid WHERE iid = $new_id";
                                 Database::query($sql);
 
                                 $this->course->resources[RESOURCE_SURVEY][$id]->destination_id = $new_id;
                                 foreach ($survey->question_ids as $index => $question_id) {
                                     $qid = $this->restore_survey_question($question_id, $new_id);
-                                    $sql = "UPDATE $table_que SET survey_id = $new_id
+                                    $sql = "update $table_que SET survey_id = $new_id
                                             WHERE c_id = ".$this->destination_course_id." AND question_id = $qid";
                                     Database::query($sql);
-                                    $sql = "UPDATE $table_ans SET survey_id = $new_id
+                                    $sql = "update $table_ans SET survey_id = $new_id
                                             WHERE  c_id = ".$this->destination_course_id." AND  question_id = $qid";
                                     Database::query($sql);
                                 }
@@ -2335,7 +2335,7 @@ 
                             $new_id = Database::insert($table_sur, $params);
 
                             if ($new_id) {
-                                $sql = "UPDATE $table_sur SET survey_id = iid WHERE iid = $new_id";
+                                $sql = "update $table_sur SET survey_id = iid WHERE iid = $new_id";
                                 Database::query($sql);
 
                                 $this->course->resources[RESOURCE_SURVEY][$id]->destination_id = $new_id;
@@ -2344,10 +2344,10 @@ 
                                         $question_id,
                                         $new_id
                                     );
-                                    $sql = "UPDATE $table_que SET survey_id = $new_id
+                                    $sql = "update $table_que SET survey_id = $new_id
                                             WHERE c_id = ".$this->destination_course_id." AND question_id = $qid";
                                     Database::query($sql);
-                                    $sql = "UPDATE $table_ans SET survey_id = $new_id
+                                    $sql = "update $table_ans SET survey_id = $new_id
                                             WHERE c_id = ".$this->destination_course_id." AND question_id = $qid";
                                     Database::query($sql);
                                 }
@@ -2361,7 +2361,7 @@ 
                     $new_id = Database::insert($table_sur, $params);
 
                     if ($new_id) {
-                        $sql = "UPDATE $table_sur SET survey_id = iid WHERE iid = $new_id";
+                        $sql = "update $table_sur SET survey_id = iid WHERE iid = $new_id";
                         Database::query($sql);
 
                         $this->course->resources[RESOURCE_SURVEY][$id]->destination_id = $new_id;
@@ -2370,10 +2370,10 @@ 
                                 $question_id,
                                 $new_id
                             );
-                            $sql = "UPDATE $table_que SET survey_id = $new_id
+                            $sql = "update $table_que SET survey_id = $new_id
                                     WHERE c_id = ".$this->destination_course_id." AND question_id = $qid";
                             Database::query($sql);
-                            $sql = "UPDATE $table_ans SET survey_id = $new_id
+                            $sql = "update $table_ans SET survey_id = $new_id
                                     WHERE c_id = ".$this->destination_course_id." AND question_id = $qid";
                             Database::query($sql);
                         }
@@ -2445,7 +2445,7 @@ 
             $new_id = Database::insert($table_que, $params);
             if ($new_id) {
 
-                $sql = "UPDATE $table_que SET question_id = iid WHERE iid = $new_id";
+                $sql = "update $table_que SET question_id = iid WHERE iid = $new_id";
                 Database::query($sql);
 
                 foreach ($question->answers as $index => $answer) {
@@ -2468,7 +2468,7 @@ 
                     ];
                     $answerId = Database::insert($table_ans, $params);
                     if ($answerId) {
-                        $sql = "UPDATE $table_ans SET question_option_id = iid
+                        $sql = "update $table_ans SET question_option_id = iid
                                 WHERE iid = $answerId";
                         Database::query($sql);
                     }
@@ -2590,7 +2590,7 @@ 
                     // The following only makes sense if a new LP was
                     // created in the destination course
 
-                    $sql = "UPDATE $table_main SET id = iid WHERE iid = $new_lp_id";
+                    $sql = "update $table_main SET id = iid WHERE iid = $new_lp_id";
                     Database::query($sql);
 
                     if ($lp->visibility) {
@@ -2606,7 +2606,7 @@ 
                         ];
                         $insertId = Database::insert($table_tool, $params);
                         if ($insertId) {
-                            $sql = "UPDATE $table_tool SET id = iid WHERE iid = $insertId";
+                            $sql = "update $table_tool SET id = iid WHERE iid = $insertId";
                             Database::query($sql);
                         }
                     }
@@ -2685,7 +2685,7 @@ 
 
                         $new_item_id = Database::insert($table_item, $params);
 
-                        $sql = "UPDATE $table_item SET id = iid WHERE iid = $new_item_id";
+                        $sql = "update $table_item SET id = iid WHERE iid = $new_item_id";
                         Database::query($sql);
 
                         //save a link between old and new item IDs
@@ -2838,7 +2838,7 @@ 
                     $sub_folder['session_id'] = $sessionId ? $sessionId : null;
                     $new_item_id = Database::insert($item_property_table, $sub_folder);
                     if ($new_item_id) {
-                        $sql = "UPDATE $item_property_table SET id = iid WHERE iid = $new_item_id";
+                        $sql = "update $item_property_table SET id = iid WHERE iid = $new_item_id";
                         Database::query($sql);
                     }
                 }
@@ -2859,7 +2859,7 @@ 
                     $assignmentId = Database::insert($work_assignment_table, $assign);
 
                     if ($assignmentId) {
-                        $sql = "UPDATE $work_assignment_table SET id = iid WHERE iid = $assignmentId";
+                        $sql = "update $work_assignment_table SET id = iid WHERE iid = $assignmentId";
                         Database::query($sql);
                     }
                 }
@@ -2971,7 +2971,7 @@ 
                 $my_id = Database::insert($table_glossary, $params);
                 if ($my_id) {
 
-                    $sql = "UPDATE $table_glossary SET glossary_id = iid WHERE iid = $my_id";
+                    $sql = "update $table_glossary SET glossary_id = iid WHERE iid = $my_id";
                     Database::query($sql);
 
                     api_item_property_update(
@@ -3046,7 +3046,7 @@ 
                 $new_id = Database::insert($table_wiki, $params);
 
                 if ($new_id) {
-                    $sql = "UPDATE $table_wiki SET page_id = '$new_id', id = iid
+                    $sql = "update $table_wiki SET page_id = '$new_id', id = iid
                             WHERE c_id = ".$this->destination_course_id." AND iid = '$new_id'";
                     Database::query($sql);
 
@@ -3107,7 +3107,7 @@ 
 
                 if ($last_id) {
 
-                    $sql = "UPDATE $table_thematic SET id = iid WHERE iid = $last_id";
+                    $sql = "update $table_thematic SET id = iid WHERE iid = $last_id";
                     Database::query($sql);
 
                     api_item_property_update(
@@ -3132,7 +3132,7 @@ 
                         );
 
                         if ($my_id) {
-                            $sql = "UPDATE $table_thematic_advance SET id = iid WHERE iid = $my_id";
+                            $sql = "update $table_thematic_advance SET id = iid WHERE iid = $my_id";
                             Database::query($sql);
 
                             api_item_property_update(
@@ -3153,7 +3153,7 @@ 
                         $my_id = Database::insert($table_thematic_plan, $thematic_plan, false);
 
                         if ($my_id) {
-                            $sql = "UPDATE $table_thematic_plan SET id = iid WHERE iid = $my_id";
+                            $sql = "update $table_thematic_plan SET id = iid WHERE iid = $my_id";
                             Database::query($sql);
 
                             api_item_property_update(
@@ -3197,7 +3197,7 @@ 
                 $last_id = Database::insert($table_attendance, $obj->params);
 
                 if (is_numeric($last_id)) {
-                    $sql = "UPDATE $table_attendance SET id = iid WHERE iid = $last_id";
+                    $sql = "update $table_attendance SET id = iid WHERE iid = $last_id";
                     Database::query($sql);
 
                     $this->course->resources[RESOURCE_ATTENDANCE][$id]->destination_id = $last_id;
@@ -3221,7 +3221,7 @@ 
                             $attendance_calendar
                         );
 
-                        $sql = "UPDATE $table_attendance_calendar SET id = iid WHERE iid = $attendanceCalendarId";
+                        $sql = "update $table_attendance_calendar SET id = iid WHERE iid = $attendanceCalendarId";
                         Database::query($sql);
                     }
                 }

main/install/update-configuration.inc.php

Indentation +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
         if (stripos($line, '$_configuration[\'system_version\']') !== false) {
             $found_version = true;
             $line = '$_configuration[\'system_version\'] = \''.$GLOBALS['new_version'].'\';'."\r\n";
-       } elseif (stripos($line, '$_configuration[\'system_stable\']') !== false) {
+        } elseif (stripos($line, '$_configuration[\'system_stable\']') !== false) {
             $found_stable = true;
             $line = '$_configuration[\'system_stable\'] = '.($GLOBALS['new_version_stable'] ? 'true' : 'false').';'."\r\n";
         } elseif (stripos($line, '$_configuration[\'software_name\']') !== false) {

main/admin/ldap_form_add_users_group.php

Indentation +28 added lines, -28 removed lines

@@ -29,35 +29,35 @@
 echo '<input type="hidden" name="confirmed" value="yes">';
 echo '<table border="0" cellspacing="0" width="100%">';
 echo '<tr align="center" id="header3">'.
-		'<td width="15%"><input type="button" value="'.get_lang('AllSlashNone').'" onClick="checkAll();"></td>'.
-		'<td width="40%"><b>'.get_lang('Email').'</b></td>'.
-		($is_western_name_order
-			? '<td width="15%"><b>'.get_lang('FirstName').'</b></td>'.
-			'<td width="15%"><b>'.get_lang('Name').'</b></td>'
-			: '<td width="15%"><b>'.get_lang('Name').'</b></td>'.
-			'<td width="15%"><b>'.get_lang('FirstName').'</b></td>').
-		'<td width="15%"><b>'.get_lang('Login').'</b></td>'.
-	  '</tr>'."\n";
+        '<td width="15%"><input type="button" value="'.get_lang('AllSlashNone').'" onClick="checkAll();"></td>'.
+        '<td width="40%"><b>'.get_lang('Email').'</b></td>'.
+        ($is_western_name_order
+            ? '<td width="15%"><b>'.get_lang('FirstName').'</b></td>'.
+            '<td width="15%"><b>'.get_lang('Name').'</b></td>'
+            : '<td width="15%"><b>'.get_lang('Name').'</b></td>'.
+            '<td width="15%"><b>'.get_lang('FirstName').'</b></td>').
+        '<td width="15%"><b>'.get_lang('Login').'</b></td>'.
+        '</tr>'."\n";
 while (list ($key, $val) = each($nom_form)) {
-	$nbre = $nbre + 1;
-	if ($nbre & 1) $ndiv = 2; else $ndiv = 3;
-	echo '<tr align="center" id="header'.$ndiv.'">';
-	echo '<td><input type="checkbox" name="checkboxes[]" value="'.$key.'" checked="checked"></td>';
-	echo '<td>'.$email_form[$key].'<input type="hidden" name="email_form['.$key.']" size="40" value="'.$email_form[$key].'"></td>';
-	if ($is_western_name_order) {
-		echo '<td>'.$prenom_form[$key].'<input type="hidden" name="prenom_form['.$key.']" size="20" value="'.$prenom_form[$key].'"></td>';
-		echo '<td>'.$nom_form[$key].'<input type="hidden" name="nom_form['.$key.']" size="20" value="'.$nom_form[$key].'"></td>';
-	} else {
-		echo '<td>'.$nom_form[$key].'<input type="hidden" name="nom_form['.$key.']" size="20" value="'.$nom_form[$key].'"></td>';
-		echo '<td>'.$prenom_form[$key].'<input type="hidden" name="prenom_form['.$key.']" size="20" value="'.$prenom_form[$key].'"></td>';
-	}
-	echo '<td>'.$username_form[$key].'<input type="hidden" name="username_form['.$key.']" size="10" value="'.$username_form[$key].'">';
-	echo '<input type="hidden" name="tutor_form['.$key.']" value="0">';
-	echo '<input type="hidden" name="admin_form['.$key.']" value="1">';
-	echo '<input type="hidden" name="password_form['.$key.']"  value="'.$password_form[$key].'">';
-	echo '<input type="hidden" name="statut['.$key.']"  value="'.$statut.'">';
-	echo '</td>';
-	echo '</tr>';
+    $nbre = $nbre + 1;
+    if ($nbre & 1) $ndiv = 2; else $ndiv = 3;
+    echo '<tr align="center" id="header'.$ndiv.'">';
+    echo '<td><input type="checkbox" name="checkboxes[]" value="'.$key.'" checked="checked"></td>';
+    echo '<td>'.$email_form[$key].'<input type="hidden" name="email_form['.$key.']" size="40" value="'.$email_form[$key].'"></td>';
+    if ($is_western_name_order) {
+        echo '<td>'.$prenom_form[$key].'<input type="hidden" name="prenom_form['.$key.']" size="20" value="'.$prenom_form[$key].'"></td>';
+        echo '<td>'.$nom_form[$key].'<input type="hidden" name="nom_form['.$key.']" size="20" value="'.$nom_form[$key].'"></td>';
+    } else {
+        echo '<td>'.$nom_form[$key].'<input type="hidden" name="nom_form['.$key.']" size="20" value="'.$nom_form[$key].'"></td>';
+        echo '<td>'.$prenom_form[$key].'<input type="hidden" name="prenom_form['.$key.']" size="20" value="'.$prenom_form[$key].'"></td>';
+    }
+    echo '<td>'.$username_form[$key].'<input type="hidden" name="username_form['.$key.']" size="10" value="'.$username_form[$key].'">';
+    echo '<input type="hidden" name="tutor_form['.$key.']" value="0">';
+    echo '<input type="hidden" name="admin_form['.$key.']" value="1">';
+    echo '<input type="hidden" name="password_form['.$key.']"  value="'.$password_form[$key].'">';
+    echo '<input type="hidden" name="statut['.$key.']"  value="'.$statut.'">';
+    echo '</td>';
+    echo '</tr>';
 }
 echo '</table>';
 echo '<br />';

Braces +5 added lines, -1 removed lines

@@ -40,7 +40,11 @@
 	  '</tr>'."\n";
 while (list ($key, $val) = each($nom_form)) {
 	$nbre = $nbre + 1;
-	if ($nbre & 1) $ndiv = 2; else $ndiv = 3;
+	if ($nbre & 1) {
+	    $ndiv = 2;
+	} else {
+	    $ndiv = 3;
+	}
 	echo '<tr align="center" id="header'.$ndiv.'">';
 	echo '<td><input type="checkbox" name="checkboxes[]" value="'.$key.'" checked="checked"></td>';
 	echo '<td>'.$email_form[$key].'<input type="hidden" name="email_form['.$key.']" size="40" value="'.$email_form[$key].'"></td>';

main/inc/lib/add_course.lib.inc.php

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

@@ -872,7 +872,7 @@ 
                                     $image_id = Database:: insert_id();
                                     if ($image_id) {
 
-                                        $sql = "UPDATE $TABLETOOLDOCUMENT SET id = iid WHERE iid = $image_id";
+                                        $sql = "update $TABLETOOLDOCUMENT SET id = iid WHERE iid = $image_id";
                                         Database::query($sql);
 
                                         if ($path_documents.$value['file'] == '/certificates/default.html') {
@@ -895,7 +895,7 @@ 
                                             ]
                                         );
                                         if ($docId) {
-                                            $sql = "UPDATE $TABLEITEMPROPERTY SET id = iid WHERE iid = $docId";
+                                            $sql = "update $TABLEITEMPROPERTY SET id = iid WHERE iid = $docId";
                                             Database::query($sql);
                                         }
                                     }
@@ -1111,7 +1111,7 @@ 
         $authorId = empty($authorId) ? api_get_user_id() : (int) $authorId;
 
         if ($docId) {
-            $sql = "UPDATE $tableDocument SET id = iid WHERE iid = $docId";
+            $sql = "update $tableDocument SET id = iid WHERE iid = $docId";
             Database::query($sql);
 
             $id = Database::insert(
@@ -1133,7 +1133,7 @@ 
             );
 
             if ($id) {
-                $sql = "UPDATE $tableItem SET id = iid WHERE iid = $id";
+                $sql = "update $tableItem SET id = iid WHERE iid = $id";
                 Database::query($sql);
             }
         }
@@ -1279,7 +1279,7 @@ 
                     'creation_date' => $time,
                     'expiration_date' => $expiration_date,
                     'last_edit' => $time,
-                    'last_visit' => NULL,
+                    'last_visit' => null,
                     'tutor_name' => $tutor_name,
                     'department_name' => $department_name,
                     'department_url' => $department_url,

Doc Comments +1 added lines, -1 removed lines

@@ -1132,7 +1132,7 @@
      * and the api_get_setting('course_create_active_tools') should be 0 or 1 (used for
      * the visibility of the tool)
      * @param string $variable
-     * @return bool
+     * @return boolean|null
      * @author Patrick Cool, patrick.cool@ugent.be
      * @assert ('true') === true
      * @assert ('false') === false

main/lp/lp_move_item.php

Indentation +3 added lines, -3 removed lines

@@ -43,10 +43,10 @@
 " .
 "\n".
 '$().ready(function() {'."\n".
-  'if ($(\'#previous\')) {'."\n".
+    'if ($(\'#previous\')) {'."\n".
     'if(\'parent is\'+$(\'#idParent\').val()) {'.
-      'load_cbo($(\'#idParent\').val());'."\n".
-  '}}'."\n".
+        'load_cbo($(\'#idParent\').val());'."\n".
+    '}}'."\n".
 '});</script>'."\n";
 
 /* Constants and variables */

main/document/save_pixlr.php

Indentation +2 added lines, -2 removed lines

@@ -72,8 +72,8 @@
 $filename = disable_dangerous_file($filename);
 
 if (strlen(trim($filename)) == 0) {
-     echo "The title is empty"; //if title is empty, headers Content-Type = application/octet-stream, then not create a new title here please
-     exit;
+        echo "The title is empty"; //if title is empty, headers Content-Type = application/octet-stream, then not create a new title here please
+        exit;
 }
 
 //check file_get_contents

Braces +3 added lines, -1 removed lines

@@ -124,7 +124,9 @@
 
     if (file_exists($saveDir.'/'.$filename.'.'.$extension)) {
         $i = 1;
-        while (file_exists($saveDir.'/'.$filename.'_'.$i.'.'.$extension)) $i++;
+        while (file_exists($saveDir.'/'.$filename.'_'.$i.'.'.$extension)) {
+            $i++;
+        }
         $paintFileName = $filename.'_'.$i.'.'.$extension;
         $title = $filename.'_'.$i.'.'.$extension;
     }

main/document/webcam_receiver.php

Braces +1 added lines, -2 removed lines

@@ -17,8 +17,7 @@
     $webcamname = $params['webcamname'];
     $webcamdir = $params['webcamdir'];
     $webcamuserid = $params['webcamuserid'];
-}
-else {
+} else {
     api_not_allowed();
     die();
 }