Skip to content

Check if get_parent_post exists as it may throw a fatal error on WP 5.7#48

Closed
audrasjb wants to merge 1 commit intojamiechong:masterfrom
audrasjb:fix/get_parent_post-wp57-issue
Closed

Check if get_parent_post exists as it may throw a fatal error on WP 5.7#48
audrasjb wants to merge 1 commit intojamiechong:masterfrom
audrasjb:fix/get_parent_post-wp57-issue

Conversation

@audrasjb
Copy link
Copy Markdown

@audrasjb audrasjb commented Feb 1, 2021

Hi,

WordPress 5.7 will introduce a get_parent_post() function in WP Core.
It appears that this plugin is the only one in the WordPress plugin repository to use in.
This pull request adds a function_exists check to avoid fatal errors.

For reference, see https://core.trac.wordpress.org/ticket/33045#comment:42

Thank you,
Jb

@thefrosty
Copy link
Copy Markdown
Contributor

thefrosty commented Feb 1, 2021

@audrasjb I'm not the developer, but that function in the plugin is namespaced, so it shouldn't throw a "fatal". I am going to keep an eye on this as I was looking to see why there are 404 errors to their "pro" site which seems to be no longer accessible.

@jamiechong
Copy link
Copy Markdown
Owner

@audrasjb Indeed @thefrosty is correct in that the get_parent_post function is not the Wordpress core function, but rather a namespaced function specific to Revisionize. I'm going to close this PR since it's not applicable. Thanks for contributing however!

@thefrosty the 404 errors should hopefully be resolved now.

@jamiechong jamiechong closed this Feb 8, 2021
@audrasjb
Copy link
Copy Markdown
Author

Indeed, that was a false positive in our search results 👍

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.

3 participants