MDL-39761: mod_assign: Always check submission plugin status and visibility.

2013-05-20 16:39:57 +01:00
parent 5e3412d94d
commit 7e8514c5fd
1 changed files with 4 additions and 2 deletions
@@ -2643,7 +2643,9 @@ class assign {
            // Give each submission plugin a chance to process the submission
            $plugins = $this->get_submission_plugins();
            foreach ($plugins as $plugin) {
-                $plugin->submit_for_grading();
+                if ($plugin->is_enabled() && $plugin->is_visible()) {
+                    $plugin->submit_for_grading();
+                }
            }

            $submission->status = ASSIGN_SUBMISSION_STATUS_SUBMITTED;
@@ -2921,7 +2923,7 @@ class assign {
            $allempty = true;
            $pluginerror = false;
            foreach ($this->submissionplugins as $plugin) {
-                if ($plugin->is_enabled()) {
+                if ($plugin->is_enabled() && $plugin->is_visible()) {
                    if (!$plugin->save($submission, $data)) {
                        $notices[] = $plugin->get_error();
                        $pluginerror = true;