L'édition d'un groupe K-Fêt via son site supprime des permissions liées à Wagtail.

manet commented

2017-09-10 02:10:22 +02:00

(Migrated from git.eleves.ens.fr)

Et du coup, quand j'ai rajouté la permission "fermer manuellement la K-Fêt" au groupe Cheffe, ça a supprimé cette permission (et une autre permission, d'ailleurs).

Mais si c'était voulu, si jamais on veut pas que cette permission soit "gérable à la main" (càd si on veut que seul le groupe Cheffe puisse modifier le site, pour les siècles des siècles, ce qui me semble vachement contestable), alors il faudrait aussi s'assurer qu'on ne puisse pas la supprimer en enregistrant un nouvel ensemble de permissions à la place.

EDIT: Voir #168 pour le second point.

Et du coup, quand j'ai rajouté la permission "fermer manuellement la K-Fêt" au groupe Cheffe, ça a supprimé cette permission (et une autre permission, d'ailleurs). Mais si c'était voulu, si jamais on veut pas que cette permission soit "gérable à la main" (càd si on veut que seul le groupe Cheffe puisse modifier le site, pour les siècles des siècles, ce qui me semble vachement contestable), alors il faudrait aussi s'assurer qu'on ne puisse pas la supprimer en enregistrant un nouvel ensemble de permissions à la place. EDIT: Voir #168 pour le second point.

manet commented

2017-09-10 02:11:05 +02:00

(Migrated from git.eleves.ens.fr)

added ~35 label

manet commented

2017-09-10 02:13:28 +02:00

(Migrated from git.eleves.ens.fr)

