Uploaded image for project: 'jBPM'
  1. jBPM
  2. JBPM-8450

Stunner - Relax constraint on signal name

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Critical Critical
    • None
    • 7.20.0.Final
    • Designer
    • 1
    • NEW
    • NEW
    • Hide
      1. Open stunner bpmn project
      2. add Signal node into the canvas
      3. in properties set Signal to 'Milestone 2: Order shipped'

      Actual result

      Validation fails on checking signal name value

      Expected result

      Validation does not fail on checking signal name value

      Show
      Open stunner bpmn project add Signal node into the canvas in properties set Signal to 'Milestone 2: Order shipped' Actual result Validation fails on checking signal name value Expected result Validation does not fail on checking signal name value
    • 2019 Week 23-25, 2019 Week 47-49 (from Nov 18), 2019 Week 50-52 (from Dec 9)

      Once migrating the it-orders case mgmt example from the legacy designer to the new one, an issue appeared: It seems there is a validation error on the process, the example is using two signals with symbol ":" and spaces.

      This warning can be ignored and the process can still be saved, but you can't recreate it from scratch as the editor will prevent you from creating a new signal with colon or spaces in it. The fact that we use the node name as the signal name in case management (to trigger that node) makes this a more severe issue as you're more likely to use spaces and possibly other special characters in node names. The underlying XML does not have any restrictions on signal name (xsd:string), and the engine also accepts any string afaik, so as long as the parser can support it as an attribute value we can support more characters.
      <bpmn2:signal id="_825e5b59-8a03-369d-b7ed-e5e3824400c9" name="Milestone 2: Order shipped"/>

              dtikhomi@redhat.com Dmitrii Tikhomirov
              romartin@redhat.com Roger Martinez
              Votes:
              1 Vote for this issue
              Watchers:
              9 Start watching this issue

                Created:
                Updated:
                Resolved: