Sylius: pre/post payment events not being triggered

[kochen]

Following this example: http://docs.sylius.org/en/latest/cookbook/disabling-order-confirmation-email.html#overriding-service-definition

I've noticed that:

1. the event name should actually be: sylius.payment.post_complete
2. overriding the service definition does cancel the original orderConfirmation email but does not send one after payment, because non of the pre/post payment complete events are triggered.

PR for the documentation will follow after the main bug is fixed.

[kochen]

related to: #8109

[kochen]

@pjedzejewski shouldn't this be fixed (later release?), instead of jut changing the docs because it doesn't work?

[pjedrzejewski]

@kochen Sorry it was automatically closed via reference. We removed docs to avoid confusion but we need to figure out a better way to solve it.

[kochen]

@pjedzejewski I know.
I say move it to v1.1.0, it's not a must have for v1.0.0

[stale[bot]]

This issue has been automatically marked as stale because it has not had any recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

[kochen]

@pjedzejewski any plan to pick this up?

[stale[bot]]

This issue has been automatically marked as stale because it has not had any recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

[czende]

@kochen Is this solved or not? I have same problem as described here #8109 :/

[kochen]

@czende As far as I know it was never address directly.
But might have been addressed indirectly - never checked...

[pamil]

IIRC there was no work done with this one.

[stale[bot]]

This issue has been automatically marked as stale because it has not had any recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

[tsogoo]

I think sylius.payment.post_complete event is still not triggered

[Gounlaf]

I think sylius.payment.post_complete event is still not triggered

I didn't found any trigger of this event too =/
The official InvoicingPlugin doesn't rely on it too.