Re: [Galette-devel] Bug #9705: pb d'interpretaion html (mailing adherents)

2008-08-21 Par sujet Kevin Klein

 Ok merci, je vais voir à merger tout ça sur le SVN... J'avais fait
 quelque chose à ce sujet, mais peut être une amélioration est-elle
 possible :)

   

ok mon code php n'est surement pas très 'pro', je suis débutant dans ce 
domaine, donc il y a certainement des façons d'écrire ça en plus 
court/plus propre... mais ça marche ;)
 Heu... Le warning vient du navigateur, pas du parseur :)

 Les pages sont envoyées avec une déclaration XML, si le XML n'est pas
 valide, le browser va alors refuser d'afficher la page. Peut être un
 coup de tidy pourrait résoudre ce problème ? À vrai dire, je n'ai aps du
 tout testé ce genre de choses, je n'envoie que des mails au format texte :p
   

Ah... ok ;) J'ai regardé la doc de tidy, c'est une super extension... 
J'ai testé, ci-dessous modif à faire dans le bloc '#si le mail est en html'.
Cela ne marche que si l'extension tidy est installée... j'ai pas trouvé 
d'include externe (comme gettext par exemple). Il faudrait également 
regénérer le .po pour la nouvelle erreur que j'ai inclue dans le code si 
tidy n'arrive pas à corriger.

Kevin

#si le mail est en html
if (isset($_POST['mailing_html'])) {
$data['mailing_html']=$_POST['mailing_html'];
If (get_magic_quotes_gpc()==1) {
$data['mailing_corps'] = 
htmlentities(stripslashes($_POST['mailing_corps']));

$data['mailing_corps_display']=nl2br(html_entity_decode($data['mailing_corps']));

$data['mailing_corps_to_send']=$data['mailing_corps_display'];
} else {
$data['mailing_corps'] = 
htmlentities($_POST['mailing_corps']);

$data['mailing_corps_display']=nl2br(html_entity_decode($data['mailing_corps']));

$data['mailing_corps_to_send']=$data['mailing_corps_display'];
}
   
#HTML check with Tidy
If (extension_loaded('tidy'))
{
$tidy_config = array(
'doctype' = omit,
'output-xhtml' = true,
'show-warnings' = false,
'show-body-only' = true);
$tidy = new tidy();

$tidy-parseString(html_entity_decode($data['mailing_corps_to_send']),$tidy_config);
$tidy-cleanRepair();
   
if ($tidy-errorBuffer) {
$error_detected[] = _T(You have HTML syntax 
error(s):).'p'.htmlentities($tidy-errorBuffer).'/p';
} else {
$data['mailing_corps_to_send'] = $tidy;
$data['mailing_corps_display'] = $tidy;
$data['mailing_corps'] = htmlentities($tidy);
}
}
   
}

___
Galette-devel mailing list
Galette-devel@gna.org
https://mail.gna.org/listinfo/galette-devel


Re: [Galette-devel] Bug #9705: pb d'interpretaion html (mailing adherents)

2008-08-20 Par sujet Johan Cwiklinski
Salut,

Kevin Klein a écrit :
 Kevin Klein a écrit :
   
 Bonjour,

 Je ne sais pas si c'est l'endroit adéquat, je vous soumets mes 
 modifications concernant le bug#9705 concernant un pb d'interpretation 
 lors de la création d'un email via le mailing adherents. Cette modif 
 prends en compte les magic quotes, les retours à la ligne et la 
 prévisualisation des mails html (auparavant les codes html créés 
 étaient sniffés chez moi lors de l'edit d'un message déjà prévisualisé).

 NB: la version utilisée et la 0.63 bugfix. Je vous joins également les 
 fichiers modifiés.

 En ésperant que ça fasse avancer le schmilblick pour une v.0.63 finale ;)
 

Ok merci, je vais voir à merger tout ça sur le SVN... J'avais fait
quelque chose à ce sujet, mais peut être une amélioration est-elle
possible :)

 Cordialement,

 Kevin Klein

 Les fichiers modifiés sont: mailing_adherents.php et mailing_adhrents.tpl


 
 Rebonjour,

 J'oubliais de préciser que lors d'un envoi html, si il y a erreur dans 
 la syntaxe (par ex. oubli de fermer une balise), cela me donne une 
 erreur XML lors de la prévisualisation. Logique à priori, mais est-il 
 possible de désactiver ce warning du parseur?
   

Heu... Le warning vient du navigateur, pas du parseur :)

