gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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>
This commit is contained in:
Raphael Straub 2024-05-02 09:19:32 +00:00 committed by Katrin Fischer
parent 356d3226f9
commit 3c6280f4a2
No known key found for this signature in database
GPG Key ID: 0EF6E2C03357A834
1 changed files with 19 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

46
Koha/Auth/Client.pm
View File

@ -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});
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 $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;