nav[aria-label="Primary Navigation"] { padding: 0; & ul { list-style: none; width: 100%; display: flex; flex-direction: row; justify-content: start; align-items: start; gap: 30px; padding: 0; & li { margin: 0; } & ul li { list-style: none; } } }

As we transition to the new Hyland Community, some post links may be temporarily disrupted as we finalize redirects. We apologize for the inconvenience and are working to resolve this as quickly as possible. Details.

Probable defect in EventSubscriptionEntityManager

franck102

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

has an obvious defect:

// add events created in this command (not visible yet in query)
    for (SignalEventSubscriptionEntity entity : createdSignalSubscriptions) {
      if(processInstanceId.equals(processInstanceId) && eventName.equals(entity.getEventName())) {
        selectList.add(entity);
      }
    }

The first test most likely should be if (entity.getProcessInstanceId.equals(processInstanceId) && …
Other methods in the same class seem to have the same issue.

This is in Activiti 5.15.

Franck

Find more posts tagged with

Comments

franck102

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.

Franck