Jetpack_Google_Analytics::insert_code() - Code Metrics - Inspection of "Comments: Allow comment_form_action actions" - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — fix/7357 ( 13a0c4...764bff )

unknown

created 2017-12-21 15:21 UTC

Jetpack_Google_Analytics::insert_code() B

↳ Parent: Project

Complexity

Conditions	6
Paths	8

Size

Total Lines	64
Code Lines	26

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	6
eloc	26
nc	8
nop	1
dl	0
loc	64
rs	8.6346
c	0
b	0
f	0

How to fix Long Method

<?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() {
		if ( Jetpack_Google_Analytics_Options::enhanced_ecommerce_tracking_is_enabled() ) {
			$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;
	}
}

global $jetpack_google_analytics;
$jetpack_google_analytics = Jetpack_Google_Analytics::get_instance();


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-12-04 13:32 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			if ( Jetpack_Google_Analytics_Options::enhanced_ecommerce_tracking_is_enabled() ) {
54			$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...
55			} else {
56			$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...
57			}
58
59			}
60
61			/**
62			* Function to instantiate our class and make it a singleton
63			*/
64			public static function get_instance() {
65			if ( ! self::$instance ) {
66			self::$instance = new self;
67			}
68
69			return self::$instance;
70			}
71			}
72
73			global $jetpack_google_analytics;
74			$jetpack_google_analytics = Jetpack_Google_Analytics::get_instance();
75

Automattic / jetpack

Push — fix/7357 ( 13a0c4...764bff )

Jetpack_Google_Analytics::insert_code() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like