Bugzilla – Attachment 128208 Details for
Bug 29457
Fee Cancellation records the wrong manager_id
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 29457: (QA follow-up) Report about checked manager_id's
Bug-29457-QA-follow-up-Report-about-checked-manage.patch (text/plain), 2.02 KB, created by
Marcel de Rooy
on 2021-12-03 10:00:30 UTC
(
hide
)
Description:
Bug 29457: (QA follow-up) Report about checked manager_id's
Filename:
MIME Type:
Creator:
Marcel de Rooy
Created:
2021-12-03 10:00:30 UTC
Size:
2.02 KB
patch
obsolete
>From f4eb618d1908c1962e568be363b5cd067b35d865 Mon Sep 17 00:00:00 2001 >From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >Date: Fri, 26 Nov 2021 07:53:44 +0000 >Subject: [PATCH] Bug 29457: (QA follow-up) Report about checked manager_id's >Content-Type: text/plain; charset=utf-8 > >We print the number of missing or incorrect manager_id's now. > >Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >--- > .../data/mysql/atomicupdate/bug_29457.pl | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > create mode 100755 installer/data/mysql/atomicupdate/bug_29457.pl > >diff --git a/installer/data/mysql/atomicupdate/bug_29457.pl b/installer/data/mysql/atomicupdate/bug_29457.pl >new file mode 100755 >index 0000000000..23e9d13016 >--- /dev/null >+++ b/installer/data/mysql/atomicupdate/bug_29457.pl >@@ -0,0 +1,19 @@ >+use Modern::Perl; >+ >+return { >+ bug_number => "29457", >+ description => "Check manager_id in cancelled account lines", >+ up => sub { >+ my ($args) = @_; >+ my ($dbh, $out) = @$args{qw(dbh out)}; >+ my ( $count_empty ) = $dbh->selectrow_array( >+ q{SELECT COUNT(*) FROM accountlines WHERE credit_type_code = 'CANCELLATION' AND manager_id IS NULL} >+ ); >+ my ( $count_bad ) = $dbh->selectrow_array( >+ q{SELECT COUNT(*) FROM accountlines WHERE credit_type_code = 'CANCELLATION' AND manager_id IS NOT NULL} >+ ); >+ say $out "Note: We counted $count_empty cancellations in accountlines with a missing manager_id. We regret that, but it can't hurt. This release/upgrade fixed the bug causing it." if $count_empty; >+ say $out "Warning: We counted $count_bad cancellations in accountlines with an incorrect manager_id. This does not really hurt, but if you want to clear or even correct the data, please go to the following URL: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29457#c31" if $count_bad; >+ say $out "Your data did not contain missing or incorrect manager_id's in cancelled account lines" unless $count_empty or $count_bad; >+ }, >+} >-- >2.20.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 29457
:
127527
|
127596
|
127739
|
128024
|
128025
|
128030
|
128031
|
128032
|
128207
|
128208
|
128649
|
128650
|
128651