Based on jeneric's CSV download templates;

- try to reduce code duplication by using same route handler for ajax and csv,
  using request->is_ajax to switch the template, and set content-type

- use new HTML5 "download" attribute on links so content-disposition header is
  no longer necessary

- download CSV icon is placed on all tables (per report/device/serach section)

- update download CSV link using javascript just before table content is
  fetched - this is necessary to make sure updated sidebar query params are
  included

The idea here is to allow us to support CSV download in the pages which
display tables by only doing the following:

- (existing routes:) replace "ajax" with "get" route handler

- add logic to switch template in handler, based on request->is_ajax

- write _csv.tt version of the template, to spit out CSV file content

This makes it much easier for new devs to write reports supporting CSV, I
think?
This commit is contained in:
Oliver Gorwits
2013-09-20 09:23:57 +01:00
parent d3553d2623
commit ce57cdba69
4 changed files with 27 additions and 24 deletions

View File

@@ -1,3 +1,18 @@
// csv download icon on any table page
// needs to be dynamically updated to use current search options
function update_csv_download_link (type, tab) {
var form = '#' + tab + '_form';
var query = $(form).serialize();
// this is needed otherwise we can 404 on url with traling slash
if (query.length) { query = '/' + query }
$('#nd_csv-download').attr('href', '/ajax/content/' + type + '/' + tab + query);
}
// page title includes tab name and possibly device name
// this is nice for when you have multiple netdisco pages open in the
// browser
function update_page_title (tab) {
var pgtitle = 'Netdisco';
if ($('#nd_device-name').text().length) {
@@ -74,7 +89,8 @@
[% IF report %]
// for the report pages
$('[% "#${report.tag}_form" %]').submit(function (event) {
update_page_title('[% tab.tag %]');
update_page_title('[% report.tag %]');
update_csv_download_link('report', '[% report.tag %]');
do_search(event, '[% report.tag %]');
});
[% END -%]
@@ -82,7 +98,7 @@
[% IF task %]
// for the admin pages
$('[% "#${task.tag}_form" %]').submit(function (event) {
update_page_title('[% tab.tag %]');
update_page_title('[% task.tag %]');
do_search(event, '[% task.tag %]');
});
[% END %]