open-orchestra/open-orchestra-cms-bundle

View on GitHub
ApiBundle/Facade/NodeFacade.php

Summary

Maintainability
A
0 mins
Test Coverage

The class NodeFacade has 18 fields. Consider redesigning NodeFacade to keep the number of fields under 15.
Open

class NodeFacade extends DeletedFacade
{
    use BlameableFacade;

    /**
Severity: Minor
Found in ApiBundle/Facade/NodeFacade.php by phpmd

TooManyFields

Since: 0.1

Classes that have too many fields could be redesigned to have fewer fields, possibly through some nested object grouping of some of the information. For example, a class with city/state/zip fields could instead have one Address field.

Example

class Person {
   protected $one;
   private $two;
   private $three;
   [... many more fields ...]
}

Source https://phpmd.org/rules/codesize.html#toomanyfields

Class uses undeclared trait \OpenOrchestra\BaseApi\Facade\Traits\BlameableFacade
Open

class NodeFacade extends DeletedFacade
Severity: Critical
Found in ApiBundle/Facade/NodeFacade.php by phan

Possibly zero references to use statement for classlike/namespace Serializer (\JMS\Serializer\Annotation)
Open

use JMS\Serializer\Annotation as Serializer;
Severity: Minor
Found in ApiBundle/Facade/NodeFacade.php by phan

Parameter $previewLink has undeclared type \OpenOrchestra\BaseApi\Facade\FacadeInterface
Open

    public function addPreviewLink(FacadeInterface $previewLink)
Severity: Minor
Found in ApiBundle/Facade/NodeFacade.php by phan

Parameter $facade has undeclared type \OpenOrchestra\BaseApi\Facade\FacadeInterface
Open

    public function setAreas(FacadeInterface $facade, $key)
Severity: Minor
Found in ApiBundle/Facade/NodeFacade.php by phan

There are no issues that match your filters.

Category
Status