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

Note more TODOs

parent 058a267d
No related branches found
No related tags found
No related merge requests found
......@@ -18,6 +18,7 @@ function is_command_allowed( $command ) {
* @return array
*/
function get_command_whitelist() {
// TODO: constant!
// Supported built-in commands
$whitelist = array(
'cache',
......@@ -54,6 +55,7 @@ function get_command_whitelist() {
* @return array
*/
function get_command_blacklist() {
// TODO: constant!
return array(
'cli',
'config',
......
......@@ -32,6 +32,7 @@ function run_event( $command ) {
// Command failed
if ( ! is_object( $output ) || is_wp_error( $output ) ) {
// TODO: message consistency, revisit using ACTION as the "key", seems wrong
trigger_error( 'WP-CLI command failed. (' . var_export( $command, true ) . ')', E_USER_WARNING );
trigger_error( var_export( $output, true ), E_USER_WARNING );
return;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment