Jetpack_Sync_Module_Posts - Code Metrics - Inspection of "Make subscriptions work again." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — fix/send-publish-action ( 74c5e6 )

unknown

created 2016-12-15 01:17 UTC

Jetpack_Sync_Module_Posts B

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	283
Duplicated Lines	15.9 %

Coupling/Cohesion

Components	3
Dependencies	3

Importance

Changes

Metric	Value
dl	45
loc	283
rs	8.5454
c	0
b	0
f	0
wmc	49
lcom	3
cbo	3

21 Methods

Rating	Name	Duplication	Size	Complexity
A	name()	0	3	1
A	get_object_by_id()	0	7	3
A	set_defaults()	0	2	1
A	init_listeners()	0	13	1
A	init_full_sync_listeners()	0	3	1
A	init_before_send()	0	6	1
A	enqueue_full_sync_actions()	0	5	1
A	estimate_full_sync_actions()	0	8	1
A	get_where_sql()	10	10	2
A	get_full_sync_actions()	0	3	1
A	expand_wp_insert_post()	0	3	1
A	filter_blacklisted_post_types()	0	9	2
A	filter_meta()	0	7	3
A	is_whitelisted_post_meta()	0	4	2
A	is_post_type_allowed()	0	4	1
A	remove_embed()	8	8	1
A	add_embed()	8	8	1
A	save_published()	0	5	3
D	filter_post_content_and_add_links()	19	92	14
C	send_published()	0	39	7
A	expand_post_ids()	0	13	1

How to fix Duplicated Code Complexity

Duplicated Code

Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.

Common duplication problems, and corresponding solutions are:

If you have the same expression in different places: Extract expression to a method
If you have the same method in different sub-classes: Extract method, and pull up field to the parent class
If you have the same code in unrelated classes: Consider extracting the code to a new class, and injecting that class

Complex Class

Tip: Before tackling complexity, make sure that you eliminate any duplication first. This often can reduce the size of classes significantly.

Complex classes like Jetpack_Sync_Module_Posts often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.

Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.

While breaking up the class, it is a good idea to analyze how other classes use Jetpack_Sync_Module_Posts, and based on these observations, apply Extract Interface, too.

<?php

require_once dirname( __FILE__ ) . '/class.jetpack-sync-settings.php';

class Jetpack_Sync_Module_Posts extends Jetpack_Sync_Module {

	private $just_published = array();

	public function name() {
		return 'posts';
	}

	public function get_object_by_id( $object_type, $id ) {
		if ( $object_type === 'post' && $post = get_post( intval( $id ) ) ) {
			return $this->filter_post_content_and_add_links( $post );
		}

		return false;
	}

	public function set_defaults() {
	}

	public function init_listeners( $callable ) {
		add_action( 'wp_insert_post', $callable, 10, 3 );
		add_action( 'wp_insert_post', array( $this, 'send_published'), 11, 3 );
		add_action( 'deleted_post', $callable, 10 );
		add_action( 'jetpack_publicize_post', $callable );
		add_action( 'jetpack_published_post', $callable, 10, 2 );
		add_action( 'transition_post_status', array( $this, 'save_published' ), 10, 3 );
		add_filter( 'jetpack_sync_before_enqueue_wp_insert_post', array( $this, 'filter_blacklisted_post_types' ) );

		// listen for meta changes
		$this->init_listeners_for_meta_type( 'post', $callable );
		$this->init_meta_whitelist_handler( 'post', array( $this, 'filter_meta' ) );
	}

	public function init_full_sync_listeners( $callable ) {
		add_action( 'jetpack_full_sync_posts', $callable ); // also sends post meta
	}

	public function init_before_send() {
		add_filter( 'jetpack_sync_before_send_wp_insert_post', array( $this, 'expand_wp_insert_post' ) );

		// full sync
		add_filter( 'jetpack_sync_before_send_jetpack_full_sync_posts', array( $this, 'expand_post_ids' ) );
	}

	public function enqueue_full_sync_actions( $config, $max_items_to_enqueue, $state ) {
		global $wpdb;

		return $this->enqueue_all_ids_as_action( 'jetpack_full_sync_posts', $wpdb->posts, 'ID', $this->get_where_sql( $config ), $max_items_to_enqueue, $state );
	}

