Skip to content

chore(php): prevent PHP 8.5 null array key deprecation#24668

Closed
bshaffer wants to merge 1 commit intoprotocolbuffers:mainfrom
bshaffer:bshaffer-patch-2
Closed

chore(php): prevent PHP 8.5 null array key deprecation#24668
bshaffer wants to merge 1 commit intoprotocolbuffers:mainfrom
bshaffer:bshaffer-patch-2

Conversation

@bshaffer
Copy link
Copy Markdown
Contributor

@bshaffer bshaffer commented Dec 1, 2025 •
edited
Loading

Using null as an array offset is now deprecated PHP 8.5. The changes in this PR will fix the following:

PHP Deprecated:  Using null as an array offset is deprecated, use an empty string instead in php/src/Google/Protobuf/Internal/Descriptor.php on line 57
PHP Deprecated:  Using null as an array offset is deprecated, use an empty string instead in php/src/Google/Protobuf/Internal/DescriptorPool.php on line 78
PHP Deprecated:  Using null as an array offset is deprecated, use an empty string instead in php/src/Google/Protobuf/Internal/DescriptorPool.php on line 79
PHP Deprecated:  Using null as an array offset is deprecated, use an empty string instead in php/src/Google/Protobuf/Internal/DescriptorPool.php on line 93

@bshaffer bshaffer requested a review from a team as a code owner December 1, 2025 19:38
@bshaffer bshaffer requested review from zhangskz and removed request for a team December 1, 2025 19:38
@zhangskz zhangskz added the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Dec 1, 2025
@github-actions github-actions bot removed the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Dec 1, 2025
@copybara-service copybara-service bot closed this in 48b3aa1 Dec 2, 2025
@VincentLanglet VincentLanglet mentioned this pull request Dec 15, 2025
Closed
@bshaffer bshaffer mentioned this pull request Dec 17, 2025
Closed
@ruudk ruudk mentioned this pull request Jan 5, 2026
Closed
This was referenced Jan 30, 2026
Closed
Closed
Merged
@th-lange th-lange mentioned this pull request Mar 13, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zhangskz zhangskz zhangskz approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bshaffer @zhangskz