Bug 14343: Incorrect links to results pages in Receive Shipment List
authorLyon3 Team <koha@univ-lyon3.fr>
Fri, 5 Jun 2015 13:37:57 +0000 (15:37 +0200)
committerChris Cormack <chris@bigballofwax.co.nz>
Tue, 22 Sep 2015 16:36:35 +0000 (04:36 +1200)
commit388b0e43639283cf02fe11f7f24b6c2e832ce10e
tree3e0e19af6cf297c456105441cc7adbdde18ab6a4
parent87ed1269fb1f6dd2624aeace532e25e663e4c509
Bug 14343: Incorrect links to results pages in Receive Shipment List

Numbered links have incorrect url. Links to result pages
don't work in Receive Shipment List (but fortunately, Next
and Previous buttons work) It's because the booksellerid is
not furnished in the url.

Test Plan :
1) Go to Acquisitions module, enter a bookseller name that you
   know you can get many page of invoices for and search for it.

2) click on Receive shipment button.

3) On bottom of the first results page, click on page number 2
   link. (cf joined screencast)

You'll see that the results include invoices from other
booksellerid. Indeed, I suppose that you get results from all booksellerid.

Intall patch and redo 3 steps.

NOTE: I did not follow this test plan.
      I read the acqui/parcels.pl code.
      The template parameter numbers is assigned in a function which has
      no reference to booksellerid at all!
      Additionally, the booksellerid is set directly elsewhere.
      It is also strange that the booksellerid references before and after
      this loop do not use the numbers.booksellerid, but just booksellerid.
      The change from numbers.booksellerid to booksellerid is correct!

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit 2dc5ae238ae0eb58040be4612cae641330b868ba)
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcels.tt