http://redmine.ulysses.fr/http://redmine.ulysses.fr/themes/PurpleMine2/favicon/favicon.ico?16318893672019-05-25T05:18:07ZGalette bugs & featuresGalette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192442019-05-25T05:18:07ZJohan Cwiklinskijohan@x-tnd.be
<ul><li><strong>Catégorie</strong> mis à <i>Core</i></li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>In Progress</i></li><li><strong>Assigné à</strong> mis à <i>Johan Cwiklinski</i></li></ul><p>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.</p>
<p>Le second point est quant à lui effectivement un bug.</p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192452019-05-25T05:25:11ZJohan Cwiklinskijohan@x-tnd.be
<ul><li><strong>Statut</strong> changé de <i>In Progress</i> à <i>Résolu</i></li><li><strong>% réalisé</strong> changé de <i>0</i> à <i>100</i></li></ul><p>Appliqué par commit <a class="changeset" title="Fix call to a non existing method; closes #1280" href="http://redmine.ulysses.fr/projects/galette/repository/62/revisions/349b6fc2cabb2a933c11ba561825eb2c932f0b04">349b6fc2cabb2a933c11ba561825eb2c932f0b04</a>.</p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192462019-05-25T06:43:46ZJohan Cwiklinskijohan@x-tnd.be
<ul><li><strong>Statut</strong> changé de <i>Résolu</i> à <i>Fermé</i></li></ul> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192492019-05-25T07:26:58ZAlain Paris
<ul></ul><p>Bonjour,<br />Je crois que je viens de trouver le problème:<br />Le soucis vient du nom de l’expéditeur dans Paramètres courriel le fait de mettre une apostrophe provoque les 2 anomalies.<br />L'apostrophe :ne fonctionne pas <br />L apostrophe : ok</p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192502019-05-25T07:29:27ZJohan Cwiklinskijohan@x-tnd.be
<ul><li><strong>Statut</strong> changé de <i>Fermé</i> à <i>In Progress</i></li></ul><p>Ok je vais voir a corriger le souci d'apostrophe également :)</p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192582019-06-28T13:11:31ZBenjamin Leduc
<ul></ul><p>Johan Cwiklinski a écrit :</p>
<blockquote>
<p>Appliqué par commit <a class="changeset" title="Fix call to a non existing method; closes #1280" href="http://redmine.ulysses.fr/projects/galette/repository/62/revisions/349b6fc2cabb2a933c11ba561825eb2c932f0b04">349b6fc2cabb2a933c11ba561825eb2c932f0b04</a>.</p>
</blockquote>
<p>Bonjour,</p>
<p>Le point #1 n'est toujours pas fonctionnel chez moi, nightly du jour. Comment puis-je fournir des informations utile pour avancer?</p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192592019-06-28T14:45:51ZJohan Cwiklinskijohan@x-tnd.be
<ul></ul><p>Si le problème est le même que celui d'Alain (apostrophe), ce n'est juste pas corrigé, pas eu le temps.</p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192862019-09-13T09:47:19ZGuillaume AGNIERAY
<ul><li><strong>Fichier</strong> <a href="/attachments/236">0001-Fix-unescaped-variable-in-mailing-template.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/236/0001-Fix-unescaped-variable-in-mailing-template.patch">0001-Fix-unescaped-variable-in-mailing-template.patch</a> ajouté</li></ul><p>La variable correspondante n'est pas échapée dans le template des mailings.<br />Ci-joint un correctif basé sur la branche develop, et ajoutant un modifieur smarty pour échapper la variable javascript fautive.</p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192872019-09-13T12:09:09ZJohan Cwiklinskijohan@x-tnd.be
<ul></ul><p>Merci pour le patch, ça devrait effectivement corriger le problème :)</p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192882019-09-16T15:18:15ZAlain Paris
<ul></ul><p>Bonjour,<br />Le patch doit modifier quoi pour l'utilisateur ?</p>
J'ai essayé la modification du fichier (mailing_adherents.tpl) chez moi et je n'ai pas trouvé .<br /><ins><br />Cas no 1 : Apostrophe dans le nom de l’expéditeur (exemple "l’expéditeur")</ins> situé dans Configuration (webroot/preferences) Courriel expéditeur :<br /> Sur la page webroot/mailing
<ul>
<li> le bouton "Gestion des adhérents sélectionnés" ne fonctionne plus.</li>
<li> le bouton pour modifier l’expéditeur non plus (reste sur le courriel de "depuis les préférences").</li>
<li> lors de l'appui sur prévisualisation il y a un message d'avertissement si le sujet est vide ou message est vide.<br /> 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).</li>
</ul>
<ins>Cas no 2 : pas d'apostrophe dans le nom (exemple "l expéditeur"):</ins>
<ul>
<li> Tous les boutons fonctionnent.</li>
<li> Il n'y a plus le message d'avertissement que le sujet ou le corps du message est vide par contre.</li>
<li> La fenêtre pop-up de prévisualisation fonctionne.<br /> 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é.</li>
</ul>
<p>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).</p>
<p>Il faudrait peut être une démo fonctionnelle en ligne pour voir le comportement normal.</p>
<pre><code>Bonne Journée</code></pre> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192922019-09-17T05:40:11ZGuillaume AGNIERAY
<ul><li><strong>Statut</strong> changé de <i>In Progress</i> à <i>Résolu</i></li></ul><p>Appliqué par commit <a class="changeset" title="Fix unescaped variable in mailing template; fixes #1280" href="http://redmine.ulysses.fr/projects/galette/repository/62/revisions/e4249da84d8e0907fa72226af75217e7765c8ff3">e4249da84d8e0907fa72226af75217e7765c8ff3</a>.</p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192932019-09-17T05:51:02ZJohan Cwiklinskijohan@x-tnd.be
<ul></ul><p>La modification du seul fichier template sur la dernière stable ne sera clairement pas suffisante pour corriger les problèmes de javascript.<br />Je viens de tester avec la dernière nightly, je n'ai pas rencontré de problèmes.</p>
<p>À priori, sur la branche develop, le patch fourni par Guillaume termine de corriger les problèmes.</p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192952019-09-17T14:20:10ZGuillaume AGNIERAY
<ul><li><strong>Fichier</strong> <a href="/attachments/241">0001-Fix-wrong-smarty-modifier-value-fixes-1280.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/241/0001-Fix-wrong-smarty-modifier-value-fixes-1280.patch">0001-Fix-wrong-smarty-modifier-value-fixes-1280.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/242">0001-Hotfix-refs-1280.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/242/0001-Hotfix-refs-1280.patch">0001-Hotfix-refs-1280.patch</a> ajouté</li></ul><p>A posteriori, le patch ne termine pas de corriger le problème :/</p>
<p>Échapper la variable est la solution.</p>
<p>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 : <em>js</em>. Mais <a href="https://www.smarty.net/docsv2/fr/language.modifier.escape.tpl" class="external">selon la documentation smarty</a>, la valeur dans ce cas devrait être : <em>javascript</em></p>
<p>Ci-joint un nouveau patch sur la branche develop corrigeant cette valeur.<br />Et un "hotfix" complet incluant la révision <a class="changeset" title="Fix call to a non existing method; closes #1280" href="http://redmine.ulysses.fr/projects/galette/repository/62/revisions/349b6fc2cabb2a933c11ba561825eb2c932f0b04">349b6fc2</a> pour la dernière branche stable.</p>
<p>Cette fois j'ai testé préalablement sur les 2 branches et ça fonctionne correctement :)</p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192972019-09-21T07:45:34ZJohan Cwiklinskijohan@x-tnd.be
<ul></ul><p><code>js</code> pour l'échappement est utilisé partout dans le code de Galette, à priori, ça fonctionne sans problèmes ; beaucoup d'autres choses ne fonctionneraient pas.</p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192982019-09-21T09:18:34ZGuillaume AGNIERAY
<ul></ul><p>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.</p>
<p>Dans le cas des traductions, pas de problème, c'est un plugin smarty qui s'occupe de l'échappement : <a class="source" href="http://redmine.ulysses.fr/projects/galette/repository/62/entry/galette/includes/smarty_plugins/function._T.php#L69">source:galette/includes/smarty_plugins/function._T.php#L69</a></p>
<p>Dans le cas des variables, c'est le modificateur smarty qui se charge de l'échappement. <br />Et alors, seule la valeur <em>javascript</em> peut être reconnue : <a class="external" href="https://github.com/smarty-php/smarty/blob/c9f0de05f41b9e52798b268ab1e625fac3b8578c/libs/plugins/modifier.escape.php#L177">https://github.com/smarty-php/smarty/blob/c9f0de05f41b9e52798b268ab1e625fac3b8578c/libs/plugins/modifier.escape.php#L177</a></p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=192992019-09-21T10:47:33ZJohan Cwiklinskijohan@x-tnd.be
<ul></ul><p>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.</p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=193002019-09-21T10:55:15ZGuillaume AGNIERAY
<ul></ul><p>J'étais justement en train de préparer une nouvelle demande concernant les 2 autres cas problématiques corrigés.<br />Je suis donc d'avis également que tout est d'équerre désormais :)</p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=193012019-09-21T10:59:56ZJohan Cwiklinskijohan@x-tnd.be
<ul></ul><p>:) merci pour ta contribution !</p> Galette - Anomalie #1280: problème avec le Mailinghttp://redmine.ulysses.fr/issues/1280?journal_id=193072019-09-27T05:20:08ZJohan Cwiklinskijohan@x-tnd.be
<ul><li><strong>Statut</strong> changé de <i>Résolu</i> à <i>Fermé</i></li></ul>