Apply review comment

This commit is contained in:
imperosol 2025-01-14 17:17:31 +01:00
parent 9272f53bea
commit 71b096f9ef
2 changed files with 18 additions and 4 deletions

View File

@ -174,11 +174,9 @@ class NewsUpdateView(PermissionOrAuthorRequiredMixin, UpdateView):
permission_required = "com.edit_news" permission_required = "com.edit_news"
def form_valid(self, form): def form_valid(self, form):
self.object = form.save() response = super().form_valid(form) # Does the saving part
IcsCalendar.make_internal() IcsCalendar.make_internal()
# Don't call `super().form_valid()`, return response
# because it would trigger a second call to `form.save()`
return HttpResponseRedirect(self.get_success_url())
def get_date_form_kwargs(self) -> dict[str, Any]: def get_date_form_kwargs(self) -> dict[str, Any]:
"""Get initial data for NewsDateForm""" """Get initial data for NewsDateForm"""

View File

@ -185,6 +185,22 @@ En plus de ces groupes, on peut noter :
Un utilisateur est automatiquement ajouté à ce groupe Un utilisateur est automatiquement ajouté à ce groupe
lors de sa première cotisation 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 ### Groupes de club
Chaque club est associé à deux groupes : Chaque club est associé à deux groupes :