From 94ea16db26fafcfa4eb51e27234d335264e3acc1 Mon Sep 17 00:00:00 2001
From: Rich Lott / Artful Robot <code.commits@artfulrobot.uk>
Date: Mon, 4 Dec 2023 11:52:23 +0000
Subject: [PATCH] Fix missing blank href attr on <a> links in searchkit angular

---
 ext/search_kit/ang/crmSearchDisplay/colType/buttons.html | 2 +-
 ext/search_kit/ang/crmSearchDisplay/colType/field.html   | 2 +-
 ext/search_kit/ang/crmSearchDisplay/colType/image.html   | 2 +-
 ext/search_kit/ang/crmSearchDisplay/colType/links.html   | 2 +-
 ext/search_kit/ang/crmSearchDisplay/colType/menu.html    | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/ext/search_kit/ang/crmSearchDisplay/colType/buttons.html b/ext/search_kit/ang/crmSearchDisplay/colType/buttons.html
index 4dda35662c..14d26437f3 100644
--- a/ext/search_kit/ang/crmSearchDisplay/colType/buttons.html
+++ b/ext/search_kit/ang/crmSearchDisplay/colType/buttons.html
@@ -1,5 +1,5 @@
 <span ng-repeat="link in colData.links">
-  <a class="btn {{:: $ctrl.settings.columns[colIndex].size }} btn-{{:: link.style }}" target="{{:: link.target }}" ng-href="{{:: link.url }}" title="{{:: link.title }}" ng-click="$ctrl.onClickLink(link, row.key, $event)">
+  <a class="btn {{:: $ctrl.settings.columns[colIndex].size }} btn-{{:: link.style }}" target="{{:: link.target }}" ng-attr-href="{{ link.url || '' }}" title="{{:: link.title }}" ng-click="$ctrl.onClickLink(link, row.key, $event)">
     <i ng-if=":: link.icon" class="crm-i {{:: link.icon }}"></i>
     {{:: link.text }}
   </a>
diff --git a/ext/search_kit/ang/crmSearchDisplay/colType/field.html b/ext/search_kit/ang/crmSearchDisplay/colType/field.html
index a2ffc7293d..7e762244a2 100644
--- a/ext/search_kit/ang/crmSearchDisplay/colType/field.html
+++ b/ext/search_kit/ang/crmSearchDisplay/colType/field.html
@@ -15,7 +15,7 @@
 </span>
 <span ng-if="colData.links && !$ctrl.isEditing(rowIndex, colIndex)">
   <span ng-repeat="link in colData.links">
-    <a target="{{:: link.target }}" ng-href="{{:: link.url }}" title="{{:: link.title }}" class="{{:: link.style }} ng-click="$ctrl.onClickLink(link, row.key, $event)">
+    <a target="{{:: link.target }}" ng-attr-href="{{ link.url || '' }}" title="{{:: link.title }}" class="{{:: link.style }} ng-click="$ctrl.onClickLink(link, row.key, $event)">
       <i ng-if="colData.icons.left[$index]" class="crm-i {{:: colData.icons.left[$index] }}"></i>
       {{:: link.text }}<i ng-if="colData.icons.right[$index]" class="crm-i {{:: colData.icons.right[$index] }}"> </i></a><span ng-if="!$last">,
     </span>
diff --git a/ext/search_kit/ang/crmSearchDisplay/colType/image.html b/ext/search_kit/ang/crmSearchDisplay/colType/image.html
index c7d12feb48..232fa4c2bf 100644
--- a/ext/search_kit/ang/crmSearchDisplay/colType/image.html
+++ b/ext/search_kit/ang/crmSearchDisplay/colType/image.html
@@ -1,6 +1,6 @@
 <span ng-if="::colData.links && colData.img">
   <span ng-repeat="link in colData.links">
-    <a target="{{:: link.target }}" ng-href="{{:: link.url }}">
+    <a target="{{:: link.target }}" ng-attr-href="{{ link.url || '' }}">
       <img ng-src="{{:: colData.img.src }}" alt="{{:: colData.val }}" height="{{:: colData.img.height }}" width="{{:: colData.img.width }}"/>
     </a>
   </span>
diff --git a/ext/search_kit/ang/crmSearchDisplay/colType/links.html b/ext/search_kit/ang/crmSearchDisplay/colType/links.html
index 4575d2223b..83c07fde55 100644
--- a/ext/search_kit/ang/crmSearchDisplay/colType/links.html
+++ b/ext/search_kit/ang/crmSearchDisplay/colType/links.html
@@ -1,5 +1,5 @@
 <span ng-repeat="link in colData.links">
-  <a class="text-{{:: link.style }}" target="{{:: link.target }}" ng-href="{{:: link.url }}" title="{{:: link.title }}" ng-click="$ctrl.onClickLink(link, row.key, $event)">
+  <a class="text-{{:: link.style }}" target="{{:: link.target }}" ng-attr-href="{{ link.url || '' }}" title="{{:: link.title }}" ng-click="$ctrl.onClickLink(link, row.key, $event)">
     <i ng-if=":: link.icon" class="crm-i {{:: link.icon }}"></i>
     {{:: link.text }}
   </a>
diff --git a/ext/search_kit/ang/crmSearchDisplay/colType/menu.html b/ext/search_kit/ang/crmSearchDisplay/colType/menu.html
index d08e57c9d3..1a83c3d282 100644
--- a/ext/search_kit/ang/crmSearchDisplay/colType/menu.html
+++ b/ext/search_kit/ang/crmSearchDisplay/colType/menu.html
@@ -6,7 +6,7 @@
   </button>
   <ul class="dropdown-menu {{ $ctrl.settings.columns[colIndex].alignment === 'text-right' ? 'dropdown-menu-right' : '' }}" ng-if=":: colData.open">
     <li ng-repeat="link in colData.links" class="bg-{{:: link.style }}">
-      <a ng-href="{{:: link.url }}" target="{{:: link.target }}" title="{{:: link.title }}" ng-click="$ctrl.onClickLink(link, row.key, $event)">
+      <a ng-attr-href="{{ link.url || '' }}" target="{{:: link.target }}" title="{{:: link.title }}" ng-click="$ctrl.onClickLink(link, row.key, $event)">
         <i ng-if=":: link.icon" class="crm-i {{:: link.icon }}"></i>
         {{:: link.text }}
       </a>
-- 
2.25.1