Project

General

Profile

Anomalie #638

suppression adherent sans avertissement

Added by Fr cero about 6 years ago. Updated about 6 years ago.

Status:
Fermé
Priority:
Urgent
Category:
Core
Target version:
-
Start date:
07/05/2013
Due date:
% Done:

100%

Estimated time:
Version utilisée:
Vote:

Description

Avec Galete 7.4.4 en mode liste des adhérent, un clic maladroit sur la petite icône de suppression à droite de la liste supprime directement l'adhérent sans passer par la fenêtre js d'avertissement.

Associated revisions

Revision 3e6f49fd (diff)
Added by Johan Cwiklinski about 6 years ago

No confirmation asked on member deletion; fixes #638

History

#1

Updated by Johan Cwiklinski about 6 years ago

  • Priority changed from Normal to Urgent
  • Assignee changed from Fr cero to Johan Cwiklinski

En effet, c'est pas top :/ Ça vient d'un changement à priori mineur qui impacte le texte, et javascript qui n'aime pas le nouveau résultat...

Pour corriger, en attendant que je comprenne pourquoi ; dans le template tamplates/default/gestion_adherents.tpl à la ligne 195 ; on a quelque chose du genre :

return confirm('{_T string="Do you really want to [...]" escape="js"}')

Qu'il faut remplacer par :

return confirm('{_T string="Do you really want to [...]"|escape:"javascript"}')

(Le texte est volontairement tronqué pour davantage de lisibilité sur le tracker).

#2

Updated by Johan Cwiklinski about 6 years ago

Je ne parviens pas trop à saisir ce qui s'est passé. La chaîne traduite comporte deux \n pour effectuer un saut de ligne, qui sont remplacés par un vrai saut de ligne ; sauf que je ne trouve pas où (c'est une fonction smarty qui se charge de ça ; pas évident à déboguer).

La solution proposée dans le premier commentaire fonctionne, il est aussi possible de remplacer chaque \n par \\n pour que ça fonctionne...

#3

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from Nouveau to In Progress
#4

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from In Progress to Résolu
  • % Done changed from 0 to 100
#5

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF