gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 11755: ensure that SearchOrders returns booksellerid 

The SearchOrders routine should return the booksellerid and this
patch adds it.

This fixes several problems:

[1] The link to the vendor on the order receive page breadcrumbs
    was broken.
[2] The tax calculation in finishreceive.pl didn't run.
[3] The item booksellerid field never got updated during
    receipt.

Booksellerid was returned before bug 10723.

Quick test plan:
Go on orderreceive.pl and verify that the vendor link is correct.

Followed test plan. Vendor link is now correct.
Signed-off-by: Marc Véron <veron@veron.ch>
Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

(cherry picked from commit 1e0375e91c)

Conflicts:
	t/db_dependent/Acquisition.t
This commit is contained in:
Jonathan Druart 2014-03-05 12:06:47 +01:00 committed by Fridolin Somers
parent d726bf355c
commit 8f765aa560
2 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
C4/Acquisition.pm
View File

@ -1664,6 +1664,7 @@ sub SearchOrders {
biblioitems.isbn,
biblioitems.biblioitemnumber,
aqbasket.authorisedby,
aqbasket.booksellerid,
aqbasket.closedate,
aqbasket.creationdate,
aqbasket.basketname,

3
t/db_dependent/Acquisition.t
View File

@ -8,7 +8,7 @@ use POSIX qw(strftime);
use C4::Bookseller qw( GetBookSellerFromId );
use Test::More tests => 63;
use Test::More tests => 64;
BEGIN {
use_ok('C4::Acquisition');
@ -103,6 +103,7 @@ is( $neworder->{claimed_date}, strftime( "%Y-%m-%d", localtime(time) ), "AddClai
my @expectedfields = qw( basketno
biblionumber
invoiceid
booksellerid
budgetdate
cancelledby
closedate