Changeset 25560 in spip-zone

Timestamp:

Dec 27, 2008, 8:10:26 PM (12 years ago)

Author:

marcimat@…

Message:

Utiliser un identifiant digne de ce nom pour les champs extras créés !

Location:

_plugins_/_dev_/champs_extras

Files:

• core/inc/cextras.php (modified) (3 diffs)
• extensions/interface/action/iextras.php (modified) (2 diffs)
• extensions/interface/exec/iextras_edit.php (modified) (1 diff)
• extensions/interface/formulaires/editer_champ_extra.html (modified) (1 diff)
• extensions/interface/formulaires/editer_champ_extra.php (modified) (8 diffs)
• extensions/interface/inc/iextras.php (modified) (1 diff)
• extensions/interface/prive/contenu/champs_extras.html (modified) (1 diff)

_plugins_/_dev_/champs_extras/core/inc/cextras.php

@@ -8 +8 @@
         var $type = 'textarea'; // type (input/textarea)
         var $sql = ''; // declaration sql (text NOT NULL DEFAULT '')
+        
+        var $_id = ''; // identifiant de ce champ extra
         
         // constructeur
@@ -26 +28 @@
                         }
                 }
+                // calculer l'id du champ extra
+                $this->make_id();
         }
         
@@ -36 +40 @@
         }
         
-        // transformer en tableau PHP les variable de la classe.
+        // creer l'id du champ extra :
+        function make_id(){
+                // creer un hash
+                $hash = array();
+                foreach ($this as $cle=>$val) {
+                        if ($cle[0]!=='_') {
+                                $hash[] = $val;
+                        }
+                }
+                $this->_id = substr(md5(serialize($hash)),0,6);
+        }
+        
+        // determiner un identifiant
+        function get_id(){
+                if (!$this->_id) $this->make_id();
+                return $this->_id;
+        }
+        
+        // transformer en tableau PHP les variable publiques de la classe.
         function toArray(){
                 $extra = array();
                 foreach ($this as $cle=>$val) {
-                        $extra[$cle] = $val;
+                        if ($cle[0]!=='_') {
+                                $extra[$cle] = $val;
+                        }
                 }
+                $extra['id_extra'] = $this->get_id();
                 return $extra;
         }

_plugins_/_dev_/champs_extras/extensions/interface/action/iextras.php

@@ -15 +15 @@
         }
         
-        list($arg, $id) = explode ('/', $arg);
+        list($arg, $id_extra) = explode ('/', $arg);
         
         // actions possibles
@@ -26 +26 @@
         
         // cas de suppression
-        if ($arg == 'supprimer_extra'){
+        if ($id_extra and ($arg == 'supprimer_extra')){
                 include_spip('inc/iextras');
                 $extras = iextras_get_extras();
-                if ($id = intval($id)) {
-                        // $id a 1 de plus
-                        $extra = $extras[--$id];
-                        unset($extras[$id]);
-                        iextras_set_extras($extras);
-                        
-                        extras_log("Suppression d'un champ par auteur n°".$GLOBALS['auteur_session']['id_auteur'],true);
-                        extras_log($extra, true);
-                        $table = table_objet_sql($extra->table);
-                        sql_alter("TABLE $table DROP ".$extra->champ);
+                foreach($extras as $i=>$e) {
+                        if ($e->get_id() == $id_extra) {
+                                extras_log("Suppression d'un champ par auteur ".$GLOBALS['auteur_session']['id_auteur'],true);
+                                extras_log($extra, true);
+                                
+                                $table = table_objet_sql($extra->table);
+                                sql_alter("TABLE $table DROP ".$extra->champ);
+                                
+                                unset($extras[$i]);
+                                iextras_set_extras($extras);
+                                break;
+                        }
                 }
         }

_plugins_/_dev_/champs_extras/extensions/interface/exec/iextras_edit.php

@@ -39 +39 @@
         
         // contenu
-        $id_extra = intval(_request('id_extra'));
+        $id_extra = _request('id_extra');
         $id_extra = $id_extra ? $id_extra : 'new' ;
         echo recuperer_fond('prive/editer/champs_extras', array(
-                'id_extra'=>$id_extra,
-                'titre'=>$id_extra=='new' ? _T('iextras:info_nouveau_champ_extra') : _T('iextras:info_modifier_champ_extra'),
-                'redirect'=>generer_url_ecrire("iextras"),
-                'icone_retour'=>icone_inline(_T('icone_retour'), generer_url_ecrire('iextras'), find_in_path("images/iextras-24.png"), "rien.gif",$GLOBALS['spip_lang_left']),
+                'id_extra' => $id_extra,
+                'titre' => $id_extra=='new' ? _T('iextras:info_nouveau_champ_extra') : _T('iextras:info_modifier_champ_extra'),
+                'redirect' => generer_url_ecrire("iextras"),
+                'icone_retour' => icone_inline(_T('icone_retour'), generer_url_ecrire('iextras'), find_in_path("images/iextras-24.png"), "rien.gif",$GLOBALS['spip_lang_left']),
                 ));
 

_plugins_/_dev_/champs_extras/extensions/interface/formulaires/editer_champ_extra.html

@@ -85 +85 @@
                 [(#REM) ajouter les saisies supplementaires : extra et autre, a cet endroit ]
                 <!--extra-->
-                <p class='boutons'><input type='submit' class='submit' value='[(#ENV{id_extra}|intval|?{<:bouton_modifier:>,<:bouton_ajouter:>})]' /></p>
+                <p class='boutons'><input type='submit' class='submit' value='[(#ENV{new}|?{<:bouton_ajouter:>,<:bouton_modifier:>})]' /></p>
         </div></form>
         </BOUCLE_si_editable>

_plugins_/_dev_/champs_extras/extensions/interface/formulaires/editer_champ_extra.php

@@ -5 +5 @@
 
 function formulaires_editer_champ_extra_charger_dist($id_extra='new', $redirect=''){
+        // nouveau ?
+        $new = ($id_extra == 'new') ? ' ': '';
+        
         // valeur par defaut
         $valeurs = array(
@@ -13 +16 @@
                 'sql' => "text NOT NULL DEFAULT ''",
                 'id_extra' => $id_extra,
-                'new' => intval($id_extra)?'':' ',
+                'new' => $new,
                 'redirect' => $redirect,
         );
@@ -19 +22 @@
         // si un extra est demande (pour edition)
         // remplir les valeurs avec infos de celui-ci
-        if (intval($id_extra)) {
+        if (!$new) {
                 $extras = iextras_get_extras();
-                // $id_extra = 1, mais l'entree reelle est 0 dans le tableau
-                if ($extra = $extras[--$id_extra]) {
-                        $valeurs = array_merge($valeurs, $extra->toArray());
+                foreach($extras as $extra) {
+                        if ($extra->get_id() == $id_extra) {
+                                $valeurs = array_merge($valeurs, $extra->toArray());
+                                break;
+                        }
                 }
         }
@@ -32 +37 @@
 function formulaires_editer_champ_extra_verifier_dist($id_extra='new', $redirect=''){
         $erreurs = array();
+        
+        // nouveau ?
+        $new = ($id_extra == 'new') ? ' ': '';  
         
         // recuperer les valeurs postees
@@ -54 +62 @@
         // n'existe pas deja sur la meme table
         $extras = iextras_get_extras();
-        if (!intval($id_extra) 
-        // $id_extra = 1, mais l'entree reelle est 0 dans le tableau
-        OR ($extras[--$id_extra]->champ !== $champ)) { 
-                foreach ($extras as $i=>$e) {
-                        if (($i !== $id_extra)
-                        and ($e->champ == $champ) 
-                        and ($e->table == $extra['table'])) {
-                                $erreurs['champ'] = _T('iextras:champ_deja_existant');  
+        foreach ($extras as $e) {
+                if ($new OR ($e->get_id() !== $id_extra)) {
+                        if (($e->champ == $champ) and ($e->table == $extra['table'])) {
+                                $erreurs['champ'] = _T('iextras:champ_deja_existant');
+                                break;
                         }
                 }
         }
+
 
         return $erreurs;
@@ -71 +77 @@
 
 function formulaires_editer_champ_extra_traiter_dist($id_extra='new', $redirect=''){
-        
+        // nouveau ?
+        $new = ($id_extra == 'new') ? ' ': '';
+                
         // recuperer les valeurs postees
         $extra = iextras_post_formulaire();
@@ -77 +85 @@
         // recreer le tableau de stockage des extras
         $extras = iextras_get_extras();
-        $new = false;
-        if (intval($id_extra)) {
-                // $id_extra = 1, mais l'entree reelle est 0 dans le tableau
-                $extras[--$id_extra] = new ChampExtra($extra);
+
+        // ajout du champ ou modification du champ extra de meme id.
+        if ($new) {
+                $extras[] = new ChampExtra($extra);
         } else {
-                $extras[] = new ChampExtra($extra);
-                $new = true;
+                foreach($extras as $i=>$e) {
+                        if ($e->get_id() == $id_extra) {
+                                $extras[$i] = new ChampExtra($extra);
+                                break;
+                        }
+                }               
         }
+
         // l'enregistrer
         iextras_set_extras($extras);
@@ -92 +105 @@
                 include_spip('base/create');
                 $table = table_objet_sql($extra['table']);
-                extras_log("Creation d'un nouveau champ par auteur n°".$GLOBALS['auteur_session']['id_auteur'],true);
+                extras_log("Creation d'un nouveau champ par auteur ".$GLOBALS['auteur_session']['id_auteur'],true);
                 extras_log($extra, true);
                 maj_tables($table);

_plugins_/_dev_/champs_extras/extensions/interface/inc/iextras.php

@@ -22 +22 @@
         $extras = iextras_get_extras();
         $tables = array();
-        foreach($extras as $i=>$e) {
+        foreach($extras as $e) {
                 if (!isset($tables[$e->table])) {
                         $tables[$e->table] = array();
                 }
-                $tables[$e->table][$i] = $e->toArray();
+                $tables[$e->table][] = $e->toArray();
         }
         return $tables;

_plugins_/_dev_/champs_extras/extensions/interface/prive/contenu/champs_extras.html

@@ -8 +8 @@
                         <li>
                                 [(#VALEUR|table_valeur{champ})]
-                                | <a href="[(#URL_ECRIRE{iextras_edit}|parametre_url{id_extra,#CLE|plus{1}})]"><:iextras:action_modifier:></a>
-                                | <a href="[(#URL_ACTION_AUTEUR{iextras, [supprimer_extra/(#CLE|plus{1})], #SELF})]" class="supprimer"><:iextras:action_supprimer:></a>
+                                | <a href="[(#URL_ECRIRE{iextras_edit}|parametre_url{id_extra,#VALEUR|table_valeur{id_extra}})]"><:iextras:action_modifier:></a>
+                                | <a href="[(#URL_ACTION_AUTEUR{iextras, [supprimer_extra/(#VALEUR|table_valeur{id_extra})], #SELF})]" class="supprimer"><:iextras:action_supprimer:></a>
                         </li>
                 </BOUCLE_extras>