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.
When writing my own actions I set the redirect within my custom form submit like so: $_GET['destination'] = 'my/target';
Would be good to have that option for the default actions as well. Or is there already a way how to override the default redirect from views_bulk_operations_form_submit()
?
Comment | File | Size | Author |
---|---|---|---|
#15 | rsz_schermafdruk_van_2016-10-24_114515.png | 145.13 KB | ikeigenwijs |
#6 | interdiff_3-6.txt | 1.2 KB | ikeigenwijs |
#6 | 2307381_6.patch | 2.13 KB | ikeigenwijs |
#3 | 2307381-3.patch | 2.17 KB | ikeigenwijs |
Comments
Comment #1
leymannxComment #2
ikeigenwijs CreditAttribution: ikeigenwijs commentedI would like this feature added in Bulk operations under the form of a optional custom field.
Comment #3
ikeigenwijs CreditAttribution: ikeigenwijs commentedSolved this.
Added it in the UI with an extra option field.
No changes for current projects, default behaviour is the same.
But no more form_alter for custom redirects! Much more user and not programmer friendly.
I got help from David_Hernandez on irc.
patch attached
Comment #4
David Hernández CreditAttribution: David Hernández commentedGlad to be able to help!
Comment #5
Wim LeersIs the "at end" part even necessary? Up to the maintainer to decide, but it feels strange.
Page to redirect *to*
1) Use single quotes, like all other translatable strings here.
2) The English is very difficult to parse.
Comment #6
ikeigenwijs CreditAttribution: ikeigenwijs commentedFixed the issues.
And used an other string to explain the default behaviour
Comment #7
ikeigenwijs CreditAttribution: ikeigenwijs commentedComment #8
ikeigenwijs CreditAttribution: ikeigenwijs commentedStill applies cleanly on current dev.
This runs for the last 9 months on our production environment without problems.
Comment #9
ikeigenwijs CreditAttribution: ikeigenwijs commentedbump
Comment #10
wmfinnegan CreditAttribution: wmfinnegan commentedAny chance this feature could handle tokens from the view? I've got an EVA with VBO and I want to come back to the page with the EVA.
Comment #11
bunset CreditAttribution: bunset commentedPatch #6 worked perfectly for me!
This should be included in the module, it is awesome!.
Please include this feature in new version of vbo module! :)
Thank you very much!
Comment #12
ikeigenwijs CreditAttribution: ikeigenwijs commentedI'm trying to get commit access.
Comment #13
ibuildit CreditAttribution: ibuildit commentedI wonder, how can we make this field accept tokens, for example [path]...
Comment #14
g089h515r806 CreditAttribution: g089h515r806 commentedPatch #6 worked perfectly.
Comment #15
ikeigenwijs CreditAttribution: ikeigenwijs commentedStill applies cleanly on current dev
A screenshot of the UI addition.
No change or impact for current install and views.
Comment #16
ikeigenwijs CreditAttribution: ikeigenwijs commentedI would like to see this added to -dev
Comment #17
ibuildit CreditAttribution: ibuildit commentedPatch #6 works great, but it does NOT work when "Skip batching" is selected. Needs updating there please.
Comment #18
ikeigenwijs CreditAttribution: ikeigenwijs commentedStill applies cleanly to dev
2307381_6.patch
Checking patch views/views_bulk_operations_handler_field_operations.inc...
Hunk #1 succeeded at 63 (offset 6 lines).
Hunk #2 succeeded at 142 (offset 6 lines).
Checking patch views_bulk_operations.module...
Hunk #1 succeeded at 744 (offset 12 lines).
@ibuildit #17
I took a look when skip batch is used
views_bulk_operations_direct_process( ...) is called
I would try at line 835
Comment #19
Chris Matthews CreditAttribution: Chris Matthews commentedComment #20
ikeigenwijs CreditAttribution: ikeigenwijs commentedPatch in #6 still applies without difficulty to latest 7.x-3.x-dev (16-08-2020).
Comment #21
leymannxLet's finally raise priority to push this forward. Will we need to port this to D8?
Comment #22
kopeboy CreditAttribution: kopeboy as a volunteer commentedHow to redirect from a specific bulk operation?
If you have multiple options on the same view (for example Rules' components that you created), and just one needs a redirect at the end, how can we set the redirect for that one only?
Comment #23
tcweb CreditAttribution: tcweb commentedIs there a way to use replacement patterns in the redirect URL? We would like to have it redirect back to the node edit page. Thanks