MDL-45319 events: trivial changes
authorMark Nelson <markn@moodle.com>
Fri, 2 May 2014 23:39:40 +0000 (16:39 -0700)
committerMarina Glancy <marina@moodle.com>
Wed, 7 May 2014 08:28:59 +0000 (16:28 +0800)
36 files changed:
lib/classes/event/blog_association_created.php
lib/classes/event/blog_comment_created.php
lib/classes/event/blog_comment_deleted.php
lib/classes/event/cohort_created.php
lib/classes/event/cohort_deleted.php
lib/classes/event/cohort_member_added.php
lib/classes/event/cohort_member_removed.php
lib/classes/event/cohort_updated.php
lib/classes/event/course_category_deleted.php
lib/classes/event/course_content_deleted.php
lib/classes/event/course_user_report_viewed.php
lib/classes/event/email_failed.php
lib/classes/event/group_created.php
lib/classes/event/group_deleted.php
lib/classes/event/group_updated.php
lib/classes/event/grouping_created.php
lib/classes/event/grouping_deleted.php
lib/classes/event/grouping_updated.php
lib/classes/event/user_deleted.php
lib/classes/event/user_enrolment_created.php
lib/classes/event/user_enrolment_deleted.php
lib/classes/event/user_enrolment_updated.php
lib/classes/event/user_password_updated.php
lib/classes/event/webservice_login_failed.php
lib/classes/event/webservice_service_user_added.php
lib/classes/event/webservice_service_user_removed.php
mod/chat/classes/event/message_sent.php
mod/workshop/classes/event/assessable_uploaded.php
report/completion/classes/event/report_viewed.php
report/completion/classes/event/user_report_viewed.php
report/log/classes/event/report_viewed.php
report/log/classes/event/user_report_viewed.php
report/loglive/classes/event/report_viewed.php
report/outline/classes/event/report_viewed.php
report/stats/classes/event/report_viewed.php
report/stats/classes/event/user_report_viewed.php

index 9084d70..ba083cb 100644 (file)
@@ -25,8 +25,6 @@ namespace core\event;
 defined('MOODLE_INTERNAL') || die();
 
 /**
- * blog_association_created
- *
  * Class for event to be triggered when a new blog entry is associated with a context.
  *
  * @property-read array $other {
index b538106..3c68de2 100644 (file)
@@ -26,7 +26,7 @@ namespace core\event;
 defined('MOODLE_INTERNAL') || die();
 
 /**
- * The blog comment created event.
+ * The blog comment created event class.
  *
  * @package    core
  * @since      Moodle 2.7
index 6fa8774..adf5c09 100644 (file)
@@ -26,7 +26,7 @@ namespace core\event;
 defined('MOODLE_INTERNAL') || die();
 
 /**
- * The blog comment deleted event.
+ * The blog comment deleted event class.
  *
  * @package    core
  * @since      Moodle 2.7
index 6a1d237..5451a7a 100644 (file)
@@ -85,7 +85,7 @@ class cohort_created extends base {
     /**
      * Return legacy event data.
      *
-     * @return stdClass
+     * @return \stdClass
      */
     protected function get_legacy_eventdata() {
         return $this->get_record_snapshot('cohort', $this->objectid);
index 50bf442..c5f170c 100644 (file)
@@ -85,7 +85,7 @@ class cohort_deleted extends base {
     /**
      * Return legacy event data.
      *
-     * @return stdClass
+     * @return \stdClass
      */
     protected function get_legacy_eventdata() {
         return $this->get_record_snapshot('cohort', $this->objectid);
index 922c155..6a944ea 100644 (file)
@@ -86,7 +86,7 @@ class cohort_member_added extends base {
     /**
      * Return legacy event data.
      *
-     * @return stdClass
+     * @return \stdClass
      */
     protected function get_legacy_eventdata() {
         $data = new \stdClass();
index e08ece3..68b29e3 100644 (file)
@@ -87,7 +87,7 @@ class cohort_member_removed extends base {
     /**
      * Return legacy event data.
      *
-     * @return stdClass
+     * @return \stdClass
      */
     protected function get_legacy_eventdata() {
         $data = new \stdClass();
index 50705fd..83d05b5 100644 (file)
@@ -85,7 +85,7 @@ class cohort_updated extends base {
     /**
      * Return legacy event data.
      *
-     * @return stdClass
+     * @return \stdClass
      */
     protected function get_legacy_eventdata() {
         return $this->get_record_snapshot('cohort', $this->objectid);
index 446345b..5c6bbee 100644 (file)
@@ -86,7 +86,7 @@ class course_category_deleted extends base {
     /**
      * Returns the legacy event data.
      *
-     * @return coursecat the category that was deleted
+     * @return \coursecat the category that was deleted
      */
     protected function get_legacy_eventdata() {
         return $this->coursecat;
index 62c18cf..42161dd 100644 (file)
@@ -15,7 +15,7 @@
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
 /**
- * Course content_deleted event class.
+ * Course content deleted event.
  *
  * @package    core
  * @copyright  2013 Mark Nelson <markn@moodle.com>
@@ -27,7 +27,7 @@ namespace core\event;
 defined('MOODLE_INTERNAL') || die();
 
 /**
- * Course content_deleted event class.
+ * Course content deleted event class.
  *
  * @property-read array $other {
  *      Extra information about event.
index 1023dc7..c465a03 100644 (file)
@@ -106,12 +106,12 @@ class course_user_report_viewed extends \core\event\base {
         }
 
         if (empty($this->relateduserid)) {
-            throw new \coding_exception('relateduserid needs to be set.');
+            throw new \coding_exception('The \'relateduserid\' must be set.');
         }
 
         // Make sure this class is never used without proper object details.
         if (!isset($this->other['mode'])) {
-            throw new \coding_exception('mode needs to be set in $other.');
+            throw new \coding_exception('The \'mode\' value must be set in other.');
         }
     }
 }
index 4f55008..d62e2c7 100644 (file)
@@ -81,13 +81,13 @@ class email_failed extends base {
     protected function validate_data() {
         parent::validate_data();
         if (!isset($this->other['subject'])) {
-            throw new \coding_exception('The subject needs to be set in $other');
+            throw new \coding_exception('The \'subject\' value must be set in other.');
         }
         if (!isset($this->other['message'])) {
-            throw new \coding_exception('The message needs to be set in $other');
+            throw new \coding_exception('The \'message\' value must be set in other.');
         }
         if (!isset($this->other['errorinfo'])) {
-            throw new \coding_exception('The error info needs to be set in $other');
+            throw new \coding_exception('The \'errorinfo\' value must be set in other.');
         }
     }
 }
index 939291a..504ba3d 100644 (file)
@@ -47,7 +47,7 @@ class group_created extends base {
     /**
      * Legacy event data if get_legacy_eventname() is not empty.
      *
-     * @return stdClass
+     * @return \stdClass
      */
     protected function get_legacy_eventdata() {
         return $this->get_record_snapshot('groups', $this->objectid);
index 08a4c5e..22294d1 100644 (file)
@@ -47,7 +47,7 @@ class group_deleted extends base {
     /**
      * Legacy event data if get_legacy_eventname() is not empty.
      *
-     * @return stdClass
+     * @return \stdClass
      */
     protected function get_legacy_eventdata() {
         return $this->get_record_snapshot('groups', $this->objectid);
@@ -90,5 +90,4 @@ class group_deleted extends base {
         $this->data['edulevel'] = self::LEVEL_OTHER;
         $this->data['objecttable'] = 'groups';
     }
-
 }
index 64a060f..5882b55 100644 (file)
@@ -47,7 +47,7 @@ class group_updated extends base {
     /**
      * Legacy event data if get_legacy_eventname() is not empty.
      *
-     * @return stdClass
+     * @return \stdClass
      */
     protected function get_legacy_eventdata() {
         return $this->get_record_snapshot('groups', $this->objectid);
index ceb6973..7ab5f79 100644 (file)
@@ -47,7 +47,7 @@ class grouping_created extends base {
     /**
      * Legacy event data if get_legacy_eventname() is not empty.
      *
-     * @return stdClass
+     * @return \stdClass
      */
     protected function get_legacy_eventdata() {
         return $this->get_record_snapshot('groupings', $this->objectid);
@@ -90,5 +90,4 @@ class grouping_created extends base {
         $this->data['edulevel'] = self::LEVEL_OTHER;
         $this->data['objecttable'] = 'groupings';
     }
-
 }
index b5a3c3d..8a7235a 100644 (file)
@@ -54,7 +54,7 @@ class grouping_deleted extends base {
     /**
      * Legacy event data if get_legacy_eventname() is not empty.
      *
-     * @return stdClass
+     * @return \stdClass
      */
     protected function get_legacy_eventdata() {
         return $this->get_record_snapshot('groupings', $this->objectid);
@@ -97,5 +97,4 @@ class grouping_deleted extends base {
         $this->data['edulevel'] = self::LEVEL_OTHER;
         $this->data['objecttable'] = 'groupings';
     }
-
 }
index 3b5247d..2917da7 100644 (file)
@@ -90,5 +90,4 @@ class grouping_updated extends base {
         $this->data['edulevel'] = self::LEVEL_OTHER;
         $this->data['objecttable'] = 'groupings';
     }
-
 }
index 488863d..9e419ca 100644 (file)
@@ -26,7 +26,7 @@ namespace core\event;
 defined('MOODLE_INTERNAL') || die();
 
 /**
- * Event when user profile is deleted.
+ * User deleted event class.
  *
  * @property-read array $other {
  *      Extra information about event.
@@ -117,23 +117,23 @@ class user_deleted extends base {
         parent::validate_data();
 
         if (!isset($this->other['username'])) {
-            throw new \coding_exception('username must be set in $other.');
+            throw new \coding_exception('The \'username\' value must be set in other.');
         }
 
         if (!isset($this->other['email'])) {
-            throw new \coding_exception('email must be set in $other.');
+            throw new \coding_exception('The \'email\' value must be set in other.');
         }
 
         if (!isset($this->other['idnumber'])) {
-            throw new \coding_exception('idnumber must be set in $other.');
+            throw new \coding_exception('The \'idnumber\' value must be set in other.');
         }
 
         if (!isset($this->other['picture'])) {
-            throw new \coding_exception('picture must be set in $other.');
+            throw new \coding_exception('The \'picture\' value must be set in other.');
         }
 
         if (!isset($this->other['mnethostid'])) {
-            throw new \coding_exception('mnethostid must be set in $other.');
+            throw new \coding_exception('The \'mnethostid\' value must be set in other.');
         }
     }
 }
index 63364e9..3ea29bb 100644 (file)
@@ -122,7 +122,7 @@ class user_enrolment_created extends base {
         }
 
         if (!isset($this->other['enrol'])) {
-            throw new \coding_exception('The \'enrol\' value must be set in $other.');
+            throw new \coding_exception('The \'enrol\' value must be set in other.');
         }
     }
 }
index 74b33e8..30dff69 100644 (file)
@@ -115,13 +115,13 @@ class user_enrolment_deleted extends base {
     protected function validate_data() {
         parent::validate_data();
         if (!isset($this->other['userenrolment'])) {
-            throw new \coding_exception('User enrolment must be set in $other.');
+            throw new \coding_exception('The \'userenrolment\' value must be set in other.');
         }
         if (!isset($this->other['enrol'])) {
-            throw new \coding_exception('Enrolment plugin name must be set in $other.');
+            throw new \coding_exception('The \'enrol\' value must be set in other.');
         }
         if (!isset($this->relateduserid)) {
-            throw new \coding_exception('Related user id must be set.');
+            throw new \coding_exception('The \'relateduserid\' must be set.');
         }
     }
 }
index 4482d89..0570cf8 100644 (file)
@@ -108,10 +108,10 @@ class user_enrolment_updated extends base {
     protected function validate_data() {
         parent::validate_data();
         if (!isset($this->other['enrol'])) {
-            throw new \coding_exception('Enrolment plugin name must be set in $other.');
+            throw new \coding_exception('The \'enrol\' value must be set in other.');
         }
         if (!isset($this->relateduserid)) {
-            throw new \coding_exception('Related user id must be set.');
+            throw new \coding_exception('The \'relateduserid\' must be set.');
         }
     }
 }
index 46f08ca..ef7dbe8 100644 (file)
@@ -123,11 +123,11 @@ class user_password_updated extends base {
         parent::validate_data();
 
         if (!$this->relateduserid) {
-            throw new \coding_exception('relateduserid needs to be set.');
+            throw new \coding_exception('The \'relateduserid\' must be set.');
         }
 
         if (!isset($this->other['forgottenreset'])) {
-            throw new \coding_exception('forgottenreset needs to be set in $other.');
+            throw new \coding_exception('The \'forgottenreset\' value must be set in other.');
         }
     }
 }
index 07781af..e65e6e1 100644 (file)
@@ -120,7 +120,7 @@ class webservice_login_failed extends base {
         } else if (!isset($this->other['method'])) {
            throw new \coding_exception('The \'method\' value must be set in other.');
         } else if (isset($this->other['token'])) {
-           throw new \coding_exception('The token cannot be set in $other.');
+           throw new \coding_exception('The \'token\' value must not be set in other.');
         }
     }
 }
index 7aa126a..9a3f3e6 100644 (file)
@@ -89,12 +89,13 @@ class webservice_service_user_added extends base {
     /**
      * Custom validation.
      *
+     * @throws \coding_exception
      * @return void
      */
     protected function validate_data() {
         parent::validate_data();
         if (!isset($this->relateduserid)) {
-            throw new \coding_exception('The relateduserid must be set.');
+            throw new \coding_exception('The \'relateduserid\' must be set.');
         }
     }
 
index 41af410..c2248d8 100644 (file)
@@ -89,12 +89,13 @@ class webservice_service_user_removed extends base {
     /**
      * Custom validation.
      *
+     * @throws \coding_exception
      * @return void
      */
     protected function validate_data() {
         parent::validate_data();
         if (!isset($this->relateduserid)) {
-            throw new \coding_exception('The relateduserid must be set.');
+            throw new \coding_exception('The \'relateduserid\' must be set.');
         }
     }
 
index 92be40c..d695f82 100644 (file)
@@ -97,5 +97,4 @@ class message_sent extends \core\event\base {
             throw new \coding_exception('The \'relateduserid\' must be set.');
         }
     }
-
 }
index 1eb9a69..1b7ec4d 100644 (file)
@@ -135,5 +135,4 @@ class assessable_uploaded extends \core\event\assessable_uploaded {
     public function set_legacy_logdata($legacylogdata) {
         $this->legacylogdata = $legacylogdata;
     }
-
 }
index 7eb669c..d8ef4b4 100644 (file)
@@ -119,9 +119,8 @@ class report_viewed extends \core\event\base {
             throw new \coding_exception('The \'logformat\' value must be set in other.');
         }
 
-        if (!isset($this->data['relateduserid'])) {
+        if (!isset($this->relateduserid)) {
             throw new \coding_exception('The \'relateduserid\' must be set.');
         }
     }
 }
-
index bd2c860..d553306 100644 (file)
@@ -101,9 +101,8 @@ class user_report_viewed extends \core\event\base {
             throw new \coding_exception('The \'mode\' value must be set in other.');
         }
 
-        if (empty($this->data['relateduserid'])) {
+        if (empty($this->relateduserid)) {
             throw new \coding_exception('The \'relateduserid\' must be set.');
         }
     }
 }
-
index 6ca70a0..f37ae03 100644 (file)
@@ -81,4 +81,3 @@ class report_viewed extends \core\event\base {
         return new \moodle_url('/report/loglive/index.php', array('id' => $this->courseid));
     }
 }
-
index efc38d6..8bd4f55 100644 (file)
@@ -101,7 +101,7 @@ class report_viewed extends \core\event\base {
         if (empty($this->other['mode'])) {
             throw new \coding_exception('The \'mode\' value must be set in other.');
         }
-        if (empty($this->data['relateduserid'])) {
+        if (empty($this->relateduserid)) {
             throw new \coding_exception('The \'relateduserid\' must be set.');
         }
     }
index 45b4b7e..e485a79 100644 (file)
@@ -110,7 +110,7 @@ class report_viewed extends \core\event\base {
             throw new \coding_exception('The \'mode\' value must be set in other.');
         }
 
-        if (!isset($this->data['relateduserid'])) {
+        if (!isset($this->relateduserid)) {
             throw new \coding_exception('The \'relateduserid\' must be set.');
         }
     }
index ce16020..3854175 100644 (file)
@@ -90,7 +90,7 @@ class user_report_viewed extends \core\event\base {
      */
     protected function validate_data() {
         parent::validate_data();
-        if (empty($this->data['relateduserid'])) {
+        if (empty($this->relateduserid)) {
             throw new \coding_exception('The \'relateduserid\' must be set.');
         }
     }