From 4598cc934a3c3eecbec697fafdbaca29e655eeec Mon Sep 17 00:00:00 2001
From: Rajesh Taneja <rajesh@moodle.com>
Date: Tue, 3 Jul 2012 13:42:45 +0800
Subject: [PATCH] MDL-32787 user: rule required for custom profile fields
 applies to all users editing own profile

---
 user/edit.php              |  2 +-
 user/edit_form.php         | 18 ++++++++++++------
 user/editadvanced.php      |  2 +-
 user/editadvanced_form.php | 15 ++++++++++-----
 user/profile/lib.php       |  8 +++++---
 5 files changed, 29 insertions(+), 16 deletions(-)

diff --git a/user/edit.php b/user/edit.php
index 6fa160819e9..83954acc2bf 100644
--- a/user/edit.php
+++ b/user/edit.php
@@ -153,7 +153,7 @@ $editoroptions = array(
 );
 
 $user = file_prepare_standard_editor($user, 'description', $editoroptions, $personalcontext, 'user', 'profile', 0);
-$userform = new user_edit_form(null, array('editoroptions'=>$editoroptions));
+$userform = new user_edit_form(null, array('editoroptions'=>$editoroptions, 'userid' => $user->id));
 if (empty($user->country)) {
     // MDL-16308 - we must unset the value here so $CFG->country can be used as default one
     unset($user->country);
diff --git a/user/edit_form.php b/user/edit_form.php
index 2fdfe1e596e..dfcfa48732f 100644
--- a/user/edit_form.php
+++ b/user/edit_form.php
@@ -10,13 +10,19 @@ class user_edit_form extends moodleform {
 
     // Define the form
     function definition () {
-        global $CFG, $COURSE;
+        global $CFG, $COURSE, $USER;
 
         $mform =& $this->_form;
-        if (is_array($this->_customdata) && array_key_exists('editoroptions', $this->_customdata)) {
-            $editoroptions = $this->_customdata['editoroptions'];
-        } else {
-            $editoroptions = null;
+        $editoroptions = null;
+        $userid = $USER->id;
+
+        if (is_array($this->_customdata)) {
+            if (array_key_exists('editoroptions', $this->_customdata)) {
+                $editoroptions = $this->_customdata['editoroptions'];
+            }
+            if (array_key_exists('userid', $this->_customdata)) {
+                $userid = $this->_customdata['userid'];
+            }
         }
         //Accessibility: "Required" is bad legend text.
         $strgeneral  = get_string('general');
@@ -42,7 +48,7 @@ class user_edit_form extends moodleform {
         }
 
         /// Next the customisable profile fields
-        profile_definition($mform);
+        profile_definition($mform, $userid);
 
         $this->add_action_buttons(false, get_string('updatemyprofile'));
     }
diff --git a/user/editadvanced.php b/user/editadvanced.php
index 3c660d7f546..eb03645b798 100644
--- a/user/editadvanced.php
+++ b/user/editadvanced.php
@@ -135,7 +135,7 @@ if ($user->id !== -1) {
 }
 
 //create form
-$userform = new user_editadvanced_form(null, array('editoroptions'=>$editoroptions));
+$userform = new user_editadvanced_form(null, array('editoroptions'=>$editoroptions, 'userid' => $user->id));
 $userform->set_data($user);
 
 if ($usernew = $userform->get_data()) {
diff --git a/user/editadvanced_form.php b/user/editadvanced_form.php
index 2af0b1c20c2..530685a765e 100644
--- a/user/editadvanced_form.php
+++ b/user/editadvanced_form.php
@@ -13,11 +13,16 @@ class user_editadvanced_form extends moodleform {
         global $USER, $CFG, $COURSE;
 
         $mform =& $this->_form;
+        $editoroptions = null;
+        $userid = $USER->id;
 
-        if (is_array($this->_customdata) && array_key_exists('editoroptions', $this->_customdata)) {
-            $editoroptions = $this->_customdata['editoroptions'];
-        } else {
-            $editoroptions = null;
+        if (is_array($this->_customdata)) {
+            if (array_key_exists('editoroptions', $this->_customdata)) {
+                $editoroptions = $this->_customdata['editoroptions'];
+            }
+            if (array_key_exists('userid', $this->_customdata)) {
+                $userid = $this->_customdata['userid'];
+            }
         }
 
         //Accessibility: "Required" is bad legend text.
@@ -61,7 +66,7 @@ class user_editadvanced_form extends moodleform {
         useredit_shared_definition($mform, $editoroptions);
 
         /// Next the customisable profile fields
-        profile_definition($mform);
+        profile_definition($mform, $userid);
 
         $this->add_action_buttons(false, get_string('updatemyprofile'));
     }
diff --git a/user/profile/lib.php b/user/profile/lib.php
index 906f8ca1a91..de9a711361a 100644
--- a/user/profile/lib.php
+++ b/user/profile/lib.php
@@ -169,7 +169,8 @@ class profile_field_base {
      * @param   object   instance of the moodleform class
      */
     function edit_field_set_required(&$mform) {
-        if ($this->is_required() and !has_capability('moodle/user:update', get_context_instance(CONTEXT_SYSTEM))) {
+        global $USER;
+        if ($this->is_required() && ($this->userid == $USER->id)) {
             $mform->addRule($this->inputname, get_string('required'), 'required', null, 'client');
         }
     }
@@ -352,8 +353,9 @@ function profile_load_data(&$user) {
 /**
  * Print out the customisable categories and fields for a users profile
  * @param  object   instance of the moodleform class
+ * @param int $userid id of user whose profile is being edited.
  */
-function profile_definition(&$mform) {
+function profile_definition(&$mform, $userid = 0) {
     global $CFG, $DB;
 
     // if user is "admin" fields are displayed regardless
@@ -377,7 +379,7 @@ function profile_definition(&$mform) {
                     foreach ($fields as $field) {
                         require_once($CFG->dirroot.'/user/profile/field/'.$field->datatype.'/field.class.php');
                         $newfield = 'profile_field_'.$field->datatype;
-                        $formfield = new $newfield($field->id);
+                        $formfield = new $newfield($field->id, $userid);
                         $formfield->edit_field($mform);
                     }
                 }