Pink-Crab / Perique_Admin_Menu

src/Registrar/Page_Enqueue_Action.php

Spacing +6 added lines, -6 removed lines

@@ -38,7 +38,7 @@ 
 	/** The option group being enqueued */
 	protected ?Abstract_Group $group;
 
-	public function __construct( string $hook, Page $page, ?Abstract_Group $group = null ) {
+	public function __construct(string $hook, Page $page, ?Abstract_Group $group = null) {
 		$this->hook  = $hook;
 		$this->page  = $page;
 		$this->group = $group;
@@ -50,15 +50,15 @@ 
 	 * @param string $page_hook
 	 * @return void
 	 */
-	public function __invoke( string $page_hook ) {
-		if ( $page_hook === $this->hook ) {
+	public function __invoke(string $page_hook) {
+		if ($page_hook === $this->hook) {
 
 			// Register hooks for the group if part of group
-			if ( null !== $this->group ) {
-				$this->group->enqueue( $this->group, $this->page );
+			if (null !== $this->group) {
+				$this->group->enqueue($this->group, $this->page);
 			}
 
-			$this->page->enqueue( $this->page );
+			$this->page->enqueue($this->page);
 		}
 	}
 }

src/Registrar/Page_Dispatcher.php

Spacing +48 added lines, -48 removed lines

@@ -40,7 +40,7 @@ 
 	protected View $view;
 	protected Registrar $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;
@@ -52,10 +52,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;
 		}
 
@@ -63,20 +63,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(), $group );
+			foreach ($this->get_pages($group) as $page) {
+				$this->register_subpage($page, $this->get_primary_page($group)->slug(), $group);
 			}
-			$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);
 		}
 	}
 
@@ -87,22 +87,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(
@@ -122,9 +122,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
 		);
 	}
@@ -135,17 +135,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;
@@ -159,18 +159,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();
 				}
 			)
@@ -186,22 +186,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();
 	}
 
 
@@ -213,21 +213,21 @@ 
 	 * @param \PinkCrab\Perique_Admin_Menu\Group\Abstract_Group|null $group
 	 * @return void
 	 */
-	public function register_subpage( Page $page, string $parent_slug, ?Abstract_Group $group = null ): void {
+	public function register_subpage(Page $page, string $parent_slug, ?Abstract_Group $group = null): void {
 		// If user cant access the page, bail before attempting to register.
-		if ( ! current_user_can( $page->capability() ) ) {
+		if ( ! current_user_can($page->capability())) {
 			return;
 		}
 
 		// Register view if required.
-		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, $group );
-		} catch ( \Throwable $th ) {
-			$this->admin_exception_notice( $page, $th );
+			$this->registrar->register_subpage($page, $parent_slug, $group);
+		} catch (\Throwable $th) {
+			$this->admin_exception_notice($page, $th);
 		}
 	}
 
@@ -237,16 +237,16 @@ 
 	 * @param \PinkCrab\Perique_Admin_Menu\Page\Page $page
 	 * @return void
 	 */
-	public function register_single_page( Page $page ): void {
+	public function register_single_page(Page $page): void {
 		// Register view if required.
-		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_primary( $page, null );
-		} catch ( \Throwable $th ) {
-			$this->admin_exception_notice( $page, $th );
+			$this->registrar->register_primary($page, null);
+		} catch (\Throwable $th) {
+			$this->admin_exception_notice($page, $th);
 		}
 	}
 

src/Registrar/Page_Load_Action.php

Spacing +4 added lines, -4 removed lines

@@ -34,7 +34,7 @@ 
 	/** The option group being enqueued  */
 	protected ?Abstract_Group $group;
 
-	public function __construct( Page $page, ?Abstract_Group $group = null ) {
+	public function __construct(Page $page, ?Abstract_Group $group = null) {
 		$this->page  = $page;
 		$this->group = $group;
 	}
@@ -46,10 +46,10 @@ 
 	 */
 	public function __invoke() {
 		// Register hooks for the group if part of group
-		if ( null !== $this->group ) {
-			$this->group->load( $this->group, $this->page );
+		if (null !== $this->group) {
+			$this->group->load($this->group, $this->page);
 		}
 
-		$this->page->load( $this->page );
+		$this->page->load($this->page);
 	}
 }

src/Page/Menu_Page.php

Spacing +13 added lines, -13 removed lines

@@ -106,7 +106,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;
 	}
