Ligne "titre" du ticket apparaît 2x dans les mails

eiseli · 2008-07-10 15:47:38

GLPI v0.71-RC3

Je viens de m'appercevoir que autant dans la version HTML que dans la version texte des mails envoyés, le titre du ticket apparaît deux fois.

grep -n '$LANG\["common"\]\[57\]' inc/tracking.class.php
883:                    $message.="<span style='color:#8B8C8F; font-weight:bold;  text-decoration:underline; '>".$LANG["common"][57].":</span> ".$this->fields["name"]."<br>";
924:                    $message.="<span style='color:#8B8C8F; font-weight:bold;  text-decoration:underline; '>".$LANG["common"][57].":</span> ".$this->fields["name"]."<br>";
930:                    $message.=mailRow($LANG["common"][57],$this->fields["name"]);
972:                    $message.= $LANG["common"][57]." : ".$this->fields["name"]."\n";

D'autre part, il y a une différence entre les lignes 930 et 972. Je suppose qu'une version est plus "actuelle" et l'autre est un cadavre?

Finalement, petite modification cosmétique qui s'impose à mon avis: remplacer les <span style='color...'> par un <span class='...'> et définir la classe CSS une fois pour toutes dans le <head>. 2 fichiers seraient touchés par cette modif: inc/tracking.class.php et inc/reservation.class.php.

MoYo · 2008-07-10 17:47:33

le titre est remis pour rappel avant le contenu de l'incident. C'est pour que le mail soit plus lisible.

eiseli · 2008-07-11 08:14:16

Mouais, je trouve que ce rappel n'est pas nécessaire. Surtout dans les mails html où justement le la présentation peut être faite en telle sorte qu'il est évident de quoi on parle.

Note que le titre apparaît aussi dans l'objet du mail, il est donc 3x sur chaque message; c'est une redondance qui dérange plus qu'elle n'aide. "moins est plus"!

doum · 2008-07-11 09:05:09

Si cela vous gene vous pouvez facilement commentez les lignes en trop

eiseli · 2008-07-11 09:13:11

Bien sûr, c'est déjà fait. Ce bug report avait cependant le but de donner mon retour, car si chacun commente cette ligne et ne vous donne pas de retour, c'est un peu dommage, non?

Par contre la seconde partie de mon report mettait le doigt sur les lignes 930 et 972, écrites de façon différente pour le même effet. Ne faudrait-il pas s'entendre sur une manière et donc corriger une de ces 2 lignes? Merci.

MoYo · 2008-07-11 17:31:56

c'est modifié

JMD · 2008-07-11 21:51:57

Pas de souci sur votre retour, vous avez bien fait.

Forum GLPI-Project

Announcement

#1 2008-07-10 15:47:38

Ligne "titre" du ticket apparaît 2x dans les mails

#2 2008-07-10 17:47:33

Re: Ligne "titre" du ticket apparaît 2x dans les mails

#3 2008-07-11 08:14:16

Re: Ligne "titre" du ticket apparaît 2x dans les mails

#4 2008-07-11 09:05:09

Re: Ligne "titre" du ticket apparaît 2x dans les mails

#5 2008-07-11 09:13:11

Re: Ligne "titre" du ticket apparaît 2x dans les mails

#6 2008-07-11 17:31:56

Re: Ligne "titre" du ticket apparaît 2x dans les mails

#7 2008-07-11 21:51:57

Re: Ligne "titre" du ticket apparaît 2x dans les mails

Board footer