Project

General

Profile

Anomalie #728

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

Added by Roland Telle about 6 years ago. Updated about 6 years ago.

Status:
Fermé
Priority:
Normal
Category:
Core
Target version:
-
Start date:
10/20/2013
Due date:
% Done:

100%

Estimated time:
Version utilisée:
Vote:

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.

Associated revisions

Revision e5803b83 (diff)
Added by Johan Cwiklinski about 6 years ago

Import model could not be changed; fixes #728

History

#1

Updated by Johan Cwiklinski about 6 years ago

  • Category set to Core
  • Assignee set to Johan Cwiklinski
#2

Updated by Johan Cwiklinski about 6 years ago

À 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();
     } else {
         $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

Updated by Roland Telle about 6 years ago

Johan Cwiklinski a écrit :

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

Le correctif fonctionne parfaitement ! Merci ;-)

#4

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from Nouveau to In Progress
#5

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from In Progress to Résolu
  • % Done changed from 0 to 100
#6

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF