Skip to content

Add --include-root parameter to also verify root directory #102

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 19 commits into from
Mar 24, 2023
Merged

Add --include-root parameter to also verify root directory #102

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
4df6e69
Add --include-root parameter
paulschreiber Mar 22, 2023
1893b47
Add test for --include-root parameter
paulschreiber Mar 22, 2023
31f15fc
when --allow-root is set, do not warn about wp-config.php and wp-cont…
paulschreiber Mar 22, 2023
e00b62f
PHPCS: align equals signs
paulschreiber Mar 22, 2023
81ea662
Update src/Checksum_Core_Command.php
paulschreiber Mar 23, 2023
e3e1936
when --include-root is enabled, skip entire wp-content directory from…
paulschreiber Mar 23, 2023
56e5ab8
add private $include_root variable
paulschreiber Mar 23, 2023
5653679
update help text to indicate it looks for files and folders
paulschreiber Mar 23, 2023
962472f
Add test for wp-cli.yml
paulschreiber Mar 23, 2023
b133985
Update features/checksum-core.feature
paulschreiber Mar 23, 2023
63fd9e7
Fix Scenario indentation
danielbachhuber Mar 24, 2023
d755af7
Add more tests to clarify expected behavior
danielbachhuber Mar 24, 2023
e69798e
Remove extraneous helper
danielbachhuber Mar 24, 2023
7176faf
Clean up `wp-cli.yml` scenario
danielbachhuber Mar 24, 2023
71c327f
Rebuild README with new flag
danielbachhuber Mar 24, 2023
a59cc2e
Avoid random test failures when the order changes
danielbachhuber Mar 24, 2023
a03bee1
Remove extraneous argument
danielbachhuber Mar 24, 2023
c9bab49
Merge branch 'main' into patch-1
danielbachhuber Mar 24, 2023
dc03bc8
Fix these assertions
danielbachhuber Mar 24, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 15 additions & 3 deletions src/Checksum_Core_Command.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,9 @@ class Checksum_Core_Command extends Checksum_Base_Command {
*
* ## OPTIONS
*
* [--include-root]
* : Verify all files in the root directory.
*
* [--version=<version>]
* : Verify checksums against a specific version of WordPress.
*
Expand Down Expand Up @@ -58,8 +61,9 @@ class Checksum_Core_Command extends Checksum_Base_Command {
* @when before_wp_load
*/
public function __invoke( $args, $assoc_args ) {
$wp_version = '';
$locale = '';
$wp_version = '';
$locale = '';
$this->include_root = false;

if ( ! empty( $assoc_args['version'] ) ) {
$wp_version = $assoc_args['version'];
Expand All @@ -69,6 +73,10 @@ public function __invoke( $args, $assoc_args ) {
$locale = $assoc_args['locale'];
}

if ( ! empty( $assoc_args['include-root'] ) ) {
$this->include_root = true;
}

if ( empty( $wp_version ) ) {
$details = self::get_wp_details();
$wp_version = $details['wp_version'];
Expand Down Expand Up @@ -135,7 +143,11 @@ public function __invoke( $args, $assoc_args ) {
*
* @return bool
*/
protected function filter_file( $filepath ) {
protected function filter_file( $filepath, $include_root = false ) {
if ( true === $this->include_root ) {
return true;
}

return ( 0 === strpos( $filepath, 'wp-admin/' )
|| 0 === strpos( $filepath, 'wp-includes/' )
|| 1 === preg_match( '/^wp-(?!config\.php)([^\/]*)$/', $filepath )
Expand Down