From 9e43b23e9a4ae0ecb586f67cd3bf5c359dbe7c2d Mon Sep 17 00:00:00 2001 From: Brandon Kraft Date: Tue, 19 Apr 2022 13:34:27 -0500 Subject: [PATCH] PHPCS fixes --- features/jetpack-crm.php | 21 +++++++++++++++------ 1 file changed, 15 insertions(+), 6 deletions(-) diff --git a/features/jetpack-crm.php b/features/jetpack-crm.php index 47c2156..00c34aa 100644 --- a/features/jetpack-crm.php +++ b/features/jetpack-crm.php @@ -79,6 +79,8 @@ function ( $features, $json_params ) { /** * Installs and activates a specified version of Jetpack CRM from the WP.org plugin repo. + * + * @param string $version Version of Jetpack CRM. */ function add_jpcrm_from_wporg( $version ) { @@ -100,15 +102,22 @@ function ( $s ) use ( $cmd ) { /** * Installs and activates a specified build of Jetpack CRM from our custom build URL. + * + * @param string $build Hash of build to use. */ function add_jpcrm_from_custom_build( $build ) { - // Require commit SHA-1 hash (40 char long hex). - // if ( ! preg_match( '/^[A-Fa-f0-9]{40}$/', $build ) ) { - // return new \WP_Error( 'bad_commit_hash', 'Invalid commit hash.', array( 'status' => 404 ) ); - // } + // phpcs:disable Squiz.PHP.CommentedOutCode.Found - $clean_build = str_replace('/', '_', $build); + /* + * Require commit SHA-1 hash (40 char long hex). + * if ( ! preg_match( '/^[A-Fa-f0-9]{40}$/', $build ) ) { + * return new \WP_Error( 'bad_commit_hash', 'Invalid commit hash.', array( 'status' => 404 ) ); + * } + */ + // phpcs:enable + + $clean_build = str_replace( '/', '_', $build ); // note that this public link is in a public repo $jpcrm_build_base_url = 'https://jetpackcrm-builds.s3.amazonaws.com/builds/'; @@ -150,4 +159,4 @@ function () {