@@ -122,8 +122,8 @@ 
 	 * @return string
 	 */
 	public function slug(): string {
-		if ( $this->page_slug === '' ) {
-			throw Page_Exception::undefined_property( 'page_slug', $this );
+		if ($this->page_slug === '') {
+			throw Page_Exception::undefined_property('page_slug', $this);
 		}
 		return $this->page_slug;
 	}
@@ -132,8 +132,8 @@ 
 	 * @return string
 	 */
 	public function menu_title(): string {
-		if ( $this->menu_title === '' ) {
-			throw Page_Exception::undefined_property( 'menu_title', $this );
+		if ($this->menu_title === '') {
+			throw Page_Exception::undefined_property('menu_title', $this);
 		}
 		return $this->menu_title;
 	}
@@ -167,17 +167,17 @@ 
 	 * @throws Page_Exception code 201 if template not defined.
 	 */
 	public function render_view(): callable {
-		if ( null === $this->view ) {
-			throw Page_Exception::view_not_set( $this );
+		if (null === $this->view) {
+			throw Page_Exception::view_not_set($this);
 		}
 
-		if ( '' === $this->view_template ) {
-			throw Page_Exception::undefined_property( 'view_template', $this );
+		if ('' === $this->view_template) {
+			throw Page_Exception::undefined_property('view_template', $this);
 		}
 
 		return function() {
 			// @phpstan-ignore-next-line, as we have already checked for null.
-			$this->view->render( $this->view_template, $this->view_data );
+			$this->view->render($this->view_template, $this->view_data);
 		};
 
 	}
@@ -189,7 +189,7 @@ 
 	 * @return void
 	 * @codeCoverageIgnore This can be tested as it does nothing and is extended only
 	 */
-	public function enqueue( Page $page ): void {
+	public function enqueue(Page $page): void {
 		// Do nothing.
 		// Can be extended in any child class that extends.
 	}
@@ -201,7 +201,7 @@ 
 	 * @return void
 	 * @codeCoverageIgnore This can be tested as it does nothing and is extended only
 	 */
-	public function load( Page $page ): void {
+	public function load(Page $page): void {
 		// Do nothing.
 		// Can be extended in any child class that extends.
 	}
@@ -212,7 +212,7 @@ 
 	 * @param string $page_hook
 	 * @return void
 	 */
-	final public function set_page_hook( string $page_hook ): void {
+	final public function set_page_hook(string $page_hook): void {
 		$this->page_hook = $page_hook;
 	}
 

src/Module/Admin_Menu.php

Spacing +3 added lines, -3 removed lines

@@ -38,7 +38,7 @@
 	}
 
 	## Unused methods
-	public function pre_register( App_Config $config, Hook_Loader $loader, DI_Container $di_container ): void {} // phpcs:ignore Generic.CodeAnalysis.UnusedFunctionParameter.FoundInImplementedInterfaceBeforeLastUsed
-	public function pre_boot( App_Config $config, Hook_Loader $loader, DI_Container $di_container ): void {} // phpcs:ignore Generic.CodeAnalysis.UnusedFunctionParameter.FoundInImplementedInterfaceBeforeLastUsed
-	public function post_register( App_Config $config, Hook_Loader $loader, DI_Container $di_container ): void {} // phpcs:ignore Generic.CodeAnalysis.UnusedFunctionParameter.FoundInImplementedInterfaceBeforeLastUsed
+	public function pre_register(App_Config $config, Hook_Loader $loader, DI_Container $di_container): void {} // phpcs:ignore Generic.CodeAnalysis.UnusedFunctionParameter.FoundInImplementedInterfaceBeforeLastUsed
+	public function pre_boot(App_Config $config, Hook_Loader $loader, DI_Container $di_container): void {} // phpcs:ignore Generic.CodeAnalysis.UnusedFunctionParameter.FoundInImplementedInterfaceBeforeLastUsed
+	public function post_register(App_Config $config, Hook_Loader $loader, DI_Container $di_container): void {} // phpcs:ignore Generic.CodeAnalysis.UnusedFunctionParameter.FoundInImplementedInterfaceBeforeLastUsed
 }