[SPIP Zone] [Spip-zone-commit] r32700 - in /_plugins_/clevermail/2_0: clevermail_nouveautes_html.html inc/clevermail_post_create.php

9aa2aa68144b2781ac02 · Novembre 6, 2009, 12:00

Je ne comprends vraiment pas pourquoi tu mets toute cette partie dans la boucle alors que @@NOM_LETTRE@@ et @@DESCRIPTION@@ étaient en dehors :

<!DOCTYPE html PUBLIC '-//W3C//DTD XHTML 1.0 Strict//EN' 'http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd’><html>
<head><title>Les nouveautés depuis le (#DATE|affdate)</title></head>
<body>#SET{date,oui}]
<h1>@@NOM_LETTRE@@</h1>

<div>@@DESCRIPTION@@</div>

Quel était le problème ?

Et sinon, attention à ta variable nommée "date", c'est déjà une variable d'environnement, cela pourrait avoir des effets secondaires imprévisibles...

Le 6 nov. 2009 à 09:55, pierre.fiches@free.fr a écrit :

Author: pierre.fiches@free.fr
Date: Fri Nov 6 09:55:26 2009
New Revision: 32700

Log:
sujet automatique plus valide, remplacement de eregi

Modified:
   _plugins_/clevermail/2_0/clevermail_nouveautes_html.html
   _plugins_/clevermail/2_0/inc/clevermail_post_create.php

Modified: _plugins_/clevermail/2_0/clevermail_nouveautes_html.html

--- _plugins_/clevermail/2_0/clevermail_nouveautes_html.html (original)
+++ _plugins_/clevermail/2_0/clevermail_nouveautes_html.html Fri Nov 6 09:55:26 2009
@@ -1,9 +1,11 @@
#CACHE{0}
#HTTP_HEADER{Content-type: text/html; charset=#CHARSET}
-<B_test_contenu>
+<B_test_contenu>#SET{date,''}
<BOUCLE_test_contenu(AUTEURS){0,1}{tous}><BOUCLE_articles(ARTICLES){age_relatif<=0}{par date}{inverse}{0,20}>
[(#COMPTEUR_BOUCLE|=={1}|oui)
-[<title>(#INCLURE{fond=inclure/sujet1,sujet=Les nouveautÃ©s depuis le #DATE|affdate})</title>]
+[<!DOCTYPE html PUBLIC '-//W3C//DTD XHTML 1.0 Strict//EN' 'http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd’><html>
+<head><title>Les nouveautÃ©s depuis le (#DATE|affdate)</title></head>
+<body>#SET{date,oui}]
<h1>@@NOM_LETTRE@@</h1>

<div>@@DESCRIPTION@@</div>
@@ -15,4 +17,5 @@
</BOUCLE_articles></BOUCLE_test_contenu>

<p>Cliquez sur le lien suivant pour vous dÃ©sinscrire : <a href="@@URL_DESINSCRIPTION@@">@@URL_DESINSCRIPTION@@</a></p>
+[(#GET{date}|oui)</body></html>]
</B_test_contenu>
\ No newline at end of file

Modified: _plugins_/clevermail/2_0/inc/clevermail_post_create.php

--- _plugins_/clevermail/2_0/inc/clevermail_post_create.php (original)
+++ _plugins_/clevermail/2_0/inc/clevermail_post_create.php Fri Nov 6 09:55:26 2009
@@ -12,7 +12,8 @@
    $url_text = $list['lst_url_text'].(strpos($list['lst_url_html'], '?') !== false ? '&' : '?').'date='.date("Y-m-d",$last_create);
    $post['pst_text'] = recuperer_page($url_text);
    if (trim($post['pst_html']) != '' && trim($post['pst_text']) != '') {
- if (eregi("<title>(.*)</title>", $post['pst_html'], $regs)) {
+ //if (eregi("<title>(.*)</title>", $post['pst_html'], $regs)) {
+ if (preg_match(",<title>(.*)</title>,", $post['pst_html'], $regs)) {
        $post['pst_subject'] = trim($regs[1]);
      } else {
        $post['pst_subject'] = 'Aucun sujet';

_______________________________________________
Spip-zone-commit@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone-commit

-Nicolas

--
Nicolas HOIZEY
Blog : http://www.gasteroprod.com/
Photos : http://flic.kr/nicolas-hoizey/

Pierre_FICHES · Novembre 6, 2009, 12:27

Le 6 nov. 09 à 13:00, Nicolas Hoizey a écrit :

Je ne comprends vraiment pas pourquoi tu mets toute cette partie dans la boucle alors que @@NOM_LETTRE@@ et @@DESCRIPTION@@ étaient en dehors :
Les nouveautés depuis le (#DATE|affdate) #SET{date,oui}]
@@NOM_LETTRE@@

@@DESCRIPTION@@

Quel était le problème ?

Comme hier : « Les nouveautés depuis le (#DATE|affdate… » doit -être dans la boucle et ne voulant pas mettre le h1 avant le title ni utiliser une deuxième boucle, j’ai déplacé h1 et div relativement.
Ça pose un problème ?

Si le problème est le title on peut le supprimer aussi. C’est pas un soucis pour moi qui ne cherchait qu’à faire profiter tout le monde d’un truc qui me semble utile.

Et sinon, attention à ta variable nommée « date », c’est déjà une variable d’environnement, cela pourrait avoir des effets secondaires imprévisibles…

J’en ai pas constaté mais si tu crois je vais la changer.

9aa2aa68144b2781ac02 · Novembre 6, 2009, 12:46

Le 6 nov. 2009 à 13:27, Pierre Fiches a écrit :

Le 6 nov. 09 à 13:00, Nicolas Hoizey a écrit :

Je ne comprends vraiment pas pourquoi tu mets toute cette partie dans la boucle alors que @@NOM_LETTRE@@ et @@DESCRIPTION@@ étaient en dehors :
Les nouveautés depuis le (#DATE|affdate) #SET{date,oui}]
@@NOM_LETTRE@@

@@DESCRIPTION@@

Quel était le problème ?

Comme hier : « Les nouveautés depuis le (#DATE|affdate… » doit -être dans la boucle

Bin non, #DATE est censé exister en dehors de la boucle, puisque CleverMail le passe à la page lors de la création du message, et ça vaut la date d’envoi du message précédent.

En le déplaçant dans la boucle, ça devient la date du plus ancien article publié après le dernier envoi, ce qui n’a rien à voir.

et ne voulant pas mettre le h1 avant le title ni utiliser une deuxième boucle, j’ai déplacé h1 et div relativement.
Ça pose un problème ?

Si le problème est le title on peut le supprimer aussi. C’est pas un soucis pour moi qui ne cherchait qu’à faire profiter tout le monde d’un truc qui me semble utile.

Non, ce n’est pas cet ajout que je critique, au contraire, juste son emplacement.

Et sinon, attention à ta variable nommée « date », c’est déjà une variable d’environnement, cela pourrait avoir des effets secondaires imprévisibles…

J’en ai pas constaté mais si tu crois je vais la changer.

Je pense que c’est préférable. Mais de toute façon, si on tombe d’accord sur l’autre point, ce ne sera plus nécessaire…

-Nicolas

–
Nicolas HOIZEY
Blog : http://www.gasteroprod.com/
Photos : http://flic.kr/nicolas-hoizey/

Pierre_FICHES · Novembre 6, 2009, 1:12

Le 6 nov. 09 à 13:46, Nicolas Hoizey a écrit :

Le 6 nov. 2009 à 13:27, Pierre Fiches a écrit :

Le 6 nov. 09 à 13:00, Nicolas Hoizey a écrit :

Je ne comprends vraiment pas pourquoi tu mets toute cette partie dans la boucle alors que @@NOM_LETTRE@@ et @@DESCRIPTION@@ étaient en dehors :
Les nouveautés depuis le (#DATE|affdate) #SET{date,oui}]
@@NOM_LETTRE@@

@@DESCRIPTION@@

Quel était le problème ?

Comme hier : « Les nouveautés depuis le (#DATE|affdate… » doit -être dans la boucle

Bin non, #DATE est censé exister en dehors de la boucle, puisque CleverMail le passe à la page lors de la création du message, et ça vaut la date d’envoi du message précédent.

En le déplaçant dans la boucle, ça devient la date du plus ancien article publié après le dernier envoi, ce qui n’a rien à voir.

et ne voulant pas mettre le h1 avant le title ni utiliser une deuxième boucle, j’ai déplacé h1 et div relativement.
Ça pose un problème ?

Si le problème est le title on peut le supprimer aussi. C’est pas un soucis pour moi qui ne cherchait qu’à faire profiter tout le monde d’un truc qui me semble utile.

Non, ce n’est pas cet ajout que je critique, au contraire, juste son emplacement.

Et sinon, attention à ta variable nommée « date », c’est déjà une variable d’environnement, cela pourrait avoir des effets secondaires imprévisibles…

J’en ai pas constaté mais si tu crois je vais la changer.

Je pense que c’est préférable. Mais de toute façon, si on tombe d’accord sur l’autre point, ce ne sera plus nécessaire…

Bon ok j’ai enfin compris…

une autre idée pour le nom : messager ou messageur

9aa2aa68144b2781ac02 · Novembre 6, 2009, 3:14

Le 6 nov. 2009 à 14:12, Pierre Fiches a écrit :

Bon ok j’ai enfin compris…

Super ! \o/

une autre idée pour le nom : messager ou messageur

C’est déjà mieux, mais pas forcément très explicite. On est plus sur des synonymes du facteurs, là.

Si on veut éviter le classique mais anglophone « newsletters », j’hésite plutôt entre :

infolettres
lettres d’information
bulletins d’information

-Nicolas

–
Nicolas HOIZEY
Blog : http://www.gasteroprod.com/
Photos : http://flic.kr/nicolas-hoizey/

Pierre_FICHES · Novembre 6, 2009, 3:49

Le 6 nov. 09 à 16:14, Nicolas Hoizey a écrit :

Le 6 nov. 2009 à 14:12, Pierre Fiches a écrit :

Bon ok j'ai enfin compris...

Super ! \o/

une autre idée pour le nom : messager ou messageur

C'est déjà mieux, mais pas forcément très explicite. On est plus sur des synonymes du facteurs, là.

Si on veut éviter le classique mais anglophone "newsletters", j'hésite plutôt entre :

- infolettres

je vote pour compréhensible, court, un seul mot, c'est aussi un clin d'œil à la francophonie.

- lettres d'information
- bulletins d'information