Comodojo\Extender\Orm\Entities\Schedule

Complexity

Total Complexity

15

Size/Duplication

Total Lines	252
Duplicated Lines	10.32 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	getDescription()	0	5	1
A	setDescription()	0	7	1
A	getExpression()	0	5	1
A	setExpression()	0	14	1
A	getEnabled()	0	5	1
A	setEnable()	0	9	1
A	getFirstrun()	0	5	1
A	setFirstrun()	0	7	1
A	getLastrun()	0	5	1
A	setLastrun()	0	7	1
A	shouldRunJob()	13	13	2
A	getNextPlannedRun()	13	13	2
A	buildExpression()	0	14	1

<?php namespace Comodojo\Extender\Orm\Entities;

use \Doctrine\ORM\Mapping as ORM;
use \Comodojo\Extender\Traits\BaseEntityTrait;
use \Comodojo\Extender\Traits\ProcessEntityTrait;
use \Cron\CronExpression;
use \Comodojo\Foundation\Validation\DataFilter;
use \DateTime;

/**
 * @package     Comodojo Extender
 * @author      Marco Giovinazzi <marco.giovinazzi@comodojo.org>
 * @license     MIT
 *
 * LICENSE:
 *
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
 * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 * THE SOFTWARE.
 *
 * @ORM\Table(name="extender_schedule")
 * @ORM\Entity
 */

class Schedule {

    use BaseEntityTrait;
    use ProcessEntityTrait;

    /**
     * @var string
     *
     * @ORM\Column(name="description", type="text", length=65535, nullable=true)
     */
    protected $description;

    /**
     * @var string
     *
     * @ORM\Column(name="min", type="string", length=16, nullable=false)
     */
    protected $min;

    /**
     * @var string
     *
     * @ORM\Column(name="hour", type="string", length=16, nullable=false)
     */
    protected $hour;

    /**
     * @var string
     *
     * @ORM\Column(name="day", type="string", length=16, nullable=false)
     */
    protected $day;

    /**
     * @var string
     *
     * @ORM\Column(name="month", type="string", length=16, nullable=false)
     */
    protected $month;

    /**
     * @var string
     *
     * @ORM\Column(name="weekday", type="string", length=16, nullable=false)
     */
    protected $weekday;

    /**
     * @var string
     *
     * @ORM\Column(name="year", type="string", length=16, nullable=false)
     */
    protected $year;

    /**
     * @var boolean
     *
     * @ORM\Column(name="enabled", type="boolean", nullable=false)
     */
    protected $enabled = 0;

    /**
     * @var datetime
     *
     * @ORM\Column(name="firstrun", type="datetime", nullable=true)
     */
    protected $firstrun;

    /**
     * @var datetime
     *
     * @ORM\Column(name="lastrun", type="datetime", nullable=true)
     */
    protected $lastrun;

    /**
     * Get a brief job description
     *
     * @return string
     */
    public function getDescription() {

        return $this->description;

    }

    /**
     * Set brief job description
     *
     * @param string $description
     * @return Schedule
     */
    public function setDescription($description) {

        $this->description = $description;

        return $this;

    }

    /**
     * Get cron expression of this schedule
     *
     * @return string
     */
    public function getExpression() {

        return (string) $this->buildExpression();

    }

    /**
     * set cron expression for this schedule
     *
     * @param srting $expression A cron-compatible expression
     * @return Schedule
     */
    public function setExpression($expression) {

        $exp = CronExpression::factory($expression);

        $this->minute = $exp->getExpression(0);

The property minute does not exist. Did you maybe forget to declare it?

        $this->hour = $exp->getExpression(1);
        $this->day = $exp->getExpression(2);
        $this->month = $exp->getExpression(3);
        $this->weekday = $exp->getExpression(4);
        $this->year = $exp->getExpression(5);

        return $this;

    }

    /**
     * True if job is currently enabled
     *
     * @return bool
     */
    public function getEnabled() {

        return $this->enabled;

    }

    /**
     * Set enable/disable status
     *
     * @param bool $enable
     * @return Schedule
     */
    public function setEnable($enable) {

        $ena = DataFilter::filterBoolean($enable);

        $this->enable = $ena;

The property enable does not seem to exist. Did you mean enabled?

        return $this;

    }

    /**
     * Get the first-run-date of job
     *
     * @return DateTime
     */
    public function getFirstrun() {

        return $this->firstrun;

    }

    /**
     * Set the first-run-date of job
     *
     * @param DateTime $datetime
     * @return Schedule
     */
    public function setFirstrun(DateTime $datetime) {

        $this->firstrun = $datetime;

It seems like $datetime of type object<DateTime> is incompatible with the declared type object<Comodojo\Extender\Orm\Entities\datetime> of property $firstrun.

        return $this;

    }

    /**
     * Get the first-run-date of job
     *
     * @return DateTime
     */
    public function getLastrun() {

        return $this->lastrun;

    }

    /**
     * Set the first-run-date of job
     *
     * @param DateTime $datetime
     * @return Schedule
     */
    public function setLastrun(DateTime $datetime) {

        $this->lastrun = $datetime;

It seems like $datetime of type object<DateTime> is incompatible with the declared type object<Comodojo\Extender\Orm\Entities\datetime> of property $lastrun.

        return $this;

    }

    public function shouldRunJob(DateTime $time) {

This method seems to be duplicated in your project.

        $last_run = $this->getLastrun();

        if ( is_null($lastrun) ) {

The variable $lastrun does not exist. Did you mean $last_run?

            $next_run = $this->getFirstrun();
        } else {
            $next_run = $this->buildExpression()->getNextRunDate($last_run);
        }

        return $time < $next_run;

    }

    public function getNextPlannedRun(DateTime $time) {

This method seems to be duplicated in your project.

        $last_run = $this->getLastrun();

$last_run is not used, you could remove the assignment.

        if ( is_null($lastrun) ) {

The variable $lastrun does not exist. Did you mean $last_run?

            $next_run = $this->getFirstrun();
        } else {
            $next_run = $this->buildExpression()->getNextRunDate($time);
        }

        return $next_run;

    }

    protected function buildExpression() {

        $exp_string = implode(" ", [
            $this->minute,
            $this->hour,
            $this->day,
            $this->month,
            $this->weekday,
            $this->year
        ]);

        return CronExpression::factory($exp_string);

    }
}