Project

General

Profile

Bug #83286

[authn tél] la double écriture colonne et attribut de profil continue d’avoir lieu dans le BO (?)

Added by Paul Marillonnet 8 months ago. Updated 8 months ago.

Status:
Nouveau
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
09 November 2023
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Quand j’édite la page BO d’un usager possédant un attribut de profil phone, la modification de ce dernier est répercutée dans la colonne phone, pourtant dépréciée.


Related issues

Related to Publik - Development #49212: Création de compte avec un numéro de téléphone mobileEn cours01 October 2021

Actions
Related to Authentic 2 - Development #81282: Retrait de la colonne User.phoneEn cours18 September 2023

Actions

History

#1

Updated by Paul Marillonnet 8 months ago

  • Related to Development #49212: Création de compte avec un numéro de téléphone mobile added
#2

Updated by Paul Marillonnet 8 months ago

#3

Updated by Benjamin Dauvergne 8 months ago

Certainement que le formulaire a un préférence pour les attributs natifs du modèle.

#4

Updated by Benjamin Dauvergne 8 months ago

def modelform_factory(model, **kwargs):
    """Build a modelform for the given model,

    For the user model also add attribute based fields.
    """ 

    form = kwargs.pop('form', None)
    fields = kwargs.get('fields') or []
    required = list(kwargs.pop('required', []) or [])
    d = {}
    # KV attributes are only supported for the user model currently
    modelform = None
    if issubclass(model, User):
        if not form:
            form = BaseUserForm
        attributes = models.Attribute.objects.all()
        for attribute in attributes:
            if attribute.name not in fields:
                continue
            d[attribute.name] = attribute.get_form_field()
        for field in app_settings.A2_REQUIRED_FIELDS:
            if field not in required:
                required.append(field)

Est-ce qu'on ne pourrait pas virer User.phone maintenant ?

#5

Updated by Paul Marillonnet 8 months ago

Benjamin Dauvergne a écrit :

[...]

Est-ce qu'on ne pourrait pas virer User.phone maintenant ?

Yes, c’est https://git.entrouvert.org/entrouvert/authentic/pulls/129 😊

Also available in: Atom PDF