Code: add types

Author: f3l1x

src/AbstractMenuItemsContainer.php

@@ -138,13 +138,11 @@ private function hasVisibleItemsOn(string $type): bool
 	 */
 	private function getVisibleItemsOn(string $type): array
 	{
-		return array_filter($this->getItems(), function (IMenuItem $item) use ($type): bool {
-			return match ($type) {
+		return array_filter($this->getItems(), fn (IMenuItem $item): bool => match ($type) {
 				'menu' => $item->isVisibleOnMenu(),
 				'breadcrumbs' => $item->isVisibleOnBreadcrumbs(),
 				'sitemap' => $item->isVisibleOnSitemap(),
 				default => false,
-			};
 		});
 	}
 

src/Config/MenuItemAction.php

@@ -7,11 +7,11 @@ final class MenuItemAction
 
 	public string $target;
 
-	/** @var array<string, string> */
+	/** @var array<string, mixed> */
 	public array $parameters = [];
 
 	/**
-	 * @param array<string, mixed> $array
+	 * @param array{target: string, parameters?: array<string, mixed>} $array
 	 */
 	public static function fromArray(array $array): self
 	{

src/Config/TemplatePaths.php

@@ -1,6 +1,4 @@
-<?php
-
-declare(strict_types=1);
+<?php declare(strict_types = 1);
 
 namespace Contributte\MenuControl\Config;
 

src/DI/MenuExtension.php

@@ -136,15 +136,20 @@ private function loadMenuConfiguration(
 	 */
 	private function normalizeMenuItems(Processor $processor, array $items): array
 	{
-		array_walk($items, function (array &$item, string $key) use ($processor): void {
-			$item = $processor->process($this->getItemSchema(), $item);
-
-			if ($item->title === null) {
-				$item->title = $key;
+		array_walk(
+			$items,
+			// @phpcs:ignore SlevomatCodingStandard.PHP.DisallowReference.DisallowedPassingByReference
+			function (&$item, string $key) use ($processor): void {
+				$item = $processor->process($this->getItemSchema(), $item);
+				assert($item instanceof stdClass);
+
+				if ($item->title === null) {
+					$item->title = $key;
+				}
+
+				$item->items = $this->normalizeMenuItems($processor, $item->items);
 			}
-
-			$item->items = $this->normalizeMenuItems($processor, $item->items);
-		});
+		);
 
 		return $items;
 	}

src/IMenuItem.php

@@ -15,7 +15,7 @@ public function isAllowed(): bool;
 	public function getActionTarget(): ?string;
 
 	/**
-	 * @return array<string, string>
+	 * @return array<string, mixed>
 	 */
 	public function getActionParameters(): array;
 

src/Loaders/DefaultMenuLoader.php

@@ -6,7 +6,7 @@
 use Contributte\MenuControl\IMenu;
 use Contributte\MenuControl\IMenuItem;
 use Contributte\MenuControl\IMenuItemsContainer;
-use Nette\Utils;
+use Nette\Utils\Validators;
 use stdClass;
 
 final class DefaultMenuLoader implements IMenuLoader
@@ -49,7 +49,7 @@ private function processItem(IMenuItemsContainer $parent, string $name, stdClass
 
 			if ($config->action !== null) {
 				if (is_array($config->action)) {
-					$item->setAction(MenuItemAction::fromArray($config->action));
+					$item->setAction(MenuItemAction::fromArray($config->action)); // @phpstan-ignore-line
 				} else {
 					$item->setAction(MenuItemAction::fromString($config->action));
 				}
@@ -61,7 +61,7 @@ private function processItem(IMenuItemsContainer $parent, string $name, stdClass
 
 			if ($config->include !== null) {
 				$include = array_map(function ($include) {
-					Utils\Validators::assert($include, 'string');
+					Validators::assert($include, 'string');
 
 					return $include;
 				}, is_array($config->include) ? $config->include : [$config->include]);

src/MenuItem.php

@@ -91,7 +91,7 @@ public function getActionTarget(): ?string
 	}
 
 	/**
-	 * @return array<string, string>
+	 * @return array<string, mixed>
 	 */
 	public function getActionParameters(): array
 	{

src/Traits/MenuItemData.php

@@ -5,19 +5,19 @@
 trait MenuItemData
 {
 
-	/** @var array<string, string> */
+	/** @var array<string, mixed> */
 	private array $data = [];
 
 	/**
-	 * @return array<string, string>
+	 * @return array<string, mixed>
 	 */
 	public function getData(): array
 	{
 		return $this->data;
 	}
 
 	/**
-	 * @param array<string, string> $data
+	 * @param array<string, mixed> $data
 	 */
 	public function setData(array $data): void
 	{