MAGETWO-92720: E-mail admin users when a new administrator is created.

Author: Oleksandr Gorkun

app/code/Magento/User/Controller/Adminhtml/Auth/Forgotpassword.php

@@ -19,6 +19,8 @@
 use Magento\User\Model\Spi\NotificatorInterface;
 
 /**
+ * Initiate forgot-password process.
+ *
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  */
 class Forgotpassword extends Auth

app/code/Magento/User/Controller/Adminhtml/User/Save.php

@@ -12,6 +12,8 @@
 use Magento\User\Model\Spi\NotificationExceptionInterface;
 
 /**
+ * Save admin user.
+ *
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  */
 class Save extends \Magento\User\Controller\Adminhtml\User
@@ -37,7 +39,7 @@ private function getSecurityCookie()
     }
 
     /**
-     * @return void
+     * @inheritDoc
      * @SuppressWarnings(PHPMD.CyclomaticComplexity)
      * @SuppressWarnings(PHPMD.NPathComplexity)
      */
@@ -119,6 +121,8 @@ public function execute()
     }
 
     /**
+     * Redirect to Edit form.
+     *
      * @param \Magento\User\Model\User $model
      * @param array $data
      * @return void

app/code/Magento/User/Model/Spi/NotificatorInterface.php

@@ -39,7 +39,7 @@ public function sendCreated(UserInterface $user): void;
      * Send a notification when a user is updated.
      *
      * @param UserInterface $user The user updated.
-     * @param string[] List of changed properties.
+     * @param string[] $changed List of changed properties.
      * @throws NotificationExceptionInterface
      *
      * @return void

app/code/Magento/User/Model/User.php

@@ -199,6 +199,8 @@ protected function _construct()
     }
 
     /**
+     * Removing dependencies and leaving only entity's properties.
+     *
      * @return string[]
      */
     public function __sleep()
@@ -221,6 +223,8 @@ public function __sleep()
     }
 
     /**
+     * Restoring required objects after serialization.
+     *
      * @return void
      */
     public function __wakeup()
@@ -407,7 +411,7 @@ public function deleteFromRole()
     }
 
     /**
-     * Check if such combination role/user exists
+     * Check if such combination role/user exists.
      *
      * @return bool
      */
@@ -419,6 +423,7 @@ public function roleUserExists()
 
     /**
      * Send email with reset password confirmation link.
+     *
      * @deprecated
      * @see NotificatorInterface::sendForgotPassword()
      *
@@ -738,143 +743,143 @@ public function setHasAvailableResources($hasResources)
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function getFirstName()
     {
         return $this->_getData('firstname');
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function setFirstName($firstName)
     {
         return $this->setData('firstname', $firstName);
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function getLastName()
     {
         return $this->_getData('lastname');
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function setLastName($lastName)
     {
         return $this->setData('lastname', $lastName);
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function getEmail()
     {
         return $this->_getData('email');
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function setEmail($email)
     {
         return $this->setData('email', $email);
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function getUserName()
     {
         return $this->_getData('username');
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function setUserName($userName)
     {
         return $this->setData('username', $userName);
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function getPassword()
     {
         return $this->_getData('password');
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function setPassword($password)
     {
         return $this->setData('password', $password);
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function getCreated()
     {
         return $this->_getData('created');
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function setCreated($created)
     {
         return $this->setData('created', $created);
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function getModified()
     {
         return $this->_getData('modified');
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function setModified($modified)
     {
         return $this->setData('modified', $modified);
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function getIsActive()
     {
         return $this->_getData('is_active');
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function setIsActive($isActive)
     {
         return $this->setData('is_active', $isActive);
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function getInterfaceLocale()
     {
         return $this->_getData('interface_locale');
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritDoc
      */
     public function setInterfaceLocale($interfaceLocale)
     {