From 863a961dcb1dc38ae0a928d07eba150e44a35f0b Mon Sep 17 00:00:00 2001
From: Nick <nick@bywatersolutions.com>
Date: Thu, 17 Oct 2019 15:26:18 +0000
Subject: [PATCH] Bug 23836: Don't forward form tracklinks if not tracking

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
---
 opac/tracklinks.pl | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/opac/tracklinks.pl b/opac/tracklinks.pl
index 809037ad4e..80794cb50d 100755
--- a/opac/tracklinks.pl
+++ b/opac/tracklinks.pl
@@ -33,9 +33,10 @@ my $uri = $cgi->param('uri') || '';
 my $biblionumber = $cgi->param('biblionumber') || 0;
 my $itemnumber   = $cgi->param('itemnumber')   || 0;
 
+my $tracking_method = C4::Context->preference('TrackClicks');
+output_error( $cgi, '404' ) unless $tracking_method;
 my $tracker = Koha::Linktracker->new(
-    { trackingmethod => C4::Context->preference('TrackClicks') } );
-
+    { trackingmethod => $tracking_method } );
 if ($uri && ($biblionumber || $itemnumber) ) {
     my $borrowernumber = 0;
 
-- 
2.39.5