Logging and warning/error messages for try/catch statements in WseConfigDatabaseStorage

Created on 17 June 2022, almost 3 years ago
Updated 3 October 2023, over 1 year ago

Problem/Motivation

Many methods in WseConfigDatabaseStorage contain try/catch statements but fail silently without logging if something goes wrong. Let's change this and add logging and some error messages when exceptions are thrown.

Steps to reproduce

Revert the code to before the commit 0e094f55d4d6376ef42ddf8ac1a876cda6bfe209
Inside a workspace create a workspace config, for example by creating a view
Try to delete the view

Result: The view isn't getting deleted but there's no message or log record that indicates something went wrong due to the try/catch in WseConfigDatabaseStorage::delete().

Proposed resolution

Add logging and messages via injected services in the catch branches of the try/catch statements in WseConfigDatabaseStorage.

📌 Task
Status

Fixed

Version

1.0

Component

Workspace Config

Created by

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024