Duration::create() - Code Metrics - Inspection of "taking into account PHP bugfix of DateInterval::cr..." - thephpleague/period - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 12eff8...079e85 )

by ignace nyamagana

created 2019-03-17 08:55 UTC

Duration::create() C

↳ Parent: Duration

Complexity

Conditions	12
Paths	20

Size

Total Lines	50
Code Lines	28

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	27
CRAP Score	12.0473

Importance

Changes

Metric	Value
cc	12
eloc	28
nc	20
nop	1
dl	0
loc	50
ccs	27
cts	29
cp	0.931
crap	12.0473
rs	6.9666
c	0
b	0
f	0

How to fix Complexity

<?php

/**
 * League.Period (https://period.thephpleague.com)
 *
 * (c) Ignace Nyamagana Butera <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace League\Period;

use DateInterval;
use DateTimeImmutable;
use TypeError;
use function filter_var;
use function gettype;
use function is_string;
use function method_exists;
use function preg_match;
use function property_exists;
use function rtrim;
use function sprintf;
use function str_pad;
use const FILTER_VALIDATE_INT;

/**
 * League Period Duration.
 *
 * @package League.period
 * @author  Ignace Nyamagana Butera <[email protected]>
 * @since   4.2.0
 */
final class Duration extends DateInterval
{
    private const REGEXP_MICROSECONDS_INTERVAL_SPEC = '@^(?<interval>.*)(\.|,)(?<fraction>\d{1,6})S$@';

    private const REGEXP_MICROSECONDS_DATE_SPEC = '@^(?<interval>.*)(\.)(?<fraction>\d{1,6})$@';

    private const REGEXP_CHRONO_FORMAT = '@^
        (?<sign>\+|-)?
        (((?<hour>\d+):)?(?<minute>\d+):)?
        ((?<second>\d+)(\.(?<fraction>\d{1,6}))?)
    $@x';

    /**
     * New instance.
     *
     * Returns a new instance from an Interval specification
     */
    public function __construct(string $interval_spec)
    {
        if (1 === preg_match(self::REGEXP_MICROSECONDS_INTERVAL_SPEC, $interval_spec, $matches)) {
            parent::__construct($matches['interval'].'S');
            $this->f = (float) str_pad($matches['fraction'], 6, '0') / 1e6;
            return;
        }

        if (1 === preg_match(self::REGEXP_MICROSECONDS_DATE_SPEC, $interval_spec, $matches)) {
            parent::__construct($matches['interval']);
            $this->f = (float) str_pad($matches['fraction'], 6, '0') / 1e6;
            return;
        }

        parent::__construct($interval_spec);
    }

    /**
     * Returns a continuous portion of time between two datepoints expressed as a DateInterval object.
     *
     * The duration can be
     * <ul>
     * <li>an Period object</li>
     * <li>a DateInterval object</li>
     * <li>an integer interpreted as the duration expressed in seconds.</li>
     * <li>a string parsable by DateInterval::createFromDateString</li>
     * </ul>
     *
     * @param mixed $duration a continuous portion of time
     *
     * @throws TypeError if the duration type is not a supported
     */
    public static function create($duration): self
    {
        if ($duration instanceof Period) {
            $duration = $duration->getDateInterval();
        }

        if ($duration instanceof DateInterval) {
            $new = new self('PT0S');
            foreach ($duration as $name => $value) {
                if (property_exists($new, $name)) {
                    $new->$name = $value;
                }
            }

            return $new;
        }

        if (false !== ($second = filter_var($duration, FILTER_VALIDATE_INT))) {
            return new self('PT'.$second.'S');
        }

        if (!is_string($duration) && !method_exists($duration, '__toString')) {
            throw new TypeError(sprintf('%s expects parameter 1 to be string, %s given', __METHOD__, gettype($duration)));
        }

        $duration = (string) $duration;
        if (1 !== preg_match(self::REGEXP_CHRONO_FORMAT, $duration, $matches)) {
            $new = self::createFromDateString($duration);
            if ($new !== false) {

                return $new;
            }

            throw new Exception(sprintf('Unknown or bad format (%s)', $duration));
        }

        $matches['fraction'] = str_pad($matches['fraction'] ?? '0000000', 6, '0');
        $expression = $matches['hour'].' hours '.
            $matches['minute'].' minutes '.
            $matches['second'].' seconds '.$matches['fraction'].' microseconds';

        $instance = self::createFromDateString($expression);
        if (false === $instance) {

            throw new Exception(sprintf('Unknown or bad format (%s)', $expression));
        }

        if ('-' === $matches['sign']) {
            $instance->invert = 1;
        }

        return $instance;
    }

    /**
     * @inheritdoc
     *
     * @param mixed $duration a date with relative parts
     *
     * @return self|false
     */
    public static function createFromDateString($duration)
    {
        $duration = parent::createFromDateString($duration);
        if (false === $duration) {
            return $duration;
        }

        $new = new self('PT0S');
        foreach ($duration as $name => $value) {
            $new->$name = $value;
        }

        return $new;
    }

    /**
     * DEPRECATION WARNING! This method will be removed in the next major point release.
     *
     * @deprecated deprecated since version 4.5
     * @see ::format
     *
     * Returns the ISO8601 interval string representation.
     *
     * Microseconds fractions are included
     */
    public function __toString(): string
    {
        $date = 'P';
        foreach (['Y' => 'y', 'M' => 'm', 'D' => 'd'] as $key => $value) {
            if (0 !== $this->$value) {
                $date .= '%'.$value.$key;
            }
        }

        $time = 'T';
        foreach (['H' => 'h', 'M' => 'i'] as $key => $value) {
            if (0 !== $this->$value) {
                $time .= '%'.$value.$key;
            }
        }

        if (0.0 !== $this->f) {

            $second = $this->s + $this->f;
            if (0 > $this->s) {
                $second = $this->s - $this->f;
            }
            
            $second = rtrim(sprintf('%f', $second), '0');

            return $this->format($date.$time).$second.'S';
        }

        if (0 !== $this->s) {
            $time .= '%sS';

            return $this->format($date.$time);
        }
        
        if ('T' !== $time) {
            return $this->format($date.$time);
        }

        if ('P' !== $date) {
            return $this->format($date);
        }

        return 'PT0S';
    }

    /**
     * DEPRECATION WARNING! This method will be removed in the next major point release.
     *
     * @deprecated deprecated since version 4.6
     * @see ::adjustedTo
     *
     * Returns a new instance with recalculate time and date segments to remove carry over points.
     *
     * This method MUST retain the state of the current instance, and return
     * an instance that contains the time and date segments recalculate to remove
     * carry over points.
     *
     * @param mixed $reference_date a reference datepoint {@see \League\Period\Datepoint::create}
     */
    public function withoutCarryOver($reference_date): self
    {
        return $this->adjustedTo($reference_date);
    }

    /**
     * Returns a new instance with recalculate properties according to a given datepoint.
     *
     * This method MUST retain the state of the current instance, and return
     * an instance that contains the time and date segments recalculate to remove
     * carry over points.
     *
     * @param mixed $reference_date a reference datepoint {@see \League\Period\Datepoint::create}
     */
    public function adjustedTo($reference_date): self
    {
        if (!$reference_date instanceof DateTimeImmutable) {
            $reference_date = Datepoint::create($reference_date);
        }

        $duration = self::create($reference_date->diff($reference_date->add($this)));
        if ($duration == $this) {
            return $this;
        }

        return $duration;
    }
}


