Path::withScheme() - Code Metrics - Inspection of "Use static checkers more (#18)" - Nyholm/dsn - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 3268de...6bd5d5 )

by Tobias

created 2020-10-03 13:53 UTC

Path::withScheme() A

↳ Parent: Path

Complexity

Conditions	3
Paths	2

Size

Total Lines	7
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	3
c	0
b	0
f	0
dl	0
loc	7
rs	10
cc	3
nc	2
nop	1

<?php

declare(strict_types=1);

namespace Nyholm\Dsn\Configuration;

use Nyholm\Dsn\Exception\InvalidArgumentException;

/**
 * A "path like" DSN string.
 *
 * Example:
 * - redis:///var/run/redis/redis.sock
 * - memcached://user:password@/var/local/run/memcached.socket?weight=25
 *
 * @author Tobias Nyholm <[email protected]>
 */
class Path extends Dsn
{
    use UserPasswordTrait;
    /**
     * @var string
     */
    private $path;

    /**
     * @param array{ user?: string|null, password?: string|null, } $authentication

     */
    public function __construct(string $scheme, string $path, array $parameters = [], array $authentication = [])
    {
        $this->path = $path;
        $this->setAuthentication($authentication);
        parent::__construct($scheme, $parameters);
    }

    public function getScheme(): string
    {
        return parent::getScheme();

    }

    /**
     * @return static
     */
    public function withScheme(?string $scheme)
    {
        if (null === $scheme || '' === $scheme) {
            throw new InvalidArgumentException('A Path must have a schema');
        }

        return parent::withScheme($scheme);
    }

    public function getPath(): string
    {
        return $this->path;
    }

    /**
     * @return static
     */
    public function withPath(string $path)
    {
        $new = clone $this;
        $new->path = $path;

        return $new;
    }

    /**
     * @var string
     */
    public function __toString()
    {
        $parameters = $this->getParameters();

        return
            $this->getScheme().'://'.
            $this->getUserInfoString().
            $this->getPath().
            (empty($parameters) ? '' : '?'.http_build_query($parameters));
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Nyholm\Dsn\Configuration;
6
7			use Nyholm\Dsn\Exception\InvalidArgumentException;
8
9			/**
10			* A "path like" DSN string.
11			*
12			* Example:
13			* - redis:///var/run/redis/redis.sock
14			* - memcached://user:password@/var/local/run/memcached.socket?weight=25
15			*
16			* @author Tobias Nyholm <[email protected]>
17			*/
18			class Path extends Dsn
19			{
20			use UserPasswordTrait;
21			/**
22			* @var string
23			*/
24			private $path;
25
26			/**
27			* @param array{ user?: string\|null, password?: string\|null, } $authentication
			0 ignored issues – show Documentation Bug introduced 2020-10-03 13:54 UTC by Report Bug Copy Issue Report The doc comment `user?: string\|null, password?: string\|null, }` at position `0` could not be parsed: Unknown type name 'user?' at position 0 in user?: string\|null, password?: string\|null, }. Loading history...
28			*/
29			public function __construct(string $scheme, string $path, array $parameters = [], array $authentication = [])
30			{
31			$this->path = $path;
32			$this->setAuthentication($authentication);
33			parent::__construct($scheme, $parameters);
34			}
35
36			public function getScheme(): string
37			{
38			return parent::getScheme();
			0 ignored issues – show Bug Best Practice introduced 2020-10-03 13:54 UTC by Report Bug Copy Issue Report The expression `return parent::getScheme()` could return the type `null` which is incompatible with the type-hinted return `string`. Consider adding an additional type-check to rule them out. Loading history...
39			}
40
41			/**
42			* @return static
43			*/
44			public function withScheme(?string $scheme)
45			{
46			if (null === $scheme \|\| '' === $scheme) {
47			throw new InvalidArgumentException('A Path must have a schema');
48			}
49
50			return parent::withScheme($scheme);
51			}
52
53			public function getPath(): string
54			{
55			return $this->path;
56			}
57
58			/**
59			* @return static
60			*/
61			public function withPath(string $path)
62			{
63			$new = clone $this;
64			$new->path = $path;
65
66			return $new;
67			}
68
69			/**
70			* @var string
71			*/
72			public function __toString()
73			{
74			$parameters = $this->getParameters();
75
76			return
77			$this->getScheme().'://'.
78			$this->getUserInfoString().
79			$this->getPath().
80			(empty($parameters) ? '' : '?'.http_build_query($parameters));
81			}
82			}
83

Nyholm / dsn

Push — master ( 3268de...6bd5d5 )

Path::withScheme() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like