Summary: | selecting an invoice from the historic orders page does not take you to that invoice | ||
---|---|---|---|
Product: | Koha | Reporter: | Colin Campbell <colin.campbell> |
Component: | Acquisitions | Assignee: | Colin Campbell <colin.campbell> |
Status: | CLOSED FIXED | QA Contact: | Jonathan Druart <jonathan.druart> |
Severity: | normal | ||
Priority: | P5 - low | CC: | bgkriegel, gmcharlt, jonathan.druart, julian.maurice, tomascohen |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
See Also: | http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10317 | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Proposed patch
[SIGNED-OFF] Bug 10313 Fix link to invoice from orders Bug 10313 Fix link to invoice from orders |
Description
Colin Campbell
2013-05-22 18:27:05 UTC
Created attachment 18319 [details] [review] Proposed patch To test: Without this patch selecting an invoice number on the histsearch screen returns apparently random orderlines. With patch applied it should retrieve the correct invoice and its associated orderlines Created attachment 18349 [details] [review] [SIGNED-OFF] Bug 10313 Fix link to invoice from orders Need to have invoiceid and pass it to retrieve selected invoicenumber. Wrong data passed causing incorrect records to be displayed Signed-off-by: Julian Maurice <julian.maurice@biblibre.com> Simple patch, easy to test QA comment: Clearly a bug. Another bug exist on the parcel page, I will open a new report. Marked as Passed QA. Created attachment 18351 [details] [review] Bug 10313 Fix link to invoice from orders Need to have invoiceid and pass it to retrieve selected invoicenumber. Wrong data passed causing incorrect records to be displayed Signed-off-by: Julian Maurice <julian.maurice@biblibre.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Pushed to master. Thanks, Colin! This patch has been pushed to 3.12.x, will be in 3.12.1. Pushed to 3.10.x, will be in 3.10.7 |