mirror_php/moodle
1
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-04-13 04:22:07 +02:00
Code Issues Projects Releases Wiki Activity

MDL-76603 mod_assign: Move advanced grading grades out of the tables

Browse Source
This commit is contained in:
Shamim Rezaie 2023-02-09 23:01:14 +11:00 committed by Jun Pataleta
parent 79491b8a90
commit 394a0520d0
4 changed files with 39 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
mod/assign/classes/output/renderer.php
View File

@ -448,6 +448,11 @@ class renderer extends \plugin_renderer_base {
$o .= \html_writer::table($t);
$o .= $this->output->box_end();
if (!empty($status->gradingcontrollergrade)) {
$o .= $this->output->heading(get_string('gradebreakdown', 'assign'), 4);
$o .= $status->gradingcontrollergrade;
}
$o .= $this->output->container_end();
return $o;
}
@ -813,13 +818,6 @@ class renderer extends \plugin_renderer_base {
$this->add_table_row_tuple($t, $cell1content, $cell2content, [], $cell2attributes);
}
// Grading criteria preview.
if (!empty($status->gradingcontrollerpreview)) {
$cell1content = get_string('gradingmethodpreview', 'assign');
$cell2content = $status->gradingcontrollerpreview;
$this->add_table_row_tuple($t, $cell1content, $cell2content, [], []);
}
// Last modified.
if ($submission) {
$cell1content = get_string('timemodified', 'assign');
@ -860,6 +858,12 @@ class renderer extends \plugin_renderer_base {
$o .= \html_writer::table($t);
$o .= $this->output->box_end();
// Grading criteria preview.
if (!empty($status->gradingcontrollerpreview)) {
$o .= $this->output->heading(get_string('gradingmethodpreview', 'assign'), 4);
$o .= $status->gradingcontrollerpreview;
}
$o .= $this->output->container_end();
return $o;
}

1
mod/assign/lang/en/assign.php
View File

@ -284,6 +284,7 @@ $string['gradedby'] = 'Graded by';
$string['gradedfollowupsubmit'] = 'Graded - follow-up submission received';
$string['gradedon'] = 'Graded on';
$string['gradebelowzero'] = 'Grade must be greater than or equal to zero.';
$string['gradebreakdown'] = 'Grade breakdown';
$string['gradeabovemaximum'] = 'Grade must be less than or equal to {$a}.';
$string['gradelocked'] = 'This grade is locked or overridden in the gradebook.';
$string['gradeoutof'] = 'Grade out of {$a}';

35
mod/assign/locallib.php
View File

@ -5446,15 +5446,19 @@ class assign {
$grader = null;
$gradingmanager = get_grading_manager($this->get_context(), 'mod_assign', 'submissions');
$gradingcontrollergrade = '';
if ($hasgrade) {
if ($controller = $gradingmanager->get_active_controller()) {
$menu = make_grades_menu($this->get_instance()->grade);
$controller->set_grade_range($menu, $this->get_instance()->grade > 0);
$gradefordisplay = $controller->render_grade($PAGE,
$grade->id,
$gradingitem,
$gradebookgrade->str_long_grade,
$cangrade);
$gradingcontrollergrade = $controller->render_grade(
$PAGE,
$grade->id,
$gradingitem,
'',
$cangrade
);
$gradefordisplay = $gradebookgrade->str_long_grade;
} else {
$gradefordisplay = $this->display_grade($gradebookgrade->grade, false);
}
@ -5481,15 +5485,18 @@ class assign {
if ($grade) {
\mod_assign\event\feedback_viewed::create_from_grade($this, $grade)->trigger();
}
$feedbackstatus = new assign_feedback_status($gradefordisplay,
$gradeddate,
$grader,
$this->get_feedback_plugins(),
$grade,
$this->get_course_module()->id,
$this->get_return_action(),
$this->get_return_params(),
$viewfullnames);
$feedbackstatus = new assign_feedback_status(
$gradefordisplay,
$gradeddate,
$grader,
$this->get_feedback_plugins(),
$grade,
$this->get_course_module()->id,
$this->get_return_action(),
$this->get_return_params(),
$viewfullnames,
$gradingcontrollergrade
);
// Show the grader's identity if 'Hide Grader' is disabled or has the 'Show Hidden Grader' capability.
$showgradername = (

7
mod/assign/renderable.php
View File

@ -291,6 +291,8 @@ class assign_feedback_status implements renderable {
public $returnparams = array();
/** @var bool canviewfullnames */
public $canviewfullnames = false;
/** @var string gradingcontrollergrade The grade information rendered by a grade controller */
public $gradingcontrollergrade;
/**
* Constructor
@ -303,6 +305,7 @@ class assign_feedback_status implements renderable {
* @param string $returnaction The action required to return to this page
* @param array $returnparams The list of params required to return to this page
* @param bool $canviewfullnames
* @param string $gradingcontrollergrade The grade information rendered by a grade controller
*/
public function __construct($gradefordisplay,
$gradeddate,
@ -312,7 +315,8 @@ class assign_feedback_status implements renderable {
$coursemoduleid,
$returnaction,
$returnparams,
$canviewfullnames) {
$canviewfullnames,
$gradingcontrollergrade = '') {
$this->gradefordisplay = $gradefordisplay;
$this->gradeddate = $gradeddate;
$this->grader = $grader;
@ -322,6 +326,7 @@ class assign_feedback_status implements renderable {
$this->returnaction = $returnaction;
$this->returnparams = $returnparams;
$this->canviewfullnames = $canviewfullnames;
$this->gradingcontrollergrade = $gradingcontrollergrade;
}
}