@@ -, +, @@ --- Koha/Schema/Result/CheckoutRenewal.pm | 16 +++++++++++----- .../atomicupdate/bug_30642-add_renewal_type.pl | 4 ++-- installer/data/mysql/kohastructure.sql | 2 +- .../prog/en/includes/str/checkout_renewals.inc | 6 ------ .../prog/en/modules/catalogue/moredetail.tt | 1 - .../prog/js/checkout_renewals_modal.js | 8 ++++++-- 6 files changed, 20 insertions(+), 17 deletions(-) delete mode 100644 koha-tmpl/intranet-tmpl/prog/en/includes/str/checkout_renewals.inc --- a/Koha/Schema/Result/CheckoutRenewal.pm +++ a/Koha/Schema/Result/CheckoutRenewal.pm @@ -71,9 +71,10 @@ the date and time the renewal took place =head2 renewal_type - data_type: 'varchar' + data_type: 'enum' + default_value: 'Manual' + extra: {list => ["Automatic","Manual"]} is_nullable: 0 - size: 9 whether the renewal was an automatic or manual renewal @@ -98,7 +99,12 @@ __PACKAGE__->add_columns( is_nullable => 0, }, "renewal_type", - { data_type => "varchar", is_nullable => 0, size => 9 }, + { + data_type => "enum", + default_value => "Manual", + extra => { list => ["Automatic", "Manual"] }, + is_nullable => 0, + }, ); =head1 PRIMARY KEY @@ -136,8 +142,8 @@ __PACKAGE__->belongs_to( ); -# Created by DBIx::Class::Schema::Loader v0.07049 @ 2022-12-06 16:44:53 -# DO NOT MODIFY THIS OR ANYTHING ABOVE! md5sum:BcbN0Iceh09H2DWEA6CDwA +# Created by DBIx::Class::Schema::Loader v0.07049 @ 2022-12-08 10:49:16 +# DO NOT MODIFY THIS OR ANYTHING ABOVE! md5sum:Si1gkXWqpvt98YN0dO7vgw =head2 checkout --- a/installer/data/mysql/atomicupdate/bug_30642-add_renewal_type.pl +++ a/installer/data/mysql/atomicupdate/bug_30642-add_renewal_type.pl @@ -1,7 +1,7 @@ use Modern::Perl; return { - bug_number => "BUG_30642", + bug_number => "30642", description => "Record whether a renewal has been done manually or automatically.", up => sub { my ($args) = @_; @@ -9,7 +9,7 @@ return { if( !column_exists( 'checkout_renewals', 'renewal_type' ) ) { $dbh->do(q{ - ALTER TABLE checkout_renewals ADD COLUMN `renewal_type` varchar(9) NOT NULL AFTER `timestamp` + ALTER TABLE checkout_renewals ADD COLUMN `renewal_type` enum('Automatic', 'Manual') NOT NULL DEFAULT 'Manual' AFTER `timestamp` }); say $out "Added column 'checkout_renewals.column_name'"; --- a/installer/data/mysql/kohastructure.sql +++ a/installer/data/mysql/kohastructure.sql @@ -1711,7 +1711,7 @@ CREATE TABLE `checkout_renewals` ( `seen` tinyint(1) DEFAULT 0 COMMENT 'boolean denoting whether the item was present or not', `interface` varchar(16) NOT NULL COMMENT 'the interface this renewal took place on', `timestamp` timestamp NOT NULL DEFAULT current_timestamp() COMMENT 'the date and time the renewal took place', - `renewal_type` varchar(9) NOT NULL COMMENT 'whether the renewal was an automatic or manual renewal', + `renewal_type` enum('Automatic', 'Manual') NOT NULL DEFAULT 'Manual' COMMENT 'whether the renewal was an automatic or manual renewal', PRIMARY KEY (`renewal_id`), KEY `renewer_id` (`renewer_id`), CONSTRAINT `renewals_renewer_id` FOREIGN KEY (`renewer_id`) REFERENCES `borrowers` (`borrowernumber`) ON DELETE SET NULL ON UPDATE CASCADE --- a/koha-tmpl/intranet-tmpl/prog/en/includes/str/checkout_renewals.inc +++ a/koha-tmpl/intranet-tmpl/prog/en/includes/str/checkout_renewals.inc @@ -1,6 +0,0 @@ - - --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt @@ -482,7 +482,6 @@ [% MACRO jsinclude BLOCK %] [% INCLUDE 'catalog-strings.inc' %] [% INCLUDE 'modals/checkout_renewals.inc' %] - [% INCLUDE 'str/checkout_renewals.inc' %] [% INCLUDE 'js-date-format.inc' %] [% INCLUDE 'js-patron-format.inc' %] [% Asset.js("js/catalog.js") | $raw %] --- a/koha-tmpl/intranet-tmpl/prog/js/checkout_renewals_modal.js +++ a/koha-tmpl/intranet-tmpl/prog/js/checkout_renewals_modal.js @@ -10,7 +10,7 @@ $(document).ready(function(){ $('#checkoutRenewals #retrieving').show(); $.get({ 'url': '/api/v1/checkouts/'+checkoutID+'/renewals', 'headers': { 'x-koha-embed': 'renewer' } }, function(data) { if (data.length < renewals) { - $('#checkoutRenewals #incomplete').append(renewed_prop.format(data.length, renewals)).show(); + $('#checkoutRenewals #incomplete').append(__("Note: %s out of %s renewals have been logged").format(data.length, renewals)).show(); } var items = data.map(function(item) { return createLi(item); @@ -20,6 +20,10 @@ $(document).ready(function(){ }); }); function createLi(renewal) { - return '
  • ' + $datetime(renewal.timestamp) + ' ' + renewed + ' ' + $patron_to_html(renewal.renewer) + '' + renewed_type + ' ' + renewal.renewal_type + '
  • '; + if(renewal.renewal_type === "Manual"){ + return '
  • ' + $datetime(renewal.timestamp) + ' ' + __("Renewed by") + ' ' + $patron_to_html(renewal.renewer) + " " + __("manually") + '
  • '; + } else { + return '
  • ' + $datetime(renewal.timestamp) + ' ' + __("Renewal type:") + ' ' + __("Automatic") + '
  • '; + } } }); --