Skip to content

Commit

Permalink
Merge pull request #461 from plural/symfony-reqs
Browse files Browse the repository at this point in the history
update SymfonyRequirements file to match server.
  • Loading branch information
plural authored Dec 24, 2020
2 parents f3b4fa9 + d9a6e30 commit 6108ec6
Showing 1 changed file with 3 additions and 10 deletions.
13 changes: 3 additions & 10 deletions var/SymfonyRequirements.php
Original file line number Diff line number Diff line change
Expand Up @@ -389,7 +389,7 @@ public function __construct()
{
/* mandatory requirements follow */

$installedPhpVersion = phpversion();
$installedPhpVersion = PHP_VERSION;
$requiredPhpVersion = $this->getPhpRequiredVersion();

$this->addRecommendation(
Expand Down Expand Up @@ -448,15 +448,8 @@ public function __construct()
}

if (false !== $requiredPhpVersion && version_compare($installedPhpVersion, $requiredPhpVersion, '>=')) {
$timezones = array();
foreach (DateTimeZone::listAbbreviations() as $abbreviations) {
foreach ($abbreviations as $abbreviation) {
$timezones[$abbreviation['timezone_id']] = true;
}
}

$this->addRequirement(
isset($timezones[@date_default_timezone_get()]),
in_array(@date_default_timezone_get(), DateTimeZone::listIdentifiers(), true),
sprintf('Configured default timezone "%s" must be supported by your installation of PHP', @date_default_timezone_get()),
'Your default timezone is not supported by PHP. Check for typos in your <strong>php.ini</strong> file and have a look at the list of deprecated timezones at <a href="http://php.net/manual/en/timezones.others.php">http://php.net/manual/en/timezones.others.php</a>.'
);
Expand Down Expand Up @@ -731,7 +724,7 @@ function_exists('posix_isatty'),
'Install and/or enable a <strong>PHP accelerator</strong> (highly recommended).'
);

if (strtoupper(substr(PHP_OS, 0, 3)) === 'WIN') {
if ('WIN' === strtoupper(substr(PHP_OS, 0, 3))) {
$this->addRecommendation(
$this->getRealpathCacheSize() >= 5 * 1024 * 1024,
'realpath_cache_size should be at least 5M in php.ini',
Expand Down

0 comments on commit 6108ec6

Please sign in to comment.