PhpSettingsHandler::getUserDefinedMemoryLimit() - Code Metrics - Inspection of "Simplify PhpSettings checker" - humbug/box - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#415)

by Théo

created 2019-05-19 22:12 UTC

PhpSettingsHandler::getUserDefinedMemoryLimit() A

↳ Parent: PhpSettingsHandler

Complexity

Conditions	3
Paths	3

Size

Total Lines	13
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	8
dl	0
loc	13
rs	10
c	0
b	0
f	0
cc	3
nc	3
nop	0

<?php

declare(strict_types=1);

/*
 * This file is part of the box project.
 *
 * (c) Kevin Herrera <[email protected]>
 *     Théo Fidry <[email protected]>
 *
 * This source file is subject to the MIT license that is bundled
 * with this source code in the file LICENSE.
 */

namespace KevinGH\Box\Console\Php;

use Assert\Assertion;
use Composer\XdebugHandler\XdebugHandler;
use function getenv;
use function ini_get;
use function ini_set;
use const KevinGH\Box\BOX_MEMORY_LIMIT;

use function KevinGH\Box\FileSystem\append_to_file;
use function KevinGH\Box\format_size;
use function KevinGH\Box\memory_to_bytes;
use const PHP_EOL;
use Psr\Log\LoggerInterface;
use function sprintf;
use function trim;

/**
 * @private
 */
final class PhpSettingsHandler extends XdebugHandler
{
    private $logger;
    private $pharReadonly;

    /**
     * {@inheritdoc}
     */
    public function __construct(LoggerInterface $logger)
    {
        parent::__construct('box', '--ansi');

        $this->setLogger($logger);
        $this->logger = $logger;

        $this->pharReadonly = '1' === ini_get('phar.readonly');
    }

    /**
     * {@inheritdoc}
     */
    public function check()
    {
        $this->bumpMemoryLimit();

        return parent::check();
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    }

    /**
     * {@inheritdoc}
     */
    protected function requiresRestart($isLoaded): bool
    {
        if ($this->pharReadonly) {
            $this->logger->debug('phar.readonly is enabled');

            return true;
        }

        $this->logger->debug('phar.readonly is disabled');

        return parent::requiresRestart($isLoaded);
    }

    /**
     * {@inheritdoc}
     */
    protected function restart($command): void
    {
        // Disable phar.readonly if set
        $this->disablePharReadonly();

        parent::restart($command);
    }

    private function disablePharReadonly(): void
    {
        if (ini_get('phar.readonly')) {
            Assertion::notNull($this->tmpIni);

            append_to_file($this->tmpIni, 'phar.readonly=0'.PHP_EOL);

            $this->logger->debug('Configured `phar.readonly=0`');
        }
    }

    /**
     * @see https://github.com/composer/composer/blob/34c371f5f23e25eb9aa54ccc65136cf50930612e/bin/composer#L20-L50
     */
    private function bumpMemoryLimit(): void
    {
        $userDefinedMemoryLimit = self::getUserDefinedMemoryLimit();

        $memoryLimit = trim(ini_get('memory_limit'));
        $memoryLimitInBytes = '-1' === $memoryLimit ? -1 : memory_to_bytes($memoryLimit);

        // Whether or not the memory limit should be dumped
        $bumpMemoryLimit = (
            null === $userDefinedMemoryLimit
            && -1 !== $memoryLimitInBytes
            && $memoryLimitInBytes < 1024 * 1024 * 512
        );
        // Whether or not the memory limit should be set to the user defined memory limit
        $setUserDefinedMemoryLimit = (
            null !== $userDefinedMemoryLimit
            && $memoryLimitInBytes !== $userDefinedMemoryLimit
        );

        if ($bumpMemoryLimit && false === $setUserDefinedMemoryLimit) {
            ini_set('memory_limit', '512M');

            $this->logger->debug(
                sprintf(
                    'Changed the memory limit from "%s" to "%s"',
                    format_size($memoryLimitInBytes, 0),
                    '512M'
                )
            );
        } elseif ($setUserDefinedMemoryLimit) {
            ini_set('memory_limit', (string) $userDefinedMemoryLimit);

            $this->logger->debug(
                sprintf(
                    'Changed the memory limit from "%s" to %s="%s"',
                    format_size($memoryLimitInBytes, 0),
                    BOX_MEMORY_LIMIT,
                    format_size($userDefinedMemoryLimit, 0)
                    format_size(/** @scrutinizer ignore-type */ $userDefinedMemoryLimit, 0)
                )
            );
        } else {
            $this->logger->debug(
                sprintf(
                    'Current memory limit: "%s"',
                    format_size($memoryLimitInBytes, 0)
                )
            );
        }
    }

    /**
     * @return null|int
     */
    private static function getUserDefinedMemoryLimit(): ?int
    {
        $memoryLimit = getenv(BOX_MEMORY_LIMIT);

        if (false === $memoryLimit) {
            $memoryLimitInBytes = null;
        } elseif ('-1' === $memoryLimit) {
            $memoryLimitInBytes = -1;
        } else {
            $memoryLimitInBytes = memory_to_bytes($memoryLimit);
        }

        return $memoryLimitInBytes;
    }
}


humbug / box

Pull Request — master (#415)

PhpSettingsHandler::getUserDefinedMemoryLimit() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like