Properly clone objects for comparison

Otherwise, they're handled by reference and transformed on update

Author: danielbachhuber

src/Option_Command.php

@@ -543,6 +543,9 @@ public function patch( $args, $assoc_args ) {
 
 		/* Need to make a copy of $current_value here as it is modified by reference */
 		$old_value = $current_value = sanitize_option( $key, get_option( $key ) );
+		if ( is_object( $current_value ) ) {
+			$old_value = clone $current_value;
+		}
 
 		$traverser = new RecursiveDataStructureTraverser( $current_value );
 

src/Site_Option_Command.php

@@ -372,6 +372,9 @@ public function patch( $args, $assoc_args ) {
 
 		/* Need to make a copy of $current_value here as it is modified by reference */
 		$old_value = $current_value = sanitize_option( $key, get_site_option( $key ) );
+		if ( is_object( $current_value ) ) {
+			$old_value = clone $current_value;
+		}
 
 		$traverser = new RecursiveDataStructureTraverser( $current_value );
 

src/WP_CLI/CommandWithMeta.php

@@ -390,6 +390,9 @@ public function patch( $args, $assoc_args ) {
 
 		/* Need to make a copy of $current_meta_value here as it is modified by reference */
 		$current_meta_value = $old_meta_value = sanitize_meta( $meta_key, get_metadata( $this->meta_type, $object_id, $meta_key, true ), $this->meta_type );
+		if ( is_object( $current_meta_value ) ) {
+			$old_meta_value = clone $current_meta_value;
+		}
 
 		$traverser = new RecursiveDataStructureTraverser( $current_meta_value );