Projet

Général

Profil

Development #58695

Ne plus utiliser get_role_model, get_organizational_unit, etc.. tout ce qui vient de django_rbac.utils

Ajouté par Valentin Deniaud il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
18 novembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Remplacer par des imports directs des modèles concrets.


Fichiers

Révisions associées

Révision 50e48b76 (diff)
Ajouté par Paul Marillonnet il y a plus de 2 ans

discard django_rbac generic getters in main a2 code (#58695)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

  • Tags mis à facile
  • Sujet changé de Virer get_role_model à Ne plus utiliser get_role_model, get_organizational_unit, etc.. tout ce qui vient de django_rbac.utils
#2

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

  • Description mis à jour (diff)
#3

Mis à jour par Paul Marillonnet il y a plus de 2 ans

  • Assigné à mis à Paul Marillonnet
#4

Mis à jour par Paul Marillonnet il y a plus de 2 ans

  • Tags facile supprimé

Il y a plus de choses dans django_rbac.utils qu’une simple répercussion de la duplication des modèles. Je regarde si on peut tout sortir django_rbac.utils ou bien dans un premier temps simplement les fonctions get_*_model{,_name}.

#5

Mis à jour par Paul Marillonnet il y a plus de 2 ans

Voilà, une première passe, en laissant de côté src/django_rbac/ et tests_rbac/ avec l’idée que ça serait de toute façon réintégré dans src/authentic2/a2_rbac/ et tests/ respectivement.

#6

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

  • Statut changé de Solution proposée à En cours
#7

Mis à jour par Paul Marillonnet il y a plus de 2 ans

(pylint encore, je corrige dans la branche.)

#9

Mis à jour par Paul Marillonnet il y a plus de 2 ans

(Pour info j’ai aussi exécuté les tests rbac (non pris par le job jenkins wip), qui passent normalement.)

#10

Mis à jour par Paul Marillonnet il y a plus de 2 ans

(Et en cliquant un peu partout dans mon devinst je ne vois rien de suspect non plus.)

#11

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

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

Ok, pour vendredi.

#12

Mis à jour par Paul Marillonnet il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 50e48b760ad59606684292308e65f73bb59da247
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Mon Nov 22 14:03:40 2021 +0100

    discard django_rbac generic getters in main a2 code (#58695)
#13

Mis à jour par Frédéric Péters il y a plus de 2 ans

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

Formats disponibles : Atom PDF