Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

MDL-34774 Lesson module: fixed report stats for 'high score' and 'low…

… score' values where there is only one attempt with 0 score
  • Loading branch information...
commit 819cdd80413f78bdec59969abc5f14ba614492f5 1 parent a09484c
@rwijaya rwijaya authored
Showing with 12 additions and 5 deletions.
  1. +12 −5 mod/lesson/report.php
View
17 mod/lesson/report.php
@@ -284,10 +284,10 @@
$numofattempts++;
$avescore += $try["grade"];
$avetime += $timetotake;
- if ($try["grade"] > $highscore || $highscore == NULL) {
+ if ($try["grade"] > $highscore || $highscore === NULL) {
$highscore = $try["grade"];
}
- if ($try["grade"] < $lowscore || $lowscore == NULL) {
+ if ($try["grade"] < $lowscore || $lowscore === NULL) {
$lowscore = $try["grade"];
}
if ($timetotake > $hightime || $hightime == NULL) {
@@ -346,10 +346,10 @@
} else {
$lowtime = format_time($lowtime);
}
- if ($highscore == NULL) {
+ if ($highscore === NULL) {
$highscore = get_string("notcompleted", "lesson");
}
- if ($lowscore == NULL) {
+ if ($lowscore === NULL) {
$lowscore = get_string("notcompleted", "lesson");
}
@@ -362,7 +362,14 @@
$stattable->align = array('center', 'center', 'center', 'center', 'center', 'center');
$stattable->wrap = array('nowrap', 'nowrap', 'nowrap', 'nowrap', 'nowrap', 'nowrap');
$stattable->attributes['class'] = 'standardtable generaltable';
- $stattable->data[] = array($avescore.'%', $avetime, $highscore.'%', $lowscore.'%', $hightime, $lowtime);
+
+ if (is_numeric($highscore)) {
+ $highscore .= '%';
+ }
+ if (is_numeric($lowscore)) {
+ $lowscore .= '%';
+ }
+ $stattable->data[] = array($avescore.'%', $avetime, $highscore, $lowscore, $hightime, $lowtime);
echo html_writer::table($stattable);
} else if ($action === 'reportdetail') {
Please sign in to comment.
Something went wrong with that request. Please try again.