Projet

Général

Profil

Anomalie #728

Galette v0.7.6. Le modèle personnalisé pour l'import ne peut pas être modifié.

Ajouté par Roland Telle il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
Core
Version cible:
-
Début:
20/10/2013
Echéance:
% réalisé:

100%

Version utilisée:

Description

Lorsqu'on change de modèle en réalisant une création personnelle, le modèle A ainsi obtenu ne peut pas être modifié bien que la modification soit annoncée comme effectuée après enregistrement.
Il faut supprimer le modèle A et revenir aux champs par défaut pour créer un modèle B si le modèle A ne donne pas satisfaction.

Révisions associées

Révision e5803b83
Ajouté par Johan Cwiklinski il y a presque 4 ans

Import model could not be changed; fixes #728

Historique

#1 Mis à jour par Johan Cwiklinski il y a presque 4 ans

  • Catégorie mis à Core
  • Assigné à mis à Johan Cwiklinski

#2 Mis à jour par Johan Cwiklinski il y a presque 4 ans

À priori, ça devrait être corrigé avec les modifications suivantes :

diff --git a/galette/import_model.php b/galette/import_model.php
index 9dc45a4..d578c21 100644
--- a/galette/import_model.php
+++ b/galette/import_model.php
@@ -53,12 +53,14 @@ if ( !$login->isAdmin() ) {
 }

 $model = new ImportModel();
+$model->load();

 if ( isset($_POST['fields']) ) {
     $model->setFields($_POST['fields']);
     $res = $model->store($zdb);
     if ( $res === true ) {
         $success_detected[] = _T("Import model has been successfully stored :)");
+        $model->load();
     }
         $error_detected[] = _T("Import model has not been stored :(");
     }
@@ -66,6 +68,7 @@ if ( isset($_POST['fields']) ) {

 if ( isset($_GET['remove']) ) {
     $model->remove($zdb);
+    $model->load();
 }

 $csv = new CsvIn();
@@ -75,7 +78,6 @@ $csv = new CsvIn();
  * - set fields that must be part of import, and visually disable them in the list
  */

-$model->load();
 $fields = $model->getFields();
 $defaults = $csv->getDefaultFields();
 $defaults_loaded = false;

#3 Mis à jour par Roland Telle il y a presque 4 ans

Johan Cwiklinski a écrit :

À priori, ça devrait être corrigé avec les modifications suivantes :
[...]

Le correctif fonctionne parfaitement ! Merci ;-)

#4 Mis à jour par Johan Cwiklinski il y a presque 4 ans

  • Statut changé de Nouveau à In Progress

#5 Mis à jour par Johan Cwiklinski il y a presque 4 ans

  • Statut changé de In Progress à Résolu
  • % réalisé changé de 0 à 100

#6 Mis à jour par Johan Cwiklinski il y a presque 4 ans

  • Statut changé de Résolu à Fermé

Formats disponibles : Atom PDF