Jetpack_Google_Analytics::insert_code() - Code Metrics - Inspection of "Remove addTitle function and inline it." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — pr/7794 ( cb7d3e...693e58 )

by George

created 2017-10-24 16:41 UTC

Jetpack_Google_Analytics::insert_code() B

↳ Parent: Jetpack_Google_Analytics

Complexity

Conditions	6
Paths	8

Size

Total Lines	54
Code Lines	25

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	6
eloc	25
nc	8
nop	1
dl	0
loc	54
rs	8.7449
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
 */
class Jetpack_Google_Analytics {

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

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

	 */
	private function __construct() {
		add_action( 'wp_footer', array( $this, 'insert_code' ) );
	}

	/**
	 * 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;
	}

	/**
	 * Used to generate a tracking URL
	 *
	 * @param array $track - Must have ['data'] and ['code'].
	 * @return string - Tracking URL
	 */
	private function _get_url( $track ) {
		$site_url = ( is_ssl() ? 'https://':'http://' ) . sanitize_text_field( wp_unslash( $_SERVER['HTTP_HOST'] ) ); // Input var okay.
		foreach ( $track as $k => $value ) {
			if ( strpos( strtolower( $value ), strtolower( $site_url ) ) === 0 ) {
				$track[ $k ] = substr( $track[ $k ], strlen( $site_url ) );
			}
			if ( 'data' === $k ) {
				$track[ $k ] = preg_replace( '/^https?:\/\/|^\/+/i', '', $track[ $k ] );
			}

			// This way we don't lose search data.
			if ( 'data' === $k && 'search' === $track['code'] ) {
				$track[ $k ] = rawurlencode( $track[ $k ] );
			} else {
				$track[ $k ] = preg_replace( '/[^a-z0-9\.\/\+\?=-]+/i', '_', $track[ $k ] );
			}

			$track[ $k ] = trim( $track[ $k ], '_' );
		}
		$char = ( strpos( $track['data'], '?' ) === false ) ? '?' : '&amp;';
		return str_replace( "'", "\'", "/{$track['code']}/{$track['data']}{$char}referer=" . rawurlencode( isset( $_SERVER['HTTP_REFERER'] ) ? $_SERVER['HTTP_REFERER'] : '' ) ); // Input var okay.
	}

	/**
	 * Maybe output or return, depending on the context
	 */
	private function _output_or_return( $val, $maybe ) {
		if ( $maybe ) {
			echo $val . "\r\n";
		} else {
			return $val;
		}
	}

	/**
	 * This injects the Google Analytics code into the footer of the page.
	 *
	 * @param bool[optional] $output - defaults to true, false returns but does NOT echo the code.
	 */
	public function insert_code( $output = true ) {
		// If $output is not a boolean false, set it to true (default).
		$output = ( false !== $output);

		$tracking_id = $this->_get_tracking_code();
		if ( empty( $tracking_id ) ) {
			return $this->_output_or_return( '<!-- Your Google Analytics Plugin is missing the tracking ID -->', $output );
		}

		// If we're in the admin_area, return without inserting code.
		if ( is_admin() ) {
			return $this->_output_or_return( '<!-- Your Google Analytics Plugin is set to ignore Admin area -->', $output );
		}

		$custom_vars = array(
			"_gaq.push(['_setAccount', '{$tracking_id}']);",
		);

		$track = array();
		if ( is_404() ) {
			// This is a 404 and we are supposed to track them.
			$custom_vars[] = "_gaq.push( [ '_trackEvent', '404', document.location.href, document.referrer ] );";
		} elseif ( is_search() ) {
			// Set track for searches, if it's a search, and we are supposed to.
			$track['data'] = sanitize_text_field( wp_unslash( $_REQUEST['s'] ) ); // Input var okay.
			$track['code'] = 'search';
		}

		if ( ! empty( $track ) ) {
			$track['url'] = $this->_get_url( $track );
			// adjust the code that we output, account for both types of tracking.
			$track['url'] = esc_js( str_replace( '&', '&amp;', $track['url'] ) );
			$custom_vars[] = "_gaq.push(['_trackPageview','{$track['url']}']);";
		} else {
			$custom_vars[] = "_gaq.push(['_trackPageview']);";
		}

		$async_code = "<!-- Jetpack Google Analytics -->
		<script type='text/javascript'>
							var _gaq = _gaq || [];
							%custom_vars%

							(function() {
								var ga = document.createElement('script'); ga.type = 'text/javascript'; ga.async = true;
								ga.src = ('https:' == document.location.protocol ? 'https://ssl' : 'http://www') + '.google-analytics.com/ga.js';
								var s = document.getElementsByTagName('script')[0]; s.parentNode.insertBefore(ga, s);
							})();
						</script>";

		$custom_vars_string = implode( "\r\n", $custom_vars );
		$async_code = str_replace( '%custom_vars%', $custom_vars_string, $async_code );

		return $this->_output_or_return( $async_code, $output );
	}

	/**
	 * Used to get the tracking code option
	 *
	 * @return tracking code option value.
	 */
	private function _get_tracking_code() {
		$o = get_option( 'jetpack_wga' );

		if ( isset( $o['code'] ) && preg_match( '#UA-[\d-]+#', $o['code'], $matches ) ) {
				return $o['code'];
		}

		return '';
	}
}

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			class Jetpack_Google_Analytics {
26
27			/**
28			* @var Jetpack_Google_Analytics - Static property to hold our singleton instance
29			*/
30			static $instance = false;
			0 ignored issues – show Coding Style introduced 2017-04-18 21:07 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...
31
32			/**
33			* This is our constructor, which is private to force the use of get_instance()
34			*
35			* @return void
			0 ignored issues – show Comprehensibility Best Practice introduced 2016-11-11 21:07 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...
36			*/
37			private function __construct() {
38			add_action( 'wp_footer', array( $this, 'insert_code' ) );
39			}
40
41			/**
42			* Function to instantiate our class and make it a singleton
43			*/
44			public static function get_instance() {
45			if ( ! self::$instance ) {
46			self::$instance = new self;
47			}
48
49			return self::$instance;
50			}
51
52			/**
53			* Used to generate a tracking URL
54			*
55			* @param array $track - Must have ['data'] and ['code'].
56			* @return string - Tracking URL
57			*/
58			private function _get_url( $track ) {
59			$site_url = ( is_ssl() ? 'https://':'http://' ) . sanitize_text_field( wp_unslash( $_SERVER['HTTP_HOST'] ) ); // Input var okay.
60			foreach ( $track as $k => $value ) {
61			if ( strpos( strtolower( $value ), strtolower( $site_url ) ) === 0 ) {
62			$track[ $k ] = substr( $track[ $k ], strlen( $site_url ) );
63			}
64			if ( 'data' === $k ) {
65			$track[ $k ] = preg_replace( '/^https?:\/\/\|^\/+/i', '', $track[ $k ] );
66			}
67
68			// This way we don't lose search data.
69			if ( 'data' === $k && 'search' === $track['code'] ) {
70			$track[ $k ] = rawurlencode( $track[ $k ] );
71			} else {
72			$track[ $k ] = preg_replace( '/[^a-z0-9\.\/\+\?=-]+/i', '_', $track[ $k ] );
73			}
74
75			$track[ $k ] = trim( $track[ $k ], '_' );
76			}
77			$char = ( strpos( $track['data'], '?' ) === false ) ? '?' : '&';
78			return str_replace( "'", "\'", "/{$track['code']}/{$track['data']}{$char}referer=" . rawurlencode( isset( $_SERVER['HTTP_REFERER'] ) ? $_SERVER['HTTP_REFERER'] : '' ) ); // Input var okay.
79			}
80
81			/**
82			* Maybe output or return, depending on the context
83			*/
84			private function _output_or_return( $val, $maybe ) {
85			if ( $maybe ) {
86			echo $val . "\r\n";
87			} else {
88			return $val;
89			}
90			}
91
92			/**
93			* This injects the Google Analytics code into the footer of the page.
94			*
95			* @param bool[optional] $output - defaults to true, false returns but does NOT echo the code.
96			*/
97			public function insert_code( $output = true ) {
98			// If $output is not a boolean false, set it to true (default).
99			$output = ( false !== $output);
100
101			$tracking_id = $this->_get_tracking_code();
102			if ( empty( $tracking_id ) ) {
103			return $this->_output_or_return( '<!-- Your Google Analytics Plugin is missing the tracking ID -->', $output );
104			}
105
106			// If we're in the admin_area, return without inserting code.
107			if ( is_admin() ) {
108			return $this->_output_or_return( '<!-- Your Google Analytics Plugin is set to ignore Admin area -->', $output );
109			}
110
111			$custom_vars = array(
112			"_gaq.push(['_setAccount', '{$tracking_id}']);",
113			);
114
115			$track = array();
116			if ( is_404() ) {
117			// This is a 404 and we are supposed to track them.
118			$custom_vars[] = "_gaq.push( [ '_trackEvent', '404', document.location.href, document.referrer ] );";
119			} elseif ( is_search() ) {
120			// Set track for searches, if it's a search, and we are supposed to.
121			$track['data'] = sanitize_text_field( wp_unslash( $_REQUEST['s'] ) ); // Input var okay.
122			$track['code'] = 'search';
123			}
124
125			if ( ! empty( $track ) ) {
126			$track['url'] = $this->_get_url( $track );
127			// adjust the code that we output, account for both types of tracking.
128			$track['url'] = esc_js( str_replace( '&', '&', $track['url'] ) );
129			$custom_vars[] = "_gaq.push(['_trackPageview','{$track['url']}']);";
130			} else {
131			$custom_vars[] = "_gaq.push(['_trackPageview']);";
132			}
133
134			$async_code = "<!-- Jetpack Google Analytics -->
135			<script type='text/javascript'>
136			var _gaq = _gaq \|\| [];
137			%custom_vars%
138
139			(function() {
140			var ga = document.createElement('script'); ga.type = 'text/javascript'; ga.async = true;
141			ga.src = ('https:' == document.location.protocol ? 'https://ssl' : 'http://www') + '.google-analytics.com/ga.js';
142			var s = document.getElementsByTagName('script')[0]; s.parentNode.insertBefore(ga, s);
143			})();
144			</script>";
145
146			$custom_vars_string = implode( "\r\n", $custom_vars );
147			$async_code = str_replace( '%custom_vars%', $custom_vars_string, $async_code );
148
149			return $this->_output_or_return( $async_code, $output );
150			}
151
152			/**
153			* Used to get the tracking code option
154			*
155			* @return tracking code option value.
156			*/
157			private function _get_tracking_code() {
158			$o = get_option( 'jetpack_wga' );
159
160			if ( isset( $o['code'] ) && preg_match( '#UA-[\d-]+#', $o['code'], $matches ) ) {
161			return $o['code'];
162			}
163
164			return '';
165			}
166			}
167
168			global $jetpack_google_analytics;
169			$jetpack_google_analytics = Jetpack_Google_Analytics::get_instance();
170

Automattic / jetpack

Push — pr/7794 ( cb7d3e...693e58 )

Jetpack_Google_Analytics::insert_code() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like