[SPIP Zone] [Spip-zone-commit] r23529 - in /_fondation_/acces_restreint: ./ exec/ inc/

En quoi as-tu besoin de toucher au code d'un plugin qui n'est pas compatible 2.0 sous pretexte de problemes avec la 2.0 ?

Et encore en plus tu te permet de remettre un
<necessite id="SPIP" version='[1.925;]' />
qui est faux, à la place de
<necessite id="SPIP" version='[;1.925)' />

Tu ferais mieux de mettre à jour tes sites et tes plugins plutôt que de venir casser des versions stables.
En l'occurence cette version est figée, et était fonctionnelle avec la 1.9.2 avec laquelle tu as potentiellement généré des plantages maintenant.

Au cas où tu ne serais pas au courant, il y a maintenant

Merci de revert ce commit rapidement et intégralement

Cédric

Le 21 oct. 08 à 10:28, real3t@gmail.com a écrit :

Author: real3t@gmail.com
Date: Tue Oct 21 10:28:55 2008
New Revision: 23529

Log:
Sur Linux, la beta 2 de SPIP crée 2 fois les pipelines :
function execute_pipeline_accesrestreint_liste_zones_autorisees(&$val){
et
function execute_pipeline_AccesRestreint_liste_zones_autorisees(&$val){

Ce qui évidement plante le site.

Passage en minuscule partout (enfin, j'espère).

Modified:
    _fondation_/acces_restreint/acces_restreint.php
    _fondation_/acces_restreint/acces_restreint_options.php
    _fondation_/acces_restreint/acces_restreint_pipelines.php
    _fondation_/acces_restreint/acces_restreint_prive.php
    _fondation_/acces_restreint/exec/acces_restreint.php
    _fondation_/acces_restreint/exec/acces_restreint_config.php
    _fondation_/acces_restreint/exec/acces_restreint_edit.php
    _fondation_/acces_restreint/inc/acces_restreint.php
    _fondation_/acces_restreint/inc/acces_restreint_gestion.php
    _fondation_/acces_restreint/inc/acces_restreint_install.php
    _fondation_/acces_restreint/plugin.xml

Modified: _fondation_/acces_restreint/acces_restreint.php

--- _fondation_/acces_restreint/acces_restreint.php (original)
+++ _fondation_/acces_restreint/acces_restreint.php Tue Oct 21 10:28:55 2008
@@ -22,7 +22,7 @@
   if (!isset($boucle->modificateur['tout_voir'])){
     $t = $boucle->id_table . '.id_rubrique';
     $boucle->select = array_merge($boucle->select, array($t)); // pour postgres
- $boucle->where = 'AccesRestreint_rubriques_accessibles_where("'.$t.'")';
+ $boucle->where = 'accesrestreint_rubriques_accessibles_where("'.$t.'")';
   }
   return boucle_ARTICLES_dist($id_boucle, $boucles);
}
@@ -35,7 +35,7 @@
   if (!isset($boucle->modificateur['tout_voir'])){
     $t = $boucle->id_table . '.id_rubrique';
     $boucle->select = array_merge($boucle->select, array($t)); // pour postgres
- $boucle->where = 'AccesRestreint_rubriques_accessibles_where("'.$t.'")';
+ $boucle->where = 'accesrestreint_rubriques_accessibles_where("'.$t.'")';
   }
   return boucle_BREVES_dist($id_boucle, $boucles);
}
@@ -48,15 +48,15 @@
   if (!isset($boucle->modificateur['tout_voir'])){
     $t = $boucle->id_table . '.id_rubrique';
     $boucle->select = array_merge($boucle->select, array($t)); // pour postgres
- $boucle->where = 'AccesRestreint_rubriques_accessibles_where("'.$t.'")';
+ $boucle->where = 'accesrestreint_rubriques_accessibles_where("'.$t.'")';

     $t = $boucle->id_table . '.id_article';
     $boucle->select = array_merge($boucle->select, array($t));
- $boucle->where = 'AccesRestreint_articles_accessibles_where("'.$t.'")';
+ $boucle->where = 'accesrestreint_articles_accessibles_where("'.$t.'")';

     $t = $boucle->id_table . '.id_breve';
     $boucle->select = array_merge($boucle->select, array($t));
- $boucle->where = 'AccesRestreint_breves_accessibles_where("'.$t.'")';
+ $boucle->where = 'accesrestreint_breves_accessibles_where("'.$t.'")';
   }
   return boucle_FORUMS_dist($id_boucle, $boucles);
}
@@ -69,7 +69,7 @@
   if (!isset($boucle->modificateur['tout_voir'])) {
     $t = $boucle->id_table . '.id_article';
     $boucle->select = array_merge($boucle->select, array($t));
- $boucle->where = 'AccesRestreint_articles_accessibles_where("'.$t.'")';
+ $boucle->where = 'accesrestreint_articles_accessibles_where("'.$t.'")';
   }
   return boucle_SIGNATURES_dist($id_boucle, $boucles);
}
@@ -82,7 +82,7 @@
   if (!isset($boucle->modificateur['tout_voir'])){
     $t = $boucle->id_table . '.' . $boucle->primary;
     $boucle->select = array_merge($boucle->select, array($t));
- $boucle->where = 'AccesRestreint_documents_accessibles_where("'.$t.'")';
+ $boucle->where = 'accesrestreint_documents_accessibles_where("'.$t.'")';
   }
   return boucle_DOCUMENTS_dist($id_boucle, $boucles);
}
@@ -95,7 +95,7 @@
   if (!isset($boucle->modificateur['tout_voir'])){
     $t = $boucle->id_table . '.' . $boucle->primary;
     $boucle->select = array_merge($boucle->select, array($t));
- $boucle->where = 'AccesRestreint_rubriques_accessibles_where("'.$t.'")';
+ $boucle->where = 'accesrestreint_rubriques_accessibles_where("'.$t.'")';
   }
   return boucle_RUBRIQUES_dist($id_boucle, $boucles);
}
@@ -108,7 +108,7 @@
   if (!isset($boucle->modificateur['tout_voir'])){
     $t = $boucle->id_table . '.' . $boucle->primary;
     $boucle->select = array_merge($boucle->select, array($t));
- $boucle->where = 'AccesRestreint_rubriques_accessibles_where("'.$t.'")';
+ $boucle->where = 'accesrestreint_rubriques_accessibles_where("'.$t.'")';
   }
   return boucle_HIERARCHIE_dist($id_boucle, $boucles);
}
@@ -121,7 +121,7 @@
   if (!isset($boucle->modificateur['tout_voir'])){
     $t = $boucle->id_table . '.id_rubrique';
     $boucle->select = array_merge($boucle->select, array($t));
- $boucle->where = 'AccesRestreint_rubriques_accessibles_where("'.$t.'")';
+ $boucle->where = 'accesrestreint_rubriques_accessibles_where("'.$t.'")';
   }
   return boucle_SYNDICATION_dist($id_boucle, $boucles);
}
@@ -134,7 +134,7 @@
   if (!isset($boucle->modificateur['tout_voir'])){
     $t = $boucle->id_table . '.' . $boucle->primary;
     $boucle->select = array_merge($boucle->select, array($t));
- $boucle->where = 'AccesRestreint_syndic_articles_accessibles_where("'.$t.'")';
+ $boucle->where = 'accesrestreint_syndic_articles_accessibles_where("'.$t.'")';
   }
   return boucle_SYNDIC_ARTICLES_dist($id_boucle, $boucles);
}
@@ -147,7 +147,7 @@
   if (!isset($boucle->modificateur['tout_voir'])){
     $t = $boucle->id_table . '.' . $boucle->primary;
     $boucle->select = array_merge($boucle->select, array($t));
- $boucle->where = 'AccesRestreint_evenements_accessibles_where("'.$t.'")';
+ $boucle->where = 'accesrestreint_evenements_accessibles_where("'.$t.'")';
   }
   return boucle_EVENEMENTS_dist($id_boucle, $boucles);
}

Modified: _fondation_/acces_restreint/acces_restreint_options.php

--- _fondation_/acces_restreint/acces_restreint_options.php (original)
+++ _fondation_/acces_restreint/acces_restreint_options.php Tue Oct 21 10:28:55 2008
@@ -4,24 +4,24 @@
if (!defined("_ECRIRE_INC_VERSION")) return;

// declarer le pipeline pour le core
-$GLOBALS['spip_pipeline']['AccesRestreint_liste_zones_autorisees']='';
+$GLOBALS['spip_pipeline']['accesrestreint_liste_zones_autorisees']='';

// Si on n'est pas connecte, aucune autorisation n'est disponible
// pas la peine de sortir la grosse artillerie
if (!isset($GLOBALS['auteur_session']['id_auteur'])){
- $GLOBALS['AccesRestreint_zones_autorisees'] = '';
+ $GLOBALS['accesrestreint_zones_autorisees'] = '';
}
else {
   // Pipeline : calculer les zones autorisees, sous la forme '1,2,3'
   // TODO : avec un petit cache pour eviter de solliciter la base de donnees
- $GLOBALS['AccesRestreint_zones_autorisees'] =
- pipeline('AccesRestreint_liste_zones_autorisees', '');
+ $GLOBALS['accesrestreint_zones_autorisees'] =
+ pipeline('accesrestreint_liste_zones_autorisees', '');
}

