Wordlift\Dataset\Sync_User_Hooks

Complexity

Total Complexity

11

Size/Duplication

Total Lines	97
Duplicated Lines	28.87 %

Coupling/Cohesion

Components	2
Dependencies	3

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	10	10	1
A	register_hooks()	0	12	1
A	changed_user()	0	5	1
A	changed_user_meta()	0	22	2
A	sync()	0	3	1
A	do_sync()	9	9	2
A	delete_user()	0	3	1
A	do_delete()	8	8	2

<?php

namespace Wordlift\Dataset;

use Wordlift\Object_Type_Enum;

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

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

	/**
	 * Sync_User_Hooks constructor.
	 *
	 * @param Sync_Service $sync_service
	 */
	function __construct( Sync_Service $sync_service ) {

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->register_hooks();

	}

	private function register_hooks() {
		/**
		 * Register hooks for user and meta.
		 */
		add_action( 'user_register', array( $this, 'changed_user' ) );
		add_action( 'profile_update', array( $this, 'changed_user' ) );
		add_action( 'added_user_meta', array( $this, 'changed_user_meta' ), 10, 3 );
		add_action( 'updated_user_meta', array( $this, 'changed_user_meta' ), 10, 3 );
		add_action( 'deleted_user_meta', array( $this, 'changed_user_meta' ), 10, 3 );
		add_action( 'delete_user', array( $this, 'delete_user' ) );

	}

	public function changed_user( $user_id ) {

		$this->sync( $user_id );

	}

	public function changed_user_meta( $meta_id, $user_id, $meta_key ) {

		if ( in_array( $meta_key,
			apply_filters( 'wl_dataset__sync_user_hooks__ignored_meta_keys',
				apply_filters( 'wl_dataset__sync_hooks__ignored_meta_keys',
					array(
						'rich_editing',
						'comment_shortcuts',
						'admin_color',
						'use_ssl',
						'show_admin_bar_front',
						'wptests_capabilities',
						'wptests_user_level',
						'dismissed_wp_pointers',
						'entity_url',
					) ) ) ) ) {
			return;
		}

		$this->sync( $user_id );

	}

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

	public function do_sync( $user_id ) {

This method seems to be duplicated in your project.

		try {
			$this->sync_service->sync_one( Object_Type_Enum::USER, (int) $user_id );
		} catch ( \Exception $e ) {
			$this->log->error( "An error occurred while trying to sync user $user_id: " . $e->getMessage(), $e );
		}

	}

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

	public function do_delete( $user_id ) {

This method seems to be duplicated in your project.

		try {
			$this->sync_service->delete_one( Object_Type_Enum::USER, $user_id );
		} catch ( \Exception $e ) {
			$this->log->error( "An error occurred while trying to delete user $user_id: " . $e->getMessage(), $e );
		}

	}

}