Minor - format code.

main/inc/lib/document.lib.php

@@ -548,7 +548,6 @@ class DocumentManager
 
         if ($result !== false && Database::num_rows($result) != 0) {
             while ($row = Database::fetch_array($result, 'ASSOC')) {
-
                 if (api_is_coach()) {
                     // Looking for course items that are invisible to hide it in the session
                     if (in_array($row['id'], array_keys($doc_list))) {
@@ -644,6 +643,7 @@ class DocumentManager
             } else {
                 $final_document_data = $document_data;
             }
+
             return $final_document_data;
         } else {
             return false;
@@ -658,8 +658,11 @@ class DocumentManager
      * @param boolean $can_see_invisible
      * @return array with paths
      */
-    public static function get_all_document_folders($_course, $to_group_id = '0', $can_see_invisible = false)
-    {
+    public static function get_all_document_folders(
+        $_course,
+        $to_group_id = '0',
+        $can_see_invisible = false
+    ) {
         $TABLE_ITEMPROPERTY = Database::get_course_table(TABLE_ITEM_PROPERTY);
         $TABLE_DOCUMENT = Database::get_course_table(TABLE_DOCUMENT);
 
@@ -989,7 +992,7 @@ class DocumentManager
     {
         $TABLE_DOCUMENT = Database :: get_course_table(TABLE_DOCUMENT);
         // @todo We should be able to get rid of this later when using only documentId (check further usage)
-        if (empty ($documentId)) {
+        if (empty($documentId)) {
             if (empty($path) || empty($base_work_dir)) {
                 return false;
             }
@@ -1025,7 +1028,6 @@ class DocumentManager
         }
 
         $document_exists_in_disk = file_exists($base_work_dir.$path);
-
         $new_path = $path.'_DELETED_'.$document_id;
 
         $file_deleted_from_db = false;
@@ -1661,7 +1663,7 @@ class DocumentManager
 
     /**
      * Remove default certificate
-     * @param string The course id
+     * @param string The course code
      * @param int The document id of the default certificate
      * @return void()
      */
@@ -1684,14 +1686,16 @@ class DocumentManager
             }
 
             $sql = 'UPDATE ' . $tbl_category . ' SET document_id=null
-                  WHERE course_code="' . Database::escape_string($course_id) . '" AND document_id="' . $default_certificate_id . '" ' . $sql_session;
+                    WHERE
+                        course_code = "' . Database::escape_string($course_id) . '" AND
+                        document_id="' . $default_certificate_id . '" ' . $sql_session;
             Database::query($sql);
         }
     }
 
     /**
      * Create directory certificate
-     * @param string The course id
+     * @param string The course code
      * @return void()
      */
     public static function create_directory_certificate_in_course($course_id)

main/inc/lib/groupmanager.lib.php

@@ -1343,10 +1343,14 @@ class GroupManager
         $group_id = Database::escape_string($group_id);
         $user_id = Database::escape_string($user_id);
         $course_id = api_get_course_int_id();
-        $sql = 'SELECT 1 FROM '.$table_group_user.' WHERE c_id = '.$course_id.' AND group_id = '.$group_id.' AND user_id = '.$user_id;
-        $db_result = Database::query($sql);
+        $sql = 'SELECT 1 FROM '.$table_group_user.'
+                WHERE
+                    c_id = '.$course_id.' AND
+                    group_id = '.$group_id.' AND
+                    user_id = '.$user_id;
+        $result = Database::query($sql);
 
-        return Database::num_rows($db_result) > 0;
+        return Database::num_rows($result) > 0;
     }
 
     /**