Attachment 150629 Details for Bug 33671 – Bug 33671: Fix for 30472 - Remove null-guarantors

[patch] Bug 33671: Fix for 30472 - Remove null-guarantors

Bug-33671-Fix-for-30472---Remove-null-guarantors.patch (text/plain), 1.75 KB, created by Marcel de Rooy on 2023-05-04 07:15:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2023-05-04 07:15:57 UTC

Size: 1.75 KB

patch

obsolete

>From d1c70f163beea31a97813195e6e5df49d69c26f5 Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Thu, 4 May 2023 06:41:19 +0000
>Subject: [PATCH] Bug 33671: Fix for 30472 - Remove null-guarantors
>Content-Type: text/plain; charset=utf-8
>
>These guarantors cannot be added. They are of no use.
>
>NOTE FOR QA/RM: I do not think that it is needed to add a new
>db rev for this change since it happens seldom. But if you think
>that it is, just copy it to atomicupdate/change bugno.
>
>Test plan:
>[1] Check your table definition and modify it:
>    alter table borrower_relationships modify column `guarantor_id` int(11) NULL;
>[2] Add an empty guarantor:
>    insert into borrower_relationships (guarantee_id,relationship) values (SOME_PATRON_ID,'father');
>[3] Run the dbrev and verify that the inserted record has been removed.
>[4] Check table definition with 'show create table borrower_relationships'.
>    You should see NOT NULL with guarantor_id.
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> installer/data/mysql/db_revs/220600048.pl | 5 +++++
> 1 file changed, 5 insertions(+)
>
>diff --git a/installer/data/mysql/db_revs/220600048.pl b/installer/data/mysql/db_revs/220600048.pl
>index 9c35ccec5d..8fb5e9c322 100755
>--- a/installer/data/mysql/db_revs/220600048.pl
>+++ b/installer/data/mysql/db_revs/220600048.pl
>@@ -6,6 +6,11 @@ return {
>     up => sub {
>         my ($args) = @_;
>         my ($dbh, $out) = @$args{qw(dbh out)};
>+
>+        # Delete 'empty' guarantors. No longer possible to add them via interface. Have no use.
>+        $dbh->do(q{
>+DELETE FROM borrower_relationships WHERE guarantor_id IS NULL
>+        });
>         $dbh->do(q{
> ALTER TABLE borrower_relationships CHANGE COLUMN guarantor_id guarantor_id int(11) NOT NULL
>         });
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 33671: 150629 | 150630 | 150824 | 150825 | 153367 | 153533 | 153536