Uploaded image for project: 'camunda BPM'
  1. camunda BPM
  2. CAM-8074

Exception message is not shown when a batch operation fails in Cockpit

    Details

    • Type: Bug Report
    • Status: Closed
    • Priority: L3 - Default
    • Resolution: Fixed
    • Affects Version/s: 7.8.0-alpha2
    • Fix Version/s: 7.8.0, 7.8.0-alpha3
    • Component/s: cockpit
    • Labels:
      None

      Description

      Steps to reproduce:
      1) go to batch operation page in cockpit
      2) execute for example the operation "set retries of jobs ..."
      3) add the search criteria "Finished Only"
      4) execute the batch operation

      Problem:
      The batch operation fails, but the exception message is not visible in cockpit (see attached screenshot)

      Expected:
      The exception message is shown.

      Hint:
      The same behavior can also observed in case of other batch operations.

        Activity

        roman.smirnov Smirnov Roman created issue -
        roman.smirnov Smirnov Roman made changes -
        Field Original Value New Value
        Assignee Seif Ghezala [ seif.ghezala ]
        svetlana.dorokhova Svetlana Dorokhova made changes -
        Affects Version/s 7.8.0-alpha2 [ 14906 ]
        roman.smirnov Smirnov Roman made changes -
        Rank Ranked higher
        roman.smirnov Smirnov Roman made changes -
        Rank Ranked higher
        roman.smirnov Smirnov Roman made changes -
        Rank Ranked higher
        roman.smirnov Smirnov Roman made changes -
        Rank Ranked higher
        seif.ghezala Seif Ghezala made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        seif.ghezala Seif Ghezala made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Original Estimate 0 minutes [ 0 ]
        Remaining Estimate 0 minutes [ 0 ]
        Assignee Seif Ghezala [ seif.ghezala ] Sebastian Stamm [ sebastian.stamm ]
        Resolution Fixed [ 1 ]
        Hide
        sebastian.stamm Sebastian Stamm added a comment -

        I think ['unknown', 'payload'].indexOf(executionCtrl.getErrorType()) <= -1 can never be true as the getErrorType method always returns either payload or unknown.

        We can probably get rid of the getErrorType method altogether as we don't display different messages depending on the type of the error.

        Show
        sebastian.stamm Sebastian Stamm added a comment - I think ['unknown', 'payload'].indexOf(executionCtrl.getErrorType()) <= -1 can never be true as the getErrorType method always returns either payload or unknown. We can probably get rid of the getErrorType method altogether as we don't display different messages depending on the type of the error.
        sebastian.stamm Sebastian Stamm made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Assignee Sebastian Stamm [ sebastian.stamm ] Seif Ghezala [ seif.ghezala ]
        seif.ghezala Seif Ghezala made changes -
        Status Reopened [ 4 ] In Progress [ 3 ]
        seif.ghezala Seif Ghezala made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Assignee Seif Ghezala [ seif.ghezala ] Sebastian Stamm [ sebastian.stamm ]
        Resolution Fixed [ 1 ]
        Hide
        sebastian.stamm Sebastian Stamm added a comment - - edited

        The getErrorType method is not used anymore so we should remove it from the execution component and the tests. Also, the indentation of the template file should be adjusted.

        Show
        sebastian.stamm Sebastian Stamm added a comment - - edited The getErrorType method is not used anymore so we should remove it from the execution component and the tests. Also, the indentation of the template file should be adjusted.
        sebastian.stamm Sebastian Stamm made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Assignee Sebastian Stamm [ sebastian.stamm ] Seif Ghezala [ seif.ghezala ]
        seif.ghezala Seif Ghezala made changes -
        Status Reopened [ 4 ] In Progress [ 3 ]
        seif.ghezala Seif Ghezala made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Assignee Seif Ghezala [ seif.ghezala ] Sebastian Stamm [ sebastian.stamm ]
        Resolution Fixed [ 1 ]
        sebastian.stamm Sebastian Stamm made changes -
        Status Resolved [ 5 ] In Test [ 10004 ]
        sebastian.stamm Sebastian Stamm made changes -
        Assignee Sebastian Stamm [ sebastian.stamm ] Tassilo Weidner [ tassilo.weidner ]
        tassilo.weidner Tassilo Weidner made changes -
        Status In Test [ 10004 ] Closed [ 6 ]
        tassilo.weidner Tassilo Weidner made changes -
        Assignee Tassilo Weidner [ tassilo.weidner ]
        tassilo.weidner Tassilo Weidner made changes -
        Fix Version/s 7.8.0-alpha3 [ 14907 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            roman.smirnov Smirnov Roman
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development