Pink-Crab / Perique-Route

src/Route/Route.php

Spacing +27 added lines, -27 removed lines

@@ -36,9 +36,9 @@ 
 	protected $callback = null;
 
 
-	public function __construct( string $method, string $route ) {
+	public function __construct(string $method, string $route) {
 		$this->method = $method;
-		$this->route  = $this->format_route( $route );
+		$this->route  = $this->format_route($route);
 	}
 
 	/**
@@ -47,9 +47,9 @@ 
 	 * @param string $route
 	 * @return string
 	 */
-	protected function format_route( string $route ): string {
-		$route = $this->translate_arguments( $route );
-		return '/' . ltrim( $route, '/\\' );
+	protected function format_route(string $route): string {
+		$route = $this->translate_arguments($route);
+		return '/'.ltrim($route, '/\\');
 	}
 
 	/**
@@ -59,27 +59,27 @@ 
 	 * @param string $route
 	 * @return string
 	 */
-	protected function translate_arguments( string $route ): string {
-		if ( preg_match( '/[^-:\/_{}()a-zA-Z\d]/', $route ) ) {
+	protected function translate_arguments(string $route): string {
+		if (preg_match('/[^-:\/_{}()a-zA-Z\d]/', $route)) {
 			return $route;
 		}
 
 		// Create capture group for ":parameter"
 		$allowed_chars = '[\@a-zA-Z0-9&.?:-_=#]*';
 		$route         = preg_replace(
-			'/:(' . $allowed_chars . ')/',
-			'(?<$1>' . $allowed_chars . ')',
+			'/:('.$allowed_chars.')/',
+			'(?<$1>'.$allowed_chars.')',
 			$route
 		);
 
 		// Create capture group for '{parameter}'
 		$route = preg_replace(
-			'/{(' . $allowed_chars . ')}/',
-			'(?<$1>' . $allowed_chars . ')',
-			is_string( $route ) ? $route : ''
+			'/{('.$allowed_chars.')}/',
+			'(?<$1>'.$allowed_chars.')',
+			is_string($route) ? $route : ''
 		);
 
-		return is_string( $route ) ? $route : '';
+		return is_string($route) ? $route : '';
 	}
 
 	/**
@@ -98,7 +98,7 @@ 
 	 *
 	 * @return self
 	 */
-	public function callback( callable $callback ): self {
+	public function callback(callable $callback): self {
 		$this->callback = $callback;
 		return $this;
 	}
@@ -127,7 +127,7 @@ 
 	 * @param string $namespace
 	 * @return static
 	 */
-	public function namespace( string $namespace ) {
+	public function namespace(string $namespace) {
 		$this->namespace = $namespace;
 		return $this;
 	}
@@ -138,27 +138,27 @@ 
 	 * @param string $method
 	 * @return self
 	 */
-	public function with_method( string $method ): self {
-		$clone = new self( $method, $this->get_route() );
+	public function with_method(string $method): self {
+		$clone = new self($method, $this->get_route());
 		// Arguments
-		if ( ! empty( $this->arguments ) ) {
-			foreach ( $this->arguments as $argument ) {
-				$clone->argument( $argument );
+		if ( ! empty($this->arguments)) {
+			foreach ($this->arguments as $argument) {
+				$clone->argument($argument);
 			}
 		}
 		// Authentication
-		if ( ! empty( $this->authentication ) ) {
-			foreach ( $this->authentication as $authentication ) {
-				$clone->authentication( $authentication );
+		if ( ! empty($this->authentication)) {
+			foreach ($this->authentication as $authentication) {
+				$clone->authentication($authentication);
 			}
 		}
 		// Callback
-		if ( ! empty( $this->callback ) ) {
-			$clone->callback( $this->callback );
+		if ( ! empty($this->callback)) {
+			$clone->callback($this->callback);
 		}
 		// Namespace
-		if ( ! empty( $this->namespace ) ) {
-			$clone->namespace( $this->namespace );
+		if ( ! empty($this->namespace)) {
+			$clone->namespace($this->namespace);
 		}
 
 		return $clone;

src/Registration/WP_Rest_Registrar.php

Spacing +22 added lines, -22 removed lines

@@ -26,10 +26,10 @@ 
 	 * @param \PinkCrab\Route\Route\Route $route
 	 * @return callable
 	 */
-	public function create_callback( Route $route ): callable {
-		return function() use ( $route ): void {
-			$model = $this->map_to_wp_rest( $route );
-			register_rest_route( $model->namespace, $model->route, $model->args );
+	public function create_callback(Route $route): callable {
+		return function() use ($route): void {
+			$model = $this->map_to_wp_rest($route);
+			register_rest_route($model->namespace, $model->route, $model->args);
 		};
 	}
 
@@ -39,11 +39,11 @@ 
 	 * @param \PinkCrab\Route\Route\Route $route
 	 * @return WP_Rest_Route
 	 */
-	public function map_to_wp_rest( Route $route ): WP_Rest_Route {
+	public function map_to_wp_rest(Route $route): WP_Rest_Route {
 		$wp_rest            = new WP_Rest_Route();
 		$wp_rest->namespace = $route->get_namespace();
 		$wp_rest->route     = $route->get_route();
-		$wp_rest->args      = $this->parse_options( $route );
+		$wp_rest->args      = $this->parse_options($route);
 		return $wp_rest;
 	}
 
@@ -54,23 +54,23 @@ 
 	 * @return array<mixed>
 	 * @throws Route_Exception
 	 */
-	protected function parse_options( Route $route ): array {
+	protected function parse_options(Route $route): array {
 
 		// If we have no callback defined for route, throw.
-		if ( is_null( $route->get_callback() ) ) {
-			throw Route_Exception::callback_not_defined( $route );
+		if (is_null($route->get_callback())) {
+			throw Route_Exception::callback_not_defined($route);
 		}
 
 		// If we have an invlaid method, throw
-		if ( ! $this->is_valid_method( $route->get_method() ) ) {
-			throw Route_Exception::invalid_http_method( $route );
+		if ( ! $this->is_valid_method($route->get_method())) {
+			throw Route_Exception::invalid_http_method($route);
 		}
 
 		$options                        = array();
 		$options['methods']             = $route->get_method();
 		$options['callback']            = $route->get_callback();
-		$options['permission_callback'] = $this->compose_permission_callback( $route );
-		$options['args']                = $this->parse_args( $route );
+		$options['permission_callback'] = $this->compose_permission_callback($route);
+		$options['args']                = $this->parse_args($route);
 
 		return $options;
 	}
@@ -81,11 +81,11 @@ 
 	 * @param Route $route
 	 * @return array<mixed>
 	 */
-	protected function parse_args( Route $route ): array {
+	protected function parse_args(Route $route): array {
 		return array_reduce(
 			$route->get_arguments(),
-			function( array $args, Argument $argument ) {
-				$args[ $argument->get_key() ] = Argument_Parser::as_single( $argument );
+			function(array $args, Argument $argument) {
+				$args[$argument->get_key()] = Argument_Parser::as_single($argument);
 				return $args;
 			},
 			array()
@@ -98,12 +98,12 @@ 
 	 * @param string $method
 	 * @return boolean
 	 */
-	protected function is_valid_method( string $method ): bool {
+	protected function is_valid_method(string $method): bool {
 		return in_array(
 			$method,
 			apply_filters(
 				'pinkcrab/route/accepted_http_methods', // phpcs:ignore WordPress.NamingConventions.ValidHookName
-				array( Route::DELETE, Route::POST, Route::PUT, Route::PATCH, Route::GET )
+				array(Route::DELETE, Route::POST, Route::PUT, Route::PATCH, Route::GET)
 			),
 			true
 		);
@@ -115,20 +115,20 @@ 
 	 * @param Route $route
 	 * @return callable
 	 */
-	protected function compose_permission_callback( Route $route ): callable {
+	protected function compose_permission_callback(Route $route): callable {
 		$callbacks = $route->get_authentication();
 
 		// If we have no callback defined, use return true.
-		if ( count( $callbacks ) === 0 ) {
+		if (count($callbacks) === 0) {
 			return '__return_true';
 		}
 
 		// If we only have 1, return as is.
-		if ( count( $callbacks ) === 1 ) {
+		if (count($callbacks) === 1) {
 			return $callbacks[0];
 		}
 
-		return Utils::compose_conditional_all_true( ...$callbacks );
+		return Utils::compose_conditional_all_true(...$callbacks);
 	}
 
 

src/Route_Exception.php

Spacing +8 added lines, -8 removed lines

@@ -24,9 +24,9 @@ 
 	 * @return self
 	 * @code 101
 	 */
-	public static function namespace_not_defined( string $route ): self {
+	public static function namespace_not_defined(string $route): self {
 		return new self(
-			sprintf( 'Namespace not defined in %s', $route ),
+			sprintf('Namespace not defined in %s', $route),
 			101
 		);
 	}
@@ -38,7 +38,7 @@ 
 	 * @return self
 	 * @code 102
 	 */
-	public static function callback_not_defined( Route $route ): self {
+	public static function callback_not_defined(Route $route): self {
 		// Set the namespace if exists.
 		$namespace = '' !== $route->get_namespace()
 				? $route->get_namespace()
@@ -47,9 +47,9 @@ 
 		return new self(
 			sprintf(
 				'Callback not defined for [%s] %s%s',
-				strtoupper( $route->get_method() ),
-				strtoupper( $namespace ),
-				strtoupper( $route->get_route() )
+				strtoupper($route->get_method()),
+				strtoupper($namespace),
+				strtoupper($route->get_route())
 			),
 			102
 		);
@@ -61,11 +61,11 @@ 
 	 * @param Route $route
 	 * @return self
 	 */
-	public static function invalid_http_method( Route $route ): self {
+	public static function invalid_http_method(Route $route): self {
 		return new self(
 			sprintf(
 				'%s is a none supported HTTP Mehtod.',
-				strtoupper( $route->get_method() )
+				strtoupper($route->get_method())
 			),
 			103
 		);