Jetpack_Options_Sync::update_option() - Code Metrics - Inspection of "Test showing options added to sync array and delet..." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — add/sync-rest-options ( f50eff )

unknown

created 2016-03-03 23:15 UTC

Jetpack_Options_Sync::update_option() A

↳ Parent: Jetpack_Options_Sync

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	7

Duplication

Lines	9
Ratio	100 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	9
loc	9
rs	9.6666
cc	2
eloc	7
nc	2
nop	0

<?php

class Jetpack_Options_Sync {

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

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

	static function init() {

		foreach ( self::$options_to_sync as $option ) {
			self::init_option( $option );
		}
	}

	static function init_option( $option ) {
		add_action( "delete_option_{$option}", array( __CLASS__, 'delete_option' ) );
		add_action( "update_option_{$option}", array( __CLASS__, 'update_option' ) );
		add_action( "add_option_{$option}",    array( __CLASS__, 'add_option'   ) );
	}

	static function delete_option( $option ) {
		self::$delete[] = $option;
	}

	static function update_option() {
		$option = current_filter();
		$prefix = 'update_option_';
		if ( 0 !== strpos( $option, $prefix ) ) {
			return;
		}
		$option = substr( $option, strlen( $prefix ) );
		self::$sync[] = $option;
	}

	static function add_option( $option ) {
		self::$sync[] = $option;
	}

	static function options_to_delete() {
		return array_unique( self::$delete );
	}

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

1		<?php
2
3		class Jetpack_Options_Sync {
4
5		static $options_to_sync = array();
		0 ignored issues – show Coding Style introduced 2016-03-03 23:48 UTC by Report Bug Copy Issue Report The visibility should be declared for property `$options_to_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...
6
7		static $sync = array();
		0 ignored issues – show Coding Style introduced 2016-03-03 23:48 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...
8		static $delete = array();
		0 ignored issues – show Coding Style introduced 2016-03-03 23:48 UTC by Report Bug Copy Issue Report The visibility should be declared for property `$delete`. 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...
9
10		static function init() {
11
12		foreach ( self::$options_to_sync as $option ) {
13		self::init_option( $option );
14		}
15		}
16
17		static function init_option( $option ) {
18		add_action( "delete_option_{$option}", array( __CLASS__, 'delete_option' ) );
19		add_action( "update_option_{$option}", array( __CLASS__, 'update_option' ) );
20		add_action( "add_option_{$option}", array( __CLASS__, 'add_option' ) );
21		}
22
23		static function delete_option( $option ) {
24		self::$delete[] = $option;
25		}
26
27	View Code Duplication	static function update_option() {
28		$option = current_filter();
29		$prefix = 'update_option_';
30		if ( 0 !== strpos( $option, $prefix ) ) {
31		return;
32		}
33		$option = substr( $option, strlen( $prefix ) );
34		self::$sync[] = $option;
35		}
36
37		static function add_option( $option ) {
38		self::$sync[] = $option;
39		}
40
41		static function options_to_delete() {
42		return array_unique( self::$delete );
43		}
44
45		static function options_to_sync() {
46		return array_unique( self::$sync );
47		}
48		}

Automattic / jetpack

Push — add/sync-rest-options ( f50eff )

Jetpack_Options_Sync::update_option() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like