MDL-45758 tool_monitor: Add observers to handle user and module deletion

Original issue - MDL-47376

diff --git a/admin/tool/monitor/classes/eventobservers.php b/admin/tool/monitor/classes/eventobservers.php
index a63dc0d..27d3d29 100644 (file)
--- a/admin/tool/monitor/classes/eventobservers.php
+++ b/admin/tool/monitor/classes/eventobservers.php
@@ -187,4 +187,24 @@ class eventobservers {
         $this->count = 0;
         return $events;
     }
+
+    /**
+     * Observer that monitors user deleted event and delete user subscriptions.
+     *
+     * @param \core\event\user_deleted $event the event object.
+     */
+    public static function user_deleted(\core\event\user_deleted $event) {
+        $userid = $event->objectid;
+        subscription_manager::delete_user_subscriptions($userid);
+    }
+
+    /**
+     * Observer that monitors course module deleted event and delete user subscriptions.
+     *
+     * @param \core\event\course_module_deleted $event the event object.
+     */
+    public static function course_module_deleted(\core\event\course_module_deleted $event) {
+        $cmid = $event->contextinstanceid;
+        subscription_manager::delete_cm_subscriptions($cmid);
+    }
 }

diff --git a/admin/tool/monitor/classes/subscription_manager.php b/admin/tool/monitor/classes/subscription_manager.php
index c7b67c5..3d4d0f3 100644 (file)
--- a/admin/tool/monitor/classes/subscription_manager.php
+++ b/admin/tool/monitor/classes/subscription_manager.php
@@ -84,6 +84,30 @@ class subscription_manager {
         return $DB->delete_records('tool_monitor_subscriptions', array('id' => $subscription->id));
     }
 
+    /**
+     * Delete all subscriptions for a user.
+     *
+     * @param int $userid user id.
+     *
+     * @return mixed
+     */
+    public static function delete_user_subscriptions($userid) {
+        global $DB;
+        return $DB->delete_records('tool_monitor_subscriptions', array('userid' => $userid));
+    }
+
+    /**
+     * Delete all subscriptions for a course module.
+     *
+     * @param int $cmid cm id.
+     *
+     * @return mixed
+     */
+    public static function delete_cm_subscriptions($cmid) {
+        global $DB;
+        return $DB->delete_records('tool_monitor_subscriptions', array('cmid' => $cmid));
+    }
+
     /**
      * Delete all subscribers for a given rule.
      *

diff --git a/admin/tool/monitor/db/events.php b/admin/tool/monitor/db/events.php
index 4589c58..0b1859f 100644 (file)
--- a/admin/tool/monitor/db/events.php
+++ b/admin/tool/monitor/db/events.php
@@ -26,10 +26,19 @@
 $observers = array(
     array(
         'eventname'   => '\core\event\course_deleted',
+        'priority'    => 1,
         'callback'    => '\tool_monitor\eventobservers::course_deleted',
     ),
     array(
         'eventname'   => '*',
         'callback'    => '\tool_monitor\eventobservers::process_event',
+    ),
+    array(
+        'eventname'   => '\core\event\user_deleted',
+        'callback'    => '\tool_monitor\eventobservers::user_deleted',
+    ),
+    array(
+        'eventname'   => '\core\event\course_module_deleted',
+        'callback'    => '\tool_monitor\eventobservers::course_module_deleted',
     )
 );