From be90637a694e60ce5ea8fd4f2b468b92b71f8b05 Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Wed, 24 Jan 2024 16:24:51 +0100
Subject: [PATCH] Bug 35890: Remove var loggedin

It is never used and add confusion

Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Wainui Witika-Park <wainuiwitikapark@catalyst.net.nz>
---
 C4/Auth.pm | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/C4/Auth.pm b/C4/Auth.pm
index 8adbc78173..edc5e3ba64 100644
--- a/C4/Auth.pm
+++ b/C4/Auth.pm
@@ -847,7 +847,6 @@ sub checkauth {
     _version_check( $type, $query );
 
     # state variables
-    my $loggedin = 0;
     my $auth_state = 'failed';
     my %info;
     my ( $userid, $cookie, $sessionID, $flags );
@@ -884,7 +883,6 @@ sub checkauth {
             -secure => ( C4::Context->https_enabled() ? 1 : 0 ),
             -sameSite => 'Lax',
         ));
-        $loggedin = 1;
     }
     elsif ( $emailaddress) {
         # the Google OpenID Connect passes an email address
@@ -1228,7 +1226,6 @@ sub checkauth {
                         my $domain = $branches->{$branchcode}->{'branchip'};
                         $domain =~ s|\.\*||g;
                         if ( $ip !~ /^$domain/ ) {
-                            $loggedin = 0;
                             $cookie = $cookie_mgr->replace_in_list( $cookie, $query->cookie(
                                 -name     => 'CGISESSID',
                                 -value    => '',
-- 
2.39.5