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):").''.htmlentities($tidy->errorBuffer).'';
} 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:
>
> {$data.mailing_corps_display}
>
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