diff --git a/lib/private/app/dependencyanalyzer.php b/lib/private/app/dependencyanalyzer.php index 2e83887238..795017e2b3 100644 --- a/lib/private/app/dependencyanalyzer.php +++ b/lib/private/app/dependencyanalyzer.php @@ -26,6 +26,8 @@ class DependencyAnalyzer { /** @var array */ private $dependencies = array(); + private $appInfo = array(); + /** * @param array $app * @param Platform $platform @@ -49,6 +51,7 @@ class DependencyAnalyzer { $this->analyzeCommands(); $this->analyzeLibraries(); $this->analyzeOS(); + $this->analyzeOC(); return $this->missing; } @@ -154,7 +157,31 @@ class DependencyAnalyzer { } } + private function analyzeOC() { + $minVersion = null; + if (isset($this->dependencies['owncloud']['@attributes']['min-version'])) { + $minVersion = $this->dependencies['owncloud']['@attributes']['min-version']; + } elseif (isset($this->appInfo['requiremin'])) { + $minVersion = $this->appInfo['requiremin']; + } + $maxVersion = null; + if (isset($this->dependencies['oc']['@attributes']['max-version'])) { + $maxVersion = $this->dependencies['oc']['@attributes']['max-version']; + } elseif (isset($this->appInfo['requiremax'])) { + $maxVersion = $this->appInfo['requiremax']; + } + if (!is_null($minVersion)) { + if (version_compare($this->platform->getOcVersion(), $minVersion, '<')) { + $this->addMissing((string)$this->l->t('ownCloud %s or higher is required.', $minVersion)); + } + } + if (!is_null($maxVersion)) { + if (version_compare($this->platform->getOcVersion(), $maxVersion, '>')) { + $this->addMissing((string)$this->l->t('ownCloud with a version lower than %s is required.', $maxVersion)); + } + } + } /** * @param $element diff --git a/lib/private/app/platform.php b/lib/private/app/platform.php index 6279bb5f20..ba96fba00d 100644 --- a/lib/private/app/platform.php +++ b/lib/private/app/platform.php @@ -10,6 +10,7 @@ namespace OC\App; +use OC_Util; use OCP\IConfig; /** @@ -35,6 +36,13 @@ class Platform { return phpversion(); } + /** + * @return string + */ + public function getOcVersion() { + return OC_Util::getVersion(); + } + /** * @return string */ diff --git a/tests/data/app/expected-info.json b/tests/data/app/expected-info.json index b899df7a8d..ab4f103939 100644 --- a/tests/data/app/expected-info.json +++ b/tests/data/app/expected-info.json @@ -60,6 +60,12 @@ }, "curl" ], - "os": "Linux" + "os": "Linux", + "owncloud": { + "@attributes" : { + "min-version": "7.01", + "max-version": "8" + } + } } } diff --git a/tests/data/app/valid-info.xml b/tests/data/app/valid-info.xml index 42f4e3edb7..4b22d55d7b 100644 --- a/tests/data/app/valid-info.xml +++ b/tests/data/app/valid-info.xml @@ -29,5 +29,6 @@ intl curl Linux + diff --git a/tests/lib/app/dependencyanalyzer.php b/tests/lib/app/dependencyanalyzer.php index 1cd24193ea..9c5db96e04 100644 --- a/tests/lib/app/dependencyanalyzer.php +++ b/tests/lib/app/dependencyanalyzer.php @@ -51,6 +51,9 @@ class DependencyAnalyzer extends \PHPUnit_Framework_TestCase { } return null; })); + $this->platformMock->expects($this->any()) + ->method('getOcVersion') + ->will( $this->returnValue('8.0.1')); $this->l10nMock = $this->getMockBuilder('\OCP\IL10N') ->disableOriginalConstructor() @@ -161,6 +164,35 @@ class DependencyAnalyzer extends \PHPUnit_Framework_TestCase { $this->assertEquals($expectedMissing, $missing); } + /** + * @dataProvider providesOC + * @param $expectedMissing + * @param $oc + */ + function testOC($expectedMissing, $oc) { + $app = array( + 'dependencies' => array() + ); + if (!is_null($oc)) { + $app['dependencies']['oc'] = $oc; + } + + $analyser = new \OC\App\DependencyAnalyzer($app, $this->platformMock, $this->l10nMock); + $missing = $analyser->analyze(); + + $this->assertTrue(is_array($missing)); + $this->assertEquals($expectedMissing, $missing); + } + + function providesOC() { + return array( + // no version -> no missing dependency + array(array(), null), + array(array('ownCloud 9 or higher is required.'), array('@attributes' => array('min-version' => '9'))), + array(array('ownCloud with a version lower than 5.1.2 is required.'), array('@attributes' => array('max-version' => '5.1.2'))), + ); + } + function providesOS() { return array( array(array(), null),