[SPIP Zone] [Spip-zone-commit] r22300 - in /_plugins_: _stable_/acces_restreint/acces_restreint_pipelines.php _stable_/acces_restreint/fond/inscription2_acces_restreint.html _stable_/acces_restreint/plugin.xml _test_/inscription2/inscription2_193/fond

S'lt

Tu peux mettre tout ca dans inscription 3, et faire l'inclusion sur un
simple if (defined('_DIR_PLUGIN_ACCES_RESTREINT')){} )

Ce n'est en aucun cas un manque de respect. Si tel a été pris je te
présente mes excuses.

Pour ma part, ce n'est pas à I2 de gerer le code d'acces restreint.
Si acces restreint souhaite apparaitre dans le CFG de I2 c'est à lui
de s'annoncer et pas le contraire. I2 n'a pas à gerer des noisettes
d'un autre plugin comme c'etait le cas.

Là on risque de se rejetter la faute sans fin.

Dans ce cas la solution simple est tout simplement supprimer la
noissettes acces restreint d'I2 mais cela risque de ne pas plaire non
plus.

km

2008/8/31 cedric.morin@yterium.com <cedric.morin@yterium.com>:

je ne vois pas en quoi le plugin acces restreint devrait avoir du code
specifique à inscription 2, a tartempion 25 ou à machin 15.
Si chacun y va de son morceau, ça ne peut que devenir illisible et
inutilement lourd.

En plus de ça, acces restreint n'est pas un greffon d'inscription 2, il
existait bien avant.
un peu de respect svp.

Cédric

Le 29 août 08 à 10:00, cam.lafit@azerttyu.net a écrit :

Author: cam.lafit@azerttyu.net
Date: Fri Aug 29 10:00:53 2008
New Revision: 22300

Log:
Complete 22286

acces_restreint subit le meme sort que les autres greffons d'Inscription2

Added:

_plugins_/_stable_/acces_restreint/fond/inscription2_acces_restreint.html
     - copied unchanged from r22282,
_plugins_/_test_/inscription2/inscription2_193/fonds/inscription2_acces_restreint.html
Removed:

_plugins_/_test_/inscription2/inscription2_193/fonds/inscription2_acces_restreint.html
Modified:
   _plugins_/_stable_/acces_restreint/acces_restreint_pipelines.php
   _plugins_/_stable_/acces_restreint/plugin.xml

Modified: _plugins_/_stable_/acces_restreint/acces_restreint_pipelines.php

==============================================================================
--- _plugins_/_stable_/acces_restreint/acces_restreint_pipelines.php
(original)
+++ _plugins_/_stable_/acces_restreint/acces_restreint_pipelines.php Fri
Aug 29 10:00:53 2008
@@ -47,5 +47,9 @@
               return $flux;
       }

+ function AccesRestreint_I2_cfg_form($flux){
+ $flux .= recuperer_fond('fond/inscription2_acces_restreint');
+ return $flux;
+ }

?>
\ No newline at end of file

Modified: _plugins_/_stable_/acces_restreint/plugin.xml

==============================================================================
--- _plugins_/_stable_/acces_restreint/plugin.xml (original)
+++ _plugins_/_stable_/acces_restreint/plugin.xml Fri Aug 29 10:00:53 2008
@@ -57,6 +57,11 @@
               <inclure>acces_restreint_pipelines.php</inclure>
       </pipeline>
       <pipeline>
+ <nom>I2_cfg_form</nom>
+ <action>I2_cfg_form</action>
+ <inclure>acces_restreint_pipelines.php</inclure>
+ </pipeline>
+ <pipeline>
               <nom>AccesRestreint_liste_zones_autorisees</nom>
               <action>liste_zones_autorisees</action>
               <inclure>inc/acces_restreint.php</inclure>

Removed:
_plugins_/_test_/inscription2/inscription2_193/fonds/inscription2_acces_restreint.html

==============================================================================
---
_plugins_/_test_/inscription2/inscription2_193/fonds/inscription2_acces_restreint.html
(original)
+++
_plugins_/_test_/inscription2/inscription2_193/fonds/inscription2_acces_restreint.html
(removed)
@@ -1,28 +0,0 @@
-#SET{selected,#CONFIG{inscription2/zones,'',''}}
-#SET{checked,#CONFIG{inscription2/accesrestreint,'',''}}
-
-<div>
-<label>
- <input type="checkbox" name="accesrestreint"
onclick="showhide(this, '#zones')"
[(#GET{checked}|?{#CONFIG{plugin/ACCESRESTREINT}|?{checked="checked"}})]
id="accesrestreint" />
- <strong><:inscription2:accesrestreint:></strong>
-</label>
-
-<label for="zones" style="[(#GET{checked}|=={on}|?{display: block;,
display:none;})]">
-<:inscription2:choisir_zones:>
-</label>
-<br />
-
-<B_zones>
-<select name="zones" multiple="multiple" size="[(#TOTAL_BOUCLE|<{6}|?{
[(#TOTAL_BOUCLE|plus{1})] , 5 })]" id="zones">
-<BOUCLE_zones(Spip_zones){tout}>
- <option value="#ID_ZONE"[(#ID_ZONE|in_array{#GET{selected}}|?{
selected="selected"})]>
- #TITRE
- </option>
-</BOUCLE_zones>
-</select>
-</B_zones>
-<//B_zones>
-
-<small><br /><:inscription2:select_listes:><br />
-<a href="#" onclick="$('#zones').find('option').attr('selected',
false);return false;"><:inscription2:deselect_listes:></a> </small><br />
-</div>
\ No newline at end of file

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

--
------------
-----
http://www.azerttyu.net (hébergement)
http://km.azerttyu.net (blog)
cam.lafit@azerttyu.net

Le 1 sept. 08 à 14:06, cam.lafit@azerttyu.net a écrit :

S'lt

Tu peux mettre tout ca dans inscription 3, et faire l'inclusion sur un
simple if (defined('_DIR_PLUGIN_ACCES_RESTREINT')){} )

Ce n'est en aucun cas un manque de respect. Si tel a été pris je te
présente mes excuses.

non non, je ne l'avais pas pris comme tel ...

Pour ma part, ce n'est pas à I2 de gerer le code d'acces restreint.
Si acces restreint souhaite apparaitre dans le CFG de I2 c'est à lui
de s'annoncer et pas le contraire. I2 n'a pas à gerer des noisettes
d'un autre plugin comme c'etait le cas.

Jusqu'à preuve du contraire, c'est I2 qui utilise les fonctionnalités d'accès restreint là, pas le contraire...
Je vois meme que le formulaire de cfg comporte des chaines de langues de I2 :

<:inscription2:choisir_zones:>

et tout cela est stocke in fine dans l'espce cfg de I2.
Si ce code doit être quelque part, c'est bien dans I2.
Il n'a en tout cas strictement rien à faire dans acces restreint.

Là on risque de se rejetter la faute sans fin.

Ca n'est pas une faute.
Mais enlever ce code ne prive accès restreint d'aucune fonctionnalité.
Si c'est le cas aussi pour I2, alors ce code ne sert à rien, et bon débaras.
Sinon, il a tout à faire dans I2.

Cédric