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

Add wp_stream_is_record_excluded filter #921

Merged

Conversation

Projects
None yet
2 participants
@stephenharris
Copy link
Contributor

commented Apr 21, 2017

This is a feature request. I would like to hard-code the logic of whether a record is excluded from the log. The context is that the plug-in will be mu-installed on multiple multisites, but individual blog admins will not be able to view/configure the settings (or indeed, the logs themselves).

The plug-in is mu-installed rather than network activated because there are a number of different networks, each of which needs to be configured identically, and an mu-plugin with hard-coded settings is the most straightforward way of achieving this.

I could use wp_stream_record_array but that feels too low-level.

The PR has made some minor changes to insert the filter, including renaming a local variable for clarity.

Add wp_stream_is_record_excluded to filter whether the record should …
…be excluded form the log.

Minor refactoring of the code to faciliate the filter. Renames local variable for clarity.

@lukecarbis lukecarbis merged commit d45ea19 into xwp:develop Sep 7, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.