Skip to content

Commit

Permalink
Omitted a parameter from the configuration report during system insta…
Browse files Browse the repository at this point in the history
…llation
  • Loading branch information
areksolek committed Aug 22, 2023
1 parent d3e2a71 commit 5902d5d
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions app/Utils/ConfReport.php
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,8 @@ class ConfReport
* @var string[]
*/
public static $container = ['php', 'env', 'ext', 'request', 'db', 'writableFilesAndFolders'];

/** @var string[] Skip writable files in install. */
public static $skipWritableFilesInInstall = ['config/Modules'];
/**
* Stability variables map.
*
Expand Down Expand Up @@ -1599,7 +1600,7 @@ private static function validateIsWritable(string $name, array $row, string $sap
$path = ROOT_DIRECTORY . \DIRECTORY_SEPARATOR . $path;
}
$exists = file_exists($path);
if (empty($row['mustExist']) && !$exists) {
if ((empty($row['mustExist']) || ((isset(\App\Process::$requestMode) && 'Install' === \App\Process::$requestMode) && in_array($name, static::$skipWritableFilesInInstall))) && !$exists) {
$row['mode'] = 'skipParam';
} else {
$row['status'] = \App\Fields\File::isWriteable($path, true);
Expand Down

0 comments on commit 5902d5d

Please sign in to comment.