diff --git a/lib/class.ISPConfig.inc.php b/lib/class.ISPConfig.inc.php index b3a0bfff45018aa3e440bc685084fcfe51782993..948130dcb7aa4a8a73ea9d06ad9790f7f23246b1 100644 --- a/lib/class.ISPConfig.inc.php +++ b/lib/class.ISPConfig.inc.php @@ -174,10 +174,30 @@ class ISPConfig { } public static function wantsPHP() { + // If a new version is added, the getApacheModulesToDisable function should be updated to disable the latest version (this part could be improved) + $available_php_versions = array( + '5.6', + '7.0', + '7.1', + '7.2', + '7.3', + '7.4', + '8.0' + ); if(isset($_GET['use-php']) && $_GET['use-php']) { - return $_GET['use-php']; + if ($_GET['use-php'] === 'system') { + return $_GET['use-php']; + } else { + $use_php = explode(',',$_GET['use-php']); + $php_versions = array_intersect($use_php, $available_php_versions); + if(!empty($php_versions)) { + return $php_versions; + } else { + return false; + } + } } else { - return false; + return $available_php_versions; } } @@ -224,9 +244,9 @@ Possible arguments are: --use-nginx ->Use nginx webserver instead of apache2 --use-amavis ->Use amavis instead of rspamd for mail filtering --use-unbound ->Use unbound instead of bind9 for local resolving. Only allowed if --no-dns is set. - --use-php ->Use specific PHP version instead of installing multiple PHP, e.g. --use-php=7.3 (5.6, 7.0, 7.1, 7.2 and 7.3 available). + --use-php ->Use specific PHP versions, comma separated, instead of installing multiple PHP, e.g. --use-php=7.4,8.0 (5.6, 7.0, 7.1, 7.2, 7.3, 7.4 and 8.0 available). ->--use-php=system disables the sury repository and just installs the system\'s default PHP version. - ->CURRENTLY ONLY EITHER --use-php=system OR OMITTING THE ARGUMENT (use all versions) IS SUPPORTED! + ->ommiting the argument (use all versions) --use-ftp-ports ->This option sets the passive port range for pure-ftpd. You have to specify the port range separated by hyphen, e. g. --use-ftp-ports=40110-40210. ->If not provided the passive port range will not be configured. --no-web ->Do not use ISPConfig on this server to manage webserver setting and don\'t install nginx/apache or pureftpd. This will also prevent installing an ISPConfig UI and implies --no-roundcube as well as --no-pma @@ -279,7 +299,7 @@ Possible arguments are: } elseif(isset($_GET['lang']) && !in_array($_GET['lang'], array('de', 'en'), true)) { self::printHelp(); exit; - } elseif(isset($_GET['use-php']) && $_GET['use-php'] !== 'system') {//!in_array($_GET['use-php'], array('5.6', '7.0', '7.1', '7.2', '7.3', 'system'), true)) { + } elseif(isset($_GET['use-php']) && !self::wantsPHP()) { self::printHelp(); exit; } elseif(isset($_GET['use-ftp-ports']) && (!preg_match('/^([1-9][0-9]+)-([1-9][0-9]+)$/', $_GET['use-ftp-ports'], $pmatch) || intval($pmatch[1]) >= intval($pmatch[2]))) { diff --git a/lib/os/class.ISPConfigDebianOS.inc.php b/lib/os/class.ISPConfigDebianOS.inc.php index ccaa6e97b3eb3119c4727f20b43ae37903c99b92..0a707ede91541213c1cc899ef5a88d7d6e2b4c0f 100644 --- a/lib/os/class.ISPConfigDebianOS.inc.php +++ b/lib/os/class.ISPConfigDebianOS.inc.php @@ -808,16 +808,8 @@ mailman-unsubscribe: "|/var/lib/mailman/mail/mailman unsubscribe mailman"'; if(ISPConfig::wantsPHP() === 'system') { $php_versions = array($this->getSystemPHPVersion()); } else { - // If a new version is added, the getApacheModulesToDisable function should be updated to disable the latest version (this part could be improved) - $php_versions = array( - '5.6', - '7.0', - '7.1', - '7.2', - '7.3', - '7.4', - '8.0' - ); + //ISPConfig::run() validations prevent sending here null values + $php_versions = ISPConfig::wantsPHP(); } $php_modules = array(