Move shutdown functions into the shutdown handler class.

Addressed #11. Rerrolled the patch in the MR. Updated CR

Applied the MR and searched for drupal_register_shutdown_function and seems all instances have been replaced.

Only question is the formatting of the trigger_errors. Not sure it matters but wanted to flag for committers.

I've added some comments to the MR.

Addressed #23

feedback appears to have been addressed. Left a note in IS that an api change is being proposed.

I updated CR per Slack feedback. Also updated Error::shutdownExceptionHandler definition.

Nice to see a reduction in the 'includes'!

I'm triaging RTBC issues → . I read the IS, the comments, the change record and the MR. I didn't see any unanswered questions.

When setting an issue to RTBC the comment should explain what one did to determine the issue was ready for commit to core. For example, did you see something you liked or disliked, what core gates did you check, etc.

I looked at the comments raised by alexpott in the MR and see that they have been marked 'resolved' by the person who made the changes. I do not see any comment from another person that those changes have been reviewed. Therefor, I am setting this back to NR.

I made minor changes to the change record.

Some small changes. Rebased as the current MR didn't have the pipeline code yet.

Rebased, addressed review comments. Thanks

Seems to have code failure, have not re-reviewed yet.

New tests require strict_types declaration. Fixed

Still appears to have failures in MR.

Fixed issue with parameter initialization. Ready for the review.

Changes look good to me, and feedback addressed.

+1 RTBC but per new approach for #needs-review-queue-initiative going to leave in NR for a few days for additional eyes.

Going to go ahead and mark it, been a few days.

Checking the RTBC queue.

Ah, this is the issue where I commented about the same person making changes in response to comments in MR and them setting those comments to resolved. At the time I didn't realize there are limitations on who can resolve comments.

There have been some changes to the MR but mostly just keeping it up to date.

Leaving at RTBC.

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

This will need to make changes due to 🐛 Transaction autocommit during shutdown relies on unreliable object destruction order (xdebug 3.3+ enabled) Needs review as that ended up adding a shutdown function.