[Koha-bugs] [Bug 10641] GetBooksellerWithLateOrders in C4::Bookseller.pm has some incoherences

[bugzilla-daemon at bugs.koha-community.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jonathan.druart at biblibre.co
                   |                            |m

--- Comment #9 from Jonathan Druart <jonathan.druart at biblibre.com> ---
(In reply to Katrin Fischer from comment #7)
> Hi Kenza,
> 
> I think this would be a bit better as 2 patches - one for the changes
> Booksellers.pm and one for the tests, so you can compare the results of the
> tests before and after the change. Could you split the patch up?

Katrin,
Kenza already adds a report for testing this routine (bug 10528). In this
patch, a lot of lines are uncommented in order to remove previous FIXME. I am
not sure to understand the goal of your request.

However I think this patch needs a better test plan: the feature should be
tested in the interface.

-- 
You are receiving this mail because:
You are watching all bug changes.