e107inc / e107

e107 Bootstrap CMS (Content Management System) v2 with PHP, MySQL, HTML5, jQuery and Twitter Bootstrap. Issue Discussion Room: https://gitter.im/e107inc/e107
https://e107.org
GNU General Public License v3.0
318 stars 212 forks source link

install.php: PHP 8.2 exception handler signature compatibility #5073

Closed Deltik closed 9 months ago

Deltik commented 9 months ago

Motivation and Context

Fixes: https://github.com/e107inc/e107/issues/5072

Description

The exception handler parameter has to be of type Throwable, but we are remaining compatible with PHP 5.6, which didn't have Throwable, so let's fix this by moving the type to the phpDoc block.

How Has This Been Tested?

Human static code analysis?

Types of Changes

Checklist

codeclimate[bot] commented 9 months ago

Code Climate has analyzed commit c6401ff2 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 0.0% (80% is the threshold).

This pull request will bring the total coverage in the repository to 34.7% (0.1% change).

View more on Code Climate.