flowable / flowable/flowable-engine

Owner/Assignee cannot be overridden by a StartProcessInstanceInterceptor

Offen
#3,628 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.