1		<?php
2
3		/**
4		* League.Period (https://period.thephpleague.com)
5		*
6		* (c) Ignace Nyamagana Butera <[email protected]>
7		*
8		* For the full copyright and license information, please view the LICENSE
9		* file that was distributed with this source code.
10		*/
11
12		declare(strict_types=1);
13
14		namespace League\Period;
15
16		use DateInterval;
17		use DateTimeImmutable;
18		use TypeError;
19		use function filter_var;
20		use function gettype;
21		use function is_string;
22		use function method_exists;
23		use function preg_match;
24		use function property_exists;
25		use function rtrim;
26		use function sprintf;
27		use function str_pad;
28		use const FILTER_VALIDATE_INT;
29
30		/**
31		* League Period Duration.
32		*
33		* @package League.period
34		* @author Ignace Nyamagana Butera <[email protected]>
35		* @since 4.2.0
36		*/
37		final class Duration extends DateInterval
38		{
39		private const REGEXP_MICROSECONDS_INTERVAL_SPEC = '@^(?<interval>.*)(\.\|,)(?<fraction>\d{1,6})S$@';
40
41		private const REGEXP_MICROSECONDS_DATE_SPEC = '@^(?<interval>.*)(\.)(?<fraction>\d{1,6})$@';
42
43		private const REGEXP_CHRONO_FORMAT = '@^
44		(?<sign>\+\|-)?
45		(((?<hour>\d+):)?(?<minute>\d+):)?
46		((?<second>\d+)(\.(?<fraction>\d{1,6}))?)
47		$@x';
48
49		/**
50		* New instance.
51		*
52		* Returns a new instance from an Interval specification
53		*/
54	282	public function __construct(string $interval_spec)
55		{
56	282	if (1 === preg_match(self::REGEXP_MICROSECONDS_INTERVAL_SPEC, $interval_spec, $matches)) {
57	3	parent::__construct($matches['interval'].'S');
58	3	$this->f = (float) str_pad($matches['fraction'], 6, '0') / 1e6;
59	3	return;
60		}
61
62	282	if (1 === preg_match(self::REGEXP_MICROSECONDS_DATE_SPEC, $interval_spec, $matches)) {
63	3	parent::__construct($matches['interval']);
64	3	$this->f = (float) str_pad($matches['fraction'], 6, '0') / 1e6;
65	3	return;
66		}
67
68	279	parent::__construct($interval_spec);
69	273	}
70
71		/**
72		* Returns a continuous portion of time between two datepoints expressed as a DateInterval object.
73		*
74		* The duration can be
75		* <ul>
76		* <li>an Period object</li>
77		* <li>a DateInterval object</li>
78		* <li>an integer interpreted as the duration expressed in seconds.</li>
79		* <li>a string parsable by DateInterval::createFromDateString</li>
80		* </ul>
81		*
82		* @param mixed $duration a continuous portion of time
83		*
84		* @throws TypeError if the duration type is not a supported
85		*/
86	285	public static function create($duration): self
87		{
88	285	if ($duration instanceof Period) {
89	15	$duration = $duration->getDateInterval();
90		}
91
92	285	if ($duration instanceof DateInterval) {
93	48	$new = new self('PT0S');
94	48	foreach ($duration as $name => $value) {
95	48	if (property_exists($new, $name)) {
96	48	$new->$name = $value;
97		}
98		}
99
100	48	return $new;
101		}
102
103	237	if (false !== ($second = filter_var($duration, FILTER_VALIDATE_INT))) {
104	57	return new self('PT'.$second.'S');
105		}
106
107	180	if (!is_string($duration) && !method_exists($duration, '__toString')) {
108	9	throw new TypeError(sprintf('%s expects parameter 1 to be string, %s given', __METHOD__, gettype($duration)));
109		}
110
111	171	$duration = (string) $duration;
112	171	if (1 !== preg_match(self::REGEXP_CHRONO_FORMAT, $duration, $matches)) {
113	156	$new = self::createFromDateString($duration);
114	156	if ($new !== false) {
		0 ignored issues – show introduced 2019-03-17 08:58 UTC by Report Bug Copy Issue Report The condition `$new !== false` is always `true`. Loading history...
115	156	return $new;
116		}
117
118		throw new Exception(sprintf('Unknown or bad format (%s)', $duration));
119		}
120
121	15	$matches['fraction'] = str_pad($matches['fraction'] ?? '0000000', 6, '0');
122	15	$expression = $matches['hour'].' hours '.
123	15	$matches['minute'].' minutes '.
124	15	$matches['second'].' seconds '.$matches['fraction'].' microseconds';
125
126	15	$instance = self::createFromDateString($expression);
127	15	if (false === $instance) {
		0 ignored issues – show introduced 2019-03-17 08:58 UTC by Report Bug Copy Issue Report The condition `false === $instance` is always `false`. Loading history...
128		throw new Exception(sprintf('Unknown or bad format (%s)', $expression));
129		}
130
131	15	if ('-' === $matches['sign']) {
132	6	$instance->invert = 1;
133		}
134
135	15	return $instance;
136		}
137
138		/**
139		* @inheritdoc
140		*
141		* @param mixed $duration a date with relative parts
142		*
143		* @return self\|false
144		*/
145	174	public static function createFromDateString($duration)
146		{
147	174	$duration = parent::createFromDateString($duration);
148	174	if (false === $duration) {
149		return $duration;
150		}
151
152	174	$new = new self('PT0S');
153	174	foreach ($duration as $name => $value) {
154	174	$new->$name = $value;
155		}
156
157	174	return $new;
158		}
159
160		/**
161		* DEPRECATION WARNING! This method will be removed in the next major point release.
162		*
163		* @deprecated deprecated since version 4.5
164		* @see ::format
165		*
166		* Returns the ISO8601 interval string representation.
167		*
168		* Microseconds fractions are included
169		*/
170	69	public function __toString(): string
171		{
172	69	$date = 'P';
173	69	foreach (['Y' => 'y', 'M' => 'm', 'D' => 'd'] as $key => $value) {
174	69	if (0 !== $this->$value) {
175	39	$date .= '%'.$value.$key;
176		}
177		}
178
179	69	$time = 'T';
180	69	foreach (['H' => 'h', 'M' => 'i'] as $key => $value) {
181	69	if (0 !== $this->$value) {
182	41	$time .= '%'.$value.$key;
183		}
184		}
185
186	69	if (0.0 !== $this->f) {
		0 ignored issues – show introduced 2019-02-01 08:05 UTC by Report Bug Copy Issue Report The condition `0.0 !== $this->f` is always `true`. Loading history...
187	21	$second = $this->s + $this->f;
188	21	if (0 > $this->s) {
189	3	$second = $this->s - $this->f;
190		}
191
192	21	$second = rtrim(sprintf('%f', $second), '0');
193
194	21	return $this->format($date.$time).$second.'S';
195		}
196
197	48	if (0 !== $this->s) {
198	9	$time .= '%sS';
199
200	9	return $this->format($date.$time);
201		}
202
203	39	if ('T' !== $time) {
204	12	return $this->format($date.$time);
205		}
206
207	27	if ('P' !== $date) {
208	24	return $this->format($date);
209		}
210
211	3	return 'PT0S';
212		}
213
214		/**
215		* DEPRECATION WARNING! This method will be removed in the next major point release.
216		*
217		* @deprecated deprecated since version 4.6
218		* @see ::adjustedTo
219		*
220		* Returns a new instance with recalculate time and date segments to remove carry over points.
221		*
222		* This method MUST retain the state of the current instance, and return
223		* an instance that contains the time and date segments recalculate to remove
224		* carry over points.
225		*
226		* @param mixed $reference_date a reference datepoint {@see \League\Period\Datepoint::create}
227		*/
228	27	public function withoutCarryOver($reference_date): self
229		{
230	27	return $this->adjustedTo($reference_date);
231		}
232
233		/**
234		* Returns a new instance with recalculate properties according to a given datepoint.
235		*
236		* This method MUST retain the state of the current instance, and return
237		* an instance that contains the time and date segments recalculate to remove
238		* carry over points.
239		*
240		* @param mixed $reference_date a reference datepoint {@see \League\Period\Datepoint::create}
241		*/
242	27	public function adjustedTo($reference_date): self
243		{
244	27	if (!$reference_date instanceof DateTimeImmutable) {
245	24	$reference_date = Datepoint::create($reference_date);
246		}
247
248	27	$duration = self::create($reference_date->diff($reference_date->add($this)));
249	27	if ($duration == $this) {
250	3	return $this;
251		}
252
253	24	return $duration;
254		}
255		}
256

thephpleague / period

Push — master ( 12eff8...079e85 )

Duration::create() C

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like