Skip to content

Commit 7486abb

Browse files
committed
use List::Util::first to replace some greps
1 parent 2e681a9 commit 7486abb

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

lib/Dancer2/Plugin/Auth/Extensible.pm

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ use strict;
66
use warnings;
77
use Carp;
88
use Dancer2::Core::Types qw(ArrayRef Bool HashRef Int Str);
9+
use List::Util qw(first);
910
use Module::Runtime qw(use_module);
1011
use Scalar::Util;
1112
use Session::Token;
@@ -138,7 +139,7 @@ has realm_count => (
138139
sub realm {
139140
my ( $self, $name ) = @_;
140141
croak "realm name not provided" unless $name;
141-
my ($realm) = grep { $_->{name} eq $name } @{ $self->realms };
142+
my $realm = first { $_->{name} eq $name } @{ $self->realms };
142143
return $realm;
143144
}
144145

lib/Dancer2/Plugin/Auth/Extensible/Provider/Config.pm

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
package Dancer2::Plugin::Auth::Extensible::Provider::Config;
22

33
use Dancer2::Core::Types qw/ArrayRef/;
4-
4+
use List::Util qw/first/;
55
use Moo;
66
with "Dancer2::Plugin::Auth::Extensible::Role::Provider";
77
use namespace::clean;
@@ -86,7 +86,7 @@ sub authenticate_user {
8686
# fields defined for users will just get passed through.
8787
sub get_user_details {
8888
my ($self, $username) = @_;
89-
my ($user) = grep {
89+
my $user = first {
9090
$_->{user} eq $username
9191
} @{ $self->users };
9292
return $user;

0 commit comments

Comments
 (0)