Uploaded image for project: 'Camunda Optimize'
  1. Camunda Optimize
  2. OPT-1478

Optimize craches when selecting a process defintion with bpmn2 tage

    • Icon: Bug Report Bug Report
    • Resolution: Fixed
    • Icon: L3 - Default L3 - Default
    • 2.2.0
    • None
    • frontend
    • None

      When

      • Go to analysis page
      • Selecting a process definition with bpmn2:process tag such as 'process request'
        Then
      • Optimize crashes
        Expected
      • process get selected with crashing

        This is the controller panel for Smart Panels app

            [OPT-1478] Optimize craches when selecting a process defintion with bpmn2 tage

            Does it make sense to get the process element by ignoring the namespace? We could try getElementsByTagNameNS with "*" as the namespaceURI. Maybe that works. I would also rename the variable procesNode to processNode

            Sebastian Stamm added a comment - Does it make sense to get the process element by ignoring the namespace? We could try getElementsByTagNameNS with "*" as the namespaceURI. Maybe that works. I would also rename the variable procesNode to processNode

            Great idea. Thanks.

            Omran Abazeed added a comment - Great idea. Thanks.

            Found something else, sorry

            return processNode ? processNode.getAttribute('name') : '' || '' can return null if the process has no name attribute, because the || operator binds stronger than the inline if. You can try this out by running true ? null : '' || '' , which will return null. You need to use parentheses (true ? null : '') || '' for it to work.

            Sebastian Stamm added a comment - Found something else, sorry return processNode ? processNode.getAttribute('name') : '' || '' can return null if the process has no name attribute, because the || operator binds stronger than the inline if. You can try this out by running true ? null : '' || '' , which will return null. You need to use parentheses (true ? null : '') || '' for it to work.

              Unassigned Unassigned
              omran.abazeed Omran Abazeed
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: