Skip to content
Snippets Groups Projects
Commit 2052d884 authored by Erick Hitter's avatar Erick Hitter
Browse files

PHPCS fixes

parent 4bd67d52
Branches
No related tags found
No related merge requests found
......@@ -67,6 +67,8 @@ function validate_command( $command ) {
$command .= ' --allow-root';
}
// TODO: validate further // @codingStandardsIgnoreLine
// Nothing to run.
if ( empty( $command ) ) {
return new WP_Error( 'invalid-command', 'Invalid command provided' );
......
<?php
/**
* Execution handling
*
* @package WP_CLI_Cron_Control_Offload
*/
namespace Automattic\WP\WP_CLI_Cron_Control_Offload;
/**
* Intended for non-interactive use, so all output ends up in the error log
*
* @param string $command
* @param string $command WP-CLI command to execute.
* @return null
*/
function run_event( $command ) {
if ( ! defined( 'WP_CLI' ) || ! \WP_CLI ) {
// TODO: reschedule at least once or twice
// TODO: reschedule at least once or twice.
trigger_error( sprintf( __( '%1$s: Attempted to run event without WP-CLI loaded. (%2$s)', 'wp-cli-cron-control-offload' ), MESSAGE_PREFIX, var_export( $command, true ) ), E_USER_WARNING );
return;
}
......@@ -23,14 +28,14 @@ function run_event( $command ) {
$start = microtime( true );
$output = \WP_CLI::runcommand( $command, array(
'exit_error' => false, // Don't kill the cron process if the WP-CLI command fails, otherwise we can't capture the error
'launch' => true, // Don't reuse as we're in cron context
'return' => 'all', // We want STDERR and the exit code, in addition to STDOUT
'exit_error' => false, // Don't kill the cron process if the WP-CLI command fails, otherwise we can't capture the error.
'launch' => true, // Don't reuse as we're in cron context.
'return' => 'all', // We want STDERR and the exit code, in addition to STDOUT.
) );
$end = microtime( true );
// Command failed
// Command failed.
if ( ! is_object( $output ) || is_wp_error( $output ) ) {
trigger_error( sprintf( __( '%1$s: WP-CLI command failed. (%2$s)', 'wp-cli-cron-control-offload' ), MESSAGE_PREFIX, var_export( $command, true ) ), E_USER_WARNING );
......@@ -39,7 +44,7 @@ function run_event( $command ) {
return;
}
// On success, reformat response for logging
// On success, reformat response for logging.
$output->command = $command;
$output->start = $start;
$output->end = $end;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment