Skip to content

Commit b7ceb14

Browse files
committed
Tracking: fix average total time spent on platform calculation and use
See dfca264
1 parent f41ee04 commit b7ceb14

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

main/mySpace/index.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -314,6 +314,8 @@
314314
$posts = null;
315315

316316
if ($skipData === false) {
317+
$averageTimeSpentPerStudent = '00:00:00';
318+
317319
if (!empty($students)) {
318320
// Students
319321
$studentIds = array_values($students);
@@ -325,7 +327,7 @@
325327
$numberAssignments = $countAssignments / $numberStudents;
326328
$avg_courses_per_student = $countCourses / $numberStudents;
327329
$totalTimeSpent = Tracking::get_time_spent_on_the_platform($studentIds, 'ever');
328-
$averageTimeSpentPerStudent = $totalTimeSpent / $numberStudents;
330+
$averageTimeSpentPerStudent = api_time_to_hms($totalTimeSpent / $numberStudents);
329331
$posts = Tracking::count_student_messages($studentIds);
330332
$averageScore = Tracking::getAverageStudentScore($studentIds);
331333
}
@@ -379,9 +381,7 @@
379381
? ''
380382
: round($avg_courses_per_student, 3);
381383
$report['InactivesStudents'] = $nb_inactive_students;
382-
$report['AverageTimeSpentOnThePlatform'] = is_null($averageTimeSpentPerStudent)
383-
? '00:00:00'
384-
: api_time_to_hms($averageTimeSpentPerStudent);
384+
$report['AverageTimeSpentOnThePlatform'] = $averageTimeSpentPerStudent;
385385
$report['AverageProgressInLearnpath'] = is_null($avgTotalProgress)
386386
? ''
387387
: round($avgTotalProgress, 2).'%';

0 commit comments

Comments
 (0)