Skip to content

Commit

Permalink
MDL-55491 badges: Add missing format_string call
Browse files Browse the repository at this point in the history
Extra version bump + trailing whitespace removal.
  • Loading branch information
David Monllao committed Apr 11, 2018
1 parent 728b851 commit 27fef51
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion badges/criteria/award_criteria_cohort.php
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ public function get_details($short = '') {
if (!$cohortname) {
$str = $OUTPUT->error_text(get_string('error:nosuchcohort', 'badges'));
} else {
$str = html_writer::tag('b', '"' . $cohortname . '"');
$str = html_writer::tag('b', '"' . format_string($cohortname, true) . '"');
}
$output[] = $str;
}
Expand Down
4 changes: 2 additions & 2 deletions badges/tests/behat/criteria_cohort.feature
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,7 @@ Feature: Award badges based on cohort
And I log in as "user2"
And I follow "Profile" in the user menu
And I should not see "Site Badge"

@javascript
Scenario: Award badge based on a single cohort membership or other criteria.
Given the following "cohorts" exist:
Expand Down Expand Up @@ -222,7 +222,7 @@ Feature: Award badges based on cohort
And I log in as "user3"
And I follow "Profile" in the user menu
And I should not see "Site Badge"

@javascript
Scenario: Award badge based on a multiple cohort membership or other criteria.
Given the following "cohorts" exist:
Expand Down
2 changes: 1 addition & 1 deletion version.php
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@

defined('MOODLE_INTERNAL') || die();

$version = 2018040500.02; // YYYYMMDD = weekly release date of this DEV branch.
$version = 2018040500.03; // YYYYMMDD = weekly release date of this DEV branch.
// RR = release increments - 00 in DEV branches.
// .XX = incremental changes.

Expand Down

0 comments on commit 27fef51

Please sign in to comment.