Wordlift_Sample_Data_Ajax_Adapter

Complexity

Total Complexity

3

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	create()	0	12	1
A	delete()	0	13	1

<?php
/**
 * Ajax Adapters: Sample Data Ajax Adapter.
 *
 * Provides end-points to call the {@link Wordlift_Sample_Data_Service} instance,
 * thus enabling loading sample data with a http call.
 *
 * @since   3.12.0
 * @package Wordlift
 */

/**
 * Define the {@link Wordlift_Sample_Data_Ajax_Adapter} class.
 *
 * @since   3.12.0
 * @package Wordlift
 */
class Wordlift_Sample_Data_Ajax_Adapter {

	/**
	 * The {@link Wordlift_Sample_Data_Service} instance.
	 *
	 * @since  3.12.0
	 * @access private
	 * @var \Wordlift_Sample_Data_Service $sample_data_service The {@link Wordlift_Sample_Data_Service} instance.
	 */
	private $sample_data_service;

	/**
	 * Create a {@link Wordlift_Sample_Data_Ajax_Adapter} instance.
	 *
	 * @since 3.12.0
	 *
	 * @param \Wordlift_Sample_Data_Service $sample_data_service The {@link Wordlift_Sample_Data_Service} instance.
	 */
	function __construct( $sample_data_service ) {

It is generally recommended to explicitly declare the visibility for methods.

		$this->sample_data_service = $sample_data_service;

	}

	/**
	 * Handle the `wl_sample_data_create` ajax action.
	 *
	 * @since 3.12.0
	 */
	function create() {

It is generally recommended to explicitly declare the visibility for methods.

		// Clean any potential garbage before us.
		ob_clean();

		// Create the sample data.
		$this->sample_data_service->create();

		// Send success.
		wp_send_json_success();

	}

	function delete() {

It is generally recommended to explicitly declare the visibility for methods.

		// Clean any potential garbage before us.
		ob_clean();

		// Create the sample data.
		$this->sample_data_service->delete();

		// Send success.
		@header( 'Content-Disposition: inline' );

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

		wp_send_json_success();

	}

}