Fix snapshot publish issue after settings save

PatelUtkarsh · PatelUtkarsh · commit ee517700356c · 2016-08-31T12:48:14.000+05:30
diff --git a/php/class-customize-snapshot-manager.php b/php/class-customize-snapshot-manager.php
@@ -794,13 +794,13 @@ function( $value ) {
 			},
 			$this->customize_manager->unsanitized_post_values()
 		);
-		$result = $this->snapshot->set( $settings_data );
+		$result = $this->snapshot->set( $settings_data, array( 'skip_validation' => true ) );
 		if ( ! empty( $result['errors'] ) ) {
 			add_filter( 'customize_save_response', function( $response ) use ( $result, $that ) {
 				$response['snapshot_errors'] = $that->prepare_errors_for_response( $result['errors'] );
 				return $response;
 			} );
-			return false;
+			return false; // Todo should remove?
 		}
 
 		if ( ! $this->snapshot->post() || 'publish' !== $this->snapshot->post()->post_status ) {
diff --git a/php/class-customize-snapshot.php b/php/class-customize-snapshot.php
@@ -141,6 +141,7 @@ public function status() {
 	 * @throws Exception When $settings_data is not an array of arrays.
 	 *
 	 * @param array $settings_data Settings data, mapping setting IDs to arrays containing `value` and optionally additional params.
+	 * @param array $args extra optional param to tweak.
 	 * @return array {
 	 *     Result.
 	 *
@@ -149,7 +150,7 @@ public function status() {
 	 *     @type array          $validities Setting validities.
 	 * }
 	 */
-	public function set( array $settings_data ) {
+	public function set( array $settings_data, array $args = array() ) {
 		$error = new \WP_Error();
 		$result = array(
 			'errors' => null,
@@ -188,26 +189,29 @@ public function set( array $settings_data ) {
 			)
 		);
 
-		// Validate.
-		if ( method_exists( $customize_manager, 'validate_setting_values' ) ) {
-			$result['validities'] = $customize_manager->validate_setting_values( $unsanitized_values );
-		} else {
-			// @codeCoverageIgnoreStart
-			$result['validities'] = array_map(
-				function( $sanitized ) {
-					if ( is_null( $sanitized ) ) {
-						return new \WP_Error( 'invalid_value', __( 'Invalid value', 'customize-snapshots' ) );
-					} else {
-						return true;
-					}
-				},
-				$unsanitized_values
-			);
-			// @codeCoverageIgnoreEnd
+		$invalid_setting_ids = array();
+		if ( empty( $args['skip_validation'] ) ) {
+			// Validate.
+			if ( method_exists( $customize_manager, 'validate_setting_values' ) ) {
+				$result['validities'] = $customize_manager->validate_setting_values( $unsanitized_values );
+			} else {
+				// @codeCoverageIgnoreStart
+				$result['validities'] = array_map(
+					function( $sanitized ) {
+						if ( is_null( $sanitized ) ) {
+							return new \WP_Error( 'invalid_value', __( 'Invalid value', 'customize-snapshots' ) );
+						} else {
+							return true;
+						}
+					},
+					$unsanitized_values
+				);
+				// @codeCoverageIgnoreEnd
+			}
+			$invalid_setting_ids = array_keys( array_filter( $result['validities'], function( $validity ) {
+				return is_wp_error( $validity );
+			} ) );
 		}
-		$invalid_setting_ids = array_keys( array_filter( $result['validities'], function( $validity ) {
-			return is_wp_error( $validity );
-		} ) );
 
 		// Sanitize.
 		foreach ( $unsanitized_values as $setting_id => $unsanitized_value ) {
