Table rows visibility in invoice search results (acqui/invoices.pl) is limited to "top 20" table rows after Bug 12147 has been pushed.
Are you sure it is caused by bug 12147? The patch removed "bPaginate: false,", so the pagination was already removed.
(In reply to Jonathan Druart from comment #1) > Are you sure it is caused by bug 12147? > The patch removed "bPaginate: false,", so the pagination was already removed. Pretty sure that it was caused by removing "bPaginate: false," (or, at least, that after re-adding it, things seem to be back to normal). I don't really know why it works that way, some DataTables weirdness maybe? My guess is that with all DataTables controls hidden with "sDom: t", pagination in still in efect "under the hood" (but no longer controllable in any way: no way to navigate to next result page or control the amount displayed). Anyway, it got me thinkig about the ergonomics of invoice search in general; maybe reenabling (at least some) datatables controls (instead of turning off "under the hood" pagination like in Bug 11798) would be way better aproach here? I mean, in mid-sized library, there may be hundreds/tousends invoices entered each year; after a relativelly short period of time this page would become less and less usable in it's current form (and also slower - but thats not easy remediable, unless we decide to convert it to ajax/server-side datatable variant ?). Apart from pagination controlls, IMO it would be nice to have a general purpose datatables search box/filter available in this page. Doing some testing right now (with sDom: "t" simply removed): - filtering by "show only subscription" does seem to coexist nicely with general-pupose DT search box (btw, having such a checkbox to show only open invoices would also be a great addition from an ergonomics standpoint) - checkboxes for selecting invoices to be merged are beahving a little weird (invoices to be merged need to be selected on the same result page for the selection to become effecive); no idea how to fix it..
Created attachment 30965 [details] [review] Bug 12646: Display all invoices on the same page Only first 20 invoices were displayed on the page. Without pagination, DT did not allow to show others. This page explicitely removes the pagination option in order to display all invoices on the page. Test plan: 1/ Create 21+ invoices 2/ Go on acqui/invoices 3/ Verify all invoices are displayed.
(In reply to Jacek Ablewicz from comment #2) I totally agree with all what you told about the invoices page. The patch I proposed only fix the pagination issue, I don't have enough time to provide a server-side version :)
Created attachment 31857 [details] [review] Bug 12646: Display all invoices on the same page I've applied the patch against master 3.17.00.025 Everything is OK. So I pass the patch to "Signed Off" status.
Created attachment 31991 [details] [review] [PASSED QA] Bug 12646: Display all invoices on the same page Only first 20 invoices were displayed on the page. Without pagination, DT did not allow to show others. This page explicitely removes the pagination option in order to display all invoices on the page. Test plan: 1/ Create 21+ invoices 2/ Go on acqui/invoices 3/ Verify all invoices are displayed. Signed-off-by: Paola Rossi <paola.rossi@cineca.it> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Patch pushed to master. Thanks Jonathan!
*** Bug 12761 has been marked as a duplicate of this bug. ***