APY / APYBreadcrumbTrailBundle

Generate a dynamic Twig breadcrumbs trail via Annotations, PHP Attributes or PHP methods.
81 stars 32 forks source link

It is not compatible with Symfony 7.0 #92

Open fsangar opened 11 months ago

fsangar commented 11 months ago

When attempting to update to the latest version of Symfony, I encounter the following error:

PHP Fatal error:  Declaration of APY\BreadcrumbTrailBundle\DependencyInjection\Configuration::getConfigTreeBuilder() must be compatible with Symfony\Component\Config\Definition\ConfigurationInterface::getConfigTreeBuilder(): Symfony\Component\Config\Definition\Builder\TreeBuilder in appName\vendor\apy\breadcrumbtrail-bundle\src\DependencyInjection\Configuration.php on line 19    

Fatal error: Declaration of APY\BreadcrumbTrailBundle\DependencyInjection\Configuration::getConfigTreeBuilder() must be compatible with Symfony\Component\Config\Definition\ConfigurationInterface::getConfigTreeBuilder(): Symfony\Component\Config\Definition\Builder\TreeBuilder in appName\vendor\apy\breadcrumbtrail-bundle\src\DependencyInjection\Configuration.php on line 19
Symfony\Component\ErrorHandler\Error\FatalError^ {#3256
  #message: "Compile Error: Declaration of APY\BreadcrumbTrailBundle\DependencyInjection\Configuration::getConfigTreeBuilder() must be compatible with Symfony\Component\Config\Definition\ConfigurationInterface::getConfigTreeBuilder(): Symfony\Component\Config\Definition\Builder\TreeBuilder"
  #code: 0
  #file: "appName\vendor\apy\breadcrumbtrail-bundle\src\DependencyInjection\Configuration.php"
  #line: 19
  -error: array:4 [
    "type" => 64
    "message" => "Declaration of APY\BreadcrumbTrailBundle\DependencyInjection\Configuration::getConfigTreeBuilder() must be compatible with Symfony\Component\Config\Definition\ConfigurationInterface::getConfigTreeBuilder(): Symfony\Component\Config\Definition\Builder\TreeBuilder"
    "file" => "appName\vendor\apy\breadcrumbtrail-bundle\src\DependencyInjection\Configuration.php"
    "line" => 19
  ]
}
rvanlaak commented 11 months ago

Will try to find time to add support on the new major version (congrats with the release!). @fsangar would you possibly have time to provide a PR?

maixal commented 1 month ago

Need support for Symfony 7 too :s