• Type: Bug
• Resolution: Fixed
• Priority: Major
• Fix Version/s: 3.1.2, 3.2.1
• Affects Version/s: None
• Component/s: None
• Labels:

  None

1. live-comment.eml

   2 kB

   26/Apr/11 16:46

2. online-comment.eml

   2 kB

   26/Apr/11 16:46

[CMS-2691] Mail from comments is not well displayed

Collapse comment: Raphaël Franchet added a comment - 26/Apr/11 17:08, Edited by Raphaël Franchet - 26/Apr/11 17:10

Raphaël Franchet added a comment - 26/Apr/11 17:08 - edited

Ok, got it !

The problem is the following:
the mail is built using the following i18n pattern:
* content name

{0}

* site name

{4}
* content url {1}
* comment name {2}
* comment content {3}

but when the comment is pre-validated, the content url is taken from the "referer" header. This works fine on live (we are in direct) but not behind the front proxy...
and adding null as an i18n parameter does nothing so : {2} becomes {1}... and nobody becomes {4}

two ways for correcting this :
1) tranfering the referer header on proxy
2) adding an hidden field with the url in the comment form ; this solution may be a problem for those who have overriden the comment appearance (and I don't know if we can easily get it in a content xsl...)

Expand comment: Raphaël Franchet added a comment - 26/Apr/11 17:08, Edited by Raphaël Franchet - 26/Apr/11 17:10

Raphaël Franchet added a comment - 26/Apr/11 17:08 - edited Ok, got it ! The problem is the following: the mail is built using the following i18n pattern: * content name {0} * site name {4} * content url {1} * comment name {2} * comment content {3} but when the comment is pre-validated, the content url is taken from the "referer" header. This works fine on live (we are in direct) but not behind the front proxy... and adding null as an i18n parameter does nothing so : {2} becomes {1}... and nobody becomes {4} two ways for correcting this : 1) tranfering the referer header on proxy 2) adding an hidden field with the url in the comment form ; this solution may be a problem for those who have overriden the comment appearance (and I don't know if we can easily get it in a content xsl...)

Collapse comment: Anne-Sophie Labadie (Inactive) added a comment - 26/Apr/11 16:47

Anne-Sophie Labadie (Inactive) added a comment - 26/Apr/11 16:47

Live version :
Un nouveau commentaire a été ajouté sur le contenu 'Organisation' du site 'www'.
Cliquez ici pour visualiser le contenu http://localhost:8080/cms/live/www/fr/l-universite/organisation.html

Commentaire de labadie :
1er commentaire

Online version :
Un nouveau commentaire a été ajouté sur le contenu 'Organisation' du site ''.
Cliquez ici pour visualiser le contenu anne-sophie

Commentaire de 2ème commentaire :
www

Expand comment: Anne-Sophie Labadie (Inactive) added a comment - 26/Apr/11 16:47

Anne-Sophie Labadie (Inactive) added a comment - 26/Apr/11 16:47 Live version : Un nouveau commentaire a été ajouté sur le contenu 'Organisation' du site 'www'. Cliquez ici pour visualiser le contenu http://localhost:8080/cms/live/www/fr/l-universite/organisation.html Commentaire de labadie : 1er commentaire Online version : Un nouveau commentaire a été ajouté sur le contenu 'Organisation' du site ''. Cliquez ici pour visualiser le contenu anne-sophie Commentaire de 2ème commentaire : www