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

Fix for object method calls no longer showing #2

Merged
merged 10 commits into from Dec 19, 2013

Conversation

jrfnl
Copy link
Collaborator

@jrfnl jrfnl commented Dec 17, 2013

The earlier closures fix actually caused a regression. Any callbacks given in array form, such as the below examples, were no longer showing in the filters list. As this is one of the most used (and advocated) forms in plugins, this effectively rendered the plugin nearly useless. This pull request fixes that.

add_filter( 'filter_name', array( $this, 'method_name' ) );
add_filter( 'filter_name', array( __CLASS__, 'method_name' ) );

The pull request is release ready, i.e. changelog has been updated, the version numbers have been upped and I've tested my changes in WP 3.6, 3.7 and 3.8.

Adiitionally I've added two test files with example add_filter() calls which should all work & be displayed by the plugin. These do not have to be included in the actual plugin release, but are useful for testing of future changes to the filter list display function.

subharanjanm added a commit that referenced this pull request Dec 19, 2013
Fix for object method calls no longer showing
@subharanjanm subharanjanm merged commit 790ce16 into subharanjanm:master Dec 19, 2013
@subharanjanm
Copy link
Owner

Merged. Will update WordPress.Org plugin repository very soon. Thanks !!

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.

None yet

2 participants