MDL-40200 Administration: Return the invalid user message and stop there
authorMatteo Scaramuccia <moodle@matteoscaramuccia.com>
Sat, 15 Jun 2013 10:17:54 +0000 (12:17 +0200)
committerEloy Lafuente (stronk7) <stronk7@moodle.org>
Mon, 24 Jun 2013 17:03:29 +0000 (19:03 +0200)
To avoid adding duplicate code, 'invalid user' and 'user deleted' checks
have been merged together.

Conflicts:
user/profile.php

user/profile.php

index 9162a43..3e35951 100644 (file)
@@ -56,12 +56,14 @@ if (!empty($CFG->forceloginforprofiles)) {
 }
 
 $userid = $userid ? $userid : $USER->id;       // Owner of the page
-$user = $DB->get_record('user', array('id' => $userid));
-
-if ($user->deleted) {
-    $PAGE->set_context(get_context_instance(CONTEXT_SYSTEM));
+if ((!$user = $DB->get_record('user', array('id' => $userid))) || ($user->deleted)) {
+    $PAGE->set_context(context_system::instance());
     echo $OUTPUT->header();
-    echo $OUTPUT->notification(get_string('userdeleted'));
+    if (!$user) {
+        echo $OUTPUT->notification(get_string('invaliduser', 'error'));
+    } else {
+        echo $OUTPUT->notification(get_string('userdeleted'));
+    }
     echo $OUTPUT->footer();
     die;
 }