From fbd640fa326a838d6406a333bfdfe4d618190d98 Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Thu, 10 Oct 2019 16:38:24 +0100 Subject: [PATCH] Bug 23049: Update 'HE' to 'RESERVE_EXPIRED' for consistency MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit This patch updates all cases where debit_type_code was 'HE' to 'RESERVE_EXPIRED' which is more informative and consistent with the new style guidelines. Signed-off-by: Séverine QUEUNE Signed-off-by: Marcel de Rooy Signed-off-by: Martin Renvoize --- Koha/Account.pm | 6 +++--- Koha/Hold.pm | 2 +- installer/data/mysql/account_debit_types.sql | 4 ++-- installer/data/mysql/atomicupdate/bug_23049_debit.perl | 9 ++++++++- .../data/mysql/en/mandatory/account_debit_types.sql | 4 ++-- koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc | 2 +- .../opac-tmpl/bootstrap/en/includes/account-table.inc | 2 +- 7 files changed, 18 insertions(+), 11 deletions(-) diff --git a/Koha/Account.pm b/Koha/Account.pm index 183928a152..2c8c4a1e7d 100644 --- a/Koha/Account.pm +++ b/Koha/Account.pm @@ -446,7 +446,7 @@ my $debit_line = Koha::Account->new({ patron_id => $patron_id })->add_debit( $debit_type can be any of: - ACCOUNT - ACCOUNT_RENEW - - hold_expired + - RESERVE_EXPIRED - lost_item - sundry - new_card @@ -720,7 +720,7 @@ our $offset_type = { 'RENT_DAILY_RENEW' => 'Rental Fee', 'overdue' => 'OVERDUE', 'manual_debit' => 'Manual Debit', - 'hold_expired' => 'Hold Expired' + 'RESERVE_EXPIRED' => 'Hold Expired' }; =head3 $account_type_credit @@ -742,7 +742,7 @@ our $account_type_credit = { our $account_type_debit = { 'ACCOUNT' => 'ACCOUNT', 'ACCOUNT_RENEW' => 'ACCOUNT_RENEW', - 'hold_expired' => 'HE', + 'RESERVE_EXPIRED' => 'RESERVE_EXPIRED', 'lost_item' => 'LOST', 'sundry' => 'M', 'new_card' => 'N', diff --git a/Koha/Hold.pm b/Koha/Hold.pm index 05afea8591..e2308dbd1f 100644 --- a/Koha/Hold.pm +++ b/Koha/Hold.pm @@ -373,7 +373,7 @@ sub cancel { user_id => C4::Context->userenv ? C4::Context->userenv->{'number'} : undef, interface => C4::Context->interface, library_id => C4::Context->userenv ? C4::Context->userenv->{'branch'} : undef, - type => 'hold_expired', + type => 'RESERVE_EXPIRED', item_id => $self->itemnumber } ); diff --git a/installer/data/mysql/account_debit_types.sql b/installer/data/mysql/account_debit_types.sql index 8f7d70a289..5176aa9253 100644 --- a/installer/data/mysql/account_debit_types.sql +++ b/installer/data/mysql/account_debit_types.sql @@ -1,7 +1,6 @@ INSERT INTO account_debit_types ( code, description, can_be_added_manually, default_amount, is_system ) VALUES ('ACCOUNT', 'Account creation fee', 0, NULL, 1), ('ACCOUNT_RENEW', 'Account renewal fee', 0, NULL, 1), -('HE', '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), @@ -11,4 +10,5 @@ INSERT INTO account_debit_types ( code, description, can_be_added_manually, defa ('RENT_DAILY', 'Daily rental fee', 0, NULL, 1), ('RENT_RENEW', 'Renewal of rental item', 0, NULL, 1), ('RENT_DAILY_RENEW', 'Rewewal of daily rental item', 0, NULL, 1), -('RESERVE', 'Hold fee', 0, NULL, 1); +('RESERVE', 'Hold fee', 0, NULL, 1), +('RESERVE_EXPIRED', 'Hold waiting too long', 0, NULL, 1); diff --git a/installer/data/mysql/atomicupdate/bug_23049_debit.perl b/installer/data/mysql/atomicupdate/bug_23049_debit.perl index 6ced71cdd8..d0e0a9cb70 100644 --- a/installer/data/mysql/atomicupdate/bug_23049_debit.perl +++ b/installer/data/mysql/atomicupdate/bug_23049_debit.perl @@ -40,7 +40,7 @@ if ( CheckVersion($DBversion) ) { VALUES ('ACCOUNT', 'Account creation fee', 0, NULL, 1), ('ACCOUNT_RENEW', 'Account renewal fee', 0, NULL, 1), - ('HE', 'Hold waiting too long', 0, NULL, 1), + ('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), @@ -68,6 +68,13 @@ if ( CheckVersion($DBversion) ) { } ); + # Update accountype 'HE' to 'RESERVE_EXPIRED' + $dbh->do( + qq{ + UPDATE accountlines SET accounttype = 'RESERVE_EXPIRED' WHERE accounttype = 'HE' + } + ); + # 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 8f7d70a289..5176aa9253 100644 --- a/installer/data/mysql/en/mandatory/account_debit_types.sql +++ b/installer/data/mysql/en/mandatory/account_debit_types.sql @@ -1,7 +1,6 @@ INSERT INTO account_debit_types ( code, description, can_be_added_manually, default_amount, is_system ) VALUES ('ACCOUNT', 'Account creation fee', 0, NULL, 1), ('ACCOUNT_RENEW', 'Account renewal fee', 0, NULL, 1), -('HE', '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), @@ -11,4 +10,5 @@ INSERT INTO account_debit_types ( code, description, can_be_added_manually, defa ('RENT_DAILY', 'Daily rental fee', 0, NULL, 1), ('RENT_RENEW', 'Renewal of rental item', 0, NULL, 1), ('RENT_DAILY_RENEW', 'Rewewal of daily rental item', 0, NULL, 1), -('RESERVE', 'Hold fee', 0, NULL, 1); +('RESERVE', 'Hold fee', 0, NULL, 1), +('RESERVE_EXPIRED', 'Hold waiting too long', 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 8dbe0feed1..41a9a14895 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc @@ -15,7 +15,6 @@ [%- SWITCH account.debit_type -%] [%- CASE 'ACCOUNT' -%]Account creation fee [%- CASE 'ACCOUNT_RENEW' -%]Account renewal fee - [%- CASE 'HE' -%]Hold waiting too long [%- CASE 'LOST' -%]Lost item [%- CASE 'M' -%]Sundry [%- CASE 'N' -%]New card @@ -26,6 +25,7 @@ [%- CASE 'RENT_RENEW' -%]Renewal of rental item [%- CASE 'RENT_DAILY_RENEW' -%]Rewewal of daily rental item [%- CASE 'RESERVE' -%]Hold fee + [%- CASE 'RESERVE_EXPIRED' -%]Hold waiting too long [%- CASE -%][% account.debit_type.description | html %] [%- END -%] [%- END -%] 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 d4f16c29b1..354badbb07 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/account-table.inc +++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/account-table.inc @@ -186,7 +186,6 @@ [%- SWITCH account.debit_type -%] [%- CASE 'ACCOUNT' -%]Account creation fee [%- CASE 'ACCOUNT_RENEW' -%]Account renewal fee - [%- CASE 'HE' -%]Hold waiting too long [%- CASE 'LOST' -%]Lost item [%- CASE 'M' -%]Sundry [%- CASE 'N' -%]New card @@ -197,6 +196,7 @@ [%- CASE 'RENT_RENEW' -%]Renewal of rental item [%- CASE 'RENT_DAILY_RENEW' -%]Rewewal of daily rental item [%- CASE 'RESERVE' -%]Hold fee + [%- CASE 'RESERVE_EXPIRED' -%]Hold waiting too long [%- CASE -%][% account.debit_type.description | html %] [%- END -%] [%- END -%] -- 2.39.5