Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Incorporate @caddoo's suggested changes

Co-authored-by: caddoo <matthew.caddoo@gmail.com>
  • Loading branch information
danielpunkass and caddoo authored Dec 9, 2024
1 parent 5a2b9f8 commit 2992e06
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions core/FileIntegrity.php
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ protected static function getMessagesDirectoriesFoundButNotExpected($messages)

$directories = array();
foreach ($directoriesFoundButNotExpected as $directoryFoundNotExpected) {
$directories[] = htmlspecialchars($directoryFoundNotExpected);
$directories[] = htmlspecialchars(realpath(dirname($directoryFoundNotExpected)). DIRECTORY_SEPARATOR . basename($directoryFoundNotExpected));

Check failure on line 95 in core/FileIntegrity.php

View workflow job for this annotation

GitHub Actions / PHPCS

Expected at least 1 space before "."; 0 found
}

$deleteAllAtOnce = array();
Expand Down Expand Up @@ -138,7 +138,8 @@ protected static function getMessagesFilesFoundButNotExpected($messages)

$files = array();
foreach ($filesFoundButNotExpected as $fileFoundNotExpected) {
$files[] = '"' . htmlspecialchars($fileFoundNotExpected) . '"';
$files[] = '"' . htmlspecialchars(realpath(dirname($fileFoundNotExpected)). DIRECTORY_SEPARATOR . basename($fileFoundNotExpected)) . '"';

Check failure on line 141 in core/FileIntegrity.php

View workflow job for this annotation

GitHub Actions / PHPCS

Expected at least 1 space before "."; 0 found

Check failure on line 142 in core/FileIntegrity.php

View workflow job for this annotation

GitHub Actions / PHPCS

Blank line found at end of control structure
}

$deleteAllAtOnce = array();
Expand Down

0 comments on commit 2992e06

Please sign in to comment.