Backoffice/GeneratePerimeter/GeneratePerimeterManager.php
Doc-block of $strategy
in addStrategy
contains phpdoc param type \OpenOrchestra\Backoffice\GeneratePerimeter\PerimeterStrategyInterface
which is incompatible with the param type \OpenOrchestra\Backoffice\GeneratePerimeter\Strategy\GeneratePerimeterStrategyInterface
declared in the signature Open
Open
* @param PerimeterStrategyInterface $strategy
- Create a ticketCreate a ticket
- Exclude checks
Avoid excessively long variable names like $perimetersConfiguration. Keep variable name length under 20. Open
Open
$perimetersConfiguration = array();
- Read upRead up
- Create a ticketCreate a ticket
- Exclude checks
LongVariable
Since: 0.2
Detects when a field, formal or local variable is declared with a long name.
Example
class Something {
protected $reallyLongIntName = -3; // VIOLATION - Field
public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
$otherReallyLongName = -5; // VIOLATION - Local
for ($interestingIntIndex = 0; // VIOLATION - For
$interestingIntIndex < 10;
$interestingIntIndex++ ) {
}
}
}
Source https://phpmd.org/rules/naming.html#longvariable
There must be one blank line after the last USE statement; 0 found; Open
Open
use OpenOrchestra\Backoffice\GeneratePerimeter\Strategy\GeneratePerimeterStrategyInterface;
- Create a ticketCreate a ticket
- Exclude checks
The static declaration must come after the visibility declaration Open
Open
static public function changePathToName($path) {
- Create a ticketCreate a ticket
- Exclude checks
The static declaration must come after the visibility declaration Open
Open
static public function changeNameToPath($name) {
- Create a ticketCreate a ticket
- Exclude checks
Line exceeds 120 characters; contains 121 characters Open
Open
$perimeters = array_merge($perimeters, array($strategy->getType() => $strategy->generatePerimeter($siteId)));
- Create a ticketCreate a ticket
- Exclude checks
Opening brace should be on a new line Open
Open
static public function changeNameToPath($name) {
- Create a ticketCreate a ticket
- Exclude checks
Opening brace should be on a new line Open
Open
static public function changePathToName($path) {
- Create a ticketCreate a ticket
- Exclude checks