esmero / ami

Archipelago Multi Importer. A module of mass ingest made for the masses
GNU Affero General Public License v3.0
2 stars 4 forks source link

ISSUE-197: Deals with moving target for our VBO Event subscriber conditionals #198

Closed DiegoPino closed 4 months ago

DiegoPino commented 4 months ago

See #197

Since Drupal 10.2 we can't no longer just relay on a 'op' in the input to avoid saving in our VBO temporary storage facets and filters.

This new approach here attempts first to check if the VBO operation is happening under the actual Views ROUTE (so the page that the views handles) OR, if someone is using layout builder/a block, if 'op' != 'do' (might be another moving target) and there is no batch id.

How I covered VBO with facets running as a block is probably not the cleanest way but for now is solid enough

@davidbasswwu @alliomeria will ping you when the block option is tested and all is merged