Changeset 113694 in spip-zone

Timestamp:

Jan 30, 2019, 10:48:29 PM (2 years ago)

Author:

peetdu@…

Message:

Report dernières modifs du trunk

Location:

_plugins_/lim/branches/V2

Files:

• lim_autorisations.php (modified) (2 diffs)
• lim_pipelines.php (modified) (3 diffs)
• paquet.xml (modified) (2 diffs)
• todo.md (copied) (copied from _plugins_/lim/trunk/todo.md)
• todo.txt (deleted)

_plugins_/lim/branches/V2/lim_autorisations.php

@@ -65 +65 @@
 /************* RESTRICTION DANS LES RUBRIQUES *************/
 /**
- * gérer création et modification (en fait publierdans)
+ * gérer création et modification (en fait creerobjetrdans)
  * @pipeline autoriser 
  */
@@ -127 +127 @@
 }
 
-if (!function_exists('autoriser_rubrique_publierdans')) {
-        function autoriser_rubrique_publierdans($faire, $type, $id, $qui, $opt) {
+// if (!function_exists('autoriser_rubrique_publierdans')) {
+//      function autoriser_rubrique_publierdans($faire, $type, $id, $qui, $opt) {
 
-                // Dans LIM l'appel à cette autorisation signifie que forcément $opt est renseigné
-                if (is_array($opt) AND array_key_exists('lim_except_rub',$opt) AND array_key_exists('type',$opt)) {
-                        $type = $opt['type'];
-                        $quelles_rubriques = lire_config("lim_rubriques/$type");
-                        if (!is_null($quelles_rubriques)) {
-                                $rubrique_except = array(0 => $opt['lim_except_rub']);
-                                $quelles_rubriques = array_diff($quelles_rubriques, $opt);
-                                $lim_rub = !in_array($id,$quelles_rubriques);
-                        }
-                        // cas possible : un objet peut avoir été sélectionné dans ?exec=configurer_lim_rubriques, mais aucune restriction activée
-                        else $lim_rub = true;
-                }
-                // ici gestion hors CVT
-                else $lim_rub = true;
+//              // Dans LIM l'appel à cette autorisation signifie que forcément $opt est renseigné
+//              if (is_array($opt) AND array_key_exists('lim_except_rub',$opt) AND array_key_exists('type',$opt)) {
+//                      $type = $opt['type'];
+//                      $quelles_rubriques = lire_config("lim_rubriques/$type");
+//                      if (!is_null($quelles_rubriques)) {
+//                              $rubrique_except = array(0 => $opt['lim_except_rub']);
+//                              $quelles_rubriques = array_diff($quelles_rubriques, $opt);
+//                              $lim_rub = !in_array($id,$quelles_rubriques);
+//                      }
+//                      // cas possible : un objet peut avoir été sélectionné dans ?exec=configurer_lim_rubriques, mais aucune restriction activée
+//                      else $lim_rub = true;
+//              }
+//              // ici gestion hors CVT
+//              else $lim_rub = true;
 
-                return
-                        $lim_rub
-                        AND autoriser_rubrique_publierdans_dist($faire, $type, $id, $qui, $opt);
-        }
-}
+//              return
+//                      $lim_rub
+//                      AND autoriser_rubrique_publierdans_dist($faire, $type, $id, $qui, $opt);
+//      }
+// }

_plugins_/lim/branches/V2/lim_pipelines.php

@@ -60 +60 @@
 **/
 function lim_formulaire_charger($flux) {
-        $form   = $flux['args']['form'];
-        $valid  = strpos($form, 'editer');
-        // si ce n'est pas un formulaire d'édition 
-        // ou si la restriction par rubrique n'a pas été activée, on sort
-        if ($valid === false OR is_null(lire_config('lim_objets'))) {
+        // si ce n'est pas un formulaire d'édition d'un objet ou si la restriction par rubrique n'a pas été activée, on sort.
+        if (strncmp($flux['args']['form'], 'editer_', 7) !== 0 OR is_null(lire_config('lim_objets'))) {
                 return $flux;
         }
 
-        $type                           = substr($form, 7); // 'editer_objet' devient 'objet'
-        $nom_table                      = table_objet_sql($type);
+        $objet = substr($flux['args']['form'], 7); // 'editer_objet' devient 'objet'
+        $nom_table      = table_objet_sql($objet);
         $tableau_tables_lim     = explode(',', lire_config('lim_objets'));
         
         if (in_array($nom_table, $tableau_tables_lim)) {
-                $tab_rubriques_choisies = lim_publierdansrubriques($type);
+                $tab_rubriques_choisies = lim_publierdansrubriques($objet);
                 if (count($tab_rubriques_choisies) == 1) {
                         $id_parent = $flux['data']['id_parent'];
@@ -86 +83 @@
 
 /**
- * Gestion des contenus par rubrique : vérifier si on a le droit de publier l'objet dans cette rubrique
- * en fonction des rubriques décochées dans la page exec=configurer_lim_rubriques
+ * Gestion des contenus par rubrique : 
+ * Impossible de CREER ou DEPLACER un objet dans une rubrique interdite par la configuration choisie dans exec=configurer_lim_rubriques
+ * exception : possibilité de modifier un objet si celui-ci est maintenant dans une rubrique où il est interdit de créer ce type d'objet.
  *
  * @param array $flux
@@ -94 +92 @@
 **/
 function lim_formulaire_verifier($flux) {
-        $form   = $flux['args']['form'];
-        $valid  = strpos($form, 'editer');
-
-        // si ce n'est pas un formulaire d'édition 
-        //ou si la restriction par rubrique n'a pas été activée, on sort
-        if ($valid === false OR is_null(lire_config('lim_objets'))) return $flux;
+        // si ce n'est pas un formulaire d'édition d'un objet ou si la restriction par rubrique n'a pas été activée, on sort.
+        if (strncmp($flux['args']['form'], 'editer_', 7) !== 0 OR is_null(lire_config('lim_objets'))) {
+                return $flux;
+        }
         
-        $type   = substr($form, 7); // 'editer_objet' devient 'objet'
-        $nom_table                      = table_objet_sql($type);
+        $objet = substr($flux['args']['form'], 7); // 'editer_objet' devient 'objet'
+        $nom_table      = table_objet_sql($objet);
         $tableau_tables_lim     = explode(',', lire_config('lim_objets'));
 
         if (in_array($nom_table, $tableau_tables_lim)) {
                 include_spip('inc/autoriser');
-
-                // Si modification : le rédacteur doit pouvoir modifier le contenu d'un objet existant, 
-                // même ci celui-ci est maintenant dans une rubrique où il est interdit de publier cet objet.
+                $faire = 'creer'.$objet.'dans';
+                
                 $id_objet = $flux['args']['args'][0];
                 if (is_numeric($id_objet)) {    // c'est donc une modification, 
 
-                        // récupérer l'id_rubrique actuel (en BdD) de l'objet 
-                        $faire = 'publierdans';
-                        $where = id_table_objet($type).'='.$id_objet;
-                        if ($type == 'rubrique')
-                                $id_rub_en_cours = sql_getfetsel('id_parent', $nom_table, $where);
-                        else
-                                $id_rub_en_cours = sql_getfetsel('id_rubrique', $nom_table, $where);
-                        $opt = array('lim_except_rub' => $id_rub_en_cours, 'type' => $type);
-                        $msg_error = _T('lim:info_deplacer_dans_rubrique_non_autorise');
+                        // récupérer l'id_rubrique actuel de l'objet 
+                        // note : dans l'idéal, il faudrait utiliser le plugin déclarer parent ici
+                        $where = id_table_objet($objet).'='.$id_objet;
+                        switch ($objet) {
+                                case 'rubrique':
+                                        $id_rub_en_cours = sql_getfetsel('id_parent', $nom_table, $where);
+                                        break;
+                                case 'document':
+                                        // rien à faire ici
+                                        break;
+                                default:
+                                        $id_rub_en_cours = sql_getfetsel('id_rubrique', $nom_table, $where);
+                                        break;
+                        }
+
+                        // si c'est un déplacement vers une autre rubrique, on vérifie
+                        if (isset($id_rub_en_cours) and $id_rub_en_cours !=_request('id_parent')) {
+                                if (!autoriser($faire, 'rubrique', _request('id_parent'))) {
+                                        $flux['data']['id_parent'] = _T('lim:info_deplacer_dans_rubrique_non_autorise');
+                                }
+                        }
                 }
                 else { //c'est une création
                         // en fait, cela ne sert à rien...snif...à cause de /echafaudage qui intercepte les créations avant le CVT (?!).
-                        $faire = 'creer'.$type.'dans';
-                        $opt = null;
-                        $msg_error = _T('lim:info_creer_dans_rubrique_non_autorise');
+                        // if (!autoriser($faire, 'rubrique', _request('id_parent'))) {
+                        //      $flux['data']['id_parent'] = _T('lim:info_creer_dans_rubrique_non_autorise');
                 }
-                // mise en berne car il faudrait pourvoir gérer les cas suivants :
-                // 1- cas de la création : voir #122 (juste au dessus)
-                // 2- en l'état avec SPIP, impossible de surcharger deux fois une autorisation. Du coup devient compliqué de gérer aussi le cas des rédacteurs
-                // voir à ce propos : https://www.spip.net/fr_article3517.html et https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc/autoriser.php#L555
-                /*
-                if (!autoriser($faire, 'rubrique', _request('id_parent'),'', $opt)) {
-                        $flux['data']['id_parent'] = $msg_error;
-                }
-                */
         }
+
         return $flux;
 }
 
 /**
+ * Gestion de la desactivation de l'affichage de certain champs dans le formulaire Editer Auteur
  * Inserer le JS qui gére l'affichage ou non des champs dans certains formulaires historiques
- * juste le formulaire Auteur (pour l'instant ?)
+ * juste le formulaire Auteur 
  *
  * @param array $flux

_plugins_/lim/branches/V2/paquet.xml

@@ -2 +2 @@
         prefix="lim"
         categorie="maintenance"
-        version="2.0.0"
+        version="2.0.1"
         etat="stable"
         compatibilite="[3.2.0;3.2.*]"
@@ -18 +18 @@
         <licence>GNU/GPL</licence>
 
+<<<<<<< .working
         <necessite nom="saisies" compatibilite="[1.42.11;[" />
         <necessite nom="medias" compatibilite="[2.20.15;[" />
+=======
+        <necessite nom="saisies"  compatibilite="[1.42.11;[" />
+        <necessite nom="medias"  compatibilite="[2.20.15;[" />
+>>>>>>> .merge-right.r113693
         <necessite nom="php" compatibilite="[5.5.0;[" />