thubrecht/poulpe
1
0
Fork 0
Code Issues 5 Pull requests 1 Projects Releases Packages Wiki Activity Actions

Permissions par évènement #20

Merged
lstephan merged 11 commits from Aufinal/permissions into master 2018-05-03 14:34:38 +02:00
Conversation 18 Commits 11 Files changed 15 +302 -94
lstephan commented 2017-07-21 21:10:41 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ajoute un système de permissions spécifiques à un évènement donné.

  • utilise django-guardian pour avoir un système de permissions par évènement
  • génère toutes les permissions nécessaires automatiquement
  • crée des groupes spécifiques orgas et participants par évènements (modèle GEGroup), et leur assigne les permissions associées.
Ajoute un système de permissions spécifiques à un évènement donné. - utilise `django-guardian` pour avoir un système de permissions par évènement - génère toutes les permissions nécessaires automatiquement - crée des groupes spécifiques `orgas` et `participants` par évènements (modèle `GEGroup`), et leur assigne les permissions associées.
mpepin commented 2017-07-23 19:37:47 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

On pourrait filtrer sur le content_type ici, ce serait plus précis.

Par exemple :

Permission.objects.filter(content_type__app_label="event", content_type__model="Event")
On pourrait filtrer sur le content_type ici, ce serait plus précis. Par exemple : ```python Permission.objects.filter(content_type__app_label="event", content_type__model="Event") ```
lstephan commented 2017-07-24 12:52:56 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ca donnerait aux gens la possibilité de supprimer l'évènement, donc faut voir si on veut ça.

Ca donnerait aux gens la possibilité de supprimer l'évènement, donc faut voir si on veut ça.
mpepin commented 2017-07-24 12:57:17 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Effectivement c'est pas forcément ce qu'on veut.

Mais ce n'était pas le sujet de mon objection, j'ai probablement mal compris ce qui se passe ici, je reformule : quelles permissions cette requête va chercher ? Y a-t-il un moyen plus robuste d'aller les chercher que .filter(codename__contains=…) ?

Effectivement c'est pas forcément ce qu'on veut. Mais ce n'était pas le sujet de mon objection, j'ai probablement mal compris ce qui se passe ici, je reformule : quelles permissions cette requête va chercher ? Y a-t-il un moyen plus robuste d'aller les chercher que `.filter(codename__contains=…)` ?
lstephan commented 2017-07-24 13:04:54 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

En gros, ça va chercher les permissions créées pas create_event_permissions (et qui normalement sont les seules avec le codename event_*)

En gros, ça va chercher les permissions créées pas `create_event_permissions` (et qui normalement sont les seules avec le codename `event_*`)
mpepin commented 2017-07-24 13:51:19 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

J'ai un problème avec le « normalement » : qu'est-ce qui se passe le jour où on utilise une lib pour avoir un calendrier qui a un modèle Event ?

J'ai un problème avec le « normalement » : qu'est-ce qui se passe le jour où on utilise une lib pour avoir un calendrier qui a un modèle `Event` ?
mpepin commented 2017-07-24 14:09:24 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

En fait, juste avec un content_type=… en plus dans le filter, je pense qu'on s'évite des problèmes.

En fait, juste avec un `content_type=…` en plus dans le `filter`, je pense qu'on s'évite des problèmes.
lstephan commented 2017-07-24 14:09:50 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Note bien, on a déjà un modèle Event ; mais les permissions associées sont {add,change,delete}_event donc ya pas de collision.

Note bien, on a déjà un modèle `Event` ; mais les permissions associées sont `{add,change,delete}_event` donc ya pas de collision.
mpepin commented 2017-07-24 14:11:55 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ok, j'ai bien compris qu'il n'y avait pas de collision au sein de ce content_type mais on ne contrôle pas le nom des perms sur les modèles des librairies qu'on utilise. Ça ne coûte rien de rajouter une condition dans filter

Ok, j'ai bien compris qu'il n'y avait pas de collision au sein de ce content_type mais on ne contrôle pas le nom des perms sur les modèles des librairies qu'on utilise. Ça ne coûte rien de rajouter une condition dans `filter`
lstephan commented 2017-07-24 15:47:43 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * 25df34e5 - Add filter to permissions [Compare with previous version](https://git.eleves.ens.fr/cof-geek/GestionEvenementiel/merge_requests/15/diffs?diff_id=555&start_sha=910d8fe9c094101e302d225b6c56e6d68f8bb0d2)
mpepin commented 2017-07-24 19:06:03 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

resolved all discussions

resolved all discussions
delobell commented 2017-07-25 11:42:20 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

T'as la motiv' d'écrire des tests ? Sinon je peux le faire

T'as la motiv' d'écrire des tests ? Sinon je peux le faire
lstephan commented 2017-08-17 14:24:20 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 51 commits

  • 25df34e5...677c7f33 - 50 commits from branch master
  • 64a979a4 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/GestionEvenementiel into Aufinal/permissions

Compare with previous version

added 51 commits * 25df34e5...677c7f33 - 50 commits from branch `master` * 64a979a4 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/GestionEvenementiel into Aufinal/permissions [Compare with previous version](https://git.eleves.ens.fr/cof-geek/GestionEvenementiel/merge_requests/15/diffs?diff_id=601&start_sha=25df34e57ea1697cbee4d7b673b9c816e1ab383c)
lstephan commented 2017-08-17 15:11:08 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * 0b1641a0 - Tests and group uniqueness [Compare with previous version](https://git.eleves.ens.fr/cof-geek/GestionEvenementiel/merge_requests/15/diffs?diff_id=602&start_sha=64a979a4acc5bbae647230fa4fa4719a833cc6e8)
lstephan commented 2017-09-19 10:46:40 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * c894d359 - Scrap useless perms [Compare with previous version](https://git.eleves.ens.fr/cof-geek/GestionEvenementiel/merge_requests/15/diffs?diff_id=641&start_sha=0b1641a002162152897dbe1521af95b3d07b96bf)
lstephan commented 2017-09-19 11:52:06 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * 392e5cf1 - Revert c894d35 --- Fuck django-guardian [Compare with previous version](https://git.eleves.ens.fr/cof-geek/GestionEvenementiel/merge_requests/15/diffs?diff_id=642&start_sha=c894d359324eab0c6fab607d6c4afe74fbc23dc2)
mpepin commented 2018-05-03 12:46:19 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 6 commits

  • 392e5cf1...94b2529a - 5 commits from branch master
  • 10e4393b - Merge branch 'master' into Aufinal/permissions

Compare with previous version

added 6 commits * 392e5cf1...94b2529a - 5 commits from branch `master` * 10e4393b - Merge branch 'master' into Aufinal/permissions [Compare with previous version](https://git.eleves.ens.fr/cof-geek/GestionEvenementiel/merge_requests/15/diffs?diff_id=850&start_sha=392e5cf14477a2ecb484e34f8c2261f2421f1d08)
mpepin commented 2018-05-03 14:34:38 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

merged

merged
mpepin commented 2018-05-03 14:34:39 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in commit 3aab76613a

mentioned in commit 3aab76613a502a3cd516b755fdf1fc855c80fee7
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Doing

   
Doing

   
Good first issue

   
To Do
No labels
Doing
Doing
Good first issue
To Do
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
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: thubrecht/poulpe#20
No description provided.
Delete branch "Aufinal/permissions"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?