Merge pull request #25335 from nextcloud/fix/app-fetcher-php-compat-comparison

Fix/app fetcher php compat comparison

Author: rullzer

lib/private/App/AppStore/Fetcher/AppFetcher.php

@@ -113,12 +113,12 @@ protected function fetch($ETag, $content, $allowUnstable = false) {
 							$phpVersion = $versionParser->getVersion($release['rawPhpVersionSpec']);
 							$minPhpVersion = $phpVersion->getMinimumVersion();
 							$maxPhpVersion = $phpVersion->getMaximumVersion();
-							$minPhpFulfilled = $minPhpVersion === '' || version_compare(
+							$minPhpFulfilled = $minPhpVersion === '' || $this->compareVersion->isCompatible(
 									PHP_VERSION,
 									$minPhpVersion,
 									'>='
 								);
-							$maxPhpFulfilled = $maxPhpVersion === '' || version_compare(
+							$maxPhpFulfilled = $maxPhpVersion === '' || $this->compareVersion->isCompatible(
 									PHP_VERSION,
 									$maxPhpVersion,
 									'<='

lib/private/App/CompareVersion.php

@@ -1,4 +1,7 @@
 <?php
+
+declare(strict_types=1);
+
 /**
  * @copyright 2018 Christoph Wurst <christoph@winzerhof-wurst.at>
  *
@@ -24,12 +27,13 @@
 namespace OC\App;
 
 use InvalidArgumentException;
+use function explode;
 
 class CompareVersion {
-	public const REGEX_MAJOR = '/^\d+$/';
-	public const REGEX_MAJOR_MINOR = '/^\d+.\d+$/';
-	public const REGEX_MAJOR_MINOR_PATCH = '/^\d+.\d+.\d+$/';
-	public const REGEX_SERVER = '/^\d+.\d+.\d+(.\d+)?$/';
+	private const REGEX_MAJOR = '/^\d+$/';
+	private const REGEX_MAJOR_MINOR = '/^\d+\.\d+$/';
+	private const REGEX_MAJOR_MINOR_PATCH = '/^\d+\.\d+\.\d+(?!\.\d+)/';
+	private const REGEX_ACTUAL = '/^\d+(\.\d+){1,2}/';
 
 	/**
 	 * Checks if the given server version fulfills the given (app) version requirements.
@@ -45,18 +49,19 @@ class CompareVersion {
 	 */
 	public function isCompatible(string $actual, string $required,
 		string $comparator = '>='): bool {
-		if (!preg_match(self::REGEX_SERVER, $actual)) {
-			throw new InvalidArgumentException('server version is invalid');
+		if (!preg_match(self::REGEX_ACTUAL, $actual, $matches)) {
+			throw new InvalidArgumentException("version specification $actual is invalid");
 		}
+		$cleanActual = $matches[0];
 
 		if (preg_match(self::REGEX_MAJOR, $required) === 1) {
-			return $this->compareMajor($actual, $required, $comparator);
+			return $this->compareMajor($cleanActual, $required, $comparator);
 		} elseif (preg_match(self::REGEX_MAJOR_MINOR, $required) === 1) {
-			return $this->compareMajorMinor($actual, $required, $comparator);
+			return $this->compareMajorMinor($cleanActual, $required, $comparator);
 		} elseif (preg_match(self::REGEX_MAJOR_MINOR_PATCH, $required) === 1) {
-			return $this->compareMajorMinorPatch($actual, $required, $comparator);
+			return $this->compareMajorMinorPatch($cleanActual, $required, $comparator);
 		} else {
-			throw new InvalidArgumentException('required version is invalid');
+			throw new InvalidArgumentException("required version $required is invalid");
 		}
 	}
 

tests/lib/App/CompareVersionTest.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 /**
  * @copyright 2018 Christoph Wurst <christoph@winzerhof-wurst.at>
  *
@@ -49,11 +51,17 @@ public function comparisonData() {
 			['13.0.0', '13', '>=', true],
 			['13.0.1', '13', '>=', true],
 			['13.0.1', '13', '<=', true],
+			['13.0.1.9', '13', '<=', true],
+			['13.0.1-beta.1', '13', '<=', true],
+			['7.4.14', '7.4', '<=', true],
+			['7.4.14-ubuntu', '7.4', '<=', true],
+			['7.4.14-ubuntu', '7.4.15', '<=', true],
 			// Incompatible major versions
 			['13.0.0.3', '13.0.0', '<', false],
 			['12.0.0', '13.0.0', '>=', false],
 			['12.0.0', '13.0', '>=', false],
 			['12.0.0', '13', '>=', false],
+			['7.4.15-ubuntu', '7.4.15', '>=', true],
 			// Incompatible minor and patch versions
 			['13.0.0', '13.0.1', '>=', false],
 			['13.0.0', '13.1', '>=', false],