From: moodler <moodler>
Date: Wed, 13 Sep 2006 06:56:25 +0000 (+0000)
Subject: Tidy-ups for user profile page
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=e82dc9bb41f77cdb3017a050ccc9138fdae72e61;p=moodle.git

Tidy-ups for user profile page
---

diff --git a/user/view.php b/user/view.php
index 3e9de03c35..bd7c29c40e 100644
--- a/user/view.php
+++ b/user/view.php
@@ -38,10 +38,8 @@
     add_to_log($course->id, "user", "view", "view.php?id=$user->id&course=$course->id", "$user->id");
 
     if ($course->id != SITEID) {
-        if ($student = get_record("user_students", "userid", $user->id, "course", $course->id)) {
-            $user->lastaccess = $student->timeaccess;
-        } else if ($teacher = get_record("user_teachers", "userid", $user->id, "course", $course->id)) {
-            $user->lastaccess = $teacher->timeaccess;
+        if ($lastaccess = get_record('user_lastaccess', 'userid', $user->id, 'course', $course->id)) {
+            $user->lastaccess = $lastaccess->timeaccess;
         }
     }
 
@@ -99,11 +97,6 @@
                      "$fullname", "", "", true, "&nbsp;", navmenu($course));
     }
 
-    if ($CFG->debug && $USER->id == $user->id) {   // TEMPORARY in DEV!
-        print_heading('DEBUG MODE:  User session variables');
-        print_object($USER);
-    }
-
 
     if ($course->category and ! isguest() ) {   // Need to have access to a course to see that info
         if (!has_capability('moodle/course:view', get_context_instance(CONTEXT_COURSE, $course->id))) {
@@ -274,8 +267,9 @@
     }
 /// printing roles
     
-    $rolestring = get_user_roles_in_context($id, $coursecontext->id);
-    print_row(get_string("roles").":", $rolestring);
+    if ($rolestring = get_user_roles_in_context($id, $coursecontext->id)) {
+        print_row(get_string('roles').':', $rolestring);
+    }
 
 /// Printing groups
     $isseparategroups = ($course->groupmode == SEPARATEGROUPS and $course->groupmodeforce and
@@ -377,6 +371,14 @@
     echo "<td></td>";
     echo "</tr></table></div>\n";
 
+
+    if ($CFG->debug && $USER->id == $user->id) {   // TEMPORARY in DEV!   XXX TODO
+        echo '<hr />';
+        print_heading('DEBUG MODE:  User session variables');
+        print_object($USER);
+    }
+
+
     print_footer($course);
 
 /// Functions ///////