]> git.koha-community.org Git - koha.git/commit
Bug 10641 - GetBooksellerWithLateOrders in C4::Bookseller.pm has some incoherences
authorroot <root@kenza-VirtualBox>
Thu, 25 Jul 2013 08:49:21 +0000 (10:49 +0200)
committerGalen Charlton <gmc@esilibrary.com>
Wed, 9 Oct 2013 04:02:47 +0000 (04:02 +0000)
commit9620d2d249577c460d272d4413119d3f43de33cf
tree0f8d9d1e978745726911e6dac7f93c06fc208517
parentb8dbff02830429c511d860782e956e579be4bec4
Bug 10641 - GetBooksellerWithLateOrders in C4::Bookseller.pm has some incoherences

This patch fixes some incoherences of the routine
GetBooksellerWithOrders().

Now it considers the field $estimateddeliverydateto and it replaces it
by now() only if it is undef.

Also, it doesn't test if $aqbookseller.deliverytime is not Null anymore
but if $deliverytime = null or undef, it replaces it by 0.

It also verifies if $delay is >= 0 and return undef if it is a negative
value.

To Test:
Before, this routine sorts out the BookSellerWithLateOrders. If a
bookseller did not specify a deliverytime, it would never appears in
the list of LateOrders.  Moreover, if the field "Estimated delivery
date to" was specified, it didn't take care of the value and it
returns the late order up to today's date.

Now, the returned list considers all the fields give and if the
delivery time of the bookseller is not specified, it calculates the
late orders as if the deliverytime is 0.  By default, all booksellers
which have orders in late until today are listed unless "estimated
delivery date to" is specified.

prove t/db_dependent/Bookseller.t
t/db_dependent/Bookseller.t ..
[Some warnings about uninitialized values]
WARNING: GetBooksellerWithLateOrders is called with a negative value at C4/Bookseller.pm line 135.
t/db_dependent/Bookseller.t .. ok
All tests successful.

Signed-off-by: Srdjan <srdjan@catalyst.net.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
All tests and QA script pass.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
C4/Bookseller.pm
t/db_dependent/Bookseller.t