MDL-45319 events: added debugging messages for missing relateduserid value
authorMark Nelson <markn@moodle.com>
Wed, 7 May 2014 06:32:14 +0000 (23:32 -0700)
committerMarina Glancy <marina@moodle.com>
Wed, 7 May 2014 08:28:59 +0000 (16:28 +0800)
lib/classes/event/course_completed.php
lib/classes/event/user_created.php
lib/classes/event/user_deleted.php
lib/classes/event/user_updated.php

index 6f028ff..cd4bbdc 100644 (file)
@@ -124,6 +124,6 @@ class course_completed extends base {
     protected function validate_data() {
         parent::validate_data();
 
-        // TODO: MDL-45319 add validation of relateduserid and other['relateduserid'].
+        // TODO: MDL-45445 add validation of relateduserid and other['relateduserid'].
     }
 }
index 81310d2..b716179 100644 (file)
@@ -98,4 +98,19 @@ class user_created extends base {
     protected function get_legacy_logdata() {
         return array(SITEID, 'user', 'add', '/view.php?id='.$this->objectid, fullname($this->get_legacy_eventdata()));
     }
+
+    /**
+     * Custom validation.
+     *
+     * @throws \coding_exception
+     * @return void
+     */
+    protected function validate_data() {
+        parent::validate_data();
+
+        if (!isset($this->relateduserid)) {
+            debugging('The \'relateduserid\' value must be specified in the event.', DEBUG_DEVELOPER);
+            $this->relateduserid = $this->objectid;
+        }
+    }
 }
index 9e419ca..877ec5a 100644 (file)
@@ -116,6 +116,11 @@ class user_deleted extends base {
     protected function validate_data() {
         parent::validate_data();
 
+        if (!isset($this->relateduserid)) {
+            debugging('The \'relateduserid\' value must be specified in the event.', DEBUG_DEVELOPER);
+            $this->relateduserid = $this->objectid;
+        }
+
         if (!isset($this->other['username'])) {
             throw new \coding_exception('The \'username\' value must be set in other.');
         }
index 1cbfd08..a8ef5c5 100644 (file)
@@ -98,4 +98,19 @@ class user_updated extends base {
     protected function get_legacy_logdata() {
         return array(SITEID, 'user', 'update', 'view.php?id='.$this->objectid, '');
     }
+
+    /**
+     * Custom validation.
+     *
+     * @throws \coding_exception
+     * @return void
+     */
+    protected function validate_data() {
+        parent::validate_data();
+
+        if (!isset($this->relateduserid)) {
+            debugging('The \'relateduserid\' value must be specified in the event.', DEBUG_DEVELOPER);
+            $this->relateduserid = $this->objectid;
+        }
+    }
 }