// Ajouter un marqueur de cache pour le differencier selon les autorisations
if (!isset($GLOBALS['marqueur'])) $GLOBALS['marqueur'] = '';
-$GLOBALS['marqueur'] .= ":AccesRestreint_zones_autorisees="
- .$GLOBALS['AccesRestreint_zones_autorisees'];
+$GLOBALS['marqueur'] .= ":accesrestreint_zones_autorisees="
+ .$GLOBALS['accesrestreint_zones_autorisees'];

//
// Autorisations
@@ -38,11 +38,11 @@
   if (!isset($rub_exclues[$publique]) || !is_array($rub_exclues[$publique])) {
     // Si autoriser est appelee pour un autre auteur que l'auteur connecte ou si pas d'auteur connecte , on passe $id_auteur en parametre
     if(isset($GLOBALS['auteur_session']['id_auteur']) && isset($qui['id_auteur']) && $qui['id_auteur']!=$GLOBALS['auteur_session']['id_auteur'])
- $rub_exclues[$publique] = AccesRestreint_liste_rubriques_exclues($publique,$qui['id_auteur']);
+ $rub_exclues[$publique] = accesrestreint_liste_rubriques_exclues($publique,$qui['id_auteur']);
     elseif (!isset($GLOBALS['auteur_session']['id_auteur']) && isset($qui['id_auteur']))
- $rub_exclues[$publique] = AccesRestreint_liste_rubriques_exclues($publique,$qui['id_auteur']);
+ $rub_exclues[$publique] = accesrestreint_liste_rubriques_exclues($publique,$qui['id_auteur']);
     else
- $rub_exclues[$publique] = AccesRestreint_liste_rubriques_exclues($publique);
+ $rub_exclues[$publique] = accesrestreint_liste_rubriques_exclues($publique);
     $rub_exclues[$publique] = array_flip($rub_exclues[$publique]);
   }
   return !isset($rub_exclues[$publique][$id]);
@@ -59,11 +59,11 @@
   if (!isset($art_exclus[$publique]) || !is_array($art_exclus[$publique])) {
     // Si autoriser est appelee pour un autre auteur que l'auteur connecte ou si pas d'auteur connecte , on passe $id_auteur en parametre
     if(isset($GLOBALS['auteur_session']['id_auteur']) && isset($qui['id_auteur']) && $qui['id_auteur']!=$GLOBALS['auteur_session']['id_auteur'])
- $art_exclus[$publique] = AccesRestreint_liste_articles_exclus($publique,$qui['id_auteur']);
+ $art_exclus[$publique] = accesrestreint_liste_articles_exclus($publique,$qui['id_auteur']);
     elseif (!isset($GLOBALS['auteur_session']['id_auteur']) && isset($qui['id_auteur']))
- $art_exclus[$publique] = AccesRestreint_liste_articles_exclus($publique,$qui['id_auteur']);
+ $art_exclus[$publique] = accesrestreint_liste_articles_exclus($publique,$qui['id_auteur']);
     else
- $art_exclus[$publique] = AccesRestreint_liste_articles_exclus($publique);
+ $art_exclus[$publique] = accesrestreint_liste_articles_exclus($publique);
     $art_exclus[$publique] = array_flip($art_exclus[$publique]);
   }
   return !isset($art_exclus[$publique][$id]);
@@ -80,11 +80,11 @@
   if (!isset($breves_exclues[$publique]) || !is_array($breves_exclues[$publique])) {
     // Si autoriser est appelee pour un autre auteur que l'auteur connecte ou si pas d'auteur connecte , on passe $id_auteur en parametre

[... 803 lines stripped ...]
_______________________________________________
Spip-zone-commit@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone-commit

* cedric.morin@yterium.com tapuscrivait, le 21/10/2008 10:43:

En quoi as-tu besoin de toucher au code d'un plugin qui n'est pas compatible 2.0 sous pretexte de problemes avec la 2.0 ?

Parce que j'ai essayé de mettre accès restreint 3 sur la 2.0 et qu(il finit quand il est activé par, sur certains sites, bloquer le fonctionnement d'apache : je suis obligé de faire un
/etc/init.d/apache2 restart pour débloquer le serveur.

Et encore en plus tu te permet de remettre un
<necessite id="SPIP" version='[1.925;]' />
qui est faux, à la place de
<necessite id="SPIP" version='[;1.925)' />

Oups, ça, c'était pas voulu.
En même temps, c'est pas pris en compte par SPIP 1.9, donc, ça change rien.

Tu ferais mieux de mettre à jour tes sites et tes plugins plutôt que de venir casser des versions stables.

Comme dit plus haut, ARv3 plante le serveur.

En l'occurence cette version est figée, et était fonctionnelle avec la 1.9.2 avec laquelle tu as potentiellement généré des plantages maintenant.

D'après http://trac.rezo.net/trac/spip/changeset/12890 "php ne distinguant pas la casse des noms de fonction", où est le plantage potentiel ?
Par contre, http://trac.rezo.net/trac/spip/changeset/12890 a planté plusieurs plugins qui avaient des majuscules dans leur pipeline déclarés dans plugin.xml. Bug ?

Au cas où tu ne serais pas au courant, il y a maintenant
Connexion · GitLab

Merci de revert ce commit rapidement et intégralement

OK, c'est fait.

PS : Au passage, tu n'as pas répondu sur les bug de (EVENEMENTS){id_mot}...

Jacques

Cédric

PS : t'as jamais apris à répondre à un mail sur usenet ?
http://www.usenet-fr.net/fur/usenet/repondre-sur-usenet.html
--
RealET

Le 21 oct. 08 à 11:06, RealET a écrit :

* cedric.morin@yterium.com tapuscrivait, le 21/10/2008 10:43:

En quoi as-tu besoin de toucher au code d'un plugin qui n'est pas compatible 2.0 sous pretexte de problemes avec la 2.0 ?

Parce que j'ai essayé de mettre accès restreint 3 sur la 2.0 et qu(il finit quand il est activé par, sur certains sites, bloquer le fonctionnement d'apache : je suis obligé de faire un
/etc/init.d/apache2 restart pour débloquer le serveur.

Et bien il faut trouver pouquoi !

Et encore en plus tu te permet de remettre un
<necessite id="SPIP" version='[1.925;]' />
qui est faux, à la place de
<necessite id="SPIP" version='[;1.925)' />

Oups, ça, c'était pas voulu.
En même temps, c'est pas pris en compte par SPIP 1.9, donc, ça change rien.

Si, cela change que ce plugin ne doit pas etre utilisé avec Spip 2, puisqu'il n'est pas compatible avec
Je n'ai pas envie de faire le support sur des installations bancales et erronées

Tu ferais mieux de mettre à jour tes sites et tes plugins plutôt que de venir casser des versions stables.

Comme dit plus haut, ARv3 plante le serveur.

En l'occurence cette version est figée, et était fonctionnelle avec la 1.9.2 avec laquelle tu as potentiellement généré des plantages maintenant.

D'après http://trac.rezo.net/trac/spip/changeset/12890 "php ne distinguant pas la casse des noms de fonction", où est le plantage potentiel ?
Par contre, http://trac.rezo.net/trac/spip/changeset/12890 a planté plusieurs plugins qui avaient des majuscules dans leur pipeline déclarés dans plugin.xml. Bug ?

Non, c'est volontaire, car il etait possible de déclarer 2 pipelines dont le nom ne differait que par la casse, et cela provoquait une erreur fatale.
J'ai préféré boucher cette faille.
Cédric