Verified Commit 07635356 authored by Alex Pott's avatar Alex Pott
Browse files

Issue #3137119 by munish.kumar, johnwebdev, Jaypan, jungle, xjm, catch:... 

Issue #3137119 by munish.kumar, johnwebdev, Jaypan, jungle, xjm, catch: User::setExistingPassword() does not return $this for chaining

(cherry picked from commit 51e7a083)
parent 17381d94

core/modules/user/src/Entity/User.php

+1 −0
Original line number Diff line number Diff line
@@ -398,6 +398,7 @@ public function setUsername($username) { 
   */

  public function setExistingPassword($password) {

    $this->get('pass')->existing = $password;

    return $this;

  }



  /**

core/modules/user/tests/src/Kernel/UserEntityTest.php

+12 −0
Original line number Diff line number Diff line
@@ -91,4 +91,16 @@ public function testUserValidation() { 
    $this->assertFalse((bool) $violations->count());

  }



  /**

   * Tests that ::existingPassword can be used for chaining.

   */

  public function testChainExistingPasswordMethod() {

    /** @var \Drupal\user\Entity\User $user */

    $user = User::create([

      'name' => $this->randomMachineName(),

    ]);

    $user = $user->setExistingPassword('existing_pass');

    $this->assertInstanceOf(User::class, $user);

  }



}