Project

General

Profile

Actions
Copy link

Defect #3218

closed

Some reports contain only single result line

Added by Alena Peterová over 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
High
Assignee:
Tomáš Doischer
Category:
Report
Target version:
13.0.0
Start date:
11/03/2022
Due date:
% Done:

100%

Estimated time:
Affected versions:
12.2.2
Owner:

Description

Tested on 12.2.3
Some product reports contain only single line (or lines for single result), even though there are multiple objects to report.
identity-report, identity-eav-report, core-monitoring-report, provisioning-operation-report

changes-on-system-report looks fine.

Reported by our customer.

Files

reports.png (122 KB) reports.png Alena Peterová, 11/03/2022 11:26 AM
Actions
Copy link
 #1

Updated by Tomáš Doischer over 1 year ago

  • Status changed from New to In Progress
Actions
Copy link
 #2

Updated by Tomáš Doischer over 1 year ago

  • % Done changed from 0 to 20

This appears to be an issue with all reports.

After processing the first item from a loaded page, reports try to update their state to see if they were perhaps canceled by the user. For some reason, the LRTs have running set to false.

Actions
Copy link
 #3

Updated by Alena Peterová over 1 year ago

Interesting thing:
Identities with complex combination with contracts attributes. (identity-complex-report) is OK
Identities, their main contract and their roles (identity-contract-role-report) has this issue.

Actions
Copy link
 #4

Updated by Tomáš Doischer over 1 year ago

This still works in 12.1.4 but fails in 12.2. Something marks the tasks as not running and it is not obvious what does this. It also doesn't happen always.

Some reports might still work. Those that do work because they don't always call updateState() which stops the task.

Actions
Copy link
 #5

Updated by Tomáš Doischer over 1 year ago

  • Assignee changed from Tomáš Doischer to Roman Kučera
  • Target version set to 13.0.0
  • % Done changed from 20 to 80

The issue is related to datasource changes in 12.2. The report generation needed to be covered by a transaction.

PR: https://github.com/bcvsolutions/CzechIdMng/pull/278

@kucerar, can you please give me feedback?

Actions
Copy link
 #6

Updated by Tomáš Doischer over 1 year ago

  • Status changed from In Progress to Needs feedback
Actions
Copy link
 #7

Updated by Roman Kučera over 1 year ago

  • Status changed from Needs feedback to Resolved
  • Assignee changed from Roman Kučera to Tomáš Doischer
  • % Done changed from 80 to 100

LGTM, merged, thx

Actions
Copy link
 #8

Updated by Tomáš Doischer about 1 year ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF