Development #66053
csv_import: stocker une information de source de vérification des attributs de l’usager
0%
Description
On a la base nécessaire dans #65612. Dans l’import csv il y a la possibilité de désigner des attributs usagers comme vérifiés.
On peut alors stocker le fait que la vérification provient de l’import csv.
Fichiers
Demandes liées
Historique
Mis à jour par Paul Marillonnet il y a presque 2 ans
- Lié à Development #65612: stocker l’origine (ou les multiples origines) ayant mené à la vérification d’un attribut donné pour un usager ajouté
Mis à jour par Paul Marillonnet il y a presque 2 ans
- Fichier 0001-csv_import-set-verification-source-for-user-attribut.patch 0001-csv_import-set-verification-source-for-user-attribut.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Paul Marillonnet il y a presque 2 ans
- Assigné à mis à Benjamin Dauvergne
(Basé sur #65612, bien sûr.)
Mis à jour par Paul Marillonnet il y a presque 2 ans
- Assigné à changé de Benjamin Dauvergne à Paul Marillonnet
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution proposée à Information nécessaire
Reproposer quand #65612 sera terminé. Mais je note déjà que la source est posée que l'attribut soit vérifié ou pas, c'est bizarre.
Mis à jour par Paul Marillonnet il y a plus d'un an
- Statut changé de Information nécessaire à En cours
Mis à jour par Paul Marillonnet il y a plus d'un an
- Fichier 0001-csv-link-attribute-object-to-relevant-headers-66053.patch 0001-csv-link-attribute-object-to-relevant-headers-66053.patch ajouté
- Fichier 0002-csv-set-verification-source-for-user-attributes-6605.patch 0002-csv-set-verification-source-for-user-attributes-6605.patch ajouté
- Statut changé de En cours à Solution proposée
Ok, version corrigée.
Dans 0002 un peu plus de tuyauterie car il faut gérer le cas où la valeur de l’attribut déjà existant est la même mais il n’est pas encore vérifié.
Et avant ça 0001 trivial qui facilite un peu l’affaire, en stockant directement dans l’entête l’objet attribut lié (au lieu d’un simple flag indiquant que la valeur est à aller écrire dans un attribut).
Mis à jour par Benjamin Dauvergne il y a plus d'un an
J'ai rebasé (3 lignes de modifications dans les tests au niveau de CsvField "phone" qui est devenu un attribut).
Mis à jour par Benjamin Dauvergne il y a environ un an
- Statut changé de Solution proposée à En cours
J'ai corrigé les tests et UserCsvImporter pour que .run() retourne False pour n'importe quelle erreur, ensuite il y a un souci maintenant que phone est à la fois un attribut et une propriété de la classe CustomUser qui fait que le statut vérifié n'apparaît pas dans l'objet AttributeValue (parce que header.field is True et donc header.attribute n'est jamais ni défini ni utilisé).
Des choses à revoir donc.
Mis à jour par Paul Marillonnet il y a environ un an
Benjamin Dauvergne a écrit :
[…] il y a un souci maintenant que phone est à la fois un attribut et une propriété de la classe CustomUser qui fait que le statut vérifié n'apparaît pas dans l'objet AttributeValue (parce que
header.field is Trueet doncheader.attributen'est jamais ni défini ni utilisé).
Ok, je vais regarder ce que tu as ajouté dans cette nouvelle branche et je corrige ce souci sur l’état de vérification du téléphone du à l’ambiguïté entre attribut de profil étendu et attribut immédiat de la classe User custom.
Mis à jour par Paul Marillonnet il y a environ un an
Et j’ai poussé les correctifs nécessaires dans une troisième branche au dessus de la tienne, je vais ranger proprement en commits distincts et proposer à nouveau les patches ici.