CedricZiel\ShariffBundle\CedricZielShariffBundle

Complexity

Total Complexity

3

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	3
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getContainerExtensionClass()	0	4	1
A	getContainerExtension()	0	8	2

<?php

namespace CedricZiel\ShariffBundle;

use CedricZiel\ShariffBundle\DependencyInjection\CedricZielShariffExtension;
use Symfony\Component\HttpKernel\Bundle\Bundle;

/**
 * @package CedricZiel\ShariffBundle
 */
class CedricZielShariffBundle extends Bundle
{
    /**
     * @return string
     */
    public function getContainerExtensionClass()
    {
        return CedricZielShariffExtension::class;
    }

    /**
     * {@inheritdoc}
     */
    public function getContainerExtension()
    {
        if (null === $this->extension) {
            return $this->extension = $this->createContainerExtension();
        }

        return $this->extension;

The expression $this->extension; of type Symfony\Component\DependencyInjection\Extension\ExtensionInterface|false adds false to the return on line 30 which is incompatible with the return type declared by the interface Symfony\Component\HttpKernel\Bundle\BundleInterface::getContainerExtension of type Symfony\Component\DependencyInjection\Extension\ExtensionInterface|null. It seems like you forgot to handle an error condition.

    }
}