Skip to content

Commit

Permalink
MDL-60548 profile: Clarify some users can always see email addresses.
Browse files Browse the repository at this point in the history
  • Loading branch information
David Balch committed Jan 15, 2018
1 parent 565f3f2 commit d2bba97
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 4 deletions.
2 changes: 1 addition & 1 deletion admin/settings/users.php
Expand Up @@ -28,7 +28,7 @@
$choices['1'] = new lang_string('emaildisplayyes');
$choices['2'] = new lang_string('emaildisplaycourse');
$temp->add(new admin_setting_configselect('defaultpreference_maildisplay', new lang_string('emaildisplay'),
'', 2, $choices));
new lang_string('emaildisplay_help'), 2, $choices));

$choices = array();
$choices['0'] = new lang_string('textformat');
Expand Down
1 change: 1 addition & 0 deletions admin/tool/uploaduser/user_form.php
Expand Up @@ -229,6 +229,7 @@ function definition () {
$choices = array(0 => get_string('emaildisplayno'), 1 => get_string('emaildisplayyes'), 2 => get_string('emaildisplaycourse'));
$mform->addElement('select', 'maildisplay', get_string('emaildisplay'), $choices);
$mform->setDefault('maildisplay', core_user::get_property_default('maildisplay'));
$mform->addHelpButton('maildisplay', 'emaildisplay');

$choices = array(0 => get_string('textformat'), 1 => get_string('htmlformat'));
$mform->addElement('select', 'mailformat', get_string('emailformat'), $choices);
Expand Down
1 change: 1 addition & 0 deletions enrol/lti/lib.php
Expand Up @@ -298,6 +298,7 @@ public function edit_instance_form($instance, MoodleQuickForm $mform, $context)
);
$mform->addElement('select', 'maildisplay', get_string('emaildisplay'), $choices);
$mform->setDefault('maildisplay', $emaildisplay);
$mform->addHelpButton('maildisplay', 'emaildisplay');

$city = get_config('enrol_lti', 'city');
$mform->addElement('text', 'city', get_string('city'), 'maxlength="100" size="25"');
Expand Down
4 changes: 2 additions & 2 deletions enrol/lti/settings.php
Expand Up @@ -47,8 +47,8 @@
1 => get_string('emaildisplayyes'),
2 => get_string('emaildisplaycourse'));
$maildisplay = isset($CFG->defaultpreference_maildisplay) ? $CFG->defaultpreference_maildisplay : 2;
$settings->add(new admin_setting_configselect('enrol_lti/emaildisplay', get_string('emaildisplay'), '',
$maildisplay, $choices));
$settings->add(new admin_setting_configselect('enrol_lti/emaildisplay', get_string('emaildisplay'),
get_string('emaildisplay_help'), $maildisplay, $choices));

$city = '';
if (!empty($CFG->defaultcity)) {
Expand Down
3 changes: 2 additions & 1 deletion lang/en/moodle.php
Expand Up @@ -615,9 +615,10 @@
$string['emaildisable'] = 'This email address is disabled';
$string['emaildisableclick'] = 'Click here to disable all email from being sent to this address';
$string['emaildisplay'] = 'Email display';
$string['emaildisplay_help'] = 'Privileged users (such as teachers and managers) will always be able to see your email address.';
$string['emaildisplaycourse'] = 'Allow only other course members to see my email address';
$string['emaildisplayhidden'] = 'Email hidden';
$string['emaildisplayno'] = 'Hide my email address from everyone';
$string['emaildisplayno'] = 'Hide my email address from non-privileged users';
$string['emaildisplayyes'] = 'Allow everyone to see my email address';
$string['emailenable'] = 'This email address is enabled';
$string['emailenableclick'] = 'Click here to re-enable all email being sent to this address';
Expand Down
1 change: 1 addition & 0 deletions user/editlib.php
Expand Up @@ -302,6 +302,7 @@ function useredit_shared_definition(&$mform, $editoroptions, $filemanageroptions
$choices['2'] = get_string('emaildisplaycourse');
$mform->addElement('select', 'maildisplay', get_string('emaildisplay'), $choices);
$mform->setDefault('maildisplay', core_user::get_property_default('maildisplay'));
$mform->addHelpButton('maildisplay', 'emaildisplay');

$mform->addElement('text', 'city', get_string('city'), 'maxlength="120" size="21"');
$mform->setType('city', PARAM_TEXT);
Expand Down

0 comments on commit d2bba97

Please sign in to comment.