From 3961539194cf04fcd8bff3628c472591bc8eb3ef Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Joachim=20L=C3=B8vgaard?= Date: Sun, 23 Aug 2026 14:53:02 +0200 Subject: [PATCH] Close the configuration node chain The tree builder chain was never terminated: cookie_name is missing its ->end() and so is the surrounding ->children(). It produces the right tree today because nodes are attached to their parent when they are created rather than when end() is called, so nothing was broken. It is a trap for the next edit though. Appending another node to the chain would attach it to the scalar node instead of to children(), which fails in a way that does not obviously point back here. config:dump-reference output is unchanged. Claude-Session: https://claude.ai/code/session_01P9NzuVPQGvaVR97HZFq98a --- src/DependencyInjection/Configuration.php | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/DependencyInjection/Configuration.php b/src/DependencyInjection/Configuration.php index 4257be3..d6fd388 100644 --- a/src/DependencyInjection/Configuration.php +++ b/src/DependencyInjection/Configuration.php @@ -27,6 +27,8 @@ public function getConfigTreeBuilder(): TreeBuilder ->scalarNode('cookie_name') ->defaultValue('sstv_display_with_vat') ->info('Name of the cookie used to store the user\'s VAT choice') + ->end() + ->end() ; return $treeBuilder;