From c7beee4e7ca4eea96a67dab7d3901a3c652b78b2 Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Tue, 28 Mar 2017 16:39:25 +0200 Subject: [PATCH] Bug 12913: [QA Follow-up] Reword 'change item status' and add zero status Content-Type: text/plain; charset=utf-8 The zero status should be added in @notforloans. It is a legitimate status for available books. The zero status description can be translated in the template. The 'Change item status' is confusing. We mean that we saw a notforloan status that we do not expect (read: does not exist in the table). Signed-off-by: Marcel de Rooy --- koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt | 14 ++++++++++---- tools/inventory.pl | 7 +++++-- 2 files changed, 15 insertions(+), 6 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt index a76fdef..650a510 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt @@ -177,12 +177,18 @@ $(document).ready(function(){ [% status.fieldname %] @@ -293,7 +299,7 @@ $(document).ready(function(){ [% ELSIF result.problem == 'missingitem' %]

Item missing

[% ELSIF result.problem == 'changestatus' %] -

Change item status

+

Unknown not-for-loan status

[% ELSIF result.problem == 'not_scanned' %]

Item should have been scanned

[% END %] diff --git a/tools/inventory.pl b/tools/inventory.pl index be57303..731ad77 100755 --- a/tools/inventory.pl +++ b/tools/inventory.pl @@ -94,11 +94,14 @@ for my $statfield (qw/items.notforloan items.itemlost items.withdrawn items.dama $hash->{authcode} = $mss->count ? $mss->next->authorised_value : undef; if ($hash->{authcode}){ my $arr = GetAuthorisedValues($hash->{authcode}); - $hash->{values} = $arr; - push @$statuses, $hash; if ( $statfield eq 'items.notforloan') { + # Add notforloan == 0 to the list of possible notforloan statuses + # The lib value is replaced in the template + push @$arr, { authorised_value => 0, id => 'stat0' , lib => 'ignore' }; @notforloans = map { $_->{'authorised_value'} } @$arr; } + $hash->{values} = $arr; + push @$statuses, $hash; } } -- 2.1.4