[KevinBlakeLopez]

Description

Fixes #10648.

reconnect() closes a dead socket and opens a new one. The server has already rolled back the transaction that was open on the old socket. transDepth and transStatus were left as they were, and ping() goes through simpleQuery(), which does not call handleTransStatus(). transComplete() then returned true while the pre-timeout insert was gone.

When the socket is replaced during a transaction, reconnect() now clears transDepth and marks the transaction failed. A later reconnect() while ping() is still true leaves the open transaction alone.

On develop (df8b96bee) the new live test failed before this change:

-    'depth' => 0,
-    'status' => false,
-    'completed' => false,
+    'depth' => 1,
+    'status' => true,
+    'completed' => true,
     'rows' => 0,

After the change, ReconnectDuringTransactionTest and PingTest pass (local MySQL 8.0.45, PHP 8.5.12, MySQLi).

Checklist:

• Securely signed commits
• Component(s) with PHPDoc blocks, only if necessary or adds value (without duplication)
• Unit testing, with >80% coverage
• User guide updated
• Conforms to style guide

Made with Cursor

[carson-codeigniter4]

Hi there, @KevinBlakeLopez! 👋

Thank you for sending this PR!

We expect the following in all Pull Requests (PRs).

• PRs must be sent to the appropriate branch
• All git commits must be GPG-signed
• Must follow our style guide
• Be commented in the PHP source file
• Be documented in the user guide
• Be unit tested
• Pass all checks in GitHub Actions

Important

We expect all code changes or bug-fixes to be accompanied by one or more tests added to our test suite to prove the code works.

If pull requests do not comply with the above, they will likely be closed. Since we are a team of volunteers, we don't have any more time to work
on the framework than you do. Please make it as painless for your contributions to be included as possible.

See https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/pull_request.md

[Copilot]

🟡 Changes recommended

Failed reconnection attempts bypass transaction invalidation, and OCI8 recovery leaves autocommit disabled.

2 open issues

• Restore OCI8 autocommit after abandoning lost transaction · New
• Reset transaction state before reconnect initialization · New

What changed in this PR

Addresses #10648 by invalidating transaction state when reconnect() replaces a lost database connection. Reviewed under the assumed develop patch-release policy.

Changes:

• Clears transaction depth and marks interrupted transactions as failed.
• Adds MySQLi live tests for lost and healthy connections during transactions.
• Documents the behavior in the connection guide and changelog.

Validation used targeted source inspection; automated tests and the database-driver matrix remain for CI.

File	Description
user_guide_src/​source/​database/​connecting.rst	Explains transaction invalidation after reconnection.
user_guide_src/​source/​changelogs/​v4.7.6.rst	Records the fix for #10648.
tests/​system/​Database/​Live/​ReconnectDuringTransactionTest.php	Tests interrupted and preserved MySQLi transactions.
system/​Database/​BaseConnection.php	Resets transaction state after replacing a connection.

_{Review effort: Balanced}

[michalsn]

Looks good overall. Please clarify that reconnect() does not resume the lost transaction and that failure handling must cover the entire operation, including nested transaction groups.

[michalsn]

Thank you!