- Issue created by @phenaproxima
- Status changed to Closed: duplicate
9 months ago 3:44pm 18 March 2024 - 🇺🇸United States phenaproxima Massachusetts
Closing in favor of 📌 Update lazy database creation pattern for more specific exceptions etc. Active .
Some services that create their database tables lazily (only when needed) have code like this (from \Drupal\Core\Cache\DatabaseBackend::getMultiple()):
try {
// Some database stuff
}
catch (\Exception $e) {
// Nothing to do.
}
These should not be catching the very general-purpose \Exception or \Throwable classes. They should be instead specifically capturing SchemaObjectNotExistsException (if they're trying to access a non-existent table) or SchemaObjectExistsException (if they're trying to create a table that has already been created). Additionally, they should be converted to non-capturing catches if they're not going to actually do anything with the exception.
Closed: duplicate
11.0 🔥
Last updated
Closing in favor of 📌 Update lazy database creation pattern for more specific exceptions etc. Active .