bug symfony#36030 [SecurityBundle] Minor fix in LDAP config tree builder (HeahDude)

nicolas-grekas · nicolas-grekas · commit 741b24926bdd · 2020-03-12T16:55:28.000+01:00
This PR was merged into the 4.4 branch. Discussion ---------- [SecurityBundle] Minor fix in LDAP config tree builder | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | ~ | License | MIT | Doc PR | ~ Continuation of symfony#35910 for 4.4. Commits ------- 468a201 [SecurityBundle] Minor fix in LDAP config tree builder
diff --git a/src/Symfony/Bundle/SecurityBundle/DependencyInjection/Security/UserProvider/LdapFactory.php b/src/Symfony/Bundle/SecurityBundle/DependencyInjection/Security/UserProvider/LdapFactory.php
@@ -48,6 +48,7 @@ public function getKey()
     public function addConfiguration(NodeDefinition $node)
     {
         $node
+            ->fixXmlConfig('extra_field')
             ->fixXmlConfig('default_role')
             ->children()
                 ->scalarNode('service')->isRequired()->cannotBeEmpty()->defaultValue('ldap')->end()

Original file line number	Diff line number	Diff line change
`@@ -48,6 +48,7 @@ public function getKey()`
`48`	`48`	`public function addConfiguration(NodeDefinition $node)`
`49`	`49`	`{`
`50`	`50`	`$node`
	`51`	`+ ->fixXmlConfig('extra_field')`
`51`	`52`	`->fixXmlConfig('default_role')`
`52`	`53`	`->children()`
`53`	`54`	`->scalarNode('service')->isRequired()->cannotBeEmpty()->defaultValue('ldap')->end()`