topic Probable defect in EventSubscriptionEntityManager in Alfresco Archive https://connect.hyland.com/t5/alfresco-archive/probable-defect-in-eventsubscriptionentitymanager/m-p/192929#M146059 <HTML><HEAD></HEAD><BODY><SPAN>I haven't tested this out, but just from looking at the code it looks like the logic in org.activiti.engine.impl.persistence.entity.EventSubscriptionEntityManager#findSignalEventSubscriptionsByProcessInstanceAndEventName</SPAN><BR /><BR /><SPAN>has an obvious defect:</SPAN><BR /><BR /><SPAN>// add events created in this command (not visible yet in query)</SPAN><BR /><SPAN>&nbsp;&nbsp;&nbsp; for (SignalEventSubscriptionEntity entity : createdSignalSubscriptions) {</SPAN><BR /><SPAN>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if(processInstanceId.equals(processInstanceId) &amp;&amp; eventName.equals(entity.getEventName())) {</SPAN><BR /><SPAN>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; selectList.add(entity);&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </SPAN><BR /><SPAN>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; }</SPAN><BR /><SPAN>&nbsp;&nbsp;&nbsp; }</SPAN><BR /><BR /><SPAN>The first test most likely should be if (entity.getProcessInstanceId.equals(processInstanceId) &amp;&amp; …</SPAN><BR /><SPAN>Other methods in the same class seem to have the same issue.</SPAN><BR /><BR /><SPAN>This is in Activiti 5.15.</SPAN><BR /><BR /><SPAN>Franck</SPAN></BODY></HTML> Wed, 27 May 2015 08:21:38 GMT franck102 2015-05-27T08:21:38Z Probable defect in EventSubscriptionEntityManager https://connect.hyland.com/t5/alfresco-archive/probable-defect-in-eventsubscriptionentitymanager/m-p/192929#M146059 I haven't tested this out, but just from looking at the code it looks like the logic in org.activiti.engine.impl.persistence.entity.EventSubscriptionEntityManager#findSignalEventSubscriptionsByProcessInstanceAndEventNamehas an obvious defect:// add events created in this command (not visible yet in Wed, 27 May 2015 08:21:38 GMT https://connect.hyland.com/t5/alfresco-archive/probable-defect-in-eventsubscriptionentitymanager/m-p/192929#M146059 franck102 2015-05-27T08:21:38Z Re: Probable defect in EventSubscriptionEntityManager https://connect.hyland.com/t5/alfresco-archive/probable-defect-in-eventsubscriptionentitymanager/m-p/192930#M146060 <HTML><HEAD></HEAD><BODY><SPAN>My bad, the test is strange to say the least (may throw an NPE but will otherwise always return true), but the query used to obtain selectList should ensure that the processInstanceId matches anyway.</SPAN><BR /><BR /><SPAN>Franck</SPAN></BODY></HTML> Wed, 27 May 2015 10:17:59 GMT https://connect.hyland.com/t5/alfresco-archive/probable-defect-in-eventsubscriptionentitymanager/m-p/192930#M146060 franck102 2015-05-27T10:17:59Z