Postponed
Project:
Drupal core
Version:
main
Component:
views.module
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
28 Oct 2015 at 23:00 UTC
Updated:
4 Nov 2019 at 20:45 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #3
andypostComment #4
dawehnerI'm wondering how much of #1823572: Port Views Bulk Operations to Drupal 8 we could extract and move slowly over time to core itself.
Comment #7
vierlexHi everyone!
I wrote ported some bare bones functionality from VBO 7 to Drupal 8.4.x
This is my first contribution ever so please nitpick about every little thing you can find,
which I am sure there are probably several :)
You can test it right away on the Content page.
Create some nodes and use the select-all checkbox in the header.
Some buttons will appear like they used to in VBO.
The bulk (hah!) of the changes can be found in docroot/core/modules/system/src/Plugin/views/field/BulkForm.php
in the functions viewsForm and viewsFormSubmit (on line 437 is probably the place to manage the actual batch processing)
Comment #8
andypostComment #9
vierlexAs far as i can see this is actually blocked by https://www.drupal.org/node/2701829
which has this actual problem https://www.drupal.org/node/2849674
Comment #10
vierlexComment #11
scott_euser commentedThank you for working on this functionality! Is your solution suitable without #9 being fixed?
Comment #12
vierlexHm no,.. now that i think about it, my changes are UI only, so no real "batching" takes place, an operation with a huge amount of nodes would still timeout.
My changes allow for selecting all those nodes a view returns even if a pager is used.
Comment #13
graber commentedI started working on VBO port for D8 a few months ago following this issue: https://www.drupal.org/node/1823572, sandbox: https://www.drupal.org/sandbox/graber/2862860, thought it'd be good to mention here.
I made a workaround for https://www.drupal.org/node/2701829 for now, more details in the original issue.
Comment #14
manuel garcia commentedLet's do this!
Drive by quick review on the patch...
These need to use short array syntax.
Comment #15
jhedstromComment #19
mglamanLinking to #2239269: Have Views BulkForm optionally use batching with an option to chunk it. The other issues seems to be more correct in that it sets a batch properly.
Comment #22
manuel garcia commentedClosing in favour of #2239269: Have Views BulkForm optionally use batching with an option to chunk it - let's continue the work there :)
Comment #23
andypostThe issue mostly about making JS frontens for views batching, so probably better to repurpose instead of closing
Comment #24
manuel garcia commentedTrue!
Comment #25
flocondetoileComment #26
flocondetoileComment #27
graber commentedMaybe we should chose comprehensive approach and just change the title to "Move VBO to Drupal core"? :P
Comment #28
andypostThis title is better, but I think we should postpone it on #2239269: Have Views BulkForm optionally use batching with an option to chunk it which adds batching, then "select all" could be added here
Comment #29
dww#28 sounds right to me. Changing status and title accordingly.