WordAds_Params::__construct() - Code Metrics - Inspection of "Adding in things to sync up with WPCOM version" - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — fix/sync-likes-settings-file-w... ( e241a1...a5000d )

unknown

created 2019-02-21 03:06 UTC

WordAds_Params::__construct() C

↳ Parent: WordAds_Params

Complexity

Conditions	10
Paths	160

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	10
nc	160
nop	0
dl	0
loc	51
rs	6.8024
c	0
b	0
f	0

How to fix Long Method Complexity

<?php

class WordAds_Params {

	/**
	 * Setup parameters for serving the ads
	 *
	 * @since 4.5.0
	 */
	public function __construct() {
		// WordAds setting => default
		$settings = array(
			'wordads_approved'           => false,
			'wordads_active'             => false,
			'wordads_house'              => true,
			'wordads_unsafe'             => false,
			'enable_header_ad'           => true,
			'wordads_second_belowpost'   => true,
			'wordads_display_front_page' => true,
			'wordads_display_post'       => true,
			'wordads_display_page'       => true,
			'wordads_display_archive'    => true,
			'wordads_custom_adstxt'      => '',
		);

		// grab settings, or set as default if it doesn't exist
		$this->options = array();
class MyClass { }

$x = new MyClass();
$x->foo = true;
		foreach ( $settings as $setting => $default ) {
			$option = get_option( $setting, null );

			if ( is_null( $option ) ) {
				update_option( $setting, $default, true );
				$option = $default;
			}

			$this->options[ $setting ] = 'wordads_custom_adstxt' !== $setting ? (bool) $option : $option;
		}

		$host = 'localhost';
		if ( isset( $_SERVER['HTTP_HOST'] ) ) {
			$host = $_SERVER['HTTP_HOST'];
		}

		$this->url = ( is_ssl() ? 'https' : 'http' ) . '://' . $host . $_SERVER['REQUEST_URI'];
class MyClass { }

$x = new MyClass();
$x->foo = true;
		if ( ! ( false === strpos( $this->url, '?' ) ) && ! isset( $_GET['p'] ) ) {
			$this->url = substr( $this->url, 0, strpos( $this->url, '?' ) );
		}

		$this->cloudflare     = self::is_cloudflare();
class MyClass { }

$x = new MyClass();
$x->foo = true;
		$this->blog_id        = Jetpack::get_option( 'id', 0 );
class MyClass { }

$x = new MyClass();
$x->foo = true;
		$this->mobile_device  = jetpack_is_mobile( 'any', true );
class MyClass { }

$x = new MyClass();
$x->foo = true;
		$this->targeting_tags = array(
class MyClass { }

$x = new MyClass();
$x->foo = true;
			'WordAds' => 1,
			'BlogId'  => Jetpack::is_development_mode() ? 0 : Jetpack_Options::get_option( 'id' ),
			'Domain'  => esc_js( parse_url( home_url(), PHP_URL_HOST ) ),
			'PageURL' => esc_js( $this->url ),
			'LangId'  => false !== strpos( get_bloginfo( 'language' ), 'en' ) ? 1 : 0, // TODO something else?
			'AdSafe'  => 1, // TODO
		);
	}

	/**
	 * @return boolean true if the user is browsing on a mobile device (iPad not included)
	 *
	 * @since 4.5.0
	 */
	public function is_mobile() {
		return ! empty( $this->mobile_device );
	}

	/**
	 * @return boolean true if site is being served via CloudFlare
	 *
	 * @since 4.5.0
	 */
	public static function is_cloudflare() {
		if (
			defined( 'WORDADS_CLOUDFLARE' )
			|| isset( $_SERVER['HTTP_CF_CONNECTING_IP'] )
			|| isset( $_SERVER['HTTP_CF_IPCOUNTRY'] )
			|| isset( $_SERVER['HTTP_CF_VISITOR'] )
		) {
			return true;
		}

		return false;
	}

	/**
	 * @return boolean true if user is browsing in iOS device
	 *
	 * @since 4.5.0
	 */
	public function is_ios() {
		return in_array( $this->get_device(), array( 'ipad', 'iphone', 'ipod' ) );
	}

	/**
	 * Returns the user's device (see user-agent.php) or 'desktop'
	 *
	 * @return string user device
	 *
	 * @since 4.5.0
	 */
	public function get_device() {
		global $agent_info;

		if ( ! empty( $this->mobile_device ) ) {
			return $this->mobile_device;
		}

		if ( $agent_info->is_ipad() ) {
			return 'ipad';
		}

		return 'desktop';
	}

	/**
	 * @return string The type of page that is being loaded
	 *
	 * @since 4.5.0
	 */
	public function get_page_type() {
		if ( ! empty( $this->page_type ) ) {
			return $this->page_type;
class MyClass { }

$x = new MyClass();
$x->foo = true;
		}

		if ( self::is_static_home() ) {
			$this->page_type = 'static_home';
		} elseif ( is_home() ) {
			$this->page_type = 'home';
		} elseif ( is_page() ) {
			$this->page_type = 'page';
		} elseif ( is_single() ) {
			$this->page_type = 'post';
		} elseif ( is_search() ) {
			$this->page_type = 'search';
		} elseif ( is_category() ) {
			$this->page_type = 'category';
		} elseif ( is_archive() ) {
			$this->page_type = 'archive';
		} else {
			$this->page_type = 'wtf';
		}

		return $this->page_type;
	}

	/**
	 * @return int The page type code for ipw config
	 *
	 * @since 5.6.0
	 */
	public function get_page_type_ipw() {
		if ( ! empty( $this->page_type_ipw ) ) {

			return $this->page_type_ipw;

		}

		$page_type_ipw = 6;
		if ( self::is_static_home() || is_home() || is_front_page() ) {
			$page_type_ipw = 0;
		} elseif ( is_page() ) {
			$page_type_ipw = 2;
		} elseif ( is_singular() ) {
			$page_type_ipw = 1;
		} elseif ( is_search() ) {
			$page_type_ipw = 4;
		} elseif ( is_category() || is_tag() || is_archive() || is_author() ) {
			$page_type_ipw = 3;
		} elseif ( is_404() ) {
			$page_type_ipw = 5;
		}

		$this->page_type_ipw = $page_type_ipw;

		return $page_type_ipw;
	}

	/**
	 * Returns true if page is static home
	 *
	 * @return boolean true if page is static home
	 *
	 * @since 4.5.0
	 */
	public static function is_static_home() {
		return is_front_page() &&
			'page' == get_option( 'show_on_front' ) &&
			get_option( 'page_on_front' );
	}

	/**
	 * Logic for if we should show an ad
	 *
	 * @since 4.5.0
	 */
	public function should_show() {
		global $wp_query;
		if ( ( is_front_page() || is_home() ) && ! $this->options['wordads_display_front_page'] ) {
			return false;
		}

		if ( is_single() && ! $this->options['wordads_display_post'] ) {
			return false;
		}

		if ( is_page() && ! $this->options['wordads_display_page'] ) {
			return false;
		}

		if ( ( is_archive() || is_search() ) && ! $this->options['wordads_display_archive'] ) {
			return false;
		}

		if ( is_single() || ( is_page() && ! is_home() ) ) {
			return true;
		}

		// TODO this would be a good place for allowing the user to specify
		if ( ( is_home() || is_archive() || is_search() ) && 0 == $wp_query->current_post ) {
			return true;
		}

		return false;
	}
}