(le label "urgent" est peut-être un peu exagéré, mais soit on rajoute les permissions directement dans la BDD, soit on corrige ça, soit les chefs ne peuvent pas modifier le texte (et donc c'est toujours des vieux chef-fe-s qui apparaissent, notamment)

delobell commented

2017-09-10 02:19:11 +02:00

(Migrated from git.eleves.ens.fr)

Pour la gestion des permissions du CMS depuis le site K-Fêt, c'est bien noté.
Je rajoute aussi qu'il faudrait trouver un moyen d'autoriser la modification d'un seul 'Menu additionnel' également.

Je suis pas sûr d'avoir saisi le rapport entre la permission Fermer manuellement... et les permissions du CMS. Modifier les permissions du groupe 'Chef' sur le site K-Fêt a entraîné la suppression des attributions de permissions du CMS à ce groupe ?

Pour la gestion des permissions du CMS depuis le site K-Fêt, c'est bien noté. Je rajoute aussi qu'il faudrait trouver un moyen d'autoriser la modification d'un seul 'Menu additionnel' également. Je suis pas sûr d'avoir saisi le rapport entre la permission `Fermer manuellement...` et les permissions du CMS. Modifier les permissions du groupe 'Chef' sur le site K-Fêt a entraîné la suppression des attributions de permissions du CMS à ce groupe ?

mpepin commented

2017-09-10 02:21:07 +02:00

(Migrated from git.eleves.ens.fr)

Elles ont toujours l'air là

In [12]: for p in chef_group.permissions.all():
    ...:     print("{}.{}".format(p.content_type.app_label, p.codename))
    ...:     
kfet.add_account
kfet.can_force_close
kfet.change_account
kfet.change_account_password
kfet.delete_account
kfet.edit_balance_account
kfet.is_team
kfet.manage_addcosts
kfet.manage_perms
kfet.special_add_account
kfet.add_accountnegative
kfet.change_accountnegative
kfet.delete_accountnegative
kfet.view_negs
kfet.add_article
kfet.change_article
kfet.delete_article
kfet.add_articlecategory
kfet.change_articlecategory
kfet.delete_articlecategory
kfet.add_articlerule
kfet.change_articlerule
kfet.delete_articlerule
kfet.add_checkout
kfet.change_checkout
kfet.delete_checkout
kfet.add_checkoutstatement
kfet.change_checkoutstatement
kfet.delete_checkoutstatement
kfet.add_checkouttransfer
kfet.change_checkouttransfer
kfet.delete_checkouttransfer
kfet.add_genericteamtoken
kfet.change_genericteamtoken
kfet.delete_genericteamtoken
kfet.add_inventory
kfet.change_inventory
kfet.delete_inventory
kfet.order_to_inventory
kfet.add_inventoryarticle
kfet.change_inventoryarticle
kfet.delete_inventoryarticle
kfet.add_operation
kfet.cancel_old_operations
kfet.change_operation
kfet.delete_operation
kfet.override_frozen_protection
kfet.perform_commented_operations
kfet.perform_deposit
kfet.perform_negative_operations
kfet.add_operationgroup
kfet.change_operationgroup
kfet.delete_operationgroup
kfet.add_order
kfet.change_order
kfet.delete_order
kfet.add_orderarticle
kfet.change_orderarticle
kfet.delete_orderarticle
kfet.add_supplier
kfet.change_supplier
kfet.delete_supplier
kfet.add_supplierarticle
kfet.change_supplierarticle
kfet.delete_supplierarticle
kfet.add_transfer
kfet.change_transfer
kfet.delete_transfer
kfet.add_transfergroup
kfet.change_transfergroup
kfet.delete_transfergroup
kfetcms.add_memberteam
kfetcms.change_memberteam
kfetcms.delete_memberteam
wagtailadmin.access_admin
wagtailmenus.change_flatmenu

Elles ont toujours l'air là ``` In [12]: for p in chef_group.permissions.all(): ...: print("{}.{}".format(p.content_type.app_label, p.codename)) ...: kfet.add_account kfet.can_force_close kfet.change_account kfet.change_account_password kfet.delete_account kfet.edit_balance_account kfet.is_team kfet.manage_addcosts kfet.manage_perms kfet.special_add_account kfet.add_accountnegative kfet.change_accountnegative kfet.delete_accountnegative kfet.view_negs kfet.add_article kfet.change_article kfet.delete_article kfet.add_articlecategory kfet.change_articlecategory kfet.delete_articlecategory kfet.add_articlerule kfet.change_articlerule kfet.delete_articlerule kfet.add_checkout kfet.change_checkout kfet.delete_checkout kfet.add_checkoutstatement kfet.change_checkoutstatement kfet.delete_checkoutstatement kfet.add_checkouttransfer kfet.change_checkouttransfer kfet.delete_checkouttransfer kfet.add_genericteamtoken kfet.change_genericteamtoken kfet.delete_genericteamtoken kfet.add_inventory kfet.change_inventory kfet.delete_inventory kfet.order_to_inventory kfet.add_inventoryarticle kfet.change_inventoryarticle kfet.delete_inventoryarticle kfet.add_operation kfet.cancel_old_operations kfet.change_operation kfet.delete_operation kfet.override_frozen_protection kfet.perform_commented_operations kfet.perform_deposit kfet.perform_negative_operations kfet.add_operationgroup kfet.change_operationgroup kfet.delete_operationgroup kfet.add_order kfet.change_order kfet.delete_order kfet.add_orderarticle kfet.change_orderarticle kfet.delete_orderarticle kfet.add_supplier kfet.change_supplier kfet.delete_supplier kfet.add_supplierarticle kfet.change_supplierarticle kfet.delete_supplierarticle kfet.add_transfer kfet.change_transfer kfet.delete_transfer kfet.add_transfergroup kfet.change_transfergroup kfet.delete_transfergroup kfetcms.add_memberteam kfetcms.change_memberteam kfetcms.delete_memberteam wagtailadmin.access_admin wagtailmenus.change_flatmenu ```

delobell commented

2017-09-10 02:21:36 +02:00

(Migrated from git.eleves.ens.fr)

[Les permissions du CMS ont été réattribuées au groupe K-Fêt Chef.]

manet commented

2017-09-10 02:25:33 +02:00

(Migrated from git.eleves.ens.fr)

En tout cas elles n'étaient plus dans la liste des permissions affichées, mais effectivement maintenant elles y apparaissent à nouveau.

manet commented

2017-09-10 02:33:05 +02:00

(Migrated from git.eleves.ens.fr)

removed ~35 label

delobell commented

2017-09-10 21:50:24 +02:00

(Migrated from git.eleves.ens.fr)

assigned to @delobell

delobell commented

2017-09-10 21:54:56 +02:00

(Migrated from git.eleves.ens.fr)

Même bug que #161 (qui était avec les groupes). Sans investiguer plus que cela, seules les permissions de Wagtail liées à un modèle sont supprimées (par exemple: le snippet K-Fêteux).
J'ai un fix de prêt. Il me reste à ajouter du test(s) pour éviter que cela se reproduise, ce qui sera fait ce soir ou demain.

En attendant, ne modifier pas les permissions du groupe 'Chef' pour éviter de perdre la possibilité de modifier ce snippet et donc les membres de l'équipe.

Même bug que #161 (qui était avec les groupes). Sans investiguer plus que cela, seules les permissions de Wagtail liées à un modèle sont supprimées (par exemple: le snippet `K-Fêteux`). J'ai un fix de prêt. Il me reste à ajouter du test(s) pour éviter que cela se reproduise, ce qui sera fait ce soir ou demain. En attendant, ne modifier pas les permissions du groupe 'Chef' pour éviter de perdre la possibilité de modifier ce snippet et donc les membres de l'équipe.

delobell commented

2017-09-10 22:12:48 +02:00

(Migrated from git.eleves.ens.fr)

Voir #168 au sujet des permissions de Wagtail non attribuables.
Cette issue ne concerne plus que la disparation de permissions du CMS lors de la modification d'un groupe K-Fêt via le site K-Fêt.

Voir #168 au sujet des permissions de Wagtail non attribuables. Cette issue ne concerne plus que la disparation de permissions du CMS lors de la modification d'un groupe K-Fêt via le site K-Fêt.

delobell commented

2017-09-10 22:13:28 +02:00

(Migrated from git.eleves.ens.fr)

changed title from L{-es permissions pour modifier le site n'apparaissent pas dans la liste des permissions distribuables-} to L{+'édition d'un groupe K-Fêt via son site supprime des permissions liées à Wagtail.+}

changed title from **L{-es permissions pour modifier le site n'apparaissent pas dans la liste des permissions distribuables-}** to **L{+'édition d'un groupe K-Fêt via son site supprime des permissions liées à Wagtail.+}**

delobell commented

2017-09-10 22:13:28 +02:00

(Migrated from git.eleves.ens.fr)

changed the description

delobell commented

2017-09-10 22:14:13 +02:00

(Migrated from git.eleves.ens.fr)

changed the description

delobell commented

2017-09-11 16:45:09 +02:00

(Migrated from git.eleves.ens.fr)

created branch 167-l-edition-d-un-groupe-k-fet-via-son-site-supprime-des-permissions-liees-a-wagtail

created branch [`167-l-edition-d-un-groupe-k-fet-via-son-site-supprime-des-permissions-liees-a-wagtail`](https://git.eleves.ens.fr/cof-geek/gestioCOF/compare/master...167-l-edition-d-un-groupe-k-fet-via-son-site-supprime-des-permissions-liees-a-wagtail)

delobell commented

2017-09-11 16:45:10 +02:00

(Migrated from git.eleves.ens.fr)

mentioned in merge request !251

delobell commented

2017-10-16 13:43:01 +02:00

(Migrated from git.eleves.ens.fr)

closed

delobell commented

2017-10-16 13:43:26 +02:00

(Migrated from git.eleves.ens.fr)

Fix par !252.

Rows
Columns

L'édition d'un groupe K-Fêt via son site supprime des permissions liées à Wagtail. #167