From d463a1b3473106a0b501d6247afdc0f7f84a4ff4 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Thu, 2 Aug 2018 11:47:24 -0300 Subject: [PATCH] Bug 21149: Fix superlibrarian creation in onboarding See bug 21087 for more details. Test plan: Drop your DB Start the installation process Use the onboarding process to install missing data The superlibrarian created during this process should be created sucessfully and allowed to login Signed-off-by: Owen Leonard Signed-off-by: Martin Renvoize Signed-off-by: Nick Clemens --- installer/onboarding.pl | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/installer/onboarding.pl b/installer/onboarding.pl index 930c4aad26..ceee2e47ea 100755 --- a/installer/onboarding.pl +++ b/installer/onboarding.pl @@ -173,12 +173,10 @@ if ( $step == 3 ) { branchcode => scalar $input->param('libraries'), categorycode => scalar $input->param('categorycode_entry'), userid => scalar $input->param('userid'), - password => scalar $input->param('password'), - password2 => scalar $input->param('password2'), privacy => "default", address => "", city => "", - flags => 1, # Will be superlibrarian + flags => 1, # Will be superlibrarian }; my $patron_category = @@ -187,7 +185,8 @@ if ( $step == 3 ) { $patron_category->get_expiry_date( $patron_data->{dateenrolled} ); eval { - Koha::Patron->new($patron_data)->store; + my $patron = Koha::Patron->new($patron_data)->store; + $patron->update_password($patron->userid, $firstpassword); }; #Error handling checking if the patron was created successfully @@ -195,6 +194,7 @@ if ( $step == 3 ) { push @messages, { code => 'success_on_insert_patron' }; } else { + warn $@; push @messages, { code => 'error_on_insert_patron' }; } } -- 2.39.5