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

hang on save when a comment_text filter uses 3 args #27

Open
AssemblerManiac opened this issue Mar 16, 2019 · 0 comments
Open

hang on save when a comment_text filter uses 3 args #27

AssemblerManiac opened this issue Mar 16, 2019 · 0 comments

Comments

@AssemblerManiac
Copy link

I have added a filter for 'comment_text' based on the built in filter definition using 3 args. Your call to apply_filters in function get_comment_content, appx line 977, only sends two args to the function. This causes an internal server error on our IIS server (mismatched arg count basically).

I suggest adding a 3rd argument of an empty array (as defined in the base WP system) to solve the issue. Same issue applies to 'get_comment_text'. Any other addon that tries to use the 3rd arg will cause the same hang.

I've adjusted our code to only use 2 args, but thought you might like to know the cause of a hang problem.

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

1 participant