Skip to content

Commit 2c935ac

Browse files
committed
feat: Code refactoring
1 parent 35ab6a1 commit 2c935ac

File tree

2 files changed

+8
-3
lines changed

2 files changed

+8
-3
lines changed

src/SubCommands/BlockAccessToSensitiveDirectories.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ class BlockAccessToSensitiveDirectories extends SubCommand {
1616
/**
1717
* @return array
1818
*/
19-
public function getTemplateVars() {
19+
public function getTemplateVars() : array {
2020
$directories = $this->commandArguments['directories'] ?? $this->sensitiveDirectories;
2121
if ( ! empty( $directories ) ) {
2222
$directories = explode( ',', $directories );

src/SubCommands/BlockAccessToSensitiveFiles.php

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,13 @@ class BlockAccessToSensitiveFiles extends SubCommand {
88
public string $successMessage = 'Block Access to Sensitive Files rule has been deployed.';
99
public string $removalMessage= 'Block Access to Sensitive Files rule has been removed.';
1010

11-
public function getTemplateVars() {
12-
$files = isset( $this->commandArguments['files'] ) ? $this->commandArguments['files'] : 'readme.html,readme.txt,wp-config.php,nginx.conf,/wp-admin/install.php,/wp-admin/upgrade.php';
11+
/**
12+
* @var string List of files that we are protecting by default
13+
*/
14+
private string $protectedFiles = 'readme.html,readme.txt,wp-config.php,nginx.conf,/wp-admin/install.php,/wp-admin/upgrade.php';
15+
16+
public function getTemplateVars() : array {
17+
$files = $this->commandArguments['files'] ?? $this->protectedFiles;
1318
if ( ! empty( $files ) ) {
1419
$files = explode( ',', $files );
1520
$files = array_map( 'trim', $files );

0 commit comments

Comments
 (0)