Avoid using undefined variables such as '$runtimeConfiguration' which will lead to PHP notices.

Open

        if (\in_array('*', $runtimeConfiguration['trustedProxies'], true)) {

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$this' which will lead to PHP notices.

Open

        $this->$propertyName = $newValue;

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$runtimeConfiguration' which will lead to PHP notices.

Open

        $runtimeConfiguration['trustedProxies'] = $this->decodeSet($trustedProxies);

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$runtimeConfiguration' which will lead to PHP notices.

Open

        Assertion::isArray($runtimeConfiguration['trustedProxies']);

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$runtimeConfiguration' which will lead to PHP notices.

Open

        return $runtimeConfiguration;

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Method process has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function process(ContainerBuilder $container, Proxifier $proxifier): void
    {
        /** @var array<string,string> $bundles */
        $bundles = $container->getParameter('kernel.bundles');

Method ensureXdebugDisabled has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function ensureXdebugDisabled(SymfonyStyle $io): void
    {
        $xdebugHandler = new XdebugHandler();
        if (!$xdebugHandler->shouldRestart()) {
            return;

A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 18 and the first side effect is on line 14.

Open

<?php

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        private string $wrappedServiceId,
        private Container $container,
        Mutex $mutex,
        int $instancesLimit = 50,
        ?Resetter $resetter = null,

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

Open

final class ServerStatusCommand extends Command
{
    public function __construct(
        private Sockets $sockets,
        private ApiServerClientFactory $apiServerClientFactory,

• 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) {}

    // ...
}

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

Open

class ContextualAccessForwarderGenerator implements ProxyGeneratorInterface
{
    public function __construct(private MethodForwarderBuilder $forwarderBuilder)
    {
    }

• 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) {}

    // ...
}

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

Open

abstract class AbstractServerStartCommand extends Command
{
    use ParametersHelperTrait;

    private bool $testing = false;

• 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) {}

    // ...
}

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

Open

        private HttpServer $server,
        private HttpServerConfiguration $serverConfiguration,
        private ConfiguratorInterface $serverConfigurator,
        protected ParameterBagInterface $parameterBag,
        private BootableInterface $bootManager

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

Open

        private AccessInterceptorValueHolderFactory $proxyFactory,
        private Instantiator $instantiator,
        private ServicePoolContainer $servicePoolContainer,
        private MutexFactory $limitLocking,
        private MutexFactory $newInstanceLocking

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

Open

        Filesystem $fs,
        string $containerDir,
        string $fileToLoad,
        string $class,
        string $namespace

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

Open

    private static function overrideCachedEntrypoint(Filesystem $fs, string $cacheDir, string $containerClass, string $overriddenFqcn, bool $isDebug): void

Possible parse error: class missing opening or closing brace

Open

final class ContainerModifier

• Exclude checks
  • Disable engine
  • Disable check

Multi-line use declarations must define one parameter per line

Open

            ) use ($servicePoolContainer, $instantiator, $factoryConfig, $mutex, $globalInstancesLimit) {

• Exclude checks
  • Disable engine
  • Disable check

Function overrideGeneratedContainer has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    private static function overrideGeneratedContainer(ReflectionClass $reflContainer, string $cacheDir, bool $isDebug): void
    {
        $fs = new Filesystem();
        $containerFqcn = $reflContainer->getName();
        $overriddenFqcn = $containerFqcn.'_Overridden';

• Read up
  Read up

Function prepareConnectionsForProxification has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    private function prepareConnectionsForProxification(ContainerBuilder $container, array $connectionSvcIds): void
    {
        $dbalAliveKeeperDef = $container->findDefinition(DBALPlatformAliveKeeper::class);
        $aliveKeepers = $dbalAliveKeeperDef->getArgument(1);
        $dbalAliveKeeperDef->setArgument(1, []);

• Read up
  Read up