Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
21 Jun 2017

Summary of Changes

Allow setting reset PW together with a new PW.

Testing Instructions

(- install 3.7.3-rc)

  • add a new user
  • set a PW
  • edit the user
  • set a new PW + require PW reset (like what should happen when the support is called)
  • save the user
  • notice that the require PW reset option is disabled.

Expected result

PW reset is set if selected

Actual result

PW reset is not set if we set a new PW in the backend.

Documentation Changes Required

none

Additional Infos

@mbabker The line comes from the initial PR do you know the reason why you did it that way?

avatar zero-24 zero-24 - open - 21 Jun 2017
avatar zero-24 zero-24 - change - 21 Jun 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Jun 2017
Category Libraries
avatar zero-24
zero-24 - comment - 21 Jun 2017

thanks for the report @hamby please test

avatar mbabker
mbabker - comment - 21 Jun 2017

The logic is if the account's password changes, the need to reset the password goes away.

As is, this PR will break the functionality because that line is the only spot in the CMS that sets the flag back to false. So you're going to need some extra logic somewhere to make this work with the condition you've described.

avatar zero-24
zero-24 - comment - 21 Jun 2017

Ah got it. So we need to check if the user is currently in the PW reset step or not. In the first case we need to reset the flag and in the latter case we need to keep it. Correct?

avatar mbabker
mbabker - comment - 21 Jun 2017

Why should it matter how the password reset was accomplished? As far as JUser is concerned, the user's password was reset therefore the requirement to change it is invalidated. Yes, this means a second save is required if you want to set the user password from the admin and still require that the user change their password, but without overcomplicating the code or making a potentially B/C breaking change and requiring any downstream implementor to change the flag if a password is reset (think extensions which add custom user management options that extend JUser and the core users table), I'm not sure what other options exist right now.

JUser should not have request based logic within it, so it should not be responsible for figuring out if the user profile is being updated.

avatar franz-wohlkoenig franz-wohlkoenig - change - 22 Jun 2017
Status Pending Information Required
avatar zero-24
zero-24 - comment - 11 Jul 2017

Closing based on the discussion above as expected behavior.

avatar zero-24 zero-24 - change - 11 Jul 2017
Status Information Required Closed
Closed_Date 0000-00-00 00:00:00 2017-07-11 19:59:37
Closed_By zero-24
avatar zero-24 zero-24 - close - 11 Jul 2017

Add a Comment

Login with GitHub to post a comment