camunda / camunda-modeler

An integrated modeling solution for BPMN, DMN and Forms based on bpmn.io.
https://camunda.com/products/modeler
MIT License
1.5k stars 482 forks source link

Warning "unknown attribute <background-color>" #2599

Closed hinneLinks closed 2 years ago

hinneLinks commented 2 years ago

Describe the Bug

Coloring the named Paths after a Gateway, saving the Model and reopen It, creates a Warning:

[ FILE_PATH ] unknown attribute <background-color> [ warning ]

Its not just the Warning, our automated Unittests using these Models fail, because of the created "background-color"-Attributes. See full atttached Stacktrace: bug-background-color-camunda-bpm-unittest.txt

* cvc-complex-type.3.2.2: Attribut 'background-color' darf nicht in Element 'bpmndi:BPMNEdge' vorkommen. | resource process/Process.bpmn | line 215 | column 170

Steps to Reproduce

  1. Create a simple Model with named Paths on a Gatway (see test diagram)
  2. Select every Element with STRG + A (including the Outgoing Paths of the first Gateway) and give them a Color
  3. Save the File
  4. Reopen the file, now the Log opens with two Warnings, both related to the outgoing Paths of the Gateway [ FILE_PATH ] unknown attribute <background-color> [ warning ] [ FILE_PATH ] unknown attribute <background-color> [ warning ]

Expected Behavior

No Warnings (e.g. no invalid elements) are created when saving and reopening a File with colored Elements. Models created by Camunda-Modeler run without Parsing-Errors in Camunda-Unittest and Engine.

Environment I tried both 4.11.1 und 4.10.0 Versions of Camunda-Modeler

p.s.: If necessary/Helpful i could also create an Jira-Ticket at jira.camunda.com since we are an enterprise customer for the Engine. pps: This Bug Might be related to #2452 but we are only using Camunda Modeler to edit our Models.

nikku commented 2 years ago

Thanks for your bug report. We'll have a look and get back to you soon.

smbea commented 2 years ago

Thank you for your contribution. I was able to reproduce the issue. We'll look into this.

nikku commented 2 years ago

Will be closed via bpmn-js bump.