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

Jump to bottom

Handle results of operation events #673

Closed

loic425 opened this issue Mar 29, 2023 · 0 comments

Member

loic425 commented Mar 29, 2023

No description provided.

loic425 mentioned this issue

Handle pre & post events #674

Merged

lchrusciel added a commit that referenced this issue


          feature #674 Handle pre & post events (loic425)

7f77b04

This PR was merged into the poc-new-resource-metadata branch.

Discussion
----------

| Q               | A
| --------------- | -----
| Bug fix?        | no
| New feature?    | yes
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | fixes #673
| License         | MIT

This one is not simple... 

On ResourceController:
* [PreCreate](https://github.com/Sylius/SyliusResourceBundle/blob/1.11/src/Bundle/Controller/ResourceController.php#L177)
* [PostCreate](https://github.com/Sylius/SyliusResourceBundle/blob/1.11/src/Bundle/Controller/ResourceController.php#L202)
* [PreUpdate](https://github.com/Sylius/SyliusResourceBundle/blob/1.11/src/Bundle/Controller/ResourceController.php#L256)
* [PostUpdate](https://github.com/Sylius/SyliusResourceBundle/blob/1.11/src/Bundle/Controller/ResourceController.php#L288)

Commits
-------

baa0e3d Handle pre & post events
3095794 Fix Psalm error
fadb1b7 Fix coding standard
d6629f8 Fix DI
587210a Fix operation event listener
bf3f552 Fix write listener when persist result is a response
5273dab Fix PHPStan error
f14c2b4 Improve event handler with dedicated methods for pre and post process events
a53e492 Fix coding standard

loic425 closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment