Defect #3016
closedProvisionining archive - false is displayed as empty value
90%
Description
Tested on 11.2.3 and 11.2.1
Changes in the boolean attribute (e.g. when disabling an account) displays "false" incorrectly as empty space in the provisioning archive, see:
Files
Updated by Roman Kučera over 2 years ago
- Target version changed from 12.2.0 to 13.0.0
Updated by Tomáš Doischer over 2 years ago
- Status changed from New to In Progress
- Assignee changed from Peter Štrunc to Tomáš Doischer
Updated by Tomáš Doischer over 2 years ago
- % Done changed from 0 to 30
This is not a frontend issue - the false value for Boolean is represented as null in IcAttribute. I have to find the right place to fix it.
Updated by Tomáš Doischer over 2 years ago
- % Done changed from 30 to 70
I have made a change on the backend where the empty Boolean value is now translated to false. On the frontend, I had to change the basic null checking to comparison with undefined.
This seems to work well during testing. I tested it with a virtual system and a PostgreSQL table. There are some caveats:
- this will not work retroactively but only for new provisionings
- for a virtual system, the attribute type has to be Boolean - both in the schema and in the form definition. This one I'm worried about because it will not be backward compatible. By default, you can now add a new attribute to a virtual system and only set it as Boolean in the schema, let it be Text in the form definition and everything will work fine. This will change because it will fail now. After an upgrade, those who added their custom attributes for virtual systems and set them to type Boolean will see failures in provisioning and will need to manually change the attribute type in the form definition. I don't see any way that we can prevent that. The one good news is that this will not be a problem for the ENABLE attribute.
We will discuss it before a review.
PR: https://github.com/bcvsolutions/CzechIdMng/pull/218
@kucerar, can you give me feedback please?
Updated by Roman Kučera over 2 years ago
- Status changed from In Progress to Resolved
- % Done changed from 70 to 90
LGTM, works nicely.
Changelog is updated to.
Updated by Tomáš Doischer almost 2 years ago
- Status changed from Resolved to Closed