Skip to content

Commit 1acdc7f

Browse files
committed
MDL-81678 enrol: don't escape course name in welcome message.
1 parent d3ae139 commit 1acdc7f

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

enrol/manual/tests/lib_test.php

+4-3
Original file line numberDiff line numberDiff line change
@@ -800,7 +800,7 @@ public function test_send_course_welcome_message(): void {
800800

801801
// Create course.
802802
$course = $this->getDataGenerator()->create_course([
803-
'fullname' => 'Course 1',
803+
'fullname' => 'Course 1 & 2',
804804
'shortname' => 'C1',
805805
]);
806806
// Create users.
@@ -860,7 +860,8 @@ public function test_send_course_welcome_message(): void {
860860
instance: $maninstance,
861861
userid: $student->id,
862862
sendoption: ENROL_SEND_EMAIL_FROM_COURSE_CONTACT,
863-
message: 'Your email address: {$a->email}, your first name: {$a->firstname}, your last name: {$a->lastname}',
863+
message: 'Your email address: {$a->email}, your first name: {$a->firstname}, your last name: {$a->lastname}, ' .
864+
'your course: {$a->coursename}',
864865
);
865866
$messages = $messagesink->get_messages_by_component_and_type(
866867
'moodle',
@@ -874,7 +875,7 @@ public function test_send_course_welcome_message(): void {
874875
$this->assertStringContainsString($course->fullname, $message->subject);
875876
$this->assertEquals(
876877
'Your email address: ' . $student->email . ', your first name: ' . $student->firstname . ', your last name: ' .
877-
$student->lastname,
878+
$student->lastname . ', your course: ' . $course->fullname,
878879
$message->fullmessage,
879880
);
880881
// Clear sink.

lib/enrollib.php

+2-3
Original file line numberDiff line numberDiff line change
@@ -3662,7 +3662,7 @@ public function send_course_welcome_message_to_user(
36623662
);
36633663

36643664
$a = new stdClass();
3665-
$a->coursename = format_string($course->fullname, true, ['context' => $context]);
3665+
$a->coursename = format_string($course->fullname, true, ['context' => $context, 'escape' => false]);
36663666
$a->profileurl = (new moodle_url(
36673667
url: '/user/view.php',
36683668
params: [
@@ -3707,7 +3707,6 @@ public function send_course_welcome_message_to_user(
37073707
$messagehtml = text_to_html($messagetext, null, false, true);
37083708
}
37093709

3710-
$subject = get_string('welcometocourse', 'moodle', format_string($course->fullname, true, ['context' => $context]));
37113710
$contact = $this->get_welcome_message_contact(
37123711
sendoption: $sendoption,
37133712
context: $context,
@@ -3723,7 +3722,7 @@ public function send_course_welcome_message_to_user(
37233722
$message->name = 'enrolcoursewelcomemessage';
37243723
$message->userfrom = $contact;
37253724
$message->userto = $user;
3726-
$message->subject = $subject;
3725+
$message->subject = get_string('welcometocourse', 'moodle', $a->coursename);
37273726
$message->fullmessage = $messagetext;
37283727
$message->fullmessageformat = FORMAT_MARKDOWN;
37293728
$message->fullmessagehtml = $messagehtml;

0 commit comments

Comments
 (0)