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

forms: make anonymise attribute a string (#73203) #372

Merged
ecazenave merged 3 commits from wip/73203-anonymisation into main 2023-06-26 18:20:14 +02:00
Conversation 5 Commits 3 Files Changed 11 +195 -36
ecazenave commented 2023-06-13 13:17:51 +02:00
Owner
Copy Link
No description provided.
ecazenave force-pushed wip/73203-anonymisation from aed030c9d3 to 6671308795 2023-06-13 17:32:54 +02:00 Compare
ecazenave force-pushed wip/73203-anonymisation from 6671308795 to f54f95718e 2023-06-13 17:49:48 +02:00 Compare
ecazenave force-pushed wip/73203-anonymisation from f54f95718e to ef3e1ac54a 2023-06-14 10:42:24 +02:00 Compare
ecazenave force-pushed wip/73203-anonymisation from ef3e1ac54a to ad792b779f 2023-06-14 11:10:01 +02:00 Compare
ecazenave force-pushed wip/73203-anonymisation from ad792b779f to 1b25e8c818 2023-06-14 12:11:03 +02:00 Compare
ecazenave force-pushed wip/73203-anonymisation from 1b25e8c818 to 7fff356b17 2023-06-14 13:54:12 +02:00 Compare
ecazenave force-pushed wip/73203-anonymisation from 7fff356b17 to 0c927b9065 2023-06-14 14:08:13 +02:00 Compare
ecazenave force-pushed wip/73203-anonymisation from 0c927b9065 to ab7ad76c2f 2023-06-14 15:53:12 +02:00 Compare
ecazenave force-pushed wip/73203-anonymisation from ab7ad76c2f to 2f14cd5ec7 2023-06-14 18:32:57 +02:00 Compare
ecazenave commented 2023-06-15 10:30:56 +02:00
Author
Owner
Copy Link

Le premier patch pour préparer le terrain coté champs, le deuxième pour l'action, le troisième qui fait le taf.

Sur les champs le titre de l'option était 'anonymise' (à cocher ou décocher), ça passe maintenant à 'anonymisation' (3 choix possibles : final, intermediate, no), cf champ.png.

Sur l'action on pouvait cocher ou décocher 'Only perform form/card user unlinking'. Maitenant 3 choix possibles : 'final', 'intermediate', 'Only perform form/card user unlinking',cf action.png.
J'ai casé le petit texte d'aide 'If you choose "Only perform form/card user unlinking" this action will only unlink user from the form/card ...' sous le titre de l'option. Pas particulièrement satisfaisant mais je ne sais pas quoi faire d'autre.

Le premier patch pour préparer le terrain coté champs, le deuxième pour l'action, le troisième qui fait le taf. Sur les champs le titre de l'option était 'anonymise' (à cocher ou décocher), ça passe maintenant à 'anonymisation' (3 choix possibles : final, intermediate, no), cf champ.png. Sur l'action on pouvait cocher ou décocher 'Only perform form/card user unlinking'. Maitenant 3 choix possibles : 'final', 'intermediate', 'Only perform form/card user unlinking',cf action.png. J'ai casé le petit texte d'aide 'If you choose "Only perform form/card user unlinking" this action will only unlink user from the form/card ...' sous le titre de l'option. Pas particulièrement satisfaisant mais je ne sais pas quoi faire d'autre.
champ.png
124 KiB
action.png
119 KiB
champ.png action.png
ecazenave changed title from WIP: forms: make anonymise attribute a string (73203) to forms: make anonymise attribute a string (73203) 2023-06-15 10:31:15 +02:00
ecazenave commented 2023-06-15 10:33:03 +02:00
Author
Owner
Copy Link

Pas particulièrement satisfaisant mais je ne sais pas quoi faire d'autre.

Je me dis soudainement que je pourrais virer le titre 'Anonymisation type' et mettre le texte d'aide à la place, ce serait mieux.

> Pas particulièrement satisfaisant mais je ne sais pas quoi faire d'autre. Je me dis soudainement que je pourrais virer le titre 'Anonymisation type' et mettre le texte d'aide à la place, ce serait mieux.
ecazenave force-pushed wip/73203-anonymisation from 2f14cd5ec7 to a9181d13d9 2023-06-15 10:54:17 +02:00 Compare
ecazenave commented 2023-06-15 11:41:09 +02:00
Author
Owner
Copy Link

Je me dis soudainement que je pourrais virer le titre 'Anonymisation type' et mettre le texte d'aide à la place, ce serait mieux.

J'ai fait ça, cf action2.png.

> Je me dis soudainement que je pourrais virer le titre 'Anonymisation type' et mettre le texte d'aide à la place, ce serait mieux. J'ai fait ça, cf action2.png.
action2.png
121 KiB
action2.png
fpeters changed title from forms: make anonymise attribute a string (73203) to forms: make anonymise attribute a string (#73203) 2023-06-15 20:56:37 +02:00
fpeters commented 2023-06-15 21:01:31 +02:00
Owner
Copy Link

Ce help_text déplacé en libellé de champ ne va pas côté accessibilité, je serais pour revenir à la précédente version, avec "Type d’anonymisation" comme libellé.

Peut-être ne pas reprendre le texte d’aide mais allonger le libellé de l'option, "Uniquement délier l'usager associé à la demande/fiche. Si un code de suivi existe il sera également supprimé." ?

Ce help_text déplacé en libellé de champ ne va pas côté accessibilité, je serais pour revenir à la précédente version, avec "Type d’anonymisation" comme libellé. Peut-être ne pas reprendre le texte d’aide mais allonger le libellé de l'option, "Uniquement délier l'usager associé à la demande/fiche. Si un code de suivi existe il sera également supprimé." ?
fpeters reviewed 2023-06-15 21:10:33 +02:00
wcs/fields.py Outdated
@ -625,0 +626,4 @@
self.anonymise = 'no'
if self.anonymise:
self.anonymise = 'final'
changed = True
fpeters commented 2023-06-15 21:04:28 +02:00
Owner
Copy Link

Ce code va toujours mettre "final" comme valeur, ça devrait être soit if/else, soit en une seule ligne,

self.anonymise = 'final' if self.anonymise else 'no'
Ce code va toujours mettre "final" comme valeur, ça devrait être soit if/else, soit en une seule ligne, ``` self.anonymise = 'final' if self.anonymise else 'no' ```
wcs/formdata.py Outdated
@ -1259,3 +1259,3 @@
for field in self.formdef.get_all_fields():
if field.anonymise:
if field.anonymise == kind:
field.set_value(self.data, None)
fpeters commented 2023-06-15 21:05:46 +02:00
Owner
Copy Link

Je pense que si kind == 'final' ça devrait anonymiser tout, y compris ce qui est marqué pour 'intermediate'.

Je pense que si kind == 'final' ça devrait anonymiser tout, y compris ce qui est marqué pour 'intermediate'.
wcs/formdata.py Outdated
@ -1268,2 +1263,2 @@
self.workflow_roles = None
self.submission_context = None
if kind == 'final':
self.anonymised = localtime()
fpeters commented 2023-06-15 21:08:19 +02:00
Owner
Copy Link

Je trouverais plus lisible (à la fois le diff et le code résultant) de moins déplacer, plutôt avoir un return supplémentaire, plus haut

if kind != 'final':
    self.store()
    return

self.anonymised = localtime()
self.user_id = None
...
Je trouverais plus lisible (à la fois le diff et le code résultant) de moins déplacer, plutôt avoir un return supplémentaire, plus haut ``` if kind != 'final': self.store() return self.anonymised = localtime() self.user_id = None ... ```
wcs/wf/anonymise.py Outdated
@ -26,3 +26,3 @@
key = 'anonymise'
category = 'formdata-action'
unlink_user = False
kind = 'final'
fpeters commented 2023-06-15 21:10:18 +02:00
Owner
Copy Link

Détail mais plutôt "mode" que "kind" (pour faire comme user_association_mode, operation_mode, target_mode, qui existent dans les autres actions).

Détail mais plutôt "mode" que "kind" (pour faire comme user_association_mode, operation_mode, target_mode, qui existent dans les autres actions).
ecazenave force-pushed wip/73203-anonymisation from a9181d13d9 to ad9ab6ea7c 2023-06-19 18:26:12 +02:00 Compare
ecazenave force-pushed wip/73203-anonymisation from ad9ab6ea7c to f3063551dc 2023-06-20 10:29:13 +02:00 Compare
ecazenave force-pushed wip/73203-anonymisation from f3063551dc to 1616caaf3a 2023-06-20 11:06:36 +02:00 Compare
ecazenave force-pushed wip/73203-anonymisation from 1616caaf3a to 86e6b1b168 2023-06-20 11:33:44 +02:00 Compare
ecazenave commented 2023-06-20 12:04:15 +02:00
Author
Owner
Copy Link

Toutes les remarques prises en compte.

Toutes les remarques prises en compte.
ecazenave requested review from fpeters 2023-06-20 12:04:26 +02:00
fpeters approved these changes 2023-06-26 17:47:17 +02:00
fpeters left a comment
Owner
Copy Link

Je crains que toute cette histoire d'anonymisation intermédiaire ne soit pas bien claire et qu'il faille revenir sur les libellés, l'ordre des options, que sais-je, mais passons ça.

Je crains que toute cette histoire d'anonymisation intermédiaire ne soit pas bien claire et qu'il faille revenir sur les libellés, l'ordre des options, que sais-je, mais passons ça.
wcs/fields.py Outdated
@ -282,3 +282,3 @@
# flag a field for removal by AnonymiseWorkflowStatusItem
# can be overriden in field' settings
anonymise = True
anonymise = 'final'
fpeters commented 2023-06-26 17:45:59 +02:00
Owner
Copy Link

Peut-être actualiser le commentaire qui précède, pour noter les valeurs possibles .

Peut-être actualiser le commentaire qui précède, pour noter les valeurs possibles .
ecazenave commented 2023-06-26 18:20:06 +02:00
Author
Owner
Copy Link

Fait.

Fait.
ecazenave force-pushed wip/73203-anonymisation from 86e6b1b168 to 4a5c2b1541 2023-06-26 17:59:26 +02:00 Compare
ecazenave merged commit 3ac14b7fb9 into main 2023-06-26 18:20:14 +02:00
ecazenave deleted branch wip/73203-anonymisation 2023-06-26 18:20:14 +02: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#372
No description provided.
Delete Branch "wip/73203-anonymisation"

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?