- Status changed to Needs review
about 2 years ago 10:33pm 16 January 2023 - First commit to issue fork.
- π§πͺBelgium dieterholvoet Brussels
I cleaned up the redirect fieldset a little bit by adding a Redirect to radio group. I wrote an update hook to migrate existing views.
- last update
about 1 year ago 2 pass, 2 fail - π¬π§United Kingdom danharper
Tested the MR on 2.x-dev and it works great for me.
redwan jamous β made their first commit to this issueβs fork.
HTTP_REFERER will be empty if the display link was accessed directly, in that case we would want to fallback to front page.
I updated the MR to handle this case.
Attaching a static patch file to use with composer...- πΊπΈUnited States rsnyd
Tested on core 10.4.1 with views_data_export 8.x-1.5 using merge request .diff in patches section of composer managed site. The Merge request applied cleanly and I now see 4 options for "Redirect to" under "Redirect Settings".
Active display
Other display
Custom path
Front pageActive display does keep you on the same url once batch completes.
Other display and Custom path seem to function as before.
Front page does indeed redirect you back to the front page of site. - π¬π§United Kingdom steven jones
I feel like Drupal should be able to generate a link with the
destination
parameter set? It does this a lot for other links etc. and I think that would be better than fetching the referrer header, which isn't even set by some browsers.Is there a reason I'm missing why we're not using the destination parameter?
Also, we need some tests for these changes.