- 🇳🇱Netherlands daffie
#3137883: Deprecate passing a StatementInterface object to Connection::query → added a DatabaseExceptionWrapper. The third argument should be the \Throwable and not the code.
throw new DatabaseExceptionWrapper($message, 0, $e->getCode());
Change to throw new DatabaseExceptionWrapper($message, $e->getCode(), $e);
1. Write a test in the database group to assert that bad queries have the correct exception thrown. To start with, let's focus on queries that get caught in DatabaseExceptionWrapper exceptions and not in the other exceptions. This could have a data provider to assert a query on a specific database driver.
2. Make the fix.
3. Review
No
No
No
Closed: outdated
10.1 ✨
Particularly affects sites running on the PostgreSQL database.
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.