entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 39 Releases Activity

fields: implement block field json serialization (#68964) #24

Merged
vdeniaud merged 1 commits from wip/68964-representation-publique-du-conte into main 2023-01-16 09:41:07 +01:00
Conversation 4 Commits 1 Files Changed 4 +180 -3
vdeniaud commented 2023-01-10 17:29:45 +01:00
Owner
Copy Link

(nécessaire également pour les tests)

(nécessaire également pour les tests)
fpeters requested changes 2023-01-10 17:55:45 +01:00
fpeters left a comment
Owner
Copy Link

Merci d'être parti là-dessus.

Merci d'être parti là-dessus.
tests/backoffice_pages/test_carddata.py
@ -664,2 +682,4 @@
}
],
}
fpeters commented 2023-01-10 17:50:14 +01:00
Owner
Copy Link

Je verrais bien un test qui fasse le roundtrip export/import et assure que c'est ok. Là c'est un peu trop éparpillé avec juste dans tests/api/test_formdata.py::test_formdata une rapide vérification de ce qu'on reçoit en représentation json.

Je verrais bien un test qui fasse le roundtrip export/import et assure que c'est ok. Là c'est un peu trop éparpillé avec juste dans tests/api/test_formdata.py::test_formdata une rapide vérification de ce qu'on reçoit en représentation json.
vdeniaud commented 2023-01-11 12:38:20 +01:00
Author
Owner
Copy Link

Yes, ajouté

Yes, ajouté
wcs/api.py Outdated
@ -77,3 +77,3 @@
data.update(data.pop('_unnamed'))
# create a temporary formdata so datasources using previous fields in
# create a temporary feed so datasources using previous fields in
fpeters commented 2023-01-10 17:40:30 +01:00
Owner
Copy Link

C'est l'alimentation (feed) du contexte qui est temporaire, on ne crée pas de "temporary feed".

C'est l'alimentation (feed) du contexte qui est temporaire, on ne crée pas de "temporary feed".
vdeniaud marked this conversation as resolved
wcs/api.py
@ -83,3 +81,2 @@
with get_publisher().substitutions.temporary_feed(transient_formdata, force_mode='lazy'):
with get_publisher().substitutions.temporary_feed(data, force_mode='lazy'):
# complete/adapt field values
fpeters commented 2023-01-10 17:55:16 +01:00
Owner
Copy Link

C'est quand même très délicat ici de passer d'un formdata (avec donc une représentation totale des champs et leurs subtilités) à juste un dictionnaire, ça représente peut-être une absence de test. Je n'oserais pas valider ça.

C'est quand même très délicat ici de passer d'un formdata (avec donc une représentation totale des champs et leurs subtilités) à juste un dictionnaire, ça représente peut-être une absence de test. Je n'oserais pas valider ça.
fpeters commented 2023-01-10 17:56:40 +01:00
Owner
Copy Link

Ah non parfait un test échoue justement là-dessus.

Ah non parfait un test échoue justement là-dessus.
vdeniaud commented 2023-01-10 18:12:32 +01:00
Author
Owner
Copy Link

Il n'échoue pas là dessus, pour qu'il passe ce qu'il faut revert du patch c'est

     with get_publisher().substitutions.temporary_feed(data, force_mode='lazy'):
         # complete/adapt field values
-        for field in fields:
+        for field in formdef.get_all_fields():

Noté ta remarque cependant, je vais reprendre tout ça en gardant un formdata.

Il n'échoue pas là dessus, pour qu'il passe ce qu'il faut revert du patch c'est ``` with get_publisher().substitutions.temporary_feed(data, force_mode='lazy'): # complete/adapt field values - for field in fields: + for field in formdef.get_all_fields(): ``` Noté ta remarque cependant, je vais reprendre tout ça en gardant un formdata.
vdeniaud commented 2023-01-11 12:39:30 +01:00
Author
Owner
Copy Link

Vérification trop rapide, c'était bien la suppression du formdata qui provoquait l'échec du test. Du coup je l'ai dupliqué pour le cas bloc de champs.

Vérification trop rapide, c'était bien la suppression du formdata qui provoquait l'échec du test. Du coup je l'ai dupliqué pour le cas bloc de champs.
vdeniaud force-pushed wip/68964-representation-publique-du-conte from 9dff02f90d to 9c588ce300 2023-01-11 12:37:10 +01:00 Compare
vdeniaud requested review from fpeters 2023-01-11 12:41:12 +01:00
vdeniaud commented 2023-01-11 12:42:30 +01:00
Author
Owner
Copy Link

(le bouton pour demander à nouveau une relecture devrait pas faire passer le ticket redmine en solution proposée ? ou il y a un autre moyen pour faire ça ?)

(le bouton pour demander à nouveau une relecture devrait pas faire passer le ticket redmine en solution proposée ? ou il y a un autre moyen pour faire ça ?)
fpeters commented 2023-01-11 12:56:15 +01:00
Owner
Copy Link

(le bouton pour demander à nouveau une relecture devrait pas faire passer le ticket redmine en solution proposée ? ou il y a un autre moyen pour faire ça ?)

c'est https://dev.entrouvert.org/issues/73091

> (le bouton pour demander à nouveau une relecture devrait pas faire passer le ticket redmine en solution proposée ? ou il y a un autre moyen pour faire ça ?) c'est https://dev.entrouvert.org/issues/73091
vdeniaud force-pushed wip/68964-representation-publique-du-conte from 9c588ce300 to 7114689ce9 2023-01-12 14:52:57 +01:00 Compare
vdeniaud commented 2023-01-12 14:53:25 +01:00
Author
Owner
Copy Link

(mini bout en plus pour inclure les champs sans identifiants)

(mini bout en plus pour inclure les champs sans identifiants)
vdeniaud force-pushed wip/68964-representation-publique-du-conte from 7114689ce9 to 037148c564 2023-01-12 15:27:13 +01:00 Compare
vdeniaud commented 2023-01-12 15:27:41 +01:00
Author
Owner
Copy Link

(et encore un mini bout pour le support des dates (c'est à dire reconstruire le schéma à l'import))

(et encore un mini bout pour le support des dates (c'est à dire reconstruire le schéma à l'import))
fpeters approved these changes 2023-01-14 11:00:56 +01:00
vdeniaud merged commit b7d358cfce into main 2023-01-16 09:41:07 +01:00
vdeniaud deleted branch wip/68964-representation-publique-du-conte 2023-01-16 09:41:07 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#24
No description provided.
Delete Branch "wip/68964-representation-publique-du-conte"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?