Skip to content

Commit

Permalink
MDL-65923 core grades: Add a new event for when a grade item is deleted
Browse files Browse the repository at this point in the history
  • Loading branch information
Tom Dickman committed Jul 29, 2020
1 parent 94fdac9 commit 68049ac
Show file tree
Hide file tree
Showing 4 changed files with 150 additions and 0 deletions.
1 change: 1 addition & 0 deletions lang/en/grades.php
Original file line number Diff line number Diff line change
Expand Up @@ -198,6 +198,7 @@
$string['errorupdatinggradeitemaggregationcoef'] = 'Error updating the aggregation coefficient (weight or extra credit) of grade item ID {$a->id}';
$string['eventgradedeleted'] = 'Grade deleted';
$string['eventgradeitemcreated'] = 'Grade item created';
$string['eventgradeitemdeleted'] = 'Grade item deleted';
$string['eventgradeitemupdated'] = 'Grade item updated';
$string['eventgradelettercreated'] = 'Grade letter created';
$string['eventgradeletterdeleted'] = 'Grade letter deleted';
Expand Down
67 changes: 67 additions & 0 deletions lib/classes/event/grade_item_deleted.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,67 @@
<?php
// This file is part of Moodle - http://moodle.org/
//
// Moodle is free software: you can redistribute it and/or modify
// it under the terms of the GNU General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.
//
// Moodle is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU General Public License for more details.
//
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.

/**
* Grade item deleted event.
*
* @package core
* @copyright 2020 Tom Dickman <tomdickman@catalyst-au.net>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/

namespace core\event;

/**
* Grade item deleted event class.
*
* @package core
* @copyright 2020 Tom Dickman <tomdickman@catalyst-au.net>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
class grade_item_deleted extends grade_item_created {

/**
* Init method.
*
* @return void
*/
protected function init() {
$this->data['objecttable'] = 'grade_items';
$this->data['crud'] = 'd';
$this->data['edulevel'] = self::LEVEL_OTHER;
}

/**
* Return localised event name.
*
* @return string
*/
public static function get_name() {
return get_string('eventgradeitemdeleted', 'core_grades');
}

/**
* Returns description of what happened.
*
* @return string
*/
public function get_description() {
return "The user with id '" . $this->userid . "' deleted a grade item with id '" . $this->objectid . "'" .
" of type '" . $this->other['itemtype'] . "' and name '" . $this->other['itemname'] . "'" .
" from the course with the id '" . $this->courseid . "'.";
}

}
6 changes: 6 additions & 0 deletions lib/grade/grade_item.php
Original file line number Diff line number Diff line change
Expand Up @@ -413,6 +413,12 @@ public function delete($source=null) {
$this->delete_all_grades($source);
$success = parent::delete($source);
$transaction->allow_commit();

if ($success) {
$event = \core\event\grade_item_deleted::create_from_grade_item($this);
$event->trigger();
}

return $success;
}

Expand Down
76 changes: 76 additions & 0 deletions lib/tests/event/grade_item_deleted_test.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,76 @@
<?php
// This file is part of Moodle - http://moodle.org/
//
// Moodle is free software: you can redistribute it and/or modify
// it under the terms of the GNU General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.
//
// Moodle is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU General Public License for more details.
//
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.

/**
* Grade item deleted event tests.
*
* @package core
* @category test
* @copyright 2020 Tom Dickman <tomdickman@catalyst-au.net>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/

namespace core\event;

/**
* Test for grade item deleted event.
*
* @package core
* @category test
* @copyright 2020 Tom Dickman <tomdickman@catalyst-au.net>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @coversDefaultClass \core\event\grade_item_deleted
*/
class grade_item_deleted_testcase extends \advanced_testcase {

/**
* Test the grade item deleted event.
*
* @covers ::create_from_grade_item
*/
public function test_grade_item_deleted() {
global $DB;

$this->resetAfterTest();

$course = $this->getDataGenerator()->create_course();
$gradeitemrecord = $this->getDataGenerator()->create_grade_item(['courseid' => $course->id]);
$gradeitem = \grade_item::fetch(['id' => $gradeitemrecord->id, 'courseid' => $course->id]);

$countgradeitems = $DB->count_records('grade_items');

// Trigger and capture the event for deleting a grade item.
$sink = $this->redirectEvents();
$gradeitem->delete();
$events = $sink->get_events();
$sink->close();

// Event should only be triggered once.
$this->assertCount(1, $events);
$event = reset($events);

// Expect that the grade item was deleted and the event data is valid.
$this->assertEquals($countgradeitems - 1, $DB->count_records('grade_items'));
$this->assertInstanceOf('\core\event\grade_item_deleted', $event);
$eventdata = $event->get_data();
$this->assertEquals($gradeitem->id, $eventdata['objectid']);
$this->assertEquals($gradeitem->courseid, $eventdata['courseid']);
$this->assertEquals(\context_course::instance($gradeitem->courseid)->id, $eventdata['contextid']);
$this->assertEquals($gradeitem->itemname, $eventdata['other']['itemname']);
$this->assertEquals($gradeitem->itemtype, $eventdata['other']['itemtype']);
$this->assertEquals($gradeitem->itemmodule, $eventdata['other']['itemmodule']);
}
}

0 comments on commit 68049ac

Please sign in to comment.