- Issue created by @robcarr
- @robcarr opened merge request.
- Status changed to Needs review
over 2 years ago 3:37pm 3 March 2023 - 🇬🇧United Kingdom robcarr Perthshire, Scotland
The MR (and patch) seems to work correctly, but not sure if the $field_storage_definition is not an instance of FieldStorageConfig section requires a rework
- Status changed to RTBC
over 2 years ago 4:00pm 3 March 2023 - 🇬🇧United Kingdom finn lewis Oxford
Thanks @robcarr
I can replicate this problem. We're working with LocalGov Drupal distribution.
https://www.drupal.org/project/localgov →
Steps to reproduce:
1. Install LocalGov Drupal.
2. Enable LocalGov Full page Alert Banner (this creates fields on entities with long names)
3. Install and enable security_review
4. Run the report at /admin/reports/security-reviewResult:
Batch fails and we get this error:
Drupal\Core\Database\DatabaseExceptionWrapper: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'db.localgov_alert_banner__localgov_alert_banner_body' doesn't exist: SELECT `entity_id`, `localgov_alert_banner_body_value` FROM localgov_alert_banner__localgov_alert_banner_body t; Array ( ) in Drupal\security_review\Checks\Field->run() (line 107 of /var/www/html/web/modules/contrib/security_review/src/Checks/Field.php).
Apply the patch from this merge request and the report runs just fine!
- Status changed to Fixed
over 2 years ago 7:04pm 20 March 2023 Automatically closed - issue fixed for 2 weeks with no activity.