From 3921cd2534485a6c97b37c544ccc43d3e921fccc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tomasz=20B=C5=82a=C5=BCejczyk?= <blazejc@icm.edu.pl>
Date: Mon, 1 Jun 2026 17:50:08 +0200
Subject: [PATCH] Closes #3310: Fix error when trying to open baner link.

---
 .../MessagesAndBannersFragment.java                 |  7 -------
 .../src/main/webapp/text-messagesAndBanners.xhtml   | 13 +++++++------
 2 files changed, 7 insertions(+), 13 deletions(-)

diff --git a/fairchive-webapp/src/main/java/edu/harvard/iq/dataverse/bannersandmessages/MessagesAndBannersFragment.java b/fairchive-webapp/src/main/java/edu/harvard/iq/dataverse/bannersandmessages/MessagesAndBannersFragment.java
index b5a1320267..2d49033002 100644
--- a/fairchive-webapp/src/main/java/edu/harvard/iq/dataverse/bannersandmessages/MessagesAndBannersFragment.java
+++ b/fairchive-webapp/src/main/java/edu/harvard/iq/dataverse/bannersandmessages/MessagesAndBannersFragment.java
@@ -41,11 +41,4 @@ public List<ImageWithLinkDto> banners(Long dataverseId) {
                 dataverseSession.getLocaleCode());
     }
 
-    public void redirect(String link) throws IOException {
-
-        if (!link.startsWith("http")) {
-            link = "http://".concat(link);
-        }
-        FacesContext.getCurrentInstance().getExternalContext().redirect(link);
-    }
 }
diff --git a/fairchive-webapp/src/main/webapp/text-messagesAndBanners.xhtml b/fairchive-webapp/src/main/webapp/text-messagesAndBanners.xhtml
index b6de12664b..1039780d86 100644
--- a/fairchive-webapp/src/main/webapp/text-messagesAndBanners.xhtml
+++ b/fairchive-webapp/src/main/webapp/text-messagesAndBanners.xhtml
@@ -24,18 +24,19 @@
             <ui:repeat value="#{banners}" 
                        var="banner">
                 <div style="display: flex;align-content: center;">
-                  <h:commandLink style="text-align: center; display: block; margin: auto;  padding: 10px; "
-                                   target="_blank" 
-                                   disabled="#{banner.link == null}" 
-                                   action="#{MessagesAndBannersFragment.redirect(banner.link)}">
+                  <h:outputLink style="text-align: center; display: block; margin: auto;  padding: 10px; "
+                                value="#{banner.link}"
+                                target="_blank"
+                                disabled="#{banner.link == null}"
+                                title="#{bundle['opensInNewTab']}">
                          <p:graphicImage 
                                         stream="false"
                                         value="#{banner.image}"
                                         alt="#{bundle['header.banner.alt']} #{banner.link != null ? bundle['opensInNewTab'] : ''}"
                                         title="#{bundle['header.banner.alt']} #{banner.link != null ? bundle['opensInNewTab'] : ''}"
                         />
-                 </h:commandLink> 
-                </div>
+                  </h:outputLink>
+               </div>
             </ui:repeat>
         </div>
     </h:form>