Bug 1248986
| Summary: | DataIOEditor dialog remembers old process variables as Source or Target options | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Retired] JBoss BPMS Platform 6 | Reporter: | Jeremy Lindop <jlindop> | ||||
| Component: | jBPM Designer | Assignee: | Tihomir Surdilovic <tsurdilo> | ||||
| Status: | CLOSED EOL | QA Contact: | Kirill Gaevskii <kgaevski> | ||||
| Severity: | high | Docs Contact: | |||||
| Priority: | high | ||||||
| Version: | 6.2.0 | CC: | kgaevski, kverlaen | ||||
| Target Milestone: | ER4 | ||||||
| Target Release: | 6.2.0 | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2020-03-27 19:47:50 UTC | Type: | Bug | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Attachments: |
|
||||||
|
Description
Jeremy Lindop
2015-07-31 09:33:41 UTC
Fixed master: https://github.com/droolsjbpm/jbpm-designer/commit/4d59dcaa25cccdd68b0fa5163d010a8bac04ce17 6.3.x: https://github.com/droolsjbpm/jbpm-designer/commit/0a340dccb56956e74c212ce52fc93335c77ed15d Works good, but I have a little remark. Assignments property and Source code of elements remembers old process variables which have been edited or deleted. You need to open Assignments and save it without any changes or save whole project and reopen it. For clear incorrect link. see Attachment. Also bug 1273871 was opened. Created attachment 1085118 [details]
Assignments after process variables renaming
The behaviour in the DataIOEditor is improved, in that it doesn't remember the old process variable that was renamed. Kirill - as the behaviour reported in the BZ is fixed, and you've opened https://bugzilla.redhat.com/show_bug.cgi?id=1273871 for the more general problem of process variable renaming, can we close this BZ? Hi Jeremy, remaining things are relate to this issue not to new one. But you are right if new one will be fixed this issue also will be resolved. So we have two options for 6.2 release: to finish this one and fix new one for next releases or close it and concentrate on new issue. Kris - will you take a look at this and decide what we should do for 6.2? The underlying issue is that when the user renames a process variable which is linked to a DataInput, the sourceRef of the dataInputAssociation isn't updated, so it still refers to the old process variable, which is now invalid (https://bugzilla.redhat.com/show_bug.cgi?id=1273871). The bug I reported here and made the commits for is that after renaming the process variable, the DataIOEditor still showed a link to the old process variable. The fix involved removing the link to the old process variable when the DataIOEditor is opened. This is a limited fix. To fix the issue fully, we'd need to update all references to process variables whenever a process variable is renamed, i.e. fix https://bugzilla.redhat.com/show_bug.cgi?id=1273871 I spoke to Tiho about this and he indicated that resolving this is not a minor thing. That's why I wonder whether we should close this BZ for 6.2 and fix BZ1273871 for the next release. Note that this isn't a regression. This behaviour was present in bpms6.1. I suggest we close this one of as a sufficient (temporary) fix, and keep the other one open until we implement full refactoring support. So putting back to ON_QA, so they can close this off as verified for 6.2, wdyt? OK, let's do it in this way. |