From 3f187395eaed8d6e455adb454a65f9e2503f390a Mon Sep 17 00:00:00 2001 From: Alexandre Alapetite Date: Wed, 7 May 2025 10:47:09 +0200 Subject: Move PHP minimum version check (#7560) It is too late to check for minimum version check in `lib_rss.php` because that file already contains some relatively new PHP language constructs, which will lead to a syntax error - when running with an old PHP version - instead of the expected error message. Moved to `constants.php` for now. Example of syntax error with PHP 7.4: ``` PHP Parse error: syntax error, unexpected '|', expecting '{' in /var/www/FreshRSS/lib/lib_rss.php on line 166 ``` Should help users like in: * https://github.com/FreshRSS/FreshRSS/discussions/7539 * https://github.com/FreshRSS/FreshRSS/issues/7557 --- constants.php | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'constants.php') diff --git a/constants.php b/constants.php index 2b4f563c9..8e9d76160 100644 --- a/constants.php +++ b/constants.php @@ -35,6 +35,10 @@ const CORE_EXTENSIONS_PATH = LIB_PATH . '/core-extensions'; const TESTS_PATH = FRESHRSS_PATH . '/tests'; // +if (version_compare(PHP_VERSION, FRESHRSS_MIN_PHP_VERSION, '<')) { + die(sprintf('Error: FreshRSS requires PHP %s+ but was invoked with PHP %s!', FRESHRSS_MIN_PHP_VERSION, PHP_VERSION)); +} + if (file_exists(__DIR__ . '/constants.local.php')) { //Include custom / local settings: include(__DIR__ . '/constants.local.php'); -- cgit v1.2.3