app\commands\GenerateController::actionKey()

Complexity

Conditions	3
Paths	3

Size

Total Lines	16
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	1	Features	1

<?php
    public function actionKey(bool $dryRun = false, string $filename = '.env')
    {
        $environmentFilePath = Yii::getAlias('@app/' . $filename);
        Loader::load('', $filename, true);
        foreach (['COOKIE_VALIDATION_KEY', 'JWT_SECRET'] as $item) {
            $escaped = preg_quote('=' . env($item), '/');

Are you sure env($item) of type false|string can be used in concatenation?

            $keyReplacementPattern = "/^{$item}{$escaped}/m";
            $key = Security::generateRealUniqId(32);
            if (!$dryRun) {
                file_put_contents($environmentFilePath, preg_replace(

It seems like $environmentFilePath can also be of type boolean; however, parameter $filename of file_put_contents() does only seem to accept string, maybe add an additional type check?

                    $keyReplacementPattern,
                    "{$item}={$key}",
                    file_get_contents($environmentFilePath)

It seems like $environmentFilePath can also be of type boolean; however, parameter $filename of file_get_contents() does only seem to accept string, maybe add an additional type check?

                ));
            }
            $this->stdout("{$item} key [{$key}] set successfully.\n", Console::FG_GREEN);
        }