[Koha-bugs] [Bug 5347] late order management

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Fri Jan 13 16:12:10 CET 2012


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

--- Comment #49 from Paul Poulain <paul.poulain at biblibre.com> 2012-01-13 15:12:10 UTC ---
QA comments:
1 what's the use of div.info in staff.css ? couldn't we use an existing class ?
2 there is a new sub in C4/Acquisitions.pm (AddClaim), but no unit test. I've
already pushed a patch with a new sub here and no unit test, so I won't refuse
this one, but that would be a plus.
* Other than this = good code, fixes some crapy things in previously existing
code, perlcritic OK, except for C4/Letter.pm that was not perlcritic-compliant
before the patch, so no regression here

not marking passed QA because or point 1 & 2

-- 
Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email
------- 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