Sulu\Component\DocumentManager\Event\PersistEvent

Complexity

Total Complexity

10

Size/Duplication

Total Lines	91
Duplicated Lines	26.37 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	9
Bugs	2	Features	3

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getDebugMessage()	0	8	2
A	setNode()	0	4	1
A	setParentNode()	0	4	1
A	getNode()	12	12	2
A	getParentNode()	12	12	2
A	hasParentNode()	0	4	1

<?php

/*
 * This file is part of Sulu.
 *
 * (c) MASSIVE ART WebServices GmbH
 *
 * This source file is subject to the MIT license that is bundled
 * with this source code in the file LICENSE.
 */

namespace Sulu\Component\DocumentManager\Event;

use PHPCR\NodeInterface;
use Sulu\Component\DocumentManager\DocumentHelper;

class PersistEvent extends AbstractMappingEvent
{
    /**
     * @param NodeInterface
     */
    private $parentNode;

    /**
     * @param object $document
     * @param string $locale
     * @param array $options
     */
    public function __construct($document, $locale, array $options = [])
    {
        $this->document = $document;
        $this->locale = $locale;
        $this->options = $options;

It seems like $options of type array is incompatible with the declared type object<Symfony\Component\OptionsResolver\OptionsResolver> of property $options.

    }

    /**
     * {@inheritdoc}
     */
    public function getDebugMessage()
    {
        return sprintf(
            '%s p:%s',
            parent::getDebugMessage(),
            $this->parentNode ? $this->parentNode->getPath() : '<no parent node>'
        );
    }

    /**
     * @param NodeInterface $node
     */
    public function setNode(NodeInterface $node)
    {
        $this->node = $node;
    }

    /**
     * @param NodeInterface $parentNode
     */
    public function setParentNode(NodeInterface $parentNode)
    {
        $this->parentNode = $parentNode;
    }

    /**
     * @return NodeInterface
     *
     * @throws \RuntimeException
     */
    public function getNode()

This method seems to be duplicated in your project.

    {
        if (!$this->node) {
            throw new \RuntimeException(sprintf(
                'Trying to retrieve node when no node has been set. An event ' .
                'listener should have set the node when persisting document "%s"',
                DocumentHelper::getDebugTitle($this->document)
            ));
        }

        return $this->node;
    }

    /**
     * @return NodeInterface
     *
     * @throws \RuntimeException
     */
    public function getParentNode()

This method seems to be duplicated in your project.

    {
        if (!$this->parentNode) {
            throw new \RuntimeException(sprintf(
                'Trying to retrieve parent node when no parent node has been set. An event ' .
                'listener should have set the node when persisting document "%s"',
                DocumentHelper::getDebugTitle($this->document)
            ));
        }

        return $this->parentNode;
    }

    /**
     * @return bool
     */
    public function hasParentNode()
    {
        return $this->parentNode !== null;
    }
}