FooGallery_Extensions_Compatibility::add_video_class_to_item() - Code Metrics - fooplugins/foogallery - Measure and Improve Code Quality continuously with Scrutinizer

add_video_class_to_item() A
last analyzed 2019-02-27 17:41 UTC

↳ Parent: FooGallery_Extensions_Compatibility

Complexity

Conditions	4
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
nc	3
nop	3
dl	0
loc	15
rs	9.7666
c	0
b	0
f	0

<?php
/**
 * FooGallery Extensions Compatibility Class
 * Date: 20 Sep 2017
 *
 * This class is used to make any overrides that are needed in extensions when updating FooGallery.
 * These are "hacks" that will make the upgrade process easier, and not have the requirement to upgrade the individual extensions.
 *
 */
if ( ! class_exists( 'FooGallery_Extensions_Compatibility' ) ) {

    class FooGallery_Extensions_Compatibility {
namespace YourVendor;

class YourClass { }

        function __construct() {

			add_filter( 'foogallery_attachment_html_item_classes', array( $this, 'add_video_class_to_item' ), 10, 3 );
        }

		/**
		 * @param $classes
		 * @param $foogallery_attachment
		 * @param $args
		 *
		 * @return mixed
		 */
		public function add_video_class_to_item( $classes, $foogallery_attachment, $args ) {

			if ( class_exists( 'Foo_Video' ) ) {
				$video_info = get_post_meta( $foogallery_attachment->ID, FOO_VIDEO_POST_META, true );
				if ( $video_info && isset( $video_info['id'] ) ) {
					//we are dealing with a video
					$classes[] = 'fg-video';

					//include a specific css file to override issues with video hover effects
					$css = FOOGALLERY_URL . 'css/foogallery-foovideo-overrides.css';
					foogallery_enqueue_style( 'foogallery_foovideo_overrides', $css, array(), FOOGALLERY_VERSION );
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				}
			}

            return $classes;
		}
    }
}


1			<?php
2			/**
3			* FooGallery Extensions Compatibility Class
4			* Date: 20 Sep 2017
5			*
6			* This class is used to make any overrides that are needed in extensions when updating FooGallery.
7			* These are "hacks" that will make the upgrade process easier, and not have the requirement to upgrade the individual extensions.
8			*
9			*/
10			if ( ! class_exists( 'FooGallery_Extensions_Compatibility' ) ) {
11
12			class FooGallery_Extensions_Compatibility {
			0 ignored issues – show Coding Style Compatibility introduced 2017-09-20 19:07 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
13
14			function __construct() {
			0 ignored issues – show Best Practice introduced 2017-10-24 19:58 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
15			add_filter( 'foogallery_attachment_html_item_classes', array( $this, 'add_video_class_to_item' ), 10, 3 );
16			}
17
18			/**
19			* @param $classes
20			* @param $foogallery_attachment
21			* @param $args
22			*
23			* @return mixed
24			*/
25			public function add_video_class_to_item( $classes, $foogallery_attachment, $args ) {
			0 ignored issues – show Unused Code introduced 2017-09-20 19:07 UTC by Report Bug Copy Issue Report The parameter `$args` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
26			if ( class_exists( 'Foo_Video' ) ) {
27			$video_info = get_post_meta( $foogallery_attachment->ID, FOO_VIDEO_POST_META, true );
28			if ( $video_info && isset( $video_info['id'] ) ) {
29			//we are dealing with a video
30			$classes[] = 'fg-video';
31
32			//include a specific css file to override issues with video hover effects
33			$css = FOOGALLERY_URL . 'css/foogallery-foovideo-overrides.css';
34			foogallery_enqueue_style( 'foogallery_foovideo_overrides', $css, array(), FOOGALLERY_VERSION );
			0 ignored issues – show Documentation introduced 2017-09-20 19:07 UTC by Report Bug Copy Issue Report `FOOGALLERY_VERSION` is of type `string`, but the function expects a `boolean`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
35			}
36			}
37
38			return $classes;
39			}
40			}
41			}
42

fooplugins / foogallery

GitHub Access Token became invalid

add_video_class_to_item() A last analyzed 2019-02-27 17:41 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

add_video_class_to_item() A
last analyzed 2019-02-27 17:41 UTC