Project

General

Profile

Bug #29545

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

Added by Benjamin Dauvergne 6 months ago. Updated 5 months ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
07 Jan 2019
Due date:
% Done:

100%

Patch proposed:
Yes
Planning:
No

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 View (1.64 KB) Benjamin Dauvergne, 08 Jan 2019 03:30 PM

0002-data_transfer-fix-indentation-whitespaces-29545.patch View (967 Bytes) Benjamin Dauvergne, 08 Jan 2019 03:30 PM

0003-data_transfer-save-object-in-update_model-fixes-2954.patch View (958 Bytes) Benjamin Dauvergne, 08 Jan 2019 03:30 PM

0003-data_transfer-fix-indentation-whitespaces-29545.patch View (2.38 KB) Benjamin Dauvergne, 09 Jan 2019 07:24 PM

0002-data_transfer-move-update_model-29545.patch View (1.64 KB) Benjamin Dauvergne, 09 Jan 2019 07:24 PM

0004-data_transfer-save-object-in-update_model-fixes-2954.patch View (4.32 KB) Benjamin Dauvergne, 09 Jan 2019 07:24 PM

0001-tests-simplify-import_site-tests-29545.patch View (7.12 KB) Benjamin Dauvergne, 09 Jan 2019 07:24 PM

Associated revisions

Revision 75e0b378 (diff)
Added by Benjamin Dauvergne 5 months ago

tests: simplify import_site tests (#29545)

Revision 53b9f74a (diff)
Added by Benjamin Dauvergne 5 months ago

data_transfer: move update_model (#29545)

Revision ce72b9a2 (diff)
Added by Benjamin Dauvergne 5 months ago

data_transfer: fix indentation/whitespaces (#29545)

Revision a44d45fc (diff)
Added by Benjamin Dauvergne 5 months ago

data_transfer: save object in update_model (fixes #29545)

History

#2 Updated by Benjamin Dauvergne 6 months ago

  • Assignee set to Benjamin Dauvergne

#3 Updated by Benjamin Dauvergne 6 months ago

Avec des petites choses em plus.

#4 Updated by Frédéric Péters 6 months ago

  • Status changed from Solution proposée to 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 Updated by Benjamin Dauvergne 6 months ago

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 Updated by Benjamin Dauvergne 5 months ago

  • % Done changed from 0 to 100
  • Status changed from Solution proposée to Résolu (à déployer)

#7 Updated by Frédéric Péters 5 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF