FileExtensionValidator::getMessages() - Code Metrics - Inspection of "tests back up at 100%" - delboy1978uk/form - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( fcb641...65474d )

by Derek Stephen

created 2020-09-25 16:08 UTC

FileExtensionValidator::getMessages() A

↳ Parent: FileExtensionValidator

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	14
ccs	9
cts	9
cp	1
rs	9.7998
c	0
b	0
f	0
cc	2
nc	2
nop	0
crap	2

<?php

namespace Del\Form\Validator;

class FileExtensionValidator implements ValidatorInterface
{
    /** @var array $validExtensions */
    private $validExtensions;

    /**
     * FileExtensionValidator constructor.
     * @param array $validExtensions
     */
    public function __construct(array $validExtensions)
    {
        $this->validExtensions = $validExtensions;
    }

    /**
     * @param mixed $value
     * @return bool|void
     */
    public function isValid($value)
    {
        $debris = explode('.', $value);
        $extension = end($debris);
        $extension = strtolower($extension);

        return in_array($extension, $this->validExtensions);
    }

    /**
     * @return array|void
     */
    public function getMessages()
    {
        $validExtensions = $this->validExtensions;
        $last = array_pop($validExtensions);
        $extensions = 'The file must be a ' . $last . ' file.';

        if (count($validExtensions) > 0) {
            $extensions = implode(', ', $validExtensions);
            $extensions .= ' or ' . $last;
            $extensions = 'The file must be either a ' . $extensions . ' file.';
        }

        return [$extensions];
    }
}

1		<?php
2
3		namespace Del\Form\Validator;
4
5		class FileExtensionValidator implements ValidatorInterface
6		{
7		/** @var array $validExtensions */
8		private $validExtensions;
9
10		/**
11		* FileExtensionValidator constructor.
12		* @param array $validExtensions
13		*/
14	1	public function __construct(array $validExtensions)
15		{
16	1	$this->validExtensions = $validExtensions;
17	1	}
18
19		/**
20		* @param mixed $value
21		* @return bool\|void
22		*/
23	1	public function isValid($value)
24		{
25	1	$debris = explode('.', $value);
26	1	$extension = end($debris);
27	1	$extension = strtolower($extension);
28
29	1	return in_array($extension, $this->validExtensions);
30		}
31
32		/**
33		* @return array\|void
34		*/
35	1	public function getMessages()
36		{
37	1	$validExtensions = $this->validExtensions;
38	1	$last = array_pop($validExtensions);
39	1	$extensions = 'The file must be a ' . $last . ' file.';
40
41	1	if (count($validExtensions) > 0) {
42	1	$extensions = implode(', ', $validExtensions);
43	1	$extensions .= ' or ' . $last;
44	1	$extensions = 'The file must be either a ' . $extensions . ' file.';
45		}
46
47	1	return [$extensions];
48		}
49		}

delboy1978uk / form

Push — master ( fcb641...65474d )

FileExtensionValidator::getMessages() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like