shortcodes.php ➔ jetpack_audio_atts_handler() - Code Metrics - Inspection of "Shortcodes: remove audio shortcode" - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — branch-4.1 ( 7db296...d46cf5 )

by Jeremy

created 2016-06-22 15:48 UTC

shortcodes.php ➔ jetpack_audio_atts_handler() A

↳ Parent: Project

Complexity

Conditions	2
Paths	2

Size

Total Lines	6
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	4
nc	2
nop	3
dl	0
loc	6
rs	9.4285
c	0
b	0
f	0

<?php

/**
 * Module Name: Shortcode Embeds
 * Module Description: Embed content from YouTube, Vimeo, SlideShare, and more, no coding necessary.
 * Sort Order: 3
 * First Introduced: 1.1
 * Major Changes In: 1.2
 * Requires Connection: No
 * Auto Activate: Yes
 * Module Tags: Photos and Videos, Social, Writing, Appearance
 * Additional Search Queries: shortcodes, shortcode, embeds, media, bandcamp, blip.tv, dailymotion, facebook, flickr, google calendars, google maps, google+, polldaddy, recipe, recipes, scribd, slideshare, slideshow, slideshows, soundcloud, ted, twitter, vimeo, vine, youtube
 */

/**
 * Transforms the $atts array into a string that the old functions expected
 *
 * The old way was:
 * [shortcode a=1&b=2&c=3] or [shortcode=1]
 * This is parsed as array( a => '1&b=2&c=3' ) and array( 0 => '=1' ), which is useless
 *
 * @param Array $params
 * @param Bool $old_format_support true if [shortcode=foo] format is possible.
 * @return String $params
 */
function shortcode_new_to_old_params( $params, $old_format_support = false ) {
	$str = '';

	if ( $old_format_support && isset( $params[0] ) ) {
		$str = ltrim( $params[0], '=' );
	} elseif ( is_array( $params ) ) {
		foreach ( array_keys( $params ) as $key ) {
			if ( ! is_numeric( $key ) )
				$str = $key . '=' . $params[$key];
		}
	}

	return str_replace( array( '&amp;', '&#038;' ), '&', $str );
}

function jetpack_load_shortcodes() {

	global $wp_version;

	$shortcode_includes = array();

	foreach ( Jetpack::glob_php( dirname( __FILE__ ) . '/shortcodes' ) as $file ) {
		$shortcode_includes[] = $file;
	}

/**
 * This filter allows other plugins to override which shortcodes Jetpack loads.
 *
 * @module shortcodes
 *
 * @since 2.2.1
 *
 * @param array $shortcode_includes An array of which shortcodes to include.
 */
	$shortcode_includes = apply_filters( 'jetpack_shortcodes_to_include', $shortcode_includes );

	foreach ( $shortcode_includes as $include ) {
		include $include;
	}
}

/**
 * Runs preg_replace so that replacements don't happen within open tags.
 * Parameters are the same as preg_replace, with an added optional search param for improved performance
 *
 * @param String $pattern
 * @param String $replacement
 * @param String $content
 * @param String $search

 * @return String $content
 */
function jetpack_preg_replace_outside_tags( $pattern, $replacement, $content, $search = null ) {

	if( ! function_exists( 'wp_html_split' ) ) {
		return $content;
	}

	if ( $search && false === strpos( $content, $search ) ) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
		return $content;
	}

	$textarr = wp_html_split( $content );
	unset( $content );
	foreach( $textarr as &$element ) {
	    if ( '' === $element || '<' === $element{0} )
	        continue;
	    $element = preg_replace( $pattern, $replacement, $element );
	}

	return join( $textarr );
}

/**
 * Runs preg_replace_callback so that replacements don't happen within open tags.
 * Parameters are the same as preg_replace, with an added optional search param for improved performance
 *
 * @param String $pattern
 * @param String $replacement
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
 * @param String $content
 * @param String $search

 * @return String $content
 */
function jetpack_preg_replace_callback_outside_tags( $pattern, $callback, $content, $search = null ) {

	if( ! function_exists( 'wp_html_split' ) ) {
		return $content;
	}

	if ( $search && false === strpos( $content, $search ) ) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
		return $content;
	}

	$textarr = wp_html_split( $content );
	unset( $content );
	foreach( $textarr as &$element ) {
	    if ( '' === $element || '<' === $element{0} )
	        continue;
	    $element = preg_replace_callback( $pattern, $callback, $element );
	}

	return join( $textarr );
}

if ( ! function_exists( 'jetpack_shortcode_get_wpvideo_id' ) ) {
	function jetpack_shortcode_get_wpvideo_id( $atts ) {
		if ( isset( $atts[ 0 ] ) )
			return $atts[ 0 ];
		else
			return 0;
	}
}

jetpack_load_shortcodes();


1		<?php
2
3		/**
4		* Module Name: Shortcode Embeds
5		* Module Description: Embed content from YouTube, Vimeo, SlideShare, and more, no coding necessary.
6		* Sort Order: 3
7		* First Introduced: 1.1
8		* Major Changes In: 1.2
9		* Requires Connection: No
10		* Auto Activate: Yes
11		* Module Tags: Photos and Videos, Social, Writing, Appearance
12		* Additional Search Queries: shortcodes, shortcode, embeds, media, bandcamp, blip.tv, dailymotion, facebook, flickr, google calendars, google maps, google+, polldaddy, recipe, recipes, scribd, slideshare, slideshow, slideshows, soundcloud, ted, twitter, vimeo, vine, youtube
13		*/
14
15		/**
16		* Transforms the $atts array into a string that the old functions expected
17		*
18		* The old way was:
19		* [shortcode a=1&b=2&c=3] or [shortcode=1]
20		* This is parsed as array( a => '1&b=2&c=3' ) and array( 0 => '=1' ), which is useless
21		*
22		* @param Array $params
23		* @param Bool $old_format_support true if [shortcode=foo] format is possible.
24		* @return String $params
25		*/
26		function shortcode_new_to_old_params( $params, $old_format_support = false ) {
27		$str = '';
28
29		if ( $old_format_support && isset( $params[0] ) ) {
30		$str = ltrim( $params[0], '=' );
31		} elseif ( is_array( $params ) ) {
32		foreach ( array_keys( $params ) as $key ) {
33		if ( ! is_numeric( $key ) )
34		$str = $key . '=' . $params[$key];
35		}
36		}
37
38		return str_replace( array( '&', '&' ), '&', $str );
39		}
40
41	View Code Duplication	function jetpack_load_shortcodes() {
		0 ignored issues – show Duplication introduced 2016-05-03 07:34 UTC by Report Bug Copy Issue Report This function seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
42		global $wp_version;
43
44		$shortcode_includes = array();
45
46		foreach ( Jetpack::glob_php( dirname( __FILE__ ) . '/shortcodes' ) as $file ) {
47		$shortcode_includes[] = $file;
48		}
49
50		/**
51		* This filter allows other plugins to override which shortcodes Jetpack loads.
52		*
53		* @module shortcodes
54		*
55		* @since 2.2.1
56		*
57		* @param array $shortcode_includes An array of which shortcodes to include.
58		*/
59		$shortcode_includes = apply_filters( 'jetpack_shortcodes_to_include', $shortcode_includes );
60
61		foreach ( $shortcode_includes as $include ) {
62		include $include;
63		}
64		}
65
66		/**
67		* Runs preg_replace so that replacements don't happen within open tags.
68		* Parameters are the same as preg_replace, with an added optional search param for improved performance
69		*
70		* @param String $pattern
71		* @param String $replacement
72		* @param String $content
73		* @param String $search
		0 ignored issues – show Documentation introduced 2016-06-17 16:21 UTC by Report Bug Copy Issue Report Should the type for parameter `$search` not be `string\|null`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
74		* @return String $content
75		*/
76	View Code Duplication	function jetpack_preg_replace_outside_tags( $pattern, $replacement, $content, $search = null ) {
		0 ignored issues – show Duplication introduced 2016-05-27 01:47 UTC by Report Bug Copy Issue Report This function seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
77		if( ! function_exists( 'wp_html_split' ) ) {
78		return $content;
79		}
80
81		if ( $search && false === strpos( $content, $search ) ) {
		0 ignored issues – show Bug Best Practice introduced 2016-06-17 16:21 UTC by Report Bug Copy Issue Report The expression `$search` of type `string\|null` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
82		return $content;
83		}
84
85		$textarr = wp_html_split( $content );
86		unset( $content );
87		foreach( $textarr as &$element ) {
88		if ( '' === $element \|\| '<' === $element{0} )
89		continue;
90		$element = preg_replace( $pattern, $replacement, $element );
91		}
92
93		return join( $textarr );
94		}
95
96		/**
97		* Runs preg_replace_callback so that replacements don't happen within open tags.
98		* Parameters are the same as preg_replace, with an added optional search param for improved performance
99		*
100		* @param String $pattern
101		* @param String $replacement
		0 ignored issues – show Bug introduced 2016-05-27 01:47 UTC by Report Bug Copy Issue Report There is no parameter named `$replacement`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history...
102		* @param String $content
103		* @param String $search
		0 ignored issues – show Documentation introduced 2016-06-17 16:21 UTC by Report Bug Copy Issue Report Should the type for parameter `$search` not be `string\|null`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
104		* @return String $content
105		*/
106	View Code Duplication	function jetpack_preg_replace_callback_outside_tags( $pattern, $callback, $content, $search = null ) {
		0 ignored issues – show Duplication introduced 2016-05-27 01:47 UTC by Report Bug Copy Issue Report This function seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
107		if( ! function_exists( 'wp_html_split' ) ) {
108		return $content;
109		}
110
111		if ( $search && false === strpos( $content, $search ) ) {
		0 ignored issues – show Bug Best Practice introduced 2016-06-17 16:21 UTC by Report Bug Copy Issue Report The expression `$search` of type `string\|null` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
112		return $content;
113		}
114
115		$textarr = wp_html_split( $content );
116		unset( $content );
117		foreach( $textarr as &$element ) {
118		if ( '' === $element \|\| '<' === $element{0} )
119		continue;
120		$element = preg_replace_callback( $pattern, $callback, $element );
121		}
122
123		return join( $textarr );
124		}
125
126		if ( ! function_exists( 'jetpack_shortcode_get_wpvideo_id' ) ) {
127		function jetpack_shortcode_get_wpvideo_id( $atts ) {
128		if ( isset( $atts[ 0 ] ) )
129		return $atts[ 0 ];
130		else
131		return 0;
132		}
133		}
134
135		jetpack_load_shortcodes();
136

Automattic / jetpack

Push — branch-4.1 ( 7db296...d46cf5 )

shortcodes.php ➔ jetpack_audio_atts_handler() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like