Backoffice/EventSubscriber/UpdateHistoryListSubscriber.php
The class UpdateHistoryListSubscriber has 16 public methods. Consider refactoring UpdateHistoryListSubscriber to keep number of public methods under 10. Open
Open
class UpdateHistoryListSubscriber implements EventSubscriberInterface
{
protected $tokenManager;
protected $objectManager;
protected $historyClass;
- Read upRead up
- Create a ticketCreate a ticket
- Exclude checks
TooManyPublicMethods
Since: 0.1
A class with too many public methods is probably a good suspect for refactoring, in order to reduce its complexity and find a way to have more fine grained objects.
By default it ignores methods starting with 'get' or 'set'.
Example
Source https://phpmd.org/rules/codesize.html#toomanypublicmethods
Missing class import via use statement (line '193', column '36'). Open
Open
$history->setUpdatedAt(new \DateTime());
- Read upRead up
- Create a ticketCreate a ticket
- Exclude checks
MissingImport
Since: 2.7.0
Importing all external classes in a file through use statements makes them clearly visible.
Example
function make() {
return new \stdClass();
}
Source http://phpmd.org/rules/cleancode.html#MissingImport
Reference to constant CONTENT_CREATION
from undeclared class \OpenOrchestra\ModelInterface\ContentEvents
Open
Open
ContentEvents::CONTENT_CREATION => 'addContentCreationHistory',
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_DUPLICATE
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
$this->addNodeHistory($event, NodeEvents::NODE_DUPLICATE);
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_RESTORE
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
NodeEvents::NODE_RESTORE => 'addNodeRestoreHistory',
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_CHANGE_STATUS
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
NodeEvents::NODE_CHANGE_STATUS => 'addNodeChangeStatusHistory',
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_UPDATE
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
NodeEvents::NODE_UPDATE => 'addNodeUpdateHistory',
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\ContentEvent
Open
Open
public function addContentUpdateHistory(ContentEvent $event)
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\ContentEvent
Open
Open
public function addContentRestoreHistory(ContentEvent $event)
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_RESTORE
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
$this->addNodeHistory($event, NodeEvents::NODE_RESTORE);
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_UPDATE_BLOCK
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
NodeEvents::NODE_UPDATE_BLOCK => 'addNodeUpdateBlockHistory',
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_UPDATE_BLOCK_POSITION
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
NodeEvents::NODE_UPDATE_BLOCK_POSITION => 'addNodeUpdateBlockPositionHistory',
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant CONTENT_DUPLICATE
from undeclared class \OpenOrchestra\ModelInterface\ContentEvents
Open
Open
$this->addContentHistory($event, ContentEvents::CONTENT_DUPLICATE);
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\ContentEvent
Open
Open
public function addContentChangeStatusHistory(ContentEvent $event)
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant CONTENT_CHANGE_STATUS
from undeclared class \OpenOrchestra\ModelInterface\ContentEvents
Open
Open
$this->addContentHistory($event, ContentEvents::CONTENT_CHANGE_STATUS);
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\NodeEvent
Open
Open
public function addPathUpdatedHistory(NodeEvent $event)
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\NodeEvent
Open
Open
public function addNodeDeleteBlockHistory(NodeEvent $event)
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_UPDATE_BLOCK_POSITION
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
$this->addNodeHistory($event, NodeEvents::NODE_UPDATE_BLOCK_POSITION);
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\NodeEvent
Open
Open
public function addNodeDuplicateHistory(NodeEvent $event)
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_DELETE_BLOCK
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
$this->addNodeHistory($event, NodeEvents::NODE_DELETE_BLOCK);
- Create a ticketCreate a ticket
- Exclude checks
Call to method getContent
from undeclared class \OpenOrchestra\ModelInterface\Event\ContentEvent
Open
Open
$document = $event->getContent();
- Create a ticketCreate a ticket
- Exclude checks
Checking instanceof against undeclared class \OpenOrchestra\ModelInterface\Model\HistorisableInterface
Open
Open
if ($document instanceof HistorisableInterface && !is_null($token)) {
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_DUPLICATE
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
NodeEvents::NODE_DUPLICATE => 'addNodeDuplicateHistory',
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\NodeEvent
Open
Open
public function addNodeUpdateHistory(NodeEvent $event)
- Create a ticketCreate a ticket
- Exclude checks
Call to method addHistory
from undeclared class \OpenOrchestra\ModelInterface\Model\HistorisableInterface
Open
Open
$document->addHistory($history);
- Create a ticketCreate a ticket
- Exclude checks
Parameter $objectManager
has undeclared type \Doctrine\Common\Persistence\ObjectManager
Open
Open
public function __construct(TokenStorageInterface $tokenManager, ObjectManager $objectManager, $historyClass)
- Create a ticketCreate a ticket
- Exclude checks
Parameter $tokenManager
has undeclared type \Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface
Open
Open
public function __construct(TokenStorageInterface $tokenManager, ObjectManager $objectManager, $historyClass)
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\ContentEvent
Open
Open
public function addContentCreationHistory(ContentEvent $event)
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\NodeEvent
Open
Open
public function addNodeRestoreHistory(NodeEvent $event)
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\NodeEvent
Open
Open
public function addNodeAddLanguageHistory(NodeEvent $event)
- Create a ticketCreate a ticket
- Exclude checks
Call to method getNode
from undeclared class \OpenOrchestra\ModelInterface\Event\NodeEvent
Open
Open
$document = $event->getNode();
- Create a ticketCreate a ticket
- Exclude checks
Call to method flush
from undeclared class \Doctrine\Common\Persistence\ObjectManager
Open
Open
$this->objectManager->flush();
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant CONTENT_DUPLICATE
from undeclared class \OpenOrchestra\ModelInterface\ContentEvents
Open
Open
ContentEvents::CONTENT_DUPLICATE => 'addContentDuplicateHistory',
- Create a ticketCreate a ticket
- Exclude checks
Class implements undeclared interface \Symfony\Component\EventDispatcher\EventSubscriberInterface
Open
Open
class UpdateHistoryListSubscriber implements EventSubscriberInterface
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\NodeEvent
Open
Open
public function addNodeUpdateBlockHistory(NodeEvent $event)
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_UPDATE_BLOCK
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
$this->addNodeHistory($event, NodeEvents::NODE_UPDATE_BLOCK);
- Create a ticketCreate a ticket
- Exclude checks
Call to method getToken
from undeclared class \Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface
Open
Open
$token = $this->tokenManager->getToken();
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant CONTENT_CHANGE_STATUS
from undeclared class \OpenOrchestra\ModelInterface\ContentEvents
Open
Open
ContentEvents::CONTENT_CHANGE_STATUS => 'addContentChangeStatusHistory',
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant CONTENT_RESTORE
from undeclared class \OpenOrchestra\ModelInterface\ContentEvents
Open
Open
$this->addContentHistory($event, ContentEvents::CONTENT_RESTORE);
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_CREATION
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
$this->addNodeHistory($event, NodeEvents::NODE_CREATION);
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant CONTENT_RESTORE
from undeclared class \OpenOrchestra\ModelInterface\ContentEvents
Open
Open
ContentEvents::CONTENT_RESTORE => 'addContentRestoreHistory',
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant PATH_UPDATED
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
$this->addNodeHistory($event, NodeEvents::PATH_UPDATED);
- Create a ticketCreate a ticket
- Exclude checks
Call to method getUser
from undeclared class \Symfony\Component\Security\Core\Authentication\Token\TokenInterface
Open
Open
$user = $token->getUser();
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_CREATION
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
NodeEvents::NODE_CREATION => 'addNodeCreationHistory',
- Create a ticketCreate a ticket
- Exclude checks
Call to method getToken
from undeclared class \Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface
Open
Open
$token = $this->tokenManager->getToken();
- Create a ticketCreate a ticket
- Exclude checks
Checking instanceof against undeclared class \OpenOrchestra\ModelInterface\Model\HistorisableInterface
Open
Open
if ($document instanceof HistorisableInterface && !is_null($token)) {
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant CONTENT_CREATION
from undeclared class \OpenOrchestra\ModelInterface\ContentEvents
Open
Open
$this->addContentHistory($event, ContentEvents::CONTENT_CREATION);
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_UPDATE
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
$this->addNodeHistory($event, NodeEvents::NODE_UPDATE);
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\NodeEvent
Open
Open
public function addNodeCreationHistory(NodeEvent $event)
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_ADD_LANGUAGE
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
$this->addNodeHistory($event, NodeEvents::NODE_ADD_LANGUAGE);
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\NodeEvent
Open
Open
public function addNodeChangeStatusHistory(NodeEvent $event)
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant CONTENT_UPDATE
from undeclared class \OpenOrchestra\ModelInterface\ContentEvents
Open
Open
ContentEvents::CONTENT_UPDATE => 'addContentUpdateHistory',
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_DELETE_BLOCK
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
NodeEvents::NODE_DELETE_BLOCK => 'addNodeDeleteBlockHistory',
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant CONTENT_UPDATE
from undeclared class \OpenOrchestra\ModelInterface\ContentEvents
Open
Open
$this->addContentHistory($event, ContentEvents::CONTENT_UPDATE);
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\ContentEvent
Open
Open
public function addContentDuplicateHistory(ContentEvent $event)
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\NodeEvent
Open
Open
public function addNodeUpdateBlockPositionHistory(NodeEvent $event)
- Create a ticketCreate a ticket
- Exclude checks
Parameter $document
has undeclared type \OpenOrchestra\ModelInterface\Model\HistorisableInterface
Open
Open
protected function addDocumentHistory(HistorisableInterface $document, TokenInterface $token, $eventType)
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_CHANGE_STATUS
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
$this->addNodeHistory($event, NodeEvents::NODE_CHANGE_STATUS);
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\ContentEvent
Open
Open
protected function addContentHistory(ContentEvent $event, $eventType)
- Create a ticketCreate a ticket
- Exclude checks
Parameter $event
has undeclared type \OpenOrchestra\ModelInterface\Event\NodeEvent
Open
Open
protected function addNodeHistory(NodeEvent $event, $eventType)
- Create a ticketCreate a ticket
- Exclude checks
Parameter $token
has undeclared type \Symfony\Component\Security\Core\Authentication\Token\TokenInterface
Open
Open
protected function addDocumentHistory(HistorisableInterface $document, TokenInterface $token, $eventType)
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant PATH_UPDATED
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
NodeEvents::PATH_UPDATED => 'addPathUpdatedHistory',
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant NODE_ADD_LANGUAGE
from undeclared class \OpenOrchestra\ModelInterface\NodeEvents
Open
Open
NodeEvents::NODE_ADD_LANGUAGE => 'addNodeAddLanguageHistory',
- Create a ticketCreate a ticket
- Exclude checks