Wordlift\Dataset\Sync_Post_Hooks

Complexity

Total Complexity

14

Size/Duplication

Total Lines	107
Duplicated Lines	9.35 %

Coupling/Cohesion

Components	2
Dependencies	5

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	10	10	1
A	register_hooks()	0	11	1
A	save_post()	0	9	2
A	changed_post_meta()	0	18	3
A	sync()	0	3	1
A	do_sync()	0	15	3
A	delete_post()	0	3	1
A	do_delete()	0	7	2

<?php

namespace Wordlift\Dataset;

use Wordlift\Object_Type_Enum;

class Sync_Post_Hooks extends Abstract_Sync_Hooks {
	/**
	 * @var \Wordlift_Log_Service
	 */
	private $log;

	/**
	 * @var Sync_Service
	 */
	private $sync_service;

	/**
	 * @var Sync_Object_Adapter_Factory
	 */
	private $sync_object_factory;

	/**
	 * Sync_Post_Hooks constructor.
	 *
	 * @param Sync_Service $sync_service
	 * @param Sync_Object_Adapter_Factory $sync_object_factory
	 */
	function __construct( $sync_service, $sync_object_factory ) {

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

		parent::__construct();

		$this->log = \Wordlift_Log_Service::get_logger( get_class() );

		$this->sync_service        = $sync_service;
		$this->sync_object_factory = $sync_object_factory;

		$this->register_hooks();
	}

	private function register_hooks() {
		/**
		 * Register hooks for post and meta.
		 */
		add_action( 'save_post', array( $this, 'save_post' ) );
		add_action( 'added_post_meta', array( $this, 'changed_post_meta' ), 10, 4 );
		add_action( 'updated_post_meta', array( $this, 'changed_post_meta' ), 10, 4 );
		add_action( 'deleted_post_meta', array( $this, 'changed_post_meta' ), 10, 4 );
		add_action( 'delete_post', array( $this, 'delete_post' ) );

	}

	public function save_post( $post_id ) {

		if ( ! in_array( get_post_type( $post_id ), \Wordlift_Entity_Service::valid_entity_post_types() ) ) {
			return;
		}

		$this->sync( $post_id );

	}

	public function changed_post_meta( $meta_id, $post_id, $meta_key, $_meta_value ) {

The parameter $_meta_value is not used and could be removed.

		if ( in_array( $meta_key,
				apply_filters( 'wl_dataset__sync_post_hooks__ignored_meta_keys',
					apply_filters( 'wl_dataset__sync_hooks__ignored_meta_keys',
						array(
							'_pingme',
							'_encloseme',
							'entity_url',
						) ) ) )
		     || ! in_array( get_post_type( $post_id ), \Wordlift_Entity_Service::valid_entity_post_types() )
		) {
			return;
		}

		$this->sync( $post_id );

	}

	private function sync( $post_id ) {
		$this->enqueue( array( 'do_sync', $post_id ) );
	}

	public function do_sync( $post_id ) {
		try {
			$post = get_post( $post_id );
			if ( !isset( $post ) ) {
				return;
			}
			$this->sync_service->sync_many( array(
				$this->sync_object_factory->create( Object_Type_Enum::POST, $post_id ),
				$this->sync_object_factory->create( Object_Type_Enum::USER, $post->post_author )
			) );
		} catch ( \Exception $e ) {
			$this->log->error( "An error occurred while trying to sync post $post_id: " . $e->getMessage(), $e );
		}

	}

	public function delete_post( $post_id ) {
		$this->enqueue( array( 'do_delete', $post_id ) );
	}

	public function do_delete( $post_id ) {
		try {
			$this->sync_service->delete_one( Object_Type_Enum::POST, $post_id );
		} catch ( \Exception $e ) {
			$this->log->error( "An error occurred while trying to delete post $post_id: " . $e->getMessage(), $e );
		}
	}

}