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

testdef: forbid test creation if form has deprecated fields (#73331) #77

Merged
vdeniaud merged 1 commits from wip/73331-testdef-indiquer-qu-on-ne-suppor into main 2023-01-30 15:10:32 +01:00
Conversation 0 Commits 1 Files Changed 4 +38 -2
vdeniaud commented 2023-01-30 14:29:46 +01:00
Owner
Copy Link
No description provided.
vdeniaud added 1 commit 2023-01-30 14:29:47 +01:00
fpeters requested changes 2023-01-30 14:37:59 +01:00
wcs/admin/tests.py
@ -119,1 +119,4 @@
'formdata': self.objectdef.data_class().select([StrictNotEqual('status', 'draft')]),
'has_deprecated_fields': any(
x.type in ('table', 'table-select', 'tablerows', 'ranked-items')
for x in self.objectdef.fields
fpeters commented 2023-01-30 14:37:55 +01:00
Owner
Copy Link

Il y aurait moyen d'ajouter un attribut deprecated = True/False sur les champs, et l'utiliser ici + wcs/testdef.py plus bas + wcs/backoffice/deprecations.py, j'ai peur autrement que dans une prochaine dépréciation on oublie d'actualiser un côté.

Il y aurait moyen d'ajouter un attribut `deprecated = True/False` sur les champs, et l'utiliser ici + wcs/testdef.py plus bas + wcs/backoffice/deprecations.py, j'ai peur autrement que dans une prochaine dépréciation on oublie d'actualiser un côté.
vdeniaud commented 2023-01-30 14:46:55 +01:00
Author
Owner
Copy Link

Mais ici ce qui justifie qu'on interdise de créer des tests ce n'est pas vraiment que les champs soient dépréciés, c'est qu'ils l'aient été avant l'introduction des tests.

Si demain on déprécie un nouveau type de champ on a pas envie que ça ait une incidence sur les tests existants non ? (ça irait à l'encontre du principe de la dépréciation, laisser le temps de gérer la migration en permettant de continuer une utilisation normale)

Mais ici ce qui justifie qu'on interdise de créer des tests ce n'est pas vraiment que les champs soient dépréciés, c'est qu'ils l'aient été avant l'introduction des tests. Si demain on déprécie un nouveau type de champ on a pas envie que ça ait une incidence sur les tests existants non ? (ça irait à l'encontre du principe de la dépréciation, laisser le temps de gérer la migration en permettant de continuer une utilisation normale)
fpeters commented 2023-01-30 15:01:18 +01:00
Owner
Copy Link

ok ainsi alors.

ok ainsi alors.
fpeters approved these changes 2023-01-30 15:01:29 +01:00
vdeniaud merged commit 1d64138735 into main 2023-01-30 15:10:32 +01:00
vdeniaud deleted branch wip/73331-testdef-indiquer-qu-on-ne-suppor 2023-01-30 15:10:32 +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#77
No description provided.
Delete Branch "wip/73331-testdef-indiquer-qu-on-ne-suppor"

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?