Checking the activity id is not null in the history manager

In both implementation of HistoryManager interface (DefaultHistoryManager and AsyncHistoryManager) from the ProcessEngineConfigurationImpl when you are recording to the history a new activity instance you are saying that: _"Historic activity instance could have been created (but only in cache, never persisted), for example when submitting form properties"_

Because I've my own implementation of an HistoryManager to sync with an external source (ActiveMQ) I think your validation of activityId is not null it could be wrong.


I think you wanted to validate the activity entity id, the primary key if persisted, and not whether this activity has an identifier or not.
So instead of activityInstance.getActivityId() != null maybe you want to check if activityInstance.getId() != null

Ispeziona DefaultHistoryManager e AsyncHistoryManager attraverso la loro configurazione in ProcessEngineConfigurationImpl, iniziando dalla convalida utilizzata durante la registrazione di una nuova istanza di attività. Conferma se il controllo debba utilizzare l'ID dell'entità attività invece di activityId e assicurati che le istanze memorizzate nella cache e non ancora persistite siano gestite in modo coerente in entrambe le implementazioni.

Scritto dal modello di indicizzazione a partire dal testo della issue.