From 16a7ac0fe71e74e28051cb5374923879c8c5213d 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>
(cherry picked from commit 0ce8cc4c05bc96503172018775ba574e41b40ecb)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
---
 C4/Auth.pm | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/C4/Auth.pm b/C4/Auth.pm
index c099dbc28c..027b16bbde 100644
--- a/C4/Auth.pm
+++ b/C4/Auth.pm
@@ -815,7 +815,6 @@ sub checkauth {
     _version_check( $type, $query );
 
     # state variables
-    my $loggedin = 0;
     my $auth_state = 'failed';
     my %info;
     my ( $userid, $cookie, $sessionID, $flags );
@@ -855,7 +854,6 @@ sub checkauth {
             -secure => ( C4::Context->https_enabled() ? 1 : 0 ),
             -sameSite => 'Lax',
         ));
-        $loggedin = 1;
     }
     elsif ( $emailaddress) {
         # the Google OpenID Connect passes an email address
@@ -1203,7 +1201,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