From 4e882edf5642bdf0ea1cfc5805ba4794029406e7 Mon Sep 17 00:00:00 2001
From: Elaine Osbourn <12600440+kittysquee@users.noreply.github.com>
Date: Tue, 9 Nov 2021 11:43:12 +0000
Subject: [PATCH] Stops blank target being set

With no target being defined (`target=""`) we end up with invalid HTML, this ensures this doesn't happen.
---
 app/views/cookies_eu/_consent_banner.html.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/cookies_eu/_consent_banner.html.erb b/app/views/cookies_eu/_consent_banner.html.erb
index ae6ac5b..100e4c1 100644
--- a/app/views/cookies_eu/_consent_banner.html.erb
+++ b/app/views/cookies_eu/_consent_banner.html.erb
@@ -5,7 +5,7 @@
       <span class="cookies-eu-button-holder">
       <button class="cookies-eu-ok js-cookies-eu-ok"> <%= t('cookies_eu.ok') %> </button>
       <% if defined?(link).present? %>
-        <a href="<%= link %>" class="cookies-eu-link" target="<%= defined?(target).present? ? target : '' %>"> <%= t('cookies_eu.learn_more') %> </a>
+        <a href="<%= link %>" class="cookies-eu-link" <% if defined?(target).present? %>target="<%= target %>"<% end %> <%= t('cookies_eu.learn_more') %> </a>
       <% end %>
       </span>
     </div>