TimeRange::overflowsNextDay() - Code Metrics - Inspection of "Merge pull request #112 from kylekatarnls/feature/..." - spatie/opening-hours - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( e31ec7...148746 )

by Kyle

created 2019-06-04 15:54 UTC

TimeRange::overflowsNextDay() A

↳ Parent: TimeRange

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	0

<?php

namespace Spatie\OpeningHours;

use Spatie\OpeningHours\Helpers\DataTrait;
use Spatie\OpeningHours\Exceptions\InvalidTimeRangeList;
use Spatie\OpeningHours\Exceptions\InvalidTimeRangeArray;
use Spatie\OpeningHours\Exceptions\InvalidTimeRangeString;

class TimeRange
{
    use DataTrait;

    /** @var \Spatie\OpeningHours\Time */
    protected $start;

    /** @var \Spatie\OpeningHours\Time */
    protected $end;

    protected function __construct(Time $start, Time $end)
    {
        $this->start = $start;
        $this->end = $end;
    }

    public static function fromString(string $string): self
    {
        $times = explode('-', $string);

        if (count($times) !== 2) {
            throw InvalidTimeRangeString::forString($string);
        }

        return new self(Time::fromString($times[0]), Time::fromString($times[1]));
    }

    public static function fromArray(array $array): self
    {
        $values = [];
        $keys = ['hours', 'data'];

        foreach ($keys as $key) {
            if (isset($array[$key])) {
                $values[$key] = $array[$key];
                unset($array[$key]);

                continue;
            }
        }

        foreach ($keys as $key) {
            if (! isset($values[$key])) {
                $values[$key] = array_shift($array);
            }
        }

        if (! $values['hours']) {
            throw InvalidTimeRangeArray::create();
        }

        return static::fromString($values['hours'])->setData($values['data']);
    }

    public static function fromDefinition($value): self
    {
        return is_array($value) ? static::fromArray($value) : static::fromString($value);
    }

    public static function fromList(array $ranges): self
    {
        if (count($ranges) === 0) {
            throw InvalidTimeRangeList::create();
        }

        foreach ($ranges as $range) {
            if (! ($range instanceof self)) {
                throw InvalidTimeRangeList::create();
            }
        }

        $start = $ranges[0]->start();
        $end = $ranges[0]->end();

        foreach (array_slice($ranges, 1) as $range) {
            $rangeStart = $range->start();
            if ($rangeStart->format('Gi') < $start->format('Gi')) {
                $start = $rangeStart;
            }
            $rangeEnd = $range->end();
            if ($rangeEnd->format('Gi') > $end->format('Gi')) {
                $end = $rangeEnd;
            }
        }

        return new self($start, $end);
    }

    public static function fromMidnight(Time $end)
    {
        return new self(Time::fromString('00:00'), $end);
    }

    public function start(): Time
    {
        return $this->start;
    }

    public function end(): Time
    {
        return $this->end;
    }

