Projet

Général

Profil

Actions

Anomalie #1247

fermé

Crash du code JS d'emailing si une quote est dans le nom d'un des expéditeur

Ajouté par C. S. il y a presque 6 ans. Mis à jour il y a presque 6 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
Emails
Version cible:
Début:
27/01/2019
Echéance:
% réalisé:

100%

Temps estimé:
Version utilisée:

Description

Crash du code JS d'emailing si une quote est dans le nom d'un des expéditeur

Le code en question se trouve dans mailing_adherents.tpl ligne 342

Fix: Ajouter "|escape:'quotes'" à la fin des variables

            switch (_val) {
                case '{Galette\Core\GaletteMail::SENDER_PREFS}':
                    _sender_name.val('{$preferences->pref_email_nom|escape:'quotes'}');
                    _sender_address.val('{$preferences->pref_email|escape:'quotes'}');
                    break;
                case '{Galette\Core\GaletteMail::SENDER_CURRENT}':
                    _sender_name.val('{$sender_current['name']|escape:'quotes'}');
                    _sender_address.val('{$sender_current['email']|escape:'quotes'}');
                    break;
                case '{Galette\Core\GaletteMail::SENDER_OTHER}':
                    _sender_name.val('');
                    _sender_address.val('');
                    _editable = true;
                    break;
            }

J'ai pas vérifié s'il y avait d'autres erreurs de ce genre ailleurs.


Fichiers

Mis à jour par Johan Cwiklinski il y a presque 6 ans

  • Catégorie mis à Emails
  • Assigné à mis à Johan Cwiklinski
  • Version cible mis à 0.9.3

Dommage, la version 0.9.2.1 est sortie à l'instant...

Ce serait plutôt |escape:"js" (c'est ce qui est utilisé à d'autres endroits). Il est parfaitement possible que d'autres choses du genre soient présentes ailleurs :/

Mis à jour par Christian Hardy il y a presque 6 ans

Bonjour
Galette 0.9.2.1 PHP 7.2
Envoi d'un message à 312 adhérents en utilisant le choix "autre " avec saisie du nom et d'une adresse de messagerie.
Plantage selon le rapport joint
J'ai appliqué les corrections ci-dessus, c'est donc une autre cause, a priori.
Christian

Mis à jour par Johan Cwiklinski il y a presque 6 ans

Oui, il s'agit d'un autre problème ; qui semble être lié au contenu du courriel en lui même. Y aurait-il quelque chose de spécial là dedans (attention par exemple aux copié/collé depuis un traitement de texte, etc).

Mis à jour par Christian Hardy il y a presque 6 ans

Bonsoir
Quel que soit le mode d'adressage choisi ( d'après les préférences, utilisateur connecté, autre) Le crash se produit dès que l'on utilise l'éditeur HTML.
Si l'on envoie un texte simple, le message est envoyé.
J'ai fait une copie du message de plantage. dans le texte du message j'avais écrit "Bonjour à tous" en H1.
Du coup toute la seconde partie du message d'erreur est en H1
Christian

Mis à jour par Johan Cwiklinski il y a presque 6 ans

@Christian,

Le le problème que tu rencontres est différent du problème initial ; il faudrait ouvrir un autre ticket s'il te plaît.
Penses à préciser le navigateur utilisé, la version de php et les modules installés (voir les infos système de Galette).

Mis à jour par Johan Cwiklinski il y a presque 6 ans

  • Statut changé de Nouveau à Résolu
  • % réalisé changé de 0 à 100

Mis à jour par Johan Cwiklinski il y a presque 6 ans

  • Statut changé de Résolu à Fermé
Actions

Formats disponibles : Atom PDF