diff --git a/grade/report/grader/index.php b/grade/report/grader/index.php
index b9045f47825..137404f21d1 100644
--- a/grade/report/grader/index.php
+++ b/grade/report/grader/index.php
@@ -5,10 +5,6 @@
require_once($CFG->libdir.'/tablelib.php');
include_once($CFG->libdir.'/gradelib.php');
-// Prepare language strings
-$strsortasc = get_string('sortasc', 'grades');
-$strsortdesc = get_string('sortasc', 'grades');
-
/// processing posted grades here
if ($data = data_submitted()) {
@@ -65,6 +61,15 @@ if ($perpageurl = optional_param('perpage', 0, PARAM_INT)) {
$perpage = $perpageurl;
}
+// Prepare language strings
+$strsortasc = get_string('sortasc', 'grades');
+$strsortdesc = get_string('sortasc', 'grades');
+
+// base url for sorting by first/last name
+$baseurl = 'report.php?id='.$courseid.'&report=grader&page='.$page;
+// base url for paging
+$pbarurl = 'report.php?id='.$courseid.'&report=grader&';
+
// Grab the grade_tree for this course
$gtree = new grade_tree($courseid);
@@ -166,12 +171,13 @@ grade_update_final_grades($courseid);
// roles to be displaye in the gradebook
$gradebookroles = $CFG->gradebookroles;
-// pulls out the userids of the users to be display, and sort them
-// the right outer join is needed because potentially, it is possible not
-// to have the corresponding entry in grade_grades table for some users
-// this is check for user roles because there could be some users with grades
-// but not supposed to be displayed
-
+/*
+* pulls out the userids of the users to be display, and sort them
+* the right outer join is needed because potentially, it is possible not
+* to have the corresponding entry in grade_grades table for some users
+* this is check for user roles because there could be some users with grades
+* but not supposed to be displayed
+*/
if (is_numeric($sortitemid)) {
$sql = "SELECT u.id, u.firstname, u.lastname
FROM {$CFG->prefix}grade_grades g RIGHT OUTER JOIN
@@ -227,19 +233,6 @@ if ($grades = get_records_sql($sql)) {
}
}
-print_heading('Grader Report');
-
-// Add tabs
-$currenttab = 'graderreport';
-include('tabs.php');
-
-// base url for sorting by first/last name
-$baseurl = 'report.php?id='.$courseid.'&report=grader&page='.$page;
-// base url for paging
-$pbarurl = 'report.php?id='.$courseid.'&report=grader&';
-
-print_paging_bar($numusers, $page, $perpage, $pbarurl);
-
/// With the users in an sorted array and grades fetched, we can not print the main html table
// 1. Fetch all top-level categories for this course, with all children preloaded, sorted by sortorder
@@ -270,13 +263,23 @@ if ($sortitemid === 'firstname') {
$firstarrow = '';
}
-// first name/last name column
+/********* BEGIN OUTPUT *********/
+
+print_heading('Grader Report');
+
+// Add tabs
+$currenttab = 'graderreport';
+include('tabs.php');
+
+print_paging_bar($numusers, $page, $perpage, $pbarurl);
$items = array();
+// Prepare Table Headers
$headerhtml = '';
$numrows = count($gtree->levels);
+
foreach ($gtree->levels as $key=>$row) {
if ($key == 0) {
// do not diplay course grade category
@@ -286,7 +289,8 @@ foreach ($gtree->levels as $key=>$row) {
$headerhtml .= '