	public function estimate_full_sync_actions( $config ) {
		global $wpdb;

		$query = "SELECT count(*) FROM $wpdb->posts WHERE " . $this->get_where_sql( $config );
		$count = $wpdb->get_var( $query );

		return (int) ceil( $count / self::ARRAY_CHUNK_SIZE );
	}

	private function get_where_sql( $config ) {
		$where_sql = Jetpack_Sync_Settings::get_blacklisted_post_types_sql();

		// config is a list of post IDs to sync
		if ( is_array( $config ) ) {
			$where_sql   .= ' AND ID IN (' . implode( ',', array_map( 'intval', $config ) ) . ')';
$a = "a";
$ab = "ab";
$abc = "abc";
		}

		return $where_sql;
	}

	function get_full_sync_actions() {
		return array( 'jetpack_full_sync_posts' );
	}

	/**
	 * Process content before send
	 */

	function expand_wp_insert_post( $args ) {
		return array( $args[0], $this->filter_post_content_and_add_links( $args[1] ), $args[2] );
	}

	function filter_blacklisted_post_types( $args ) {
		$post = $args[1];

		if ( in_array( $post->post_type, Jetpack_Sync_Settings::get_setting( 'post_types_blacklist' ) ) ) {
			return false;
		}

		return $args;
	}

	// Meta
	function filter_meta( $args ) {
		if ( $this->is_post_type_allowed( $args[1] ) && $this->is_whitelisted_post_meta( $args[2] ) ) {
			return $args;
		}

		return false;
	}

	function is_whitelisted_post_meta( $meta_key ) {
		// _wpas_skip_ is used by publicize
		return in_array( $meta_key, Jetpack_Sync_Settings::get_setting( 'post_meta_whitelist' ) ) || wp_startswith( $meta_key, '_wpas_skip_' );
	}

	function is_post_type_allowed( $post_id ) {
		$post = get_post( $post_id );
		return ! in_array( $post->post_type, Jetpack_Sync_Settings::get_setting( 'post_types_blacklist' ) );
	}

	function remove_embed() {
		global $wp_embed;
		remove_filter( 'the_content', array( $wp_embed, 'run_shortcode' ), 8 );
		// remove the embed shortcode since we would do the part later.
		remove_shortcode( 'embed' );
		// Attempts to embed all URLs in a post
		remove_filter( 'the_content', array( $wp_embed, 'autoembed' ), 8 );
	}

	function add_embed() {
		global $wp_embed;
		add_filter( 'the_content', array( $wp_embed, 'run_shortcode' ), 8 );
		// Shortcode placeholder for strip_shortcodes()
		add_shortcode( 'embed', '__return_false' );
		// Attempts to embed all URLs in a post
		add_filter( 'the_content', array( $wp_embed, 'autoembed' ), 8 );
	}

