bourdeau/jdhm-api

Method addPhpIniRequirement has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function addPhpIniRequirement($cfgName, $evaluation, $approveCfgAbsence = false, $testMessage = null, $helpHtml = null, $helpText = null)

Method addPhpIniRecommendation has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function addPhpIniRecommendation($cfgName, $evaluation, $approveCfgAbsence = false, $testMessage = null, $helpHtml = null, $helpText = null)

Each class must be in a file by itself

Open

class SymfonyRequirements extends RequirementCollection

• Exclude checks
  • Disable engine
  • Disable check

The class AppKernel has a coupling between objects value of 18. Consider to reduce the number of dependencies under 13.

Open

class AppKernel extends Kernel
{
    public function registerBundles()
    {
        $bundles = [

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    /**
     * @var \foo\bar\X
     */
    private $x = null;

    /**
     * @var \foo\bar\Y
     */
    private $y = null;

    /**
     * @var \foo\bar\Z
     */
    private $z = null;

    public function setFoo(\Foo $foo) {}
    public function setBar(\Bar $bar) {}
    public function setBaz(\Baz $baz) {}

    /**
     * @return \SplObjectStorage
     * @throws \OutOfRangeException
     * @throws \InvalidArgumentException
     * @throws \ErrorException
     */
    public function process(\Iterator $it) {}

    // ...
}

Each class must be in a file by itself

Open

class PhpIniRequirement extends Requirement

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a file by itself

Open

class RequirementCollection implements IteratorAggregate

• Exclude checks
  • Disable engine
  • Disable check

Method __construct has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function __construct($fulfilled, $testMessage, $helpHtml, $helpText = null, $optional = false)

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

There must be one blank line after the last USE statement; 2 found;

Open

use Symfony\Component\Form\Extension\Core\Type\TextType;

• Exclude checks
  • Disable engine
  • Disable check

Blank line found at start of control structure

Open

        if ($form->isSubmitted() && $form->isValid()) {

• Exclude checks
  • Disable engine
  • Disable check

The closing brace for the class must go on the next line after the body

Open

}

• Exclude checks
  • Disable engine
  • Disable check

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

There must be one blank line after the namespace declaration

Open

namespace JdhmApi\Entity;

• Exclude checks
  • Disable engine
  • Disable check

Blank line found at end of control structure

Open

• Exclude checks
  • Disable engine
  • Disable check

Missing class import via use statement (line '666', column '34').

Open

                $reflector = new ReflectionExtension('intl');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '17', column '41').

Open

            $client->setDateOfBirth(new \DateTime("now"));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '216', column '24').

Open

        $this->add(new Requirement($fulfilled, $testMessage, $helpHtml, $helpText, true));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Remove error control operator '@' on line 445.

Open

    public function __construct()
    {
        /* mandatory requirements follow */

        $installedPhpVersion = phpversion();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

Missing class import via use statement (line '19', column '17').

Open

            new JMS\SerializerBundle\JMSSerializerBundle(),

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}