From f750cf712c22427eb7bbf25761b459053af84b38 Mon Sep 17 00:00:00 2001 From: Damyon Wiese Date: Thu, 14 Jun 2012 15:29:06 +0800 Subject: [PATCH] MDL-33697: Merge the 2 assignment notification types This is the best solution until Moodle supports capabilities on module message providers. Unfortunately we already changed the mod_assignment language string so I can't put an AMOS command to copy the new string. --- mod/assign/db/messages.php | 5 +---- mod/assign/lang/en/assign.php | 3 +-- mod/assign/locallib.php | 6 +++--- mod/assign/version.php | 2 +- 4 files changed, 6 insertions(+), 10 deletions(-) diff --git a/mod/assign/db/messages.php b/mod/assign/db/messages.php index d4c71fd26ff..a611f5f72ba 100644 --- a/mod/assign/db/messages.php +++ b/mod/assign/db/messages.php @@ -25,10 +25,7 @@ $messageproviders = array ( // Ordinary assignment submissions - 'assign_student_notification' => array( - ), - 'assign_grader_notification' => array( - 'capability' => 'mod/assign:grade' + 'assign_notification' => array( ) ); diff --git a/mod/assign/lang/en/assign.php b/mod/assign/lang/en/assign.php index 833276b8584..66d994ede29 100644 --- a/mod/assign/lang/en/assign.php +++ b/mod/assign/lang/en/assign.php @@ -150,8 +150,7 @@ $string['locksubmissionforstudent'] = 'Prevent any more submissions for student: $string['locksubmissions'] = 'Lock submissions'; $string['manageassignfeedbackplugins'] = 'Manage assignment feedback plugins'; $string['manageassignsubmissionplugins'] = 'Manage assignment submission plugins'; -$string['messageprovider:assign_student_notification'] = 'Assignment student notifications'; -$string['messageprovider:assign_grader_notification'] = 'Assignment grader notifications'; +$string['messageprovider:assign_notification'] = 'Assignment notifications'; $string['modulename'] = 'Assignment'; $string['modulename_help'] = 'The assignment activity module enables a teacher to communicate tasks, collect work and provide grades and feedback. diff --git a/mod/assign/locallib.php b/mod/assign/locallib.php index 4c4d157ff83..adee5765e71 100644 --- a/mod/assign/locallib.php +++ b/mod/assign/locallib.php @@ -1181,7 +1181,7 @@ class assign { // need to send this to the student $messagetype = 'feedbackavailable'; - $eventtype = 'assign_student_notification'; + $eventtype = 'assign_notification'; $updatetime = $submission->lastmodified; $modulename = get_string('modulename', 'assign'); self::send_assignment_notification($grader, $user, $messagetype, $eventtype, $updatetime, $mod, $contextmodule, $course, $modulename, $submission->name); @@ -2449,7 +2449,7 @@ class assign { return; } $user = $DB->get_record('user', array('id'=>$submission->userid), '*', MUST_EXIST); - $this->send_notification($user, $user, 'submissionreceipt', 'assign_student_notification', $submission->timemodified); + $this->send_notification($user, $user, 'submissionreceipt', 'assign_notification', $submission->timemodified); } /** @@ -2471,7 +2471,7 @@ class assign { $user = $DB->get_record('user', array('id'=>$submission->userid), '*', MUST_EXIST); if ($teachers = $this->get_graders($user->id)) { foreach ($teachers as $teacher) { - $this->send_notification($user, $teacher, 'gradersubmissionupdated', 'assign_grader_notification', $submission->timemodified); + $this->send_notification($user, $teacher, 'gradersubmissionupdated', 'assign_notification', $submission->timemodified); } } } diff --git a/mod/assign/version.php b/mod/assign/version.php index af599d0b8bf..7adab47c4a8 100644 --- a/mod/assign/version.php +++ b/mod/assign/version.php @@ -25,7 +25,7 @@ defined('MOODLE_INTERNAL') || die(); $module->component = 'mod_assign'; // Full name of the plugin (used for diagnostics) -$module->version = 2012061100; // The current module version (Date: YYYYMMDDXX) +$module->version = 2012061400; // The current module version (Date: YYYYMMDDXX) $module->requires = 2012050300; // Requires this Moodle version $module->cron = 60; -- 2.43.0