Skip to content
This repository has been archived by the owner on Jun 15, 2020. It is now read-only.

Commit

Permalink
Implemented view.php
Browse files Browse the repository at this point in the history
  • Loading branch information
@andreev-artem
andreev-artem committed Jun 18, 2011
1 parent fdb4af1 commit cbaf375
Show file tree
Hide file tree
Showing 4 changed files with 173 additions and 90 deletions.
153 changes: 92 additions & 61 deletions locallib.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,7 @@ class att_view_page_params extends att_page_with_filter_controls {
const MODE_THIS_COURSE = 0;
const MODE_ALL_COURSES = 1;

public $student;
public $studentid;

public $mode;

Expand All @@ -205,7 +205,7 @@ public function __construct() {
public function get_significant_params() {
$params = array();

if (isset($this->student)) $params['student'] = $this->student;
if (isset($this->studentid)) $params['studentid'] = $this->studentid;
if ($this->mode != self::MODE_THIS_COURSE) $params['mode'] = $this->mode;

return $params;
Expand Down Expand Up @@ -605,13 +605,13 @@ private function calc_groupmode_sessgroupslist_currentgroup(){
}

$this->sessgroupslist = array();
if ($allowedgroups or $this->groupmode == VISIBLEGROUPS or $this->perm->can_access_all_groups()) {
$this->sessgroupslist[self::SELECTOR_ALL] = get_string('all', 'attforblock');
}
if ($this->groupmode == VISIBLEGROUPS) {
$this->sessgroupslist[self::SELECTOR_COMMON] = get_string('commonsessions', 'attforblock');
}
if ($allowedgroups) {
if ($this->groupmode == VISIBLEGROUPS or $this->perm->can_access_all_groups()) {
$this->sessgroupslist[self::SELECTOR_ALL] = get_string('all', 'attforblock');
}
if ($this->groupmode == VISIBLEGROUPS) {
$this->sessgroupslist[self::SELECTOR_COMMON] = get_string('commonsessions', 'attforblock');
}
foreach ($allowedgroups as $group) {
$this->sessgroupslist[$group->id] = format_string($group->name);
}
Expand Down Expand Up @@ -802,11 +802,7 @@ public function get_statuses($onlyvisible = true) {
global $DB;

if (!isset($this->statuses)) {
if ($onlyvisible) {
$this->statuses = $DB->get_records_select('attendance_statuses', "attendanceid = :aid AND visible = 1 AND deleted = 0", array('aid' => $this->id), 'grade DESC');
} else {
$this->statuses = $DB->get_records_select('attendance_statuses', "attendanceid = :aid AND deleted = 0", array('aid' => $this->id), 'grade DESC');
}
$this->statuses = get_statuses($this->id, $onlyvisible);
}

return $this->statuses;
Expand Down Expand Up @@ -839,24 +835,11 @@ public function get_user_stat($userid) {
return $ret;
}

public function get_user_taken_sessions_count($userid, $courseid=NULL) {
public function get_user_taken_sessions_count($userid) {
global $DB;

if (!isset($this->usertakensesscount)) {
$qry = "SELECT count(*) as cnt
FROM {attendance_log} al
JOIN {attendance_sessions} ats
ON al.sessionid = ats.id
WHERE ats.attendanceid = :aid AND
ats.sessdate >= :cstartdate AND
al.studentid = :uid";
$params = array(
'aid' => $this->id,
'cstartdate' => $this->course->startdate,
'uid' => $userid);

$this->usertakensesscount = $DB->count_records_sql($qry, $params);
}
if (!isset($this->usertakensesscount))
$this->usertakensesscount = get_user_taken_sessions_count($this->id, $this->course->startdate, $userid);

return $this->usertakensesscount;
}
Expand Down Expand Up @@ -884,28 +867,17 @@ public function get_user_statuses_stat($userid) {
return $this->userstatusesstat;
}

public function get_user_grade($userid, $courseid=NULL) {
$statistics = $this->get_user_statuses_stat($userid, $courseid);
$statuses = $this->get_statuses($courseid);

$sum = 0;
foreach ($statistics as $stat) {
$sum += $stat->stcnt * $statuses[$stat->statusid]->grade;
}

return $sum;
public function get_user_grade($userid) {
return get_user_grade($this->get_user_statuses_stat($userid), $this->get_statuses());
}

// For getting sessions count implemented simplest method - taken sessions.
// It can have error if users don't have attendance info for some sessions.
// In the future we can implement another methods:
// * all sessions between user start enrolment date and now;
// * all sessions between user start and end enrolment date.
public function get_user_max_grade($userid, $courseid=NULL) {
$takensessions = $this->get_user_taken_sessions_count($userid, $courseid);
$statuses = $this->get_statuses($courseid);

return current($statuses)->grade * $takensessions;
public function get_user_max_grade($userid) {
return get_user_max_grade($this->get_user_taken_sessions_count($userid), $this->get_statuses());
}

public function get_user_filtered_sessions_log($userid) {
Expand Down Expand Up @@ -938,31 +910,90 @@ public function get_user_filtered_sessions_log($userid) {

return $sessions;
}
}

public function get_user_courses_with_attendance($userid) {
global $DB;

// we can't get user courses based only on attendance_log information
// because of then users will see only courses with taked attendance
$usercourses = enrol_get_users_courses($userid);
function get_statuses($attid, $onlyvisible=true) {
global $DB;

list($usql, $uparams) = $DB->get_in_or_equal(array_keys($usercourses), SQL_PARAMS_NAMED, 'cid0');
if ($onlyvisible) {
$statuses = $DB->get_records_select('attendance_statuses', "attendanceid = :aid AND visible = 1 AND deleted = 0", array('aid' => $attid), 'grade DESC');
} else {
$statuses = $DB->get_records_select('attendance_statuses', "attendanceid = :aid AND deleted = 0", array('aid' => $attid), 'grade DESC');
}

$sql = "SELECT ats.courseid, course.fullname
FROM {attendance_sessions} ats
JOIN {attendance_log} al
ON ats.id = al.sessionid AND al.studentid = :uid
JOIN {course} course
ON ats.courseid = course.id
WHERE ats.courseid $usql
GROUP BY ats.courseid
ORDER BY course.fullname ASC";
return $statuses;
}

$params = array_merge($uparams, array('uid' => $userid));
function get_user_taken_sessions_count($attid, $coursestartdate, $userid) {
global $DB;

$qry = "SELECT count(*) as cnt
FROM {attendance_log} al
JOIN {attendance_sessions} ats
ON al.sessionid = ats.id
WHERE ats.attendanceid = :aid AND
ats.sessdate >= :cstartdate AND
al.studentid = :uid";
$params = array(
'aid' => $attid,
'cstartdate' => $coursestartdate,
'uid' => $userid);

return $DB->count_records_sql($qry, $params);
}

return $DB->get_records_sql($sql, $params);
function get_user_statuses_stat($attid, $coursestartdate, $userid) {
global $DB;

$qry = "SELECT al.statusid, count(al.statusid) AS stcnt
FROM {attendance_log} al
JOIN {attendance_sessions} ats
ON al.sessionid = ats.id
WHERE ats.attendanceid = :aid AND
ats.sessdate >= :cstartdate AND
al.studentid = :uid
GROUP BY al.statusid";
$params = array(
'aid' => $attid,
'cstartdate' => $coursestartdate,
'uid' => $userid);

return $DB->get_records_sql($qry, $params);
}

function get_user_grade($userstatusesstat, $statuses) {
$sum = 0;
foreach ($userstatusesstat as $stat) {
$sum += $stat->stcnt * $statuses[$stat->statusid]->grade;
}

return $sum;
}

function get_user_max_grade($sesscount, $statuses) {
reset($statuses);
return current($statuses)->grade * $sesscount;
}

function get_user_courses_attendances($userid) {
global $DB;

$usercourses = enrol_get_users_courses($userid);

list($usql, $uparams) = $DB->get_in_or_equal(array_keys($usercourses), SQL_PARAMS_NAMED, 'cid0');

$sql = "SELECT att.id as attid, att.course as courseid, course.fullname as coursefullname,
course.startdate as coursestartdate, att.name as attname, att.grade as attgrade
FROM {attforblock} att
JOIN {course} course
ON att.course = course.id
WHERE att.course $usql
ORDER BY coursefullname ASC, attname ASC";

$params = array_merge($uparams, array('uid' => $userid));

return $DB->get_records_sql($sql, $params);
}

?>
41 changes: 37 additions & 4 deletions renderables.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -324,7 +324,7 @@ class attforblock_user_data implements renderable {

public $sessionslog;

public $courses;
public $coursesatts;

private $urlpath;
private $urlparams;
Expand All @@ -336,13 +336,13 @@ public function __construct(attforblock $att, $userid) {

$this->pageparams = $att->pageparams;

$this->statuses = $att->get_statuses();

if (!$this->decimalpoints = grade_get_setting($att->course->id, 'decimalpoints')) {
$this->decimalpoints = $CFG->grade_decimalpoints;
}

if ($this->pageparams->mode == att_view_page_params::MODE_THIS_COURSE) {
$this->statuses = $att->get_statuses();

$this->stat = $att->get_user_stat($userid);

$this->gradable = $att->grade > 0;
Expand All @@ -357,7 +357,40 @@ public function __construct(attforblock $att, $userid) {
$this->sessionslog = $att->get_user_filtered_sessions_log($userid);
}
else {
$this->courses = $att->get_user_courses_with_attendance($userid);
$this->coursesatts = get_user_courses_attendances($userid);

$this->statuses = array();
$this->stat = array();
$this->gradable = array();
$this->grade = array();
$this->maxgrade = array();
foreach ($this->coursesatts as $ca) {
$statuses = get_statuses($ca->attid);
$user_taken_sessions_count = get_user_taken_sessions_count($ca->attid, $ca->coursestartdate, $userid);
$user_statuses_stat = get_user_statuses_stat($ca->attid, $ca->coursestartdate, $userid);

$this->statuses[$ca->attid] = $statuses;

$this->stat[$ca->attid]['completed'] = $user_taken_sessions_count;
$this->stat[$ca->attid]['statuses'] = $user_statuses_stat;

$this->gradable[$ca->attid] = $ca->attgrade > 0;

if ($this->gradable[$ca->attid]) {
$this->grade[$ca->attid] = get_user_grade($user_statuses_stat, $statuses);
// For getting sessions count implemented simplest method - taken sessions.
// It can have error if users don't have attendance info for some sessions.
// In the future we can implement another methods:
// * all sessions between user start enrolment date and now;
// * all sessions between user start and end enrolment date.
$this->maxgrade[$ca->attid] = get_user_max_grade($user_taken_sessions_count, $statuses);
}
else {
//for more comfortable and universal work with arrays
$this->grade[$ca->attid] = NULL;
$this->maxgrade[$ca->attid] = NULL;
}
}
}

$this->urlpath = $att->url_view()->out_omit_querystring();
Expand Down
Loading

0 comments on commit cbaf375

Please sign in to comment.