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 13 jours. Mis à jour il y a 11 jours.

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

0%

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

Historique

#2 Mis à jour par Benjamin Dauvergne il y a 12 jours

  • Assigné à mis à Benjamin Dauvergne

#3 Mis à jour par Benjamin Dauvergne il y a 12 jours

Avec des petites choses em plus.

#4 Mis à jour par Frédéric Péters il y a 12 jours

  • 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 11 jours

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.

Formats disponibles : Atom PDF