Bug 21324

Summary: Missing aoColumns definition in acqui/parcel receivedt table
Product: Koha Reporter: paxed <pasi.kallinen>
Component: AcquisitionsAssignee: Nick Clemens <nick>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: julian.maurice, martin.renvoize
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:
Bug Depends on: 18639    
Bug Blocks:    
Attachments: Bug 21324: Fix column defs and spans
Bug 21324: Fix column defs and spans
Bug 21324: Fix column defs and spans

Description paxed 2018-09-07 10:06:04 UTC
Bug 18639 added a column to the receivedt table in acqui/parcel, but did not add an aoColumns definition for the column. This breaks the javascript on the page.

And the receivedt table totals at the bottom don't match the columns.
Comment 1 Nick Clemens 2018-09-07 11:35:55 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2018-09-07 12:11:56 UTC
Created attachment 78514 [details] [review]
Bug 21324: Fix column defs and spans

TO test:
1 - Open a bastket in acquisitions
2 - Order some items
3 - Close the basket
4 - Receive some items
5 - Note the columns are out of alingment on parcel.pl
6 - Note a JS error breaking datatable on 'Already received'
7 - Apply patch
8 - Things should be better

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 3 Julian Maurice 2018-09-14 09:39:01 UTC
Created attachment 78642 [details] [review]
Bug 21324: Fix column defs and spans

TO test:
1 - Open a bastket in acquisitions
2 - Order some items
3 - Close the basket
4 - Receive some items
5 - Note the columns are out of alingment on parcel.pl
6 - Note a JS error breaking datatable on 'Already received'
7 - Apply patch
8 - Things should be better

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Comment 4 Nick Clemens 2018-09-14 19:58:05 UTC
Awesome work all!

Pushed to master for 18.11
Comment 5 Martin Renvoize 2018-09-17 13:15:58 UTC
Caused by bug 18639 not in 18.05.x series.