Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
hinneLinks opened this issue Nov 30, 2021 · 3 comments · Fixed by #2709
Closed

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

hinneLinks opened this issue Nov 30, 2021 · 3 comments · Fixed by #2709
Labels
bug Something isn't working
Milestone

Comments

@hinneLinks
Copy link

hinneLinks commented Nov 30, 2021

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

  • OS: Windows 10
  • Camunda Modeler Version: [4.11.1, 4.10.0]
  • Execution Platform: [Camunda Enterprise Platform 7.15.6]
  • Installed plug-ins: [4.11.1: None, 4.10.0: BPMN Linter, Camunda Property Infos, Tooltip Plugin, Transaction Boundaries]

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.

@hinneLinks hinneLinks added the bug Something isn't working label Nov 30, 2021
@nikku
Copy link
Member

nikku commented Nov 30, 2021

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

@smbea smbea added help wanted Extra attention is needed and removed help wanted Extra attention is needed labels Dec 3, 2021
@smbea
Copy link
Contributor

smbea commented Dec 3, 2021

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

@smbea smbea added the backlog Queued in backlog label Dec 3, 2021
nikku added a commit to bpmn-io/bpmn-js that referenced this issue Dec 3, 2021
@nikku nikku added the needs review Review pending label Dec 3, 2021 — with bpmn-io-tasks
@nikku nikku removed the backlog Queued in backlog label Dec 3, 2021
fake-join bot pushed a commit to bpmn-io/bpmn-js that referenced this issue Dec 3, 2021
nikku added a commit to bpmn-io/bpmn-js that referenced this issue Dec 3, 2021
@nikku nikku added the fixed upstream Requires integration of upstream change label Dec 3, 2021 — with bpmn-io-tasks
@nikku nikku removed the needs review Review pending label Dec 3, 2021
nikku added a commit to bpmn-io/bpmn-js that referenced this issue Dec 3, 2021
@nikku
Copy link
Member

nikku commented Dec 3, 2021

Will be closed via bpmn-js bump.

@nikku nikku closed this as completed Dec 3, 2021
@bpmn-io-tasks bpmn-io-tasks bot removed the fixed upstream Requires integration of upstream change label Dec 3, 2021
@nikku nikku reopened this Dec 3, 2021
@nikku nikku added the fixed upstream Requires integration of upstream change label Dec 3, 2021
@nikku nikku added this to the M52 milestone Dec 3, 2021
nikku added a commit to bpmn-io/bpmn-js that referenced this issue Jan 13, 2022
fake-join bot pushed a commit to bpmn-io/bpmn-js that referenced this issue Jan 14, 2022
barmac added a commit that referenced this issue Jan 26, 2022
Closes #2664
Closes #2374
Closes #2524
Closes #2641
Closes #2599
Closes #2585
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed fixed upstream Requires integration of upstream change labels Jan 26, 2022
barmac added a commit that referenced this issue Jan 27, 2022
Closes #2664
Closes #2374
Closes #2524
Closes #2641
Closes #2599
Closes #2585
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Jan 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants