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 < 8000 relative-time datetime="2018-03-26T10:43:22Z" class="no-wrap">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
Allow null
  • Loading branch information
@Nyholm
Nyholm committed Mar 27, 2018
commit 05ac7cfeff772bf452154152715ecea7eb2cb399
2 changes: 1 addition & 1 deletion src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -987,7 +987,7 @@ private function addMessengerSection(ArrayNodeDefinition $rootNode)
->arrayNode('doctrine_transaction')
->canBeEnabled()
->children()
->scalarNode('entity_manager_name')->info('The name of the entity manager to use')->defaultValue('default')->end()
->scalarNode('entity_manager_name')->info('The name of the entity manager to use')->defaultNull()->end()
->end()
->end()
->end()
Expand Down
2 changes: 1 addition & 1 deletion src/Symfony/Component/Messenger/Middleware/WrapsMessageHandlingInTransaction.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ class WrapsMessageHandlingInTransaction implements MiddlewareInterface
private $managerRegistry;
private $entityManagerName;

public function __construct(ManagerRegistry $managerRegistry, string $entityManagerName)
public function __construct(ManagerRegistry $managerRegistry, ?string $entityManagerName)
{
$this->managerRegistry = $managerRegistry;
$this->entityManagerName = $entityManagerName;
Expand Down
0