diff --git a/user/edit_form.php b/user/edit_form.php index ca9a7bc4f261c..a54cc7775c79b 100644 --- a/user/edit_form.php +++ b/user/edit_form.php @@ -64,7 +64,7 @@ function definition_after_data() { if ($user = $DB->get_record('user', array('id'=>$userid))) { // remove description - if (!$DB->get_record('role_assignments', array('userid'=>$userid)) && empty($user->description) && !empty($CFG->profilesforenrolledusersonly)) { + if (empty($user->description) && !empty($CFG->profilesforenrolledusersonly) && !$DB->get_record('role_assignments', array('userid'=>$userid))) { $mform->removeElement('description'); } diff --git a/user/editadvanced_form.php b/user/editadvanced_form.php index b4a4b151bf5f2..85db6614bc7a3 100644 --- a/user/editadvanced_form.php +++ b/user/editadvanced_form.php @@ -78,7 +78,7 @@ function definition_after_data() { $sitecontext = get_context_instance(CONTEXT_SYSTEM); $can_edit_user = has_capability('moodle/user:update', $sitecontext); // remove description - if (!$DB->get_record('role_assignments', array('userid'=>$userid)) && empty($user->description) && !empty($CFG->profilesforenrolledusersonly) && !$can_edit_user) { + if (empty($user->description) && !empty($CFG->profilesforenrolledusersonly) && !$can_edit_user && !$DB->get_record('role_assignments', array('userid'=>$userid))) { $mform->removeElement('description'); } diff --git a/user/view.php b/user/view.php index b41ebf9550843..6604abda12ea8 100644 --- a/user/view.php +++ b/user/view.php @@ -211,8 +211,7 @@ if ($user->description && !isset($hiddenfields['description'])) { $has_courseid = ($course->id != SITEID); - if (!$DB->get_record('role_assignments', array('userid'=>$id)) - && !$has_courseid && !empty($CFG->profilesforenrolledusersonly)) { + if (!$has_courseid && !empty($CFG->profilesforenrolledusersonly) && !$DB->get_record('role_assignments', array('userid'=>$id))) { echo get_string('profilenotshown', 'moodle').'
'; } else { echo format_text($user->description, FORMAT_MOODLE)."
";