From 1adfc4ef01b3aea327ae575eab9747950ec4da87 Mon Sep 17 00:00:00 2001
From: Tony Air <tony@twma.pro>
Date: Wed, 13 Dec 2023 01:16:35 +0200
Subject: [PATCH] IMPR: RedirectorPage open in new window

---
 _config/base-extensions.yml        |  4 ++++
 templates/Includes/NavItem_link.ss | 12 ++++++------
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/_config/base-extensions.yml b/_config/base-extensions.yml
index da60caa..99e7db8 100755
--- a/_config/base-extensions.yml
+++ b/_config/base-extensions.yml
@@ -58,3 +58,7 @@ Page:
     - A2nt\CMSNiceties\Models\TeamMember
   extensions:
     - DNADesign\Elemental\Extensions\ElementalPageExtension
+
+SilverStripe\CMS\Model\RedirectorPage:
+  extensions:
+    - A2nt\CMSNiceties\Extensions\RedirectorPageEx
diff --git a/templates/Includes/NavItem_link.ss b/templates/Includes/NavItem_link.ss
index c3e1900..433f273 100755
--- a/templates/Includes/NavItem_link.ss
+++ b/templates/Includes/NavItem_link.ss
@@ -6,21 +6,21 @@
         $ExtraClass
         {$LinkClass}
         <% if $RedirectionType = 'External' || $ExternalURL || $OpenInNewWindow %>
-            external
+            legacy
         <% end_if %>
-        <% if $isCurrent || $isSection %>active <% end_if %>
+        <% if $isCurrent || $isSection %>active<% end_if %>
         <% if $isSection %>section<% end_if %>
     "
-    <% if $RedirectionType = 'External' || $OpenInNewWindow %>
+    <% if $OpenInNewWindow %>
         rel="noreferrer"
         target="_blank"
     <% end_if %>
 
     data-text="{$MenuTitle.XML}"
 >
-	<% if $BlockIcon %>
-	    <i class="fa-icon $BlockIcon"></i>
-	<% end_if %>
+    <% if $BlockIcon %>
+        <i class="fa-icon $BlockIcon"></i>
+    <% end_if %>
     $MenuTitle.XML
     <% if $isCurrent || $isSection %><i class="visually-hidden">(current)</i><% end_if %>
 </a>