Hi,

I think that providing a path exposed in the batch config to establish which is the url to return after the process is done would be useful, attaching a patch for this.

CommentFileSizeAuthor
#3 views_data_export_1312114_3_batch_return_url.patch2.27 KBjohnv
#2 vde_1312144_2_batch_return_url.patch2.18 KBjohnv
#1 1312114-return_url_setting-1.patch1.88 KBpcambra

Comments

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Active » Needs review
StatusFileSize
new 1312114-return_url_setting-1.patch1.88 KB
johnv’s picture

johnv
Primary language Dutch
commented
Status: Needs review » Needs work
StatusFileSize
new vde_1312144_2_batch_return_url.patch2.18 KB

This is a nice solution!
The patch did not apply with git, so a new one is attached.

For me, the patch is RBTC, but it lacks 1 feature:
ATM the patch only receives one hard URL, which means it is not suited it the VDE-button is attached to multiple displays. An extra option like '

' would be nice. It would take the current path as the return URL. Marked #1009522: Error after showing VDE-page and hitting browser's left-arrow to return to previous page. as a duplicate of this one.
johnv’s picture

johnv
Primary language Dutch
commented
Status: Needs work » Needs review
StatusFileSize
new views_data_export_1312114_3_batch_return_url.patch2.27 KB

Attached patch is rerolled against latest -dev.
Older patch was not working anymore due to changed values in 'use-batch'-parameter.

My comment in #2 ( "not suited when attached to multiple displays") is not true. It works fine.

micnap’s picture

micnap commented

Applied cleanly and is working. Took me a moment to realized that I had to clean out the views data export cache.

Mickey

steven jones’s picture

steven jones commented
Status: Needs review » Fixed

Thanks so much for the patch. Committed to 7.x-3.x.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.