Skip to content
This repository has been archived by the owner on Feb 4, 2021. It is now read-only.

Fix "Require Password Reset" #314

Closed
stevenrombauts opened this issue Apr 25, 2016 · 4 comments
Closed

Fix "Require Password Reset" #314

stevenrombauts opened this issue Apr 25, 2016 · 4 comments
Assignees
Labels
Milestone

Comments

@stevenrombauts
Copy link
Member

stevenrombauts commented Apr 25, 2016

When enabling the "Require Password Reset" for a user, the user can't do anything once logged in. Resulting page:

screen shot 2016-04-25 at 13 13 54

We should remove the option or make it work again.

Update: this fails with a Manager account. An administrator or super administrator works.

@stevenrombauts stevenrombauts added this to the Backlog milestone Apr 25, 2016
@johanjanssens johanjanssens modified the milestones: 1.1, Backlog Feb 27, 2017
@johanjanssens
Copy link
Member

@stevenrombauts How do I replicate this?

@stevenrombauts
Copy link
Member Author

@johanjanssens Steps to reproduce:

  1. Create a new user in Manager group
  2. Enable "Require password reset"
  3. Log in as Manager in back-end: you will get a permission error.

The screenshot above is front-end: this is working now in current master. Back-end is still an issue:

The password reset form in back-end used to be in com_admin, we moved this to
com_users. The commit above fixes the redirect to it.

The issue that remains is that a Manager does not have permission to access com_users admin by default. Should we simply allow it now, or find another fix?

@johanjanssens
Copy link
Member

@stevenrombauts I see the issues. I moved things indeed to clean up the code, lets fix this by allowing a manager to access com_users admin. Can you handle that?

@stevenrombauts
Copy link
Member Author

@johanjanssens Ok, updated the data SQL to give Managers access to com_users.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants