flowable / flowable/flowable-engine
Owner/Assignee cannot be overridden by a StartProcessInstanceInterceptor
- Vorherrschende Sprache
- Java
- Sterne
- 9.5k
- Forks
- 2.9k
- Ø Merge
- 7 Std. 8 Min.
- Gemergte PRs (30 T.)
- 2
Beschreibung
The process start capabilities have been extended with #3449. It allows to set the owner and assignee directly at the process start.
Furthermore, the developer has foreseen the possibility to override these properties by a StartProcessInstanceInterceptor but does not use these overridden values.
See
https://github.com/flowable/flowable-engine/blob/main/modules/flowable-engine/src/main/java/org/flowable/engine/impl/util/ProcessInstanceHelper.java#L236
and
https://github.com/flowable/flowable-engine/blob/main/modules/flowable-engine/src/main/java/org/flowable/engine/impl/util/ProcessInstanceHelper.java#L239
Instead of using the variables `ownerId` and `assigneeId`, the values from `startInstanceBeforeContext.getOwnerId()` and `startInstanceBeforeContext.getAssigneeId()` should be used.
Additional context:
Note that the pull request #3449 is superseded by my own pull request #3407. If it is planned to use the #3407 feature, you might want to remove the #3449 feature and in that case, this bug report gets obsolete.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Lies modules/flowable-engine/src/main/java/org/flowable/engine/impl/util/ProcessInstanceHelper.java um die Zeilen 236 und 239 herum und konzentriere dich darauf, wie die von StartProcessInstanceInterceptor bereitgestellten Werte behandelt werden. Überprüfe, ob der von diesem Interceptor bereitgestellte owner und assignee anstelle der ursprünglichen Variablen verwendet werden, und prüfe gleichzeitig, ob der nachfolgende Pull Request dieses Issue obsolet macht.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- backend
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 35/100