Hello,
J'essaie de suivre un peu le tourbillon de ménage que tu nous prépares et je me demande si c'est exprès que tu as viré les corrections de cerdic que j'avais reportée hier sur le /action/spiplistes_changer_statut_abonne.php
Je le signale au cas ou tu l'aurais loupé par mégarde, mais peut etre que je me trompe car je dis ca en lisant les commits sans vraiment avoir testé.
Il s'agit de la ligne 29-30 ici
Que j'avais maj pour rajouter les modifs de Cedric ici :
BoOz
paladin@quesaco.org a écrit :
Author: paladin@quesaco.org Date: Fri Sep 21 14:47:53 2007 New
Revision: 15398Log: fin (peut-être) optimisation action/changer statut
Modified:
_plugins_/_test_/spip-listes/spip-listes_1_9_3/action/spiplistes_changer_statut_abonne.php
_plugins_/_test_/spip-listes/spip-listes_1_9_3/spiplistes_mes_options.php
Modified:
_plugins_/_test_/spip-listes/spip-listes_1_9_3/action/spiplistes_changer_statut_abonne.php
==============================================================================
---
_plugins_/_test_/spip-listes/spip-listes_1_9_3/action/spiplistes_changer_statut_abonne.php
(original) +++
_plugins_/_test_/spip-listes/spip-listes_1_9_3/action/spiplistes_changer_statut_abonne.php
Fri Sep 21 14:47:53 2007 @@ -1,14 +1,16 @@ <?php -if
(!defined("_ECRIRE_INC_VERSION")) return;+if (!defined("_ECRIRE_INC_VERSION")) return;
function action_spiplistes_changer_statut_abonne_dist() {
+ // les globales ne passent pas en action + //global
$connect_id_auteur; + $connect_id_auteur =
$GLOBALS['auteur_session']['id_auteur']; +
include_spip('inc/autoriser');
include_spip(_DIR_PLUGIN_SPIPLISTES.'inc/spiplistes_api');- global $connect_id_auteur; - $securiser_action =
charger_fonction('securiser_action', 'inc'); $arg =
$securiser_action(); $redirect = urldecode(_request('redirect')); @@
-17,44 +19,51 @@ $id_auteur = intval($arg[0]); $action = $arg[1]; +
if(($id_auteur > 0) && ($connect_id_auteur > 0)) { if
($action=='format'){ - //changer de statut + //modification du
format abonné ('html', 'texte' ou 'non') $statut =
_request('statut');
if(autoriser('statutabonement','auteur',$id_auteur)) {
if(in_array($statut, explode(";", _SPIPLISTES_FORMATS_ALLOWED))) { -
$res = spip_query("SELECT id_auteur FROM spip_auteurs_elargis WHERE
id_auteur="._q($id_auteur)." LIMIT 1"); + if($res =
spip_query("SELECT id_auteur FROM spip_auteurs_elargis WHERE
id_auteur=$id_auteur LIMIT 1")) { if (spip_num_rows($res)) { -
spip_query("UPDATE spip_auteurs_elargis SET
`spip_listes_format`="._q($statut)." WHERE
id_auteur="._q($id_auteur)." LIMIT 1"); + spip_query("UPDATE
spip_auteurs_elargis SET `spip_listes_format`='$statut' WHERE
id_auteur=$id_auteur LIMIT 1"); } else { - spip_query("INSERT
INTO spip_auteurs_elargis (id_auteur,`spip_listes_format`) VALUES
("._q($id_auteur).","._q($statut).")"); + spip_query("INSERT
INTO spip_auteurs_elargis (id_auteur,`spip_listes_format`) VALUES
($id_auteur,'$statut')"); + } + spiplistes_log("FORMAT
ID_AUTEUR #$id_auteur changed to [$statut] by ID_AUTEUR
#$connect_id_auteur"); } - spiplistes_log("FORMAT ID_AUTEUR
#$id_auteur changed to $statut by ID_AUTEUR #$connect_id_auteur"); }
} } if ($action=='listeabo'){ - //changer de statut - $id_auteur =
_request('id_auteur'); - if ($id_auteur && ($id_liste = $arg[2]) -
&&
autoriser('abonnerauteur','liste',$id_liste,NULL,array('id_auteur'=>$id_auteur)))
{ - $result=spip_query("DELETE FROM spip_auteurs_listes WHERE
id_auteur="._q($id_auteur)." AND id_liste="._q($id_liste)); -
$result=spip_query("INSERT INTO spip_auteurs_listes
(id_auteur,id_liste) VALUES ("._q($id_auteur).","._q($id_liste).")");
+ //abonne un auteur, force en html si pas de format + if
($id_auteur && ($id_liste = intval($arg[2])) + &&
autoriser('abonnerauteur', 'liste', $id_liste, NULL,
array('id_auteur'=>$id_auteur)) + ) { +
$result=spip_query("DELETE FROM spip_auteurs_listes WHERE
id_auteur=$id_auteur AND id_liste=$id_liste"); +
$result=spip_query("INSERT INTO spip_auteurs_listes
(id_auteur,id_liste) VALUES ($id_auteur,$id_liste)"); //attribuer un
format de reception si besoin (ancien auteur) - $abo =
spip_fetch_array(spip_query("SELECT `spip_listes_format` FROM
`spip_auteurs_elargis` WHERE `id_auteur`='$id_auteur'")) ; + $abo
= spip_fetch_array(spip_query("SELECT `spip_listes_format` FROM
`spip_auteurs_elargis` WHERE `id_auteur`='$id_auteur' LIMIT 1"));
if(!$abo){ + // si auteur sans format, force en html $ok =
spip_query("UPDATE `spip_auteurs_elargis` SET
`spip_listes_format`='html' WHERE id_auteur="._q($id_auteur)); } }
spiplistes_log("SUBSCRIBE ID_AUTEUR #$id_auteur to ID_LISTE
#$id_liste by ID_AUTEUR #$connect_id_auteur"); } if
($action=='listedesabo'){ - if ($id_liste = $arg[2]) - //if
(autoriser()) - if
(autoriser('desabonnerauteur','liste',$id_liste,NULL,array('id_auteur'=>$id_auteur)))
- spip_query("DELETE FROM spip_auteurs_listes WHERE
id_auteur="._q($id_auteur)." AND id_liste="._q($id_liste)); + //
désabonne un auteur + if ($id_liste = intval($arg[2])) { + if
(autoriser('desabonnerauteur', 'liste', $id_liste, NULL,
array('id_auteur'=>$id_auteur))) { + if(spip_query("DELETE FROM
spip_auteurs_listes WHERE id_auteur=$id_auteur AND
id_liste=$id_liste")) { + spiplistes_log("UNSUBSCRIBE ID_AUTEUR
#$id_auteur from ID_LISTE #$id_liste by ID_AUTEUR
#$connect_id_auteur"); + } + } + } + } } - if ($redirect){
redirige_par_entete(str_replace("&","&",$redirect)."#abo$id_auteur");
}Modified:
_plugins_/_test_/spip-listes/spip-listes_1_9_3/spiplistes_mes_options.php
==============================================================================
---
_plugins_/_test_/spip-listes/spip-listes_1_9_3/spiplistes_mes_options.php
(original) +++
_plugins_/_test_/spip-listes/spip-listes_1_9_3/spiplistes_mes_options.php
Fri Sep 21 14:47:53 2007 @@ -20,7 +20,7 @@
define("_SPIPLISTES_CHARSET_DEFAULT", _SPIPLISTES_CHARSET_ENVOI);// les formats d'envoi autorisés, ou non pour pseudo-désabonné
-define("_SPIPLISTES_FORMATS_ALLOWED", "html;texte;none");
+define("_SPIPLISTES_FORMATS_ALLOWED", "html;texte;non");//Balises Spip-listes
_______________________________________________
Spip-zone-commit@rezo.net -
http://listes.rezo.net/mailman/listinfo/spip-zone-commit