diff --git a/enrol/lti/classes/helper.php b/enrol/lti/classes/helper.php index ba554d60f65..7c80ea9affb 100644 --- a/enrol/lti/classes/helper.php +++ b/enrol/lti/classes/helper.php @@ -110,7 +110,13 @@ class helper { $user->country = (!empty($tool->country)) ? $tool->country : ""; $user->institution = (!empty($tool->institution)) ? $tool->institution : ""; $user->timezone = (!empty($tool->timezone)) ? $tool->timezone : ""; - $user->maildisplay = (!empty($tool->maildisplay)) ? $tool->maildisplay : $CFG->defaultpreference_maildisplay; + if (isset($tool->maildisplay)) { + $user->maildisplay = $tool->maildisplay; + } else if (isset($CFG->defaultpreference_maildisplay)) { + $user->maildisplay = $CFG->defaultpreference_maildisplay; + } else { + $user->maildisplay = 2; + } $user->mnethostid = $CFG->mnet_localhost_id; $user->confirmed = 1; $user->lang = $tool->lang; diff --git a/enrol/lti/settings.php b/enrol/lti/settings.php index 7dc6db617ea..6c92669c6bd 100644 --- a/enrol/lti/settings.php +++ b/enrol/lti/settings.php @@ -32,8 +32,9 @@ if ($ADMIN->fulltree) { $choices = array(0 => get_string('emaildisplayno'), 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'), '', - $CFG->defaultpreference_maildisplay, $choices)); + $maildisplay, $choices)); $city = ''; if (!empty($CFG->defaultcity)) {