https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14039

David Roberts <david.robe...@ptfs-europe.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Failed QA                   |Needs Signoff

--- Comment #9 from David Roberts <david.robe...@ptfs-europe.com> ---
Belatedly revisiting this. This patch was intended to only apply to the
checkout screen as this is where the customer wanted it to display, but if it
appears elsewhere then that isn't the end of the world. 

I've added a span to the title so that it can be hidden in CSS - I believe it
is better to deliver this with it enabled so that the user knows the
functionality is there, and can be easily hidden. It is much harder to inform
the user that it is possible to enable a function they don't know exists.

I've added the test to prevent errors it there is no borrower.title present in
the user record. 

Amended test plan:

1) Install this patch.
2) Ensure a user has a salutation (Mr, Dr, Miss etc) set, and checkout an item
to this user.
In the area above the barcode box, the patron's title should display, e.g:

Checking out to Mr John Doe (0001).

3) Add #patron-title { display: none; } to IntranetUserCSS
Check that the patron title no longer displays

4) With a user without a salutation, check that no errors display on checkout.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to