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

uuid sur fiches (#73675) #125

Merged
fpeters merged 2 commits from wip/73675-card-uuid into main 2023-05-12 17:23:41 +02:00
Conversation 1 Commits 2 Files Changed 10 +343 -31
fpeters commented 2023-02-17 14:17:52 +01:00
Owner
Copy Link

Suite de https://gitea.entrouvert.org/entrouvert/wcs/pulls/48

Suite de https://gitea.entrouvert.org/entrouvert/wcs/pulls/48
fpeters changed title from uuid sur fiches (#73675) to WIP: uuid sur fiches (#73675) 2023-02-17 14:22:31 +01:00
fpeters force-pushed wip/73675-card-uuid from a909f91452 to 5c74bfcc40 2023-03-06 09:30:37 +01:00 Compare
fpeters force-pushed wip/73675-card-uuid from 5c74bfcc40 to 8ba12edc78 2023-05-01 13:22:22 +02:00 Compare
fpeters force-pushed wip/73675-card-uuid from 8ba12edc78 to 6d554f4722 2023-05-12 11:43:44 +02:00 Compare
fpeters changed title from WIP: uuid sur fiches (#73675) to uuid sur fiches (#73675) 2023-05-12 14:15:40 +02:00
fpeters commented 2023-05-12 14:25:32 +02:00
Author
Owner
Copy Link
  • cards: add an UUID attribute (#73675)

C'est très léger, juste l'ajout de l'attribut/la colonne, l'initialisation se fait directement côté postgresql, gen_random_uuid().

  • cards: add possibility to update existing cards on import (#73675)

L'exploitation des uuid pour une première fonctionnalité, la possibilité lors de l'import (via l'ui ou l'api) de mettre à jour les données.

* cards: add an UUID attribute (#73675) C'est très léger, juste l'ajout de l'attribut/la colonne, l'initialisation se fait directement côté postgresql, gen_random_uuid(). * cards: add possibility to update existing cards on import (#73675) L'exploitation des uuid pour une première fonctionnalité, la possibilité lors de l'import (via l'ui ou l'api) de mettre à jour les données.
lguerin reviewed 2023-05-12 15:59:30 +02:00
tests/backoffice_pages/test_carddata.py Outdated
@ -989,3 +991,3 @@
carddata = carddef.data_class().select()[0]
assert carddata.status == 'recorded'
assert carddata.status == 'wf-recorded'
lguerin commented 2023-05-12 15:49:44 +02:00
Owner
Copy Link

pourquoi ces changements ?

pourquoi ces changements ?
fpeters commented 2023-05-12 16:18:28 +02:00
Author
Owner
Copy Link

Les données dans le test n'étaient pas correctes, dans du vrai json exporté c'est l'id sans le wf-,

-                        'id': 'wf-%s' % st2.id,
+                        'id': str(st2.id),

et une fois cette correction apportée, ce qui était stocké dans la base devenait correct (préfixé par wf-, là).

Les données dans le test n'étaient pas correctes, dans du vrai json exporté c'est l'id sans le wf-, ``` - 'id': 'wf-%s' % st2.id, + 'id': str(st2.id), ``` et une fois cette correction apportée, ce qui était stocké dans la base devenait correct (préfixé par wf-, là).
lguerin marked this conversation as resolved
tests/test_sql.py Outdated
@ -2276,0 +2288,4 @@
assert carddef.data_class().get(carddata.id).uuid
conn, cur = sql.get_connection_and_cursor()
cur.execute('UPDATE wcs_meta SET value = 79 WHERE key = %s', ('sql_level',))
lguerin commented 2023-05-12 15:41:21 +02:00
Owner
Copy Link

85, ça suffirait, non ?

85, ça suffirait, non ?
fpeters commented 2023-05-12 16:12:52 +02:00
Author
Owner
Copy Link

Oui c'est qu'avec le temps c'était 79 et je n'ai pas monté ça à chaque fois. (je vais le faire)

Oui c'est qu'avec le temps c'était 79 et je n'ai pas monté ça à chaque fois. (je vais le faire)
fpeters marked this conversation as resolved
tests/test_sql.py Outdated
@ -2276,0 +2297,4 @@
sql.migrate()
assert column_exists_in_table(cur, sql.get_formdef_table_name(carddef), 'uuid')
assert migration_level(cur) >= 80
lguerin commented 2023-05-12 15:41:28 +02:00
Owner
Copy Link

86 ?

86 ?
fpeters commented 2023-05-12 16:19:52 +02:00
Author
Owner
Copy Link

oui, corrigé.

oui, corrigé.
fpeters marked this conversation as resolved
wcs/api.py Outdated
@ -401,13 +401,16 @@ class ApiCardPage(ApiFormPageMixin, BackofficeCardPage):
raise AccessForbiddenError('cannot import cards')
afterjob = bool(get_request().form.get('async') == 'on')
do_update = bool(get_request().form.get('update') == 'on')
lguerin commented 2023-05-12 15:59:14 +02:00
Owner
Copy Link

il manquerait juste un petit test là-dessus

il manquerait juste un petit test là-dessus
fpeters commented 2023-05-12 16:13:09 +02:00
Author
Owner
Copy Link

test_cards_import_json_update ajouté.

test_cards_import_json_update ajouté.
fpeters commented 2023-05-12 16:20:06 +02:00
Author
Owner
Copy Link

En passant ça a révélé une erreur, merci.

En passant ça a révélé une erreur, merci.
fpeters force-pushed wip/73675-card-uuid from 6d554f4722 to bc01ed9871 2023-05-12 16:12:06 +02:00 Compare
fpeters force-pushed wip/73675-card-uuid from bc01ed9871 to b24bd2b85f 2023-05-12 16:19:20 +02:00 Compare
fpeters force-pushed wip/73675-card-uuid from b24bd2b85f to 328c7e70b0 2023-05-12 16:19:46 +02:00 Compare
lguerin approved these changes 2023-05-12 16:34:16 +02:00
fpeters force-pushed wip/73675-card-uuid from 328c7e70b0 to a22c2132e3 2023-05-12 16:55:27 +02:00 Compare
fpeters merged commit a22c2132e3 into main 2023-05-12 17:23:41 +02:00
fpeters deleted branch wip/73675-card-uuid 2023-05-12 17:23:41 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
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#125
No description provided.
Delete Branch "wip/73675-card-uuid"

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?