In acquisitions, receive shipment and then enter a vendor invoice and click next. If you have more than 20 pending orders (you can change $resultsperpage = 20 unless ($resultsperpage); in the code to test) there will be a pagination at the bottom. The numbers don't have invoice and datereceived as arguments like the Next>>. Patch to follow.
Created attachment 7667 [details] [review] Fix variable scope in the tt
Patch does not apply anymore on master. Can you rebase your patch and resubmit please?
Created attachment 7951 [details] [review] New version of the initial patch - corrected for new master Created a new corrected version of this patch based on the current master.
Vendor invoice and datereceived stay in URL when changing results per page or going to the next page.
QA comment : tiny patch & good test plan but this patch overlap with the DataTable jquery plugin we should/want to use now. So I think this patch should be rewritten to use the jquery plugin marking failed QA
see = wiki.koha-community.org/wiki/DataTables_HowTo
Hi Paul, I think so short before feature freeze - perhaps worth fixing the bug and doing the datatables later?
(In reply to comment #7) > Hi Paul, I think so short before feature freeze - perhaps worth fixing the > bug and doing the datatables later? correct & fair, will push this one now
The patches for this bug do not apply to 3.6. If you want this included in 3.6.x, please rebase on 3.6.x and reset the version to rel_3_6.
Created attachment 10064 [details] [review] 3.6.x patch
Created attachment 10096 [details] [review] Bug 7548: Missing arguments when receiving orders Patch redone for 3.6.x to fix TT scoping problems and taking care of changed variable names (booksellerid vs supplierid). To test: Check pagination works correctly when receiving orders. Invoice number and date received should always appear in your URL and be shown at the top of the website. Signed-off-by: MJ Ray <mjr@phonecoop.coop>
This patch will be in 3.6.6.