Problem/Motivation

The ViewsBulkOperationsBulkForm class is serving two different purposes but only one at a time (admin config form or views form). No point loading all the dependencies every time, also the code would be more readable if this could be split to 2 files. Also

Proposed resolution

Use traits and execute relevant methods in the main class constructor if possible.
Distinguishing between the two forms can be made in the create function using \Drupal::routeMatch()->getRouteObject()->getDefault('view_id');

API changes

There will be no API changes resulting from the solution.

Comments

Graber created an issue. See original summary.

Graber’s picture

Priority: Normal » Minor
Graber’s picture

Issue summary: View changes
Graber’s picture

Status: Active » Needs work
Graber’s picture

Status: Needs work » Closed (works as designed)
Graber’s picture

Assigned: Graber » Unassigned