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
Sync: Add Jetpack Post Trashed event #6885
Conversation
We current don't have a good way to track when the post is marked as trashed. Adding the jetpack_post_trashed event will let us know when we have post is marked as trashed.
@@ -238,14 +241,20 @@ function filter_post_content_and_add_links( $post_object ) { | |||
} | |||
|
|||
public function save_published( $new_status, $old_status, $post ) { | |||
error_log( print_r( array( $new_status, $old_status ) ,1 )); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🐼
wp_delete_post( $this->post->ID ); | ||
$this->sender->do_sync(); | ||
|
||
// Since the post status is not changing here we don't exect the post be trashed again. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exect the post be trashed again.
=> expect the post to be trashed again.
Nice work, left a couple of minor comments |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Changelog: initial commit for 4.9 release. * Changelog: add #6929 * Changelog: move old changelogs to changelog.txt * Readme: restore deleted release post link. The post is now live. * Changelog: add #6853 * Changelog: add #6856 * Changelog: add #6857 * Changelog: add #6884 * Changelog: add #6885 * Changelog: add #6892 * Changelog: add #6894 * Changelog: add #6898 * Changelog: add #6899 * Changelog: add #6900 * Changelog: add #6909 * Changelog: add #6927 * Changelog: add #6947 * Chagelog: add #6958 * Changelog: add #6961 * Changelog: add #6963 * Changelog: add #6965 * Changelog: add #6986 * Changelog: add #7000 * Changelog: add #7013 * Changelog: add #7015 * Changelog: add #7019 * Changelog: add #7028 * Changelog: add #6998 * Changelog: add #6999 * Changelog: add #7044 * Changelog: add #6881 * Changelog: add #6922 * Changelog: add #6940 * Changelog: add #6962 * Changelog: add #6942 * Changelog: add #6959 * Changelog: add #7018 * Changelog: add #6948 * Changelog: add #6657 * Changelog: add #7030 * Changelog: add #7048 * Changelog: add #7031 * Changelog: add #6990 * Changelog: add #6957 * Changelog: add #7027
We current don't have a good way to track when the post is marked as
trashed.
The jetpack_post_trashed follows the same pattern as the published post event.
Changes proposed in this Pull Request:
post is marked as trashed.
Testing instructions: