From 1697659dd3e8ff022a2f5e935ff3e52f753a78cd Mon Sep 17 00:00:00 2001 From: Katrin Fischer Date: Mon, 2 Feb 2026 21:04:38 +0000 Subject: [PATCH] Bug 35617: Fix file permission and tidy - Fixed file permission - Make output use say_success - Clean-up comments and tidy Signed-off-by: Katrin Fischer --- .../atomicupdate/extend_identity_providers.pl | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) mode change 100644 => 100755 installer/data/mysql/atomicupdate/extend_identity_providers.pl diff --git a/installer/data/mysql/atomicupdate/extend_identity_providers.pl b/installer/data/mysql/atomicupdate/extend_identity_providers.pl old mode 100644 new mode 100755 index 222529ff8a1..5d2d232e09d --- a/installer/data/mysql/atomicupdate/extend_identity_providers.pl +++ b/installer/data/mysql/atomicupdate/extend_identity_providers.pl @@ -1,4 +1,5 @@ use Modern::Perl; +use Koha::Installer::Output qw(say_warning say_success say_info); return { bug_number => "35617", @@ -7,17 +8,14 @@ return { my ($args) = @_; my ( $dbh, $out ) = @$args{qw(dbh out)}; - # Do you stuffs here - # now is `matchpoint` enum('email','phone','userid','cardnumber') NOT NULL COMMENT - $dbh->do(q{ + $dbh->do( + q{ ALTER TABLE `identity_providers` MODIFY COLUMN `matchpoint` enum('email','phone','userid','cardnumber') NOT NULL COMMENT 'The field of the patron that will be used to match the user from the identity provider'; - }); - - # Print useful stuff here - # tables - say $out "Extend 'matchpoint' field in 'identity_providers'"; + } + ); + say_success( $out, "Add 'phone' to 'matchpoint' field in 'identity_providers'" ); }, }; -- 2.39.5