Merge pull request #13817 from owncloud/core-apps-dependencies

encryption requires openssl and user_ldap requires ldap 🙊
This commit is contained in:
Morris Jobke 2015-02-02 12:49:45 +01:00
commit 1bb8d5978c
3 changed files with 8 additions and 5 deletions

View File

@ -19,4 +19,7 @@
<filesystem/>
</types>
<ocsid>166047</ocsid>
<dependencies>
<lib>openssl</lib>
</dependencies>
</info>

View File

@ -18,4 +18,7 @@ A user logs into ownCloud with their LDAP or AD credentials, and is granted acce
<admin>admin-ldap</admin>
</documentation>
<ocsid>166061</ocsid>
<dependencies>
<lib>ldap</lib>
</dependencies>
</info>

View File

@ -28,13 +28,10 @@ class PlatformRepository {
$ext = new \ReflectionExtension($name);
try {
$prettyVersion = $ext->getVersion();
} catch (\UnexpectedValueException $e) {
$prettyVersion = '0';
}
try {
$prettyVersion = $this->normalizeVersion($prettyVersion);
} catch (\UnexpectedValueException $e) {
continue;
$prettyVersion = '0';
$prettyVersion = $this->normalizeVersion($prettyVersion);
}
$packages[$this->buildPackageName($name)] = $prettyVersion;