From edd8b27923245caa4ce68fb4fcc5045658fdbb30 Mon Sep 17 00:00:00 2001 From: Zachary Durber Date: Thu, 25 Sep 2014 11:01:55 +0800 Subject: [PATCH] MDL-18229 gradebook: Fix for loading activities. Was using local variable iteminstance rather than cmid. --- grade/report/single_view/screens/user/lib.php | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/grade/report/single_view/screens/user/lib.php b/grade/report/single_view/screens/user/lib.php index 394dd447619..d011e223631 100644 --- a/grade/report/single_view/screens/user/lib.php +++ b/grade/report/single_view/screens/user/lib.php @@ -55,6 +55,12 @@ class single_view_user extends single_view_tablelike implements selectable_items $params = array('courseid' => $this->courseid); $seq = new grade_seq($this->courseid, true); + foreach ($seq->items as $key => $item) { + if (isset($item->itemmodule)) { + list($courseid, $cmid) = get_course_and_cm_from_instance($item->iteminstance, $item->itemmodule); + $seq->items[$key]->cmid = $cmid->id; + } + } $this->items = array_filter($seq->items, grade_report_single_view::filters()); @@ -95,7 +101,7 @@ class single_view_user extends single_view_tablelike implements selectable_items if ( $locked_grade || $locked_grade_item ) // check both grade and grade item. $lockicon = $OUTPUT->pix_icon('t/locked', 'grade is locked'); - $url = new moodle_url("/mod/$item->itemmodule/view.php", array('id' => $item->iteminstance)); + $url = new moodle_url("/mod/$item->itemmodule/view.php", array('id' => $item->cmid)); $line = array( $OUTPUT->action_icon($this->format_link('grade', $item->id), new pix_icon('t/editstring', get_string('filtergrades', 'gradereport_single_view', $item->get_name()))),