bug #29395 [FrameworkBundle][Messenger] Restore check for messenger serializer default id (ogizanagi)
This PR was merged into the 4.2 branch.
Discussion
----------
[FrameworkBundle][Messenger] Restore check for messenger serializer default id
| Q | A
| ------------- | ---
| Branch? | 4.2 <!-- see below -->
| Bug fix? | yes
| New feature? | no <!-- don't forget to update src/**/CHANGELOG.md files -->
| BC breaks? | no <!-- see https://symfony.com/bc -->
| Deprecations? | no <!-- don't forget to update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tests pass? | yes <!-- please add some, will be required by reviewers -->
| Fixed tickets | https://symfony-devs.slack.com/archives/C9PQ75TV3/p1543590611003500 <!-- #-prefixed issue number(s), if any -->
| License | MIT
| Doc PR | N/A
From Slack:
> @adamquaile [4:10 PM]
> So, I just updated to 4.2 today, and got this message:
> > The default Messenger serializer cannot be enabled as the Serializer support is not available. Try enabling it or running "composer require symfony/serializer-pack"
>
> In the docs it's stated:
> > In order to use Symfony's built-in AMQP transport, you will need the Serializer Component. Ensure that it is installed with:
>
>But I haven't yet configured AMQP - I'm using my own transport. Should I be getting this exception?
---
This check was removed in https://github.com/symfony/symfony/pull/28405, but is actually still necessary to not fail as soon as you can install the Messenger component without the Serializer one installed.
Commits
-------
1cf17c08e9
[FrameworkBundle][Messenger] Restore check for messenger serializer default id
This commit is contained in:
commit
c3ad8a5c9b
@ -1059,7 +1059,7 @@ class Configuration implements ConfigurationInterface
|
|||||||
})
|
})
|
||||||
->end()
|
->end()
|
||||||
->children()
|
->children()
|
||||||
->scalarNode('id')->defaultValue('messenger.transport.symfony_serializer')->end()
|
->scalarNode('id')->defaultValue(!class_exists(FullStack::class) && class_exists(Serializer::class) ? 'messenger.transport.symfony_serializer' : null)->end()
|
||||||
->scalarNode('format')->defaultValue('json')->end()
|
->scalarNode('format')->defaultValue('json')->end()
|
||||||
->arrayNode('context')
|
->arrayNode('context')
|
||||||
->normalizeKeys(false)
|
->normalizeKeys(false)
|
||||||
|
@ -18,6 +18,7 @@ use Symfony\Component\Config\Definition\Exception\InvalidConfigurationException;
|
|||||||
use Symfony\Component\Config\Definition\Processor;
|
use Symfony\Component\Config\Definition\Processor;
|
||||||
use Symfony\Component\Lock\Store\SemaphoreStore;
|
use Symfony\Component\Lock\Store\SemaphoreStore;
|
||||||
use Symfony\Component\Messenger\MessageBusInterface;
|
use Symfony\Component\Messenger\MessageBusInterface;
|
||||||
|
use Symfony\Component\Serializer\Serializer;
|
||||||
|
|
||||||
class ConfigurationTest extends TestCase
|
class ConfigurationTest extends TestCase
|
||||||
{
|
{
|
||||||
@ -293,7 +294,7 @@ class ConfigurationTest extends TestCase
|
|||||||
'routing' => array(),
|
'routing' => array(),
|
||||||
'transports' => array(),
|
'transports' => array(),
|
||||||
'serializer' => array(
|
'serializer' => array(
|
||||||
'id' => 'messenger.transport.symfony_serializer',
|
'id' => !class_exists(FullStack::class) && class_exists(Serializer::class) ? 'messenger.transport.symfony_serializer' : null,
|
||||||
'format' => 'json',
|
'format' => 'json',
|
||||||
'context' => array(),
|
'context' => array(),
|
||||||
),
|
),
|
||||||
|
@ -3,6 +3,7 @@
|
|||||||
$container->loadFromExtension('framework', array(
|
$container->loadFromExtension('framework', array(
|
||||||
'serializer' => true,
|
'serializer' => true,
|
||||||
'messenger' => array(
|
'messenger' => array(
|
||||||
|
'serializer' => 'messenger.transport.symfony_serializer',
|
||||||
'routing' => array(
|
'routing' => array(
|
||||||
'Symfony\Component\Messenger\Tests\Fixtures\DummyMessage' => array('amqp', 'audit'),
|
'Symfony\Component\Messenger\Tests\Fixtures\DummyMessage' => array('amqp', 'audit'),
|
||||||
'Symfony\Component\Messenger\Tests\Fixtures\SecondMessage' => array(
|
'Symfony\Component\Messenger\Tests\Fixtures\SecondMessage' => array(
|
||||||
|
@ -4,6 +4,7 @@ $container->loadFromExtension('framework', array(
|
|||||||
'serializer' => true,
|
'serializer' => true,
|
||||||
'messenger' => array(
|
'messenger' => array(
|
||||||
'serializer' => array(
|
'serializer' => array(
|
||||||
|
'id' => 'messenger.transport.symfony_serializer',
|
||||||
'format' => 'csv',
|
'format' => 'csv',
|
||||||
'context' => array('enable_max_depth' => true),
|
'context' => array('enable_max_depth' => true),
|
||||||
),
|
),
|
||||||
|
@ -5,6 +5,7 @@ $container->loadFromExtension('framework', array(
|
|||||||
'enabled' => false,
|
'enabled' => false,
|
||||||
),
|
),
|
||||||
'messenger' => array(
|
'messenger' => array(
|
||||||
|
'serializer' => 'messenger.transport.symfony_serializer',
|
||||||
'transports' => array(
|
'transports' => array(
|
||||||
'default' => 'amqp://localhost/%2f/messages',
|
'default' => 'amqp://localhost/%2f/messages',
|
||||||
),
|
),
|
||||||
|
@ -3,6 +3,7 @@
|
|||||||
$container->loadFromExtension('framework', array(
|
$container->loadFromExtension('framework', array(
|
||||||
'serializer' => true,
|
'serializer' => true,
|
||||||
'messenger' => array(
|
'messenger' => array(
|
||||||
|
'serializer' => 'serializer: messenger.transport.symfony_serializer',
|
||||||
'transports' => array(
|
'transports' => array(
|
||||||
'default' => 'amqp://localhost/%2f/messages',
|
'default' => 'amqp://localhost/%2f/messages',
|
||||||
'customised' => array(
|
'customised' => array(
|
||||||
|
@ -8,6 +8,7 @@
|
|||||||
<framework:config>
|
<framework:config>
|
||||||
<framework:serializer enabled="true" />
|
<framework:serializer enabled="true" />
|
||||||
<framework:messenger>
|
<framework:messenger>
|
||||||
|
<framework:serializer id="messenger.transport.symfony_serializer" />
|
||||||
<framework:routing message-class="Symfony\Component\Messenger\Tests\Fixtures\DummyMessage">
|
<framework:routing message-class="Symfony\Component\Messenger\Tests\Fixtures\DummyMessage">
|
||||||
<framework:sender service="amqp" />
|
<framework:sender service="amqp" />
|
||||||
<framework:sender service="audit" />
|
<framework:sender service="audit" />
|
||||||
|
@ -8,7 +8,7 @@
|
|||||||
<framework:config>
|
<framework:config>
|
||||||
<framework:serializer enabled="true" />
|
<framework:serializer enabled="true" />
|
||||||
<framework:messenger>
|
<framework:messenger>
|
||||||
<framework:serializer format="csv">
|
<framework:serializer id="messenger.transport.symfony_serializer" format="csv">
|
||||||
<framework:context>
|
<framework:context>
|
||||||
<framework:enable_max_depth>true</framework:enable_max_depth>
|
<framework:enable_max_depth>true</framework:enable_max_depth>
|
||||||
</framework:context>
|
</framework:context>
|
||||||
|
@ -8,6 +8,7 @@
|
|||||||
<framework:config>
|
<framework:config>
|
||||||
<framework:serializer enabled="false" />
|
<framework:serializer enabled="false" />
|
||||||
<framework:messenger>
|
<framework:messenger>
|
||||||
|
<framework:serializer id="messenger.transport.symfony_serializer" />
|
||||||
<framework:transport name="default" dsn="amqp://localhost/%2f/messages" />
|
<framework:transport name="default" dsn="amqp://localhost/%2f/messages" />
|
||||||
</framework:messenger>
|
</framework:messenger>
|
||||||
</framework:config>
|
</framework:config>
|
||||||
|
@ -8,6 +8,7 @@
|
|||||||
<framework:config>
|
<framework:config>
|
||||||
<framework:serializer enabled="true" />
|
<framework:serializer enabled="true" />
|
||||||
<framework:messenger>
|
<framework:messenger>
|
||||||
|
<framework:serializer id="messenger.transport.symfony_serializer" />
|
||||||
<framework:transport name="default" dsn="amqp://localhost/%2f/messages" />
|
<framework:transport name="default" dsn="amqp://localhost/%2f/messages" />
|
||||||
<framework:transport name="customised" dsn="amqp://localhost/%2f/messages?exchange_name=exchange_name">
|
<framework:transport name="customised" dsn="amqp://localhost/%2f/messages?exchange_name=exchange_name">
|
||||||
<framework:options>
|
<framework:options>
|
||||||
|
@ -1,6 +1,7 @@
|
|||||||
framework:
|
framework:
|
||||||
serializer: true
|
serializer: true
|
||||||
messenger:
|
messenger:
|
||||||
|
serializer: messenger.transport.symfony_serializer
|
||||||
routing:
|
routing:
|
||||||
'Symfony\Component\Messenger\Tests\Fixtures\DummyMessage': [amqp, audit]
|
'Symfony\Component\Messenger\Tests\Fixtures\DummyMessage': [amqp, audit]
|
||||||
'Symfony\Component\Messenger\Tests\Fixtures\SecondMessage':
|
'Symfony\Component\Messenger\Tests\Fixtures\SecondMessage':
|
||||||
|
@ -2,6 +2,7 @@ framework:
|
|||||||
serializer: true
|
serializer: true
|
||||||
messenger:
|
messenger:
|
||||||
serializer:
|
serializer:
|
||||||
|
id: messenger.transport.symfony_serializer
|
||||||
format: csv
|
format: csv
|
||||||
context:
|
context:
|
||||||
enable_max_depth: true
|
enable_max_depth: true
|
||||||
|
@ -2,5 +2,6 @@ framework:
|
|||||||
serializer:
|
serializer:
|
||||||
enabled: false
|
enabled: false
|
||||||
messenger:
|
messenger:
|
||||||
|
serializer: messenger.transport.symfony_serializer
|
||||||
transports:
|
transports:
|
||||||
default: 'amqp://localhost/%2f/messages'
|
default: 'amqp://localhost/%2f/messages'
|
||||||
|
@ -1,6 +1,7 @@
|
|||||||
framework:
|
framework:
|
||||||
serializer: true
|
serializer: true
|
||||||
messenger:
|
messenger:
|
||||||
|
serializer: messenger.transport.symfony_serializer
|
||||||
transports:
|
transports:
|
||||||
default: 'amqp://localhost/%2f/messages'
|
default: 'amqp://localhost/%2f/messages'
|
||||||
customised:
|
customised:
|
||||||
|
Reference in New Issue
Block a user