From 056cbcaca9f9d857101ca33064508f0c2aa2ce0a 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 --- installer/onboarding.pl | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/installer/onboarding.pl b/installer/onboarding.pl index 930c4aa..ceee2e4 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.1.4