Jetpack_Sync_Module_Callables::unlock_sync_callable() - Code Metrics - Inspection of "Add translations." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — add/sync-status-to-debugger ( 8d9a33...da1c65 )

unknown

created 2016-07-28 18:22 UTC

unlock_sync_callable() A

↳ Parent: Jetpack_Sync_Module_Callables

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	1

Metric	Value
cc	1
eloc	2
c	2
b	0
f	1
nc	1
nop	0
dl	0
loc	3
rs	10

<?php

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

class Jetpack_Sync_Module_Callables extends Jetpack_Sync_Module {
	const CALLABLES_CHECKSUM_OPTION_NAME = 'jetpack_callables_sync_checksum';
	const CALLABLES_AWAIT_TRANSIENT_NAME = 'jetpack_sync_callables_await';

	private $callable_whitelist;

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

	public function set_defaults() {
		if ( is_multisite() ) {
			$this->callable_whitelist = array_merge( Jetpack_Sync_Defaults::$default_callable_whitelist, Jetpack_Sync_Defaults::$default_multisite_callable_whitelist );

		} else {
			$this->callable_whitelist = Jetpack_Sync_Defaults::$default_callable_whitelist;

		}
	}

	public function init_listeners( $callable ) {
		add_action( 'jetpack_sync_callable', $callable, 10, 2 );

		// full sync
		add_action( 'jetpack_full_sync_callables', $callable );

		// always send change to active modules right away
		add_action( 'update_option_jetpack_active_modules', array( $this, 'unlock_sync_callable' ) );

		// get_plugins and wp_version
		// gets fired when new code gets installed, updates etc.
		add_action( 'upgrader_process_complete', array( $this, 'unlock_sync_callable' ) );
	}

	public function init_full_sync_listeners( $callable ) {
		add_action( 'jetpack_full_sync_callables', $callable );
	}

	public function init_before_send() {
		add_action( 'jetpack_sync_before_send_queue_sync', array( $this, 'maybe_sync_callables' ) );

		// full sync
		add_filter( 'jetpack_sync_before_send_jetpack_full_sync_callables', array( $this, 'expand_callables' ) );
	}

	public function reset_data() {
		delete_option( self::CALLABLES_CHECKSUM_OPTION_NAME );
		delete_transient( self::CALLABLES_AWAIT_TRANSIENT_NAME );
	}

	function set_callable_whitelist( $callables ) {
		$this->callable_whitelist = $callables;
	}

	function get_callable_whitelist() {
		return $this->callable_whitelist;
	}

	public function get_all_callables() {
		// get_all_callables should run as the master user always.
		$current_user_id = get_current_user_id();
		wp_set_current_user( Jetpack_Options::get_option( 'master_user' ) );
		$callables = array_combine(
			array_keys( $this->callable_whitelist ),
			array_map( array( $this, 'get_callable' ), array_values( $this->callable_whitelist ) )
		);
		wp_set_current_user( $current_user_id );

		return $callables;
	}

	private function get_callable( $callable ) {
		return call_user_func( $callable );
	}

	public function enqueue_full_sync_actions() {
		/**
		 * Tells the client to sync all callables to the server
		 *
		 * @since 4.2.0
		 *
		 * @param boolean Whether to expand callables (should always be true)
		 */
		do_action( 'jetpack_full_sync_callables', true );

		return 1; // The number of actions enqueued
	}

	public function get_full_sync_actions() {
		return array( 'jetpack_full_sync_callables' );
	}

	public function unlock_sync_callable() {
		delete_transient( self::CALLABLES_AWAIT_TRANSIENT_NAME );
	}
	
