Installer::addMakeOnInstallOrUpdate() - Code Metrics - Inspection of "Merge pull request #1072 from WyriHaximus/only-add..." - WyriHaximus/php-test-utilities - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( e09157...64b15f )

by Cees-Jan

created 2025-08-18 18:39 UTC

Installer::addMakeOnInstallOrUpdate() F

↳ Parent: Installer

Complexity

Conditions	18
Paths	401

Size

Total Lines	57
Code Lines	34

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	18
eloc	34
c	1
b	0
f	0
nc	401
nop	2
dl	0
loc	57
rs	1.5319

How to fix Long Method Complexity

<?php

declare(strict_types=1);

namespace WyriHaximus\TestUtilities\Composer;

use Composer\Composer;
use Composer\EventDispatcher\EventSubscriberInterface;
use Composer\IO\IOInterface;
use Composer\Plugin\PluginInterface;
use Composer\Script\Event;
use Composer\Script\ScriptEvents;
use Exception;

use function array_key_exists;
use function array_keys;
use function dirname;
use function file_exists;
use function file_get_contents;
use function file_put_contents;
use function hash;
use function hash_equals;
use function is_array;
use function is_string;
use function json_decode;
use function json_encode;
use function preg_replace;

use const DIRECTORY_SEPARATOR;
use const JSON_PRETTY_PRINT;
use const JSON_UNESCAPED_SLASHES;
use const PHP_INT_MAX;

final class Installer implements PluginInterface, EventSubscriberInterface
{
    /** @return array<string, array<string|int>> */
    public static function getSubscribedEvents(): array
    {
        return [ScriptEvents::PRE_AUTOLOAD_DUMP => ['findEventListeners', PHP_INT_MAX]];
    }

    public function activate(Composer $composer, IOInterface $io): void
    {
        // does nothing, see getSubscribedEvents() instead.
    }

    public function deactivate(Composer $composer, IOInterface $io): void
    {
        // does nothing, see getSubscribedEvents() instead.
    }

    public function uninstall(Composer $composer, IOInterface $io): void
    {
        // does nothing, see getSubscribedEvents() instead.
    }

    /**
     * Called before every dump autoload, generates a fresh PHP class.
     *
     * @phpstan-ignore shipmonk.deadMethod
     */
    public static function findEventListeners(Event $event): void
    {
        $rootPackagePath = dirname(self::getVendorDir($event->getComposer())) . DIRECTORY_SEPARATOR;
        if (! file_exists($rootPackagePath . '/composer.json')) {
            return;
        }

        $jsonRaw = file_get_contents($rootPackagePath . '/composer.json');
        if (! is_string($jsonRaw)) {

            return;
        }

        $json = json_decode($jsonRaw, true);
        if (! is_array($json)) {
            return;
        }

        if (! array_key_exists('require-dev', $json)) {
            return;
        }

        if (! is_array($json['require-dev'])) {
            return;
        }

        $hasMakefiles = false;
        foreach (array_keys($json['require-dev']) as $package) {
            if ($package === 'wyrihaximus/makefiles') {
                $hasMakefiles = true;
                break;
            }
        }

        if (! $hasMakefiles) {
            return;
        }

        unset($hasMakefiles);

        if (array_key_exists('name', $json) && $json['name'] === 'wyrihaximus/test-utilities') {
            self::addMakeOnInstallOrUpdate($event->getIO(), $rootPackagePath);

            return;
        }

        foreach (array_keys($json['require-dev']) as $package) {
            if ($package === 'wyrihaximus/test-utilities') {
                self::addMakeOnInstallOrUpdate($event->getIO(), $rootPackagePath);

                return;
            }
        }
    }

    private static function addMakeOnInstallOrUpdate(IOInterface $io, string $rootPackagePath): void
    {
        $io->write('<info>wyrihaximus/test-utilities:</info> Adding <fg=cyan>make on-install-or-update || true</> to scripts');
        $composerJsonString = file_get_contents($rootPackagePath . '/composer.json');
        if (! is_string($composerJsonString)) {

            $io->write('<error>wyrihaximus/test-utilities:</error> Unable to read <fg=cyan>composer.json</> aborting');

            return;
        }

        $composerJson     = json_decode($composerJsonString, true);
        $composerJsonHash = hash('sha512', (string) json_encode($composerJson, JSON_UNESCAPED_SLASHES | JSON_PRETTY_PRINT));
        if (is_array($composerJson) && array_key_exists('scripts', $composerJson) && is_array($composerJson['scripts'])) {
            if (array_key_exists('post-install-cmd', $composerJson['scripts'])) {
                $composerJson['scripts']['post-install-cmd'] = self::addMakeOnInstallOrUpdateToScriptsSectionAndRemoveCommandsItReplaces($composerJson['scripts']['post-install-cmd']);
            }

            if (array_key_exists('post-update-cmd', $composerJson['scripts'])) {
                $composerJson['scripts']['post-update-cmd'] = self::addMakeOnInstallOrUpdateToScriptsSectionAndRemoveCommandsItReplaces($composerJson['scripts']['post-update-cmd']);
            }
        }

        $replacementComposerJsonString = json_encode($composerJson, JSON_UNESCAPED_SLASHES | JSON_PRETTY_PRINT);
        if (is_string($replacementComposerJsonString)) {

            $replacementComposerJsonHash = hash('sha512', $replacementComposerJsonString);
            if (! hash_equals($composerJsonHash, $replacementComposerJsonHash)) {
                $replacementComposerJsonString = preg_replace('/^(  +?)\\1(?=[^ ])/m', '$1', $replacementComposerJsonString);
                if (is_string($replacementComposerJsonString)) {

                    $io->write('<info>wyrihaximus/test-utilities:</info> Writing new <fg=cyan>composer.json</>');
                    file_put_contents($rootPackagePath . '/composer.json', $replacementComposerJsonString);
                }
            }
        }

        if (is_array($composerJson) && array_key_exists('scripts', $composerJson) && is_array($composerJson['scripts'])) {
            if (array_key_exists('post-install-cmd', $composerJson['scripts'])) {
                $composerJson['scripts']['post-install-cmd'] = self::addMakeOnInstallOrUpdateToScriptsSectionAndRemoveCommandsItReplaces($composerJson['scripts']['post-install-cmd']);
            }

            if (array_key_exists('post-update-cmd', $composerJson['scripts'])) {
                $composerJson['scripts']['post-update-cmd'] = self::addMakeOnInstallOrUpdateToScriptsSectionAndRemoveCommandsItReplaces($composerJson['scripts']['post-update-cmd']);
            }
        }

        $replacementComposerJsonString = json_encode($composerJson, JSON_UNESCAPED_SLASHES | JSON_PRETTY_PRINT);
        if (is_string($replacementComposerJsonString)) {

            $replacementComposerJsonHash = hash('sha512', $replacementComposerJsonString);
            if (! hash_equals($composerJsonHash, $replacementComposerJsonHash)) {
                $replacementComposerJsonString = preg_replace('/^(  +?)\\1(?=[^ ])/m', '$1', $replacementComposerJsonString);
                if (is_string($replacementComposerJsonString)) {

                    $io->write('<info>wyrihaximus/test-utilities:</info> Writing new <fg=cyan>composer.json</>');
                    file_put_contents($rootPackagePath . '/composer.json', $replacementComposerJsonString . "\r\n");
                }
            }
        }

        $io->write('<info>wyrihaximus/test-utilities:</info> Finished <fg=cyan>make on-install-or-update || true</> to scripts');
    }

    /**
     * @param array<int, string> $scriptsSection
     *
     * @return array<int, string>
     */
    private static function addMakeOnInstallOrUpdateToScriptsSectionAndRemoveCommandsItReplaces(array $scriptsSection): array
    {
        foreach ($scriptsSection as $script) {
            if ($script === 'make on-install-or-update || true') {
                return $scriptsSection;
            }
        }

        $scripts = [];
        foreach ($scriptsSection as $script) {
            if ($script === 'composer normalize' || $script === 'composer update --lock --no-scripts') {
                continue;
            }

            $scripts[] = $script;
        }

        $scripts[] = 'make on-install-or-update || true';

        return $scripts;
    }

    /** @return non-empty-string */

    private static function getVendorDir(Composer $composer): string
    {
        $vendorDir = $composer->getConfig()->get('vendor-dir');
        if ($vendorDir === '' || ! file_exists($vendorDir)) {
            throw new Exception('vendor-dir must be a string');
        }

        return $vendorDir;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace WyriHaximus\TestUtilities\Composer;
6
7			use Composer\Composer;
8			use Composer\EventDispatcher\EventSubscriberInterface;
9			use Composer\IO\IOInterface;
10			use Composer\Plugin\PluginInterface;
11			use Composer\Script\Event;
12			use Composer\Script\ScriptEvents;
13			use Exception;
14
15			use function array_key_exists;
16			use function array_keys;
17			use function dirname;
18			use function file_exists;
19			use function file_get_contents;
20			use function file_put_contents;
21			use function hash;
22			use function hash_equals;
23			use function is_array;
24			use function is_string;
25			use function json_decode;
26			use function json_encode;
27			use function preg_replace;
28
29			use const DIRECTORY_SEPARATOR;
30			use const JSON_PRETTY_PRINT;
31			use const JSON_UNESCAPED_SLASHES;
32			use const PHP_INT_MAX;
33
34			final class Installer implements PluginInterface, EventSubscriberInterface
35			{
36			/** @return array<string, array<string\|int>> */
37			public static function getSubscribedEvents(): array
38			{
39			return [ScriptEvents::PRE_AUTOLOAD_DUMP => ['findEventListeners', PHP_INT_MAX]];
40			}
41
42			public function activate(Composer $composer, IOInterface $io): void
43			{
44			// does nothing, see getSubscribedEvents() instead.
45			}
46
47			public function deactivate(Composer $composer, IOInterface $io): void
48			{
49			// does nothing, see getSubscribedEvents() instead.
50			}
51
52			public function uninstall(Composer $composer, IOInterface $io): void
53			{
54			// does nothing, see getSubscribedEvents() instead.
55			}
56
57			/**
58			* Called before every dump autoload, generates a fresh PHP class.
59			*
60			* @phpstan-ignore shipmonk.deadMethod
61			*/
62			public static function findEventListeners(Event $event): void
63			{
64			$rootPackagePath = dirname(self::getVendorDir($event->getComposer())) . DIRECTORY_SEPARATOR;
65			if (! file_exists($rootPackagePath . '/composer.json')) {
66			return;
67			}
68
69			$jsonRaw = file_get_contents($rootPackagePath . '/composer.json');
70			if (! is_string($jsonRaw)) {
			0 ignored issues – show introduced 2025-08-18 16:47 UTC by Report Bug Copy Issue Report The condition `is_string($jsonRaw)` is always `true`. Loading history...
71			return;
72			}
73
74			$json = json_decode($jsonRaw, true);
75			if (! is_array($json)) {
76			return;
77			}
78
79			if (! array_key_exists('require-dev', $json)) {
80			return;
81			}
82
83			if (! is_array($json['require-dev'])) {
84			return;
85			}
86
87			$hasMakefiles = false;
88			foreach (array_keys($json['require-dev']) as $package) {
89			if ($package === 'wyrihaximus/makefiles') {
90			$hasMakefiles = true;
91			break;
92			}
93			}
94
95			if (! $hasMakefiles) {
96			return;
97			}
98
99			unset($hasMakefiles);
100
101			if (array_key_exists('name', $json) && $json['name'] === 'wyrihaximus/test-utilities') {
102			self::addMakeOnInstallOrUpdate($event->getIO(), $rootPackagePath);
103
104			return;
105			}
106
107			foreach (array_keys($json['require-dev']) as $package) {
108			if ($package === 'wyrihaximus/test-utilities') {
109			self::addMakeOnInstallOrUpdate($event->getIO(), $rootPackagePath);
110
111			return;
112			}
113			}
114			}
115
116			private static function addMakeOnInstallOrUpdate(IOInterface $io, string $rootPackagePath): void
117			{
118			$io->write('<info>wyrihaximus/test-utilities:</info> Adding <fg=cyan>make on-install-or-update \|\| true</> to scripts');
119			$composerJsonString = file_get_contents($rootPackagePath . '/composer.json');
120			if (! is_string($composerJsonString)) {
			0 ignored issues – show introduced 2025-08-18 16:47 UTC by Report Bug Copy Issue Report The condition `is_string($composerJsonString)` is always `true`. Loading history...
121			$io->write('<error>wyrihaximus/test-utilities:</error> Unable to read <fg=cyan>composer.json</> aborting');
122
123			return;
124			}
125
126			$composerJson = json_decode($composerJsonString, true);
127			$composerJsonHash = hash('sha512', (string) json_encode($composerJson, JSON_UNESCAPED_SLASHES \| JSON_PRETTY_PRINT));
128			if (is_array($composerJson) && array_key_exists('scripts', $composerJson) && is_array($composerJson['scripts'])) {
129			if (array_key_exists('post-install-cmd', $composerJson['scripts'])) {
130			$composerJson['scripts']['post-install-cmd'] = self::addMakeOnInstallOrUpdateToScriptsSectionAndRemoveCommandsItReplaces($composerJson['scripts']['post-install-cmd']);
131			}
132
133			if (array_key_exists('post-update-cmd', $composerJson['scripts'])) {
134			$composerJson['scripts']['post-update-cmd'] = self::addMakeOnInstallOrUpdateToScriptsSectionAndRemoveCommandsItReplaces($composerJson['scripts']['post-update-cmd']);
135			}
136			}
137
138			$replacementComposerJsonString = json_encode($composerJson, JSON_UNESCAPED_SLASHES \| JSON_PRETTY_PRINT);
139			if (is_string($replacementComposerJsonString)) {
			0 ignored issues – show introduced 2025-08-18 16:47 UTC by Report Bug Copy Issue Report The condition `is_string($replacementComposerJsonString)` is always `true`. Loading history...
140			$replacementComposerJsonHash = hash('sha512', $replacementComposerJsonString);
141			if (! hash_equals($composerJsonHash, $replacementComposerJsonHash)) {
142			$replacementComposerJsonString = preg_replace('/^( +?)\\1(?=[^ ])/m', '$1', $replacementComposerJsonString);
143			if (is_string($replacementComposerJsonString)) {
			0 ignored issues – show introduced 2025-08-18 16:47 UTC by Report Bug Copy Issue Report The condition `is_string($replacementComposerJsonString)` is always `true`. Loading history...
144			$io->write('<info>wyrihaximus/test-utilities:</info> Writing new <fg=cyan>composer.json</>');
145			file_put_contents($rootPackagePath . '/composer.json', $replacementComposerJsonString);
146			}
147			}
148			}
149
150			if (is_array($composerJson) && array_key_exists('scripts', $composerJson) && is_array($composerJson['scripts'])) {
151			if (array_key_exists('post-install-cmd', $composerJson['scripts'])) {
152			$composerJson['scripts']['post-install-cmd'] = self::addMakeOnInstallOrUpdateToScriptsSectionAndRemoveCommandsItReplaces($composerJson['scripts']['post-install-cmd']);
153			}
154
155			if (array_key_exists('post-update-cmd', $composerJson['scripts'])) {
156			$composerJson['scripts']['post-update-cmd'] = self::addMakeOnInstallOrUpdateToScriptsSectionAndRemoveCommandsItReplaces($composerJson['scripts']['post-update-cmd']);
157			}
158			}
159
160			$replacementComposerJsonString = json_encode($composerJson, JSON_UNESCAPED_SLASHES \| JSON_PRETTY_PRINT);
161			if (is_string($replacementComposerJsonString)) {
			0 ignored issues – show introduced 2025-08-18 16:47 UTC by Report Bug Copy Issue Report The condition `is_string($replacementComposerJsonString)` is always `true`. Loading history...
162			$replacementComposerJsonHash = hash('sha512', $replacementComposerJsonString);
163			if (! hash_equals($composerJsonHash, $replacementComposerJsonHash)) {
164			$replacementComposerJsonString = preg_replace('/^( +?)\\1(?=[^ ])/m', '$1', $replacementComposerJsonString);
165			if (is_string($replacementComposerJsonString)) {
			0 ignored issues – show introduced 2025-08-18 16:47 UTC by Report Bug Copy Issue Report The condition `is_string($replacementComposerJsonString)` is always `true`. Loading history...
166			$io->write('<info>wyrihaximus/test-utilities:</info> Writing new <fg=cyan>composer.json</>');
167			file_put_contents($rootPackagePath . '/composer.json', $replacementComposerJsonString . "\r\n");
168			}
169			}
170			}
171
172			$io->write('<info>wyrihaximus/test-utilities:</info> Finished <fg=cyan>make on-install-or-update \|\| true</> to scripts');
173			}
174
175			/**
176			* @param array<int, string> $scriptsSection
177			*
178			* @return array<int, string>
179			*/
180			private static function addMakeOnInstallOrUpdateToScriptsSectionAndRemoveCommandsItReplaces(array $scriptsSection): array
181			{
182			foreach ($scriptsSection as $script) {
183			if ($script === 'make on-install-or-update \|\| true') {
184			return $scriptsSection;
185			}
186			}
187
188			$scripts = [];
189			foreach ($scriptsSection as $script) {
190			if ($script === 'composer normalize' \|\| $script === 'composer update --lock --no-scripts') {
191			continue;
192			}
193
194			$scripts[] = $script;
195			}
196
197			$scripts[] = 'make on-install-or-update \|\| true';
198
199			return $scripts;
200			}
201
202			/** @return non-empty-string */
			0 ignored issues – show Documentation Bug introduced 2025-08-18 16:47 UTC by Report Bug Copy Issue Report The doc comment `non-empty-string` at position `0` could not be parsed: Unknown type name 'non-empty-string' at position 0 in non-empty-string. Loading history...
203			private static function getVendorDir(Composer $composer): string
204			{
205			$vendorDir = $composer->getConfig()->get('vendor-dir');
206			if ($vendorDir === '' \|\| ! file_exists($vendorDir)) {
207			throw new Exception('vendor-dir must be a string');
208			}
209
210			return $vendorDir;
211			}
212			}
213

WyriHaximus / php-test-utilities

GitHub Access Token became invalid

Push — master ( e09157...64b15f )

Installer::addMakeOnInstallOrUpdate() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like