	// Expands wp_insert_post to include filtered content
	function filter_post_content_and_add_links( $post_object ) {
		global $post, $shortcode_tags;
		$post = $post_object;

		// return non existant post 
		$post_type = get_post_type_object( $post->post_type );
		if ( empty( $post_type) || ! is_object( $post_type ) ) {
			$non_existant_post                    = new stdClass();
			$non_existant_post->ID                = $post->ID;
			$non_existant_post->post_modified     = $post->post_modified;
			$non_existant_post->post_modified_gmt = $post->post_modified_gmt;
			$non_existant_post->post_status       = 'jetpack_sync_non_registered_post_type';
			
			return $non_existant_post;
		}
		/**
		 * Filters whether to prevent sending post data to .com
		 *
		 * Passing true to the filter will prevent the post data from being sent
		 * to the WordPress.com.
		 * Instead we pass data that will still enable us to do a checksum against the
		 * Jetpacks data but will prevent us from displaying the data on in the API as well as
		 * other services.
		 * @since 4.2.0
		 *
		 * @param boolean false prevent post data from being synced to WordPress.com
		 * @param mixed $post WP_POST object
		 */
		if ( apply_filters( 'jetpack_sync_prevent_sending_post_data', false, $post ) ) {
			// We only send the bare necessary object to be able to create a checksum.
			$blocked_post                    = new stdClass();
			$blocked_post->ID                = $post->ID;
			$blocked_post->post_modified     = $post->post_modified;
			$blocked_post->post_modified_gmt = $post->post_modified_gmt;
			$blocked_post->post_status       = 'jetpack_sync_blocked';

			return $blocked_post;
		}

		// lets not do oembed just yet.
		$this->remove_embed();

		if ( 0 < strlen( $post->post_password ) ) {
			$post->post_password = 'auto-' . wp_generate_password( 10, false );
		}

		/**
		 * Filter prevents some shortcodes from expanding.
		 *
		 * Since we can can expand some type of shortcode better on the .com side and make the
		 * expansion more relevant to contexts. For example [galleries] and subscription emails
		 *
		 * @since 4.5.0
		 *
		 * @param array of shortcode tags to remove.
		 */
		$shortcodes_to_remove = apply_filters( 'jetpack_sync_do_not_expand_shortcodes', array( 'gallery', 'slideshow' ) );
		foreach ( $shortcodes_to_remove as $shortcode ) {
			if ( isset ( $shortcode_tags[ $shortcode ] )  ) {
				$shortcodes_and_callbacks_to_remove[ $shortcode ] =  $shortcode_tags[ $shortcode ];
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
			}
		}

		array_map( 'remove_shortcode' , array_keys( $shortcodes_and_callbacks_to_remove ) );
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

		/** This filter is already documented in core. wp-includes/post-template.php */
		if ( Jetpack_Sync_Settings::get_setting( 'render_filtered_content' ) && $post_type->public  ) {
			$post->post_content_filtered   = apply_filters( 'the_content', $post->post_content );
			$post->post_excerpt_filtered   = apply_filters( 'the_excerpt', $post->post_excerpt );
		}

		foreach ( $shortcodes_and_callbacks_to_remove as $shortcode => $callback ) {
			add_shortcode( $shortcode, $callback );
		}

		$this->add_embed();

		if ( has_post_thumbnail( $post->ID ) ) {
			$image_attributes = wp_get_attachment_image_src( get_post_thumbnail_id( $post->ID ), 'full' );
			if ( is_array( $image_attributes ) && isset( $image_attributes[0] ) ) {
				$post->featured_image = $image_attributes[0];
			}
		}

		$post->permalink               = get_permalink( $post->ID );
		$post->shortlink               = wp_get_shortlink( $post->ID );
		$post->dont_email_post_to_subs = Jetpack::is_module_active( 'subscriptions' ) ?
				get_post_meta( $post->ID, '_jetpack_dont_email_post_to_subs', true ) :
				true; // Don't email subscription if the subscription module is not active.

		return $post;
	}

	public function save_published( $new_status, $old_status, $post ) {
		if ( 'publish' === $new_status && 'publish' !== $old_status ) {
			$this->just_published[] = $post->ID;
		}
	}

	public function send_published( $post_ID, $post, $update ) {

		// Post revisions cause race conditions where this send_published add the action before the actual post gets synced
		if ( wp_is_post_autosave( $post ) || wp_is_post_revision( $post ) ) {
			return;
		}

		if ( ! empty( $this->just_published ) && in_array( $post_ID, $this->just_published ) ) {
			$published = array_reverse( array_unique( $this->just_published ) );
			
			// Pre 4.7 WP does not have run though send_published for every save_published call
			// So lets clear out any just_published that we recorded 
			foreach ( $published as $just_published_post_ID ) {
				if ( $post_ID !== $just_published_post_ID ) {
					$post = get_post( $just_published_post_ID );
				}

				/**
				 * Filter that is used to add to the post flags ( meta data ) when a post gets published
				 *
				 * @since 4.4.0
				 *
				 * @param mixed array post flags that are added to the post
				 * @param mixed $post WP_POST object
				 */
				$flags = apply_filters( 'jetpack_published_post_flags', array(), $post );

				/**
				 * Action that gets synced when a post type gets published.
				 *
				 * @since 4.4.0
				 *
				 * @param int post_id
				 * @param mixed array post flags that are added to the post
				 */
				do_action( 'jetpack_published_post', $just_published_post_ID, $flags );
			}
			$this->just_published = array();
		}
	}

	public function expand_post_ids( $args ) {
		$post_ids = $args[0];

		$posts = array_filter( array_map( array( 'WP_Post', 'get_instance' ), $post_ids ) );
		$posts = array_map( array( $this, 'filter_post_content_and_add_links' ), $posts );
		$posts = array_values( $posts ); // reindex in case posts were deleted

		return array(
			$posts,
			$this->get_metadata( $post_ids, 'post', Jetpack_Sync_Settings::get_setting( 'post_meta_whitelist' ) ),
			$this->get_term_relationships( $post_ids ),
		);
	}
}


