MDL-41000 Events: Fixed return values for get_name() and get_description() in existing events

Changes done for following events:
- \core_auth\event\user_loggedin
- \core\event\role_assigned
- \core\event\role_unassigned

auth/classes/event/user_loggedin.php

@@ -36,12 +36,12 @@ defined('MOODLE_INTERNAL') || die();
 class user_loggedin extends \core\event\base {
 
     /**
-     * Returns localised description of what happened.
+     * Returns non-localised event description with id's for admin use only.
      *
-     * @return \lang_string.
+     * @return string
      */
     public function get_description() {
-        return new \lang_string('event_user_loggedin_desc', '', $this->get_username());
+        return 'Userid ' . $this->userid . ' has logged in';
     }
 
     /**
@@ -57,10 +57,10 @@ class user_loggedin extends \core\event\base {
     /**
      * Return localised event name.
      *
-     * @return \lang_string
+     * @return string
      */
     public static function get_name() {
-        return new \lang_string('event_user_loggedin');
+        return new get_string('event_user_loggedin', 'auth');
     }
 
     /**

lang/en/auth.php

@@ -82,7 +82,6 @@ $string['errorminpasswordnonalphanum'] = 'Passwords must have at least {$a} non-
 $string['errorminpasswordupper'] = 'Passwords must have at least {$a} upper case letter(s).';
 $string['errorpasswordupdate'] = 'Error updating password, password not changed';
 $string['event_user_loggedin'] = 'User has logged in';
-$string['event_user_loggedin_desc'] = 'User {$a} has logged in';
 $string['forcechangepassword'] = 'Force change password';
 $string['forcechangepasswordfirst_help'] = 'Force users to change password on their first login to Moodle.';
 $string['forcechangepassword_help'] = 'Force users to change password on their next login to Moodle.';

lang/en/role.php

@@ -181,6 +181,8 @@ $string['errorbadrolename'] = 'Incorrect role name';
 $string['errorbadroleshortname'] = 'Incorrect role short name';
 $string['errorexistsrolename'] = 'Role name already exists';
 $string['errorexistsroleshortname'] = 'Role name already exists';
+$string['eventroleassigned'] = 'Role assigned';
+$string['eventroleunassigned'] = 'Role unassigned';
 $string['existingadmins'] = 'Current site administrators';
 $string['existingusers'] = '{$a} existing users';
 $string['explanation'] = 'Explanation';

lib/classes/event/role_assigned.php

@@ -35,20 +35,18 @@ class role_assigned extends base {
     /**
      * Returns localised general event name.
      *
-     * @return string|\lang_string
+     * @return string
      */
     public static function get_name() {
-        //TODO: MDL-37658 localise
-        return 'Role assigned';
+        return get_string('eventroleassigned', 'role');
     }
 
     /**
-     * Returns localised description of what happened.
+     * Returns non-localised event description with id's for admin use only.
      *
-     * @return string|\lang_string
+     * @return string
      */
     public function get_description() {
-        //TODO: MDL-37658 localise
         return 'Role '.$this->objectid.' was assigned to user '.$this->relateduserid.' in context '.$this->contextid;
     }
 

lib/classes/event/role_unassigned.php

@@ -35,20 +35,18 @@ class role_unassigned extends base {
     /**
      * Returns localised general event name.
      *
-     * @return string|\lang_string
+     * @return string
      */
     public static function get_name() {
-        //TODO: MDL-37658 localise
-        return 'Role unassigned';
+        return get_string('eventroleunassigned', 'role');
     }
 
     /**
-     * Returns localised description of what happened.
+     * Returns non-localised event description with id's for admin use only.
      *
-     * @return string|\lang_string
+     * @return string
      */
     public function get_description() {
-        //TODO: MDL-37658 localise
         return 'Role '.$this->objectid.' was unassigned from user '.$this->relateduserid.' in context '.$this->contextid;
     }