@@ -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 | } |
@@ -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) [ |
@@ -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; |
@@ -51,7 +51,7 @@ discard block |
||
51 | 51 | * |
52 | 52 | * @return \ComponentManager\Moodle |
53 | 53 | */ |
54 | - protected function getMoodle($moodleDirectory=null) { |
|
54 | + protected function getMoodle($moodleDirectory = null) { |
|
55 | 55 | if ($this->moodle === null) { |
56 | 56 | $moodleDirectory = ($moodleDirectory === null) |
57 | 57 | ? PlatformUtil::workingDirectory() : $moodleDirectory; |
@@ -70,7 +70,7 @@ discard block |
||
70 | 70 | * |
71 | 71 | * @return \ComponentManager\Project\Project |
72 | 72 | */ |
73 | - protected function getProject($projectFilename=null, $projectLockFilename=null) { |
|
73 | + protected function getProject($projectFilename = null, $projectLockFilename = null) { |
|
74 | 74 | if ($this->project === null) { |
75 | 75 | if ($projectFilename === null) { |
76 | 76 | $projectFilename = PlatformUtil::workingDirectory() |
@@ -27,7 +27,7 @@ |
||
27 | 27 | * |
28 | 28 | * @return void |
29 | 29 | */ |
30 | - public function setContainer(ContainerInterface $container=null) { |
|
30 | + public function setContainer(ContainerInterface $container = null) { |
|
31 | 31 | $this->container = $container; |
32 | 32 | } |
33 | 33 | } |
@@ -100,7 +100,7 @@ |
||
100 | 100 | * |
101 | 101 | * @return void |
102 | 102 | */ |
103 | - public function fetch($remote, $withTags=true) { |
|
103 | + public function fetch($remote, $withTags = true) { |
|
104 | 104 | $process = $this->getProcess(['fetch', $remote]); |
105 | 105 | $process->run(); |
106 | 106 |