Attachment 94323 Details for Bug 23049 – Bug 23049: Update 'N' to 'NEW_CARD' for consistency

[patch] Bug 23049: Update 'N' to 'NEW_CARD' for consistency

Bug-23049-Update-N-to-NEWCARD-for-consistency.patch (text/plain), 5.39 KB, created by Martin Renvoize (ashimema) on 2019-10-16 15:31:21 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2019-10-16 15:31:21 UTC

Size: 5.39 KB

patch

obsolete

>From 7fc6177cf2fd654bea451170e2bf74626cf8be06 Mon Sep 17 00:00:00 2001
>From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>Date: Thu, 10 Oct 2019 16:53:52 +0100
>Subject: [PATCH] Bug 23049: Update 'N' to 'NEW_CARD' for consistency
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>Signed-off-by: SÃ©verine QUEUNE <severine.queune@bulac.fr>
>---
> Koha/Account.pm                                          | 4 ++--
> installer/data/mysql/account_debit_types.sql             | 2 +-
> installer/data/mysql/atomicupdate/bug_23049_debit.perl   | 9 ++++++++-
> .../data/mysql/en/mandatory/account_debit_types.sql      | 2 +-
> koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc    | 2 +-
> .../opac-tmpl/bootstrap/en/includes/account-table.inc    | 2 +-
> 6 files changed, 14 insertions(+), 7 deletions(-)
>
>diff --git a/Koha/Account.pm b/Koha/Account.pm
>index 91f5879c1c..48ddd829b7 100644
>--- a/Koha/Account.pm
>+++ b/Koha/Account.pm
>@@ -449,7 +449,7 @@ $debit_type can be any of:
>   - RESERVE_EXPIRED
>   - LOST
>   - sundry
>-  - new_card
>+  - NEW_CARD
>   - OVERDUE
>   - PROCESSING
>   - RENT
>@@ -742,7 +742,7 @@ our $account_type_debit = {
>     'ACCOUNT_RENEW'    => 'ACCOUNT_RENEW',
>     'RESERVE_EXPIRED'  => 'RESERVE_EXPIRED',
>     'LOST_ITEM'        => 'LOST',
>-    'new_card'         => 'N',
>+    'NEW_CARD'         => 'NEW_CARD',
>     'OVERDUE'          => 'OVERDUE',
>     'PROCESSING'       => 'PROCESSING',
>     'RENT'             => 'RENT',
>diff --git a/installer/data/mysql/account_debit_types.sql b/installer/data/mysql/account_debit_types.sql
>index 5176aa9253..7391a1f9a6 100644
>--- a/installer/data/mysql/account_debit_types.sql
>+++ b/installer/data/mysql/account_debit_types.sql
>@@ -3,7 +3,7 @@ INSERT INTO account_debit_types ( code, description, can_be_added_manually, defa
> ('ACCOUNT_RENEW', 'Account renewal fee', 0, NULL, 1),
> ('LOST', 'Lost item', 1, NULL, 1),
> ('M', 'Manual fee', 1, NULL, 0),
>-('N', 'New card fee', 1, NULL, 1),
>+('NEW_CARD', 'New card fee', 1, NULL, 1),
> ('OVERDUE', 'Overdue fine', 0, NULL, 1),
> ('PROCESSING', 'Lost item processing fee', 0, NULL, 1),
> ('RENT', 'Rental fee', 0, NULL, 1),
>diff --git a/installer/data/mysql/atomicupdate/bug_23049_debit.perl b/installer/data/mysql/atomicupdate/bug_23049_debit.perl
>index b5f6c94d66..0fab0a600d 100644
>--- a/installer/data/mysql/atomicupdate/bug_23049_debit.perl
>+++ b/installer/data/mysql/atomicupdate/bug_23049_debit.perl
>@@ -43,7 +43,7 @@ if ( CheckVersion($DBversion) ) {
>               ('RESERVE_EXPIRED', 'Hold waiting too long', 0, NULL, 1),
>               ('LOST', 'Lost item', 1, NULL, 1),
>               ('M', 'Manual fee', 1, NULL, 0),
>-              ('N', 'New card fee', 1, NULL, 1),
>+              ('NEW_CARD', 'New card fee', 1, NULL, 1),
>               ('OVERDUE', 'Overdue fine', 0, NULL, 1),
>               ('PROCESSING', 'Lost item processing fee', 0, NULL, 1),
>               ('RENT', 'Rental fee', 0, NULL, 1),
>@@ -75,6 +75,13 @@ if ( CheckVersion($DBversion) ) {
>         }
>     );
> 
>+    # Update accountype 'N' to 'NEW_CARD'
>+    $dbh->do(
>+        qq{
>+          UPDATE accountlines SET accounttype = 'NEW_CARD' WHERE accounttype = 'N'
>+        }
>+    );
>+
>     # Moving MANUAL_INV to account_debit_types
>     $dbh->do(
>         qq{
>diff --git a/installer/data/mysql/en/mandatory/account_debit_types.sql b/installer/data/mysql/en/mandatory/account_debit_types.sql
>index 5176aa9253..7391a1f9a6 100644
>--- a/installer/data/mysql/en/mandatory/account_debit_types.sql
>+++ b/installer/data/mysql/en/mandatory/account_debit_types.sql
>@@ -3,7 +3,7 @@ INSERT INTO account_debit_types ( code, description, can_be_added_manually, defa
> ('ACCOUNT_RENEW', 'Account renewal fee', 0, NULL, 1),
> ('LOST', 'Lost item', 1, NULL, 1),
> ('M', 'Manual fee', 1, NULL, 0),
>-('N', 'New card fee', 1, NULL, 1),
>+('NEW_CARD', 'New card fee', 1, NULL, 1),
> ('OVERDUE', 'Overdue fine', 0, NULL, 1),
> ('PROCESSING', 'Lost item processing fee', 0, NULL, 1),
> ('RENT', 'Rental fee', 0, NULL, 1),
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc
>index 41a9a14895..b7f67b05da 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc
>@@ -17,7 +17,7 @@
>            [%- CASE 'ACCOUNT_RENEW'    -%]Account renewal fee
>            [%- CASE 'LOST'             -%]Lost item
>            [%- CASE 'M'                -%]Sundry
>-           [%- CASE 'N'                -%]New card
>+           [%- CASE 'NEW_CARD'         -%]New card
>            [%- CASE 'OVERDUE'          -%]Fine
>            [%- CASE 'PROCESSING'       -%]Lost item processing fee
>            [%- CASE 'RENT'             -%]Rental fee
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/account-table.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/account-table.inc
>index 354badbb07..b3496ed141 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/account-table.inc
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/account-table.inc
>@@ -188,7 +188,7 @@
>            [%- CASE 'ACCOUNT_RENEW'    -%]Account renewal fee
>            [%- CASE 'LOST'             -%]Lost item
>            [%- CASE 'M'                -%]Sundry
>-           [%- CASE 'N'                -%]New card
>+           [%- CASE 'NEW_CARD'         -%]New card
>            [%- CASE 'OVERDUE'          -%]Fine
>            [%- CASE 'PROCESSING'       -%]Lost item processing fee
>            [%- CASE 'RENT'             -%]Rental fee
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 23049: 93513 | 93514 | 93515 | 93516 | 93518 | 93519 | 93520 | 93521 | 93522 | 93523 | 93529 | 93557 | 93564 | 93565 | 93567 | 93568 | 93569 | 93570 | 93571 | 93572 | 93573 | 93574 | 93575 | 93576 | 93577 | 93578 | 93579 | 93960 | 93961 | 93962 | 93963 | 93964 | 93965 | 93966 | 93967 | 93968 | 93969 | 93970 | 93972 | 93973 | 93974 | 93975 | 93976 | 93977 | 93978 | 93979 | 93989 | 93990 | 93991 | 93992 | 93993 | 93994 | 93995 | 93996 | 93997 | 93998 | 93999 | 94000 | 94001 | 94002 | 94003 | 94004 | 94005 | 94006 | 94007 | 94008 | 94073 | 94074 | 94075 | 94076 | 94077 | 94078 | 94079 | 94080 | 94081 | 94082 | 94083 | 94084 | 94085 | 94086 | 94087 | 94088 | 94089 | 94090 | 94091 | 94092 | 94093 | 94111 | 94112 | 94113 | 94114 | 94115 | 94116 | 94117 | 94118 | 94119 | 94120 | 94121 | 94122 | 94123 | 94124 | 94125 | 94126 | 94127 | 94128 | 94129 | 94130 | 94131 | 94132 | 94133 | 94134 | 94135 | 94136 | 94137 | 94138 | 94139 | 94140 | 94141 | 94142 | 94143 | 94144 | 94145 | 94146 | 94147 | 94148 | 94149 | 94150 | 94151 | 94152 | 94165 | 94307 | 94308 | 94309 | 94310 | 94311 | 94312 | 94313 | 94314 | 94315 | 94316 | 94317 | 94318 | 94319 | 94320 | 94321 | 94323 | 94324 | 94325 | 94326 | 94327 | 94328 | 94329 | 94331 | 94332 | 94333 | 94334 | 94335 | 94336 | 94337 | 94338 | 94339 | 94340 | 94341 | 94342 | 94343 | 94344 | 94345 | 94346 | 94347 | 94348 | 94349 | 94350 | 94351 | 94352 | 94648