Skip to content

fix: fix phpstan errors#7917

Closed
Maxcastel wants to merge 1 commit intoapi-platform:mainfrom
Maxcastel:fix-phpstan-errors
Closed

fix: fix phpstan errors#7917
Maxcastel wants to merge 1 commit intoapi-platform:mainfrom
Maxcastel:fix-phpstan-errors

Conversation

@Maxcastel
Copy link
Copy Markdown
Contributor

@Maxcastel Maxcastel commented Apr 17, 2026

Fix the followig errors:

PHPStan - PHP Static Analysis Tool 2.1.50
Note: Using configuration file /home/runner/work/core/core/phpstan.neon.dist.
Error: Property Mcp\Schema\Request\CallToolRequest::$arguments (array<string, mixed>) on left side of ?? is not nullable.
Error: Property Doctrine\ORM\Mapping\AssociationMapping::$targetEntity (class-string) on left side of ?? is not nullable.
 ------ ----------------------------------------------------------------------- 
  Line   src/Mcp/Server/Handler.php                                             
 ------ ----------------------------------------------------------------------- 
  77     Property Mcp\Schema\Request\CallToolRequest::$arguments (array<string  
         , mixed>) on left side of ?? is not nullable.                          
         🪪  nullCoalesce.property                                              
 ------ ----------------------------------------------------------------------- 

 ------ ----------------------------------------------------------------- 
  Line   src/Symfony/Doctrine/EventListener/PurgeHttpCacheListener.php    
 ------ ----------------------------------------------------------------- 
  148    Property Doctrine\ORM\Mapping\AssociationMapping::$targetEntity  
         (class-string) on left side of ?? is not nullable.               
         🪪  nullCoalesce.property                                        
 ------ -----------------------------------------------------------------

@soyuka
Copy link
Copy Markdown
Member

soyuka commented Apr 27, 2026

I fixed these on 4.3 thanks!

@soyuka soyuka closed this Apr 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants