From 7e16e85e4f3551479e1fc8f014becc7049f47c81 Mon Sep 17 00:00:00 2001
From: Jake Deery <jake.deery@openfifth.co.uk>
Date: Tue, 3 Mar 2026 13:31:07 +0000
Subject: [PATCH] Bug 41976: LinkWrapper no longer mangles links

This patch fixes a bug where, context-dependent, LinkWrapper can keep
referencing back to links that should be out of scope.

This was causing odd behaviour whilst writing Bug 14962 - namely, links
returning as the same value in a table of multiple rows due to
LinkWrapper treating linkData.href as one and the same for each row.

TO TEST:
== APPLY PATCH ==
a)  enable ERMModule syspref, go to /cgi-bin/koha/erm/erm.pl
b)  notice all links and buttons still work as expected
c)  go to /cgi-bin/koha/sip2/sip2.pl
d)  notice all links and buttons still work as expected
e)  go to /cgi-bin/koha/acquisition/vendors
f)  notice all links and buttons still work as expected
g)  enable PreservationModule syspref, go to
/cgi-bin/koha/preservation/home.pl
h)  notice all links and buttons still work as expected
== SIGN OFF ==

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Andrew Fuerste Henry <andrew@bywatersolutions.com>
Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com>
(cherry picked from commit 12fdf33139389a5bbe2f053b49071950ed22173e)
Signed-off-by: Baptiste Wojtkowski <baptiste.wojtkowski@biblibre.com>
---
 .../prog/js/vue/components/LinkWrapper.vue    | 57 +++++++++++++++++++
 1 file changed, 57 insertions(+)
 create mode 100644 koha-tmpl/intranet-tmpl/prog/js/vue/components/LinkWrapper.vue

diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/LinkWrapper.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/LinkWrapper.vue
new file mode 100644
index 0000000000..df207c4140
--- /dev/null
+++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/LinkWrapper.vue
@@ -0,0 +1,57 @@
+<template>
+    <router-link
+        v-if="linkData && linkData.name"
+        :to="{
+            name: linkData.name,
+            params: formattedParams,
+        }"
+    >
+        <slot />
+    </router-link>
+    <a
+        v-else-if="linkData && linkData.href"
+        :href="formattedHref"
+        :class="{ disabled: linkData.disabled }"
+    >
+        <slot />
+    </a>
+    <slot v-else />
+</template>
+
+<script>
+import { ref } from "vue";
+export default {
+    props: {
+        linkData: Object,
+        resource: Object,
+    },
+    setup(props) {
+        const formattedHref = ref(props.linkData?.href);
+        const formattedParams = ref({});
+
+        if (props.linkData && props.linkData.params) {
+            Object.keys(props.linkData.params).forEach(key => {
+                formattedParams.value[key] =
+                    props.resource[props.linkData.params[key]];
+            });
+        }
+
+        if (props.linkData?.href && props.linkData.params) {
+            formattedHref.value += "?";
+            Object.keys(props.linkData.params).forEach(key => {
+                formattedHref.value += `${key}=${formattedParams.value[key]}&`;
+            });
+            formattedHref.value = formattedHref.value.slice(0, -1);
+        }
+        if (props.linkData?.href && props.linkData.slug) {
+            formattedHref.value += `/${props.resource[props.linkData.slug]}`;
+        }
+        return {
+            formattedHref,
+            formattedParams,
+        };
+    },
+};
+</script>
+
+<style></style>