Image::resize() - Code Metrics - Inspection of "Update Image.php" - Cecilapp/Cecil - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — feature-images ( b3daac...7e65a5 )

by Arnaud

created 2020-03-17 23:10 UTC

Image::resize() B

↳ Parent: Image

Complexity

Conditions	9
Paths	28

Size

Total Lines	59
Code Lines	29

Duplication

Lines	0
Ratio	0 %

Importance

Changes	4
Bugs	0	Features	0

Metric	Value
cc	9
eloc	29
c	4
b	0
f	0
nc	28
nop	2
dl	0
loc	59
rs	8.0555

How to fix Long Method

<?php
/*
 * This file is part of the Cecil/Cecil package.
 *
 * Copyright (c) Arnaud Ligny <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Cecil\Assets;

use Cecil\Builder;
use Cecil\Config;
use Cecil\Exception\Exception;
use Cecil\Util;
use Intervention\Image\Exception\NotReadableException;
use Intervention\Image\ImageManagerStatic as ImageManager;

class Image
{
    /** @var Config */
    private $config;
    /** @var string */
    private $path;
    /** @var int */
    private $size;
    /** @var bool */
    private $local = true;
    /** @var string */
    private $source;
    /** @var string */
    private $destination = null;

    const CACHE_IMAGES_THUMBS_DIR = 'images/thumbs';

    public function __construct(Builder $builder)
    {
        $this->config = $builder->getConfig();
    }

    /**
     * Resize an image.
     *
     * @param string $path       Image path (relative from static/ dir or external)
     * @param int    $this->size Image new size (width)
     *
     * @return string Path to image thumbnail
     */
    public function resize(string $path, int $size): string
    {
        // is not a local image?
        if (Util::isExternalUrl($path)) {
            $this->local = false;
        }

        $this->path = '/'.ltrim($path, '/');
        if (!$this->local) {
            $this->path = $path;
        }
        $this->size = $size;
        $returnPath = '/'.self::CACHE_IMAGES_THUMBS_DIR.'/'.$this->size.$this->path;

        // source file
        $this->setSource();

        // images cache path
        $this->cachePath = $this->config->getCachePath().'/'.self::CACHE_IMAGES_THUMBS_DIR.'/';


        // is size is already OK?
        list($width, $height) = getimagesize($this->source);
        if ($width <= $this->size && $height <= $this->size) {
            return $this->path;
        }

        // if GD extension is not installed: can't process
        if (!extension_loaded('gd')) {
            throw new Exception('GD extension is required to use images resize.');
        }

        $this->destination = $this->cachePath.$this->size.$this->path;

        if (Util::getFS()->exists($this->destination)) {
            return $returnPath;
        }

        // image object
        try {
            $img = ImageManager::make($this->source);
            $img->resize($this->size, null, function (\Intervention\Image\Constraint $constraint) {
                $constraint->aspectRatio();
                $constraint->upsize();
            });
        } catch (NotReadableException $e) {
            throw new Exception(sprintf('Cannot get image "%s"', $this->path));
        }

        // return data:image for external image
        if (!$this->local) {
            return (string) $img->encode('data-url');
        }

        // save file
        Util::getFS()->mkdir(dirname($this->destination));
        $img->save($this->destination);

        // return new path
        return $returnPath;
    }

    /**
     * Set source.
     */
    private function setSource(): void
    {
        if ($this->local) {
            $this->source = $this->config->getStaticPath().$this->path;
            if (!Util::getFS()->exists($this->source)) {
                throw new Exception(sprintf('Can\'t process "%s": file doesn\'t exists.', $this->source));
            }

            return;
        }

        $this->source = $this->path;
        if (!Util::isUrlFileExists($this->source)) {
            throw new Exception(sprintf('Can\'t process "%s": remonte file doesn\'t exists.', $this->source));
        }
    }
}


