Skip to content

admin use only fix#1

Open
majick777 wants to merge 1 commit intomichael-cannon:masterfrom
majick777:master
Open

admin use only fix#1
majick777 wants to merge 1 commit intomichael-cannon:masterfrom
majick777:master

Conversation

@majick777
Copy link
Copy Markdown

Blackbox needs to be able to debug on the front-end, but only show for developers (ie. administrator role.)
Using is_admin() (a change from the original Blackbox) prevented this version from any front-end use at all.
This is a hacky fix after the fact, but it works. It could be better implemented in the Blackbox class.
And possibly provide a filter for allowed roles and/or capabilities so the plugin only loads when it is needed.

Blackbox needs to be able to debug on the front-end, but only show for developers (ie. administrator role.)
Using is_admin() (a change from the original Blackbox) prevented this version from any front-end use at all.
This is a hacky fix after the fact, but it works. It could be better implemented in the Blackbox class.
And possibly provide a filter for allowed roles and/or capabilities so the plugin only loads when it is needed.
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