Pink-Crab / Perique-Ajax

src/Ajax_Exception.php

Spacing +4 added lines, -4 removed lines

@@ -35,9 +35,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);
 	}
 
 	/**
@@ -47,8 +47,8 @@ 
 	 * @param string $class_string Ajax class
 	 * @return Ajax_Exception
 	 */
-	public static function undefined_action( string $class_string ): Ajax_Exception {
+	public static function undefined_action(string $class_string): Ajax_Exception {
 		$message = "{$class_string} has no defined action property";
-		return new Ajax_Exception( $message, 101 );
+		return new Ajax_Exception($message, 101);
 	}
 }

src/Dispatcher/Ajax_Dispatcher.php

Spacing +5 added lines, -5 removed lines

@@ -34,7 +34,7 @@ 
 	protected Hook_Loader $loader;
 	protected Ajax_Controller $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;
 	}
@@ -46,15 +46,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( esc_attr( \get_class( $ajax ) ) );
+		if ( ! $ajax->has_valid_action()) {
+			throw Ajax_Exception::undefined_action(esc_attr(\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/Ajax_Request_Validator.php

Spacing +11 added lines, -11 removed lines

@@ -33,7 +33,7 @@ 
 
 	protected ServerRequestInterface $server_request;
 
-	public function __construct( ServerRequestInterface $server_request ) {
+	public function __construct(ServerRequestInterface $server_request) {
 		$this->server_request = $server_request;
 	}
 
@@ -43,22 +43,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);
 	}
 
 	/**
@@ -67,11 +67,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_Controller.php

Spacing +12 added lines, -12 removed lines

@@ -57,8 +57,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);
 	}
 
 	/**
@@ -68,9 +68,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
 		);
 	}
@@ -84,32 +84,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/Module/Ajax_Middleware.php

Spacing +4 added lines, -4 removed lines

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

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_string ): Ajax {
-		if ( ! \is_subclass_of( $class_string, Ajax::class ) ) {
-			throw Ajax_Exception::non_ajax_model( 'get reflection' );
+	private static function get_reflected(string $class_string): Ajax {
+		if ( ! \is_subclass_of($class_string, Ajax::class)) {
+			throw Ajax_Exception::non_ajax_model('get reflection');
 		}
 
-		if ( ! array_key_exists( $class_string, self::$class_cache ) ) {
-			$reflection                         = new ReflectionClass( $class_string );
-			self::$class_cache[ $class_string ] = $reflection->newInstanceWithoutConstructor();
+		if ( ! array_key_exists($class_string, self::$class_cache)) {
+			$reflection                         = new ReflectionClass($class_string);
+			self::$class_cache[$class_string] = $reflection->newInstanceWithoutConstructor();
 		}
 
-		return self::$class_cache[ $class_string ];
+		return self::$class_cache[$class_string];
 	}
 
 	/**
@@ -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 ): ?string {
-		$instance = self::get_reflected( $class_string );
+	public static function get_action(string $class_string): ?string {
+		$instance = self::get_reflected($class_string);
 
-		if ( ! $instance->has_valid_action() ) {
-			throw Ajax_Exception::undefined_action( esc_attr( $class_string ) );
+		if ( ! $instance->has_valid_action()) {
+			throw Ajax_Exception::undefined_action(esc_attr($class_string));
 		}
 
 		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_string ): bool {
-		return self::get_reflected( $class_string )->has_nonce();
+	public static function has_nonce(string $class_string): bool {
+		return self::get_reflected($class_string)->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_string ): ?Nonce {
-		$instance = self::get_reflected( $class_string );
+	public static function get_nonce(string $class_string): ?Nonce {
+		$instance = self::get_reflected($class_string);
 
 		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 ): string {
-		return self::get_reflected( $class_string )->get_nonce_field();
+	public static function get_nonce_field(string $class_string): string {
+		return self::get_reflected($class_string)->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':