FooGallery_Thumbnails::resize() - Code Metrics - Inspection of "Merge branch 'release/1.4.30'" - fooplugins/foogallery - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 6e44b1...c642a0 )

by Brad

created 2018-05-01 18:23 UTC

FooGallery_Thumbnails::resize() F

↳ Parent: FooGallery_Thumbnails

Complexity

Conditions	22
Paths	1228

Size

Total Lines	95
Code Lines	50

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	22
eloc	50
nc	1228
nop	3
dl	0
loc	95
rs	2
c	0
b	0
f	0

How to fix Long Method Complexity

<?php
/*
 * FooGallery Thumbnail Resizing class
 */

if ( !class_exists( 'FooGallery_Thumbnails' ) ) {

	class FooGallery_Thumbnails {
namespace YourVendor;

class YourClass { }

		function __construct() {
			//generate thumbs using WPThumb
			add_filter( 'foogallery_attachment_resize_thumbnail', array( $this, 'resize' ), 10, 3 );

			add_filter( 'foogallery_thumbnail_resize_args', array( $this, 'check_for_force_original_thumb') );
		}

		function check_for_force_original_thumb( $args ){

			global $current_foogallery;
function myFunction($a, $b) {
    // Do something
}

			if ( isset( $current_foogallery ) ) {
				$args['force_use_original_thumb'] = $current_foogallery->force_use_original_thumbs;
			}

			return $args;
		}

		function resize( $original_image_src, $args, $thumbnail_object ) {
			global $current_foogallery;
function myFunction($a, $b) {
    // Do something
}
		    global $foogallery_last_generated_thumb_url;
function myFunction($a, $b) {
    // Do something
}

			$arg_defaults = array(
				'width'                   => 0,
				'height'                  => 0,
				'crop'                    => true,
				'jpeg_quality'            => foogallery_thumbnail_jpeg_quality(),
				'thumb_resize_animations' => foogallery_get_setting( 'thumb_resize_animations' ),
				'foogallery_attachment_id'=> $thumbnail_object->ID
			);

			if ( isset( $current_foogallery ) ) {
				$arg_defaults['foogallery_id'] = $current_foogallery->ID;
			}

			$args = wp_parse_args( $args, $arg_defaults );

			//allow for plugins to change the thumbnail creation args
			$args = apply_filters( 'foogallery_thumbnail_resize_args', $args, $original_image_src, $thumbnail_object );

			//check the current arguments passed in by the shortcode
			global $current_foogallery_arguments;
			if ( isset( $current_foogallery_arguments ) && isset( $current_foogallery_arguments['template'] ) ) {
				$thumbnail_args = apply_filters( 'foogallery_calculate_thumbnail_dimensions-' . $current_foogallery_arguments['template'], $args, $current_foogallery_arguments );
				$args = wp_parse_args( $thumbnail_args, $args );
			}

			$width  = (int)$args['width'];
			$height = (int)$args['height'];
			$crop   = (bool)$args['crop'];

			if ( 0 === $width && 0 === $height ) {
				return $original_image_src;
			}

			//we can force the use of the originally uploaded full-size image
			$force_use_original_image = isset( $args['force_use_original_image'] ) && true === $args['force_use_original_image'];

			if ( $force_use_original_image ) {
				$fullsize = wp_get_attachment_image_src( $thumbnail_object->ID, 'fullsize' );

				return $fullsize[0];
			}

			//we can force the use of the original WP icon or WP-generated thumb by passing through args individually
			$force_use_original_thumb = isset( $args['force_use_original_thumb'] ) && true === $args['force_use_original_thumb'];

			if ( $force_use_original_thumb ) {
				$thumbnail_icon = wp_get_attachment_image_src( $thumbnail_object->ID, array( $width, $height ) );

				return $thumbnail_icon[0];
			}

			//we can force the use of original WP thumbs by passing through args individually, or by saved settings
			$use_original_thumbs = ( isset( $args['use_original_thumbs'] ) && true === $args['use_original_thumbs'] ) || 'on' === foogallery_get_setting( 'use_original_thumbs' );

			if ( $use_original_thumbs ) {

				$option_thumbnail_size_w = get_option( 'thumbnail_size_w' );
				$option_thumbnail_size_h = get_option( 'thumbnail_size_h' );
				$option_thumbnail_crop = get_option( 'thumbnail_crop' );

				//check if we are trying to get back the default thumbnail that we already have
				if ( $thumbnail_object->ID > 0 && $width == $option_thumbnail_size_w && $height == $option_thumbnail_size_h && $crop == $option_thumbnail_crop ) {
					$thumbnail_attributes = wp_get_attachment_image_src( $thumbnail_object->ID );

					return $thumbnail_attributes[0];
				}
			}

			if ( $thumbnail_object->ID > 0 ) {
				$crop_from_position = get_post_meta( $thumbnail_object->ID, 'wpthumb_crop_pos', true );

				if ( !empty( $crop_from_position ) ) {
					$args['crop_from_position'] = $crop_from_position;
				}
			}

			//remove invalid resize args
			if ( array_key_exists( 'height', $args ) && 0 === $args['height'] ) {
				unset( $args['height'] );
			}

			//do some checks to see if the image is smaller
			if ( $this->should_resize( $thumbnail_object, $args ) ) {
				//save the generated thumb url to a global so that we can use it later if needed
				$foogallery_last_generated_thumb_url = wpthumb( $original_image_src, $args );
			} else {
				$foogallery_last_generated_thumb_url = $original_image_src;
			}

            return $foogallery_last_generated_thumb_url;
		}

		function should_resize($thumbnail_object, $args) {

			$original_width = $thumbnail_object->width;
			$original_height = $thumbnail_object->height;
			$new_width = isset( $args['width'] ) ? $args['width'] : 0;
			$new_height = isset( $args['height'] ) ? $args['height'] : 0;

			if ( $new_width > 0 && $new_height > 0 ) {
				return $original_width > $new_width || $original_height > $new_height;
			} else if ( $new_width > 0 ) {
				return $original_width > $new_width;
			}
			return $original_height > $new_height;
		}

		function run_thumbnail_generation_tests() {

            $test_image_url = foogallery_test_thumb_url();

			//next, generate a thumbnail
			$test_args = array(
				'width'                   => 20,
				'height'                  => 20,
				'crop'                    => true,
				'jpeg_quality'            => foogallery_thumbnail_jpeg_quality()
			);

            //first, clear any previous cached files
            $thumb = new WP_Thumb( $test_image_url, $test_args );
            wpthumb_rmdir_recursive( $thumb->getCacheFileDirectory() );

			$test_thumb = new WP_Thumb( $test_image_url, $test_args );
            $generated_thumb = $test_thumb->returnImage();
            $success = $test_image_url !== $generated_thumb;
			$file_info = wp_check_filetype( $test_image_url );

			$test_results = array(
			    'success' => $success,
				'thumb' => $generated_thumb,
				'error' => $test_thumb->errored() ? $test_thumb->error : '',
				'file_info' => $file_info
			);

            do_action( 'foogallery_thumbnail_generation_test', $test_results );

            return $test_results;
		}
	}
}


1			<?php
2			/*
3			* FooGallery Thumbnail Resizing class
4			*/
5
6			if ( !class_exists( 'FooGallery_Thumbnails' ) ) {
7
8			class FooGallery_Thumbnails {
			0 ignored issues – show Coding Style Compatibility introduced 2017-09-12 19:27 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...
9
10			function __construct() {
11			//generate thumbs using WPThumb
12			add_filter( 'foogallery_attachment_resize_thumbnail', array( $this, 'resize' ), 10, 3 );
13
14			add_filter( 'foogallery_thumbnail_resize_args', array( $this, 'check_for_force_original_thumb') );
15			}
16
17			function check_for_force_original_thumb( $args ){
			0 ignored issues – show Best Practice introduced 2017-05-07 14:56 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...
18			global $current_foogallery;
			0 ignored issues – show Compatibility Best Practice introduced 2017-05-07 14:56 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
19
20			if ( isset( $current_foogallery ) ) {
21			$args['force_use_original_thumb'] = $current_foogallery->force_use_original_thumbs;
22			}
23
24			return $args;
25			}
26
27			function resize( $original_image_src, $args, $thumbnail_object ) {
28			global $current_foogallery;
			0 ignored issues – show Compatibility Best Practice introduced 2017-03-21 17:22 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
29			global $foogallery_last_generated_thumb_url;
			0 ignored issues – show Compatibility Best Practice introduced 2016-07-24 16:18 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
30
31			$arg_defaults = array(
32			'width' => 0,
33			'height' => 0,
34			'crop' => true,
35			'jpeg_quality' => foogallery_thumbnail_jpeg_quality(),
36			'thumb_resize_animations' => foogallery_get_setting( 'thumb_resize_animations' ),
37			'foogallery_attachment_id'=> $thumbnail_object->ID
38			);
39
40			if ( isset( $current_foogallery ) ) {
41			$arg_defaults['foogallery_id'] = $current_foogallery->ID;
42			}
43
44			$args = wp_parse_args( $args, $arg_defaults );
45
46			//allow for plugins to change the thumbnail creation args
47			$args = apply_filters( 'foogallery_thumbnail_resize_args', $args, $original_image_src, $thumbnail_object );
48
49			//check the current arguments passed in by the shortcode
50			global $current_foogallery_arguments;
51			if ( isset( $current_foogallery_arguments ) && isset( $current_foogallery_arguments['template'] ) ) {
52			$thumbnail_args = apply_filters( 'foogallery_calculate_thumbnail_dimensions-' . $current_foogallery_arguments['template'], $args, $current_foogallery_arguments );
53			$args = wp_parse_args( $thumbnail_args, $args );
54			}
55
56			$width = (int)$args['width'];
57			$height = (int)$args['height'];
58			$crop = (bool)$args['crop'];
59
60			if ( 0 === $width && 0 === $height ) {
61			return $original_image_src;
62			}
63
64			//we can force the use of the originally uploaded full-size image
65			$force_use_original_image = isset( $args['force_use_original_image'] ) && true === $args['force_use_original_image'];
66
67			if ( $force_use_original_image ) {
68			$fullsize = wp_get_attachment_image_src( $thumbnail_object->ID, 'fullsize' );
69
70			return $fullsize[0];
71			}
72
73			//we can force the use of the original WP icon or WP-generated thumb by passing through args individually
74			$force_use_original_thumb = isset( $args['force_use_original_thumb'] ) && true === $args['force_use_original_thumb'];
75
76			if ( $force_use_original_thumb ) {
77			$thumbnail_icon = wp_get_attachment_image_src( $thumbnail_object->ID, array( $width, $height ) );
78
79			return $thumbnail_icon[0];
80			}
81
82			//we can force the use of original WP thumbs by passing through args individually, or by saved settings
83			$use_original_thumbs = ( isset( $args['use_original_thumbs'] ) && true === $args['use_original_thumbs'] ) \|\| 'on' === foogallery_get_setting( 'use_original_thumbs' );
84
85			if ( $use_original_thumbs ) {
86
87			$option_thumbnail_size_w = get_option( 'thumbnail_size_w' );
88			$option_thumbnail_size_h = get_option( 'thumbnail_size_h' );
89			$option_thumbnail_crop = get_option( 'thumbnail_crop' );
90
91			//check if we are trying to get back the default thumbnail that we already have
92			if ( $thumbnail_object->ID > 0 && $width == $option_thumbnail_size_w && $height == $option_thumbnail_size_h && $crop == $option_thumbnail_crop ) {
93			$thumbnail_attributes = wp_get_attachment_image_src( $thumbnail_object->ID );
94
95			return $thumbnail_attributes[0];
96			}
97			}
98
99			if ( $thumbnail_object->ID > 0 ) {
100			$crop_from_position = get_post_meta( $thumbnail_object->ID, 'wpthumb_crop_pos', true );
101
102			if ( !empty( $crop_from_position ) ) {
103			$args['crop_from_position'] = $crop_from_position;
104			}
105			}
106
107			//remove invalid resize args
108			if ( array_key_exists( 'height', $args ) && 0 === $args['height'] ) {
109			unset( $args['height'] );
110			}
111
112			//do some checks to see if the image is smaller
113			if ( $this->should_resize( $thumbnail_object, $args ) ) {
114			//save the generated thumb url to a global so that we can use it later if needed
115			$foogallery_last_generated_thumb_url = wpthumb( $original_image_src, $args );
116			} else {
117			$foogallery_last_generated_thumb_url = $original_image_src;
118			}
119
120			return $foogallery_last_generated_thumb_url;
121			}
122
123			function should_resize($thumbnail_object, $args) {
			0 ignored issues – show Best Practice introduced 2018-05-01 18:26 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...
124			$original_width = $thumbnail_object->width;
125			$original_height = $thumbnail_object->height;
126			$new_width = isset( $args['width'] ) ? $args['width'] : 0;
127			$new_height = isset( $args['height'] ) ? $args['height'] : 0;
128
129			if ( $new_width > 0 && $new_height > 0 ) {
130			return $original_width > $new_width \|\| $original_height > $new_height;
131			} else if ( $new_width > 0 ) {
132			return $original_width > $new_width;
133			}
134			return $original_height > $new_height;
135			}
136
137			function run_thumbnail_generation_tests() {
			0 ignored issues – show Best Practice introduced 2016-07-24 16:18 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...
138			$test_image_url = foogallery_test_thumb_url();
139
140			//next, generate a thumbnail
141			$test_args = array(
142			'width' => 20,
143			'height' => 20,
144			'crop' => true,
145			'jpeg_quality' => foogallery_thumbnail_jpeg_quality()
146			);
147
148			//first, clear any previous cached files
149			$thumb = new WP_Thumb( $test_image_url, $test_args );
150			wpthumb_rmdir_recursive( $thumb->getCacheFileDirectory() );
151
152			$test_thumb = new WP_Thumb( $test_image_url, $test_args );
153			$generated_thumb = $test_thumb->returnImage();
154			$success = $test_image_url !== $generated_thumb;
155			$file_info = wp_check_filetype( $test_image_url );
156
157			$test_results = array(
158			'success' => $success,
159			'thumb' => $generated_thumb,
160			'error' => $test_thumb->errored() ? $test_thumb->error : '',
161			'file_info' => $file_info
162			);
163
164			do_action( 'foogallery_thumbnail_generation_test', $test_results );
165
166			return $test_results;
167			}
168			}
169			}
170

fooplugins / foogallery

GitHub Access Token became invalid

Push — master ( 6e44b1...c642a0 )

FooGallery_Thumbnails::resize() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

Duplication Side-by-Side

Filter issues like