8000 [Messenger] Add a middleware that wraps all handlers in one Doctrine transaction. by Nyholm · Pull Request #26647 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[Messenger] Add a middleware that wraps all handlers in one Doctrine transaction. #26647

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 24 commits into from
Closed

[Messenger] Add a middleware that wraps all handlers in one Doctrine transaction. #26647

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
e62a0c9
Add a middleware that wraps all handlers in one Doctrine transaction.
Nyholm Mar 23, 2018
78f021d
Updated extension
Nyholm Mar 23, 2018
50f2cb8
cs
Nyholm Mar 23, 2018
11a3aab
Fixed tests
Nyholm Mar 23, 2018
373a2d4
Handle a return value
Nyholm Mar 23, 2018
4f54988
minor
Nyholm Mar 23, 2018
1eb6f4d
Updates according to feedback
Nyholm Mar 23, 2018
8b87f2b
Do not use an anonymous function
Nyholm Mar 23, 2018
a32e6b8
Added tests
Nyholm Mar 23, 2018
0261a51
Adding configuration definition
Nyholm Mar 23, 2018
f362f50
cs
Nyholm Mar 23, 2018
a629620
cs
Nyholm Mar 23, 2018
05ac7cf
Allow null
Nyholm Mar 26, 2018
0e0ab07
Moved to DoctrineBridge
Nyholm Mar 26, 2018
af65748
Added extra check
Nyholm Mar 26, 2018
701c665
Updating configuration
Nyholm Mar 26, 2018
b336d04
Fixed tests
Nyholm Mar 26, 2018
7d84394
cs
Nyholm Mar 26, 2018
09e5daf
cs and minors
Nyholm Mar 27, 2018
dcb07ca
Fixed tests
Nyholm Mar 27, 2018
1603708
Fixed EOL
Nyholm Mar 27, 2018
399f7c5
Added config key "middlewares"
Nyholm Mar 27, 2018
5a6b8de
cs
Nyholm Mar 27, 2018
68131f4
Use `addDefaultsIfNotSet()`
Nyholm Mar 27, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
cs and minors
  • Loading branch information
@Nyholm
Nyholm committed Mar 27, 2018
commit 09e5daf988decda2f70e3bb07962d67942671b80
2 changes: 1 addition & 1 deletion src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1449,7 +1449,7 @@ private function registerMessengerConfiguration(array $config, ContainerBuilder

if ($config['doctrine_transaction']['enabled']) {
if (!class_exists(DoctrineTransactionMiddleware::class)) {
throw new LogicException('You must install symfony/doctrine-bridge to use the "DoctrineTransactionMiddleware"');
throw new LogicException('The Doctrine transaction middleware is only available when the doctrine bridge is installed. Try running "composer require symfony/doctrine-bridge".');
}
$container->getDefinition('messenger.middleware.doctrine_transaction')->replaceArgument(1, $config['doctrine_transaction']['entity_manager_name']);
} else {
Expand Down
4 changes: 1 addition & 3 deletions src/Symfony/Bundle/FrameworkBundle/Resources/config/schema/symfony-1.0.xsd
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -363,9 +363,7 @@
</xsd:complexType>

<xsd:complexType name="messenger_doctrine_transaction">
<xsd:sequence>
<xsd:element name="entity-manager-name" type="xsd:string" />
</xsd:sequence>
<xsd:attribute name="entity-manager-name" type="xsd:string" />
<xsd:attribute name="enabled" type="xsd:boolean" />
</xsd:complexType>
</xsd:schema>
6 changes: 3 additions & 3 deletions src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Fixtures/xml/messenger.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,11 +8,11 @@
<framework:config>
<framework:messenger>
<framework:routing message-class="App\Bar">
<framework:sender service="sender.bar"></framework:sender>
<framework:sender service="sender.biz"></framework:sender>
<framework:sender service="sender.bar" />
<framework:sender service="sender.biz" />
</framework:routing>
<framework:routing message-class="App\Foo">
<framework:sender service="sender.foo"></framework:sender>
<framework:sender service="sender.foo" />
</framework:routing>
</framework:messenger>
</framework:config>
Expand Down
4 changes: 1 addition & 3 deletions ...fony/Bundle/FrameworkBundle/Tests/DependencyInjection/Fixtures/xml/messenger_doctrine.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,7 @@

<framework:config>
<framework:messenger>
<framework:doctrine-transaction>
<framework:entity-manager-name>foobar</framework:entity-manager-name>
</framework:doctrine-transaction>
<framework:doctrine-transaction entity-manager-name="foobar" />
</framework:messenger>
</framework:config>
</container>
0