Project

General

Profile

Actions
Copy link

Defect #1866

closed

Null pointer in CompareValueWithSystemReport

Added by Ondřej Kopr over 5 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Peter Štrunc
Target version:
2.7.1
Start date:
09/19/2019
Due date:
% Done:

100%

Estimated time:
Affected versions:
2.8.0
Owner:

Description

Moved version of report CompareValueWithSystemReport isn't the last project version and contains bug with null pointer in merged values. This bug was repaired in project version of this report, but someone didn't moved right version :(

Please update this behavior.

Example result of report before:

Example result of report after:

All these users has null in multivalued attribute:

Affected version: 1.1.0

Files

Download all files
after.png (50.8 KB) after.png Ondřej Kopr, 09/19/2019 08:12 AM
before.png (64.1 KB) before.png Ondřej Kopr, 09/19/2019 08:12 AM
nullValue.png (14.7 KB) nullValue.png Ondřej Kopr, 09/19/2019 08:15 AM
Actions
Copy link
 #1

Updated by Ondřej Kopr almost 5 years ago

  • Assignee changed from Patrik Stloukal to Ondřej Kopr
Actions
Copy link
 #2

Updated by Roman Kubica about 4 years ago

  • Assignee changed from Ondřej Kopr to Peter Štrunc
  • Affected versions 2.8.0 added

Possible null pointer error appears when we are comparing roles in IdM and AD groups in the end system. If there are no AD groups (any multivalued attribute) assigned to a user in the end system, then the whole Compare ROW fails without returning error. It breaks whole row for the identity in XLS report, not only the cell with null multivalued attribute.

User has no memberOf (or null) attribute in AD = report fails

Actions
Copy link
 #3

Updated by Peter Štrunc almost 4 years ago

  • Status changed from New to In Progress
  • % Done changed from 0 to 30
Actions
Copy link
 #5

Updated by Peter Štrunc almost 4 years ago

  • Status changed from In Progress to Needs feedback
  • % Done changed from 30 to 80

Fixed in pstrunc/1866-npe-CompareValueWithSystemReport

Actions
Copy link
 #6

Updated by Peter Štrunc almost 4 years ago

  • Assignee changed from Peter Štrunc to Roman Kubica

@kubicar would you please review this ticket? Thank you

Actions
Copy link
 #7

Updated by Roman Kubica almost 4 years ago

  • Status changed from Needs feedback to In Progress
  • Assignee changed from Roman Kubica to Peter Štrunc
  • Target version set to 2.8.0
  • % Done changed from 80 to 90

Tested in proper environment and code review done. OK.

Actions
Copy link
 #8

Updated by Peter Štrunc almost 4 years ago

  • Status changed from In Progress to Closed
  • Target version changed from 2.8.0 to 2.7.1
  • % Done changed from 90 to 100

Merged to develop

Actions
Copy link

Also available in: Atom PDF