flowable / flowable/flowable-engine
Deployment of App fails when modelKey of included model has been updated and the model has a message start event subscription
- Lingua principale
- Java
- Stelle
- 9.5k
- Fork
- 2.9k
- Merge medio
- 7h 8m
- PR unite (30g)
- 2
Descrizione
**Describe the bug**
We have a process model which is initiated via a message start event. Just found out that the deployment of the App containing this model fails when the model key was updated, with a somewhat hard to understand error message
```there already is a message event subscription for the message with name 'startMyFantasticProcess' ```
**Expected behavior**
The deployment to succeed
**Code**
If applicable, add the code that you are using when having the issue.
**Additional context**
This is on Flowable 6.7.2.
The reason for this error is that when deploying the method BpmnDeploymentHelper.updateTimersAndEvents calls
```eventSubscriptionManager.removeObsoleteMessageEventSubscriptions(previousProcessDefinition);```
which will not find the EventSubscriptionEntity to delete before the deployment since the name was changed.
So the fix would probably be to update the modelKey reference in that table whenever a modelKey is updated.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Direzione di ricerca
Inizia da BpmnDeploymentHelper.updateTimersAndEvents e segui eventSubscriptionManager.removeObsoleteMessageEventSubscriptions(previousProcessDefinition). Esamina come le sottoscrizioni agli eventi messaggio mantengono il modelKey quando viene rinominato un modello incluso. Il lavoro è completato quando il deployment dell’App riesce dopo la modifica del modelKey e la sottoscrizione esistente all’evento di avvio messaggio non causa più un errore per nome duplicato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- java
- Ambito
- backend
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 45/100