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

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

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

--- Comment #6 from M. de Rooy <[email protected]> 2011-11-07 14:42:15 
UTC ---
As mentioned above, test plan is wanted.
Just looking at the code, the patch could change the flow through code. It
seems that the check on existence should be handled within the if block, and
provide the alternative in case it fails. Do I overlook something?

-- 
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.
_______________________________________________
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