Projet

Général

Profil

Bug #22586

action de modif de profil : transmission des champs date à authentic

Ajouté par Frédéric Péters il y a environ 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
16 mars 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

On fait juste payload = new_data.copy() mais quand il y a des champs de type date authentic en face va plutôt attendre une chaine de caractère yyyy-mm-dd.


Fichiers

Révisions associées

Révision 3e7c60d4 (diff)
Ajouté par Frédéric Péters il y a environ 6 ans

workflows: add date support for idp profile change (#22586)

Historique

#1

Mis à jour par Frédéric Péters il y a environ 6 ans

(vient après #22578).

#2

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

Rien à voir mais l.177 vim me signal un sys.exc_info() où sys est indéfini.

Ça ça devrait aller dans l'autre ticket:

                # also change initial value to the converted one, as the
                # initial dictionary is used when sending the profile changes
                # to the identity provider.
                new_data[field.varname] = field_value

le comentaire concernant les date il devrait expliquer pourquoi on a du struct_time après le coup du dessus parce que je m'y perds, de plus struct_time ça peut aussi bien être un datetime qu'un date, là dans le use case d'authentic on considère qu'on aura rarement un datetime mais si ça devait arriver... pas moyen d'avoir un simple datetime.date() depuis la source dans new_data ?

#3

Mis à jour par Frédéric Péters il y a environ 6 ans

Ça ça devrait aller dans l'autre ticket:

Non, c'est explicitement pour celui-ci, l'autre n'est pas concerné par l'idp.

La situation struct_time/datetime est complexe (il y a #9876 avec un patch pour la faire évoluer mais depuis longtemps bloqué).

pas moyen d'avoir un simple datetime.date() depuis la source dans new_data ?

Ce serait l'objectif mais c'est l'évolution #9876, une toute autre affaire.

#4

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

Ok, ack.

#5

Mis à jour par Frédéric Péters il y a environ 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 3e7c60d4d473073117d8aa1838127ff242c5acde
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Mar 20 09:04:04 2018 +0100

    workflows: add date support for idp profile change (#22586)
#6

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF