moodle/moodle
3
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-08-05 00:46:50 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

MDL-32787 user: rule required for custom profile fields applies to all users editing own profile

Browse source
This commit is contained in:
Rajesh Taneja 2012-05-15 21:53:09 +04:00
parent 1494f20da3
commit 28f23328c5
5 changed files with 39 additions and 26 deletions
Download patch file Download diff file Expand all files Collapse all files

3
user/edit.php
View file

@ -165,7 +165,8 @@ $user->imagefile = $draftitemid;
//create form
$userform = new user_edit_form(null, array(
'editoroptions' => $editoroptions,
'filemanageroptions' => $filemanageroptions));
'filemanageroptions' => $filemanageroptions,
'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);

27
user/edit_form.php
View file

@ -10,18 +10,23 @@ 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;
}
if (is_array($this->_customdata) && array_key_exists('filemanageroptions', $this->_customdata)) {
$filemanageroptions = $this->_customdata['filemanageroptions'];
} else {
$filemanageroptions = null;
$editoroptions = null;
$filemanageroptions = null;
$userid = $USER->id;
if (is_array($this->_customdata)) {
if (array_key_exists('editoroptions', $this->_customdata)) {
$editoroptions = $this->_customdata['editoroptions'];
}
if (array_key_exists('filemanageroptions', $this->_customdata)) {
$filemanageroptions = $this->_customdata['filemanageroptions'];
}
if (array_key_exists('userid', $this->_customdata)) {
$userid = $this->_customdata['userid'];
}
}
//Accessibility: "Required" is bad legend text.
$strgeneral = get_string('general');
@ -47,7 +52,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'));
}

3
user/editadvanced.php
View file

@ -146,7 +146,8 @@ $user->imagefile = $draftitemid;
//create form
$userform = new user_editadvanced_form(null, array(
'editoroptions' => $editoroptions,
'filemanageroptions' => $filemanageroptions));
'filemanageroptions' => $filemanageroptions,
'userid' => $user->id));
$userform->set_data($user);
if ($usernew = $userform->get_data()) {

24
user/editadvanced_form.php
View file

@ -13,16 +13,20 @@ class user_editadvanced_form extends moodleform {
global $USER, $CFG, $COURSE;
$mform =& $this->_form;
$editoroptions = null;
$filemanageroptions = 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) && array_key_exists('filemanageroptions', $this->_customdata)) {
$filemanageroptions = $this->_customdata['filemanageroptions'];
} else {
$filemanageroptions = null;
if (is_array($this->_customdata)) {
if (array_key_exists('editoroptions', $this->_customdata)) {
$editoroptions = $this->_customdata['editoroptions'];
}
if (array_key_exists('filemanageroptions', $this->_customdata)) {
$filemanageroptions = $this->_customdata['filemanageroptions'];
}
if (array_key_exists('userid', $this->_customdata)) {
$userid = $this->_customdata['userid'];
}
}
//Accessibility: "Required" is bad legend text.
@ -66,7 +70,7 @@ class user_editadvanced_form extends moodleform {
useredit_shared_definition($mform, $editoroptions, $filemanageroptions);
/// Next the customisable profile fields
profile_definition($mform);
profile_definition($mform, $userid);
$this->add_action_buttons(false, get_string('updatemyprofile'));
}

8
user/profile/lib.php
View file

@ -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);
}
}