From 4dcdf2b1cf8ba61812625258f9cec975446fdeb0 Mon Sep 17 00:00:00 2001
From: Brandon Shipley <bshipley@hipowered.dev>
Date: Sat, 29 Mar 2025 03:04:45 -0700
Subject: [PATCH] allow for matching either or targets

---
 src/View/Helper/ActiveLinkHelper.php | 71 ++++++++++++++++++++--------
 1 file changed, 50 insertions(+), 21 deletions(-)

diff --git a/src/View/Helper/ActiveLinkHelper.php b/src/View/Helper/ActiveLinkHelper.php
index 04520eb..efc40e0 100644
--- a/src/View/Helper/ActiveLinkHelper.php
+++ b/src/View/Helper/ActiveLinkHelper.php
@@ -47,35 +47,35 @@ class ActiveLinkHelper extends Helper
         $target = $options['target'];
         unset($options['target']);
         if (is_string($target)) {
-            if (Router::normalize($currentUrl) == Router::normalize($target)) {
-                $options['class'] = $this->_addClass($options);
-
-                return $this->Html->link($title, $url, $options);
-            }
+            return $this->_linkFromStringTarget($currentUrl, $target, $title, $url, $options);
         }
-        if (is_array($target)) {
-            if (!array_key_exists('plugin', $currentUrl)) {
-                $currentUrl['plugin'] = false;
-            }
-            if (!array_key_exists('prefix', $currentUrl)) {
-                $currentUrl['prefix'] = false;
-            }
-            foreach ($target as $targetKey => $targetValue) {
-                if (is_array($targetValue)) {
-                    if (!in_array($currentUrl[$targetKey], $targetValue)) {
-                        return $this->Html->link($title, $url, $options);
-                    }
+        if (!is_array($target)) {
+            return $this->Html->link($title, $url, $options);
+        }
+
+        if (!array_key_exists('plugin', $currentUrl)) {
+            $currentUrl['plugin'] = false;
+        }
+        if (!array_key_exists('prefix', $currentUrl)) {
+            $currentUrl['prefix'] = false;
+        }
+        if (isset($target['or']) && $target['or']) {
+            foreach ($target['or'] as $singleTargetToMatch) {
+                if ($this->_matchesUrlFromArrayTarget($currentUrl, $singleTargetToMatch)) {
                     $options['class'] = $this->_addClass($options);
 
                     return $this->Html->link($title, $url, $options);
                 }
-                if (!array_key_exists($targetKey, $currentUrl) || $targetValue != $currentUrl[$targetKey]) {
-                    return $this->Html->link($title, $url, $options);
-                }
             }
-            $options['class'] = $this->_addClass($options);
+            return $this->Html->link($title, $url, $options);
         }
 
+        if (!$this->_matchesUrlFromArrayTarget($currentUrl, $target)) {
+            return $this->Html->link($title, $url, $options);
+        }
+
+        $options['class'] = $this->_addClass($options);
+
         return $this->Html->link($title, $url, $options);
     }
 
@@ -90,4 +90,33 @@ class ActiveLinkHelper extends Helper
         
         return array_key_exists('class', $providedOptions) ? $providedOptions['class'] . ' ' . $activeClass : $activeClass;
     }
+
+    protected function _linkFromStringTarget(array $current, string $targetString, string $title, array|string|null $url, array $options)
+    {
+        if (Router::normalize($current) == Router::normalize($targetString)) {
+            $options['class'] = $this->_addClass($options);
+
+            return $this->Html->link($title, $url, $options);
+        }
+
+        return $this->Html->link($title, $url, $options);
+    }
+
+    protected function _matchesUrlFromArrayTarget(array $current, array $targetUrl)
+    {
+        foreach ($targetUrl as $targetKey => $targetValue) {
+            if (is_array($targetValue)) {
+                if (!in_array($current[$targetKey], $targetValue)) {
+                    return false;
+                }
+
+                continue;
+            }
+            if (!array_key_exists($targetKey, $current) || $targetValue != $current[$targetKey]) {
+                return false;
+            }
+        }
+
+        return true;
+    }
 }