Bug 10028: fix how ModReceiveOrder() calls NewOrder()
authorJonathan Druart <jonathan.druart@biblibre.com>
Thu, 11 Apr 2013 10:00:45 +0000 (12:00 +0200)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Fri, 13 Sep 2013 13:41:40 +0000 (10:41 -0300)
commit27f96f606c90f85756b290e1a832377f5465bf40
tree612615e595fb854e6794c49b6ea4020469898427
parenta85f9e1928a89e5c8f01983f1b853b090ec8cd1e
Bug 10028: fix how ModReceiveOrder() calls NewOrder()

In C4::Acquisition::ModReceiveOrder, a call to NewOrder is badly used.

NewOrder returns ($basketno, $ordernumber) but in ModReceiveOrder the
ordernumber is got with
  my $ordernumber = NewOrder( $args );

It works because:
sub t{
    return ("a", "b");
}
my $a = t();
say $a;

Will display 'b'.

But it is not really clear.

Test plan:
Check that there is no regression for partial receives.

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit 29689de399a8927547c36e53461c7766ecc604f0)
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
C4/Acquisition.pm