Bug #29545
Le code d'import ne met jamais à jour les rôles
100%
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.
Fichiers
Révisions associées
data_transfer: move update_model (#29545)
data_transfer: fix indentation/whitespaces (#29545)
data_transfer: save object in update_model (fixes #29545)
Historique
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier 0001-data_transfer-move-update_model-29545.patch 0001-data_transfer-move-update_model-29545.patch ajouté
- Fichier 0002-data_transfer-fix-indentation-whitespaces-29545.patch 0002-data_transfer-fix-indentation-whitespaces-29545.patch ajouté
- Fichier 0003-data_transfer-save-object-in-update_model-fixes-2954.patch 0003-data_transfer-save-object-in-update_model-fixes-2954.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Avec des petites choses em plus.
Mis à jour par Frédéric Péters il y a environ 5 ans
- 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.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier 0003-data_transfer-fix-indentation-whitespaces-29545.patch 0003-data_transfer-fix-indentation-whitespaces-29545.patch ajouté
- Fichier 0002-data_transfer-move-update_model-29545.patch 0002-data_transfer-move-update_model-29545.patch ajouté
- Fichier 0004-data_transfer-save-object-in-update_model-fixes-2954.patch 0004-data_transfer-save-object-in-update_model-fixes-2954.patch ajouté
- Fichier 0001-tests-simplify-import_site-tests-29545.patch 0001-tests-simplify-import_site-tests-29545.patch ajouté
- Statut changé de Solution validée à Solution proposée
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.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution proposée à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit authentic2|a44d45fc2402d3e43c0bbdd07dcd71072569afa1.
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
tests: simplify import_site tests (#29545)