From 3c6280f4a2fcfa1dd7da5c215854ea840c9a09f0 Mon Sep 17 00:00:00 2001
From: Raphael Straub <raphael.straub@kit.edu>
Date: Thu, 2 May 2024 09:19:32 +0000
Subject: [PATCH] Bug 36503: (follow-up) Use the call method of Koha::Plugins

This removes the option to call multiple plugins in a prioritized order,
but this is not needed if there is only one plugin that uses this hook.

Sponsored-by: Karlsruhe Institute of Technology (KIT)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
---
 Koha/Auth/Client.pm | 48 +++++++++++++++++++--------------------------
 1 file changed, 20 insertions(+), 28 deletions(-)

diff --git a/Koha/Auth/Client.pm b/Koha/Auth/Client.pm
index d51f400bba..bd6a515521 100644
--- a/Koha/Auth/Client.pm
+++ b/Koha/Auth/Client.pm
@@ -60,38 +60,30 @@ sub get_user {
     my $interface     = $params->{interface};
     my $config        = $params->{config};
 
-    my $provider = Koha::Auth::Identity::Providers->search({ code => $provider_code })->next;
+    my $provider = Koha::Auth::Identity::Providers->search( { code => $provider_code } )->next;
 
-    my ( $mapped_data, $patron ) = $self->_get_data_and_patron({ provider => $provider, data => $data, config => $config });
+    my ( $mapped_data, $patron ) =
+        $self->_get_data_and_patron( { provider => $provider, data => $data, config => $config } );
 
     $mapped_data //= {};
 
-    my $domain = $self->has_valid_domain_config({ provider => $provider, email => $mapped_data->{email}, interface => $interface});
-
-    # Call the plugin hook "auth_client_get_user" of all plugins in
-    # ascending priority.
-    if ( C4::Context->config('enable_plugins') ) {
-        my @plugins = Koha::Plugins->new()->GetPlugins(
-            {
-                method => 'auth_client_get_user',
-            }
-        );
-        @plugins = sort { $a->retrieve_data('priority') <=> $b->retrieve_data('priority') } @plugins;
-        my $args = {
-            provider    => $provider,
-            data        => $data,
-            config      => $config,
-            mapped_data => $mapped_data,
-            patron      => $patron,
-            domain      => $domain,
-        };
-        foreach my $plugin (@plugins) {
-            $plugin->auth_client_get_user($args);
-        }
-        $mapped_data = $args->{'mapped_data'};
-        $patron      = $args->{'patron'};
-        $domain      = $args->{'domain'};
-    }
+    my $domain = $self->has_valid_domain_config(
+        { provider => $provider, email => $mapped_data->{email}, interface => $interface } );
+
+    my $args = {
+        provider    => $provider,
+        data        => $data,
+        config      => $config,
+        mapped_data => $mapped_data,
+        patron      => $patron,
+        domain      => $domain,
+    };
+
+    # Call the plugin hook "auth_client_get_user" of all plugins.
+    Koha::Plugins->call( 'auth_client_get_user', $args );
+    $mapped_data = $args->{'mapped_data'};
+    $patron      = $args->{'patron'};
+    $domain      = $args->{'domain'};
 
     $patron->set($mapped_data)->store if $patron && $domain->update_on_auth;
 
-- 
2.39.5