Operations list in UI module throws WSOD if scripts can not longer be run

Open on Drupal.org →

Created on 15 January 2025, 3 months ago

Problem/Motivation

When I enabled the codit_batch_operations_ui module and went to /admin/config/development/batch_operations/operations, I got a WSOD, telling me

Drupal\Core\Entity\Query\QueryException: 'field_mental_health_phone' not found in Drupal\Core\Entity\Query\Sql\Tables->ensureEntityTable() (line 432 of core/lib/Drupal/Core/Entity/Query/Sql/Tables.php).

It is true that the field in question is now gone, as we ran the script to migrate content from one field to another and eventually removed the original field.

It's possible that I'm thinking of the UI module incorrectly and that the only time we would want to enable it would be for scripts we would want to run again.

If not, I'd like there to be a way to see the operations list without having to keep all of the old fields around. Conversely, I'd like to be able to load the operations page, even if we have to show the scripts as having errors now, rather than a WSOD.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report

Status

Active

Version

1.0

Component

User interface

Created by

🇺🇸United States skyriter

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!29Operations list in UI module throws WSOD if scripts can not longer be run
Merged
🇺🇸United States swirt
updated 3 months ago

Comments & Activities

Issue created by @skyriter
Comment 3 months ago →
🇺🇸United States swirt Florida
This is a great find @skyriter. We definitely did not adequately defend against old processes rusting.

I think this can be solved by wrapping the call to `->gatherItemsToProcess()` in a try - catch and then surface the error notice to the column if it catches one. The UI calls that function to see how many items is has to show in the display for each BatchOperation.
Comment 3 months ago →
🇺🇸United States swirt Florida
Merge request !29Issue #3499831 by skyriter, swirt: Operations list in UI module throws WSOD if... → (Merged) created by swirt
Comment 3 months ago →
🇺🇸United States swirt Florida

Comment 3 months ago →

swirt → committed 742c4c10 on 1.0.x

Issue #3499831 by skyriter, swirt: Operations list in UI module throws...

Comment 3 months ago →
🇺🇸United States swirt Florida
Comment 3 months ago →
🇺🇸United States swirt Florida
Released as part of 1.0.8 →

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024