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

M. de Rooy <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA
                 CC|                            |[email protected]
   Patch complexity|---                         |Small patch
         QA Contact|                            |[email protected]

--- Comment #3 from M. de Rooy <[email protected]> ---
QA Comment:
Works as expected. Code looks good to me too.
Small sidenote: POD documentation of getitemtypeinfo was copied from
getitemtype. Still contains wrong procedure name. You touched that line. Could
you correct it when you look at the following point? 

Also note that the second parameter is optional. It is called more often than
from lists. If you do not test if $interface is defined, you will create a lot
of noise in the logs due to it being uninitialized. See the line with test
$interface eq 'opac' in C4/Koha.pm routine getitemtypeinfo.
For the last reason parking this report to Failed QA. Please correct or send a
follow-up. Thanks.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
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