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

Incompatibility /w Multisite & Beaver Builder #25

Closed
raptor235 opened this issue Jul 25, 2018 · 2 comments
Closed

Incompatibility /w Multisite & Beaver Builder #25

raptor235 opened this issue Jul 25, 2018 · 2 comments

Comments

@raptor235
Copy link

We had installed this for a client a few weeks ago and it's caused us good 8 hours of research as to why administrators on some pages would get denied access to edit Beaver Builder pages...

It seems that by running

newrelic_set_user_attributes( $user->ID, '', array_shift( $user->roles ) );

You guys are unsetting user roles for that particular user... users administrator roles were being removed... after disabling that line in class-wp-nr-apm.php functionality to BB has returned to normal...

Not a great way to spend 8 hours of company time trying to track this down today :(

@oscarssanchez
Copy link
Contributor

oscarssanchez commented Jul 30, 2018

HI @raptor235 , thanks for raising this issue!

It seems that by using array_shift() to retreive the current user role, the role property ends up null if the user does not have any other roles. Using another approach to retrieve the first key of the property would probably solve this issue, that's weird though as it seems $user is used in the function scope only.

Have a great day.

@oscarssanchez
Copy link
Contributor

Hi @raptor235 , does #27 works for you?

@allan23 allan23 closed this as completed Aug 30, 2018
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

No branches or pull requests

3 participants