Bug 16892: Follow up use AddMember as per QA comment

Same Test plan as before.

Signed-off-by: Eugene Jose Espinoza <eugenegf@yahoo.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

diff --git a/opac/svc/auth/googleopenidconnect b/opac/svc/auth/googleopenidconnect
index eae7b34..c08b3c1 100755 (executable)
--- a/opac/svc/auth/googleopenidconnect
+++ b/opac/svc/auth/googleopenidconnect
@@ -192,7 +192,7 @@ elsif ( defined $query->param('code') ) {
                     my $categorycode = C4::Context->preference('GoogleOpenIDConnectDefaultCategory') // q{};
                     my $branchcode = C4::Context->preference('GoogleOpenIDConnectDefaultBranch') // q{};
                     my $password = undef;
-                    $borrower = Koha::Patron->new( {
+                    my $borrowernumber = C4::Members::AddMember(
                         cardnumber   => $cardnumber,
                         firstname    => $firstname,
                         surname      => $surname,
@@ -200,9 +200,10 @@ elsif ( defined $query->param('code') ) {
                         categorycode => $categorycode,
                         branchcode   => $branchcode,
                         userid       => $userid,
-                        password     => $password,
-                    } );
-                    $borrower->store();
+                        password     => $password
+                    );
+                    $borrower = Koha::Patrons->find( {
+                        borrowernumber => $borrowernumber } );
                 }
                 my ( $userid, $cookie, $session_id ) =
                   checkauth( $query, 1, {}, 'opac', $email );