Bug 26085: Add the copy, print and export DataTables buttons to lost items report

This patch removes the in-page DataTables DOM configuration option so
that the default option is used. This will allow all the standard
controls to appear, including copy/print/export.

Since the default DataTables information includes the number of results
in the table, the template markup showing this information is removed.

To test, apply the patch and go to Reports.

- Run the "Items lost" report using parameters which will return
  multiple results.
- On the results page confirm that the standard controls are present:
  Result count, search form, columns filter, export buttons.
- Confirm that the "Activate filters" feature works, and that selecting
  results activates the option to download a CSV using a CSV export
  profile.

  Note that the "Activate filters" feature doesn't play well with hidden
  columns. However this issue predates the patch.

Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
This commit is contained in:
Owen Leonard 2020-07-29 12:58:22 +00:00 committed by Jonathan Druart
parent 9f11ba114e
commit a7c85ad163
1 changed files with 0 additions and 9 deletions

View File

@ -28,14 +28,6 @@
[% IF ( get_items ) %]
<div class="results">
[% IF items.count%]
[% items.count | html %] lost items found
[% ELSE %]
No lost items found
[% END %]
</div>
[% IF items.count %]
[% IF csv_profiles.count %]
<div class="lostitems-table_table_controls">
@ -196,7 +188,6 @@
$(document).ready(function() {
var columns_settings = [% TablesSettings.GetColumns( 'reports', 'lostitems', 'lostitems-table', 'json' ) | $raw %];
var lostitems_table = KohaTable("lostitems-table", {
"dom": 'B<"clearfix">t',
"aaSorting": [],
"aoColumnDefs": [
{ "bSortable": false, "bSearchable": false, 'aTargets': [ 'NoSort' ] },