profile: disable searchable for first_name and last_name fields (#76673) #35

Merged
bdauvergne merged 1 commits from wip/76673-profile-griser-la-case-a-cocher into main 2023-04-21 10:18:11 +02:00
Owner
No description provided.
bdauvergne added 1 commit 2023-04-15 10:26:21 +02:00
bdauvergne changed title from WIP: profile: disable searchable for first_name and last_name fields (#76673) to profile: disable searchable for first_name and last_name fields (#76673) 2023-04-15 10:57:52 +02:00
vdeniaud requested changes 2023-04-17 16:25:23 +02:00
@ -27,0 +33,4 @@
instance = kwargs.get('instance', None)
self.disable_searchable = instance and instance.name in ['first_name', 'last_name']
super().__init__(*args, **kwargs)
if self.disable_searchable:
Owner

Je ne vois pas de raison de ne pas écrire if self.instance.name in ['first_name', 'last_name']: en se passant de la gymnastique au dessus ?

Je ne vois pas de raison de ne pas écrire `if self.instance.name in ['first_name', 'last_name']:` en se passant de la gymnastique au dessus ?
@ -27,0 +55,4 @@
]
class AttributeDefinitionUpdaeForm(AttributeDefinitionCreateForm):
Owner

Typo

Typo
@ -132,13 +132,15 @@ def test_reorder_view(logged_app):
]
def test_profile_attribute_option_view(logged_app):
Owner

Pour moi il faut laisser ce test tranquille et en écrire un autre, qui vérifie notamment que searchable n'est pas disabled pour les autres attributs.

Pour moi il faut laisser ce test tranquille et en écrire un autre, qui vérifie notamment que searchable n'est pas disabled pour les autres attributs.
bdauvergne force-pushed wip/76673-profile-griser-la-case-a-cocher from a125a62533 to 9d21e99a0e 2023-04-17 18:09:41 +02:00 Compare
bdauvergne force-pushed wip/76673-profile-griser-la-case-a-cocher from 9d21e99a0e to 2d208baef0 2023-04-17 18:58:55 +02:00 Compare
bdauvergne requested review from vdeniaud 2023-04-17 18:58:59 +02:00
vdeniaud approved these changes 2023-04-18 10:25:21 +02:00
@ -143,2 +143,4 @@
def test_profile_attribute_searchable_is_disabled(logged_app):
for name in AttributeDefinition.objects.values_list('name', flat=True):
Owner

Pas fan de ce test qui passe même si il n'y a aucun AttributeDefinition en base, itérer sur ['first_name', 'last_name', un_autre_attribut] me paraît plus solide (et en vérité plus besoin de boucle dans ce cas là).

Pas fan de ce test qui passe même si il n'y a aucun AttributeDefinition en base, itérer sur ['first_name', 'last_name', un_autre_attribut] me paraît plus solide (et en vérité plus besoin de boucle dans ce cas là).
Author
Owner

Pour moi l'intérêt c'est de vérifier le profil par défaut qui est installé et pas un choix à l'instant T, on sait qu'il y a toujours un profil par défaut installé.

Pour moi l'intérêt c'est de vérifier le profil par défaut qui est installé et pas un choix à l'instant T, on sait qu'il y a toujours un profil par défaut installé.
bdauvergne force-pushed wip/76673-profile-griser-la-case-a-cocher from 2d208baef0 to ca1f3e1b9b 2023-04-18 12:22:47 +02:00 Compare
bdauvergne force-pushed wip/76673-profile-griser-la-case-a-cocher from ca1f3e1b9b to 3454e764c3 2023-04-18 12:30:56 +02:00 Compare
bdauvergne merged commit 35e6872ec3 into main 2023-04-21 10:18:11 +02:00
bdauvergne deleted branch wip/76673-profile-griser-la-case-a-cocher 2023-04-21 10:18:11 +02:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
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/hobo#35
No description provided.