Pheanstalk\Structure\Job

Complexity

Total Complexity

7

Size/Duplication

Total Lines	76
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	setTasks()	0	6	1
A	getTasks()	0	3	1
A	getXml()	0	14	2
A	removeTask()	0	4	1
A	addTask()	0	4	1

<?php


namespace Pheanstalk\Structure;

use Doctrine\Common\Collections\ArrayCollection;

class Job
{
    /** @var ArrayCollection[Task] */

The doc comment ArrayCollection[Task] at position 1 could not be parsed: Expected ']' at position 1, but found '['.

    private $tasks;

    /**
     * Job constructor.
     *
     * @param ArrayCollection[Task] $tasks

The doc comment ArrayCollection[Task] at position 1 could not be parsed: Expected ']' at position 1, but found '['.

     */
    public function __construct(ArrayCollection $tasks)
    {
        $this->setTasks($tasks);
    }

    /**
     * @return ArrayCollection
     */
    public function getTasks(): ArrayCollection
    {
        return $this->tasks;
    }

    /**
     * @param ArrayCollection $tasks
     *
     * @return Job
     */
    public function setTasks(ArrayCollection $tasks): Job
    {
        $this->tasks = $tasks->filter(function(Task $task) {

The parameter $task is not used and could be removed.

            return true;
        });
        return $this;
    }

    /**
     * @param Task $task
     *
     * @return Job
     */
    public function addTask(Task $task): Job
    {
        $this->tasks[] = $task;
        return $this;
    }

    /**
     * @param Task $task
     *
     * @return Job
     */
    public function removeTask(Task $task): Job
    {
        $this->tasks->removeElement($task);
        return $this;
    }

    /**
     * @return \DOMDocument
     * @throws \ReflectionException
     */
    public function getXml()
    {
        $dom = new \DOMDocument("1.0", "utf-8");
        $root = $dom->createElement("job");
        $tasks = $dom->createElement("tasks");
        /** @var Task $task */
        foreach ($this->getTasks() as $task) {
            $taskItem = $dom->createElement("task");

The assignment to $taskItem is dead and can be removed.

            $taskNode = $task->getXml()->getElementsByTagName('task')->item(0);
            $tasks->appendChild($dom->importNode($taskNode, true));
        }
        $root->appendChild($tasks);
        $dom->appendChild($root);
        return $dom;
    }
}