Pink-Crab / Perique_Admin_Menu

src/Exception/Page_Exception.php

Spacing +10 added lines, -10 removed lines

@@ -37,11 +37,11 @@ 
 	 * @return self
 	 * @code 200
 	 */
-	public static function view_not_set( Page $page ): self {
+	public static function view_not_set(Page $page): self {
 		return new self(
 			sprintf(
 				'View must be defined in %s to render template',
-				get_class( $page )
+				get_class($page)
 			),
 			200
 		);
@@ -54,12 +54,12 @@ 
 	 * @return self
 	 * @code 201
 	 */
-	public static function undefined_property( string $property, Page $page ): self {
+	public static function undefined_property(string $property, Page $page): self {
 		return new self(
 			sprintf(
 				'%s is a required property, not set in %s',
 				$property,
-				get_class( $page )
+				get_class($page)
 			),
 			201
 		);
@@ -73,11 +73,11 @@ 
 	 * @return self
 	 * @code 202
 	 */
-	public static function invalid_page_type( $page ): self {
+	public static function invalid_page_type($page): self {
 		return new self(
 			sprintf(
 				'%s is not defined in the Registrar and can not be registered.',
-				is_object( $page ) ? get_class( $page ) : 'UNKONW TYPE'
+				is_object($page) ? get_class($page) : 'UNKONW TYPE'
 			),
 			202
 		);
@@ -90,11 +90,11 @@ 
 	 * @return self
 	 * @code 202
 	 */
-	public static function acf_not_active( Page $page ): self {
+	public static function acf_not_active(Page $page): self {
 		return new self(
 			sprintf(
 				'%s can not be registered as ACF is not currently active.',
-				get_class( $page )
+				get_class($page)
 			),
 			203
 		);
@@ -107,11 +107,11 @@ 
 	 * @return self
 	 * @code 204
 	 */
-	public static function failed_to_register_page( Page $page ) {
+	public static function failed_to_register_page(Page $page) {
 		return new self(
 			sprintf(
 				'%s could not be registered',
-				get_class( $page )
+				get_class($page)
 			),
 			204
 		);

src/Exception/Group_Exception.php

Spacing +7 added lines, -7 removed lines

@@ -38,11 +38,11 @@ 
 	 * @return self
 	 * @code 250
 	 */
-	public static function primary_page_undefined( Abstract_Group $group ): self {
+	public static function primary_page_undefined(Abstract_Group $group): self {
 		return new self(
 			sprintf(
 				'The primary page is not defined in %s',
-				get_class( $group )
+				get_class($group)
 			),
 			250
 		);
@@ -55,11 +55,11 @@ 
 	 * @return self
 	 * @code 251
 	 */
-	public static function group_title_undefined( Abstract_Group $group ): self {
+	public static function group_title_undefined(Abstract_Group $group): self {
 		return new self(
 			sprintf(
 				'The group title is not defined in %s',
-				get_class( $group )
+				get_class($group)
 			),
 			251
 		);
@@ -73,12 +73,12 @@ 
 	 * @return self
 	 * @code 252
 	 */
-	public static function failed_validation( Group_Validator $validator, Abstract_Group $group ): self {
+	public static function failed_validation(Group_Validator $validator, Abstract_Group $group): self {
 		return new self(
 			sprintf(
 				'%s failed Group validation (%s)',
-				get_class( $group ),
-				join( ',', $validator->get_errors() )
+				get_class($group),
+				join(',', $validator->get_errors())
 			),
 			252
 		);

src/Group/Abstract_Group.php

Spacing +4 added lines, -4 removed lines

@@ -80,8 +80,8 @@ 
 	 * @return string
 	 */
 	public function get_group_title(): string {
-		if ( $this->group_title === null ) {
-			throw Group_Exception::group_title_undefined( $this );
+		if ($this->group_title === null) {
+			throw Group_Exception::group_title_undefined($this);
 		}
 		return $this->group_title;
 	}
@@ -110,8 +110,8 @@ 
 	 * @return string
 	 */
 	public function get_primary_page(): string {
-		if ( $this->primary_page === null ) {
-			throw Group_Exception::primary_page_undefined( $this );
+		if ($this->primary_page === null) {
+			throw Group_Exception::primary_page_undefined($this);
 		}
 		return $this->primary_page;
 	}

src/Registration_Middleware/Page_Middleware.php

Spacing +10 added lines, -10 removed lines

@@ -56,29 +56,29 @@ 
 	 * @param object $class
 	 * @return object
 	 */
-	public function process( $class ) {
+	public function process($class) {
 		// If we have a valid page.
 		if (
-			is_a( $class, Page::class )
+			is_a($class, Page::class)
 			&& is_admin()
-			&& ! is_null( $class->parent_slug() )
+			&& ! is_null($class->parent_slug())
 		) {
 			$this->add_to_loader(
-				function () use ( $class ) : void {
-					$this->dispatcher->register_subpage( $class, $class->parent_slug() );
+				function() use ($class) : void {
+					$this->dispatcher->register_subpage($class, $class->parent_slug());
 				}
 			);
 		}
 
 		// If we have a valid group.
 		if (
-			is_a( $class, Abstract_Group::class )
+			is_a($class, Abstract_Group::class)
 			&& is_admin()
 		) {
 
 			$this->add_to_loader(
-				function () use ( $class ): void {
-					$this->dispatcher->register_group( $class );
+				function() use ($class): void {
+					$this->dispatcher->register_group($class);
 				}
 			);
 		}
@@ -91,8 +91,8 @@ 
 	 * @param callable $callback
 	 * @return void
 	 */
-	protected function add_to_loader( callable $callback ): void {
-		$this->hook_loader->action( 'admin_menu', $callback );
+	protected function add_to_loader(callable $callback): void {
+		$this->hook_loader->action('admin_menu', $callback);
 	}
 
 

src/Registrar/Page_Dispatcher.php

Spacing +42 added lines, -42 removed lines

@@ -45,7 +45,7 @@ 
 	/** @var \PinkCrab\Perique_Admin_Menu\Registrar\Registrar */
 	protected $registrar;
 
-	public function __construct( DI_Container $di_container, View $view, Registrar $registrar ) {
+	public function __construct(DI_Container $di_container, View $view, Registrar $registrar) {
 		$this->di_container = $di_container;
 		$this->view         = $view;
 		$this->registrar    = $registrar;
@@ -57,10 +57,10 @@ 
 	 * @param \PinkCrab\Perique_Admin_Menu\Group\Abstract_Group $group
 	 * @return void
 	 */
-	public function register_group( Abstract_Group $group ): void {
+	public function register_group(Abstract_Group $group): void {
 
 		// If current user can not access the group, bail without attempting to register.
-		if ( ! current_user_can( $group->get_capability() ) ) {
+		if ( ! current_user_can($group->get_capability())) {
 			return;
 		}
 
@@ -68,20 +68,20 @@ 
 
 			// Validate the group.
 			$validator = new Group_Validator();
-			if ( ! $validator->validate( $group ) ) {
-				throw Group_Exception::failed_validation( $validator, $group );
+			if ( ! $validator->validate($group)) {
+				throw Group_Exception::failed_validation($validator, $group);
 			}
 
-			$this->register_primary_page( $group );
+			$this->register_primary_page($group);
 
 			// Register all pages and attempt to set primary page name in menu.
-			foreach ( $this->get_pages( $group ) as $page ) {
-				$this->register_subpage( $page, $this->get_primary_page( $group )->slug() );
+			foreach ($this->get_pages($group) as $page) {
+				$this->register_subpage($page, $this->get_primary_page($group)->slug());
 			}
-			$this->set_primary_page_details( $group );
+			$this->set_primary_page_details($group);
 
-		} catch ( \Throwable $th ) {
-			$this->admin_exception_notice( $group, $th );
+		} catch (\Throwable $th) {
+			$this->admin_exception_notice($group, $th);
 		}
 	}
 
@@ -92,22 +92,22 @@ 
 	 * @param \Throwable $exception
 	 * @return void
 	 */
-	public function admin_exception_notice( $object, Throwable $exception ): void {
+	public function admin_exception_notice($object, Throwable $exception): void {
 		add_action(
 			'admin_notices',
-			function() use ( $object, $exception ) {
+			function() use ($object, $exception) {
 				$class   = 'notice notice-error';
 				$message = sprintf(
 					'%s <i>%s</i> generated errors while being registered. This might result in admin pages being missing or broken. <br><b>%s(%s: %s)</b>',
-					get_class( $object ) === Page::class ? 'Page' : 'Menu Group',
-					get_class( $object ),
+					get_class($object) === Page::class ? 'Page' : 'Menu Group',
+					get_class($object),
 					$exception->getMessage(),
 					$exception->getFile(),
 					$exception->getLine()
 				);
 				printf(
 					'<div class="%1$s"><p>%2$s</p></div>',
-					esc_attr( $class ),
+					esc_attr($class),
 					wp_kses(
 						$message,
 						array(
@@ -127,9 +127,9 @@ 
 	 * @param \PinkCrab\Perique_Admin_Menu\Group\Abstract_Group $group
 	 * @return void
 	 */
-	protected function register_primary_page( Abstract_Group $group ): void {
+	protected function register_primary_page(Abstract_Group $group): void {
 		$this->registrar->register_primary(
-			$this->get_primary_page( $group ),
+			$this->get_primary_page($group),
 			$group
 		);
 	}
@@ -140,17 +140,17 @@ 
 	 * @param \PinkCrab\Perique_Admin_Menu\Group\Abstract_Group $group
 	 * @return \PinkCrab\Perique_Admin_Menu\Page\Page
 	 */
-	protected function get_primary_page( Abstract_Group $group ): Page {
+	protected function get_primary_page(Abstract_Group $group): Page {
 		/** @var Page */
-		$page = $this->di_container->create( $group->get_primary_page() );
+		$page = $this->di_container->create($group->get_primary_page());
 
-		if ( ! is_object( $page ) || ! is_a( $page, Page::class ) ) {
-			throw Page_Exception::invalid_page_type( $page );
+		if ( ! is_object($page) || ! is_a($page, Page::class)) {
+			throw Page_Exception::invalid_page_type($page);
 		}
 
 		// Register view if requied.
-		if ( \method_exists( $page, 'set_view' ) ) {
-			$page->set_view( $this->view );
+		if (\method_exists($page, 'set_view')) {
+			$page->set_view($this->view);
 		}
 
 		return $page;
@@ -164,18 +164,18 @@ 
 	 * @return array<Page>
 	 * @throws Page_Exception (Code 202)
 	 */
-	protected function get_pages( Abstract_Group $group ): array {
+	protected function get_pages(Abstract_Group $group): array {
 		return array_map(
-			function( string $page ): Page {
-				$constructed_page = $this->di_container->create( $page );
-				if ( ! is_object( $constructed_page ) || ! is_a( $constructed_page, Page::class ) ) {
-					throw Page_Exception::invalid_page_type( $constructed_page );
+			function(string $page): Page {
+				$constructed_page = $this->di_container->create($page);
+				if ( ! is_object($constructed_page) || ! is_a($constructed_page, Page::class)) {
+					throw Page_Exception::invalid_page_type($constructed_page);
 				}
 				return $constructed_page;
 			},
 			array_filter(
 				$group->get_pages(),
-				function( string $page ) use ( $group ) {
+				function(string $page) use ($group) {
 					return $page !== $group->get_primary_page();
 				}
 			)
@@ -191,22 +191,22 @@ 
 	 * @phpcs:disable WordPress.WP.GlobalVariablesOverride.Prohibited
 	 * @throws Group_Exception (code 253)
 	 */
-	protected function set_primary_page_details( Abstract_Group $group ) {
+	protected function set_primary_page_details(Abstract_Group $group) {
 		global $submenu;
 
-		$primary = $this->get_primary_page( $group );
+		$primary = $this->get_primary_page($group);
 
-		if ( ! array_key_exists( $primary->slug(), $submenu ) ) {
+		if ( ! array_key_exists($primary->slug(), $submenu)) {
 			return;
 		}
 
 		$primary_page_key = array_search(
 			$primary->slug(),
-			\array_column( $submenu[ $primary->slug() ], 2 ),
+			\array_column($submenu[$primary->slug()], 2),
 			true
 		) ?: 0;
 
-		$submenu[ $primary->slug() ][ $primary_page_key ][0] = $primary->menu_title();
+		$submenu[$primary->slug()][$primary_page_key][0] = $primary->menu_title();
 	}
 
 
@@ -217,21 +217,21 @@ 
 	 * @param string $parent_slug
 	 * @return void
 	 */
-	public function register_subpage( Page $page, string $parent_slug ): void {
+	public function register_subpage(Page $page, string $parent_slug): void {
 		// If user cant access the page, bail before attemptin to register.
-		if ( ! current_user_can( $page->capability() ) ) {
+		if ( ! current_user_can($page->capability())) {
 			return;
 		}
 
 		// Register view if requied.
-		if ( \method_exists( $page, 'set_view' ) ) {
-			$page->set_view( $this->view );
+		if (\method_exists($page, 'set_view')) {
+			$page->set_view($this->view);
 		}
 
 		try {
-			$this->registrar->register_subpage( $page, $parent_slug );
-		} catch ( \Throwable $th ) {
-			$this->admin_exception_notice( $page, $th );
+			$this->registrar->register_subpage($page, $parent_slug);
+		} catch (\Throwable $th) {
+			$this->admin_exception_notice($page, $th);
 		}
 	}
 

src/Registrar/Registrar.php

Spacing +12 added lines, -12 removed lines

@@ -42,13 +42,13 @@ 
 	 * @throws Page_Exception (Code 204)
 	 * @throws TypeError
 	 */
-	public function register_primary( Page $page, Abstract_Group $group = null ): void {
+	public function register_primary(Page $page, Abstract_Group $group = null): void {
 
-		switch ( get_parent_class( $page ) ) {
+		switch (get_parent_class($page)) {
 			// For menu pages
 			case Menu_Page::class:
-				if ( $group === null ) {
-					throw new TypeError( 'Valid group must be passed to create Menu_Page' );
+				if ($group === null) {
+					throw new TypeError('Valid group must be passed to create Menu_Page');
 				}
 
 				$result = add_menu_page(
@@ -62,13 +62,13 @@ 
 				);
 
 				// Call failed action for logging etc.
-				if ( ! is_string( $result ) ) {
-					throw Page_Exception::failed_to_register_page( $page );
+				if ( ! is_string($result)) {
+					throw Page_Exception::failed_to_register_page($page);
 				}
 				break;
 
 			default:
-				do_action( Hooks::PAGE_REGISTRAR_PRIMARY, $page, $group );
+				do_action(Hooks::PAGE_REGISTRAR_PRIMARY, $page, $group);
 		}
 	}
 
@@ -80,8 +80,8 @@ 
 	 * @return void
 	 * @throws Page_Exception (Code 204)
 	 */
-	public function register_subpage( Page $page, string $parent_slug ): void {
-		switch ( get_parent_class( $page ) ) {
+	public function register_subpage(Page $page, string $parent_slug): void {
+		switch (get_parent_class($page)) {
 			case Menu_Page::class:
 				$hook = add_submenu_page(
 					$parent_slug,
@@ -93,12 +93,12 @@ 
 					$page->position()
 				);
 
-				if ( ! is_string( $hook ) ) {
-					throw Page_Exception::failed_to_register_page( $page );
+				if ( ! is_string($hook)) {
+					throw Page_Exception::failed_to_register_page($page);
 				}
 				break;
 			default:
-				do_action( Hooks::PAGE_REGISTRAR_SUB, $page, $parent_slug );
+				do_action(Hooks::PAGE_REGISTRAR_SUB, $page, $parent_slug);
 		}
 	}
 }

src/Page/Menu_Page.php

Spacing +10 added lines, -10 removed lines

@@ -99,7 +99,7 @@ 
 	 * @param View $view  View
 	 * @return self
 	 */
-	public function set_view( View $view ): self {
+	public function set_view(View $view): self {
 		$this->view = $view;
 		return $this;
 	}
@@ -115,8 +115,8 @@ 
 	 * @return string
 	 */
 	public function slug(): string {
-		if ( $this->page_slug === null ) {
-			throw Page_Exception::undefined_property( 'page_slug', $this );
+		if ($this->page_slug === null) {
+			throw Page_Exception::undefined_property('page_slug', $this);
 		}
 		return $this->page_slug;
 	}
@@ -125,8 +125,8 @@ 
 	 * @return string
 	 */
 	public function menu_title(): string {
-		if ( $this->menu_title === null ) {
-			throw Page_Exception::undefined_property( 'menu_title', $this );
+		if ($this->menu_title === null) {
+			throw Page_Exception::undefined_property('menu_title', $this);
 		}
 		return $this->menu_title;
 	}
@@ -160,16 +160,16 @@ 
 	 * @throws Page_Exception code 201 if template not defined.
 	 */
 	public function render_view(): callable {
-		if ( $this->view === null ) {
-			throw Page_Exception::view_not_set( $this );
+		if ($this->view === null) {
+			throw Page_Exception::view_not_set($this);
 		}
 
-		if ( $this->view_template === null ) {
-			throw Page_Exception::undefined_property( 'view_template', $this );
+		if ($this->view_template === null) {
+			throw Page_Exception::undefined_property('view_template', $this);
 		}
 
 		return function() {
-			$this->view->render( $this->view_template, $this->view_data );
+			$this->view->render($this->view_template, $this->view_data);
 		};
 
 	}

src/Validator/Abstract_Validator.php

Indentation +4 added lines, -4 removed lines

@@ -27,10 +27,10 @@
 abstract class Abstract_Validator {
 
 		/**
-	 * Holds any errors encounted when validation a group.
-	 *
-	 * @var array<string>
-	 */
+		 * Holds any errors encounted when validation a group.
+		 *
+		 * @var array<string>
+		 */
 	protected $errors = array();
 
 	/**

Spacing +3 added lines, -3 removed lines

@@ -48,7 +48,7 @@ 
 	 * @return bool
 	 */
 	public function has_errors(): bool {
-		return count( $this->errors ) > 0;
+		return count($this->errors) > 0;
 	}
 
 	/**
@@ -66,7 +66,7 @@ 
 	 * @param string $error
 	 * @return void
 	 */
-	public function push_error( string $error ): void {
+	public function push_error(string $error): void {
 		$this->errors[] = $error;
 	}
 
@@ -76,5 +76,5 @@ 
 	 * @param mixed $subject
 	 * @return bool
 	 */
-	abstract public function validate( $subject): bool;
+	abstract public function validate($subject): bool;
 }

src/Validator/Group_Validator.php

Spacing +7 added lines, -7 removed lines

@@ -35,15 +35,15 @@ 
 	 * @param mixed $group
 	 * @return bool
 	 */
-	public function validate( $group ): bool {
+	public function validate($group): bool {
 		$this->reset_errors();
 
-		if ( ! is_a( $group, Abstract_Group::class ) ) {
-			$this->push_error( sprintf( '%s Is not a valid group type.', get_class( $group ) ) );
+		if ( ! is_a($group, Abstract_Group::class)) {
+			$this->push_error(sprintf('%s Is not a valid group type.', get_class($group)));
 			return false;
 		}
 
-		return $this->check_properties( $group );
+		return $this->check_properties($group);
 	}
 
 	/**
@@ -53,12 +53,12 @@ 
 	 * @param \PinkCrab\Perique_Admin_Menu\Group\Abstract_Group $group
 	 * @return bool
 	 */
-	protected function check_properties( Abstract_Group $group ): bool {
+	protected function check_properties(Abstract_Group $group): bool {
 		try {
 			$group->get_primary_page();
 			$group->get_group_title();
-		} catch ( \Throwable $th ) {
-			$this->push_error( $th->getMessage() );
+		} catch (\Throwable $th) {
+			$this->push_error($th->getMessage());
 			return false;
 		}
 		return true;