Project

General

Profile

Actions
Copy link

Defect #2292

closed

Role import with multivalue values separated with | creates multiple roles with one character names

Added by Vladimír Kotýnek over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Tomáš Doischer
Target version:
2.4.0
Start date:
06/03/2020
Due date:
% Done:

100%

Estimated time:
Affected versions:
Owner:

Description

I am using ImportRolesFromCSVExecutor from Extras modul 2.2.0.
I have CSV file:

name_for_user;guarantee;guarantee_type;guarantee_role;guarantee_role_type
Approved_role;user1;first-approver;;
Approved_role;user1;second-approver;;
Second_role;;;Approver_Role1;first-approver
Third_role;user2;first-approver;Approver_role2;second-approver
Fourth_role;user3|user4|user5;first-approver;;

In configuration of the LRT I set the separator of multivalued attirbutes as |
When I run this LRT none of the roles is updated. But there waw about 36 new IdmRole entities created. Those new roles have codes and names that contains only one char, e.g. "T" or "o". Some of those new roles are set as guarantors of some other of those new roles.

Please examine this behavior and eventualy pepare a bugfix.

Related issues

Related to extras - Task #2274: Refactor ImportRolesFromCSVExecutor and CSVToIdMClosedPeter Štrunc05/26/2020

Actions
Actions
Copy link
 #1

Updated by Vladimír Kotýnek over 4 years ago

  • Related to Task #2274: Refactor ImportRolesFromCSVExecutor and CSVToIdM added
Actions
Copy link
 #2

Updated by Tomáš Doischer over 4 years ago

  • Status changed from New to Needs feedback
  • Assignee set to Tomáš Doischer
  • Target version set to 2.4.0
  • % Done changed from 0 to 80

The separator was not escaped in the code. I am not sure why the parsing behaved this way with unescaped | but I fixed it together with #2274.

Actions
Copy link
 #3

Updated by Peter Štrunc over 4 years ago

  • Status changed from Needs feedback to Closed
  • % Done changed from 80 to 100

OK 2274 is closed. closing this too.

Actions
Copy link

Also available in: Atom PDF