Merge branch '5.4' into 6.4

* 5.4:
  [FrameworkBundle] Fix mailer config with XML

Author: lyrixx

DependencyInjection/Configuration.php

@@ -2199,6 +2199,7 @@ private function addMailerSection(ArrayNodeDefinition $rootNode, callable $enabl
                         ->end()
                         ->arrayNode('envelope')
                             ->info('Mailer Envelope configuration')
+                            ->fixXmlConfig('recipient')
                             ->children()
                                 ->scalarNode('sender')->end()
                                 ->arrayNode('recipients')

Resources/config/schema/symfony-1.0.xsd

@@ -781,7 +781,7 @@
     <xsd:complexType name="mailer_envelope">
         <xsd:sequence>
             <xsd:element name="sender" type="xsd:string" minOccurs="0" maxOccurs="1" />
-            <xsd:element name="recipients" type="xsd:string" minOccurs="0" maxOccurs="unbounded" />
+            <xsd:element name="recipient" type="xsd:string" minOccurs="0" maxOccurs="unbounded" />
         </xsd:sequence>
     </xsd:complexType>
 

Tests/DependencyInjection/Fixtures/php/mailer_with_dsn.php

@@ -12,7 +12,7 @@
             'dsn' => 'smtp://example.com',
             'envelope' => [
                 'sender' => '[email protected]',
-                'recipients' => ['[email protected]', '[email protected]'],
+                'recipients' => ['[email protected]'],
             ],
             'headers' => [
                 'from' => '[email protected]',

Tests/DependencyInjection/Fixtures/xml/mailer_with_dsn.xml

@@ -12,8 +12,7 @@
         <framework:mailer dsn="smtp://example.com">
             <framework:envelope>
                 <framework:sender>[email protected]</framework:sender>
-                <framework:recipients>[email protected]</framework:recipients>
-                <framework:recipients>[email protected]</framework:recipients>
+                <framework:recipient>[email protected]</framework:recipient>
             </framework:envelope>
             <framework:header name="from">[email protected]</framework:header>
             <framework:header name="bcc">[email protected]</framework:header>

Tests/DependencyInjection/Fixtures/xml/mailer_with_transports.xml

@@ -14,8 +14,8 @@
             <framework:transport name="transport2">smtp://example2.com</framework:transport>
             <framework:envelope>
                 <framework:sender>[email protected]</framework:sender>
-                <framework:recipients>[email protected]</framework:recipients>
-                <framework:recipients>[email protected]</framework:recipients>
+                <framework:recipient>[email protected]</framework:recipient>
+                <framework:recipient>[email protected]</framework:recipient>
             </framework:envelope>
             <framework:header name="from">[email protected]</framework:header>
             <framework:header name="bcc">[email protected]</framework:header>

Tests/DependencyInjection/Fixtures/yml/mailer_with_dsn.yml

@@ -10,7 +10,6 @@ framework:
             sender: [email protected]
             recipients:
                 - [email protected]
-                - [email protected]
         headers:
             from: [email protected]
             bcc: [[email protected], [email protected]]

Tests/DependencyInjection/FrameworkExtensionTestCase.php

@@ -361,7 +361,7 @@ public function testWorkflows()
         $this->assertSame('state_machine.pull_request.metadata_store', (string) $metadataStoreReference);
 
         $metadataStoreDefinition = $container->getDefinition('state_machine.pull_request.metadata_store');
-        $this->assertSame(Workflow\Metadata\InMemoryMetadataStore::class, $metadataStoreDefinition->getClass());
+        $this->assertSame(InMemoryMetadataStore::class, $metadataStoreDefinition->getClass());
         $this->assertSame(InMemoryMetadataStore::class, $metadataStoreDefinition->getClass());
 
         $workflowMetadata = $metadataStoreDefinition->getArgument(0);
@@ -2056,21 +2056,27 @@ public function testHttpClientFullDefaultOptions()
         $this->assertSame(['foo' => ['bar' => 'baz']], $defaultOptions['extra']);
     }
 
-    public static function provideMailer(): array
+    public static function provideMailer(): iterable
     {
-        return [
-            ['mailer_with_dsn', ['main' => 'smtp://example.com']],
-            ['mailer_with_transports', [
+        yield [
+            'mailer_with_dsn',
+            ['main' => 'smtp://example.com'],
+            ['[email protected]'],
+        ];
+        yield [
+            'mailer_with_transports',
+            [
                 'transport1' => 'smtp://example1.com',
                 'transport2' => 'smtp://example2.com',
-            ]],
+            ],
+            ['[email protected]', '[email protected]'],
         ];
     }
 
     /**
      * @dataProvider provideMailer
      */
-    public function testMailer(string $configFile, array $expectedTransports)
+    public function testMailer(string $configFile, array $expectedTransports, array $expectedRecipients)
     {
         $container = $this->createContainerFromFile($configFile);
 
@@ -2082,7 +2088,7 @@ public function testMailer(string $configFile, array $expectedTransports)
         $this->assertTrue($container->hasDefinition('mailer.envelope_listener'));
         $l = $container->getDefinition('mailer.envelope_listener');
         $this->assertSame('[email protected]', $l->getArgument(0));
-        $this->assertSame(['[email protected]', '[email protected]'], $l->getArgument(1));
+        $this->assertSame($expectedRecipients, $l->getArgument(1));
         $this->assertEquals(new Reference('messenger.default_bus', ContainerInterface::NULL_ON_INVALID_REFERENCE), $container->getDefinition('mailer.mailer')->getArgument(1));
 
         $this->assertTrue($container->hasDefinition('mailer.message_listener'));