MDL-17143 Some improvements suggested by Petr applied. Merged from 19…

…_STABLE
moodle · Nov 11, 2008 · 1ba3293 · 1ba3293
1 parent d9c2628
commit 1ba3293
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git 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
@@ -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
@@ -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').'<hr />';
         } else {
             echo format_text($user->description, FORMAT_MOODLE)."<hr />";