[Koha-bugs] [Bug 6506] deleting patron history missing errors

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Tue Apr 9 12:12:32 CEST 2013


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

Jonathan Druart <jonathan.druart at biblibre.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA
                 CC|                            |jonathan.druart at biblibre.co
                   |                            |m

--- Comment #10 from Jonathan Druart <jonathan.druart at biblibre.com> ---
QA comment:

1/ In C4::Circ::AnonymiseIssueHistory: why don't return the error after
checking the syspref's values?
The query will raise an error only if AnonymousPatron=0 and OpacPrivacy=1 isn't
it?

2/ Maybe it could be better to raise an error (die) and catch it (eval) instead
of to return another variable which should be checked for each call.

3/ There is no css class error (the error is displayed without css style).

1 and 2 for discussion.
Marked as Failed QA for 3.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.


More information about the Koha-bugs mailing list