1			<?php
2
3			class WordAds_Params {
4
5			/**
6			* Setup parameters for serving the ads
7			*
8			* @since 4.5.0
9			*/
10			public function __construct() {
11			// WordAds setting => default
12			$settings = array(
13			'wordads_approved' => false,
14			'wordads_active' => false,
15			'wordads_house' => true,
16			'wordads_unsafe' => false,
17			'enable_header_ad' => true,
18			'wordads_second_belowpost' => true,
19			'wordads_display_front_page' => true,
20			'wordads_display_post' => true,
21			'wordads_display_page' => true,
22			'wordads_display_archive' => true,
23			'wordads_custom_adstxt' => '',
24			);
25
26			// grab settings, or set as default if it doesn't exist
27			$this->options = array();
			0 ignored issues – show Bug introduced 2017-09-13 18:41 UTC by Report Bug Copy Issue Report The property `options` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
28			foreach ( $settings as $setting => $default ) {
29			$option = get_option( $setting, null );
30
31			if ( is_null( $option ) ) {
32			update_option( $setting, $default, true );
33			$option = $default;
34			}
35
36			$this->options[ $setting ] = 'wordads_custom_adstxt' !== $setting ? (bool) $option : $option;
37			}
38
39			$host = 'localhost';
40			if ( isset( $_SERVER['HTTP_HOST'] ) ) {
41			$host = $_SERVER['HTTP_HOST'];
42			}
43
44			$this->url = ( is_ssl() ? 'https' : 'http' ) . '://' . $host . $_SERVER['REQUEST_URI'];
			0 ignored issues – show Bug introduced 2016-12-14 19:13 UTC by Report Bug Copy Issue Report The property `url` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
45			if ( ! ( false === strpos( $this->url, '?' ) ) && ! isset( $_GET['p'] ) ) {
46			$this->url = substr( $this->url, 0, strpos( $this->url, '?' ) );
47			}
48
49			$this->cloudflare = self::is_cloudflare();
			0 ignored issues – show Bug introduced 2016-10-12 21:02 UTC by Report Bug Copy Issue Report The property `cloudflare` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
50			$this->blog_id = Jetpack::get_option( 'id', 0 );
			0 ignored issues – show Bug introduced 2016-10-12 21:02 UTC by Report Bug Copy Issue Report The property `blog_id` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
51			$this->mobile_device = jetpack_is_mobile( 'any', true );
			0 ignored issues – show Bug introduced 2016-10-12 21:02 UTC by Report Bug Copy Issue Report The property `mobile_device` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
52			$this->targeting_tags = array(
			0 ignored issues – show Bug introduced 2016-10-12 21:02 UTC by Report Bug Copy Issue Report The property `targeting_tags` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
53			'WordAds' => 1,
54			'BlogId' => Jetpack::is_development_mode() ? 0 : Jetpack_Options::get_option( 'id' ),
55			'Domain' => esc_js( parse_url( home_url(), PHP_URL_HOST ) ),
56			'PageURL' => esc_js( $this->url ),
57			'LangId' => false !== strpos( get_bloginfo( 'language' ), 'en' ) ? 1 : 0, // TODO something else?
58			'AdSafe' => 1, // TODO
59			);
60			}
61
62			/**
63			* @return boolean true if the user is browsing on a mobile device (iPad not included)
64			*
65			* @since 4.5.0
66			*/
67			public function is_mobile() {
68			return ! empty( $this->mobile_device );
69			}
70
71			/**
72			* @return boolean true if site is being served via CloudFlare
73			*
74			* @since 4.5.0
75			*/
76			public static function is_cloudflare() {
77			if (
78			defined( 'WORDADS_CLOUDFLARE' )
79			\|\| isset( $_SERVER['HTTP_CF_CONNECTING_IP'] )
80			\|\| isset( $_SERVER['HTTP_CF_IPCOUNTRY'] )
81			\|\| isset( $_SERVER['HTTP_CF_VISITOR'] )
82			) {
83			return true;
84			}
85
86			return false;
87			}
88
89			/**
90			* @return boolean true if user is browsing in iOS device
91			*
92			* @since 4.5.0
93			*/
94			public function is_ios() {
95			return in_array( $this->get_device(), array( 'ipad', 'iphone', 'ipod' ) );
96			}
97
98			/**
99			* Returns the user's device (see user-agent.php) or 'desktop'
100			*
101			* @return string user device
102			*
103			* @since 4.5.0
104			*/
105			public function get_device() {
106			global $agent_info;
107
108			if ( ! empty( $this->mobile_device ) ) {
109			return $this->mobile_device;
110			}
111
112			if ( $agent_info->is_ipad() ) {
113			return 'ipad';
114			}
115
116			return 'desktop';
117			}
118
119			/**
120			* @return string The type of page that is being loaded
121			*
122			* @since 4.5.0
123			*/
124			public function get_page_type() {
125			if ( ! empty( $this->page_type ) ) {
126			return $this->page_type;
			0 ignored issues – show Bug introduced 2016-10-12 21:02 UTC by Report Bug Copy Issue Report The property `page_type` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
127			}
128
129			if ( self::is_static_home() ) {
130			$this->page_type = 'static_home';
131			} elseif ( is_home() ) {
132			$this->page_type = 'home';
133			} elseif ( is_page() ) {
134			$this->page_type = 'page';
135			} elseif ( is_single() ) {
136			$this->page_type = 'post';
137			} elseif ( is_search() ) {
138			$this->page_type = 'search';
139			} elseif ( is_category() ) {
140			$this->page_type = 'category';
141			} elseif ( is_archive() ) {
142			$this->page_type = 'archive';
143			} else {
144			$this->page_type = 'wtf';
145			}
146
147			return $this->page_type;
148			}
149
150			/**
151			* @return int The page type code for ipw config
152			*
153			* @since 5.6.0
154			*/
155			public function get_page_type_ipw() {
156			if ( ! empty( $this->page_type_ipw ) ) {
			0 ignored issues – show Bug introduced 2017-11-27 20:50 UTC by Report Bug Copy Issue Report The property `page_type_ipw` does not seem to exist. Did you mean `page_type`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
157			return $this->page_type_ipw;
			0 ignored issues – show Bug introduced 2018-01-06 22:11 UTC by Report Bug Copy Issue Report The property `page_type_ipw` does not seem to exist. Did you mean `page_type`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
158			}
159
160			$page_type_ipw = 6;
161			if ( self::is_static_home() \|\| is_home() \|\| is_front_page() ) {
162			$page_type_ipw = 0;
163			} elseif ( is_page() ) {
164			$page_type_ipw = 2;
165			} elseif ( is_singular() ) {
166			$page_type_ipw = 1;
167			} elseif ( is_search() ) {
168			$page_type_ipw = 4;
169			} elseif ( is_category() \|\| is_tag() \|\| is_archive() \|\| is_author() ) {
170			$page_type_ipw = 3;
171			} elseif ( is_404() ) {
172			$page_type_ipw = 5;
173			}
174
175			$this->page_type_ipw = $page_type_ipw;
			0 ignored issues – show Bug introduced 2017-11-27 20:50 UTC by Report Bug Copy Issue Report The property `page_type_ipw` does not seem to exist. Did you mean `page_type`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
176			return $page_type_ipw;
177			}
178
179			/**
180			* Returns true if page is static home
181			*
182			* @return boolean true if page is static home
183			*
184			* @since 4.5.0
185			*/
186			public static function is_static_home() {
187			return is_front_page() &&
188			'page' == get_option( 'show_on_front' ) &&
189			get_option( 'page_on_front' );
190			}
191
192			/**
193			* Logic for if we should show an ad
194			*
195			* @since 4.5.0
196			*/
197			public function should_show() {
198			global $wp_query;
199			if ( ( is_front_page() \|\| is_home() ) && ! $this->options['wordads_display_front_page'] ) {
200			return false;
201			}
202
203			if ( is_single() && ! $this->options['wordads_display_post'] ) {
204			return false;
205			}
206
207			if ( is_page() && ! $this->options['wordads_display_page'] ) {
208			return false;
209			}
210
211			if ( ( is_archive() \|\| is_search() ) && ! $this->options['wordads_display_archive'] ) {
212			return false;
213			}
214
215			if ( is_single() \|\| ( is_page() && ! is_home() ) ) {
216			return true;
217			}
218
219			// TODO this would be a good place for allowing the user to specify
220			if ( ( is_home() \|\| is_archive() \|\| is_search() ) && 0 == $wp_query->current_post ) {
221			return true;
222			}
223
224			return false;
225			}
226			}
227

Automattic / jetpack

Push — fix/sync-likes-settings-file-w... ( e241a1...a5000d )

WordAds_Params::__construct() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like