diff --git a/enrol/locallib.php b/enrol/locallib.php index 03e2f6c38ca..5e2a928f02d 100644 --- a/enrol/locallib.php +++ b/enrol/locallib.php @@ -1099,19 +1099,26 @@ class course_enrolment_manager { */ private function prepare_user_for_display($user, $extrafields, $now) { $details = array( - 'userid' => $user->id, - 'courseid' => $this->get_course()->id, - 'picture' => new user_picture($user), - 'firstname' => fullname($user, has_capability('moodle/site:viewfullnames', $this->get_context())), - 'lastseen' => get_string('never'), + 'userid' => $user->id, + 'courseid' => $this->get_course()->id, + 'picture' => new user_picture($user), + 'firstname' => fullname($user, has_capability('moodle/site:viewfullnames', $this->get_context())), + 'lastseen' => get_string('never'), + 'lastcourseaccess' => get_string('never'), ); foreach ($extrafields as $field) { $details[$field] = $user->{$field}; } + // Last time user has accessed the site. if ($user->lastaccess) { $details['lastseen'] = format_time($now - $user->lastaccess); } + + // Last time user has accessed the course. + if ($user->lastseen) { + $details['lastcourseaccess'] = format_time($now - $user->lastseen); + } return $details; } diff --git a/enrol/users.php b/enrol/users.php index b82ebe4bad5..49421ef0d50 100644 --- a/enrol/users.php +++ b/enrol/users.php @@ -186,7 +186,7 @@ foreach ($extrafields as $field) { $fields = array( 'userdetails' => $userdetails, - 'lastseen' => get_string('lastaccess'), + 'lastcourseaccess' => get_string('lastcourseaccess'), 'role' => get_string('roles', 'role'), 'group' => get_string('groups', 'group'), 'enrol' => get_string('enrolmentinstances', 'enrol') @@ -196,7 +196,7 @@ $fields = array( if (!has_capability('moodle/course:viewhiddenuserfields', $context)) { $hiddenfields = array_flip(explode(',', $CFG->hiddenuserfields)); if (isset($hiddenfields['lastaccess'])) { - unset($fields['lastseen']); + unset($fields['lastcourseaccess']); } if (isset($hiddenfields['groups'])) { unset($fields['group']); diff --git a/lang/en/moodle.php b/lang/en/moodle.php index be887038af4..3bb13f04230 100644 --- a/lang/en/moodle.php +++ b/lang/en/moodle.php @@ -779,6 +779,7 @@ $string['first'] = 'First'; $string['firstaccess'] = 'First access'; $string['firstname'] = 'First name'; $string['firstnamephonetic'] = 'First name - phonetic'; +$string['firstsiteaccess'] = 'First access to site'; $string['firsttime'] = 'Is this your first time here?'; $string['folder'] = 'Folder'; $string['folderclosed'] = 'Closed folder'; @@ -1007,12 +1008,14 @@ $string['language'] = 'Language'; $string['languagegood'] = 'This language pack is up-to-date! :-)'; $string['last'] = 'Last'; $string['lastaccess'] = 'Last access'; +$string['lastcourseaccess'] = 'Last access to course'; $string['lastedited'] = 'Last edited'; $string['lastip'] = 'Last IP address'; $string['lastlogin'] = 'Last login'; $string['lastmodified'] = 'Last modified'; $string['lastname'] = 'Surname'; $string['lastnamephonetic'] = 'Surname - phonetic'; +$string['lastsiteaccess'] = 'Last access to site'; $string['lastyear'] = 'Last year'; $string['latestlanguagepack'] = 'Check for latest language pack on moodle.org'; $string['layouttable'] = 'Layout table'; diff --git a/user/index.php b/user/index.php index e0737b4f3e7..72e62fb4e89 100644 --- a/user/index.php +++ b/user/index.php @@ -363,7 +363,12 @@ if ($mode === MODE_BRIEF && !isset($hiddenfields['country'])) { } if (!isset($hiddenfields['lastaccess'])) { $tablecolumns[] = 'lastaccess'; - $tableheaders[] = get_string('lastaccess'); + if ($course->id == SITEID) { + // Exception case for viewing participants on site home. + $tableheaders[] = get_string('lastsiteaccess'); + } else { + $tableheaders[] = get_string('lastcourseaccess'); + } } if ($bulkoperations && $mode === MODE_USERDETAILS) { @@ -376,8 +381,8 @@ $table->define_columns($tablecolumns); $table->define_headers($tableheaders); $table->define_baseurl($baseurl->out()); -if (!isset($hiddenfields['lastaccess'])) { - $table->sortable(true, 'lastaccess', SORT_DESC); +if (!isset($hiddenfields['lastcourseaccess'])) { + $table->sortable(true, 'lastcourseaccess', SORT_DESC); } else { $table->sortable(true, 'firstname', SORT_ASC); } diff --git a/user/profile.php b/user/profile.php index 8b1bb5c8801..a5de397d5fd 100644 --- a/user/profile.php +++ b/user/profile.php @@ -407,7 +407,7 @@ if (!isset($hiddenfields['firstaccess'])) { } else { $datestring = get_string("never"); } - echo html_writer::tag('dt', get_string('firstaccess')); + echo html_writer::tag('dt', get_string('firstsiteaccess')); echo html_writer::tag('dd', $datestring); } if (!isset($hiddenfields['lastaccess'])) { @@ -416,7 +416,7 @@ if (!isset($hiddenfields['lastaccess'])) { } else { $datestring = get_string("never"); } - echo html_writer::tag('dt', get_string('lastaccess')); + echo html_writer::tag('dt', get_string('lastsiteaccess')); echo html_writer::tag('dd', $datestring); } diff --git a/user/view.php b/user/view.php index 443099151b5..48e56c65a36 100644 --- a/user/view.php +++ b/user/view.php @@ -273,7 +273,7 @@ if (!isset($hiddenfields['lastaccess'])) { } else { $datestring = get_string("never"); } - echo html_writer::tag('dt', get_string('lastaccess')); + echo html_writer::tag('dt', get_string('lastcourseaccess')); echo html_writer::tag('dd', $datestring); }