@@ -118,7 +118,7 @@ |
||
118 | 118 | * |
119 | 119 | * @param string $property |
120 | 120 | * |
121 | - * @return mixed |
|
121 | + * @return string |
|
122 | 122 | * |
123 | 123 | * @throws \OutOfBoundsException |
124 | 124 | */ |
@@ -66,8 +66,8 @@ |
||
66 | 66 | * @param string $packageSource |
67 | 67 | * @param \stdClass $extra |
68 | 68 | */ |
69 | - public function __construct($name, $version, $packageRepository=null, |
|
70 | - $packageSource=null, stdClass $extra=null) { |
|
69 | + public function __construct($name, $version, $packageRepository = null, |
|
70 | + $packageSource = null, stdClass $extra = null) { |
|
71 | 71 | $this->name = $name; |
72 | 72 | $this->version = $version; |
73 | 73 |
@@ -71,7 +71,7 @@ |
||
71 | 71 | * @param \ComponentManager\Project\ProjectLockFile $projectLockFile |
72 | 72 | * @param \ComponentManager\PackageRepository\PackageRepositoryFactory $packageRepositoryFactory |
73 | 73 | * @param \ComponentManager\PackageSource\PackageSourceFactory $packageSourceFactory |
74 | - * @param \ComponentManager\PackageSource\PackageFormatFactory $packageFormatFactory |
|
74 | + * @param PackageFormatFactory $packageFormatFactory |
|
75 | 75 | */ |
76 | 76 | public function __construct(ProjectFile $projectFile, |
77 | 77 | ProjectLockFile $projectLockFile, |
@@ -38,8 +38,6 @@ |
||
38 | 38 | /** |
39 | 39 | * Initialiser. |
40 | 40 | * |
41 | - * @param \ComponentManager\MoodleVersion $version |
|
42 | - * @param \Psr\Log\LoggerInterface $logger |
|
43 | 41 | * @param string $archive |
44 | 42 | * @param string $destination |
45 | 43 | */ |
@@ -63,7 +63,7 @@ discard block |
||
63 | 63 | * @param \ComponentManager\ComponentVersion[] $versions |
64 | 64 | * @param \ComponentManager\PackageRepository\PackageRepository $packageRepository |
65 | 65 | */ |
66 | - public function __construct($name, $versions, $packageRepository=null) { |
|
66 | + public function __construct($name, $versions, $packageRepository = null) { |
|
67 | 67 | $this->name = $name; |
68 | 68 | $this->versions = $versions; |
69 | 69 | |
@@ -120,7 +120,7 @@ discard block |
||
120 | 120 | * @return string |
121 | 121 | */ |
122 | 122 | public function getPluginType() { |
123 | - list($type, ) = $this->getNameParts(); |
|
123 | + list($type,) = $this->getNameParts(); |
|
124 | 124 | |
125 | 125 | return $type; |
126 | 126 | } |
@@ -69,9 +69,9 @@ |
||
69 | 69 | */ |
70 | 70 | protected function getMetadataCacheFilename() { |
71 | 71 | return sprintf(static::METADATA_CACHE_FILENAME, |
72 | - $this->cacheDirectory, |
|
73 | - PlatformUtil::directorySeparator(), |
|
74 | - PlatformUtil::directorySeparator()); |
|
72 | + $this->cacheDirectory, |
|
73 | + PlatformUtil::directorySeparator(), |
|
74 | + PlatformUtil::directorySeparator()); |
|
75 | 75 | } |
76 | 76 | |
77 | 77 | /** |
@@ -272,13 +272,13 @@ discard block |
||
272 | 272 | */ |
273 | 273 | protected function getMetadataCacheFilename() { |
274 | 274 | $urlHash = parse_url($this->options->uri, PHP_URL_HOST) . '-' |
275 | - . $this->options->project; |
|
275 | + . $this->options->project; |
|
276 | 276 | |
277 | 277 | return sprintf(static::METADATA_CACHE_FILENAME, |
278 | - $this->cacheDirectory, |
|
279 | - PlatformUtil::directorySeparator(), |
|
280 | - PlatformUtil::directorySeparator(), |
|
281 | - $urlHash); |
|
278 | + $this->cacheDirectory, |
|
279 | + PlatformUtil::directorySeparator(), |
|
280 | + PlatformUtil::directorySeparator(), |
|
281 | + $urlHash); |
|
282 | 282 | } |
283 | 283 | |
284 | 284 | /** |
@@ -372,7 +372,7 @@ discard block |
||
372 | 372 | */ |
373 | 373 | protected function getProjectRepositoryListUrl() { |
374 | 374 | return sprintf(static::PROJECT_REPOSITORY_LIST_PATH, |
375 | - $this->options->project); |
|
375 | + $this->options->project); |
|
376 | 376 | } |
377 | 377 | |
378 | 378 | /** |
@@ -384,7 +384,7 @@ discard block |
||
384 | 384 | */ |
385 | 385 | protected function getRepositoryBranchesPath($componentName) { |
386 | 386 | return sprintf(static::REPOSITORY_BRANCHES_PATH, $this->options->project, |
387 | - $componentName); |
|
387 | + $componentName); |
|
388 | 388 | } |
389 | 389 | |
390 | 390 | /** |
@@ -396,6 +396,6 @@ discard block |
||
396 | 396 | */ |
397 | 397 | protected function getRepositoryTagsPath($componentName) { |
398 | 398 | return sprintf(static::REPOSITORY_TAGS_PATH, $this->options->project, |
399 | - $componentName); |
|
399 | + $componentName); |
|
400 | 400 | } |
401 | 401 | } |
@@ -322,7 +322,7 @@ discard block |
||
322 | 322 | * |
323 | 323 | * @return mixed The JSON-decoded representation of the response body. |
324 | 324 | */ |
325 | - protected function get($path, array $queryParams=[]) { |
|
325 | + protected function get($path, array $queryParams = []) { |
|
326 | 326 | $uri = $this->options->uri . $path; |
327 | 327 | |
328 | 328 | $client = new Client(); |
@@ -344,7 +344,7 @@ discard block |
||
344 | 344 | * |
345 | 345 | * @return mixed The value attribute of the JSON-decoded response body. |
346 | 346 | */ |
347 | - protected function getAllPages($path, array $queryParams=[]) { |
|
347 | + protected function getAllPages($path, array $queryParams = []) { |
|
348 | 348 | $values = []; |
349 | 349 | |
350 | 350 | $responseBody = (object) [ |
@@ -328,7 +328,7 @@ |
||
328 | 328 | $client = new Client(); |
329 | 329 | $response = $client->get($uri, [ |
330 | 330 | 'headers' => [ |
331 | - 'Authorization' => "Basic {$this->options->authentication}", |
|
331 | + 'Authorization' => "basic {$this->options->authentication}", |
|
332 | 332 | ], |
333 | 333 | 'query' => $queryParams, |
334 | 334 | ]); |
@@ -61,6 +61,6 @@ |
||
61 | 61 | $className = sprintf(self::CLASS_NAME_FORMAT, $id); |
62 | 62 | |
63 | 63 | return new $className($this->filesystem, $this->cacheDirectory, |
64 | - $options); |
|
64 | + $options); |
|
65 | 65 | } |
66 | 66 | } |
@@ -62,8 +62,8 @@ discard block |
||
62 | 62 | */ |
63 | 63 | protected function getMetadataCacheDirectory() { |
64 | 64 | return sprintf(static::CACHE_DIRECTORY, |
65 | - $this->cacheDirectory, |
|
66 | - PlatformUtil::directorySeparator()); |
|
65 | + $this->cacheDirectory, |
|
66 | + PlatformUtil::directorySeparator()); |
|
67 | 67 | } |
68 | 68 | |
69 | 69 | /** |
@@ -106,7 +106,7 @@ discard block |
||
106 | 106 | } |
107 | 107 | |
108 | 108 | return new Component($componentSpecification->getName(), $versions, |
109 | - $this); |
|
109 | + $this); |
|
110 | 110 | } |
111 | 111 | |
112 | 112 | /** |
@@ -83,7 +83,7 @@ |
||
83 | 83 | * @param integer $maturity |
84 | 84 | * @param \ComponentManager\ComponentSource\ComponentSource[] $sources |
85 | 85 | */ |
86 | - public function __construct($version, $release, $maturity, $sources=null) { |
|
86 | + public function __construct($version, $release, $maturity, $sources = null) { |
|
87 | 87 | $this->version = $version; |
88 | 88 | $this->release = $release; |
89 | 89 | $this->maturity = $maturity; |