Bug 13675: Do not set aqorders.budget_id to NULL
authorJonathan Druart <jonathan.druart@biblibre.com>
Fri, 6 Feb 2015 15:19:14 +0000 (16:19 +0100)
committerLiz Rea <wizzyrea@gmail.com>
Thu, 15 Oct 2015 22:28:42 +0000 (11:28 +1300)
commitfda6ea7c322fccf79b07bec04d68c9c79a8a382d
tree29b21131e6f16b72f9ed0be9203178c1653b83b8
parentd33fc314274b6e03608e95bf9d2a38a874882a8a
Bug 13675: Do not set aqorders.budget_id to NULL

C4::Acquisition::ModReceiveOrder updates the aqorders with
budget_id=NULL if no budget_id given in parameter.
Actually the same budget_id should be used.
In tests (especially t/db_dependent/Acquisition/TransferOrder.t),
ModReceiveOrder is not called with a budget_id param and set to NULL the
budget_id value.

test plan:
  prove t/db_dependent/Acquisition/TransferOrder.t
should return green

Note that this bug should not appear using the interface.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
The fix makes sense, and running
 $ prove t/db_dependent/Acq*
returns all green. koha-qa.pl also likes it.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
(cherry picked from commit 2df541712fd691ee817568ed9d14e94bba13424e)
Signed-off-by: Liz Rea <wizzyrea@gmail.com>
C4/Acquisition.pm