Job::createFromString() - Code Metrics - Inspection of "ignore timezone if DateTime instance given" - hutnikau/job-scheduler - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — fix/job_timezone ( 63b1b2...15f766 )

by Aleh

created 2018-01-03 12:28 UTC

Job::createFromString() A

↳ Parent: Job

Complexity

Conditions	4
Paths	8

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	4

Importance

Changes

Metric	Value
dl	0
loc	14
ccs	9
cts	9
cp	1
rs	9.2
c	0
b	0
f	0
cc	4
eloc	9
nc	8
nop	4
crap	4

<?php

namespace Scheduler\Job;

use DateTimeInterface;
use DateTimeZone;
use DateTime;

/**
 * Class Job
 * @package Job
 * @author Aleh Hutnikau, <[email protected]>
 */
class Job implements JobInterface
{
    /** @var RRule */
    private $rRule;

    /** @var callable */
    private $callable;

    /**
     * Job constructor.
     * @param RRule $rRule - recurrence rules (@see https://github.com/simshaun/recurr)
     * @param callable $callable
     */
    public function __construct(RRule $rRule, callable $callable)
    {
        $this->rRule = $rRule;
        $this->callable = $callable;
    }

    /**
     * @return RRule
     */
    public function getRRule()
    {
        return $this->rRule;
    }

    /**
     * @return callable
     */
    public function getCallable()
    {
        return $this->callable;
    }

    /**
     * @param string $rRule RRULE string
     * @param string|DateTimeInterface $startDate - @see DateTime supported formats
     * @param callable $callback
     * @param string|DateTimeZone $timezone - If $timezone is omitted, the current timezone will be used.
     *                                        If $startDate is instance of `DateTimeInterface` then $timezone parameter will be ignored
     * @return Job
     */
    public static function createFromString($rRule, $startDate, callable $callback, $timezone = null)
    {
        if ($timezone === null) {
            $timezone = date_default_timezone_get();
        }
        if (is_string($timezone)) {
            $timezone = new DateTimeZone($timezone);
        }
        if (!$startDate instanceof DateTimeInterface) {
            $startDate = new DateTime($startDate, $timezone);
        }
        $rRule = new RRule($rRule, $startDate);
        return new self($rRule, $callback);
    }
}

1		<?php
2
3		namespace Scheduler\Job;
4
5		use DateTimeInterface;
6		use DateTimeZone;
7		use DateTime;
8
9		/**
10		* Class Job
11		* @package Job
12		* @author Aleh Hutnikau, <[email protected]>
13		*/
14		class Job implements JobInterface
15		{
16		/** @var RRule */
17		private $rRule;
18
19		/** @var callable */
20		private $callable;
21
22		/**
23		* Job constructor.
24		* @param RRule $rRule - recurrence rules (@see https://github.com/simshaun/recurr)
25		* @param callable $callable
26		*/
27	8	public function __construct(RRule $rRule, callable $callable)
28		{
29	8	$this->rRule = $rRule;
30	8	$this->callable = $callable;
31	8	}
32
33		/**
34		* @return RRule
35		*/
36	15	public function getRRule()
37		{
38	15	return $this->rRule;
39		}
40
41		/**
42		* @return callable
43		*/
44	2	public function getCallable()
45		{
46	2	return $this->callable;
47		}
48
49		/**
50		* @param string $rRule RRULE string
51		* @param string\|DateTimeInterface $startDate - @see DateTime supported formats
52		* @param callable $callback
53		* @param string\|DateTimeZone $timezone - If $timezone is omitted, the current timezone will be used.
54		* If $startDate is instance of `DateTimeInterface` then $timezone parameter will be ignored
55		* @return Job
56		*/
57	1	public static function createFromString($rRule, $startDate, callable $callback, $timezone = null)
58		{
59	1	if ($timezone === null) {
60	1	$timezone = date_default_timezone_get();
61		}
62	1	if (is_string($timezone)) {
63	1	$timezone = new DateTimeZone($timezone);
64		}
65	1	if (!$startDate instanceof DateTimeInterface) {
66	1	$startDate = new DateTime($startDate, $timezone);
67		}
68	1	$rRule = new RRule($rRule, $startDate);
69	1	return new self($rRule, $callback);
70		}
71		}

hutnikau / job-scheduler

GitHub Access Token became invalid

Push — fix/job_timezone ( 63b1b2...15f766 )

Job::createFromString() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like