commit e17ab378c7154d3e2cd2e32ffb0d31794af43cec Author: Ashley Holman Date: Thu May 16 12:44:11 2013 +0930 FLO-329: mod/assign: Fix missing capability check for "grand extension" feature Note: not a security issue - was just showing an erroneous UI option. If they didn't have the cap they would subsequently get an error upon trying to set an extension. diff --git a/mod/assign/gradingbatchoperationsform.php b/mod/assign/gradingbatchoperationsform.php index e2a7284..0f1aef8 100644 --- a/mod/assign/gradingbatchoperationsform.php +++ b/mod/assign/gradingbatchoperationsform.php @@ -52,7 +52,7 @@ class mod_assign_grading_batch_operations_form extends moodleform { if ($instance['submissiondrafts']) { $options['reverttodraft'] = get_string('reverttodraft', 'assign'); } - if ($instance['duedate']) { + if ($instance['duedate'] && has_capability('mod/assign:grantextension', $instance['context'])) { $options['grantextension'] = get_string('grantextension', 'assign'); } diff --git a/mod/assign/locallib.php b/mod/assign/locallib.php index 511f745..6e79f54 100644 --- a/mod/assign/locallib.php +++ b/mod/assign/locallib.php @@ -2574,7 +2574,8 @@ class assign { array('cm'=>$this->get_course_module()->id, 'submissiondrafts'=>$this->get_instance()->submissiondrafts, 'duedate'=>$this->get_instance()->duedate, - 'feedbackplugins'=>$this->get_feedback_plugins()), + 'feedbackplugins'=>$this->get_feedback_plugins(), + 'context'=>$this->context), 'post', '', array('class'=>'gradingbatchoperationsform')); @@ -2828,7 +2829,8 @@ class assign { array('cm'=>$this->get_course_module()->id, 'submissiondrafts'=>$this->get_instance()->submissiondrafts, 'duedate'=>$this->get_instance()->duedate, - 'feedbackplugins'=>$this->get_feedback_plugins()), + 'feedbackplugins'=>$this->get_feedback_plugins(), + 'context'=>$this->context), 'post', '', array('class'=>'gradingbatchoperationsform'));