Projet

Général

Profil

Bug #29545

Le code d'import ne met jamais à jour les rôles

Ajouté par Benjamin Dauvergne il y a 2 mois. Mis à jour il y a environ un mois.

Statut:
Solution déployée
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
07 jan. 2019
Echéance:
% réalisé:

100%

Patch proposed:
Oui

Description

Ici :

118         obj = search_role(self._role_d)
119         if obj:  # Role already exist
120             self._obj = obj
121             status = 'updated'
122             update_model(self._obj, kwargs)

il manque un save soit dans update_model() soit juste après. Je serais pour virer update_model() qui ne fait qu'obscurcir le code, et avoir localement quelque chose qui update le modèle en fonction de la liste des champs connu via Model._meta.

Ça a pour conséquence que la fonctionnalité d'import/export ne peut pas actuellement être utilisé pour travailler par exemple sur un renommage des rôles hors ligne.

0001-data_transfer-move-update_model-29545.patch Voir (1,64 ko) Benjamin Dauvergne, 08 jan. 2019 15:30

0002-data_transfer-fix-indentation-whitespaces-29545.patch Voir (967 octets) Benjamin Dauvergne, 08 jan. 2019 15:30

0003-data_transfer-save-object-in-update_model-fixes-2954.patch Voir (958 octets) Benjamin Dauvergne, 08 jan. 2019 15:30

0003-data_transfer-fix-indentation-whitespaces-29545.patch Voir (2,38 ko) Benjamin Dauvergne, 09 jan. 2019 19:24

0002-data_transfer-move-update_model-29545.patch Voir (1,64 ko) Benjamin Dauvergne, 09 jan. 2019 19:24

0004-data_transfer-save-object-in-update_model-fixes-2954.patch Voir (4,32 ko) Benjamin Dauvergne, 09 jan. 2019 19:24

0001-tests-simplify-import_site-tests-29545.patch Voir (7,12 ko) Benjamin Dauvergne, 09 jan. 2019 19:24

Révisions associées

Révision 75e0b378 (diff)
Ajouté par Benjamin Dauvergne il y a environ 2 mois

tests: simplify import_site tests (#29545)

Révision 53b9f74a (diff)
Ajouté par Benjamin Dauvergne il y a environ 2 mois

data_transfer: move update_model (#29545)

Révision ce72b9a2 (diff)
Ajouté par Benjamin Dauvergne il y a environ 2 mois

data_transfer: fix indentation/whitespaces (#29545)

Révision a44d45fc (diff)
Ajouté par Benjamin Dauvergne il y a environ 2 mois

data_transfer: save object in update_model (fixes #29545)

Historique

#2 Mis à jour par Benjamin Dauvergne il y a 2 mois

  • Assigné à mis à Benjamin Dauvergne

#3 Mis à jour par Benjamin Dauvergne il y a 2 mois

Avec des petites choses em plus.

#4 Mis à jour par Frédéric Péters il y a 2 mois

  • Statut changé de Solution proposée à Solution validée

À propos de 0002, vu qu'il concerne les espaces, en anglais on ne met pas d'espace avant la ponctuation double.

À part ça, si jamais tu te sens d'ajouter un test sur ce cas, histoire de compléter test_import_export_site_cmd.py, c'est cool, mais sinon ok quand même.

#5 Mis à jour par Benjamin Dauvergne il y a 2 mois

J'en ai profité pour faire un peu de nettoyage dans les tests que ce soit plus clair, surtout quand on en ajoutera d'autres.

#6 Mis à jour par Benjamin Dauvergne il y a environ 2 mois

  • % réalisé changé de 0 à 100
  • Statut changé de Solution proposée à Résolu (à déployer)

#7 Mis à jour par Frédéric Péters il y a environ un mois

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF