Project

General

Profile

Anomalie #1280

problème avec le Mailing

Added by Alain Paris 7 months ago. Updated 3 months ago.

Status:
Fermé
Priority:
Normal
Category:
Core
Target version:
Start date:
05/14/2019
Due date:
% Done:

100%

Estimated time:
Version utilisée:
Vote:

Description

Bonjour,
Je viens de découvrir semble t'il une anomalie dans le processus d'envoi de mail.

Je sélectionne dans la liste des adhérents, les destinataires de l'envoi.
Je me retrouve sur la page: http://galette9/webroot/mailing?new=new

1 le bouton "Gestion des adhérents sélectionnés" ne fonctionne pas.
2 le bouton "prévisualisation" renvoi la page : http://galette9/webroot/mailings ,je n'ai pas de fenêtre de prévisualisation.

Les messages sont envoyés correctement...

3 si on a ajouté un fichier joint et "prévisualisation" alors je me retrouve avec une erreur:

Type : Error      Code : 0      Message : Call to undefined method Galette\Core\Mailing::getPhpErrorMessage()     
Fichier : B:\galette9\includes\routes\members.routes.php Ligne : 2103

Testé sur Version0.9.2.1 et nightly.

Pour info l'envoi des courriers se fait en SMTP (PHP mail ne fonctionne pas chez moi). Je ne pense pas que cela influence l'anomalie.

Bonne fin de Journée


Files

0001-Fix-unescaped-variable-in-mailing-template.patch (1.39 KB) 0001-Fix-unescaped-variable-in-mailing-template.patch Guillaume AGNIERAY, 09/13/2019 11:43 AM
0001-Fix-wrong-smarty-modifier-value-fixes-1280.patch (1.67 KB) 0001-Fix-wrong-smarty-modifier-value-fixes-1280.patch Guillaume AGNIERAY, 09/17/2019 04:19 PM
0001-Hotfix-refs-1280.patch (2.59 KB) 0001-Hotfix-refs-1280.patch Hotfix pour la dernière branche stable uniquement Guillaume AGNIERAY, 09/17/2019 04:19 PM

Associated revisions

Revision 349b6fc2 (diff)
Added by Johan Cwiklinski 7 months ago

Fix call to a non existing method; closes #1280

Revision e4249da8 (diff)
Added by Guillaume AGNIERAY 3 months ago

Fix unescaped variable in mailing template; fixes #1280

Revision 740a0a76 (diff)
Added by Johan Cwiklinski 3 months ago

Fix escaping; refs #1280

History

#1

Updated by Johan Cwiklinski 7 months ago

  • Category set to Core
  • Status changed from Nouveau to In Progress
  • Assignee set to Johan Cwiklinski

Je ne reproduit pas le premier point ; tout au moins sur la nightly courante. Puisque pas mal de choses ont été changées là dessus, ça a pu être corrigé ; il faudrait essayer.

Le second point est quant à lui effectivement un bug.

#2

Updated by Johan Cwiklinski 7 months ago

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

Updated by Johan Cwiklinski 7 months ago

  • Status changed from Résolu to Fermé
#4

Updated by Alain Paris 7 months ago

Bonjour,
Je crois que je viens de trouver le problème:
Le soucis vient du nom de l’expéditeur dans Paramètres courriel le fait de mettre une apostrophe provoque les 2 anomalies.
L'apostrophe :ne fonctionne pas
L apostrophe : ok

#5

Updated by Johan Cwiklinski 7 months ago

  • Status changed from Fermé to In Progress

Ok je vais voir a corriger le souci d'apostrophe également :)

#6

Updated by Benjamin Leduc 6 months ago

Johan Cwiklinski a écrit :

Appliqué par commit 349b6fc2cabb2a933c11ba561825eb2c932f0b04.

Bonjour,

Le point #1 n'est toujours pas fonctionnel chez moi, nightly du jour. Comment puis-je fournir des informations utile pour avancer?

#7

Updated by Johan Cwiklinski 6 months ago

Si le problème est le même que celui d'Alain (apostrophe), ce n'est juste pas corrigé, pas eu le temps.

#8

Updated by Guillaume AGNIERAY 3 months ago

La variable correspondante n'est pas échapée dans le template des mailings.
Ci-joint un correctif basé sur la branche develop, et ajoutant un modifieur smarty pour échapper la variable javascript fautive.

#9

Updated by Johan Cwiklinski 3 months ago

Merci pour le patch, ça devrait effectivement corriger le problème :)

#10

Updated by Alain Paris 3 months ago

Bonjour,
Le patch doit modifier quoi pour l'utilisateur ?

J'ai essayé la modification du fichier (mailing_adherents.tpl) chez moi et je n'ai pas trouvé .

Cas no 1 : Apostrophe dans le nom de l’expéditeur (exemple "l’expéditeur")
situé dans Configuration (webroot/preferences) Courriel expéditeur :
Sur la page webroot/mailing
  • le bouton "Gestion des adhérents sélectionnés" ne fonctionne plus.
  • le bouton pour modifier l’expéditeur non plus (reste sur le courriel de "depuis les préférences").
  • lors de l'appui sur prévisualisation il y a un message d'avertissement si le sujet est vide ou message est vide.
    Si sujet et message sont complétés je n'ai pas la fenêtre pop-up avec la prévisualisation du mail.je me retrouve sur la page de gestion des mails (webroot/mailings).
Cas no 2 : pas d'apostrophe dans le nom (exemple "l expéditeur"):
  • Tous les boutons fonctionnent.
  • Il n'y a plus le message d'avertissement que le sujet ou le corps du message est vide par contre.
  • La fenêtre pop-up de prévisualisation fonctionne.
    Si l'on modifie l’expéditeur option "autre" et que celui-ci comporte une ou plusieurs apostrophe(s) prévisualisation fonctionne. ** Lorsque l'on choisi "Utilisateur actuellement connecté" ou "autre" on ne peut revenir sur "depuis les préférences" lorsque l'on a prévisualisé.

Le fonctionnement peut il dépendre de PHP ou de Mysql ou d'un de leurs modules? (testé sur PHP 7.2 et Mysql 5.53).

Il faudrait peut être une démo fonctionnelle en ligne pour voir le comportement normal.

Bonne Journée
#11

Updated by Guillaume AGNIERAY 3 months ago

  • Status changed from In Progress to Résolu
#12

Updated by Johan Cwiklinski 3 months ago

La modification du seul fichier template sur la dernière stable ne sera clairement pas suffisante pour corriger les problèmes de javascript.
Je viens de tester avec la dernière nightly, je n'ai pas rencontré de problèmes.

À priori, sur la branche develop, le patch fourni par Guillaume termine de corriger les problèmes.

#13

Updated by Guillaume AGNIERAY 3 months ago

A posteriori, le patch ne termine pas de corriger le problème :/

Échapper la variable est la solution.

Mais je n'avais pas testé avant de produire le patch. En me basant sur l'utilisation du même modifieur ailleurs dans le code, j'avais réutilisé la même valeur : js. Mais selon la documentation smarty, la valeur dans ce cas devrait être : javascript

Ci-joint un nouveau patch sur la branche develop corrigeant cette valeur.
Et un "hotfix" complet incluant la révision 349b6fc2 pour la dernière branche stable.

Cette fois j'ai testé préalablement sur les 2 branches et ça fonctionne correctement :)

#14

Updated by Johan Cwiklinski 3 months ago

js pour l'échappement est utilisé partout dans le code de Galette, à priori, ça fonctionne sans problèmes ; beaucoup d'autres choses ne fonctionneraient pas.

#15

Updated by Guillaume AGNIERAY 3 months ago

Effectivement js est utilisé partout. Mais pas partout dans le même contexte : il y a l'échappement des chaînes de traduction et l'échappement des variables dans les templates.

Dans le cas des traductions, pas de problème, c'est un plugin smarty qui s'occupe de l'échappement : source:galette/includes/smarty_plugins/function._T.php#L69

Dans le cas des variables, c'est le modificateur smarty qui se charge de l'échappement.
Et alors, seule la valeur javascript peut être reconnue : https://github.com/smarty-php/smarty/blob/c9f0de05f41b9e52798b268ab1e625fac3b8578c/libs/plugins/modifier.escape.php#L177

#16

Updated by Johan Cwiklinski 3 months ago

En effet, tu as raison ; il y a deux cas de figure distincts, le "modifier" smarty est finalement peu utilisé. Il y avait d'autres cas problématiques que j'ai corrigés également au passage ; normalement, tout est d'équerre désormais.

#17

Updated by Guillaume AGNIERAY 3 months ago

J'étais justement en train de préparer une nouvelle demande concernant les 2 autres cas problématiques corrigés.
Je suis donc d'avis également que tout est d'équerre désormais :)

#18

Updated by Johan Cwiklinski 3 months ago

:) merci pour ta contribution !

#19

Updated by Johan Cwiklinski 3 months ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF