Firing CDI events on BeforeCommit, AfterCommit and AfterRollback.

[rafaelGuerreiro]

I also added one more test case, improved the setUp() on the tests and tested if the events were fired only when needed.

[Turini]

Sounds nice to me, could you just add some docs about that on README?

[rafaelGuerreiro]

Sure!!

[csokol]

I havent seen the code yet but it seems to be a great idea! I dont know why
but this reminds me the beforeredirectlistener from the core. I think we
should fire an event and deprecate the listener, what do you think?

Em 10:06 ter, 24/02/2015, Rodrigo Turini <notifications
notifications@github.com@ notifications@github.comgithub.com
notifications@github.com> escreveu:

[rafaelGuerreiro]

@Turini, I've added the docs. Could you please validate it?

[lucascs]

seems 👌 to me =)

[Turini]

thanks @rafaelGuerreiro, it looks great!

[Turini]

I havent seen the code yet but it seems to be a great idea! I dont know
why but this reminds me the beforeredirectlistener from the core. I think
we should fire an event and deprecate the listener, what do you think?

@csokol, which listener are you talking about?

[csokol]

Sorry, I was on mobile. The right name is RedirectListener:
https://github.com/caelum/vraptor4/blob/master/vraptor-core/src/main/java/br/com/caelum/vraptor/http/VRaptorResponse.java#L52

I think we could deprecate that listener and that method to add the
listener and create a new event to fire before redirect.

On Wed Feb 25 2015 at 10:32:33 AM Rodrigo Turini notifications@github.com
wrote:

I havent seen the code yet but it seems to be a great idea! I dont know
why but this reminds me the beforeredirectlistener from the core. I think
we should fire an event and deprecate the listener, what do you think?

@csokol https://github.com/csokol, which listener are you talking
about?

—
Reply to this email directly or view it on GitHub
#30 (comment).

[Turini]

hum, now I see your point @csokol.
I've just opened an issue caelum/vraptor4#950 to track this

[garcia-jj]

@Turini can you release a new version for this plugin? Can help users with this feature.

[Turini]

Sure! I'll do it asap

[Turini]

I've just released vs 4.0.4:
https://github.com/caelum/vraptor-jpa/releases/tag/vraptor-jpa-4.0.4
it'l be available on maven's repo in a few hours. thks