Projet

Général

Profil

Bug #51274

ldap: fix misleading variable /timeBeforeExpiration/expiration_date/

Ajouté par Loïc Dachary il y a environ 3 ans. Mis à jour il y a environ 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
19 février 2021
Echéance:
% réalisé:

100%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

La traduction

Votre mot de passe ne sera plus valable dans {timeBeforeExpiration}.

devrait être

Votre mot de passe ne sera plus valable le {timeBeforeExpiration}.

Une confusion compréhensible car la variable aurait du être renommée.


Fichiers

Révisions associées

Révision 40930583 (diff)
Ajouté par Loïc Dachary il y a environ 3 ans

ldap: fix misleading variable /timeBeforeExpiration/expiration_date/ (#51274)

The timeBeforeExpiration obtained from ppolicy is the number of
seconds left until the password expires. Before it is inserted in the
error message it is translated into a date because it is easier to
internationalize. The variable name should be changed to reflect this
modification otherwise it will be misleading to translators.

Fixes: #51274

License: MIT

Historique

#2

Mis à jour par Benjamin Dauvergne il y a environ 3 ans

Dans le cas d'une correction à une traduction existante, tu peux introduire la correction directement dans ton commit.

#3

Mis à jour par Benjamin Dauvergne il y a environ 3 ans

  • Statut changé de Nouveau à En cours
#5

Mis à jour par Loïc Dachary il y a environ 3 ans

J'ai vérifié que le patch s'applique toujours sans conflit à ce jour. Est-ce qu'il faudrait que je fasse autre chose afin qu'il soit validé ?

#6

Mis à jour par Valentin Deniaud il y a environ 3 ans

  • Statut changé de En cours à Solution validée
  • Assigné à mis à Loïc Dachary

Nop tout est bon, je valide.

Pour le suivi des tickets à relire on utilise le statut « Solution Proposée », ça peut expliquer que ce ticket simple soit passé à la trappe. Notre petite gymnastique redmine c'est de d'abord s'assigner un ticket au moment de commencer à le traiter (ou de se l'assigner au moment de le créer si on sait qu'on va travailler dessus), puis passer en « Solution Proposée » au moment de soumettre un patch bon pour relecture.

#7

Mis à jour par Loïc Dachary il y a environ 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
  • % réalisé changé de 0 à 100
#8

Mis à jour par Valentin Deniaud il y a environ 3 ans

commit 409305831c99d2a2ca9305c508dc5bfd24c67c2e
Author: Loïc Dachary <ldachary@easter-eggs.com>
Date:   Fri Feb 19 08:27:40 2021 +0100

    ldap: fix misleading variable /timeBeforeExpiration/expiration_date/ (#51274)

    The timeBeforeExpiration obtained from ppolicy is the number of
    seconds left until the password expires. Before it is inserted in the
    error message it is translated into a date because it is easier to
    internationalize. The variable name should be changed to reflect this
    modification otherwise it will be misleading to translators.

    Fixes: #51274

    License: MIT
#9

Mis à jour par Loïc Dachary il y a environ 3 ans

Nop tout est bon, je valide.

Ah merci pour les explications et la validation. Je passerais bien un ticket en "Solution Proposé" pour participer à la gymnastique mais je ne crois pas avoir le droit de le faire. Ca ne me dérange pas particulièrement: je garde une copie des tickets en cours dans org-mode et comme je n'en ai pas beaucoup dans la radar ce n'est pas vraiment gênant.

#11

Mis à jour par Frédéric Péters il y a environ 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF