Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed for associative array of roles #27

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fixed for associative array of roles #27

wants to merge 4 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Nov 5, 2018

No description provided.

Copy link

@hm-linter hm-linter bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Linting failed (8 errors).

Chad added 3 commits November 5, 2018 12:13
If both $roles['sites'] and $roles['network'] are being used, thekey $network_roles stays blank and the update fails to add roles.
We found a bug that brought users to the last blog in their blogs list upon login instead of logging them into the blog they first visited.  This code addition fixes that.
@ghost
Copy link
Author

ghost commented Nov 21, 2018

Added restore_current_blog() which fixes login redirecting to wrong site.

@paulgibbs
Copy link
Contributor

I appreciate this is an old PR @chadsterBAM but thank you for submitting it. Can you link to an issue explaining what the problem you're trying to fix is, please?

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

Successfully merging this pull request may close these issues.

1 participant