	public function maybe_sync_callables() {
		if ( get_transient( self::CALLABLES_AWAIT_TRANSIENT_NAME ) ) {
			return;
		}

		$callables = $this->get_all_callables();

		if ( empty( $callables ) ) {
			return;
		}

		set_transient( self::CALLABLES_AWAIT_TRANSIENT_NAME, microtime( true ), Jetpack_Sync_Defaults::$default_sync_callables_wait_time );


		$callable_checksums = (array) get_option( self::CALLABLES_CHECKSUM_OPTION_NAME, array() );

		// only send the callables that have changed
		foreach ( $callables as $name => $value ) {
			$checksum = $this->get_check_sum( $value );
			// explicitly not using Identical comparison as get_option returns a string
			if ( ! $this->still_valid_checksum( $callable_checksums, $name, $checksum ) && ! is_null( $value ) ) {
				/**
				 * Tells the client to sync a callable (aka function) to the server
				 *
				 * @since 4.2.0
				 *
				 * @param string The name of the callable
				 * @param mixed The value of the callable
				 */
				do_action( 'jetpack_sync_callable', $name, $value );
				$callable_checksums[ $name ] = $checksum;
			} else {
				$callable_checksums[ $name ] = $checksum;
			}
		}
		update_option( self::CALLABLES_CHECKSUM_OPTION_NAME, $callable_checksums );
	}

	public function expand_callables( $args ) {
		if ( $args[0] ) {
			return $this->get_all_callables();
		}

		return $args;
	}
}


1		<?php
2
3		require_once dirname( __FILE__ ) . '/class.jetpack-sync-functions.php';
4
5		class Jetpack_Sync_Module_Callables extends Jetpack_Sync_Module {
6		const CALLABLES_CHECKSUM_OPTION_NAME = 'jetpack_callables_sync_checksum';
7		const CALLABLES_AWAIT_TRANSIENT_NAME = 'jetpack_sync_callables_await';
8
9		private $callable_whitelist;
10
11		public function name() {
12		return 'functions';
13		}
14
15		public function set_defaults() {
16		if ( is_multisite() ) {
17		$this->callable_whitelist = array_merge( Jetpack_Sync_Defaults::$default_callable_whitelist, Jetpack_Sync_Defaults::$default_multisite_callable_whitelist );
		0 ignored issues – show Bug introduced 2016-06-29 22:35 UTC by Report Bug Copy Issue Report The property `default_callable_whitelist` cannot be accessed from this context as it is declared `private` in class `Jetpack_Sync_Defaults`. This check looks for access to properties that are not accessible from the current context. If you need to make a property accessible to another context you can either raise its visibility level or provide an accessible getter in the defining class. Loading history... Bug introduced 2016-06-29 22:35 UTC by Report Bug Copy Issue Report The property `default_multisite_callable_whitelist` cannot be accessed from this context as it is declared `private` in class `Jetpack_Sync_Defaults`. This check looks for access to properties that are not accessible from the current context. If you need to make a property accessible to another context you can either raise its visibility level or provide an accessible getter in the defining class. Loading history...
18		} else {
19		$this->callable_whitelist = Jetpack_Sync_Defaults::$default_callable_whitelist;
		0 ignored issues – show Bug introduced 2016-06-29 22:35 UTC by Report Bug Copy Issue Report The property `default_callable_whitelist` cannot be accessed from this context as it is declared `private` in class `Jetpack_Sync_Defaults`. This check looks for access to properties that are not accessible from the current context. If you need to make a property accessible to another context you can either raise its visibility level or provide an accessible getter in the defining class. Loading history...
20		}
21		}
22
23		public function init_listeners( $callable ) {
24		add_action( 'jetpack_sync_callable', $callable, 10, 2 );
25
26		// full sync
27		add_action( 'jetpack_full_sync_callables', $callable );
28
29		// always send change to active modules right away
30		add_action( 'update_option_jetpack_active_modules', array( $this, 'unlock_sync_callable' ) );
31
32		// get_plugins and wp_version
33		// gets fired when new code gets installed, updates etc.
34		add_action( 'upgrader_process_complete', array( $this, 'unlock_sync_callable' ) );
35		}
36
37		public function init_full_sync_listeners( $callable ) {
38		add_action( 'jetpack_full_sync_callables', $callable );
39		}
40
41		public function init_before_send() {
42		add_action( 'jetpack_sync_before_send_queue_sync', array( $this, 'maybe_sync_callables' ) );
43
44		// full sync
45		add_filter( 'jetpack_sync_before_send_jetpack_full_sync_callables', array( $this, 'expand_callables' ) );
46		}
47
48		public function reset_data() {
49		delete_option( self::CALLABLES_CHECKSUM_OPTION_NAME );
50		delete_transient( self::CALLABLES_AWAIT_TRANSIENT_NAME );
51		}
52
53		function set_callable_whitelist( $callables ) {
54		$this->callable_whitelist = $callables;
55		}
56
57		function get_callable_whitelist() {
58		return $this->callable_whitelist;
59		}
60
61		public function get_all_callables() {
62		// get_all_callables should run as the master user always.
63		$current_user_id = get_current_user_id();
64		wp_set_current_user( Jetpack_Options::get_option( 'master_user' ) );
65		$callables = array_combine(
66		array_keys( $this->callable_whitelist ),
67		array_map( array( $this, 'get_callable' ), array_values( $this->callable_whitelist ) )
68		);
69		wp_set_current_user( $current_user_id );
70
71		return $callables;
72		}
73
74		private function get_callable( $callable ) {
75		return call_user_func( $callable );
76		}
77
78		public function enqueue_full_sync_actions() {
79		/**
80		* Tells the client to sync all callables to the server
81		*
82		* @since 4.2.0
83		*
84		* @param boolean Whether to expand callables (should always be true)
85		*/
86		do_action( 'jetpack_full_sync_callables', true );
87
88		return 1; // The number of actions enqueued
89		}
90
91		public function get_full_sync_actions() {
92		return array( 'jetpack_full_sync_callables' );
93		}
94
95		public function unlock_sync_callable() {
96		delete_transient( self::CALLABLES_AWAIT_TRANSIENT_NAME );
97		}
98
99	View Code Duplication	public function maybe_sync_callables() {
100		if ( get_transient( self::CALLABLES_AWAIT_TRANSIENT_NAME ) ) {
101		return;
102		}
103
104		$callables = $this->get_all_callables();
105
106		if ( empty( $callables ) ) {
107		return;
108		}
109
110		set_transient( self::CALLABLES_AWAIT_TRANSIENT_NAME, microtime( true ), Jetpack_Sync_Defaults::$default_sync_callables_wait_time );
		0 ignored issues – show Bug introduced 2016-07-28 00:08 UTC by Report Bug Copy Issue Report The property `default_sync_callables_wait_time` cannot be accessed from this context as it is declared `private` in class `Jetpack_Sync_Defaults`. This check looks for access to properties that are not accessible from the current context. If you need to make a property accessible to another context you can either raise its visibility level or provide an accessible getter in the defining class. Loading history...
111
112		$callable_checksums = (array) get_option( self::CALLABLES_CHECKSUM_OPTION_NAME, array() );
113
114		// only send the callables that have changed
115		foreach ( $callables as $name => $value ) {
116		$checksum = $this->get_check_sum( $value );
117		// explicitly not using Identical comparison as get_option returns a string
118		if ( ! $this->still_valid_checksum( $callable_checksums, $name, $checksum ) && ! is_null( $value ) ) {
119		/**
120		* Tells the client to sync a callable (aka function) to the server
121		*
122		* @since 4.2.0
123		*
124		* @param string The name of the callable
125		* @param mixed The value of the callable
126		*/
127		do_action( 'jetpack_sync_callable', $name, $value );
128		$callable_checksums[ $name ] = $checksum;
129		} else {
130		$callable_checksums[ $name ] = $checksum;
131		}
132		}
133		update_option( self::CALLABLES_CHECKSUM_OPTION_NAME, $callable_checksums );
134		}
135
136		public function expand_callables( $args ) {
137		if ( $args[0] ) {
138		return $this->get_all_callables();
139		}
140
141		return $args;
142		}
143		}
144

Automattic / jetpack

Push — add/sync-status-to-debugger ( 8d9a33...da1c65 )

unlock_sync_callable() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like