open-orchestra/open-orchestra-cms-bundle

View on GitHub
Backoffice/GenerateForm/Strategies/ConfigurableContentStrategy.php

Summary

Maintainability
A
0 mins
Test Coverage

Avoid unused parameters such as '$options'.
Open

    public function buildForm(FormBuilderInterface $builder, array $options)

UnusedFormalParameter

Since: 0.2

Avoid passing parameters to methods or constructors and then not using those parameters.

Example

class Foo
{
    private function bar($howdy)
    {
        // $howdy is not used
    }
}

Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter

Parameter $block has undeclared type \OpenOrchestra\ModelInterface\Model\BlockInterface
Open

    public function support(BlockInterface $block)

Call to method add from undeclared class \Symfony\Component\Form\FormBuilderInterface
Open

        $builder->add('contentSearch', 'oo_content_search', array(

Reference to constant NAME from undeclared class \OpenOrchestra\DisplayBundle\DisplayBlock\Strategies\ConfigurableContentStrategy (Did you mean class \OpenOrchestra\Backoffice\GenerateForm\Strategies\ConfigurableContentStrategy)
Open

        return BaseConfigurableContentStrategy::NAME === $block->getComponent();

Call to method getComponent from undeclared class \OpenOrchestra\ModelInterface\Model\BlockInterface
Open

        return BaseConfigurableContentStrategy::NAME === $block->getComponent();

Parameter $builder has undeclared type \Symfony\Component\Form\FormBuilderInterface
Open

    public function buildForm(FormBuilderInterface $builder, array $options)

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

}

There are no issues that match your filters.

Category
Status