Opened 5 years ago
Closed 5 years ago
#1485435 closed Feature Patches (fixed)
allow to delete identity even if multiple_identities is set to false
| Reported by: | tensor | Owned by: | |
|---|---|---|---|
| Priority: | 5 | Milestone: | 0.2-stable |
| Component: | User Interface | Version: | 0.2-beta |
| Severity: | trivial | Keywords: | |
| Cc: |
Description
When user has several multiple identities, and admin changes "multiple_identities" from true to false, user cannot delete identities at all. The wording of "multiple_identities" setting implies that users are allowed to have several identities. If they are not allowed to have several, why prohibit them to delete existing identities to be compliant with the setting?
Or maybe RC should delete all non-default identities if "multiple_identities" is set to false? That would be very rough for users.
Or maybe there should be another setting like "allow_identity_editing". If it exists, it should has something like "identity_query" to query for default or fixed identities from database.
Anyway, I prepared the patch which allows to delete identities regardless of "multiple_identities" setting.
Attachments (1)
Change History (7)
Changed 5 years ago by tensor
comment:1 Changed 5 years ago by alec
comment:2 Changed 5 years ago by tensor
What do you mean by disabling identities?
comment:3 Changed 5 years ago by alec
Exactly 0-3 will be enough, but I was thinking about hidding the Identities whole interface (tab).
comment:4 Changed 5 years ago by tensor
And how user should configure his name, from address and other fields when interface is disabled?
Only via SQL or LDAP query?
comment:5 Changed 5 years ago by alec
This option will be probably for environements where user shouldn't have such possibility. It was requested somewhere, but I think we can skip that option.
comment:6 Changed 5 years ago by alec
- Resolution set to fixed
- Status changed from new to closed
Fixed and implemented identities_level option with 0-3 values ([ec01712a]).

By the way, multiple_identities option should be replaced by some 'identities_level' option with predefined values:
0 - can create/edit/delete identities, can change address (like now)
1 - can create/edit/delete identities, can not change address
2 - can edit (one) identity, can change address (like multiple_identities=true)
3 - can edit (one) identity, can not change address
4 - disable identities at all