BDD: relation role -> département #108

Closed
opened 2021-08-22 11:27:58 +02:00 by jmplace · 1 comment
Collaborator

un role vu étant comme un couple (fonction , département) il pourrait être opportun de vérifier l intégrité référentielle.

Ca n'a pas été fait lors de la migration vers scodoc9 car le nom département est cité dans une chaine (ex: Ens_RT) alors de la clé de dept est un entier

exemple d impact de cette remarque dans le code (app/auth/models.py)

`
class UserRole(db.Model):
"""Associate user to role, in a dept.
If dept is None, the role applies to all departments (eg super admin).
"""

id = db.Column(db.Integer, primary_key=True)
user_id = db.Column(db.Integer, db.ForeignKey("user.id"))
role_id = db.Column(db.Integer, db.ForeignKey("role.id"))
dept = db.Column(db.String(64))  # dept acronym

`

un role vu étant comme un couple (fonction , département) il pourrait être opportun de vérifier l intégrité référentielle. Ca n'a pas été fait lors de la migration vers scodoc9 car le nom département est cité dans une chaine (ex: Ens_RT) alors de la clé de dept est un entier exemple d impact de cette remarque dans le code (app/auth/models.py) ` class UserRole(db.Model): """Associate user to role, in a dept. If dept is None, the role applies to all departments (eg super admin). """ id = db.Column(db.Integer, primary_key=True) user_id = db.Column(db.Integer, db.ForeignKey("user.id")) role_id = db.Column(db.Integer, db.ForeignKey("role.id")) dept = db.Column(db.String(64)) # dept acronym `
viennet added the
Dev
label 2021-08-22 11:30:42 +02:00
viennet added the
wontfix
label 2021-10-08 16:31:10 +02:00
Owner

Cela contraindrait trop les migrations.

Dans certains cas, on souhaite migrer des utilisateurs (et leurs rôles) sans migrer
tous les départements en même temps.

Cela contraindrait trop les migrations. Dans certains cas, on souhaite migrer des utilisateurs (et leurs rôles) sans migrer tous les départements en même temps.
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: ScoDoc/ScoDoc#108
No description provided.