@@ -116,9 +116,8 @@ discard block |
||
116 | 116 | * `$sourceLocation` or you can just have a string here |
117 | 117 | * that returns the data directly.... |
118 | 118 | * |
119 | - * @param string $fileContent |
|
120 | 119 | * |
121 | - * @return bool - true on success, false on failure |
|
120 | + * @return string|false - true on success, false on failure |
|
122 | 121 | */ |
123 | 122 | protected function getStandardFile() |
124 | 123 | { |
@@ -153,7 +152,8 @@ discard block |
||
153 | 152 | * takes the standard file and adds any |
154 | 153 | * customisations to it from the module |
155 | 154 | * |
156 | - * @return bool - true on success, false on failure |
|
155 | + * @param string|false $fileContent |
|
156 | + * @return boolean|null - true on success, false on failure |
|
157 | 157 | */ |
158 | 158 | protected function customiseStandardFile($fileContent) |
159 | 159 | { |
@@ -166,7 +166,7 @@ discard block |
||
166 | 166 | * |
167 | 167 | * @param string $fileContent |
168 | 168 | * |
169 | - * @return bool - true on success, false on failure |
|
169 | + * @return null|false - true on success, false on failure |
|
170 | 170 | */ |
171 | 171 | protected function saveFile($fileContent) |
172 | 172 | { |
@@ -216,6 +216,9 @@ discard block |
||
216 | 216 | } |
217 | 217 | |
218 | 218 | |
219 | + /** |
|
220 | + * @param string $componentName |
|
221 | + */ |
|
219 | 222 | protected function getReadMeComponent($componentName) |
220 | 223 | { |
221 | 224 | $temp_dir = GitHubModule::Config()->get('absolute_temp_folder'); |
@@ -287,8 +290,6 @@ discard block |
||
287 | 290 | |
288 | 291 | |
289 | 292 | /** |
290 | - * @param string $file |
|
291 | - * @param GitHubModule $gitObject |
|
292 | 293 | * |
293 | 294 | * @return string |
294 | 295 | */ |
@@ -116,7 +116,7 @@ |
||
116 | 116 | * |
117 | 117 | * @param string $name |
118 | 118 | * @param string $variable |
119 | - * @return boolean |
|
119 | + * @return boolean|null |
|
120 | 120 | */ |
121 | 121 | protected function checkForDetailsInComposerFile($name, $variable) |
122 | 122 | { |
@@ -180,6 +180,9 @@ |
||
180 | 180 | return 'https://github.com/'.$username.'/'.$this->ModuleName; |
181 | 181 | } |
182 | 182 | |
183 | + /** |
|
184 | + * @param string|null $directory |
|
185 | + */ |
|
183 | 186 | protected function IsDirGitRepo($directory) |
184 | 187 | { |
185 | 188 | return file_exists($directory."/.git"); |
@@ -322,6 +322,9 @@ discard block |
||
322 | 322 | $configYml = ConfigYML::create($module)->reWrite(); |
323 | 323 | } |
324 | 324 | |
325 | + /** |
|
326 | + * @param string $filename |
|
327 | + */ |
|
325 | 328 | private function checkFile($module, $filename) |
326 | 329 | { |
327 | 330 | $folder = GitHubModule::Config()->get('absolute_temp_folder'); |
@@ -333,6 +336,9 @@ discard block |
||
333 | 336 | return $this->checkFile($module, "README.MD"); |
334 | 337 | } |
335 | 338 | |
339 | + /** |
|
340 | + * @param string $directory |
|
341 | + */ |
|
336 | 342 | private function checkDirExcludedWords($directory, $wordArray) |
337 | 343 | { |
338 | 344 | $filesAndFolders = scandir($directory); |
@@ -361,6 +367,9 @@ discard block |
||
361 | 367 | return $problem_files; |
362 | 368 | } |
363 | 369 | |
370 | + /** |
|
371 | + * @param string $fileName |
|
372 | + */ |
|
364 | 373 | private function checkFileExcludedWords($fileName, $wordArray) |
365 | 374 | { |
366 | 375 | $matchedWords = array(); |