We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.
@@ -27,18 +27,18 @@ discard block |
||
| 27 | 27 | $this->legacyFiles = []; |
| 28 | 28 | $this->legacyDirectoryDetected = false; |
| 29 | 29 | |
| 30 | - if (! $this->context()->hasComposerPackage('backpack/filemanager')) { |
|
| 30 | + if (!$this->context()->hasComposerPackage('backpack/filemanager')) { |
|
| 31 | 31 | return StepResult::success('File Manager add-on not detected; no published views to review.'); |
| 32 | 32 | } |
| 33 | 33 | |
| 34 | - if (! $this->context()->fileExists(self::LEGACY_VIEWS_DIRECTORY)) { |
|
| 34 | + if (!$this->context()->fileExists(self::LEGACY_VIEWS_DIRECTORY)) { |
|
| 35 | 35 | return StepResult::success('No legacy File Manager views found in resources/views/vendor/elfinder.'); |
| 36 | 36 | } |
| 37 | 37 | |
| 38 | 38 | $filesystem = new Filesystem(); |
| 39 | 39 | $absoluteDirectory = $this->context()->basePath(self::LEGACY_VIEWS_DIRECTORY); |
| 40 | 40 | |
| 41 | - if (! $filesystem->isDirectory($absoluteDirectory)) { |
|
| 41 | + if (!$filesystem->isDirectory($absoluteDirectory)) { |
|
| 42 | 42 | return StepResult::success('No legacy File Manager views found in resources/views/vendor/elfinder.'); |
| 43 | 43 | } |
| 44 | 44 | |
@@ -73,7 +73,7 @@ discard block |
||
| 73 | 73 | |
| 74 | 74 | public function canFix(StepResult $result): bool |
| 75 | 75 | { |
| 76 | - if (! $result->status->isWarning()) { |
|
| 76 | + if (!$result->status->isWarning()) { |
|
| 77 | 77 | return false; |
| 78 | 78 | } |
| 79 | 79 | |
@@ -87,12 +87,12 @@ discard block |
||
| 87 | 87 | |
| 88 | 88 | public function fix(StepResult $result): StepResult |
| 89 | 89 | { |
| 90 | - if (! $this->legacyDirectoryDetected) { |
|
| 90 | + if (!$this->legacyDirectoryDetected) { |
|
| 91 | 91 | return StepResult::skipped('Legacy File Manager directory no longer present.'); |
| 92 | 92 | } |
| 93 | 93 | |
| 94 | 94 | foreach ($this->legacyFiles as $path) { |
| 95 | - if (! $this->context()->deleteFile($path)) { |
|
| 95 | + if (!$this->context()->deleteFile($path)) { |
|
| 96 | 96 | return StepResult::failure("Could not delete {$path} automatically."); |
| 97 | 97 | } |
| 98 | 98 | } |
@@ -100,7 +100,7 @@ discard block |
||
| 100 | 100 | $filesystem = new Filesystem(); |
| 101 | 101 | $absoluteDirectory = $this->context()->basePath(self::LEGACY_VIEWS_DIRECTORY); |
| 102 | 102 | |
| 103 | - if ($filesystem->isDirectory($absoluteDirectory) && ! $filesystem->deleteDirectory($absoluteDirectory)) { |
|
| 103 | + if ($filesystem->isDirectory($absoluteDirectory) && !$filesystem->deleteDirectory($absoluteDirectory)) { |
|
| 104 | 104 | return StepResult::failure('Could not delete resources/views/vendor/elfinder automatically.'); |
| 105 | 105 | } |
| 106 | 106 | |
@@ -27,7 +27,7 @@ discard block |
||
| 27 | 27 | return StepResult::skipped('show.php config file is not published, core defaults already use the new datagrid component.'); |
| 28 | 28 | } |
| 29 | 29 | |
| 30 | - if (! str_contains($contents, "'component'")) { |
|
| 30 | + if (!str_contains($contents, "'component'")) { |
|
| 31 | 31 | $this->missingComponent = true; |
| 32 | 32 | |
| 33 | 33 | return StepResult::warning( |
@@ -79,7 +79,7 @@ discard block |
||
| 79 | 79 | return StepResult::failure('Could not update show.php automatically.'); |
| 80 | 80 | } |
| 81 | 81 | |
| 82 | - if (! $this->context()->writeFile($this->relativePath, $updatedContents)) { |
|
| 82 | + if (!$this->context()->writeFile($this->relativePath, $updatedContents)) { |
|
| 83 | 83 | return StepResult::failure('Could not save the updated show.php configuration.'); |
| 84 | 84 | } |
| 85 | 85 | |