Skip to content

Conversation

@nuzil
Copy link
Contributor

@nuzil nuzil commented Feb 22, 2018

This PR fixing bug described in this issue:
#13768

Problem.

After redirection admin user to user account page (when user password expires), on page pops-up message with text “Something went wrong”.

Reason.

When user password expires, user object has an access only for 3 routes: 'adminhtml_system_account_index',
'adminhtml_system_account_save' and 'adminhtml_auth_logout'.

Any other request (request which lead to other route) will be redirected to “adminhtml/system_account/”. When system loads admin account page (after force password change redirection), fires ajax request (ui form component logic) which leads to route 'mui_index_render' and it redirects to “adminhtml/system_account/”. As result we get those error message and pop-up.

Fix.

Modify “Magento\User\Observer\Backend\ForceAdminPasswordChangeObserver::execute” method by adding to $actionList array 'mui_index_render' route.

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

@okorshenko okorshenko self-requested a review February 28, 2018 00:10
@magento-engcom-team
Copy link
Contributor

Hi @okorshenko, thank you for the review.
ENGCOM-587 has been created to process this Pull Request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants