Minor - Format code.

main/gradebook/gradebook.php

@@ -587,12 +587,26 @@ $gradebooktable= new GradebookTable($cats[0], $allcat, $alleval, $alllink, $addp
 if (((empty ($allcat)) && (empty ($alleval)) && (empty ($alllink)) && (!$is_platform_admin) && ($is_course_admin) && (!isset ($_GET['selectcat']))) && api_is_course_tutor()) {
     Display :: display_normal_message(get_lang('GradebookWelcomeMessage') . '<br /><br /><form name="createcat" method="post" action="' . api_get_self() . '?createallcategories=1"><input type="submit" value="' . get_lang('CreateAllCat') . '"></form>',false);
 }
-//here we are in a sub category
+// Here we are in a sub category
 if ($category != '0') {
-    DisplayGradebook :: display_header_gradebook($cats[0], 1, $_GET['selectcat'], $is_course_admin, $is_platform_admin, $simple_search_form);
+    DisplayGradebook:: display_header_gradebook(
+        $cats[0],
+        1,
+        $_GET['selectcat'],
+        $is_course_admin,
+        $is_platform_admin,
+        $simple_search_form
+    );
 } else {
-    //this is the root category
-    DisplayGradebook :: display_header_gradebook($cats[0], (((count($allcat) == '0') && (!isset ($_GET['search']))) ? 0 : 1), 0, $is_course_admin, $is_platform_admin, $simple_search_form);
+    // This is the root category
+    DisplayGradebook:: display_header_gradebook(
+        $cats[0],
+        (((count($allcat) == '0') && (!isset ($_GET['search']))) ? 0 : 1),
+        0,
+        $is_course_admin,
+        $is_platform_admin,
+        $simple_search_form
+    );
 }
 $gradebooktable->display();
 Display :: display_footer();

main/gradebook/index.php

@@ -612,11 +612,11 @@ if (isset ($move_form)){
 
 // LOAD DATA & DISPLAY TABLE
 
-$is_platform_admin  = api_is_platform_admin();
-$is_course_admin    = api_is_allowed_to_edit(null, true);
+$is_platform_admin = api_is_platform_admin();
+$is_course_admin = api_is_allowed_to_edit(null, true);
 
 //load data for category, evaluation and links
-if (empty ($_GET['selectcat'])) {
+if (empty($_GET['selectcat'])) {
     $category= 0;
 } else {
     $category= $_GET['selectcat'];
@@ -834,7 +834,7 @@ if (isset($first_time) && $first_time==1 && api_is_allowed_to_edit(null,true)) {
 
                             $gradebook->save($params);
                         }
-                        //Reloading cats
+                        // Reloading cats
                         $cats = Category :: load(null, null, $course_code, null, null, $session_id, false);
                     } else {
                         $form_grade->display();
@@ -856,6 +856,7 @@ if (isset($first_time) && $first_time==1 && api_is_allowed_to_edit(null,true)) {
             } else {
                 // This is the father
                 // Create gradebook/add gradebook links.
+
                 DisplayGradebook::display_header_gradebook(
                     $cat,
                     0,

main/gradebook/lib/fe/usertable.class.php

@@ -19,7 +19,7 @@ class UserTable extends SortableTable
 	/**
 	 * Constructor
 	 */
-	public function UserTable ($userid, $evals = array(), $links = array(), $addparams = null)
+	public function UserTable($userid, $evals = array(), $links = array(), $addparams = null)
 	{
 		parent :: __construct ('userlist', null, null, 0);
 		$this->userid = $userid;
@@ -41,11 +41,10 @@ class UserTable extends SortableTable
 		}
 	}
 
-
 	/**
 	 * Function used by SortableTable to get total number of items in the table
 	 */
-	function get_total_number_of_items ()
+	function get_total_number_of_items()
 	{
 		return $this->datagen->get_total_items_count();
 	}
@@ -53,7 +52,7 @@ class UserTable extends SortableTable
 	/**
 	 * Function used by SortableTable to generate the data to display
 	 */
-	function get_table_data ($from = 1)
+	public function get_table_data($from = 1, $per_page = null, $column = null, $direction = null, $sort = null)
 	{
 		$scoredisplay = ScoreDisplay :: instance();
 
@@ -104,14 +103,23 @@ class UserTable extends SortableTable
 				$sortable_data[] = $row;
 			}
 		}
+
 		return $sortable_data;
 	}
 
+	/**
+	 * @param $item
+	 * @return string
+	 */
 	private function build_type_column($item)
 	{
 		return build_type_icon_tag($item->get_icon_name());
 	}
 
+	/**
+	 * @param $item
+	 * @return string
+	 */
 	private function build_name_link($item)
 	{
 		switch ($item->get_item_type()) {

main/gradebook/lib/gradebook_functions.inc.php

@@ -91,6 +91,7 @@ function update_resource_from_course_gradebook($link_id, $course_code, $weight)
                 WHERE course_code = "' . $course_code . '" AND id = ' . $link_id;
         Database::query($sql);
     }
+
     return true;
 }