Wordlift_Key_Validation_Service::validate_key() - Code Metrics - Inspection of "update release date" - insideout10/wordlift-plugin - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 159d3b...c30b21 )

by David

created 2016-12-29 07:54 UTC

Wordlift_Key_Validation_Service::validate_key() A

↳ Parent: Wordlift_Key_Validation_Service

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	5
nc	2
nop	0
dl	0
loc	14
rs	9.4285
c	0
b	0
f	0

<?php
/**
 * Wordlift_Key_Validation_Service class
 *
 * The {@link Wordlift_Key_Validation_Service} class provides WordLift's key validation services.
 *
 * @link    https://wordlift.io
 *
 * @package Wordlift
 * @since   3.9.0
 */

/**
 * Define the {@link Wordlift_Key_Validation_Service} class.
 *
 * @since 3.9.0
 */
class Wordlift_Key_Validation_Service {

	/**
	 * Validate the provided key.
	 *
	 * @since 3.9.0
	 *
	 * @param string $key WordLift's key to validate.
	 *
	 * @return bool True if the key is valid, otherwise false.
	 */
	public function is_valid( $key ) {

		// Request the dataset URI as a way to validate the key
		$response = wp_remote_get( wl_configuration_get_accounts_by_key_dataset_uri( $key ), unserialize( WL_REDLINK_API_HTTP_OPTIONS ) );

		// If the response is valid, the key is valid.
		return ! is_wp_error( $response ) && 200 === (int) $response['response']['code'];
	}

	/**
	 * This function is hooked to the `wl_validate_key` AJAX call.
	 *
	 * @since 3.9.0
	 */
	public function validate_key() {
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}

		// Ensure we don't have garbage before us.
		ob_clean();

		// Check if we have a key.
		if ( ! isset( $_POST['key'] ) ) {
			wp_send_json_error( 'The key parameter is required.' );
		}

		// Set a response with valid set to true or false according to the key validity.
		wp_send_json( array( 'valid' => $this->is_valid( $_POST['key'] ) ) );

	}

}


1			<?php
2			/**
3			* Wordlift_Key_Validation_Service class
4			*
5			* The {@link Wordlift_Key_Validation_Service} class provides WordLift's key validation services.
6			*
7			* @link https://wordlift.io
8			*
9			* @package Wordlift
10			* @since 3.9.0
11			*/
12
13			/**
14			* Define the {@link Wordlift_Key_Validation_Service} class.
15			*
16			* @since 3.9.0
17			*/
18			class Wordlift_Key_Validation_Service {
19
20			/**
21			* Validate the provided key.
22			*
23			* @since 3.9.0
24			*
25			* @param string $key WordLift's key to validate.
26			*
27			* @return bool True if the key is valid, otherwise false.
28			*/
29			public function is_valid( $key ) {
30
31			// Request the dataset URI as a way to validate the key
32			$response = wp_remote_get( wl_configuration_get_accounts_by_key_dataset_uri( $key ), unserialize( WL_REDLINK_API_HTTP_OPTIONS ) );
33
34			// If the response is valid, the key is valid.
35			return ! is_wp_error( $response ) && 200 === (int) $response['response']['code'];
36			}
37
38			/**
39			* This function is hooked to the `wl_validate_key` AJAX call.
40			*
41			* @since 3.9.0
42			*/
43			public function validate_key() {
			0 ignored issues – show Coding Style introduced 2016-12-21 14:15 UTC by Report Bug Copy Issue Report `validate_key` uses the super-global variable `$_POST` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
44
45			// Ensure we don't have garbage before us.
46			ob_clean();
47
48			// Check if we have a key.
49			if ( ! isset( $_POST['key'] ) ) {
50			wp_send_json_error( 'The key parameter is required.' );
51			}
52
53			// Set a response with valid set to true or false according to the key validity.
54			wp_send_json( array( 'valid' => $this->is_valid( $_POST['key'] ) ) );
55
56			}
57
58			}
59

insideout10 / wordlift-plugin

Push — master ( 159d3b...c30b21 )

Wordlift_Key_Validation_Service::validate_key() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like