Summary: | Add basket name, PO number and order date on receive | ||
---|---|---|---|
Product: | Koha | Reporter: | Amit Gupta <amitddng135> |
Component: | Acquisitions | Assignee: | Bugs List <koha-bugs> |
Status: | NEW --- | QA Contact: | |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | amitddng135, colin.campbell, jonathan.druart, katrin.fischer, michaela.sieber, vimalibre |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Attachments: |
Basket name and PO number on receipt screen
[SIGNED-OFF] Bug 10531 - Basket name and PO number on receipt screen |
Description
Amit Gupta
2013-07-03 05:35:30 UTC
Created attachment 20285 [details] [review] Basket name and PO number on receipt screen To list the basket name or PO number on the receipt screen to identify the lot that you are receiving Test Plan: 1) Create an basket 2) Create an order 3) Click on Receive shipment button enter vendor invoice no and shipment date click on Next button. 4) Basket name and Purchase order number will display both in pending and already receive screen. Created attachment 20636 [details] [review] [SIGNED-OFF] Bug 10531 - Basket name and PO number on receipt screen To list the basket name or PO number on the receipt screen to identify the lot that you are receiving Test Plan: 1) Create an basket 2) Create an order 3) Click on Receive shipment button enter vendor invoice no and shipment date click on Next button. 4) Basket name and Purchase order number will display both in pending and already receive screen. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Two minor edits on signoff: Changed "basketname" to "basket name," and converted tabs to spaces on added lines. QA comment: 1/ This patch breaks sorts on the already received table. 2/ The sort on the basket name column does not work (on the Pending orders table). 3/ Prefer to list the fields you want for the aqbasket table: + SELECT aqorders.*, biblio.*, aqbasket.* + SELECT aqorders.*, biblio.*, aqbasket.purchase_order, aqbasket.basketname Marked as Failed QA. In fact, there is a js error on this page: ReferenceError: YAHOO is not defined Already exists on master. (In reply to Jonathan Druart from comment #4) > In fact, there is a js error on this page: > ReferenceError: YAHOO is not defined > > Already exists on master. error caused by the following line: <script type="text/javascript" src="[% yuipath %]/json/json-min.js"></script> It could be removed (see bug 10786). But this patch adds another one: TypeError: a.aoColumns[c] is undefined which does not exist on master. Hi Jonathan, I cannot find any error in this patch. Can you please explain me in detail. Mean while i am changing the status of the patch. (In reply to Amit from comment #6) > Hi Jonathan, > > I cannot find any error in this patch. Can you please explain me in detail. > Mean while i am changing the status of the patch. Amit, To reproduce: Open firebug, Go on acqui/parcel.pl?invoiceid=X, you will see a JS error: "TypeError: a.aoColumns[c] is undefined". When you add a new column in a table managed by DataTables, you should adapt the aoColumns values. Back to assigned. *** Bug 8120 has been marked as a duplicate of this bug. *** |