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

Doc corrections: $new_post is ID, not obj or array #149

Merged
merged 3 commits into from
Jul 26, 2018

Conversation

stephanieleary
Copy link
Contributor

Doc blocks for push() and pull() specified $new_post as object or array. It should be the ID, which is what wp_insert_post() returns.

Doc blocks for push() and pull() specified $new_post as object or array. It should be the ID, which is what wp_insert_post() returns.
@barryceelen
Copy link
Member

barryceelen commented Jul 3, 2018

Awesome! Perhaps we want to rename the variables from $new_post to $new_post_id as well. The naming has been throwing me off already before...

@stephanieleary
Copy link
Contributor Author

I agree; I keep expecting to have the full object at my disposal in those filters.

@barryceelen barryceelen self-assigned this Jul 3, 2018
@tlovett1
Copy link
Member

tlovett1 commented Jul 3, 2018

@sillybean mind renaming the variables?

@stephanieleary
Copy link
Contributor Author

Let me know if I fouled up the tabs there.

@barryceelen
Copy link
Member

This looks great thanks!

@tlovett1
Copy link
Member

Awesome. Thank you!

@tlovett1 tlovett1 merged commit 7151e96 into 10up:master Jul 26, 2018
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