Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.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.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | views_data_export_1312114_3_batch_return_url.patch | 2.27 KB | johnv |
| #2 | vde_1312144_2_batch_return_url.patch | 2.18 KB | johnv |
| #1 | 1312114-return_url_setting-1.patch | 1.88 KB | pcambra |
Comments
Comment #1
pcambraComment #2
johnvThis 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:
' 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.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 '
Comment #3
johnvAttached 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.
Comment #4
micnap CreditAttribution: micnap commentedApplied cleanly and is working. Took me a moment to realized that I had to clean out the views data export cache.
Mickey
Comment #5
Steven Jones CreditAttribution: Steven Jones commentedThanks so much for the patch. Committed to 7.x-3.x.