[PR #49] [MERGED] PHP 8 compatibility #59

Closed
opened 2026-01-24 11:24:59 +01:00 by admin · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/doctrine/reflection/pull/49
Author: @alexpott
Created: 10/27/2020
Status: Merged
Merged: 10/27/2020
Merged by: @greg0ire

Base: 1.2.xHead: php8-compatibility


📝 Commits (6)

  • a82bebf Initial commit
  • 2336d15 Update dist to use a more recent PHP 8.0
  • 06eec07 Ignore compatibility files in phpstan
  • f3ade86 Ensure phpstan runs with locked dependencies
  • a8d8225 Fix up things after merging 1.2.x
  • dc269e0 doctrine/coding-standard needs a PHP 8.0 compatible version

📊 Changes

12 files changed (+137 additions, -39 deletions)

View changed files

📝 .travis.yml (+4 -1)
📝 composer.json (+6 -9)
📝 composer.lock (+2 -5)
lib/Doctrine/Common/Reflection/Compatibility/Php7/ReflectionClass.php (+24 -0)
lib/Doctrine/Common/Reflection/Compatibility/Php7/ReflectionMethod.php (+16 -0)
lib/Doctrine/Common/Reflection/Compatibility/Php8/ReflectionClass.php (+24 -0)
lib/Doctrine/Common/Reflection/Compatibility/Php8/ReflectionMethod.php (+16 -0)
lib/Doctrine/Common/Reflection/Compatibility/ReflectionClass.php (+18 -0)
lib/Doctrine/Common/Reflection/Compatibility/ReflectionMethod.php (+18 -0)
📝 lib/Doctrine/Common/Reflection/StaticReflectionClass.php (+3 -16)
📝 lib/Doctrine/Common/Reflection/StaticReflectionMethod.php (+3 -8)
📝 phpstan.neon.dist (+3 -0)

📄 Description

Fixes doctrine/reflection on PHP 8. Updates composer.json to allow PHP 8 and uses class aliases and traits to account for changes in \ReflectionClass and \ReflectionMethod

Related to #43


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/doctrine/reflection/pull/49 **Author:** [@alexpott](https://github.com/alexpott) **Created:** 10/27/2020 **Status:** ✅ Merged **Merged:** 10/27/2020 **Merged by:** [@greg0ire](https://github.com/greg0ire) **Base:** `1.2.x` ← **Head:** `php8-compatibility` --- ### 📝 Commits (6) - [`a82bebf`](https://github.com/doctrine/reflection/commit/a82bebf80129e21f84e18a3020d668d74431b255) Initial commit - [`2336d15`](https://github.com/doctrine/reflection/commit/2336d15b1aef3890e2a8887d6d97d97a2b4b4379) Update dist to use a more recent PHP 8.0 - [`06eec07`](https://github.com/doctrine/reflection/commit/06eec07aa34b6011da2ad9772971f551bc88cf20) Ignore compatibility files in phpstan - [`f3ade86`](https://github.com/doctrine/reflection/commit/f3ade869eb756856322e70a0947fa91b8722816b) Ensure phpstan runs with locked dependencies - [`a8d8225`](https://github.com/doctrine/reflection/commit/a8d82251c6ba6a6f61ad40e10a663bd82241a40e) Fix up things after merging 1.2.x - [`dc269e0`](https://github.com/doctrine/reflection/commit/dc269e064a8555479c18a75f9732987f769fc318) doctrine/coding-standard needs a PHP 8.0 compatible version ### 📊 Changes **12 files changed** (+137 additions, -39 deletions) <details> <summary>View changed files</summary> 📝 `.travis.yml` (+4 -1) 📝 `composer.json` (+6 -9) 📝 `composer.lock` (+2 -5) ➕ `lib/Doctrine/Common/Reflection/Compatibility/Php7/ReflectionClass.php` (+24 -0) ➕ `lib/Doctrine/Common/Reflection/Compatibility/Php7/ReflectionMethod.php` (+16 -0) ➕ `lib/Doctrine/Common/Reflection/Compatibility/Php8/ReflectionClass.php` (+24 -0) ➕ `lib/Doctrine/Common/Reflection/Compatibility/Php8/ReflectionMethod.php` (+16 -0) ➕ `lib/Doctrine/Common/Reflection/Compatibility/ReflectionClass.php` (+18 -0) ➕ `lib/Doctrine/Common/Reflection/Compatibility/ReflectionMethod.php` (+18 -0) 📝 `lib/Doctrine/Common/Reflection/StaticReflectionClass.php` (+3 -16) 📝 `lib/Doctrine/Common/Reflection/StaticReflectionMethod.php` (+3 -8) 📝 `phpstan.neon.dist` (+3 -0) </details> ### 📄 Description Fixes doctrine/reflection on PHP 8. Updates composer.json to allow PHP 8 and uses class aliases and traits to account for changes in \ReflectionClass and \ReflectionMethod Related to #43 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
admin added the pull-request label 2026-01-24 11:24:59 +01:00
admin closed this issue 2026-01-24 11:24:59 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/reflection#59