insideout10 / wordlift-plugin

src/wordlift/configuration/class-config.php

Indentation +183 added lines, -183 removed lines

@@ -3,191 +3,191 @@
 namespace Wordlift\Configuration;
 
 class Config {
-	/**
-	 * @var \Wordlift_Admin_Setup
-	 */
-	private $admin_setup;
-	/**
-	 * @var \Wordlift_Key_Validation_Service
-	 */
-	private $key_validation_service;
-
-	/**
-	 * Config constructor.
-	 *
-	 * @param $admin_setup \Wordlift_Admin_Setup
-	 * @param $key_validation_service \Wordlift_Key_Validation_Service
-	 */
-	public function __construct( $admin_setup, $key_validation_service ) {
-
-		$this->admin_setup            = $admin_setup;
-		$this->key_validation_service = $key_validation_service;
-		add_action( 'wp_ajax_nopriv_wl_config_plugin', array( $this, 'config' ) );
-
-	}
-
-	/**
-	 * Check if the key is valid and also not bound to any domain.
-	 *
-	 * @param $key string
-	 *
-	 * @return bool
-	 */
-	private function is_key_valid_and_not_bound_to_any_domain( $key ) {
-		$account_info = $this->key_validation_service->get_account_info( $key );
-
-		/**
-		 * we need to check if the key is not associated with any account
-		 * before setting it, we should check if the url is null.
-		 */
-		if ( is_wp_error( $account_info )
-			 || wp_remote_retrieve_response_code( $account_info ) !== 200 ) {
-			return false;
-		}
-
-		$account_info_json = $account_info['body'];
-
-		$account_info_data = json_decode( $account_info_json, true );
-
-		if ( ! $account_info_data ) {
-			// Invalid json returned by api.
-			return false;
-		}
-
-		$site_url = apply_filters( 'wl_production_site_url', untrailingslashit( get_option( 'home' ) ) );
-
-		if ( null === $account_info_data['url'] ) {
-			return true;
-		}
-
-		// Check if the key belongs to same site.
-		if ( untrailingslashit( $account_info_data['url'] ) !== $site_url ) {
-			// key already associated with another account.
-			return false;
-		}
-
-		// Return true if the key domain and site domain are the same.
-		return true;
-	}
-
-	public function config() {
-
-		// Perform validation check for all the parameters.
-		$required_fields = array(
-			'diagnostic',
-			'vocabulary',
-			// Don't ask for language from webapp.
-			// 'language',
-				'country',
-			'publisherName',
-			'publisher',
-			'license',
-		);
-
-		header( 'Access-Control-Allow-Origin: *' );
-
-		// validate all the fields before processing
-		foreach ( $required_fields as $field ) {
-			if ( ! array_key_exists( $field, $_POST ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
-				/* translators: %s: field name */
-				wp_send_json_error( sprintf( __( 'Field %s is required', 'wordlift' ), $field ), 422 );
-
-				return;
-			}
-		}
-
-		$key = isset( $_POST['license'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['license'] ) ) : ''; // phpcs:ignore WordPress.Security.NonceVerification.Missing
-
-		if ( ! $this->is_key_valid_and_not_bound_to_any_domain( $key ) ) {
-			wp_send_json_error( __( 'Key is not valid or associated with other domain', 'wordlift' ), 403 );
-
-			// exit if not valid.
-			return;
-		}
-
-		// check if key is already configured, if yes then dont save settings.
-		if ( \Wordlift_Configuration_Service::get_instance()->get_key() ) {
-			wp_send_json_error( __( 'Key already configured.', 'wordlift' ), 403 );
-
-			// key already configured
-			return;
-		}
-
-		$this->admin_setup->save_configuration( $this->get_params() );
-
-		// This prevents invalid key issue with automatic installation in sites which have redis cache enabled.
-		wp_cache_flush();
-
-		wp_send_json_success( __( 'Configuration Saved', 'wordlift' ) );
-	}
-
-	/**
-	 *
-	 * @return array
-	 */
-	private function get_params() {
-
-		$attachment_id = $this->may_be_get_attachment_id();
-
-		$params = array(
-			'key'             => isset( $_POST['license'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['license'] ) ) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
-			'vocabulary'      => isset( $_POST['vocabulary'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['vocabulary'] ) ) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
-			'wl-country-code' => isset( $_POST['country'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['country'] ) ) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
-			'name'            => isset( $_POST['publisherName'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['publisherName'] ) ) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
-			'user_type'       => isset( $_POST['publisher'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['publisher'] ) ) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
-			'logo'            => $attachment_id,
-		);
-
-		$diagnostic = isset( $_POST['diagnostic'] ) ? (bool) $_POST['diagnostic'] : false; // phpcs:ignore WordPress.Security.NonceVerification.Missing
-		if ( $diagnostic ) {
-			$params['share-diagnostic'] = 'on';
-		}
+    /**
+     * @var \Wordlift_Admin_Setup
+     */
+    private $admin_setup;
+    /**
+     * @var \Wordlift_Key_Validation_Service
+     */
+    private $key_validation_service;
+
+    /**
+     * Config constructor.
+     *
+     * @param $admin_setup \Wordlift_Admin_Setup
+     * @param $key_validation_service \Wordlift_Key_Validation_Service
+     */
+    public function __construct( $admin_setup, $key_validation_service ) {
+
+        $this->admin_setup            = $admin_setup;
+        $this->key_validation_service = $key_validation_service;
+        add_action( 'wp_ajax_nopriv_wl_config_plugin', array( $this, 'config' ) );
+
+    }
+
+    /**
+     * Check if the key is valid and also not bound to any domain.
+     *
+     * @param $key string
+     *
+     * @return bool
+     */
+    private function is_key_valid_and_not_bound_to_any_domain( $key ) {
+        $account_info = $this->key_validation_service->get_account_info( $key );
+
+        /**
+         * we need to check if the key is not associated with any account
+         * before setting it, we should check if the url is null.
+         */
+        if ( is_wp_error( $account_info )
+             || wp_remote_retrieve_response_code( $account_info ) !== 200 ) {
+            return false;
+        }
+
+        $account_info_json = $account_info['body'];
+
+        $account_info_data = json_decode( $account_info_json, true );
+
+        if ( ! $account_info_data ) {
+            // Invalid json returned by api.
+            return false;
+        }
+
+        $site_url = apply_filters( 'wl_production_site_url', untrailingslashit( get_option( 'home' ) ) );
+
+        if ( null === $account_info_data['url'] ) {
+            return true;
+        }
+
+        // Check if the key belongs to same site.
+        if ( untrailingslashit( $account_info_data['url'] ) !== $site_url ) {
+            // key already associated with another account.
+            return false;
+        }
+
+        // Return true if the key domain and site domain are the same.
+        return true;
+    }
+
+    public function config() {
+
+        // Perform validation check for all the parameters.
+        $required_fields = array(
+            'diagnostic',
+            'vocabulary',
+            // Don't ask for language from webapp.
+            // 'language',
+                'country',
+            'publisherName',
+            'publisher',
+            'license',
+        );
+
+        header( 'Access-Control-Allow-Origin: *' );
+
+        // validate all the fields before processing
+        foreach ( $required_fields as $field ) {
+            if ( ! array_key_exists( $field, $_POST ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
+                /* translators: %s: field name */
+                wp_send_json_error( sprintf( __( 'Field %s is required', 'wordlift' ), $field ), 422 );
+
+                return;
+            }
+        }
+
+        $key = isset( $_POST['license'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['license'] ) ) : ''; // phpcs:ignore WordPress.Security.NonceVerification.Missing
+
+        if ( ! $this->is_key_valid_and_not_bound_to_any_domain( $key ) ) {
+            wp_send_json_error( __( 'Key is not valid or associated with other domain', 'wordlift' ), 403 );
+
+            // exit if not valid.
+            return;
+        }
+
+        // check if key is already configured, if yes then dont save settings.
+        if ( \Wordlift_Configuration_Service::get_instance()->get_key() ) {
+            wp_send_json_error( __( 'Key already configured.', 'wordlift' ), 403 );
+
+            // key already configured
+            return;
+        }
+
+        $this->admin_setup->save_configuration( $this->get_params() );
+
+        // This prevents invalid key issue with automatic installation in sites which have redis cache enabled.
+        wp_cache_flush();
+
+        wp_send_json_success( __( 'Configuration Saved', 'wordlift' ) );
+    }
+
+    /**
+     *
+     * @return array
+     */
+    private function get_params() {
+
+        $attachment_id = $this->may_be_get_attachment_id();
+
+        $params = array(
+            'key'             => isset( $_POST['license'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['license'] ) ) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
+            'vocabulary'      => isset( $_POST['vocabulary'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['vocabulary'] ) ) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
+            'wl-country-code' => isset( $_POST['country'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['country'] ) ) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
+            'name'            => isset( $_POST['publisherName'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['publisherName'] ) ) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
+            'user_type'       => isset( $_POST['publisher'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['publisher'] ) ) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
+            'logo'            => $attachment_id,
+        );
+
+        $diagnostic = isset( $_POST['diagnostic'] ) ? (bool) $_POST['diagnostic'] : false; // phpcs:ignore WordPress.Security.NonceVerification.Missing
+        if ( $diagnostic ) {
+            $params['share-diagnostic'] = 'on';
+        }
 
-		return $params;
-	}
-
-	/**
-	 * @return int | bool
-	 */
-	private function may_be_get_attachment_id() {
-
-		// if image or image extension not posted then return false.
-		if ( ! isset( $_POST['image'] ) || ! isset( $_POST['imageExtension'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
-			return false;
-		}
-
-		$allowed_extensions = array( 'png', 'jpeg', 'jpg' );
-		$image_string       = sanitize_text_field( wp_unslash( (string) $_POST['image'] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Missing
-		$image_ext          = sanitize_text_field( wp_unslash( (string) $_POST['imageExtension'] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Missing
-
-		if ( ! in_array( $image_ext, $allowed_extensions, true ) ) {
-			return false;
-		}
-
-		// phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.obfuscation_base64_decode
-		$image_decoded_string = base64_decode( $image_string );
-
-		$upload_dir = wp_upload_dir();
-
-		$file_path = $upload_dir['path'] . DIRECTORY_SEPARATOR . md5( $image_string ) . '.' . $image_ext;
-
-		// phpcs:ignore WordPress.WP.AlternativeFunctions.file_system_read_file_put_contents
-		file_put_contents( $file_path, $image_decoded_string );
-
-		$attachment_id = wp_insert_attachment(
-			array(
-				'post_status'    => 'inherit',
-				'post_mime_type' => "image/$image_ext",
-			),
-			$file_path
-		);
+        return $params;
+    }
+
+    /**
+     * @return int | bool
+     */
+    private function may_be_get_attachment_id() {
+
+        // if image or image extension not posted then return false.
+        if ( ! isset( $_POST['image'] ) || ! isset( $_POST['imageExtension'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
+            return false;
+        }
+
+        $allowed_extensions = array( 'png', 'jpeg', 'jpg' );
+        $image_string       = sanitize_text_field( wp_unslash( (string) $_POST['image'] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Missing
+        $image_ext          = sanitize_text_field( wp_unslash( (string) $_POST['imageExtension'] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Missing
+
+        if ( ! in_array( $image_ext, $allowed_extensions, true ) ) {
+            return false;
+        }
+
+        // phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.obfuscation_base64_decode
+        $image_decoded_string = base64_decode( $image_string );
+
+        $upload_dir = wp_upload_dir();
+
+        $file_path = $upload_dir['path'] . DIRECTORY_SEPARATOR . md5( $image_string ) . '.' . $image_ext;
+
+        // phpcs:ignore WordPress.WP.AlternativeFunctions.file_system_read_file_put_contents
+        file_put_contents( $file_path, $image_decoded_string );
+
+        $attachment_id = wp_insert_attachment(
+            array(
+                'post_status'    => 'inherit',
+                'post_mime_type' => "image/$image_ext",
+            ),
+            $file_path
+        );
 
-		// Generate the metadata for the attachment, and update the database record.
-		$attachment_data = wp_generate_attachment_metadata( $attachment_id, $file_path );
-		// Update the attachment metadata.
-		wp_update_attachment_metadata( $attachment_id, $attachment_data );
+        // Generate the metadata for the attachment, and update the database record.
+        $attachment_data = wp_generate_attachment_metadata( $attachment_id, $file_path );
+        // Update the attachment metadata.
+        wp_update_attachment_metadata( $attachment_id, $attachment_data );
 
-		return $attachment_id;
-	}
+        return $attachment_id;
+    }
 
 }

Spacing +39 added lines, -39 removed lines

@@ -18,11 +18,11 @@ 
 	 * @param $admin_setup \Wordlift_Admin_Setup
 	 * @param $key_validation_service \Wordlift_Key_Validation_Service
 	 */
-	public function __construct( $admin_setup, $key_validation_service ) {
+	public function __construct($admin_setup, $key_validation_service) {
 
 		$this->admin_setup            = $admin_setup;
 		$this->key_validation_service = $key_validation_service;
-		add_action( 'wp_ajax_nopriv_wl_config_plugin', array( $this, 'config' ) );
+		add_action('wp_ajax_nopriv_wl_config_plugin', array($this, 'config'));
 
 	}
 
@@ -33,35 +33,35 @@ 
 	 *
 	 * @return bool
 	 */
-	private function is_key_valid_and_not_bound_to_any_domain( $key ) {
-		$account_info = $this->key_validation_service->get_account_info( $key );
+	private function is_key_valid_and_not_bound_to_any_domain($key) {
+		$account_info = $this->key_validation_service->get_account_info($key);
 
 		/**
 		 * we need to check if the key is not associated with any account
 		 * before setting it, we should check if the url is null.
 		 */
-		if ( is_wp_error( $account_info )
-			 || wp_remote_retrieve_response_code( $account_info ) !== 200 ) {
+		if (is_wp_error($account_info)
+			 || wp_remote_retrieve_response_code($account_info) !== 200) {
 			return false;
 		}
 
 		$account_info_json = $account_info['body'];
 
-		$account_info_data = json_decode( $account_info_json, true );
+		$account_info_data = json_decode($account_info_json, true);
 
-		if ( ! $account_info_data ) {
+		if ( ! $account_info_data) {
 			// Invalid json returned by api.
 			return false;
 		}
 
-		$site_url = apply_filters( 'wl_production_site_url', untrailingslashit( get_option( 'home' ) ) );
+		$site_url = apply_filters('wl_production_site_url', untrailingslashit(get_option('home')));
 
-		if ( null === $account_info_data['url'] ) {
+		if (null === $account_info_data['url']) {
 			return true;
 		}
 
 		// Check if the key belongs to same site.
-		if ( untrailingslashit( $account_info_data['url'] ) !== $site_url ) {
+		if (untrailingslashit($account_info_data['url']) !== $site_url) {
 			// key already associated with another account.
 			return false;
 		}
@@ -84,41 +84,41 @@ 
 			'license',
 		);
 
-		header( 'Access-Control-Allow-Origin: *' );
+		header('Access-Control-Allow-Origin: *');
 
 		// validate all the fields before processing
-		foreach ( $required_fields as $field ) {
-			if ( ! array_key_exists( $field, $_POST ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
+		foreach ($required_fields as $field) {
+			if ( ! array_key_exists($field, $_POST)) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
 				/* translators: %s: field name */
-				wp_send_json_error( sprintf( __( 'Field %s is required', 'wordlift' ), $field ), 422 );
+				wp_send_json_error(sprintf(__('Field %s is required', 'wordlift'), $field), 422);
 
 				return;
 			}
 		}
 
-		$key = isset( $_POST['license'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['license'] ) ) : ''; // phpcs:ignore WordPress.Security.NonceVerification.Missing
+		$key = isset($_POST['license']) ? sanitize_text_field(wp_unslash((string) $_POST['license'])) : ''; // phpcs:ignore WordPress.Security.NonceVerification.Missing
 
-		if ( ! $this->is_key_valid_and_not_bound_to_any_domain( $key ) ) {
-			wp_send_json_error( __( 'Key is not valid or associated with other domain', 'wordlift' ), 403 );
+		if ( ! $this->is_key_valid_and_not_bound_to_any_domain($key)) {
+			wp_send_json_error(__('Key is not valid or associated with other domain', 'wordlift'), 403);
 
 			// exit if not valid.
 			return;
 		}
 
 		// check if key is already configured, if yes then dont save settings.
-		if ( \Wordlift_Configuration_Service::get_instance()->get_key() ) {
-			wp_send_json_error( __( 'Key already configured.', 'wordlift' ), 403 );
+		if (\Wordlift_Configuration_Service::get_instance()->get_key()) {
+			wp_send_json_error(__('Key already configured.', 'wordlift'), 403);
 
 			// key already configured
 			return;
 		}
 
-		$this->admin_setup->save_configuration( $this->get_params() );
+		$this->admin_setup->save_configuration($this->get_params());
 
 		// This prevents invalid key issue with automatic installation in sites which have redis cache enabled.
 		wp_cache_flush();
 
-		wp_send_json_success( __( 'Configuration Saved', 'wordlift' ) );
+		wp_send_json_success(__('Configuration Saved', 'wordlift'));
 	}
 
 	/**
@@ -130,16 +130,16 @@ 
 		$attachment_id = $this->may_be_get_attachment_id();
 
 		$params = array(
-			'key'             => isset( $_POST['license'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['license'] ) ) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
-			'vocabulary'      => isset( $_POST['vocabulary'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['vocabulary'] ) ) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
-			'wl-country-code' => isset( $_POST['country'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['country'] ) ) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
-			'name'            => isset( $_POST['publisherName'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['publisherName'] ) ) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
-			'user_type'       => isset( $_POST['publisher'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['publisher'] ) ) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
+			'key'             => isset($_POST['license']) ? sanitize_text_field(wp_unslash((string) $_POST['license'])) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
+			'vocabulary'      => isset($_POST['vocabulary']) ? sanitize_text_field(wp_unslash((string) $_POST['vocabulary'])) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
+			'wl-country-code' => isset($_POST['country']) ? sanitize_text_field(wp_unslash((string) $_POST['country'])) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
+			'name'            => isset($_POST['publisherName']) ? sanitize_text_field(wp_unslash((string) $_POST['publisherName'])) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
+			'user_type'       => isset($_POST['publisher']) ? sanitize_text_field(wp_unslash((string) $_POST['publisher'])) : '', // phpcs:ignore WordPress.Security.NonceVerification.Missing
 			'logo'            => $attachment_id,
 		);
 
-		$diagnostic = isset( $_POST['diagnostic'] ) ? (bool) $_POST['diagnostic'] : false; // phpcs:ignore WordPress.Security.NonceVerification.Missing
-		if ( $diagnostic ) {
+		$diagnostic = isset($_POST['diagnostic']) ? (bool) $_POST['diagnostic'] : false; // phpcs:ignore WordPress.Security.NonceVerification.Missing
+		if ($diagnostic) {
 			$params['share-diagnostic'] = 'on';
 		}
 
@@ -152,27 +152,27 @@ 
 	private function may_be_get_attachment_id() {
 
 		// if image or image extension not posted then return false.
-		if ( ! isset( $_POST['image'] ) || ! isset( $_POST['imageExtension'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
+		if ( ! isset($_POST['image']) || ! isset($_POST['imageExtension'])) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
 			return false;
 		}
 
-		$allowed_extensions = array( 'png', 'jpeg', 'jpg' );
-		$image_string       = sanitize_text_field( wp_unslash( (string) $_POST['image'] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Missing
-		$image_ext          = sanitize_text_field( wp_unslash( (string) $_POST['imageExtension'] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Missing
+		$allowed_extensions = array('png', 'jpeg', 'jpg');
+		$image_string       = sanitize_text_field(wp_unslash((string) $_POST['image'])); // phpcs:ignore WordPress.Security.NonceVerification.Missing
+		$image_ext          = sanitize_text_field(wp_unslash((string) $_POST['imageExtension'])); // phpcs:ignore WordPress.Security.NonceVerification.Missing
 
-		if ( ! in_array( $image_ext, $allowed_extensions, true ) ) {
+		if ( ! in_array($image_ext, $allowed_extensions, true)) {
 			return false;
 		}
 
 		// phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.obfuscation_base64_decode
-		$image_decoded_string = base64_decode( $image_string );
+		$image_decoded_string = base64_decode($image_string);
 
 		$upload_dir = wp_upload_dir();
 
-		$file_path = $upload_dir['path'] . DIRECTORY_SEPARATOR . md5( $image_string ) . '.' . $image_ext;
+		$file_path = $upload_dir['path'].DIRECTORY_SEPARATOR.md5($image_string).'.'.$image_ext;
 
 		// phpcs:ignore WordPress.WP.AlternativeFunctions.file_system_read_file_put_contents
-		file_put_contents( $file_path, $image_decoded_string );
+		file_put_contents($file_path, $image_decoded_string);
 
 		$attachment_id = wp_insert_attachment(
 			array(
@@ -183,9 +183,9 @@ 
 		);
 
 		// Generate the metadata for the attachment, and update the database record.
-		$attachment_data = wp_generate_attachment_metadata( $attachment_id, $file_path );
+		$attachment_data = wp_generate_attachment_metadata($attachment_id, $file_path);
 		// Update the attachment metadata.
-		wp_update_attachment_metadata( $attachment_id, $attachment_data );
+		wp_update_attachment_metadata($attachment_id, $attachment_data);
 
 		return $attachment_id;
 	}