Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Le groupe staff des fixtures n'a pas les bonnes permissions #4266

Closed
gllmc opened this issue Apr 3, 2017 · 4 comments · Fixed by #6045
Closed

Le groupe staff des fixtures n'a pas les bonnes permissions #4266

gllmc opened this issue Apr 3, 2017 · 4 comments · Fixed by #6045
Labels
C-DevelopmentEnv Amélioration de l'environnement de dev S-BUG Corrige un problème

Comments

@gllmc
Copy link
Member

gllmc commented Apr 3, 2017

Actuellement, un groupe staff est créé par les fixtures ici. Cependant, les permissions qu'il se voit assigner ne sont pas les bonnes. Probablement car elles sont données via les pk, ce qui n'est pas viable (ils peuvent changer à tout moment et en plus, on n'a aucune idée de quelles permissions il s'agit en lisant les fixtures).

Il faudrait donc revoir les fixtures. L'idéal serait de trouver une meilleure solution que de mettre à jour les pk. ;)
@gllmc gllmc added C-Back Concerne le back-end Django S-BUG Corrige un problème labels Apr 3, 2017
@artragis
Copy link
Member

est-ce toujours le cas @gcodeur ?

@pierre-24
Copy link
Member

Oh que oui.

@gllmc
Copy link
Member Author

gllmc commented Aug 12, 2017

Oui. En fait, je pense que la meilleure solution serait d'appliquer dynamiquement toutes les permissions en change_ comme on fait pour les tests, mais je ne sais pas comment ça se fait.

@artragis artragis added C-DevelopmentEnv Amélioration de l'environnement de dev and removed C-Back Concerne le back-end Django labels Apr 10, 2020
@Arnaud-D Arnaud-D mentioned this issue Feb 22, 2021
Closed
@artragis
Copy link
Member

Dupliqué dans un autre ticket où une solution meilleure a été trouvée

AmauryCarrade added a commit to AmauryCarrade/zds-site that referenced this issue Feb 24, 2021
@AmauryCarrade
Les fixtures ne référencent plus les permissions par PK
96b1b01 
Les clefs naturelles sont utilisées à la place, pour tout ce qui
concerne les utilisateurs.

Fixes zestedesavoir#6022
Fixes zestedesavoir#4266
@AmauryCarrade AmauryCarrade mentioned this issue Feb 24, 2021
Merged
artragis pushed a commit that referenced this issue Mar 12, 2021
@AmauryCarrade @artragis
Les fixtures ne référencent plus les permissions par PK
356a9ea 
Les clefs naturelles sont utilisées à la place, pour tout ce qui
concerne les utilisateurs.

Fixes #6022
Fixes #4266
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-DevelopmentEnv Amélioration de l'environnement de dev S-BUG Corrige un problème
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Les fixtures ne référencent plus les permissions par PK AmauryCarrade/zds-site
3 participants
@artragis @pierre-24 @gllmc