MDL-65928 mod_choice: Use proper string for 'select/deselect all' labels
authorJun Pataleta <jun@moodle.com>
Fri, 12 Jul 2019 07:43:40 +0000 (15:43 +0800)
committerJun Pataleta <jun@moodle.com>
Fri, 12 Jul 2019 08:33:35 +0000 (16:33 +0800)
* Because string concatenation is bad!

mod/choice/lang/en/choice.php
mod/choice/renderer.php

index f9ec368..7aa6252 100644 (file)
@@ -70,6 +70,7 @@ $string['choice:view'] = 'View choice activity';
 $string['chooseaction'] = 'Choose an action ...';
 $string['chooseoption'] = 'Choose: {$a}';
 $string['description'] = 'Description';
+$string['deselectalloption'] = 'Deselect all "{$a}"';
 $string['includeinactive'] = 'Include responses from inactive/suspended users';
 $string['indicator:cognitivedepth'] = 'Choice cognitive';
 $string['indicator:cognitivedepth_help'] = 'This indicator is based on the cognitive depth reached by the student in a Choice activity.';
@@ -134,6 +135,7 @@ $string['responsesto'] = 'Responses to {$a}';
 $string['results'] = 'Results';
 $string['savemychoice'] = 'Save my choice';
 $string['search:activity'] = 'Choice - activity information';
+$string['selectalloption'] = 'Select all "{$a}"';
 $string['showpreview'] = 'Show preview';
 $string['showpreview_help'] = 'Allow students to preview the available options before the choice is opened for submission.';
 $string['showunanswered'] = 'Show column for unanswered';
index ecd3e6a..2811996 100644 (file)
@@ -193,8 +193,8 @@ class mod_choice_renderer extends plugin_renderer_base {
                 // Build the select/deselect all for this option.
                 $selectallid = 'select-response-option-' . $optionid;
                 $togglegroup = 'responses response-option-' . $optionid;
-                $selectalltext = get_string('selectall', 'moodle') . ' ' . $headertitle;
-                $deselectalltext = get_string('deselectall', 'moodle') . ' ' . $headertitle;
+                $selectalltext = get_string('selectalloption', 'choice', $headertitle);
+                $deselectalltext = get_string('deselectalloption', 'choice', $headertitle);
                 $mastercheckbox = new \core\output\checkbox_toggleall($togglegroup, true, [
                     'id' => $selectallid,
                     'name' => $selectallid,