diff --git a/completion/criteria/completion_criteria_activity.php b/completion/criteria/completion_criteria_activity.php index 4fa66b536b638..c64805e01cfc3 100644 --- a/completion/criteria/completion_criteria_activity.php +++ b/completion/criteria/completion_criteria_activity.php @@ -60,7 +60,11 @@ public static function fetch($params) { * @param stdClass $data details of various modules */ public function config_form_display(&$mform, $data = null) { - $mform->addElement('checkbox', 'criteria_activity['.$data->id.']', ucfirst(self::get_mod_name($data->module)).' - '.$data->name); + $modnames = get_module_types_names(); + $mform->addElement('checkbox', 'criteria_activity['.$data->id.']', + $modnames[self::get_mod_name($data->module)]. + ' - '. + format_string($data->name)); if ($this->id) { $mform->setDefault('criteria_activity['.$data->id.']', 1); @@ -177,7 +181,8 @@ public function get_title_detailed() { $module = $DB->get_record('course_modules', array('id' => $this->moduleinstance)); $activity = $DB->get_record($this->module, array('id' => $module->instance)); - return shorten_text(urldecode($activity->name)); + return shorten_text(format_string($activity->name, true, + array('context' => context_module::instance($module->id)))); } /** diff --git a/report/completion/index.php b/report/completion/index.php index 5f84c2669524c..0791018ac9652 100644 --- a/report/completion/index.php +++ b/report/completion/index.php @@ -468,7 +468,7 @@ // Display icon $icon = $OUTPUT->pix_url('icon', $criterion->module); $iconlink = $CFG->wwwroot.'/mod/'.$criterion->module.'/view.php?id='.$criterion->moduleinstance; - $icontitle = $modinfo->cms[$criterion->moduleinstance]->name; + $icontitle = $modinfo->cms[$criterion->moduleinstance]->get_formatted_name(); $iconalt = get_string('modulename', $criterion->module); break; @@ -532,8 +532,9 @@ // Load activity $mod = $criterion->get_mod_instance(); - $row[] = $mod->name; - $row[] = $mod->name . ' - ' . get_string('completiondate', 'report_completion'); + $row[] = $formattedname = format_string($mod->name, true, + array('context' => context_module::instance($criterion->moduleinstance))); + $row[] = $formattedname . ' - ' . get_string('completiondate', 'report_completion'); } else { // Handle all other criteria @@ -616,7 +617,7 @@ $a->state = $describe; $a->date = $date; $a->user = fullname($user); - $a->activity = strip_tags($activity->name); + $a->activity = $activity->get_formatted_name(); $fulldescribe = get_string('progress-title', 'completion', $a); if ($csv) { @@ -626,7 +627,7 @@ print ''; print ''.$describe.''; + '" alt="'.s($describe).'" class="icon" title="'.s($fulldescribe).'" />'; print ''; } @@ -672,11 +673,12 @@ ) ); - print '' . + print '' . ''.$describe.''; + '" alt="'.s($describe).'" class="icon" />'; } else { - print ''.$describe.''; + print ''.s($describe).
+                        ''; } print ''; @@ -717,7 +719,7 @@ // Display course completion status icon print ''.$describe.''; + '" alt="'.s($describe).'" class="icon" title="'.s($fulldescribe).'" />'; print ''; } diff --git a/report/progress/index.php b/report/progress/index.php index 8f87baa8431fb..ac781f638d959 100644 --- a/report/progress/index.php +++ b/report/progress/index.php @@ -317,18 +317,19 @@ function csv_quote($value) { } // Some names (labels) come URL-encoded and can be very long, so shorten them - $displayname = shorten_text($activity->name); + $displayname = format_string($activity->name, true, array('context' => $activity->context)); if ($csv) { - print $sep.csv_quote(strip_tags($displayname)).$sep.csv_quote($datetext); + print $sep.csv_quote($displayname).$sep.csv_quote($datetext); } else { - $formattedactivityname = format_string($displayname, true, array('context' => $activity->context)); + $shortenedname = shorten_text($displayname); print ''. ''. + '/view.php?id='.$activity->id.'" title="' . s($displayname) . '">'. ''.
-            get_string('modulename',$activity->modname).' '. - $formattedactivityname.''; + s(get_string('modulename', $activity->modname)). + '" /> '. + $shortenedname.''; if ($activity->completionexpected) { print '
'.$datetext.'
'; } @@ -392,7 +393,7 @@ function csv_quote($value) { $a->state=$describe; $a->date=$date; $a->user=fullname($user); - $a->activity = format_string($formattedactivities[$activity->id]->displayname, true, array('context' => $activity->context)); + $a->activity = $formattedactivities[$activity->id]->displayname; $fulldescribe=get_string('progress-title','completion',$a); if ($csv) { @@ -400,7 +401,7 @@ function csv_quote($value) { } else { print ''. ''.$describe.''; + '" alt="'.s($describe).'" title="'.s($fulldescribe).'" />'; } }