wp-google-analytics.php ➔ jetpack_wga_init() - Code Metrics - Inspection of "Allow universal analytics (analytics.js) to be use..." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — fix/9129-allow-analytics-js-wi... ( 8a1a92...41fabb )

unknown

created 2018-04-17 20:03 UTC

wp-google-analytics.php ➔ jetpack_wga_init() A

↳ Parent: Project

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php
/*
    Copyright 2006 Aaron D. Campbell (email : [email protected])

    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
    the Free Software Foundation; either version 2 of the License, or
    (at your option) any later version.

    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
    GNU General Public License for more details.

    You should have received a copy of the GNU General Public License
    along with this program; if not, write to the Free Software
    Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
*/

/**
 * Jetpack_Google_Analytics is the class that handles ALL of the plugin functionality.
 * It helps us avoid name collisions
 * http://codex.wordpress.org/Writing_a_Plugin#Avoiding_Function_Name_Collisions
 */

if ( ! defined( 'ABSPATH' ) ) {
	exit;
}

require_once( plugin_basename( 'classes/wp-google-analytics-utils.php' ) );
require_once( plugin_basename( 'classes/wp-google-analytics-options.php' ) );
require_once( plugin_basename( 'classes/wp-google-analytics-legacy.php' ) );
require_once( plugin_basename( 'classes/wp-google-analytics-universal.php' ) );

class Jetpack_Google_Analytics {

	/**
	 * @var Jetpack_Google_Analytics - Static property to hold our singleton instance
	 */
	static $instance = false;
class A {
    var $property;
}

	/**
	 * @var Static property to hold concrete analytics impl that does the work (universal or legacy)
	 */
	static $analytics = false;
class A {
    var $property;
}

	/**
	 * This is our constructor, which is private to force the use of get_instance()
	 *
	 * @return void

	 */
	private function __construct() {
		/**
		 * Allow for universal analytics (analytics.js) to be used outside of ecommerce contexts
		 *
		 * @since 6.1.0
		 *
		 * @param bool Whether universal analytics (analytics.js) should be used (true) or legacy (ga.js) should be used (false)
		 */
		$use_universal_analytics = apply_filters( 'jetpack_wga_use_universal_analytics',
			Jetpack_Google_Analytics_Options::enhanced_ecommerce_tracking_is_enabled()
		);

		if ( $use_universal_analytics ) {
			$analytics = new Jetpack_Google_Analytics_Universal();
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
		} else {
			$analytics = new Jetpack_Google_Analytics_Legacy();
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
		}

	}

	/**
	 * Function to instantiate our class and make it a singleton
	 */
	public static function get_instance() {
		if ( ! self::$instance ) {
			self::$instance = new self;
		}

		return self::$instance;
	}
}

/**
 * Bootstrap analytics on the init action. This allows filtering of things like
 * jetpack_wga_use_universal_analytics
 *
 * @since 6.1.0
 *
 * @return void
 */
function jetpack_wga_init() {
	global $jetpack_google_analytics;
	$jetpack_google_analytics = Jetpack_Google_Analytics::get_instance();
}

add_action( 'init', 'jetpack_wga_init' );


1			<?php
2			/*
3			Copyright 2006 Aaron D. Campbell (email : [email protected])
4
5			This program is free software; you can redistribute it and/or modify
6			it under the terms of the GNU General Public License as published by
7			the Free Software Foundation; either version 2 of the License, or
8			(at your option) any later version.
9
10			This program is distributed in the hope that it will be useful,
11			but WITHOUT ANY WARRANTY; without even the implied warranty of
12			MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
13			GNU General Public License for more details.
14
15			You should have received a copy of the GNU General Public License
16			along with this program; if not, write to the Free Software
17			Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
18			*/
19
20			/**
21			* Jetpack_Google_Analytics is the class that handles ALL of the plugin functionality.
22			* It helps us avoid name collisions
23			* http://codex.wordpress.org/Writing_a_Plugin#Avoiding_Function_Name_Collisions
24			*/
25
26			if ( ! defined( 'ABSPATH' ) ) {
27			exit;
28			}
29
30			require_once( plugin_basename( 'classes/wp-google-analytics-utils.php' ) );
31			require_once( plugin_basename( 'classes/wp-google-analytics-options.php' ) );
32			require_once( plugin_basename( 'classes/wp-google-analytics-legacy.php' ) );
33			require_once( plugin_basename( 'classes/wp-google-analytics-universal.php' ) );
34
35			class Jetpack_Google_Analytics {
36
37			/**
38			* @var Jetpack_Google_Analytics - Static property to hold our singleton instance
39			*/
40			static $instance = false;
			0 ignored issues – show Coding Style introduced 2017-11-17 19:55 UTC by Report Bug Copy Issue Report The visibility should be declared for property `$instance`. 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...
41
42			/**
43			* @var Static property to hold concrete analytics impl that does the work (universal or legacy)
44			*/
45			static $analytics = false;
			0 ignored issues – show Coding Style introduced 2017-11-17 19:55 UTC by Report Bug Copy Issue Report The visibility should be declared for property `$analytics`. 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...
46
47			/**
48			* This is our constructor, which is private to force the use of get_instance()
49			*
50			* @return void
			0 ignored issues – show Comprehensibility Best Practice introduced 2017-09-13 19:17 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
51			*/
52			private function __construct() {
53			/**
54			* Allow for universal analytics (analytics.js) to be used outside of ecommerce contexts
55			*
56			* @since 6.1.0
57			*
58			* @param bool Whether universal analytics (analytics.js) should be used (true) or legacy (ga.js) should be used (false)
59			*/
60			$use_universal_analytics = apply_filters( 'jetpack_wga_use_universal_analytics',
61			Jetpack_Google_Analytics_Options::enhanced_ecommerce_tracking_is_enabled()
62			);
63
64			if ( $use_universal_analytics ) {
65			$analytics = new Jetpack_Google_Analytics_Universal();
			0 ignored issues – show Unused Code introduced 2017-11-17 19:55 UTC by Report Bug Copy Issue Report `$analytics` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
66			} else {
67			$analytics = new Jetpack_Google_Analytics_Legacy();
			0 ignored issues – show Unused Code introduced 2017-11-17 19:55 UTC by Report Bug Copy Issue Report `$analytics` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
68			}
69
70			}
71
72			/**
73			* Function to instantiate our class and make it a singleton
74			*/
75			public static function get_instance() {
76			if ( ! self::$instance ) {
77			self::$instance = new self;
78			}
79
80			return self::$instance;
81			}
82			}
83
84			/**
85			* Bootstrap analytics on the init action. This allows filtering of things like
86			* jetpack_wga_use_universal_analytics
87			*
88			* @since 6.1.0
89			*
90			* @return void
91			*/
92			function jetpack_wga_init() {
93			global $jetpack_google_analytics;
94			$jetpack_google_analytics = Jetpack_Google_Analytics::get_instance();
95			}
96
97			add_action( 'init', 'jetpack_wga_init' );
98

Automattic / jetpack

Push — fix/9129-allow-analytics-js-wi... ( 8a1a92...41fabb )

wp-google-analytics.php ➔ jetpack_wga_init() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like