Changeset 95836 in spip-zone

Timestamp:

Mar 8, 2016, 12:08:54 AM (5 years ago)

Author:

rastapopoulos@…

Message:

Oups au commit précédent : il fallait aussi modifier les rangs lors de la suppression d'un contenu, et aussi pour la génération du dernier rang lors de l'ajout d'un contenu ! (plus la même méthode)

Location:

_plugins_/selections_editoriales/trunk

Files:

• action/editer_selections_contenu.php (modified) (1 diff)
• action/supprimer_selection.php (modified) (2 diffs)
• formulaires/editer_selections_contenu.php (modified) (4 diffs)
• paquet.xml (modified) (1 diff)

_plugins_/selections_editoriales/trunk/action/editer_selections_contenu.php

@@ -9 +9 @@
         
         if ($id_selections_contenu > 0) {
+                // On récupère le rang et la sélection du truc à supprimer
+                $contenu = sql_fetsel('rang, id_selection', 'spip_selections_contenus', 'id_selections_contenu = '.$id_selections_contenu);
+                
+                // On supprime
                 $ok = sql_delete(
                         'spip_selections_contenus',
                         'id_selections_contenu = '.$id_selections_contenu
                 );
+                
+                // Si c'est bon, il faut décaler le rang de tout ce qui est après
+                if ($ok) {
+                        sql_update(
+                                'spip_selections_contenus',
+                                array('rang' => 'rang - 1'),
+                                array(
+                                        'id_selection = '.$contenu['id_selection'],
+                                        'rang > '.$contenu['rang'],
+                                )
+                        );
+                }
         }
         

_plugins_/selections_editoriales/trunk/action/supprimer_selection.php

@@ -5 +5 @@
 
 function action_supprimer_selection_dist(){
-        
         $securiser_action = charger_fonction('securiser_action', 'inc');
         $arg = $securiser_action();
@@ -14 +13 @@
                 selection_supprimer($id_selection);
         }
-        
 }

_plugins_/selections_editoriales/trunk/formulaires/editer_selections_contenu.php

@@ -99 +99 @@
                 $url = _request('url');
                 
-                // On cherche la sélection parente
-                if ($id_contenu = intval($id_selections_contenu)){
-                        $id_selection = sql_getfetsel('id_selection', 'spip_selections_contenus', 'id_selections_contenu = '.$id_contenu);
-                }
-                
-                // On cherche le numéro à lui mettre pour le placer suivant son rang
-                $rang = intval(sql_countsel('spip_selections_contenus', 'id_selection = '.intval($id_selection)));
-                $rang = ($id_contenu ? $rang : ($rang+1)) * 10; // Si c'est une modif on garde le même rang
-                
                 // Si on ne trouve pas d'objet SPIP
                 if (!$infos = traiter_lien_implicite($url, '', 'tout')) {
@@ -125 +116 @@
                 
                 // On  génère le titre
-                set_request('titre', "$rang. $titre");
+                set_request('titre', $titre);
         }
         
@@ -154 +145 @@
  */
 function formulaires_editer_selections_contenu_traiter_dist($id_selections_contenu='new', $id_selection=0, $retour='', $config_fonc='', $row=array(), $hidden=''){
-        // Si création, on met en mémoire la sélection parente
-        if (intval($id_selections_contenu) <= 0 and intval($id_selection) > 0){
-                set_request('id_selection', intval($id_selection));
+        $id_selection = intval($id_selection);
+        
+        // Si création, on met en mémoire la sélection parente et on génère le dernier rang
+        if (intval($id_selections_contenu) <= 0 and $id_selection > 0){
+                set_request('id_selection', $id_selection);
+                
+                $dernier_rang = sql_getfetsel('rang', 'spip_selections_contenus', 'id_selection = '.$id_selection, '', 'rang desc', '0,1');
+                set_request('rang', $dernier_rang + 1);
         }
         
@@ -162 +158 @@
         $retours = formulaires_editer_objet_traiter('selections_contenu',$id_selections_contenu,'',0,$retour,$config_fonc,$row,$hidden);
         
-        // On va chercher la vrai sélection si on a bien un contenu
+        // On va chercher la vraie sélection si on a bien un contenu
         if ($id_contenu = intval($retours['id_selections_contenu'])) {
                 $id_selection = intval(sql_getfetsel('id_selection', 'spip_selections_contenus', 'id_selections_contenu = '.$id_contenu));

_plugins_/selections_editoriales/trunk/paquet.xml

@@ -2 +2 @@
         prefix="selections_editoriales"
         categorie="navigation"
-        version="1.4.0"
+        version="1.4.1"
         etat="stable"
         compatibilite="[3.0.0;3.1.*]"