fjtirado commented on code in PR #3431:
URL: 
https://github.com/apache/incubator-kie-kogito-runtimes/pull/3431#discussion_r1519450850


##########
api/kogito-api/src/main/java/org/kie/kogito/uow/events/UnitOfWorkProcessEventListener.java:
##########
@@ -60,11 +60,11 @@ private void intercept(UserTaskEvent event) {
 
     @Override
     public void beforeProcessStarted(ProcessStartedEvent event) {
-        intercept(event);
     }
 
     @Override
     public void afterProcessStarted(ProcessStartedEvent event) {
+        intercept(event);

Review Comment:
   @elguardian As part of the fix for issue incubator-kie-issues#987 (thats why 
its this issue in on the header commit) the intercept was moved from after 
method to before method (actually it was a leftover, an experiment that was 
accidentally commited). This causes problems on ItTest that were not detected 
because CI was broken last week (always returned green). Once realized this 
change was breaking test (and not really intender to have been merged in the 
first place) it has been reverted with this PR, so now intercept is on the same 
place than last week.  



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to