debug

Author: petitphp

features/cache-patch.feature

@@ -6,9 +6,8 @@ Feature: Patch command available for the object cache
       """php
       <?php
       $set_foo = function(){
-        //TODO: DEBUG, remove before merging branch.
-        WP_CLI::log( var_export( wp_cache_set( 'my_key', ['foo' => 'bar'] ), true ) );
-        WP_CLI::log( var_export( wp_cache_set( 'other_key', ['fuz' => 'biz'] ), true ) );
+        wp_cache_set( 'my_key', ['foo' => 'bar'] );
+        wp_cache_set( 'other_key', ['fuz' => 'biz'] );
 
         $complex_key = (object) [
             'foo' => (object) [

src/Cache/RecursiveDataStructureTraverser.php

@@ -92,6 +92,7 @@ public function delete( $key_path ) {
 	 * @param mixed $value
 	 */
 	public function insert( $key_path, $value ) {
+		\WP_CLI::log( var_export( [$key_path, $value], true ) );
 		try {
 			$this->update( $key_path, $value );
 		} catch ( NonExistentKeyException $exception ) {

src/Cache_Command.php

@@ -517,6 +517,8 @@ public function patch( $args, $assoc_args ) {
 		$group                = Utils\get_flag_value( $assoc_args, 'group' );
 		$expiration           = Utils\get_flag_value( $assoc_args, 'expiration' );
 
+		\WP_CLI::log( var_export( [ $action, $key ], true ) );
+
 		$key_path = array_map(
 			function ( $key ) {
 				if ( is_numeric( $key ) && ( (string) intval( $key ) === $key ) ) {
@@ -528,14 +530,20 @@ function ( $key ) {
 			array_slice( $args, 2 )
 		);
 
+		\WP_CLI::log( var_export( $key_path, true ) );
+
 		if ( 'delete' === $action ) {
 			$patch_value = null;
 		} elseif ( \WP_CLI\Entity\Utils::has_stdin() ) {
 			$stdin_value = WP_CLI::get_value_from_arg_or_stdin( $args, - 1 );
 			$patch_value = WP_CLI::read_value( trim( $stdin_value ), $assoc_args );
+			\WP_CLI::log( 'STDIN : ' . var_export( $key_path, true ) );
+			\WP_CLI::log( 'STDIN : ' . var_export( $patch_value, true ) );
 		} else {
 			// Take the patch value as the last positional argument. Mutates $key_path to be 1 element shorter!
 			$patch_value = WP_CLI::read_value( array_pop( $key_path ), $assoc_args );
+			\WP_CLI::log( 'NO STDIN : ' . var_export( $key_path, true ) );
+			\WP_CLI::log( 'NO STDIN : ' . var_export( $patch_value, true ) );
 		}
 
 		/* Need to make a copy of $current_value here as it is modified by reference */