Développement #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.
Files
Related issues
History
Updated by Paul Marillonnet almost 3 years ago
- Related to Développement #65612: stocker l’origine (ou les multiples origines) ayant mené à la vérification d’un attribut donné pour un usager added
Updated by Paul Marillonnet almost 3 years ago
- File 0001-csv_import-set-verification-source-for-user-attribut.patch 0001-csv_import-set-verification-source-for-user-attribut.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Paul Marillonnet almost 3 years ago
- Assignee set to Benjamin Dauvergne
(Basé sur #65612, bien sûr.)
Updated by Paul Marillonnet almost 3 years ago
- Assignee changed from Benjamin Dauvergne to Paul Marillonnet
Updated by Benjamin Dauvergne almost 3 years ago
- Status changed from Solution proposée to 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.
Updated by Paul Marillonnet over 2 years ago
- Status changed from Information nécessaire to En cours
Updated by Paul Marillonnet over 2 years ago
- File 0001-csv-link-attribute-object-to-relevant-headers-66053.patch 0001-csv-link-attribute-object-to-relevant-headers-66053.patch added
- File 0002-csv-set-verification-source-for-user-attributes-6605.patch 0002-csv-set-verification-source-for-user-attributes-6605.patch added
- Status changed from En cours to 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).
Updated by Benjamin Dauvergne over 2 years ago
J'ai rebasé (3 lignes de modifications dans les tests au niveau de CsvField "phone" qui est devenu un attribut).
Updated by Benjamin Dauvergne over 2 years ago
- Status changed from Solution proposée to 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.
Updated by Paul Marillonnet over 2 years ago
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.
Updated by Paul Marillonnet over 2 years ago
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.