Changeset 115393 in spip-zone

Timestamp:

May 14, 2019, 7:57:11 PM (13 months ago)

Author:

real3t@…

Message:

[Dictionnaires] Ne pas cacher la poussière sous le tapis (dixit Rastapopoulos) : ces fonctions n'étaient *jamais* appelée par les mécanismes d'édition de SPIP parce qu'elles ne portaient pas les bons noms.
Les noms des fonctions ayant changés, les plugins y faisant appel doivent être mis à jour.
Voici la liste des changements de nom :

• insert_dictionnaire -> dictionnaire_inserer
• dictionnaire_set -> dictionnaire_modifier
• instituer_dictionnaire -> dictionnaire_instituer
• insert_definition -> definition_inserer
• definition_set -> definition_modifier
• instituer_definition -> definition_instituer

Location:

_plugins_/dictionnaires/trunk

Files:

• action/editer_definition.php (modified) (5 diffs)
• action/editer_dictionnaire.php (modified) (5 diffs)
• dictionnaires_administrations.php (modified) (2 diffs)
• dictionnaires_pipelines.php (modified) (2 diffs)
• paquet.xml (modified) (2 diffs)

_plugins_/dictionnaires/trunk/action/editer_definition.php

@@ -32 +32 @@
         // si id_definition n'est pas un nombre, c'est une creation
         if (!$id_definition = intval($arg)) {
-                $id_definition = insert_definition();
+                $id_definition = definition_inserer();
         }
 
         // Enregistre l'envoi dans la BD
-        if ($id_definition > 0) $err = definition_set($id_definition);
+        if ($id_definition > 0) $err = definition_modifier($id_definition);
 
         return array($id_definition, $err);
@@ -50 +50 @@
  *     Identifiant de la nouvelle définition
  */
-function insert_definition($champs=array()) {
+function definition_inserer($champs=array()) {
         $lang = "";
         // La langue a la creation : si les liens de traduction sont autorises
@@ -111 +111 @@
  *     Chaîne contenant un texte d'erreur sinon.
  */
-function definition_set($id_definition, $set=null, $purger_cache=true) {
+function definition_modifier($id_definition, $set=null, $purger_cache=true) {
         $err = '';
 
@@ -140 +140 @@
 
         $c = collecter_requests(array('date', 'statut', 'id_dictionnaire'),array(),$set);
-        $err = instituer_definition($id_definition, $c, $purger_cache);
+        $err = definition_instituer($id_definition, $c, $purger_cache);
         return $err;
 }
@@ -160 +160 @@
  *     Null si aucun champ à modifier, chaîne vide sinon.
  */
-function instituer_definition($id_definition, $c, $purger_cache=true){
+function definition_instituer($id_definition, $c, $purger_cache=true){
         include_spip('inc/autoriser');
         include_spip('inc/rubriques');

_plugins_/dictionnaires/trunk/action/editer_dictionnaire.php

@@ -32 +32 @@
         // si id_dictionnaire n'est pas un nombre, c'est une creation
         if (!$id_dictionnaire = intval($arg)) {
-                $id_dictionnaire = insert_dictionnaire();
+                $id_dictionnaire = dictionnaire_inserer();
         }
 
         // Enregistre l'envoi dans la BD
-        if ($id_dictionnaire > 0) $err = dictionnaire_set($id_dictionnaire);
+        if ($id_dictionnaire > 0) $err = dictionnaire_modifier($id_dictionnaire);
 
         return array($id_dictionnaire,$err);
@@ -49 +49 @@
  *     Identifiant du nouveau dictionnaire
  */
-function insert_dictionnaire($champs=array()) {
+function dictionnaire_inserer($champs=array()) {
         // Envoyer aux plugins avant insertion
         $champs = pipeline('pre_insertion',
@@ -89 +89 @@
  *     Chaîne contenant un texte d'erreur sinon.
  */
-function dictionnaire_set($id_dictionnaire, $set=null) {
+function dictionnaire_modifier($id_dictionnaire, $set=null) {
         $err = '';
 
@@ -116 +116 @@
 
         $c = collecter_requests(array('statut'),array(),$set);
-        $err = instituer_dictionnaire($id_dictionnaire, $c);
+        $err = dictionnaire_instituer($id_dictionnaire, $c);
         return $err;
 }
@@ -134 +134 @@
  *     Null si aucun champ à modifier, chaîne vide sinon.
  */
-function instituer_dictionnaire($id_dictionnaire, $c){
+function dictionnaire_instituer($id_dictionnaire, $c){
         include_spip('inc/autoriser');
         include_spip('base/objets');

_plugins_/dictionnaires/trunk/dictionnaires_administrations.php

@@ -80 +80 @@
                         // On commence par créer un dictionnaire pour l'importation
                         include_spip('action/editer_dictionnaire');
-                        if ($id_dictionnaire = insert_dictionnaire()){
+                        if ($id_dictionnaire = dictionnaire_inserer()){
                                 // On lui met des champs par défaut
                                 autoriser_exception('modifier', 'dictionnaire', $id_dictionnaire);
-                                dictionnaire_set($id_dictionnaire, array(
+                                dictionnaire_modifier($id_dictionnaire, array(
                                         'titre' => _T('dictionnaire:importer_acronymes_titre'),
                                         'statut' => 'actif',
@@ -109 +109 @@
                                                 // On crée la définition dans la base SANS calculer le cache
                                                 include_spip('action/editer_definition');
-                                                if ($id_definition = insert_definition()){
+                                                if ($id_definition = definition_inserer()){
                                                         // 0 et pas $id_dictionnaire 
                                                         // car insert_definition a utilisé les valeurs par défaut
                                                         autoriser_exception('publierdans', 'dictionnaire', 0);
                                                         autoriser_exception('modifier', 'definition', $id_definition);
-                                                        definition_set($id_definition, $definition, false);
+                                                        definition_modifier($id_definition, $definition, false);
                                                         autoriser_exception('modifier', 'definition', $id_definition, false);
                                                         autoriser_exception('publierdans', 'dictionnaire', 0, false);

_plugins_/dictionnaires/trunk/dictionnaires_pipelines.php

@@ -13 +13 @@
 
 /**
- * Crée des liaisons entre les objets et les définitions (à faire)
- * Met à jour le cache des définitions
+ * Crée des liaisons entre les objets et les définitions.
  *
  * Pour chaque objet édité, regarde si les contenus possèdent des définitions
@@ -32 +31 @@
 **/
 function dictionnaires_post_edition($flux) {
-        if($flux['args']['action'] == 'instituer') {
-                $objet = objet_type($flux['args']['table']);
-                if (($objet === 'dictionnaire') or ($objet === 'definition')) {
-                        include_spip('inc/dictionnaires');
-                        dictionnaires_lister_definitions(true);
-                }
-        }
-        
         // TOUT CELA EST A FAIRE
         return $flux;

_plugins_/dictionnaires/trunk/paquet.xml

@@ -2 +2 @@
         prefix="dictionnaires"
         categorie="divers"
-        version="1.2.4"
+        version="1.3.0"
         etat="stable"
         compatibilite="[3.0.0;3.2.*]"
@@ -24 +24 @@
         <pipeline nom="optimiser_base_disparus" inclure="dictionnaires_pipelines.php" />
 
-        <pipeline nom="post_edition" inclure="dictionnaires_pipelines.php" />
+        <!-- <pipeline nom="post_edition" inclure="dictionnaires_pipelines.php" /> -->
         <pipeline nom="post_propre" inclure="dictionnaires_pipelines.php" />