1		<?php
2
3		require_once dirname( __FILE__ ) . '/class.jetpack-sync-settings.php';
4
5		class Jetpack_Sync_Module_Posts extends Jetpack_Sync_Module {
6
7		private $just_published = array();
8
9		public function name() {
10		return 'posts';
11		}
12
13		public function get_object_by_id( $object_type, $id ) {
14		if ( $object_type === 'post' && $post = get_post( intval( $id ) ) ) {
15		return $this->filter_post_content_and_add_links( $post );
16		}
17
18		return false;
19		}
20
21		public function set_defaults() {
22		}
23
24		public function init_listeners( $callable ) {
25		add_action( 'wp_insert_post', $callable, 10, 3 );
26		add_action( 'wp_insert_post', array( $this, 'send_published'), 11, 3 );
27		add_action( 'deleted_post', $callable, 10 );
28		add_action( 'jetpack_publicize_post', $callable );
29		add_action( 'jetpack_published_post', $callable, 10, 2 );
30		add_action( 'transition_post_status', array( $this, 'save_published' ), 10, 3 );
31		add_filter( 'jetpack_sync_before_enqueue_wp_insert_post', array( $this, 'filter_blacklisted_post_types' ) );
32
33		// listen for meta changes
34		$this->init_listeners_for_meta_type( 'post', $callable );
35		$this->init_meta_whitelist_handler( 'post', array( $this, 'filter_meta' ) );
36		}
37
38		public function init_full_sync_listeners( $callable ) {
39		add_action( 'jetpack_full_sync_posts', $callable ); // also sends post meta
40		}
41
42		public function init_before_send() {
43		add_filter( 'jetpack_sync_before_send_wp_insert_post', array( $this, 'expand_wp_insert_post' ) );
44
45		// full sync
46		add_filter( 'jetpack_sync_before_send_jetpack_full_sync_posts', array( $this, 'expand_post_ids' ) );
47		}
48
49		public function enqueue_full_sync_actions( $config, $max_items_to_enqueue, $state ) {
50		global $wpdb;
51
52		return $this->enqueue_all_ids_as_action( 'jetpack_full_sync_posts', $wpdb->posts, 'ID', $this->get_where_sql( $config ), $max_items_to_enqueue, $state );
53		}
54
55		public function estimate_full_sync_actions( $config ) {
56		global $wpdb;
57
58		$query = "SELECT count(*) FROM $wpdb->posts WHERE " . $this->get_where_sql( $config );
59		$count = $wpdb->get_var( $query );
60
61		return (int) ceil( $count / self::ARRAY_CHUNK_SIZE );
62		}
63
64	View Code Duplication	private function get_where_sql( $config ) {
65		$where_sql = Jetpack_Sync_Settings::get_blacklisted_post_types_sql();
66
67		// config is a list of post IDs to sync
68		if ( is_array( $config ) ) {
69		$where_sql .= ' AND ID IN (' . implode( ',', array_map( 'intval', $config ) ) . ')';
		0 ignored issues – show Coding Style introduced 2016-07-29 22:58 UTC by Report Bug Copy Issue Report Equals sign not aligned correctly; expected 1 space but found 3 spaces This check looks for improperly formatted assignments. Every assignment must have exactly one space before and one space after the equals operator. To illustrate: $a = "a"; $ab = "ab"; $abc = "abc"; will have no issues, while $a = "a"; $ab = "ab"; $abc = "abc"; will report issues in lines 1 and 2. Loading history...
70		}
71
72		return $where_sql;
73		}
74
75		function get_full_sync_actions() {
76		return array( 'jetpack_full_sync_posts' );
77		}
78
79		/**
80		* Process content before send
81		*/
82
83		function expand_wp_insert_post( $args ) {
84		return array( $args[0], $this->filter_post_content_and_add_links( $args[1] ), $args[2] );
85		}
86
87		function filter_blacklisted_post_types( $args ) {
88		$post = $args[1];
89
90		if ( in_array( $post->post_type, Jetpack_Sync_Settings::get_setting( 'post_types_blacklist' ) ) ) {
91		return false;
92		}
93
94		return $args;
95		}
96
97		// Meta
98		function filter_meta( $args ) {
99		if ( $this->is_post_type_allowed( $args[1] ) && $this->is_whitelisted_post_meta( $args[2] ) ) {
100		return $args;
101		}
102
103		return false;
104		}
105
106		function is_whitelisted_post_meta( $meta_key ) {
107		// _wpas_skip_ is used by publicize
108		return in_array( $meta_key, Jetpack_Sync_Settings::get_setting( 'post_meta_whitelist' ) ) \|\| wp_startswith( $meta_key, '_wpas_skip_' );
109		}
110
111		function is_post_type_allowed( $post_id ) {
112		$post = get_post( $post_id );
113		return ! in_array( $post->post_type, Jetpack_Sync_Settings::get_setting( 'post_types_blacklist' ) );
114		}
115
116	View Code Duplication	function remove_embed() {
117		global $wp_embed;
118		remove_filter( 'the_content', array( $wp_embed, 'run_shortcode' ), 8 );
119		// remove the embed shortcode since we would do the part later.
120		remove_shortcode( 'embed' );
121		// Attempts to embed all URLs in a post
122		remove_filter( 'the_content', array( $wp_embed, 'autoembed' ), 8 );
123		}
124
125	View Code Duplication	function add_embed() {
126		global $wp_embed;
127		add_filter( 'the_content', array( $wp_embed, 'run_shortcode' ), 8 );
128		// Shortcode placeholder for strip_shortcodes()
129		add_shortcode( 'embed', '__return_false' );
130		// Attempts to embed all URLs in a post
131		add_filter( 'the_content', array( $wp_embed, 'autoembed' ), 8 );
132		}
133
134		// Expands wp_insert_post to include filtered content
135		function filter_post_content_and_add_links( $post_object ) {
136		global $post, $shortcode_tags;
137		$post = $post_object;
138
139		// return non existant post
140		$post_type = get_post_type_object( $post->post_type );
141	View Code Duplication	if ( empty( $post_type) \|\| ! is_object( $post_type ) ) {
142		$non_existant_post = new stdClass();
143		$non_existant_post->ID = $post->ID;
144		$non_existant_post->post_modified = $post->post_modified;
145		$non_existant_post->post_modified_gmt = $post->post_modified_gmt;
146		$non_existant_post->post_status = 'jetpack_sync_non_registered_post_type';
147
148		return $non_existant_post;
149		}
150		/**
151		* Filters whether to prevent sending post data to .com
152		*
153		* Passing true to the filter will prevent the post data from being sent
154		* to the WordPress.com.
155		* Instead we pass data that will still enable us to do a checksum against the
156		* Jetpacks data but will prevent us from displaying the data on in the API as well as
157		* other services.
158		* @since 4.2.0
159		*
160		* @param boolean false prevent post data from being synced to WordPress.com
161		* @param mixed $post WP_POST object
162		*/
163	View Code Duplication	if ( apply_filters( 'jetpack_sync_prevent_sending_post_data', false, $post ) ) {
164		// We only send the bare necessary object to be able to create a checksum.
165		$blocked_post = new stdClass();
166		$blocked_post->ID = $post->ID;
167		$blocked_post->post_modified = $post->post_modified;
168		$blocked_post->post_modified_gmt = $post->post_modified_gmt;
169		$blocked_post->post_status = 'jetpack_sync_blocked';
170
171		return $blocked_post;
172		}
173
174		// lets not do oembed just yet.
175		$this->remove_embed();
176
177		if ( 0 < strlen( $post->post_password ) ) {
178		$post->post_password = 'auto-' . wp_generate_password( 10, false );
179		}
180
181		/**
182		* Filter prevents some shortcodes from expanding.
183		*
184		* Since we can can expand some type of shortcode better on the .com side and make the
185		* expansion more relevant to contexts. For example [galleries] and subscription emails
186		*
187		* @since 4.5.0
188		*
189		* @param array of shortcode tags to remove.
190		*/
191		$shortcodes_to_remove = apply_filters( 'jetpack_sync_do_not_expand_shortcodes', array( 'gallery', 'slideshow' ) );
192		foreach ( $shortcodes_to_remove as $shortcode ) {
193		if ( isset ( $shortcode_tags[ $shortcode ] ) ) {
194		$shortcodes_and_callbacks_to_remove[ $shortcode ] = $shortcode_tags[ $shortcode ];
		0 ignored issues – show Coding Style Comprehensibility introduced 2016-12-10 04:37 UTC by Report Bug Copy Issue Report `$shortcodes_and_callbacks_to_remove` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$shortcodes_and_callbacks_to_remove = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
195		}
196		}
197
198		array_map( 'remove_shortcode' , array_keys( $shortcodes_and_callbacks_to_remove ) );
		0 ignored issues – show Bug introduced 2016-12-10 04:37 UTC by Report Bug Copy Issue Report The variable `$shortcodes_and_callbacks_to_remove` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
199
200		/** This filter is already documented in core. wp-includes/post-template.php */
201		if ( Jetpack_Sync_Settings::get_setting( 'render_filtered_content' ) && $post_type->public ) {
202		$post->post_content_filtered = apply_filters( 'the_content', $post->post_content );
203		$post->post_excerpt_filtered = apply_filters( 'the_excerpt', $post->post_excerpt );
204		}
205
206		foreach ( $shortcodes_and_callbacks_to_remove as $shortcode => $callback ) {
207		add_shortcode( $shortcode, $callback );
208		}
209
210		$this->add_embed();
211
212		if ( has_post_thumbnail( $post->ID ) ) {
213		$image_attributes = wp_get_attachment_image_src( get_post_thumbnail_id( $post->ID ), 'full' );
214		if ( is_array( $image_attributes ) && isset( $image_attributes[0] ) ) {
215		$post->featured_image = $image_attributes[0];
216		}
217		}
218
219		$post->permalink = get_permalink( $post->ID );
220		$post->shortlink = wp_get_shortlink( $post->ID );
221		$post->dont_email_post_to_subs = Jetpack::is_module_active( 'subscriptions' ) ?
222		get_post_meta( $post->ID, '_jetpack_dont_email_post_to_subs', true ) :
223		true; // Don't email subscription if the subscription module is not active.
224
225		return $post;
226		}
227
228		public function save_published( $new_status, $old_status, $post ) {
229		if ( 'publish' === $new_status && 'publish' !== $old_status ) {
230		$this->just_published[] = $post->ID;
231		}
232		}
233
234		public function send_published( $post_ID, $post, $update ) {
		0 ignored issues – show Unused Code introduced 2016-11-10 05:58 UTC by Report Bug Copy Issue Report The parameter `$update` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
235		// Post revisions cause race conditions where this send_published add the action before the actual post gets synced
236		if ( wp_is_post_autosave( $post ) \|\| wp_is_post_revision( $post ) ) {
237		return;
238		}
239
240		if ( ! empty( $this->just_published ) && in_array( $post_ID, $this->just_published ) ) {
241		$published = array_reverse( array_unique( $this->just_published ) );
242
243		// Pre 4.7 WP does not have run though send_published for every save_published call
244		// So lets clear out any just_published that we recorded
245		foreach ( $published as $just_published_post_ID ) {
246		if ( $post_ID !== $just_published_post_ID ) {
247		$post = get_post( $just_published_post_ID );
248		}
249
250		/**
251		* Filter that is used to add to the post flags ( meta data ) when a post gets published
252		*
253		* @since 4.4.0
254		*
255		* @param mixed array post flags that are added to the post
256		* @param mixed $post WP_POST object
257		*/
258		$flags = apply_filters( 'jetpack_published_post_flags', array(), $post );
259
260		/**
261		* Action that gets synced when a post type gets published.
262		*
263		* @since 4.4.0
264		*
265		* @param int post_id
266		* @param mixed array post flags that are added to the post
267		*/
268		do_action( 'jetpack_published_post', $just_published_post_ID, $flags );
269		}
270		$this->just_published = array();
271		}
272		}
273
274		public function expand_post_ids( $args ) {
275		$post_ids = $args[0];
276
277		$posts = array_filter( array_map( array( 'WP_Post', 'get_instance' ), $post_ids ) );
278		$posts = array_map( array( $this, 'filter_post_content_and_add_links' ), $posts );
279		$posts = array_values( $posts ); // reindex in case posts were deleted
280
281		return array(
282		$posts,
283		$this->get_metadata( $post_ids, 'post', Jetpack_Sync_Settings::get_setting( 'post_meta_whitelist' ) ),
284		$this->get_term_relationships( $post_ids ),
285		);
286		}
287		}
288

Automattic / jetpack