[SPIP Zone] [Spip-zone-commit] r17896 - /_plugins_/_stable_/crayons/tetecrayons.php

Non ce n'est pas bon, car ça ralentit tout le monde, à chaque hit,
pour gérer un cas exceptionnel. Pour gérer ce que tu dis il *faut*
indiquer dans mes_options :

function analyse_droits_rapide() {
return true;
}

c'est ce qu'indique la ligne de code juste en-dessous.

On Jan 6, 2008 5:32 PM, <cam.lafit@azerttyu.net> wrote:

Author: cam.lafit@azerttyu.net
Date: Sun Jan 6 16:32:49 2008
New Revision: 17896

Log:
Il semblerait qu'il y ai une petite coquille dans le code.

Il faut appeller explicitement autoriser.php sinon autorité n'est pas appellé.
Ce qui empechait la possibilité d'avoir le mode d'édition libre si le mode wiki est activé dans autorité.

Modified:
    _plugins_/_stable_/crayons/tetecrayons.php

Modified: _plugins_/_stable_/crayons/tetecrayons.php

--- _plugins_/_stable_/crayons/tetecrayons.php (original)
+++ _plugins_/_stable_/crayons/tetecrayons.php Sun Jan 6 16:32:49 2008
@@ -40,6 +41,8 @@
// Le pipeline affichage_final, execute a chaque hit sur toute la page
function &Crayons_affichage_final(&$page) {

+ include_spip('inc/autoriser');
+
     // ne pas se fatiguer si le visiteur n'a aucun droit
     if (!(function_exists('analyse_droits_rapide')?analyse_droits_rapide():analyse_droits_rapide_dist()))
         return $page;

-- Fil

S'lt

Je revert demain matin. Mais je me demande l'intérêt du truc sur la
partie autorité aussi.

Car si crayon prévoit un test et qu'autorité possède une surcharge
analyse_droits_rapide() mais n'est jamais appelé sans ce
include_spip()

J'ai du mal à voir l'intérêt de faire cette surcharge coté autorité.

Km

revert fait
commit 17904

Car si crayon prévoit un test et qu'autorité possède une surcharge
analyse_droits_rapide() mais n'est jamais appelé sans ce
include_spip()

oui c'est pour ça que je disais qu'il fallait que ce soit dans
mes_options, mais du coup ce n'est pas léger à installer en plugin ;

J'ai du mal à voir l'intérêt de faire cette surcharge coté autorité.

je pense qu'une solution simple serait d'introduire une meta
spécifique pour communiquer aux crayons le fait qu'un visiteur pas
connecté peut avoir des droits. Mais c'est très important en termes de
perfs de pouvoir ne *pas* charger les droits, ni les calculer un par
un, quand on sait à l'avance que le visiteur n'en a aucun

-- Fil

S'lt

Et si tout bêtement, on passe la fonction analyse_droit_rapide dans
mes _options de autorité ?

Ainsi Crayons lui n'est plus impacté. Mais si cela n'a pas été fait
c'est qu'il y a sûrement une raison, mais laquelle ?

Km

Et si tout bêtement, on passe la fonction analyse_droit_rapide dans
mes _options de autorité ?

Oui ça devrait marcher.

Ainsi Crayons lui n'est plus impacté. Mais si cela n'a pas été fait
c'est qu'il y a sûrement une raison, mais laquelle ?

C'est parce qu'Autorité n'a pas de fichier mes_options

-- Fil

S'lt

Bon je teste le bousin et ça ne semble pas encore coller.
Il se trouve que la fonction est testée mais ne semble pas interprétée.

Mais je vais l'avoir :slight_smile:

Km