    public function spillsOverToNextDay(): bool
    {
        return $this->end->isBefore($this->start);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    public function containsTime(Time $time): bool
    {
        if ($this->spillsOverToNextDay()) {
            if ($time->isSameOrAfter($this->start)) {
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                return $time->isAfter($this->end);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            }

            return $time->isBefore($this->end);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        }

        return $time->isSameOrAfter($this->start) && $time->isBefore($this->end);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    public function overlaps(self $timeRange): bool
    {
        return $this->containsTime($timeRange->start) || $this->containsTime($timeRange->end);
    }

    public function format(string $timeFormat = 'H:i', string $rangeFormat = '%s-%s'): string
    {
        return sprintf($rangeFormat, $this->start->format($timeFormat), $this->end->format($timeFormat));
    }

    public function isReversed()
    {
        return $this->start()->isAfter($this->end());
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    public function overflowsNextDay()
    {
        return $this->isReversed();
    }

    public function __toString(): string
    {
        return $this->format();
    }
}


1			<?php
2
3			namespace Spatie\OpeningHours;
4
5			use Spatie\OpeningHours\Helpers\DataTrait;
6			use Spatie\OpeningHours\Exceptions\InvalidTimeRangeList;
7			use Spatie\OpeningHours\Exceptions\InvalidTimeRangeArray;
8			use Spatie\OpeningHours\Exceptions\InvalidTimeRangeString;
9
10			class TimeRange
11			{
12			use DataTrait;
13
14			/** @var \Spatie\OpeningHours\Time */
15			protected $start;
16
17			/** @var \Spatie\OpeningHours\Time */
18			protected $end;
19
20			protected function __construct(Time $start, Time $end)
21			{
22			$this->start = $start;
23			$this->end = $end;
24			}
25
26			public static function fromString(string $string): self
27			{
28			$times = explode('-', $string);
29
30			if (count($times) !== 2) {
31			throw InvalidTimeRangeString::forString($string);
32			}
33
34			return new self(Time::fromString($times[0]), Time::fromString($times[1]));
35			}
36
37			public static function fromArray(array $array): self
38			{
39			$values = [];
40			$keys = ['hours', 'data'];
41
42			foreach ($keys as $key) {
43			if (isset($array[$key])) {
44			$values[$key] = $array[$key];
45			unset($array[$key]);
46
47			continue;
48			}
49			}
50
51			foreach ($keys as $key) {
52			if (! isset($values[$key])) {
53			$values[$key] = array_shift($array);
54			}
55			}
56
57			if (! $values['hours']) {
58			throw InvalidTimeRangeArray::create();
59			}
60
61			return static::fromString($values['hours'])->setData($values['data']);
62			}
63
64			public static function fromDefinition($value): self
65			{
66			return is_array($value) ? static::fromArray($value) : static::fromString($value);
67			}
68
69			public static function fromList(array $ranges): self
70			{
71			if (count($ranges) === 0) {
72			throw InvalidTimeRangeList::create();
73			}
74
75			foreach ($ranges as $range) {
76			if (! ($range instanceof self)) {
77			throw InvalidTimeRangeList::create();
78			}
79			}
80
81			$start = $ranges[0]->start();
82			$end = $ranges[0]->end();
83
84			foreach (array_slice($ranges, 1) as $range) {
85			$rangeStart = $range->start();
86			if ($rangeStart->format('Gi') < $start->format('Gi')) {
87			$start = $rangeStart;
88			}
89			$rangeEnd = $range->end();
90			if ($rangeEnd->format('Gi') > $end->format('Gi')) {
91			$end = $rangeEnd;
92			}
93			}
94
95			return new self($start, $end);
96			}
97
98			public static function fromMidnight(Time $end)
99			{
100			return new self(Time::fromString('00:00'), $end);
101			}
102
103			public function start(): Time
104			{
105			return $this->start;
106			}
107
108			public function end(): Time
109			{
110			return $this->end;
111			}
112
113			public function spillsOverToNextDay(): bool
114			{
115			return $this->end->isBefore($this->start);
			0 ignored issues – show Documentation introduced 2017-11-09 10:47 UTC by Report Bug Copy Issue Report `$this->start` is of type `object<Spatie\OpeningHours\Time>`, but the function expects a `object<self>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
116			}
117
118			public function containsTime(Time $time): bool
119			{
120			if ($this->spillsOverToNextDay()) {
121			if ($time->isSameOrAfter($this->start)) {
			0 ignored issues – show Documentation introduced 2018-10-02 12:17 UTC by Report Bug Copy Issue Report `$this->start` is of type `object<Spatie\OpeningHours\Time>`, but the function expects a `object<self>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
122			return $time->isAfter($this->end);
			0 ignored issues – show Documentation introduced 2017-11-09 10:47 UTC by Report Bug Copy Issue Report `$this->end` is of type `object<Spatie\OpeningHours\Time>`, but the function expects a `object<self>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
123			}
124
125			return $time->isBefore($this->end);
			0 ignored issues – show Documentation introduced 2017-11-09 10:47 UTC by Report Bug Copy Issue Report `$this->end` is of type `object<Spatie\OpeningHours\Time>`, but the function expects a `object<self>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
126			}
127
128			return $time->isSameOrAfter($this->start) && $time->isBefore($this->end);
			0 ignored issues – show Documentation introduced 2017-11-09 10:47 UTC by Report Bug Copy Issue Report `$this->start` is of type `object<Spatie\OpeningHours\Time>`, but the function expects a `object<self>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history... Documentation introduced 2017-11-09 10:47 UTC by Report Bug Copy Issue Report `$this->end` is of type `object<Spatie\OpeningHours\Time>`, but the function expects a `object<self>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
129			}
130
131			public function overlaps(self $timeRange): bool
132			{
133			return $this->containsTime($timeRange->start) \|\| $this->containsTime($timeRange->end);
134			}
135
136			public function format(string $timeFormat = 'H:i', string $rangeFormat = '%s-%s'): string
137			{
138			return sprintf($rangeFormat, $this->start->format($timeFormat), $this->end->format($timeFormat));
139			}
140
141			public function isReversed()
142			{
143			return $this->start()->isAfter($this->end());
			0 ignored issues – show Documentation introduced 2019-06-04 15:47 UTC by Report Bug Copy Issue Report `$this->end()` is of type `object<Spatie\OpeningHours\Time>`, but the function expects a `object<self>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
144			}
145
146			public function overflowsNextDay()
147			{
148			return $this->isReversed();
149			}
150
151			public function __toString(): string
152			{
153			return $this->format();
154			}
155			}
156

spatie / opening-hours

GitHub Access Token became invalid

Push — master ( e31ec7...148746 )

TimeRange::overflowsNextDay() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like