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 environ 5 ans. Mis à jour il y a environ 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
07 janvier 2019
Echéance:
% réalisé:

100%

Temps estimé:
Patch proposed:
Oui
Planning:

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

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

tests: simplify import_site tests (#29545)

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

data_transfer: move update_model (#29545)

Révision ce72b9a2 (diff)
Ajouté par Benjamin Dauvergne il y a environ 5 ans

data_transfer: fix indentation/whitespaces (#29545)

Révision a44d45fc (diff)
Ajouté par Benjamin Dauvergne il y a environ 5 ans

data_transfer: save object in update_model (fixes #29545)

Historique

#2

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Assigné à mis à Benjamin Dauvergne
#4

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.

#6

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
#7

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

Formats disponibles : Atom PDF