| @@ -34,109 +34,109 @@ | ||
| 34 | 34 | use OCP\ILogger; | 
| 35 | 35 | |
| 36 | 36 |  class AppFetcher extends Fetcher { | 
| 37 | - /** | |
| 38 | - * @param Factory $appDataFactory | |
| 39 | - * @param IClientService $clientService | |
| 40 | - * @param ITimeFactory $timeFactory | |
| 41 | - * @param IConfig $config | |
| 42 | - * @param ILogger $logger | |
| 43 | - */ | |
| 44 | - public function __construct(Factory $appDataFactory, | |
| 45 | - IClientService $clientService, | |
| 46 | - ITimeFactory $timeFactory, | |
| 47 | - IConfig $config, | |
| 48 | -								ILogger $logger) { | |
| 49 | - parent::__construct( | |
| 50 | - $appDataFactory, | |
| 51 | - $clientService, | |
| 52 | - $timeFactory, | |
| 53 | - $config, | |
| 54 | - $logger | |
| 55 | - ); | |
| 37 | + /** | |
| 38 | + * @param Factory $appDataFactory | |
| 39 | + * @param IClientService $clientService | |
| 40 | + * @param ITimeFactory $timeFactory | |
| 41 | + * @param IConfig $config | |
| 42 | + * @param ILogger $logger | |
| 43 | + */ | |
| 44 | + public function __construct(Factory $appDataFactory, | |
| 45 | + IClientService $clientService, | |
| 46 | + ITimeFactory $timeFactory, | |
| 47 | + IConfig $config, | |
| 48 | +                                ILogger $logger) { | |
| 49 | + parent::__construct( | |
| 50 | + $appDataFactory, | |
| 51 | + $clientService, | |
| 52 | + $timeFactory, | |
| 53 | + $config, | |
| 54 | + $logger | |
| 55 | + ); | |
| 56 | 56 | |
| 57 | - $this->fileName = 'apps.json'; | |
| 58 | - $this->setEndpoint(); | |
| 59 | - } | |
| 57 | + $this->fileName = 'apps.json'; | |
| 58 | + $this->setEndpoint(); | |
| 59 | + } | |
| 60 | 60 | |
| 61 | - /** | |
| 62 | - * Only returns the latest compatible app release in the releases array | |
| 63 | - * | |
| 64 | - * @param string $ETag | |
| 65 | - * @param string $content | |
| 66 | - * | |
| 67 | - * @return array | |
| 68 | - */ | |
| 69 | -	protected function fetch($ETag, $content) { | |
| 70 | - /** @var mixed[] $response */ | |
| 71 | - $response = parent::fetch($ETag, $content); | |
| 61 | + /** | |
| 62 | + * Only returns the latest compatible app release in the releases array | |
| 63 | + * | |
| 64 | + * @param string $ETag | |
| 65 | + * @param string $content | |
| 66 | + * | |
| 67 | + * @return array | |
| 68 | + */ | |
| 69 | +    protected function fetch($ETag, $content) { | |
| 70 | + /** @var mixed[] $response */ | |
| 71 | + $response = parent::fetch($ETag, $content); | |
| 72 | 72 | |
| 73 | - $ncVersion = $this->getVersion(); | |
| 74 | -		foreach($response['data'] as $dataKey => $app) { | |
| 75 | - $releases = []; | |
| 73 | + $ncVersion = $this->getVersion(); | |
| 74 | +        foreach($response['data'] as $dataKey => $app) { | |
| 75 | + $releases = []; | |
| 76 | 76 | |
| 77 | - // Filter all compatible releases | |
| 78 | -			foreach($app['releases'] as $release) { | |
| 79 | - // Exclude all nightly and pre-releases | |
| 80 | - if($release['isNightly'] === false | |
| 81 | -					&& strpos($release['version'], '-') === false) { | |
| 82 | - // Exclude all versions not compatible with the current version | |
| 83 | - $versionParser = new VersionParser(); | |
| 84 | - $version = $versionParser->getVersion($release['rawPlatformVersionSpec']); | |
| 85 | - if ( | |
| 86 | - // Version is bigger or equals to the minimum version of the app | |
| 87 | - version_compare($ncVersion, $version->getMinimumVersion(), '>=') | |
| 88 | - // Version is smaller or equals to the maximum version of the app | |
| 89 | - && version_compare($ncVersion, $version->getMaximumVersion(), '<=') | |
| 90 | -					) { | |
| 91 | - $releases[] = $release; | |
| 92 | - } | |
| 93 | - } | |
| 94 | - } | |
| 77 | + // Filter all compatible releases | |
| 78 | +            foreach($app['releases'] as $release) { | |
| 79 | + // Exclude all nightly and pre-releases | |
| 80 | + if($release['isNightly'] === false | |
| 81 | +                    && strpos($release['version'], '-') === false) { | |
| 82 | + // Exclude all versions not compatible with the current version | |
| 83 | + $versionParser = new VersionParser(); | |
| 84 | + $version = $versionParser->getVersion($release['rawPlatformVersionSpec']); | |
| 85 | + if ( | |
| 86 | + // Version is bigger or equals to the minimum version of the app | |
| 87 | + version_compare($ncVersion, $version->getMinimumVersion(), '>=') | |
| 88 | + // Version is smaller or equals to the maximum version of the app | |
| 89 | + && version_compare($ncVersion, $version->getMaximumVersion(), '<=') | |
| 90 | +                    ) { | |
| 91 | + $releases[] = $release; | |
| 92 | + } | |
| 93 | + } | |
| 94 | + } | |
| 95 | 95 | |
| 96 | - // Get the highest version | |
| 97 | - $versions = []; | |
| 98 | -			foreach($releases as $release) { | |
| 99 | - $versions[] = $release['version']; | |
| 100 | - } | |
| 101 | - usort($versions, 'version_compare'); | |
| 102 | - $versions = array_reverse($versions); | |
| 103 | - $compatible = false; | |
| 104 | -			if(isset($versions[0])) { | |
| 105 | - $highestVersion = $versions[0]; | |
| 106 | -				foreach ($releases as $release) { | |
| 107 | -					if ((string)$release['version'] === (string)$highestVersion) { | |
| 108 | - $compatible = true; | |
| 109 | - $response['data'][$dataKey]['releases'] = [$release]; | |
| 110 | - break; | |
| 111 | - } | |
| 112 | - } | |
| 113 | - } | |
| 114 | -			if(!$compatible) { | |
| 115 | - unset($response['data'][$dataKey]); | |
| 116 | - } | |
| 117 | - } | |
| 96 | + // Get the highest version | |
| 97 | + $versions = []; | |
| 98 | +            foreach($releases as $release) { | |
| 99 | + $versions[] = $release['version']; | |
| 100 | + } | |
| 101 | + usort($versions, 'version_compare'); | |
| 102 | + $versions = array_reverse($versions); | |
| 103 | + $compatible = false; | |
| 104 | +            if(isset($versions[0])) { | |
| 105 | + $highestVersion = $versions[0]; | |
| 106 | +                foreach ($releases as $release) { | |
| 107 | +                    if ((string)$release['version'] === (string)$highestVersion) { | |
| 108 | + $compatible = true; | |
| 109 | + $response['data'][$dataKey]['releases'] = [$release]; | |
| 110 | + break; | |
| 111 | + } | |
| 112 | + } | |
| 113 | + } | |
| 114 | +            if(!$compatible) { | |
| 115 | + unset($response['data'][$dataKey]); | |
| 116 | + } | |
| 117 | + } | |
| 118 | 118 | |
| 119 | - $response['data'] = array_values($response['data']); | |
| 120 | - return $response; | |
| 121 | - } | |
| 119 | + $response['data'] = array_values($response['data']); | |
| 120 | + return $response; | |
| 121 | + } | |
| 122 | 122 | |
| 123 | -	private function setEndpoint() { | |
| 124 | -		$versionArray = explode('.', $this->getVersion()); | |
| 125 | - $this->endpointUrl = sprintf( | |
| 126 | - 'https://apps.nextcloud.com/api/v1/platform/%d.%d.%d/apps.json', | |
| 127 | - $versionArray[0], | |
| 128 | - $versionArray[1], | |
| 129 | - $versionArray[2] | |
| 130 | - ); | |
| 131 | - } | |
| 123 | +    private function setEndpoint() { | |
| 124 | +        $versionArray = explode('.', $this->getVersion()); | |
| 125 | + $this->endpointUrl = sprintf( | |
| 126 | + 'https://apps.nextcloud.com/api/v1/platform/%d.%d.%d/apps.json', | |
| 127 | + $versionArray[0], | |
| 128 | + $versionArray[1], | |
| 129 | + $versionArray[2] | |
| 130 | + ); | |
| 131 | + } | |
| 132 | 132 | |
| 133 | - /** | |
| 134 | - * @param string $version | |
| 135 | - * @param string $filename | |
| 136 | - */ | |
| 137 | -	public function setVersion(string $version, string $fileName = 'apps.json') { | |
| 138 | - parent::setVersion($version); | |
| 139 | - $this->fileName = $fileName; | |
| 140 | - $this->setEndpoint(); | |
| 141 | - } | |
| 133 | + /** | |
| 134 | + * @param string $version | |
| 135 | + * @param string $filename | |
| 136 | + */ | |
| 137 | +    public function setVersion(string $version, string $fileName = 'apps.json') { | |
| 138 | + parent::setVersion($version); | |
| 139 | + $this->fileName = $fileName; | |
| 140 | + $this->setEndpoint(); | |
| 141 | + } | |
| 142 | 142 | } | 
| @@ -71,13 +71,13 @@ discard block | ||
| 71 | 71 | $response = parent::fetch($ETag, $content); | 
| 72 | 72 | |
| 73 | 73 | $ncVersion = $this->getVersion(); | 
| 74 | -		foreach($response['data'] as $dataKey => $app) { | |
| 74 | +		foreach ($response['data'] as $dataKey => $app) { | |
| 75 | 75 | $releases = []; | 
| 76 | 76 | |
| 77 | 77 | // Filter all compatible releases | 
| 78 | -			foreach($app['releases'] as $release) { | |
| 78 | +			foreach ($app['releases'] as $release) { | |
| 79 | 79 | // Exclude all nightly and pre-releases | 
| 80 | - if($release['isNightly'] === false | |
| 80 | + if ($release['isNightly'] === false | |
| 81 | 81 |  					&& strpos($release['version'], '-') === false) { | 
| 82 | 82 | // Exclude all versions not compatible with the current version | 
| 83 | 83 | $versionParser = new VersionParser(); | 
| @@ -95,23 +95,23 @@ discard block | ||
| 95 | 95 | |
| 96 | 96 | // Get the highest version | 
| 97 | 97 | $versions = []; | 
| 98 | -			foreach($releases as $release) { | |
| 98 | +			foreach ($releases as $release) { | |
| 99 | 99 | $versions[] = $release['version']; | 
| 100 | 100 | } | 
| 101 | 101 | usort($versions, 'version_compare'); | 
| 102 | 102 | $versions = array_reverse($versions); | 
| 103 | 103 | $compatible = false; | 
| 104 | -			if(isset($versions[0])) { | |
| 104 | +			if (isset($versions[0])) { | |
| 105 | 105 | $highestVersion = $versions[0]; | 
| 106 | 106 |  				foreach ($releases as $release) { | 
| 107 | -					if ((string)$release['version'] === (string)$highestVersion) { | |
| 107 | +					if ((string) $release['version'] === (string) $highestVersion) { | |
| 108 | 108 | $compatible = true; | 
| 109 | 109 | $response['data'][$dataKey]['releases'] = [$release]; | 
| 110 | 110 | break; | 
| 111 | 111 | } | 
| 112 | 112 | } | 
| 113 | 113 | } | 
| 114 | -			if(!$compatible) { | |
| 114 | +			if (!$compatible) { | |
| 115 | 115 | unset($response['data'][$dataKey]); | 
| 116 | 116 | } | 
| 117 | 117 | } |