Jetpack_Sync_Functions - Code Metrics - Inspection of "Add function class aka mock options." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — add/sync-rest-options ( c0c257...3a27a6 )

unknown

created 2016-03-11 23:42 UTC

Jetpack_Sync_Functions A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	70
Duplicated Lines	47.14 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
wmc	15
c	1
b	0
f	1
lcom	1
cbo	1
dl	33
loc	70
rs	10

8 Methods

Rating	Name	Duplication	Size	Complexity
A	sync()	0	3	1
A	sync_sometimes()	14	14	3
A	sync_all()	10	10	3
A	get_check_sum()	0	6	2
A	get_query_string()	0	3	1
A	values()	9	9	3
A	get_function_to_sync()	0	3	1
A	get()	0	3	1

How to fix Duplicated Code

<?php

class Jetpack_Sync_Functions {

	static $functions = array(
class A {
    var $property;
}
		'wp_version' => array( 'Jetpack', 'get_wp_version' ),
		'wp_max_upload_size' => 'wp_max_upload_size',
		'network_name' => array( 'Jetpack', 'network_name' ),
	);

	static $sync = array();
class A {
    var $property;
}

	static function sync() {
		return self::values( self::get_function_to_sync() );
	}

	static function sync_sometimes() {
		// Since there are option in the sync we know that things have changed.
		if ( ! empty ( self::$sync ) ) {
			return self::sync_all();
		}

		$values           = self::values( self::$functions );
		$check_sum        = self::get_check_sum( $values );

		if ( Jetpack_Options::get_option( 'options_check_sum' ) !== $check_sum ) {
			return self::sync_all( $values, $check_sum );
		}
		return array();
	}

	static function sync_all( $values = null, $check_sum = null ) {
		if ( is_null( $values ) ) {
			$values           = self::values( self::$functions );
$a = "a";
$ab = "ab";
$abc = "abc";
		}
		if( is_null( $check_sum ) ) {
			$check_sum = self::get_check_sum( $values );
		}
		Jetpack_Options::update_option( 'function_check_sum', $check_sum );
		return $values;
	}

	static function get_check_sum( $values = null ) {
		if( is_null( $values ) ){
			$values = self::values();
		}
		return crc32( self::get_query_string( $values ) );
	}

	static function get_query_string( $values ) {
		return build_query( $values );
	}

	static function values( $sync = array() ) {
		$values = array();
		if ( ! empty( $sync ) ) {
			foreach ( $sync as $key => $function ) {
				$values[ $key ] = self::get( $function );
			}
		}
		return $values;
	}

	static function get_function_to_sync() {
		return array_unique( self::$sync );
	}

	static function get( $function ) {
		return call_user_func( $function );
	}

}







1		<?php
2
3		class Jetpack_Sync_Functions {
4
5		static $functions = array(
		0 ignored issues – show Coding Style introduced 2016-03-11 23:50 UTC by Report Bug Copy Issue Report The visibility should be declared for property `$functions`. The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using class A { var $property; } the property is implicitly global. To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2. Loading history...
6		'wp_version' => array( 'Jetpack', 'get_wp_version' ),
7		'wp_max_upload_size' => 'wp_max_upload_size',
8		'network_name' => array( 'Jetpack', 'network_name' ),
9		);
10
11		static $sync = array();
		0 ignored issues – show Coding Style introduced 2016-03-11 23:50 UTC by Report Bug Copy Issue Report The visibility should be declared for property `$sync`. The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using class A { var $property; } the property is implicitly global. To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2. Loading history...
12
13		static function sync() {
14		return self::values( self::get_function_to_sync() );
15		}
16
17	View Code Duplication	static function sync_sometimes() {
18		// Since there are option in the sync we know that things have changed.
19		if ( ! empty ( self::$sync ) ) {
20		return self::sync_all();
21		}
22
23		$values = self::values( self::$functions );
24		$check_sum = self::get_check_sum( $values );
25
26		if ( Jetpack_Options::get_option( 'options_check_sum' ) !== $check_sum ) {
27		return self::sync_all( $values, $check_sum );
28		}
29		return array();
30		}
31
32	View Code Duplication	static function sync_all( $values = null, $check_sum = null ) {
33		if ( is_null( $values ) ) {
34		$values = self::values( self::$functions );
		0 ignored issues – show Coding Style introduced 2016-03-11 23:50 UTC by Report Bug Copy Issue Report Equals sign not aligned correctly; expected 1 space but found 11 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...
35		}
36		if( is_null( $check_sum ) ) {
37		$check_sum = self::get_check_sum( $values );
38		}
39		Jetpack_Options::update_option( 'function_check_sum', $check_sum );
40		return $values;
41		}
42
43		static function get_check_sum( $values = null ) {
44		if( is_null( $values ) ){
45		$values = self::values();
46		}
47		return crc32( self::get_query_string( $values ) );
48		}
49
50		static function get_query_string( $values ) {
51		return build_query( $values );
52		}
53
54	View Code Duplication	static function values( $sync = array() ) {
55		$values = array();
56		if ( ! empty( $sync ) ) {
57		foreach ( $sync as $key => $function ) {
58		$values[ $key ] = self::get( $function );
59		}
60		}
61		return $values;
62		}
63
64		static function get_function_to_sync() {
65		return array_unique( self::$sync );
66		}
67
68		static function get( $function ) {
69		return call_user_func( $function );
70		}
71
72		}
73
74
75
76
77
78

Automattic / jetpack

Push — add/sync-rest-options ( c0c257...3a27a6 )

Jetpack_Sync_Functions A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

8 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like