Skip to content

Conversation

javiereguiluz
Copy link
Member

Fixes #21284.

No need to mention the addition of the $context argument in parse() method because we already explain the context argument in sanitizeFor() method, which is what users will really use.

@javiereguiluz
Copy link
Member Author

I don't understand the reported issue:

!!  
!!  In ArrayNode.php line 304:
!!                                                                                 
!!    Unrecognized option "exception_controller" under "twig". Available options   
!!    are "auto_reload", "autoescape_service", "autoescape_service_method", "base  
!!    _template_class", "cache", "charset", "date", "debug", "default_path", "fil  
!!    e_name_pattern", "form_themes", "globals", "mailer", "number_format", "opti  
!!    mizations", "paths", "strict_variables".                                     
!!                                                                                  

I started seeing this error in other projects too. Why is it happening now? If Symfony changed this option, it'd do it in a BC-compatible way.

javiereguiluz added a commit to symfony-tools/symfony-application that referenced this pull request Sep 25, 2025
This PR was merged into the 5.4 branch.

Discussion
----------

Remove legacy twig.exception_controller option

Related to symfony/symfony-docs#21410 /cc `@javiereguiluz`

Commits
-------

46b6074 Remove legacy twig.exception_controller option
@javiereguiluz javiereguiluz merged commit 2c55083 into symfony:8.0 Sep 25, 2025
1 of 3 checks passed
@javiereguiluz javiereguiluz deleted the fix_21284 branch September 25, 2025 07:43
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.

3 participants