From: David Mudrak <david.mudrak@gmail.com>
Date: Mon, 4 Jan 2010 18:28:46 +0000 (+0000)
Subject: workshop: do not allow editing outside the given frame
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=a0aa8b2c016d9e3b04cb4eee3f5ba7ec0f58aa22;p=moodle.git

workshop: do not allow editing outside the given frame
---

diff --git a/mod/workshop/submission.php b/mod/workshop/submission.php
index b04540ab07..c0ce6c6cf2 100644
--- a/mod/workshop/submission.php
+++ b/mod/workshop/submission.php
@@ -58,6 +58,8 @@ $canviewall     = has_capability('mod/workshop:viewallsubmissions', $workshop->c
 $cansubmit      = has_capability('mod/workshop:submit', $workshop->context);
 $canoverride    = ($workshop->phase == workshop::PHASE_EVALUATION) and has_capability('mod/workshop:overridegrades', $workshop->context);
 $isreviewer     = $DB->record_exists('workshop_assessments', array('submissionid' => $submission->id, 'reviewerid' => $USER->id));
+$editable       = $workshop->submitting_allowed();
+$edit           = $editable and $edit;
 
 if ($submission->id and ($ownsubmission or $canviewall or $isreviewer)) {
     // ok you can go
@@ -168,7 +170,7 @@ if ($submission->id) {
     echo $OUTPUT->box(get_string('noyoursubmission', 'workshop'));
 }
 
-if ($ownsubmission and $workshop->submitting_allowed()) {
+if ($ownsubmission and $editable) {
     $editbutton                 = new html_form();
     $editbutton->method         = 'get';
     $editbutton->button->text   = get_string('editsubmission', 'workshop');
@@ -183,8 +185,6 @@ if (has_capability('mod/workshop:viewallassessments', $PAGE->context)) {
     $canviewallassessments = true;
 } elseif ($ownsubmission and $workshop->assessments_available()) {
     $canviewallassessments = true;
-} else {
-    $canviewallassessments = false;
 }
 
 $canviewgrades = false;