Skip to content

Commit 6e50d00

Browse files
committed
Rename certificate issue events (#675)
1 parent cab9b4e commit 6e50d00

File tree

7 files changed

+11
-13
lines changed

7 files changed

+11
-13
lines changed

classes/certificate.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -542,7 +542,7 @@ public static function issue_certificate($certificateid, $userid) {
542542
$context = \context_module::instance($cm->id);
543543

544544
// Trigger event.
545-
$event = \mod_customcert\event\certificate_issued::create([
545+
$event = \mod_customcert\event\issue_created::create([
546546
'objectid' => $issueid,
547547
'context' => $context,
548548
'relateduserid' => $userid

classes/event/certificate_issued.php renamed to classes/event/issue_created.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
* @copyright 2025 William Entriken <@fulldecent>
3232
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
3333
*/
34-
class certificate_issued extends \core\event\base {
34+
class issue_created extends \core\event\base {
3535

3636
/**
3737
* Initialises the event.
@@ -48,7 +48,7 @@ protected function init() {
4848
* @return string The name of the event.
4949
*/
5050
public static function get_name() {
51-
return get_string('eventcertificateissued', 'mod_customcert');
51+
return get_string('eventissuecreated', 'mod_customcert');
5252
}
5353

5454
/**

classes/event/certificate_deleted.php renamed to classes/event/issue_deleted.php

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,10 @@
3131
* @copyright 2025 William Entriken <@fulldecent>
3232
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
3333
*/
34-
class certificate_deleted extends \core\event\base {
34+
class issue_deleted extends \core\event\base {
3535

3636
/**
3737
* Initialises the event.
38-
*
3938
*/
4039
protected function init() {
4140
$this->data['crud'] = 'd'; // A 'delete' operation.
@@ -49,7 +48,7 @@ protected function init() {
4948
* @return string
5049
*/
5150
public static function get_name() {
52-
return get_string('eventcertificatedeleted', 'mod_customcert');
51+
return get_string('eventissuedeleted', 'mod_customcert');
5352
}
5453

5554
/**

classes/external.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -218,13 +218,12 @@ public static function delete_issue($certificateid, $issueid) {
218218
self::validate_context($context);
219219
require_capability('mod/customcert:manage', $context);
220220

221-
// Delete the issue.
222221
// Delete the issue.
223222
$deleted = $DB->delete_records('customcert_issues', ['id' => $issue->id]);
224223

225224
// Trigger event if deletion succeeded.
226225
if ($deleted) {
227-
$event = \mod_customcert\event\certificate_deleted::create([
226+
$event = \mod_customcert\event\issue_deleted::create([
228227
'objectid' => $issue->id,
229228
'context' => $context,
230229
'relateduserid' => $issue->userid,

lang/en/customcert.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -110,14 +110,14 @@
110110
$string['eventelementcreated'] = 'Custom certificate element created';
111111
$string['eventelementdeleted'] = 'Custom certificate element deleted';
112112
$string['eventelementupdated'] = 'Custom certificate element updated';
113+
$string['eventissuecreated'] = 'Custom certificate issue created';
114+
$string['eventissuedeleted'] = 'Custom certificate issue deleted';
113115
$string['eventpagecreated'] = 'Custom certificate page created';
114116
$string['eventpagedeleted'] = 'Custom certificate page deleted';
115117
$string['eventpageupdated'] = 'Custom certificate page updated';
116118
$string['eventtemplatecreated'] = 'Custom certificate template created';
117119
$string['eventtemplatedeleted'] = 'Custom certificate template deleted';
118120
$string['eventtemplateupdated'] = 'Custom certificate template updated';
119-
$string['eventcertificateupdated'] = 'Certificate updated';
120-
$string['eventcertificatedeleted'] = 'Certificate deleted';
121121
$string['exampledatawarning'] = 'Some of these values may just be an example to ensure positioning of the elements is possible.';
122122
$string['font'] = 'Font';
123123
$string['font_help'] = 'The font used when generating this element.';

lib.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,10 +89,10 @@ function customcert_delete_instance($id) {
8989

9090
$context = context_module::instance($cm->id);
9191

92-
// Trigger certificate_deleted events for each issue.
92+
// Trigger issue_deleted events for each issue.
9393
$issues = $DB->get_records('customcert_issues', ['customcertid' => $id]);
9494
foreach ($issues as $issue) {
95-
$event = \mod_customcert\event\certificate_deleted::create([
95+
$event = \mod_customcert\event\issue_deleted::create([
9696
'objectid' => $issue->id,
9797
'context' => $context,
9898
'relateduserid' => $issue->userid,

view.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@
9393
// Trigger event.
9494
$cm = get_coursemodule_from_instance('customcert', $customcert->id, 0, false, MUST_EXIST);
9595
$context = \context_module::instance($cm->id);
96-
$event = \mod_customcert\event\certificate_deleted::create([
96+
$event = \mod_customcert\event\issue_deleted::create([
9797
'objectid' => $issue->id,
9898
'context' => $context,
9999
'relateduserid' => $issue->userid,

0 commit comments

Comments
 (0)