This patch allows to use some arguments in the name of the exported file. If the argument is a node nid, its value is replaced by the node title.

CommentFileSizeAuthor
#8 views_bonus-702180-9.patch4.9 KBBoobaa
#5 views_bonus-HEAD_args_in_filename.patch4.22 KBdboulet
#2 views_bonus__export_filename.patch2.76 KBB-Prod
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

neclimdul’s picture

neclimdul
He/Him
CreditAttribution: neclimdul commented

patch missing

B-Prod’s picture

B-Prod CreditAttribution: B-Prod commented
FileSize
views_bonus__export_filename.patch2.76 KB

Sorry, stupid omission!

djroshi’s picture

djroshi CreditAttribution: djroshi commented

I'm getting an error at

/admin/build/views/ajax/display/[view name]/feed_1/style_options

caused by the following line in views_bonus_plugin_style_export.inc

'#description' => t('The filename that will be suggested to the browser for downloading purposes. %view will be replaced with the view name , !arg.',array('!arg' => implode( ',', array_map( create_function('$item','return "%$item";'), array_keys($view->display[$view->current_display]->handler->get_option('arguments')))))),

B-Prod’s picture

B-Prod CreditAttribution: B-Prod commented

@djroshi: Is it possible to have more details about this error, like the PHP message if any.
array_map error ?
isset error ?

Thanks

dboulet’s picture

dboulet CreditAttribution: dboulet commented
FileSize
views_bonus-HEAD_args_in_filename.patch4.22 KB

Here's an alternate patch, for review.

Cyberwolf’s picture

Cyberwolf CreditAttribution: Cyberwolf commented

Subscribing.

dambrisco’s picture

dambrisco CreditAttribution: dambrisco commented

Subscribing.

Boobaa’s picture

Boobaa
he/him
CreditAttribution: Boobaa commented
Title: Use view arguments in file name » Use more tokens in file name
Version: 6.x-1.x-dev » 6.x-1.1
Component: Code » Views Export
FileSize
views_bonus-702180-9.patch4.9 KB

Taking dboulet's patch in #5 as a foundation, I have added two more tokens: %timestamp (provides something like 2010-10-06_12-45-50) and %exposed (provides something like filter1_foo-filter2_bar).

ionmedia’s picture

ionmedia CreditAttribution: ionmedia commented

#8 looks very interesting, can you, Boobaa, attach patched file ? (i have no access to shell on my hosting, only ftp)

maartenzam’s picture

maartenzam CreditAttribution: maartenzam commented

Applied patch from #5 and works as expected.
Thanks!

sgtsaughter’s picture

sgtsaughter CreditAttribution: sgtsaughter commented

This is an old patch, but just confirming that patch #5 works great.

John Franklin’s picture

John Franklin CreditAttribution: John Franklin commented
Status: Needs review » Reviewed & tested by the community

Patch in #5 is still a great patch. So is #9. One of them should be committed.

John Franklin’s picture

John Franklin CreditAttribution: John Franklin commented
Issue summary: View changes

In the patch from #8, $view->exposed_input[$handler->options['expose']['identifier']] may be an array, not a value. The value will need to be flattened somehow to prevent check_plain() warnings.

neclimdul’s picture

neclimdul
He/Him
CreditAttribution: neclimdul commented
Status: Reviewed & tested by the community » Closed (outdated)