Task #2846
closed
Reports IdentityAccountChangesReportExecutor and IdentityPasswordChangeReportExecutor doesn't contain all changes
100%
Description
Tested on version 2.2.0
The reports contain only some random records. E.g. I have 516 records in the provisioning archive, 37 successful creates, but the report contains only 33 records.
There is a problem with pagination.
Updated by Alena Peterová over 3 years ago
- Subject changed from Report IdentityAccountChangesReportExecutor doesn't contain all changes to Reports IdentityAccountChangesReportExecutor and IdentityPasswordChangeReportExecutor doesn't contain all changes
- Description updated (diff)
Updated by Tomáš Doischer over 3 years ago
- Assignee set to Tomáš Doischer
- Target version set to 2.3.0
Updated by Tomáš Doischer over 3 years ago
- Status changed from New to In Progress
Updated by Tomáš Doischer over 3 years ago
- Status changed from In Progress to Needs feedback
- % Done changed from 0 to 80
Fixed. I also refactored the code a little bit, it should now be less memory hungry. Also, the counters showing processed items were not functioning properly, I fixed that too.
Implemented here: https://git.bcvsolutions.eu/modules/reports/-/commit/e410c2db392654465d10a87117184324e9bf2210
@kucerar, can you give me feedback please? I can show you that the reports work over video, you need quite a lot of data to simulate the previous issue.
Updated by Roman Kučera over 3 years ago
LGTM :) thanks for fix.
Report now reporting correct data.
Updated by Tomáš Doischer over 3 years ago
- Status changed from Needs feedback to Resolved
- Target version changed from 2.3.0 to 2.2.1
- % Done changed from 80 to 100
Merged to develop.
Updated by Tomáš Doischer over 3 years ago
- Status changed from Resolved to Closed
Version 2.2.1 released.