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)

allow_identity_delete.patch (3.1 KB) - added by tensor 5 years ago.

Download all attachments as: .zip

Change History (7)

Changed 5 years ago by tensor

comment:1 Changed 5 years ago by alec

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

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]).

Note: See TracTickets for help on using tickets.