WebPStrategy::getFileExtension() - Code Metrics - Inspection of "switch strategy before save test" - delboy1978uk/image - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( aae6d4...77a0b7 )

by Derek Stephen

created 2020-09-10 21:51 UTC

WebPStrategy::getFileExtension() A

↳ Parent: WebPStrategy

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
eloc	1
c	0
b	0
f	0
dl	0
loc	3
ccs	0
cts	2
cp	0
rs	10
cc	1
nc	1
nop	0
crap	2

<?php

namespace Del\Image\Strategy;

class WebPStrategy extends TransparentStrategy implements ImageTypeStrategyInterface
{
    /**
     * @param string $filename
     * @return resource
     */
    public function create(string $filename)
    {
        return \imagecreatefromwebp($filename);

    }

    /**
     * @param resource $resource
     * @param string $filename
     * @param int $compression
     * @return void
     */
    public function save($resource, string $filename = null, int $compression = 100): void
    {
        \imagewebp($resource, $filename, $compression);
    }

    /**
     * @return string
     */
    public function getContentType(): string
    {
        return 'image/webp';
    }

    /**
     * @param $resource
     */
    public function render($resource): void
    {
        imagealphablending($resource, true);
        imagesavealpha($resource, true);
        imagewebp($resource);
    }

    /**
     * @return string
     */
    public function getFileExtension(): string
    {
        return 'webp';
    }
}

1		<?php
2
3		namespace Del\Image\Strategy;
4
5		class WebPStrategy extends TransparentStrategy implements ImageTypeStrategyInterface
6		{
7		/**
8		* @param string $filename
9		* @return resource
10		*/
11	2	public function create(string $filename)
12		{
13	2	return \imagecreatefromwebp($filename);
		0 ignored issues – show Bug Best Practice introduced 2020-06-04 17:43 UTC by Report Bug Copy Issue Report The expression `return imagecreatefromwebp($filename)` could also return `false` which is incompatible with the documented return type `resource`. Did you maybe forget to handle an error condition? If the returned type also contains false, it is an indicator that maybe an error condition leading to the specific return statement remains unhandled. Loading history...
14		}
15
16		/**
17		* @param resource $resource
18		* @param string $filename
19		* @param int $compression
20		* @return void
21		*/
22	2	public function save($resource, string $filename = null, int $compression = 100): void
23		{
24	2	\imagewebp($resource, $filename, $compression);
25	2	}
26
27		/**
28		* @return string
29		*/
30	1	public function getContentType(): string
31		{
32	1	return 'image/webp';
33		}
34
35		/**
36		* @param $resource
37		*/
38	2	public function render($resource): void
39		{
40	2	imagealphablending($resource, true);
41	2	imagesavealpha($resource, true);
42	2	imagewebp($resource);
43	2	}
44
45		/**
46		* @return string
47		*/
48		public function getFileExtension(): string
49		{
50		return 'webp';
51		}
52		}

delboy1978uk / image

Push — master ( aae6d4...77a0b7 )

WebPStrategy::getFileExtension() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like