Skip to content

[DependencyInjection][ProxyManagerBridge] Don't call class_exists() on null#41233

Merged
nicolas-grekas merged 1 commit intosymfony:4.4from
derrabus:bugfix/class-exists-null
May 15, 2021
Merged

[DependencyInjection][ProxyManagerBridge] Don't call class_exists() on null#41233
nicolas-grekas merged 1 commit intosymfony:4.4from
derrabus:bugfix/class-exists-null

Conversation

@derrabus
Copy link
Copy Markdown
Member

@derrabus derrabus commented May 14, 2021

Q A
Branch? 4.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets N/A
License MIT
Doc PR N/A

PHP 8.1 complains if we pass null to class_exists() or interface_exists():

class_exists(): Passing null to parameter #1 ($class) of type string is deprecated

@carsonbot carsonbot added this to the 4.4 milestone May 14, 2021
@carsonbot carsonbot changed the title Don't call class_exists() on null [DependencyInjection][ProxyManagerBridge] Don't call class_exists() on null May 14, 2021
@nicolas-grekas
Copy link
Copy Markdown
Member

Thank you @derrabus.

@nicolas-grekas nicolas-grekas merged commit 9e12a38 into symfony:4.4 May 15, 2021
@derrabus derrabus deleted the bugfix/class-exists-null branch May 15, 2021 08:20
This was referenced May 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants