Don't add invalid empty target element to navigation links (#16341)

src/OrchardCore.Modules/OrchardCore.Menu/Views/MenuItemLink-LinkMenuItem.cshtml

@@ -16,7 +16,11 @@
         url = Url.Content(linkMenuItemPart.Url);
     }
 
-    tag.Attributes["target"] = linkMenuItemPart.Target;
+    if (!string.IsNullOrEmpty(linkMenuItemPart.Target))
+    {
+        tag.Attributes["target"] = linkMenuItemPart.Target;
+    }
+
     tag.Attributes["href"] = url;
     tag.InnerHtml.Append(contentItem.DisplayText);
 }

src/OrchardCore.Themes/TheAgencyTheme/Views/MenuItemLink-HtmlMenuItem.liquid

@@ -1,2 +1,5 @@
 {% assign link = Model.ContentItem.Content.HtmlMenuItemPart %}
-<a class="nav-link" href="{{ link.Url | href }}" target="{{ link.Target }}">{{ link.Html | raw }}</a>
+<a class="nav-link" href="{{ link.Url | href }}"
+    {% if link.Target != null and link.Target != "" %}
+        target="{{ link.Target }}"
+    {% endif %}>{{ link.Html | raw }}</a>

src/OrchardCore.Themes/TheAgencyTheme/Views/MenuItemLink-LinkMenuItem.liquid

@@ -1,6 +1,8 @@
 {% assign link = Model.ContentItem.Content.LinkMenuItemPart %}
-<a class="nav-link" href="{{ link.Url | href }}" target="{{ link.Target }}"
+<a class="nav-link" href="{{ link.Url | href }}"
+    {% if link.Target != null and link.Target != "" %}
+        target="{{ link.Target }}"
+    {% endif %}
     {% if link.Url contains "#" %}
         data-bs-target="{{ link.Url | split: '/' | last }}"
-    {% endif %}>
-{{ Model.ContentItem.DisplayText }}</a>
+    {% endif %}>{{ Model.ContentItem.DisplayText }}</a>

src/OrchardCore.Themes/TheBlogTheme/Views/MenuItemLink-HtmlMenuItem.liquid

@@ -3,5 +3,8 @@
 {% if Model.HasItems %}
     <a href="{{ link.Url | href }}" class="nav-link px-lg-3 py-3 py-lg-4 dropdown-toggle" data-bs-toggle="dropdown">{{ link.Html | raw }}<b class="caret"></b></a>
 {% else %}
-    <a class="nav-link px-lg-3 py-3 py-lg-4" href="{{ link.Url | href }}" target="{{ link.Target }}">{{ link.Html | raw }}</a>
+    <a class="nav-link px-lg-3 py-3 py-lg-4" href="{{ link.Url | href }}"
+        {% if link.Target != null and link.Target != "" %}
+            target="{{ link.Target }}"
+        {% endif %}>{{ link.Html | raw }}</a>
 {% endif %}

src/OrchardCore.Themes/TheBlogTheme/Views/MenuItemLink-LinkMenuItem.liquid

@@ -3,5 +3,8 @@
 {% if Model.HasItems %}
     <a href="{{ link.Url | href }}" class="nav-link px-lg-3 py-3 py-lg-4 dropdown-toggle" data-bs-toggle="dropdown">{{ Model.ContentItem.DisplayText }}<b class="caret"></b></a>
 {% else %}
-    <a class="nav-link px-lg-3 py-3 py-lg-4" href="{{ link.Url | href }}" target="{{ link.Target }}">{{ Model.ContentItem.DisplayText }}</a>
+    <a class="nav-link px-lg-3 py-3 py-lg-4" href="{{ link.Url | href }}"
+        {% if link.Target != null and link.Target != "" %}
+            target="{{ link.Target }}"
+        {% endif %}>{{ Model.ContentItem.DisplayText }}</a>
 {% endif %}

src/OrchardCore/OrchardCore.Navigation.Core/NavigationItemBuilder.cs

@@ -49,7 +49,7 @@ public NavigationItemBuilder Url(string url)
         public NavigationItemBuilder Target(string target)
         {
             _item.Target = target;
-            
+
             return this;
         }