1			<?php
2			/*
3			* This file is part of the Cecil/Cecil package.
4			*
5			* Copyright (c) Arnaud Ligny <[email protected]>
6			*
7			* For the full copyright and license information, please view the LICENSE
8			* file that was distributed with this source code.
9			*/
10
11			namespace Cecil\Assets;
12
13			use Cecil\Builder;
14			use Cecil\Config;
15			use Cecil\Exception\Exception;
16			use Cecil\Util;
17			use Intervention\Image\Exception\NotReadableException;
18			use Intervention\Image\ImageManagerStatic as ImageManager;
19
20			class Image
21			{
22			/** @var Config */
23			private $config;
24			/** @var string */
25			private $path;
26			/** @var int */
27			private $size;
28			/** @var bool */
29			private $local = true;
30			/** @var string */
31			private $source;
32			/** @var string */
33			private $destination = null;
34
35			const CACHE_IMAGES_THUMBS_DIR = 'images/thumbs';
36
37			public function __construct(Builder $builder)
38			{
39			$this->config = $builder->getConfig();
40			}
41
42			/**
43			* Resize an image.
44			*
45			* @param string $path Image path (relative from static/ dir or external)
46			* @param int $this->size Image new size (width)
47			*
48			* @return string Path to image thumbnail
49			*/
50			public function resize(string $path, int $size): string
51			{
52			// is not a local image?
53			if (Util::isExternalUrl($path)) {
54			$this->local = false;
55			}
56
57			$this->path = '/'.ltrim($path, '/');
58			if (!$this->local) {
59			$this->path = $path;
60			}
61			$this->size = $size;
62			$returnPath = '/'.self::CACHE_IMAGES_THUMBS_DIR.'/'.$this->size.$this->path;
63
64			// source file
65			$this->setSource();
66
67			// images cache path
68			$this->cachePath = $this->config->getCachePath().'/'.self::CACHE_IMAGES_THUMBS_DIR.'/';
			0 ignored issues – show Bug Best Practice introduced 2020-03-14 21:02 UTC by Report Bug Copy Issue Report The property `cachePath` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
69
70			// is size is already OK?
71			list($width, $height) = getimagesize($this->source);
72			if ($width <= $this->size && $height <= $this->size) {
73			return $this->path;
74			}
75
76			// if GD extension is not installed: can't process
77			if (!extension_loaded('gd')) {
78			throw new Exception('GD extension is required to use images resize.');
79			}
80
81			$this->destination = $this->cachePath.$this->size.$this->path;
82
83			if (Util::getFS()->exists($this->destination)) {
84			return $returnPath;
85			}
86
87			// image object
88			try {
89			$img = ImageManager::make($this->source);
90			$img->resize($this->size, null, function (\Intervention\Image\Constraint $constraint) {
91			$constraint->aspectRatio();
92			$constraint->upsize();
93			});
94			} catch (NotReadableException $e) {
95			throw new Exception(sprintf('Cannot get image "%s"', $this->path));
96			}
97
98			// return data:image for external image
99			if (!$this->local) {
100			return (string) $img->encode('data-url');
101			}
102
103			// save file
104			Util::getFS()->mkdir(dirname($this->destination));
105			$img->save($this->destination);
106
107			// return new path
108			return $returnPath;
109			}
110
111			/**
112			* Set source.
113			*/
114			private function setSource(): void
115			{
116			if ($this->local) {
117			$this->source = $this->config->getStaticPath().$this->path;
118			if (!Util::getFS()->exists($this->source)) {
119			throw new Exception(sprintf('Can\'t process "%s": file doesn\'t exists.', $this->source));
120			}
121
122			return;
123			}
124
125			$this->source = $this->path;
126			if (!Util::isUrlFileExists($this->source)) {
127			throw new Exception(sprintf('Can\'t process "%s": remonte file doesn\'t exists.', $this->source));
128			}
129			}
130			}
131

Cecilapp / Cecil

Push — feature-images ( b3daac...7e65a5 )

Image::resize() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like