Pink-Crab / Perique-Ajax

src/Dispatcher/Ajax_Request_Validator.php

Spacing +11 added lines, -11 removed lines

@@ -34,7 +34,7 @@ 
 	/** @var ServerRequestInterface */
 	protected $server_request;
 
-	public function __construct( ServerRequestInterface $server_request ) {
+	public function __construct(ServerRequestInterface $server_request) {
 		$this->server_request = $server_request;
 	}
 
@@ -44,22 +44,22 @@ 
 	 * @param \PinkCrab\Ajax\Ajax $ajax
 	 * @return bool
 	 */
-	public function validate( Ajax $ajax ): bool {
-		if ( ! $ajax->has_nonce() ) {
+	public function validate(Ajax $ajax): bool {
+		if ( ! $ajax->has_nonce()) {
 			return true;
 		}
 
 		// Find nonce value in request
-		$nonce_value = $this->find_nonce( $ajax->get_nonce_field() );
+		$nonce_value = $this->find_nonce($ajax->get_nonce_field());
 
 		// If no nonce value found in request.
-		if ( is_null( $nonce_value ) ) {
+		if (is_null($nonce_value)) {
 			return false;
 		}
 
 		/* @phpstan-ignore-next-line, nonce handle checked at start of method*/
-		return ( new Nonce( $ajax->get_nonce_handle() ) )
-			->validate( $nonce_value );
+		return (new Nonce($ajax->get_nonce_handle()))
+			->validate($nonce_value);
 	}
 
 	/**
@@ -68,11 +68,11 @@ 
 	 * @param string $nonce_field
 	 * @return string|null
 	 */
-	protected function find_nonce( string $nonce_field ): ?string {
-		$args = Ajax_Helper::extract_server_request_args( $this->server_request );
+	protected function find_nonce(string $nonce_field): ?string {
+		$args = Ajax_Helper::extract_server_request_args($this->server_request);
 
-		return \array_key_exists( $nonce_field, $args )
-			? $args[ $nonce_field ]
+		return \array_key_exists($nonce_field, $args)
+			? $args[$nonce_field]
 			: null;
 	}
 

src/Dispatcher/Ajax_Dispatcher.php

Spacing +5 added lines, -5 removed lines

@@ -37,7 +37,7 @@ 
 	/** @var Ajax_Controller */
 	protected $ajax_controller;
 
-	public function __construct( Ajax_Controller $ajax_controller ) {
+	public function __construct(Ajax_Controller $ajax_controller) {
 		$this->loader          = new Hook_Loader();
 		$this->ajax_controller = $ajax_controller;
 	}
@@ -49,15 +49,15 @@ 
 	 * @return void
 	 * @throws Ajax_Exception (code 101) If no action defined
 	 */
-	public function add_ajax_call( Ajax $ajax ): void {
+	public function add_ajax_call(Ajax $ajax): void {
 
-		if ( ! $ajax->has_valid_action() ) {
-			throw Ajax_Exception::undefined_action( \get_class( $ajax ) );
+		if ( ! $ajax->has_valid_action()) {
+			throw Ajax_Exception::undefined_action(\get_class($ajax));
 		}
 
 		$this->loader->ajax(
 			$ajax->get_action(), /* @phpstan-ignore-line, action existance checked above */
-			$this->ajax_controller->create_callback( $ajax ),
+			$this->ajax_controller->create_callback($ajax),
 			$ajax->get_logged_out(),
 			$ajax->get_logged_in()
 		);

src/Dispatcher/Response_Factory.php

Spacing +13 added lines, -13 removed lines

@@ -35,7 +35,7 @@ 
 	/** @var HTTP */
 	protected $http;
 
-	public function __construct( HTTP $http ) {
+	public function __construct(HTTP $http) {
 		$this->http = $http;
 	}
 
@@ -45,11 +45,11 @@ 
 	 * @param int $code The HTTP status code. Defaults to 200.
 	 * @param string $reasonPhrase
 	 */
-	public function createResponse( int $code = 200, string $reasonPhrase = '' ): ResponseInterface {
+	public function createResponse(int $code = 200, string $reasonPhrase = ''): ResponseInterface {
 		return $this->http->psr7_response()
-			->withStatus( $code )
-			->withHeader( 'Content-Type', 'application/json; charset=' . get_option( 'blog_charset' ) )
-			->withBody( Stream::create( $reasonPhrase ) );
+			->withStatus($code)
+			->withHeader('Content-Type', 'application/json; charset=' . get_option('blog_charset'))
+			->withBody(Stream::create($reasonPhrase));
 	}
 
 	/**
@@ -58,8 +58,8 @@ 
 	 * @param array<mixed> $payload
 	 * @return ResponseInterface
 	 */
-	public function success( array $payload = array() ): ResponseInterface {
-		return $this->createResponse( 200, json_encode( $payload ) ?: '' ); // phpcs:ignore WordPress.WP.AlternativeFunctions.json_encode_json_encode
+	public function success(array $payload = array()): ResponseInterface {
+		return $this->createResponse(200, json_encode($payload) ?: ''); // phpcs:ignore WordPress.WP.AlternativeFunctions.json_encode_json_encode
 	}
 
 	/**
@@ -69,8 +69,8 @@ 
 	 * @param array<mixed> $payload
 	 * @return ResponseInterface
 	 */
-	public function unauthorised( array $payload = array( 'error' => 'unauthorised' ) ): ResponseInterface {
-		return $this->createResponse( 401, json_encode( $payload ) ?: '' ); // phpcs:ignore WordPress.WP.AlternativeFunctions.json_encode_json_encode
+	public function unauthorised(array $payload = array('error' => 'unauthorised')): ResponseInterface {
+		return $this->createResponse(401, json_encode($payload) ?: ''); // phpcs:ignore WordPress.WP.AlternativeFunctions.json_encode_json_encode
 	}
 
 	/**
@@ -80,8 +80,8 @@ 
 	 * @param array<mixed> $payload
 	 * @return ResponseInterface
 	 */
-	public function failure( array $payload = array( 'error' => 'error' ) ): ResponseInterface {
-		return $this->createResponse( 500, json_encode( $payload ) ?: '' ); // phpcs:ignore WordPress.WP.AlternativeFunctions.json_encode_json_encode
+	public function failure(array $payload = array('error' => 'error')): ResponseInterface {
+		return $this->createResponse(500, json_encode($payload) ?: ''); // phpcs:ignore WordPress.WP.AlternativeFunctions.json_encode_json_encode
 	}
 
 	/**
@@ -91,7 +91,7 @@ 
 	 * @param array<mixed> $payload
 	 * @return ResponseInterface
 	 */
-	public function not_found( array $payload = array( 'error' => 'not found' ) ): ResponseInterface {
-		return $this->createResponse( 404, json_encode( $payload ) ?: '' ); // phpcs:ignore WordPress.WP.AlternativeFunctions.json_encode_json_encode
+	public function not_found(array $payload = array('error' => 'not found')): ResponseInterface {
+		return $this->createResponse(404, json_encode($payload) ?: ''); // phpcs:ignore WordPress.WP.AlternativeFunctions.json_encode_json_encode
 	}
 }

src/Ajax_Bootstrap.php

Spacing +4 added lines, -4 removed lines

@@ -41,18 +41,18 @@
 	public static function use(): void {
 		add_filter(
 			Hooks::APP_INIT_SET_DI_RULES,
-			function( array $rules ): array {
+			function(array $rules): array {
 
 				// Ensure the global rules exist.
-				if ( ! \array_key_exists( '*', $rules ) ) {
+				if ( ! \array_key_exists('*', $rules)) {
 					$rules['*'] = array();
 				}
-				if ( ! \array_key_exists( 'substitutions', $rules['*'] ) ) {
+				if ( ! \array_key_exists('substitutions', $rules['*'])) {
 					$rules['*']['substitutions'] = array();
 				}
 
 				// Set the global rule for ServerRequestInterface
-				$rules['*']['substitutions'][ ServerRequestInterface::class ]
+				$rules['*']['substitutions'][ServerRequestInterface::class]
 					= HTTP_Helper::global_server_request();
 
 				return $rules;

src/Ajax.php

Spacing +3 added lines, -3 removed lines

@@ -132,8 +132,8 @@ 
 	 * @return bool
 	 */
 	public function has_valid_action(): bool {
-		return is_string( $this->get_action() )
-		&& \mb_strlen( $this->get_action() ) > 0;
+		return is_string($this->get_action())
+		&& \mb_strlen($this->get_action()) > 0;
 	}
 
 	/**
@@ -142,6 +142,6 @@ 
 	 * @return bool
 	 */
 	public function has_nonce(): bool {
-		return ! is_null( $this->get_nonce_handle() );
+		return ! is_null($this->get_nonce_handle());
 	}
 }

src/Ajax_Helper.php

Spacing +23 added lines, -23 removed lines

@@ -49,7 +49,7 @@ 
 	 * @return string
 	 */
 	public static function admin_ajax_url(): string {
-		return admin_url( 'admin-ajax.php' );
+		return admin_url('admin-ajax.php');
 	}
 
 	/**
@@ -60,17 +60,17 @@ 
 	 * @return Ajax
 	 * @throws Ajax_Exception (code 100) If non valid Ajax class passed.
 	 */
-	private static function get_reflected( string $class ): Ajax {
-		if ( ! \is_subclass_of( $class, Ajax::class ) ) {
-			throw Ajax_Exception::non_ajax_model( 'get reflection' );
+	private static function get_reflected(string $class): Ajax {
+		if ( ! \is_subclass_of($class, Ajax::class)) {
+			throw Ajax_Exception::non_ajax_model('get reflection');
 		}
 
-		if ( ! array_key_exists( $class, self::$class_cache ) ) {
-			$reflection                  = new ReflectionClass( $class );
-			self::$class_cache[ $class ] = $reflection->newInstanceWithoutConstructor();
+		if ( ! array_key_exists($class, self::$class_cache)) {
+			$reflection                  = new ReflectionClass($class);
+			self::$class_cache[$class] = $reflection->newInstanceWithoutConstructor();
 		}
 
-		return self::$class_cache[ $class ];
+		return self::$class_cache[$class];
 	}
 
 	/**
@@ -82,11 +82,11 @@ 
 	 * @throws Ajax_Exception (code 100) If non valid Ajax class passed.
 	 * @throws Ajax_Exception (code 101) If no action defined
 	 */
-	public static function get_action( string $class ):? string {
-		$instance = self::get_reflected( $class );
+	public static function get_action(string $class): ? string {
+		$instance = self::get_reflected($class);
 
-		if ( ! $instance->has_valid_action() ) {
-			throw Ajax_Exception::undefined_action( $class );
+		if ( ! $instance->has_valid_action()) {
+			throw Ajax_Exception::undefined_action($class);
 		}
 
 		return $instance->get_action();
@@ -99,8 +99,8 @@ 
 	 * @return boolean
 	 * @throws Ajax_Exception (code 100) If non valid Ajax class passed.
 	 */
-	public static function has_nonce( string $class ): bool {
-		return self::get_reflected( $class )->has_nonce();
+	public static function has_nonce(string $class): bool {
+		return self::get_reflected($class)->has_nonce();
 	}
 
 	/**
@@ -110,11 +110,11 @@ 
 	 * @return Nonce|null
 	 * @throws Ajax_Exception (code 100) If non valid Ajax class passed.
 	 */
-	public static function get_nonce( string $class ): ?Nonce {
-		$instance = self::get_reflected( $class );
+	public static function get_nonce(string $class): ?Nonce {
+		$instance = self::get_reflected($class);
 
 		return $instance->has_nonce()
-			? new Nonce( $instance->get_nonce_handle() ?? '' ) // has_nonce conditional should catch null here
+			? new Nonce($instance->get_nonce_handle() ?? '') // has_nonce conditional should catch null here
 			: null;
 	}
 
@@ -125,8 +125,8 @@ 
 	 * @return string
 	 * @throws Ajax_Exception (code 100) If non valid Ajax class passed.
 	 */
-	public static function get_nonce_field( string $class ): string {
-		return self::get_reflected( $class )->get_nonce_field();
+	public static function get_nonce_field(string $class): string {
+		return self::get_reflected($class)->get_nonce_field();
 	}
 
 	/**
@@ -136,14 +136,14 @@ 
 	 * @param ServerRequestInterface $request
 	 * @return array<string, string>
 	 */
-	public static function extract_server_request_args( ServerRequestInterface $request ): array {
-		switch ( $request->getMethod() ) {
+	public static function extract_server_request_args(ServerRequestInterface $request): array {
+		switch ($request->getMethod()) {
 			case 'POST':
 				// Return different post types.
-				if ( str_contains( $request->getHeaderLine( 'Content-Type' ), 'application/x-www-form-urlencoded;' ) ) {
+				if (str_contains($request->getHeaderLine('Content-Type'), 'application/x-www-form-urlencoded;')) {
 					$params = (array) $request->getParsedBody();
 				} else {
-					$params = json_decode( (string) $request->getBody(), true ) ?? array();
+					$params = json_decode((string) $request->getBody(), true) ?? array();
 				}
 				break;
 			case 'GET':

src/Ajax_Exception.php

Spacing +4 added lines, -4 removed lines

@@ -34,9 +34,9 @@ 
 	 * @param string $operation The operation being carries out.
 	 * @return Ajax_Exception
 	 */
-	public static function non_ajax_model( string $operation = 'unknown operation' ): Ajax_Exception {
+	public static function non_ajax_model(string $operation = 'unknown operation'): Ajax_Exception {
 		$message = 'None Ajax Model passed to ' . $operation;
-		return new Ajax_Exception( $message, 100 );
+		return new Ajax_Exception($message, 100);
 	}
 
 	/**
@@ -45,9 +45,9 @@ 
 	 * @param string $class Ajax class
 	 * @return Ajax_Exception
 	 */
-	public static function undefined_action( string $class ): Ajax_Exception {
+	public static function undefined_action(string $class): Ajax_Exception {
 		$message = "{$class} has no defined action property";
-		return new Ajax_Exception( $message, 101 );
+		return new Ajax_Exception($message, 101);
 	}
 
 }

src/Dispatcher/Ajax_Controller.php

Spacing +12 added lines, -12 removed lines

@@ -64,8 +64,8 @@ 
 	 * @param \PinkCrab\Ajax\Ajax $ajax_class
 	 * @return bool
 	 */
-	public function validate_request( Ajax $ajax_class ): bool {
-		return $this->request_validator->validate( $ajax_class );
+	public function validate_request(Ajax $ajax_class): bool {
+		return $this->request_validator->validate($ajax_class);
 	}
 
 	/**
@@ -75,9 +75,9 @@ 
 	 * @return \Psr\Http\Message\ResponseInterface
 	 * @filter Ajax_Hooks::CALLBACK_REQUEST_FILTER
 	 */
-	public function invoke_callback( Ajax $ajax_class ): ResponseInterface {
+	public function invoke_callback(Ajax $ajax_class): ResponseInterface {
 		return $ajax_class->callback(
-			\apply_filters( Ajax_Hooks::CALLBACK_REQUEST_FILTER, $this->server_request, $ajax_class ),
+			\apply_filters(Ajax_Hooks::CALLBACK_REQUEST_FILTER, $this->server_request, $ajax_class),
 			$this->response_factory
 		);
 	}
@@ -91,32 +91,32 @@ 
 	 * @action Ajax_Hooks::CALLBACK_EXECUTION_EXCEPTION
 	 * @filter Ajax_Hooks::CALLBACK_RESPONSE_FILTER
 	 */
-	public function create_callback( Ajax $ajax_class ): Closure {
+	public function create_callback(Ajax $ajax_class): Closure {
 		/**
 		 * @param \PinkCrab\Ajax\Ajax $ajax_class
 		 * @return noreturn
 		 */
-		return function() use ( $ajax_class ): void {
+		return function() use ($ajax_class): void {
 
 			$valid_nonce = apply_filters(
 				Ajax_Hooks::REQUEST_NONCE_VERIFICATION,
-				$this->validate_request( $ajax_class ),
+				$this->validate_request($ajax_class),
 				$ajax_class,
 				$this->server_request
 			);
 
 			try {
 				$response = $valid_nonce
-				? $this->invoke_callback( $ajax_class )
+				? $this->invoke_callback($ajax_class)
 				: $this->response_factory->unauthorised();
-			} catch ( Exception $th ) {
+			} catch (Exception $th) {
 
-				do_action( Ajax_Hooks::CALLBACK_EXECUTION_EXCEPTION, $th, $ajax_class );
+				do_action(Ajax_Hooks::CALLBACK_EXECUTION_EXCEPTION, $th, $ajax_class);
 
-				$response = $this->response_factory->failure( array( 'error' => $th->getMessage() ) );
+				$response = $this->response_factory->failure(array('error' => $th->getMessage()));
 			}
 			$this->http_helper->emit_psr7_response(
-				\apply_filters( Ajax_Hooks::CALLBACK_RESPONSE_FILTER, $response, $ajax_class, $this->server_request )
+				\apply_filters(Ajax_Hooks::CALLBACK_RESPONSE_FILTER, $response, $ajax_class, $this->server_request)
 			);
 
 			\wp_die();

src/Registration_Middleware/Ajax_Middleware.php

Spacing +4 added lines, -4 removed lines

@@ -23,7 +23,7 @@ 
 	/** @var Ajax_Dispatcher */
 	public $dispatcher;
 
-	public function __construct( Ajax_Dispatcher $dispatcher ) {
+	public function __construct(Ajax_Dispatcher $dispatcher) {
 		$this->dispatcher = $dispatcher;
 	}
 
@@ -33,12 +33,12 @@ 
 	 * @param object $class
 	 * @return object
 	 */
-	public function process( $class ) {
-		if ( is_a( $class, Ajax::class )
+	public function process($class) {
+		if (is_a($class, Ajax::class)
 		&& is_admin()
 		&& wp_doing_ajax()
 		) {
-			$this->dispatcher->add_ajax_call( $class );
+			$this->dispatcher->add_ajax_call($class);
 		}
 		return $class;
 	}