GravityView_Image

Complexity

Total Complexity

29

Size/Duplication

Total Lines	196
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__toString()	0	3	1
B	__construct()	0	25	3
A	validate_image_src()	0	14	3
C	set_image_size()	0	66	17
B	html()	0	23	5

<?php
/**
 * Generic class for generating image tag
 */
class GravityView_Image {

	var $alt;

The visibility should be declared for property $alt.

	var $src;

The visibility should be declared for property $src.

	var $title;

The visibility should be declared for property $title.

	var $width;

The visibility should be declared for property $width.

	var $height;

The visibility should be declared for property $height.

	/**
	 * HTML class attribute.
	 * @var string
	 */
	var $class = NULL;

The visibility should be declared for property $class.

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

	/**
	 * HTML style attribute
	 * @var string
	 */
	var $style = NULL;

The visibility should be declared for property $style.

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

	/**
 	 * String representing size of image - Choose from "full", "medium", "thumb", "tiny"
	 * @var string
	 */
	var $size = NULL;

The visibility should be declared for property $size.

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

	/**
	 * Use PHP getimagesize function to auto-calculate image size?
	 * @var boolean
	 */
	var $getimagesize = false;

The visibility should be declared for property $getimagesize.

	/**
	 * Check the src to make sure it looks like a valid image URL
	 * @var boolean
	 */
	var $validate_src = true;

The visibility should be declared for property $validate_src.

	/**
	 * Handle being treated as a string by returning the HTML
	 * @return string HTML of image
	 */
	function __toString() {

It is generally recommended to explicitly declare the visibility for methods.

		return $this->html();
	}


	function __construct( $atts = array() ) {

It is generally recommended to explicitly declare the visibility for methods.

		$defaults = array(
			'width' => $this->width,
			'height' => $this->height,
			'alt' => $this->alt,
			'title' => $this->title,
			'size' => $this->size,
			'src' => $this->src,
			'class' => $this->class,
			'getimagesize' => false,
			'validate_src' => true

Each line in an array declaration must end in a comma

		);

		$atts = wp_parse_args( $atts, $defaults );

		foreach( $atts as $key => $val ) {
			$this->{$key} = $val;
		}

		$this->class = !empty( $this->class ) ? esc_attr( implode( ' ', (array)$this->class ) ) : $this->class;

Expected 1 space after "!"; 0 found

No space after closing casting parenthesis is prohibited

		$this->set_image_size();

	}

	/**
	 * Verify that the src URL matches image patterns.
	 *
	 * Yes, images can not have extensions, but this is a basic check. To disable this,
	 * set `validate_src` to false when instantiating the object.
	 *
	 * @return boolean     True: matches pattern; False: does not match pattern.
	 */
	function validate_image_src() {

It is generally recommended to explicitly declare the visibility for methods.

		if ( !$this->validate_src ) { return true; }

Expected 1 space after "!"; 0 found

		$info = pathinfo( $this->src );

		/**
		 * @filter `gravityview_image_extensions` Extensions that GravityView recognizes as valid images to be shown in an `img` tag
		 * @param array $image_exts Default: `['jpg', 'jpeg', 'jpe', 'gif', 'png', 'bmp', 'tif', 'tiff', 'ico']`
		 */
		$image_exts = apply_filters( 'gravityview_image_extensions', array( 'jpg', 'jpeg', 'jpe', 'gif', 'png', 'bmp', 'tif', 'tiff', 'ico' ));

Expected 1 spaces before closing bracket; 0 found

		return isset( $info['extension'] ) && in_array( strtolower( $info['extension'] ), $image_exts);

Expected 1 spaces before closing bracket; 0 found

	}

	/**
	 * Get default widths and heights for image size.
	 *
	 * @return void
	 */
	public function set_image_size( $string = NULL, $width = NULL, $height = NULL ) {

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

Functions must not contain multiple empty lines in a row; found 2 empty lines

		// If there is no width or height passed
		if ( empty( $width ) || empty( $height ) ) {

			// And there is no string size passed
			// 		And we want to get the image size using PHP
			if ( empty( $string ) && !empty( $this->getimagesize ) ) {

Expected 1 space after "!"; 0 found

				$image_size = @getimagesize( $this->src );

Silencing errors is discouraged

				// If it didn't return a response, it may be a HTTPS/SSL error
				if ( empty( $image_size[0] ) ) {
					$image_size = @getimagesize( set_url_scheme( $this->src, 'http' ) );

Silencing errors is discouraged

				}

				if ( !empty( $image_size ) ) {

Expected 1 space after "!"; 0 found

					list( $width, $height ) = $image_size;
				}

			}
			// Otherwise, we calculate based on the string size value
			else {

				/**
				 * @filter `gravityview_image_sizes` Modify the image size presets used by GravityView_Image class
				 * @param array $image_sizes Array of image sizes with the key being the size slug, and the value being an array with `width` and `height` defined, in pixels
				 */
				$image_sizes = apply_filters( 'gravityview_image_sizes', array(
					'tiny' => array('width' => 40, 'height' => 30),

No space after opening parenthesis of array is bad style

No space before closing parenthesis of array is bad style

					'small' => array('width' => 100, 'height' => 75),

No space after opening parenthesis of array is bad style

No space before closing parenthesis of array is bad style

					'medium' => array('width' => 250, 'height' => 188),

No space after opening parenthesis of array is bad style

No space before closing parenthesis of array is bad style

					'large' => array('width' => 448, 'height' => 336),

No space after opening parenthesis of array is bad style

No space before closing parenthesis of array is bad style

				) );

				switch( $this->size ) {
					case 'tiny':
						extract($image_sizes['tiny']);

extract() usage is highly discouraged, due to the complexity and unintended issues it might cause.

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

						break;
					case 'small':
					case 's':
					case 'thumb':
						extract($image_sizes['small']);

extract() usage is highly discouraged, due to the complexity and unintended issues it might cause.

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

						break;
					case 'm':
					case 'medium':
						extract($image_sizes['medium']);

extract() usage is highly discouraged, due to the complexity and unintended issues it might cause.

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

						break;
					case 'large':
					case 'l':
						extract($image_sizes['large']);

extract() usage is highly discouraged, due to the complexity and unintended issues it might cause.

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

						break;
					default:
						// Verify that the passed sizes are integers.
						$width = !empty( $width ) ? intval( $width ) : intval( $this->width );

Expected 1 space after "!"; 0 found

						$height = !empty( $height ) ? intval( $height ) : intval( $this->height );

Expected 1 space after "!"; 0 found

				}

			}

		}

		$this->width = $width;
		$this->height = $height;
	}

	/**
	 * Return the HTML tag for the image
	 * @return string HTML of the image
	 */
	public function html() {

		$html = '';

		if ( $this->validate_image_src() && ! empty( $this->src ) ) {
			$atts = '';
			foreach ( array( 'width', 'height', 'alt', 'title', 'class' ) as $attr ) {

				if ( empty( $this->{$attr} ) ) { continue; }

				$atts .= sprintf( ' %s="%s"', $attr, esc_attr( $this->{$attr} ) );
			}

			$html = sprintf( '<img src="%s"%s />', esc_url_raw( $this->src ), $atts );
		}

		/**
		 * @filter `gravityview_image_html` Filter the HTML image output
		 * @param string $html the generated image html
		 * @param GravityView_Image $this The current image object
		 */
		return apply_filters( 'gravityview_image_html', $html, $this );
	}
}