[SPIP Zone] [Spip-zone-commit] r15745 - in /_plugins_/_stable_/forms/forms_et_tables_1_9_1: forms_filtres.php forms_fonctions.php plugin.xml

Pardon ?
Qu'est ce que ce (tres) vieux code vient faire ici ?
Cedric

superyms@yahoo.fr a écrit :

Author: superyms@yahoo.fr
Date: Thu Oct 4 08:04:05 2007
New Revision: 15745

Log:
- Ajout fonctions block dépliant pour Forms & Tables (s'inspirant du plugin derebloc).
* insertion de l'appel javascript dans forms_filtres.php (pour l'insert_header)
* insertion des fonctions du block dépliant dans forms_fonctions.php
* changement numéro version 0.302 e tajout de mon nom aux auteurs dans plugin.xml

Modified:
    _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_filtres.php
    _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_fonctions.php
    _plugins_/_stable_/forms/forms_et_tables_1_9_1/plugin.xml

Modified: _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_filtres.php

--- _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_filtres.php (original)
+++ _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_filtres.php Thu Oct 4 08:04:05 2007
@@ -22,9 +22,43 @@
         $flux .= "<link rel='stylesheet' href='".find_in_path('donnees_tous.css')."' type='text/css' media='all' />\n";
         $flux .= "<link rel='stylesheet' href='"._DIR_PLUGIN_FORMS."img_pack/date_picker.css' type='text/css' media='all' />\n";
         $flux .= "<link rel='stylesheet' href='"._DIR_PLUGIN_FORMS."img_pack/jtip.css' type='text/css' media='all' />\n";
+/*
++--------------------------------------------+
+| derebloc v. 0.3 - 17/04/07
++--------------------------------------------+
+| plugin spip 1.9.2
++--------------------------------------------+
+| H. AROUX . Scoty . koakidi.com
+| Script certifié KOAK2.0 strict, mais si !
++--------------------------------------------+
+| appel layer.js
+| insere dans le <head> des squelettes via balise spip #INSERT_HEAD
++--------------------------------------------+
+| Superyms (Mehdi Cherifi) Net Studio
+| Ajouter block depliant pour Forms & Tables
++--------------------------------------------+
+*/
+ $flux .= "\n".'<script type="text/javascript" src="'._DIR_JAVASCRIPT.'layer.js"></script>'."\n";
       }
       else $flux .= "<link rel='stylesheet' href='".generer_url_public('forms_styles.css')."' type='text/css' media='all' />\n";
+/*
++--------------------------------------------+
+| derebloc v. 0.3 - 17/04/07
++--------------------------------------------+
+| plugin spip 1.9.2
++--------------------------------------------+
+| H. AROUX . Scoty . koakidi.com
+| Script certifié KOAK2.0 strict, mais si !
++--------------------------------------------+
+| appel layer.js
+| insere dans le <head> des squelettes via balise spip #INSERT_HEAD
++--------------------------------------------+
+| Superyms (Mehdi Cherifi) Net Studio
+| Ajouter block depliant pour Forms & Tables
++--------------------------------------------+
+*/
+ $flux .= "\n".'<script type="text/javascript" src="'._DIR_JAVASCRIPT.'layer.js"></script>'."\n";
     }
     return $flux;
   }

Modified: _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_fonctions.php

--- _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_fonctions.php (original)
+++ _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_fonctions.php Thu Oct 4 08:04:05 2007
@@ -308,4 +308,49 @@
   function forms_ajoute_styles($texte){
     return pipeline('forms_ajoute_styles',$texte);
   }
+ /*
++--------------------------------------------+
+| derebloc v. 0.3 - 17/04/07
++--------------------------------------------+
+| plugin spip 1.9.2
++--------------------------------------------+
+| H. AROUX . Scoty . koakidi.com
+| Script certifié KOAK2.0 strict, mais si !
++--------------------------------------------+
+| Definition des 3 balises pour block depliant.
+| pipeline .. mes_fonctions
++--------------------------------------------+
+| Superyms (Mehdi Cherifi) Net Studio
+| Ajouter block depliant pour Forms & Tables
++--------------------------------------------+
+*/
+
+include_spip('inc/minipres');
+include_spip('inc/layer');
+
+function calculer_id_block($a='') {
+ static $id_block = 0;
+ $id_block++;
+ if($a) { $id_block = $id_block-1; }
+ return $id_block;
+}
+
+function balise_BOUTON_BLOCK($p) {
+ $p->code = "bouton_block_invisible('bout'.calculer_id_block())";
+ $p->interdire_scripts = false;
+ return $p;
+}
+
+function balise_DEBUT_BLOCK($p) {
+ $p->code = "debut_block_invisible('bout'.calculer_id_block('1'))";
+ $p->interdire_scripts = false;
+ return $p;
+}
+
+function balise_FIN_BLOCK($p) {
+ $p->code = "fin_block()";
+ $p->interdire_scripts = false;
+ return $p;
+}
?>

Modified: _plugins_/_stable_/forms/forms_et_tables_1_9_1/plugin.xml

--- _plugins_/_stable_/forms/forms_et_tables_1_9_1/plugin.xml (original)
+++ _plugins_/_stable_/forms/forms_et_tables_1_9_1/plugin.xml Thu Oct 4 08:04:05 2007
@@ -5,13 +5,13 @@
   <icon>img_pack/form-24.png</icon>
   <auteur> <!-- Auteur du plugin -->
     <multi>[ar]&#1575;&#1606;&#1591;&#1608;&#1575;&#1606; &#1576;&#1578;&#1585;&#1608;&#1548; &#1587;&#1610;&#1583;&#1585;&#1610;&#1603; &#1605;&#1608;&#1585;&#1575;&#1606;&#1548; &#1585;&#1610;&#1606;&#1575;&#1578;&#1608;
- [fr]Antoine Pitrou, Cedric Morin, Renato Formato
- [it]Antoine Pitrou, Cedric Morin, Renato Formato
- [es]Antoine Pitrou, Cedric Morin, Renato Formato
+ [fr]Antoine Pitrou, Cedric Morin, Renato Formato, Mehdi Cherifi
+ [it]Antoine Pitrou, Cedric Morin, Renato Formato, Mehdi Cherifi
+ [es]Antoine Pitrou, Cedric Morin, Renato Formato, Mehdi Cherifi </multi>
   </auteur>
   <version>
- 0.301
+ 0.302
   </version>
   <etat>
   stable

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

Sur le fond :
- si la généralisation du formulaire est un squelette c'est pour que chacun puisse le personaliser selon ses besoins specifiques. En cela, tu n'as pas à imposer brutalement que les formulaires sont repliés à tout le monde, cela casse l'apparence de tous les sites qui utilisent le plugin.
- le code que tu as utilisé est vraiment obsolète. Une débauche de balise, d'inclusion javascript etc ... alors que tu peux faire la meme chose en une ligne de javascript ajoutée dans le head de tes pages, sans meme toucher à forms.html

Sur la forme :
- mettre des placards de crédits et t'ajouter comme auteur du plugin pour un vieux morceau de code recopié autre part, lui même fortement issu du code de spip 1.8 n'est pas d'une élégance folle, et cela dévalorise pour le moins le travail des autres contributeurs qui en ont fait bien plus sans meme jamais prendre le soin de s'ajouter aux auteurs.

En conséquence [15747] annule tes deux commits 15745 et 15746
Cedric

cedric.morin@yterium.com a écrit :

Pardon ?
Qu'est ce que ce (tres) vieux code vient faire ici ?
Cedric

superyms@yahoo.fr a écrit :
  

Author: superyms@yahoo.fr
Date: Thu Oct 4 08:04:05 2007
New Revision: 15745

Log:
- Ajout fonctions block dépliant pour Forms & Tables (s'inspirant du plugin derebloc).
* insertion de l'appel javascript dans forms_filtres.php (pour l'insert_header)
* insertion des fonctions du block dépliant dans forms_fonctions.php
* changement numéro version 0.302 e tajout de mon nom aux auteurs dans plugin.xml

Modified:
    _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_filtres.php
    _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_fonctions.php
    _plugins_/_stable_/forms/forms_et_tables_1_9_1/plugin.xml

Modified: _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_filtres.php

--- _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_filtres.php (original)
+++ _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_filtres.php Thu Oct 4 08:04:05 2007
@@ -22,9 +22,43 @@
         $flux .= "<link rel='stylesheet' href='".find_in_path('donnees_tous.css')."' type='text/css' media='all' />\n";
         $flux .= "<link rel='stylesheet' href='"._DIR_PLUGIN_FORMS."img_pack/date_picker.css' type='text/css' media='all' />\n";
         $flux .= "<link rel='stylesheet' href='"._DIR_PLUGIN_FORMS."img_pack/jtip.css' type='text/css' media='all' />\n";
+/*
++--------------------------------------------+
+| derebloc v. 0.3 - 17/04/07
++--------------------------------------------+
+| plugin spip 1.9.2
++--------------------------------------------+
+| H. AROUX . Scoty . koakidi.com
+| Script certifié KOAK2.0 strict, mais si !
++--------------------------------------------+
+| appel layer.js
+| insere dans le <head> des squelettes via balise spip #INSERT_HEAD
++--------------------------------------------+
+| Superyms (Mehdi Cherifi) Net Studio
+| Ajouter block depliant pour Forms & Tables
++--------------------------------------------+
+*/
+ $flux .= "\n".'<script type="text/javascript" src="'._DIR_JAVASCRIPT.'layer.js"></script>'."\n";
       }
       else $flux .= "<link rel='stylesheet' href='".generer_url_public('forms_styles.css')."' type='text/css' media='all' />\n";
+/*
++--------------------------------------------+
+| derebloc v. 0.3 - 17/04/07
++--------------------------------------------+
+| plugin spip 1.9.2
++--------------------------------------------+
+| H. AROUX . Scoty . koakidi.com
+| Script certifié KOAK2.0 strict, mais si !
++--------------------------------------------+
+| appel layer.js
+| insere dans le <head> des squelettes via balise spip #INSERT_HEAD
++--------------------------------------------+
+| Superyms (Mehdi Cherifi) Net Studio
+| Ajouter block depliant pour Forms & Tables
++--------------------------------------------+
+*/
+ $flux .= "\n".'<script type="text/javascript" src="'._DIR_JAVASCRIPT.'layer.js"></script>'."\n";
     }
     return $flux;
   }

Modified: _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_fonctions.php

--- _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_fonctions.php (original)
+++ _plugins_/_stable_/forms/forms_et_tables_1_9_1/forms_fonctions.php Thu Oct 4 08:04:05 2007
@@ -308,4 +308,49 @@
   function forms_ajoute_styles($texte){
     return pipeline('forms_ajoute_styles',$texte);
   }
+ /*
++--------------------------------------------+
+| derebloc v. 0.3 - 17/04/07
++--------------------------------------------+
+| plugin spip 1.9.2
++--------------------------------------------+
+| H. AROUX . Scoty . koakidi.com
+| Script certifié KOAK2.0 strict, mais si !
++--------------------------------------------+
+| Definition des 3 balises pour block depliant.
+| pipeline .. mes_fonctions
++--------------------------------------------+
+| Superyms (Mehdi Cherifi) Net Studio
+| Ajouter block depliant pour Forms & Tables
++--------------------------------------------+
+*/
+
+include_spip('inc/minipres');
+include_spip('inc/layer');
+
+function calculer_id_block($a='') {
+ static $id_block = 0;
+ $id_block++;
+ if($a) { $id_block = $id_block-1; }
+ return $id_block;
+}
+
+function balise_BOUTON_BLOCK($p) {
+ $p->code = "bouton_block_invisible('bout'.calculer_id_block())";
+ $p->interdire_scripts = false;
+ return $p;
+}
+
+function balise_DEBUT_BLOCK($p) {
+ $p->code = "debut_block_invisible('bout'.calculer_id_block('1'))";
+ $p->interdire_scripts = false;
+ return $p;
+}
+
+function balise_FIN_BLOCK($p) {
+ $p->code = "fin_block()";
+ $p->interdire_scripts = false;
+ return $p;
+}
?>

Modified: _plugins_/_stable_/forms/forms_et_tables_1_9_1/plugin.xml

--- _plugins_/_stable_/forms/forms_et_tables_1_9_1/plugin.xml (original)
+++ _plugins_/_stable_/forms/forms_et_tables_1_9_1/plugin.xml Thu Oct 4 08:04:05 2007
@@ -5,13 +5,13 @@
   <icon>img_pack/form-24.png</icon>
   <auteur> <!-- Auteur du plugin -->
     <multi>[ar]&#1575;&#1606;&#1591;&#1608;&#1575;&#1606; &#1576;&#1578;&#1585;&#1608;&#1548; &#1587;&#1610;&#1583;&#1585;&#1610;&#1603; &#1605;&#1608;&#1585;&#1575;&#1606;&#1548; &#1585;&#1610;&#1606;&#1575;&#1578;&#1608;
- [fr]Antoine Pitrou, Cedric Morin, Renato Formato
- [it]Antoine Pitrou, Cedric Morin, Renato Formato
- [es]Antoine Pitrou, Cedric Morin, Renato Formato
+ [fr]Antoine Pitrou, Cedric Morin, Renato Formato, Mehdi Cherifi
+ [it]Antoine Pitrou, Cedric Morin, Renato Formato, Mehdi Cherifi
+ [es]Antoine Pitrou, Cedric Morin, Renato Formato, Mehdi Cherifi </multi>
   </auteur>
   <version>
- 0.301
+ 0.302
   </version>
   <etat>
   stable

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