kohabug 2180 - improve display when fines block checkouts
If a patron's fine balance is over the limit set by the noissuecharges syspref, checkouts are blocked. However, this was not made very clear in the interface. Improved the display by doing the following: [1] If the fine balance is over the noissuecharges limit, use the "blocker" styling (i.e., make all of the text red). [2] Add wording to explicitly signal whether or not the fine balance is blocking charges. Documentation changes: new screenshots of blocked checkouts. Signed-off-by: Joshua Ferraro <jmf@liblime.com>
This commit is contained in:
parent
e613ef5a8c
commit
28fb7a6c19
|
|
@ -568,7 +568,8 @@ foreach $flag ( sort keys %$flags ) {
|
|||
if ( $flag eq 'CHARGES' ) {
|
||||
$template->param(
|
||||
charges => 'true',
|
||||
chargesmsg => $flags->{'CHARGES'}->{'message'}
|
||||
chargesmsg => $flags->{'CHARGES'}->{'message'},
|
||||
charges_is_blocker => 1
|
||||
);
|
||||
}
|
||||
if ( $flag eq 'CREDITS' ) {
|
||||
|
|
|
|||
|
|
@ -330,7 +330,15 @@ No patron matched <span class="ex"><!-- TMPL_VAR name="message" --></span>
|
|||
<!-- /TMPL_IF -->
|
||||
|
||||
<!-- TMPL_IF NAME="charges" -->
|
||||
<li><span class="circ-hlt">Fines:</span> Patron has <a href="/cgi-bin/koha/members/boraccount.pl?borrowernumber=<!-- TMPL_VAR name="borrowernumber" -->">Outstanding fines</a>.
|
||||
<!-- TMPL_IF NAME="charges_is_blocker" -->
|
||||
<li class="blocker">
|
||||
<!-- TMPL_ELSE -->
|
||||
<li>
|
||||
<!-- /TMPL_IF -->
|
||||
<span class="circ-hlt">Fines:</span> Patron has <a href="/cgi-bin/koha/members/boraccount.pl?borrowernumber=<!-- TMPL_VAR name="borrowernumber" -->">Outstanding fines</a>.
|
||||
<!-- TMPL_IF NAME="charges_is_blocker" -->
|
||||
Checkouts are blocked because fine balance is over the limit.
|
||||
<!-- /TMPL_IF -->
|
||||
Make <a href="/cgi-bin/koha/members/pay.pl?borrowernumber=<!-- TMPL_VAR name="borrowernumber" -->">Payment</a></li>
|
||||
<!-- /TMPL_IF -->
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue