[Koha-bugs] [Bug 9011] Add the ability to store the last patron to return an item

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9011

--- Comment #24 from Galen Charlton <gmcharlt at gmail.com> ---
Comment on attachment 20343
  --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=20343
Bug 9011 - QA Followup

Review of attachment 20343:
 --> (http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=9011&attachment=20343)
-----------------------------------------------------------------

::: koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt
@@ +214,4 @@
>  
>                  <li><span class="label">Last seen:</span>[% IF ( ITEM_DAT.datelastseen ) %][% ITEM_DAT.datelastseen | $KohaDates %] [%END %] </li>
>                  <li><span class="label">Last borrowed:</span>[% IF (ITEM_DAT.datelastborrowed ) %][% ITEM_DAT.datelastborrowed | $KohaDates %][% END %] </li>
> +                [% IF ( ITEM_DAT.last_returned_by && Koha.Preference('StoreLastBorrower') && !Koha.Preference('AnonymousPatron') ) %]<li><span class="label">Last returned by:</span> <a href="/cgi-bin/koha/circ/circulation.pl?borrowernumber=[% ITEM_DAT.last_returned_by %]">[% KohaBorrowers.BorrowernumberToCardnumber( ITEM_DAT.last_returned_by ) %]</a> </li>[% END %]

Is "Koha.Preference('AnonymousPatron')" what is meant, not
"!Koha.Preference('AnonymousPatron')"?

The AnonymousPatron system preference is not a Boolean, it's the borrowernumber
of the record to which anonymized transactions should be linked.

Consequently, I think the QA follow-up has exactly the opposite of the intended
effect.

-- 
You are receiving this mail because:
You are watching all bug changes.