Les pages sont envoyées avec une déclaration XML, si le XML n'est pas
valide, le browser va alors refuser d'afficher la page. Peut être un
coup de tidy pourrait résoudre ce problème ? À vrai dire, je n'ai aps du
tout testé ce genre de choses, je n'envoie que des mails au format texte :p
 Merci,

 Kevin
   

Johan



signature.asc
Description: OpenPGP digital signature
___
Galette-devel mailing list
Galette-devel@gna.org
https://mail.gna.org/listinfo/galette-devel


Re: [Galette-devel] Bug #9705: pb d'interpretaion html (mailing adherents)

2008-08-19 Par sujet Kevin Klein
Kevin Klein a écrit :
 Bonjour,

 Je ne sais pas si c'est l'endroit adéquat, je vous soumets mes 
 modifications concernant le bug#9705 concernant un pb d'interpretation 
 lors de la création d'un email via le mailing adherents. Cette modif 
 prends en compte les magic quotes, les retours à la ligne et la 
 prévisualisation des mails html (auparavant les codes html créés 
 étaient sniffés chez moi lors de l'edit d'un message déjà prévisualisé).

 NB: la version utilisée et la 0.63 bugfix. Je vous joins également les 
 fichiers modifiés.

 En ésperant que ça fasse avancer le schmilblick pour une v.0.63 finale ;)

 Cordialement,

 Kevin Klein

 Les fichiers modifiés sont: mailing_adherents.php et mailing_adhrents.tpl

 Modifs mailing adherents.php

* Lignes 76 à 94 supprimées et remplacées par:

 if (isset($_POST[mailing_go]) || isset($_POST[mailing_reset]) || 
 isset($_POST[mailing_confirm]))
{
if ($_POST['mailing_objet']==)
$error_detected[] = _T(Please type an object for the 
 message.);
else
If (get_magic_quotes_gpc()==1) {

 $data['mailing_objet']=htmlentities(stripslashes($_POST['mailing_objet'])); 


 $data['mailing_objet_to_send']=html_entity_decode($data['mailing_objet']); 

} else {

 $data['mailing_objet']=htmlentities($_POST['mailing_objet']);

 $data['mailing_objet_to_send']=html_entity_decode($data['mailing_objet']); 

}

if ($_POST['mailing_corps']==) {
$error_detected[] = _T(Please enter a message.);
} else {
#si le mail est en html
if (isset($_POST['mailing_html'])) {
$data['mailing_html']=$_POST['mailing_html'];
If (get_magic_quotes_gpc()==1) {
$data['mailing_corps'] = 
 htmlentities(stripslashes($_POST['mailing_corps']));

 $data['mailing_corps_display']=nl2br(html_entity_decode($data['mailing_corps']));
  


 $data['mailing_corps_to_send']=$data['mailing_corps_display'];
} else {
$data['mailing_corps'] = 
 htmlentities($_POST['mailing_corps']);

 $data['mailing_corps_display']=nl2br(html_entity_decode($data['mailing_corps']));
  


 $data['mailing_corps_to_send']=$data['mailing_corps_display'];
}
} else {
#si le mail est en texte
$data['mailing_html']=0;
If (get_magic_quotes_gpc()==1) {
$data['mailing_corps'] = 
 htmlentities(stripslashes($_POST['mailing_corps']));
$data['mailing_corps_display']=$data['mailing_corps'];

 $data['mailing_corps_to_send']=html_entity_decode($data['mailing_corps_display']);
  

} else {
$data['mailing_corps'] = 
 htmlentities($_POST['mailing_corps']);
$data['mailing_corps_display']=$data['mailing_corps'];

 $data['mailing_corps_to_send']=html_entity_decode($data['mailing_corps_display']);
  

}
}
}

* Lignes 120 et 121 supprimée et remplacées par:

 $data['mailing_objet_to_send'],
 $data['mailing_corps_to_send'],


 Modif faite dans mailing_adherents.tpl:

* Lignes 55 à 59 supprimée et remplacées par:

 pre{$data.mailing_corps_display}/pre

Rebonjour,

J'oubliais de préciser que lors d'un envoi html, si il y a erreur dans 
la syntaxe (par ex. oubli de fermer une balise), cela me donne une 
erreur XML lors de la prévisualisation. Logique à priori, mais est-il 
possible de désactiver ce warning du parseur?

Merci,

Kevin

___
Galette-devel mailing list
Galette-devel@gna.org
https://mail.gna.org/listinfo/galette-devel