Changeset 90570 in spip-zone

Timestamp:

Jun 28, 2015, 7:49:49 PM (6 years ago)

Author:

cedric@…

Message:

normalisation de action/editer_produit (SPIP 3)

File:

• _plugins_/produits/trunk/action/editer_produit.php (modified) (6 diffs)

_plugins_/produits/trunk/action/editer_produit.php

@@ -6 +6 @@
 /**
  * Action de création / Modification d'un produit
- * @param unknown_type $arg
- * @return unknown_type
+ * @param string|null $arg
+ * @return array
  */
 function action_editer_produit_dist($arg=null) {
+        include_spip('inc/autoriser');
+        $err="";
         if (is_null($arg)){
                 $securiser_action = charger_fonction('securiser_action', 'inc');
@@ -17 +19 @@
         // si id_produit n'est pas un nombre, c'est une creation
         if (!$id_produit = intval($arg)) {
-                $id_produit = insert_produit(array('id_rubrique'=>_request('id_parent')));
+                $id_produit = produit_inserer(_request('id_parent'));
         }
 
         // Enregistre l'envoi dans la BD
-        if ($id_produit > 0) $err = produit_set($id_produit);
+        if ($id_produit > 0) $err = produit_modifier($id_produit);
+
+        if ($err)
+                spip_log("echec editeur produit: $err",_LOG_ERREUR);
 
         return array($id_produit,$err);
@@ -29 +34 @@
  * Crée un nouveau produit et retourne son ID
  *
- * @param array $champs Un tableau avec les champs par défaut lors de l'insertion
- * @return int id_produit
- */
-function insert_produit($champs=array()) {
+ * @param int $id_rubrique
+ * @param array $set
+ *   Un tableau avec les champs par défaut lors de l'insertion
+ * @return int
+ */
+function produit_inserer($id_rubrique,$set=null) {
         $id_produit = false;
         
         // On insère seulement s'il y a une rubrique correcte
-        if (isset($champs['id_rubrique']) and $champs['id_rubrique'] = intval($champs['id_rubrique'])){
+        if ($id_rubrique = intval($id_rubrique)){
+                $champs = array();
                 // Si id_rubrique vaut 0 ou n'est pas definie, creer le produit dans la premiere rubrique racine
-                if (!$id_rubrique = intval($champs['id_rubrique'])) {
+                if (!$id_rubrique = intval($id_rubrique)) {
                         $row = sql_fetsel('id_rubrique, id_secteur, lang', 'spip_rubriques', 'id_parent=0','', '0+titre,titre', "1");
                         $id_rubrique = $row['id_rubrique'];
-                } else $row = sql_fetsel('lang, id_secteur', 'spip_rubriques', "id_rubrique=$id_rubrique");
-
+                } else $row = sql_fetsel('lang, id_secteur', 'spip_rubriques', "id_rubrique=".intval($id_rubrique));
+
+                $lang_rub = $row['lang'];
+                $champs['id_rubrique'] = $id_rubrique;
                 // On propage le secteur
                 $champs['id_secteur'] = $row['id_secteur'];
@@ -55 +65 @@
                 $champs['statut'] = 'prop';
                 
+                if ($set)
+                        $champs = array_merge($champs, $set);
+
                 // Envoyer aux plugins avant insertion
                 $champs = pipeline('pre_insertion',
@@ -84 +97 @@
  *
  * @param int $id_produit
- * @param unknown_type $set
- * @return $err
- */
-function produit_set($id_produit, $set=null) {
+ * @param array $set
+ * @return string
+ */
+function produit_modifier($id_produit, $set=null) {
         $err = '';
-        
-        include_spip('inc/saisies');
-        $saisies = saisies_chercher_formulaire('editer_produit', array($id_produit));
-        $champs = saisies_lister_champs($saisies, false);
-        
-        $c = array();
-        foreach ($champs as $champ)
-                $c[$champ] = _request($champ,$set);
-
-        // on donne une taxe null si le champ est vide
-        // MAIS spip < 2.3 ne gere pas les null avec sql_updateq..
-        // nous appliquons une requete en plus pour ce cas
-        if (isset($c['taxe']) and !strlen(trim($c['taxe']))) {
-                sql_update('spip_produits', array('taxe'=>'NULL'), 'id_produit='.$id_produit);
-                unset ($c['taxe']);
-        }
-                
+
         include_spip('inc/modifier');
-        revision_produit($id_produit, $c);
-        
-        // Modification de statut, changement de rubrique ?
-        $c = array();
-        foreach (array(
-                'date', 'statut', 'id_parent'
-        ) as $champ)
-                $c[$champ] = _request($champ, $set);
-        $err .= instituer_produit($id_produit, $c);
-        
-        return $err;
-}
-
-/**
- * Enregistre une révision de produit
- *
- * @param int $id_produit
- * @param array $c
- * @return
- */
-function revision_produit($id_produit, $c=false) {
-        $invalideur = "id='id_produit/$id_produit'";
-
-        modifier_contenu('produit', $id_produit,
+        include_spip('inc/filtres');
+        $c = collecter_requests(
+                // white list
+                objet_info('produit','champs_editables'),
+                // black list
+                array('date','statut','id_parent'),
+                // donnees eventuellement fournies
+                $set
+        );
+
+        // Si le produit est publie, invalider les caches et demander sa reindexation
+        $t = sql_getfetsel("statut", "spip_produits", "id_produit=".intval($id_produit));
+        $invalideur = $indexation = false;
+        if ($t == 'publie') {
+                $invalideur = "id='produit/$id_produit'";
+                $indexation = true;
+        }
+
+        if ($err = objet_modifier_champs('produit', $id_produit,
                 array(
                         'nonvide' => array('titre' => _T('info_sans_titre')),
-                        'invalideur' => $invalideur
+                        'invalideur' => $invalideur,
+                        'indexation' => $indexation,
                 ),
-                $c);
-
-        return ''; // pas d'erreur
-}
+                $c))
+                return $err;
+
+        // Modification de statut, changement de rubrique ?
+        $c = collecter_requests(array('date', 'statut', 'id_parent'),array(),$set);
+        $err = produit_instituer($id_produit, $c);
+
+        return $err;
+}
+
 
 /**
@@ -145 +144 @@
  * @param int $id_produit
  * @param array $c
- * @return
- */
-function instituer_produit($id_produit, $c, $calcul_rub=true){
+ * @param bool $calcul_rub
+ * @return string
+ */
+function produit_instituer($id_produit, $c, $calcul_rub=true){
         include_spip('inc/autoriser');
         include_spip('inc/rubriques');