From 71b096f9ef3de8c34aefe9e798d90f7a496fa10e Mon Sep 17 00:00:00 2001 From: imperosol Date: Tue, 14 Jan 2025 17:17:31 +0100 Subject: [PATCH] Apply review comment --- com/views.py | 6 ++---- docs/tutorial/groups.md | 16 ++++++++++++++++ 2 files changed, 18 insertions(+), 4 deletions(-) diff --git a/com/views.py b/com/views.py index 7d29dd62..a6faf214 100644 --- a/com/views.py +++ b/com/views.py @@ -174,11 +174,9 @@ class NewsUpdateView(PermissionOrAuthorRequiredMixin, UpdateView): permission_required = "com.edit_news" def form_valid(self, form): - self.object = form.save() + response = super().form_valid(form) # Does the saving part IcsCalendar.make_internal() - # Don't call `super().form_valid()`, - # because it would trigger a second call to `form.save()` - return HttpResponseRedirect(self.get_success_url()) + return response def get_date_form_kwargs(self) -> dict[str, Any]: """Get initial data for NewsDateForm""" diff --git a/docs/tutorial/groups.md b/docs/tutorial/groups.md index 9e310c75..bccd713f 100644 --- a/docs/tutorial/groups.md +++ b/docs/tutorial/groups.md @@ -185,6 +185,22 @@ En plus de ces groupes, on peut noter : Un utilisateur est automatiquement ajouté à ce groupe lors de sa première cotisation +!!!note "Utilisation du groupe Public" + + Le groupe Public est un groupe particulier. + Tout le monde faisant partie de ce groupe + (même les utilisateurs non-connectés en sont implicitement + considérés comme membres), + il ne doit pas être utilisé pour résoudre les + permissions d'une vue. + + En revanche, il est utile pour attribuer une ressource + à tout le monde. + Par exemple, un produit avec le groupe de vente Public + est considéré comme achetable par tous utilisateurs. + S'il n'avait eu aucun group de vente, il n'aurait + été accessible à personne. + ### Groupes de club Chaque club est associé à deux groupes :