Bug 768196

Summary: Process Variables currently have be entered free-form in the web designer
Product: [JBoss] JBoss Enterprise BRMS Platform 5 Reporter: Babak Mozaffari <bmozaffa>
Component: jBPM 5Assignee: Kris Verlaenen <kverlaen>
Status: VERIFIED --- QA Contact: Lukáš Petrovický <lpetrovi>
Severity: high Docs Contact:
Priority: unspecified    
Version: BRMS 5.3.0.GA   
Target Milestone: ---   
Target Release: BRMS 5.3.0.GA   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 768198 (view as bug list) Environment:
Last Closed: Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Babak Mozaffari 2011-12-16 01:03:20 UTC
Description of problem:

To enter a process variable, the user needs to know the internal format and type the variables types/names as follows:

var1:String,var2:Integer

This format is unintuitive and error-prone. It is best that a dialog would be created for entering process variables where the variable name would be input in one field while another field would allow the user to pick from common types (first the default ones such as String and Integer, then possibly imported fact types) or enter a class name. An example of this can be found on the Edit tab of a package in Guvnor when you try to add a Global, except that including common types such as String and Integer is critical in this case.


Version-Release number of selected component (if applicable):
ER6

How reproducible:
Consistent


Steps to Reproduce:
1. Open a process in the web designer
2. Click on any node in the jBPM diagram
3. Click on the white space behind the process diagram
4- In the right panel, under more properties, try to enter variables in the "variable definitions" field.
  
Actual results:
A large text box for free-form entry


Expected results:
A user-friendly dialog


Additional info:

Comment 1 Jeff DeLong 2012-01-31 20:46:21 UTC
I would like to expand this to include the specification and mapping of task variable. Currently it requires the listing of the variable in a DataInputSet, DataOutputSet, and Assignments. This is too error prone. And if you make a mistake, the, your get an error on saving and an exception in the log. Business analysts will not  be able to add task variables this way.

Similar to defining the process variables, this requires an user-friendly box / table where the user can specify and map process variables onto task variables.

I also changed the severity to high.

Comment 2 Tihomir Surdilovic 2012-02-03 15:45:39 UTC
The process variable editor has been added (available at people.redhat.com/tsurdilo/designer/master). It is used to define process variables as well as variables in subprocesses. 

Still working on the rest of the editors.

Comment 3 Ryan Zhang 2012-02-15 09:12:18 UTC
Please verify the issue on 5.3 ER4.