Bug 1130292
| Summary: | Guided rule editor does not save any then clause values, but at least it tells me it won't save them | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Retired] JBoss BRMS Platform 6 | Reporter: | Jim Tyrrell <jtyrrell> | ||||||||||
| Component: | Business Central | Assignee: | Toni Rikkola <trikkola> | ||||||||||
| Status: | CLOSED EOL | QA Contact: | Marián Macik <mmacik> | ||||||||||
| Severity: | unspecified | Docs Contact: | |||||||||||
| Priority: | high | ||||||||||||
| Version: | 6.0.2 | CC: | jtyrrell, kverlaen, lpetrovi | ||||||||||
| Target Milestone: | DR1 | ||||||||||||
| 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:07:57 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: |
|
||||||||||||
Created attachment 926888 [details]
It doesn't always seem I get this error, but for sure my data is gone!!!
Can't reproduce with BRMS 6.0.2 in Chrome 36 and Firefox 31. Haven't tried Safari. That is mighty annoying, I have it broken in Safari. Breaks in Firefox 31 on Max OS X, hmm seems like I am special. What do you want me to do to help get this resolved. I cannot reproduce either. The popup "Are you sure you want to discard unsaved data" is shown when you make a change to the rule's definition and try to close it. Even if you change something and then set it back to it's original value you will still receive the prompt. Can you please provide steps to reproduce? I have tried to reproduce it on ER4 with EAP 6.3, but I wasn't able to. 1. I created a rule and saved it. 2. I changed something and verified that source changed too. Then I changed definition again so source looked exactly like in the first point after saving. 3. I closed the editor and no message was displayed. I was about to mark it as verified but then I noticed something different, but related to this issue. The issue is this: When you change a value of a setter (so the guided editor is displaying "Set value of MyClass") and you try to close the editor - it warns you: "Are you sure you want to discard unsaved data?". This is expected behaviour. But when you change a value of a method parameter (so the guided editor is displaying Call [p.setVariable]) and you try to close the editor - it simply closes the editor and does not warn you. I will attach an image of my rule - I am using WorkflowProcessInstance setVariable method for method call and class MyClass with one String field called string, so I can call setString() setter on it. I will also attach source code of my rule just to make things clear. I've also tried it with a call of a method which has one parameter - effect is the same (no warning when you close the editor), so it is not related with the number of parameters. I think it should be fixed. I will mark it as verified then. Created attachment 991610 [details]
The rule
Created attachment 991611 [details]
Source code of the rule
Had to revert the changes into drools-workbench-models-guided-dtree. They are not mandatory for this fix. http://github.com/droolsjbpm/drools/commit/6ea324fdb It works as intended. Verified with 6.2.0.ER3. |
Created attachment 926887 [details] Values in the editor. Description of problem: Create some values in the then clause as seen in the screen shot. Then click the save button, it says it saved the rule. Click the x then it shows a message saying some values won't be saved. Re open and the inputed values are gone. Version-Release number of selected component (if applicable): How reproducible: Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info: