close
Skip to content

Parameter #1 ($post) of type is nullable.#3639

Open
yusuke-omae wants to merge 2 commits intoWordPress:trunkfrom
yusuke-omae:get-the-terms-param-post-nullable
Open

Parameter #1 ($post) of type is nullable.#3639
yusuke-omae wants to merge 2 commits intoWordPress:trunkfrom
yusuke-omae:get-the-terms-param-post-nullable

Conversation

@yusuke-omae
Copy link
Copy Markdown

From documentation on the return value of get_the_terms(), and the usage of get_the_category(), it may be assumed that this function may be used in $term = get_the_terms( null, 'category' );.

Trac ticket: https://core.trac.wordpress.org/ticket/57131


This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.

@audrasjb
Copy link
Copy Markdown
Contributor

Hello and thanks for the patch. Looks like there is an indentation issue in your patch, though :)

@yusuke-omae
Copy link
Copy Markdown
Author

Hello and thanks for the patch. Looks like there is an indentation issue in your patch, though :)

Thank you! I fixed it.

@yusuke-omae yusuke-omae force-pushed the get-the-terms-param-post-nullable branch from ef36822 to 0c30e95 Compare November 23, 2022 01:17
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.

2 participants