MDL-45758 tool_monitor: add missing alt on manage icons

Original issue - MDL-47460

diff --git a/admin/tool/monitor/classes/output/managerules/renderable.php b/admin/tool/monitor/classes/output/managerules/renderable.php
index dabf325..c2d9586 100644 (file)
--- a/admin/tool/monitor/classes/output/managerules/renderable.php
+++ b/admin/tool/monitor/classes/output/managerules/renderable.php
@@ -163,16 +163,16 @@ class renderable extends \table_sql implements \renderable {
             $deleteurl = new \moodle_url($CFG->wwwroot. '/admin/tool/monitor/managerules.php', array('ruleid' => $rule->id,
                     'action' => 'delete', 'courseid' => $rule->courseid));
 
-            $icon = $OUTPUT->render(new \pix_icon('t/edit', ''));
+            $icon = $OUTPUT->render(new \pix_icon('t/edit', get_string('editrule', 'tool_monitor')));
             $manage .= \html_writer::link($editurl, $icon, array('class' => 'action-icon'));
 
-            $icon = $OUTPUT->render(new \pix_icon('t/copy', ''));
+            $icon = $OUTPUT->render(new \pix_icon('t/copy', get_string('duplicaterule', 'tool_monitor')));
             $manage .= \html_writer::link($copyurl, $icon, array('class' => 'action-icon'));
 
             $a = $rule->get_name($this->context);
             $action = new \component_action('click', 'M.util.show_confirm_dialog', array('message' => get_string('ruleareyousure',
                     'tool_monitor', $a)));
-            $icon = $OUTPUT->action_link($deleteurl, new \pix_icon('t/delete', ''), $action);
+            $icon = $OUTPUT->action_link($deleteurl, new \pix_icon('t/delete', get_string('deleterule', 'tool_monitor')), $action);
             $manage .= $icon;
         } else {
             $manage = '-';

diff --git a/admin/tool/monitor/classes/output/managesubs/subs.php b/admin/tool/monitor/classes/output/managesubs/subs.php
index 773f1be..493f96b 100644 (file)
--- a/admin/tool/monitor/classes/output/managesubs/subs.php
+++ b/admin/tool/monitor/classes/output/managesubs/subs.php
@@ -127,7 +127,8 @@ class subs extends \table_sql implements \renderable {
                 'action' => 'unsubscribe', 'courseid' => $this->courseid));
         $action = new \component_action('click', 'M.util.show_confirm_dialog', array('message' => get_string('subareyousure',
             'tool_monitor', $a)));
-        $icon = $OUTPUT->action_link($deleteurl, new \pix_icon('t/delete', ''), $action);
+        $icon = $OUTPUT->action_link($deleteurl,
+                new \pix_icon('t/delete', get_string('deletesubscription', 'tool_monitor')), $action);
         return $icon;
     }
 

diff --git a/admin/tool/monitor/lang/en/tool_monitor.php b/admin/tool/monitor/lang/en/tool_monitor.php
index eb888f5..1b67553 100644 (file)
--- a/admin/tool/monitor/lang/en/tool_monitor.php
+++ b/admin/tool/monitor/lang/en/tool_monitor.php
@@ -31,9 +31,13 @@ $string['core'] = 'Core';
 $string['customizefilters'] = 'Select the frequency of the events';
 $string['customizemessage'] = 'Cutomize the notification message';
 $string['currentsubscriptions'] = 'Your current subscriptions';
-$string['description'] = 'Description:';
 $string['description_help'] = "Description is displayed to users when they want to subscribe to this rule. This helps them understand what the rule is about.";
 $string['defaultmessagetpl'] = 'Rule "{rulename}" has happened. You can find further details at {link}';
+$string['deleterule'] = 'Delete rule';
+$string['deletesubscription'] = 'Delete subscription';
+$string['description'] = 'Description:';
+$string['duplicaterule'] = 'Duplicate rule';
+$string['editrule'] = 'Edit rule';
 $string['eventnotfound'] = 'Event not found';
 $string['errorincorrectevent'] = 'Please select an event related to the selected plugin';
 $string['freqdesc'] = '{$a->freq} times in {$a->mins} minutes';
@@ -59,7 +63,7 @@ $string['name_help'] = "Choose a name for the rule.";
 $string['pluginname'] = 'Event monitor';
 $string['processevents'] = 'Process events';
 $string['ruleareyousure'] = 'Are you sure you want to delete rule "{$a}"?';
-$string['rulecopysuccess'] = 'Rule successfully copied';
+$string['rulecopysuccess'] = 'Rule successfully duplicated';
 $string['ruledeletesuccess'] = 'Rule successfully deleted';
 $string['rulehelp'] = 'Rule details';
 $string['rulehelp_help'] = 'This rule listens for when the event \'{$a->eventname}\' in \'{$a->eventcomponent}\' has been triggered {$a->frequency} time(s) in {$a->minutes} minute(s).';