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

provide clearer erroneous template filter use message (#86062) #1066

Merged
pmarillonnet merged 1 commits from wip/86062-invalid-queryset-translation-fr into main 2024-04-02 12:57:42 +02:00
Conversation 0 Commits 1 Files Changed 2 +5 -3
pmarillonnet commented 2024-01-24 09:29:48 +01:00
Owner
Copy Link
No description provided.
pmarillonnet added 1 commit 2024-01-24 09:29:48 +01:00
fpeters reviewed 2024-01-24 12:04:49 +01:00
wcs/locale/fr/LC_MESSAGES/django.po Outdated
@ -8792,3 +8792,3 @@
#, python-format
msgid "|%s used on invalid queryset (%r)"
msgstr "|%s utilisé sur une requête invalide (%r)"
msgstr "|%s utilisé sur un ensemble d’objets invalide (%r)"
fpeters commented 2024-01-24 12:04:49 +01:00
Owner
Copy Link

Je trouve pas ça terrible, "ensemble d'objets", ça dit quoi côté CPF ?

Je trouve pas ça terrible, "ensemble d'objets", ça dit quoi côté CPF ?
fpeters commented 2024-01-24 13:31:14 +01:00
Owner
Copy Link

Je regardais la documentation django, il y a à un moment "résultat de requête" comme traduction mais ça ne me va pas tellement non plus. À revenir ici regarder à quoi l'erreur correspond,

    def f(queryset, *args, **kwargs):
        if not hasattr(queryset, attr):
            get_publisher().record_error(_('|%s used on invalid queryset (%r)') % (name, queryset))
            return None
        return func(queryset, *args, **kwargs)

J'en viendrais à suggérer de tout bonnement remplacer queryset, et plutôt faire |%s used on inadequate object (%r), qu'on traduirait "|%s appelé sur un objet inadéquat (%r)".

Je regardais la documentation django, il y a à un moment "résultat de requête" comme traduction mais ça ne me va pas tellement non plus. À revenir ici regarder à quoi l'erreur correspond, ``` def f(queryset, *args, **kwargs): if not hasattr(queryset, attr): get_publisher().record_error(_('|%s used on invalid queryset (%r)') % (name, queryset)) return None return func(queryset, *args, **kwargs) ``` J'en viendrais à suggérer de tout bonnement remplacer queryset, et plutôt faire `|%s used on inadequate object (%r)`, qu'on traduirait "|%s appelé sur un objet inadéquat (%r)".
pmarillonnet commented 2024-01-24 13:35:53 +01:00
Author
Owner
Copy Link

Oui c’est mieux encore, merci pour le tuyau. Je vais faire cela.

Oui c’est mieux encore, merci pour le tuyau. Je vais faire cela.
pmarillonnet commented 2024-01-24 14:38:50 +01:00
Author
Owner
Copy Link

Oui c’est mieux encore, merci pour le tuyau. Je vais faire cela.

C’est modifié.

> Oui c’est mieux encore, merci pour le tuyau. Je vais faire cela. C’est modifié.
fpeters commented 2024-01-25 12:04:47 +01:00
Owner
Copy Link

(attention prolongation de la conversation dans le ticket).

(attention prolongation de la conversation dans le ticket).
tnoel commented 2024-01-25 15:23:58 +01:00
Owner
Copy Link

Dans le ticket :

on parle de filtres tels que |filter_by, qui seraient appliqués à quelque chose qui n'est pas un queryset.

Et donc je disais dans le ticket que je trouve que "objet inadequat" n'est pas très compréhensible pour les êtres humains, peut-être un truc plus bateau, genre « |filter_by utilisé en dehors d'une requête » ou « |filter_by utilisé sur autre chose qu'une requête » ?

Dans le ticket : > on parle de filtres tels que |filter_by, qui seraient appliqués à quelque chose qui n'est pas un queryset. Et donc je disais dans le ticket que je trouve que "objet inadequat" n'est pas très compréhensible pour les êtres humains, peut-être un truc plus bateau, genre « |filter_by utilisé en dehors d'une requête » ou « |filter_by utilisé sur autre chose qu'une requête » ?
fpeters commented 2024-02-29 17:10:11 +01:00
Owner
Copy Link

(reprise de mon commentaire qui était dans redmine, même si un peu obsolète)

Pour être sûr qu'on parle bien de la même chose, on parle de filtres tels que |filter_by, qui seraient appliqués à quelque chose qui n'est pas un queryset.

Par exemple sur des données de blocs de champs ou sur des listes de dictionnaires (#57066), qui sont deux utilisations actuellement pas possibles.

Mais ça vient à la base du ticket #85066, "remonter une erreur sur un |filter_by appelé sur un objet invalide", qui est un autre cas :

{{cards|objects:"arbrefruitier"|filter_by:"arbre"|filter_value:"poirier"|first|get:"max"|subtract:cards|objects:"arbrefruitier"|filter_by:"arbre"|filter_value:"poirier"|first|get:"max" }}

Le néophyte, face à ça, "|filter_by utilisé sur une requête invalide" (situation qui va passer en prod) (édit : qui est depuis passée en prod) vs "|filter_by appelé sur un objet invalide" (situation proposée), je n'ai perso aucune idée de ce qu'il aura en tête.

Je pense que je serais pour opter pour " |filter_by utilisé sur autre chose qu'une requête", ça me semble plus facile à comprendre comme pointant "l'élément juste devant le |filter_by"."

(reprise de mon commentaire qui était dans redmine, même si un peu obsolète) Pour être sûr qu'on parle bien de la même chose, on parle de filtres tels que |filter_by, qui seraient appliqués à quelque chose qui n'est pas un queryset. Par exemple sur des données de blocs de champs ou sur des listes de dictionnaires (#57066), qui sont deux utilisations actuellement pas possibles. Mais ça vient à la base du ticket #85066, "remonter une erreur sur un |filter_by appelé sur un objet invalide", qui est un autre cas : {{cards|objects:"arbrefruitier"|filter_by:"arbre"|filter_value:"poirier"|first|get:"max"|subtract:cards|objects:"arbrefruitier"|filter_by:"arbre"|filter_value:"poirier"|first|get:"max" }} Le néophyte, face à ça, "|filter_by utilisé sur une requête invalide" (situation qui va passer en prod) (édit : qui est depuis passée en prod) vs "|filter_by appelé sur un objet invalide" (situation proposée), je n'ai perso aucune idée de ce qu'il aura en tête. ---- Je pense que je serais pour opter pour " |filter_by utilisé sur autre chose qu'une requête", ça me semble plus facile à comprendre comme pointant "l'élément juste devant le |filter_by"."
fpeters commented 2024-02-29 17:11:25 +01:00
Owner
Copy Link

@pmarillonnet , donc, qqch comme "... used on something else than a queryset", avec la traduction "utilisé sur autre chose qu'une requête", ça t'irait ?

@pmarillonnet , donc, qqch comme "... used on something else than a queryset", avec la traduction "utilisé sur autre chose qu'une requête", ça t'irait ?
pmarillonnet commented 2024-03-26 11:17:03 +01:00
Author
Owner
Copy Link

Oui, je trouve cette proposition préférable à la situation actuelle. Je vais mettre à jour la PR pour aller vers ça.

Oui, je trouve cette proposition préférable à la situation actuelle. Je vais mettre à jour la PR pour aller vers ça.
pmarillonnet force-pushed wip/86062-invalid-queryset-translation-fr from 0c6c63551d to fd6a336f5e 2024-01-24 13:39:54 +01:00 Compare
pmarillonnet changed title from provide clearer 'invalid queryset' french translation (#86062) to provide clearer erroneous template filter use message (#86062) 2024-01-24 13:40:13 +01:00
pmarillonnet changed title from provide clearer erroneous template filter use message (#86062) to WIP: provide clearer erroneous template filter use message (#86062) 2024-03-26 11:12:07 +01:00
pmarillonnet force-pushed wip/86062-invalid-queryset-translation-fr from fd6a336f5e to 8a9c1fc8e8 2024-03-26 11:12:18 +01:00 Compare
pmarillonnet force-pushed wip/86062-invalid-queryset-translation-fr from 8a9c1fc8e8 to 1db58c9d65 2024-03-26 11:17:23 +01:00 Compare
pmarillonnet force-pushed wip/86062-invalid-queryset-translation-fr from 1db58c9d65 to 82bb6d0e89 2024-03-26 11:25:51 +01:00 Compare
pmarillonnet changed title from WIP: provide clearer erroneous template filter use message (#86062) to provide clearer erroneous template filter use message (#86062) 2024-03-26 11:32:19 +01:00
fpeters approved these changes 2024-03-29 09:41:43 +01:00
pmarillonnet merged commit 378758e0c5 into main 2024-04-02 12:57:42 +02:00
pmarillonnet deleted branch wip/86062-invalid-queryset-translation-fr 2024-04-02 12:57:43 +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
3 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#1066
No description provided.
Delete Branch "wip/86062-invalid-queryset-translation-fr"

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?