[Koha-bugs] [Bug 23626] Add a system preference to limit the number of rows of data used in a chart when viewing report results

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23626

Marcel de Rooy <m.de.rooy at rijksmuseum.nl> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |m.de.rooy at rijksmuseum.nl
             Status|Signed Off                  |Failed QA

--- Comment #28 from Marcel de Rooy <m.de.rooy at rijksmuseum.nl> ---
QA Comment:

We are ignoring execute_query's parameters: $offset, $limit
In Guided.pm you can find code around stripping limits etc. too.
And now we are looping thru all results in order to apply the limit.
Does not look good to me.
Note that the interface should not offer more results than we allow. And should
we offer the offset parameter on Reporting ?

Minor detail:
ReportsMaxRowsInChart: I think that it should go under Tools, not under
Webservices.

-- 
You are receiving this mail because:
You are watching all bug changes.