mirror of
https://github.com/ae-utbm/sith.git
synced 2026-09-01 18:19:21 +00:00
Compare commits
3
Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
d21c90dc72 | ||
|
|
d2f9398937 | ||
|
|
8153d7a105 |
+2
-2
@@ -46,8 +46,8 @@ class ClubAdmin(admin.ModelAdmin):
|
|||||||
@admin.register(ClubRole)
|
@admin.register(ClubRole)
|
||||||
class ClubRoleAdmin(admin.ModelAdmin):
|
class ClubRoleAdmin(admin.ModelAdmin):
|
||||||
list_display = ("name", "club", "is_board", "is_presidency")
|
list_display = ("name", "club", "is_board", "is_presidency")
|
||||||
search_fields = ("name",)
|
search_fields = ("name", "club__name")
|
||||||
autocomplete_fields = ("club",)
|
autocomplete_fields = ("club", "linked_groups")
|
||||||
list_select_related = ("club",)
|
list_select_related = ("club",)
|
||||||
list_filter = (
|
list_filter = (
|
||||||
"is_board",
|
"is_board",
|
||||||
|
|||||||
@@ -479,6 +479,13 @@ class ClubRoleCreateForm(forms.ModelForm):
|
|||||||
class ClubRoleBaseFormSet(forms.BaseInlineFormSet):
|
class ClubRoleBaseFormSet(forms.BaseInlineFormSet):
|
||||||
ordering_widget = forms.HiddenInput()
|
ordering_widget = forms.HiddenInput()
|
||||||
|
|
||||||
|
def __init__(self, *args, queryset=None, **kwargs):
|
||||||
|
if queryset is None:
|
||||||
|
queryset = self.model._default_manager
|
||||||
|
super().__init__(
|
||||||
|
*args, queryset=queryset.prefetch_related("linked_groups"), **kwargs
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
ClubRoleFormSet = forms.inlineformset_factory(
|
ClubRoleFormSet = forms.inlineformset_factory(
|
||||||
Club,
|
Club,
|
||||||
|
|||||||
@@ -0,0 +1,26 @@
|
|||||||
|
# Generated by Django 5.2.17 on 2026-09-01 14:36
|
||||||
|
|
||||||
|
from django.db import migrations, models
|
||||||
|
|
||||||
|
|
||||||
|
class Migration(migrations.Migration):
|
||||||
|
dependencies = [
|
||||||
|
("club", "0017_linktype_clublink"),
|
||||||
|
("core", "0050_alter_sithfile_moderator"),
|
||||||
|
]
|
||||||
|
|
||||||
|
operations = [
|
||||||
|
migrations.AddField(
|
||||||
|
model_name="clubrole",
|
||||||
|
name="linked_groups",
|
||||||
|
field=models.ManyToManyField(
|
||||||
|
help_text=(
|
||||||
|
"Groups that are automatically given or removed "
|
||||||
|
"to user receiving or losing this club role"
|
||||||
|
),
|
||||||
|
related_name="club_roles",
|
||||||
|
to="core.group",
|
||||||
|
verbose_name="Linked groups",
|
||||||
|
),
|
||||||
|
),
|
||||||
|
]
|
||||||
+32
-11
@@ -23,6 +23,8 @@
|
|||||||
#
|
#
|
||||||
from __future__ import annotations
|
from __future__ import annotations
|
||||||
|
|
||||||
|
import operator
|
||||||
|
from functools import reduce
|
||||||
from typing import Iterable, Self
|
from typing import Iterable, Self
|
||||||
|
|
||||||
from django.conf import settings
|
from django.conf import settings
|
||||||
@@ -282,6 +284,15 @@ class ClubRole(OrderedModel):
|
|||||||
"If the role is inactive, people joining the club won't be able to get it."
|
"If the role is inactive, people joining the club won't be able to get it."
|
||||||
),
|
),
|
||||||
)
|
)
|
||||||
|
linked_groups = models.ManyToManyField(
|
||||||
|
Group,
|
||||||
|
verbose_name=_("Linked groups"),
|
||||||
|
help_text=_(
|
||||||
|
"Groups that are automatically given or removed "
|
||||||
|
"to user receiving or losing this club role"
|
||||||
|
),
|
||||||
|
related_name="linked_roles",
|
||||||
|
)
|
||||||
|
|
||||||
order_with_respect_to = "club"
|
order_with_respect_to = "club"
|
||||||
|
|
||||||
@@ -534,7 +545,7 @@ class Membership(models.Model):
|
|||||||
def _remove_club_groups(
|
def _remove_club_groups(
|
||||||
memberships: Iterable[Membership],
|
memberships: Iterable[Membership],
|
||||||
) -> tuple[int, dict[str, int]]:
|
) -> tuple[int, dict[str, int]]:
|
||||||
"""Remove users of those memberships from the club groups.
|
"""Remove users of those memberships from the club and club role groups.
|
||||||
|
|
||||||
For example, if a user is in the Troll club board,
|
For example, if a user is in the Troll club board,
|
||||||
he is in the board group and the members group of the Troll.
|
he is in the board group and the members group of the Troll.
|
||||||
@@ -553,15 +564,19 @@ class Membership(models.Model):
|
|||||||
clubs = {m.club_id for m in memberships}
|
clubs = {m.club_id for m in memberships}
|
||||||
users = {m.user_id for m in memberships}
|
users = {m.user_id for m in memberships}
|
||||||
groups = Group.objects.filter(Q(club__in=clubs) | Q(club_board__in=clubs))
|
groups = Group.objects.filter(Q(club__in=clubs) | Q(club_board__in=clubs))
|
||||||
|
role_groups = [
|
||||||
|
Q(user_id=m.user_id, group__linked_roles=m.role_id) for m in memberships
|
||||||
|
]
|
||||||
return User.groups.through.objects.filter(
|
return User.groups.through.objects.filter(
|
||||||
Q(group__in=groups) & Q(user__in=users)
|
(Q(group__in=groups) & Q(user__in=users))
|
||||||
|
| reduce(operator.or_, role_groups)
|
||||||
).delete()
|
).delete()
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def _add_club_groups(
|
def _add_club_groups(
|
||||||
memberships: Iterable[Membership],
|
memberships: Iterable[Membership],
|
||||||
) -> list[User.groups.through]:
|
) -> list[User.groups.through]:
|
||||||
"""Add users of those memberships to the club groups.
|
"""Add users of those memberships to the club and club role groups.
|
||||||
|
|
||||||
For example, if a user just joined the Troll club board,
|
For example, if a user just joined the Troll club board,
|
||||||
he will be added in both the members group and the board group
|
he will be added in both the members group and the board group
|
||||||
@@ -582,34 +597,40 @@ class Membership(models.Model):
|
|||||||
memberships = [m for m in memberships if m.end_date is None]
|
memberships = [m for m in memberships if m.end_date is None]
|
||||||
if not memberships:
|
if not memberships:
|
||||||
return []
|
return []
|
||||||
|
nb_prefetched = sum(
|
||||||
if sum(1 for m in memberships if not hasattr(m, "club")) > 1:
|
1 for m in memberships if not hasattr(m, "club") or not hasattr(m, "role")
|
||||||
|
)
|
||||||
|
if nb_prefetched > 1:
|
||||||
# if more than one membership hasn't its `club` attribute set
|
# if more than one membership hasn't its `club` attribute set
|
||||||
# it's less expensive to reload the whole query with
|
# it's less expensive to reload the whole query with
|
||||||
# a select_related than perform a distinct query
|
# a select_related than perform a distinct query
|
||||||
# to fetch each club.
|
# to fetch each club.
|
||||||
ids = {m.id for m in memberships}
|
ids = {m.id for m in memberships}
|
||||||
memberships = list(
|
memberships = list(
|
||||||
Membership.objects.filter(id__in=ids).select_related("club")
|
Membership.objects.filter(id__in=ids)
|
||||||
|
.select_related("club", "role")
|
||||||
|
.prefetch_related("role__linked_groups")
|
||||||
)
|
)
|
||||||
club_groups = []
|
groups = []
|
||||||
for membership in memberships:
|
for membership in memberships:
|
||||||
club_groups.append(
|
groups.append(
|
||||||
User.groups.through(
|
User.groups.through(
|
||||||
user_id=membership.user_id,
|
user_id=membership.user_id,
|
||||||
group_id=membership.club.members_group_id,
|
group_id=membership.club.members_group_id,
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
if membership.role.is_board:
|
if membership.role.is_board:
|
||||||
club_groups.append(
|
groups.append(
|
||||||
User.groups.through(
|
User.groups.through(
|
||||||
user_id=membership.user_id,
|
user_id=membership.user_id,
|
||||||
group_id=membership.club.board_group_id,
|
group_id=membership.club.board_group_id,
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
return User.groups.through.objects.bulk_create(
|
groups.extend(
|
||||||
club_groups, ignore_conflicts=True
|
User.groups.through(user_id=membership.user_id, group_id=g.id)
|
||||||
|
for g in membership.role.linked_groups.all()
|
||||||
)
|
)
|
||||||
|
return User.groups.through.objects.bulk_create(groups, ignore_conflicts=True)
|
||||||
|
|
||||||
|
|
||||||
class Mailing(models.Model):
|
class Mailing(models.Model):
|
||||||
|
|||||||
@@ -49,6 +49,20 @@
|
|||||||
{{ subform.is_active.help_text }}
|
{{ subform.is_active.help_text }}
|
||||||
</span>
|
</span>
|
||||||
</div>
|
</div>
|
||||||
|
{% set groups = subform.instance.linked_groups.all()|list %}
|
||||||
|
{% if groups %}
|
||||||
|
<div>
|
||||||
|
<p>
|
||||||
|
<strong>{% trans %}Linked groups : {% endtrans %}</strong>
|
||||||
|
{{ groups|map(attribute="name")|join(", ") }}
|
||||||
|
</p>
|
||||||
|
<p class="helptext">
|
||||||
|
{% trans trimmed %}
|
||||||
|
Users receiving this role will also be assigned to those groups
|
||||||
|
{% endtrans %}
|
||||||
|
</p>
|
||||||
|
</div>
|
||||||
|
{% endif %}
|
||||||
</div>
|
</div>
|
||||||
</details>
|
</details>
|
||||||
</div>
|
</div>
|
||||||
|
|||||||
@@ -17,7 +17,7 @@ from club.forms import ClubAddMemberForm, JoinClubForm
|
|||||||
from club.models import Club, ClubRole, Membership
|
from club.models import Club, ClubRole, Membership
|
||||||
from club.tests.base import TestClub
|
from club.tests.base import TestClub
|
||||||
from core.baker_recipes import subscriber_user
|
from core.baker_recipes import subscriber_user
|
||||||
from core.models import AnonymousUser, User
|
from core.models import AnonymousUser, Group, User
|
||||||
|
|
||||||
|
|
||||||
class TestMembershipQuerySet(TestClub):
|
class TestMembershipQuerySet(TestClub):
|
||||||
@@ -500,6 +500,29 @@ class TestMembership(TestClub):
|
|||||||
assert self.subscriber.groups.contains(self.club.members_group)
|
assert self.subscriber.groups.contains(self.club.members_group)
|
||||||
assert self.subscriber.groups.contains(self.club.board_group)
|
assert self.subscriber.groups.contains(self.club.board_group)
|
||||||
|
|
||||||
|
def test_add_to_club_role_group(self):
|
||||||
|
groups = baker.make(Group, _quantity=4)
|
||||||
|
self.subscriber.groups.set(groups[:1])
|
||||||
|
self.board_role.linked_groups.set(groups[1:3])
|
||||||
|
baker.make(
|
||||||
|
Membership, club=self.club, user=self.subscriber, role=self.board_role
|
||||||
|
)
|
||||||
|
assert set(self.subscriber.groups.all()) == {
|
||||||
|
*groups[:3],
|
||||||
|
self.club.board_group,
|
||||||
|
self.club.members_group,
|
||||||
|
}
|
||||||
|
|
||||||
|
def test_remove_from_club_role_group(self):
|
||||||
|
groups = baker.make(Group, _quantity=3)
|
||||||
|
baker.make(
|
||||||
|
Membership, club=self.club, user=self.subscriber, role=self.board_role
|
||||||
|
)
|
||||||
|
self.subscriber.groups.set(groups[:1])
|
||||||
|
self.board_role.linked_groups.set(groups[1:])
|
||||||
|
self.subscriber.memberships.update(end_date=localdate())
|
||||||
|
assert set(self.subscriber.groups.all()) == {groups[0]}
|
||||||
|
|
||||||
def test_change_position_in_club(self):
|
def test_change_position_in_club(self):
|
||||||
"""Test that when moving from board to members, club group change"""
|
"""Test that when moving from board to members, club group change"""
|
||||||
membership = baker.make(
|
membership = baker.make(
|
||||||
|
|||||||
@@ -6,7 +6,7 @@
|
|||||||
msgid ""
|
msgid ""
|
||||||
msgstr ""
|
msgstr ""
|
||||||
"Report-Msgid-Bugs-To: \n"
|
"Report-Msgid-Bugs-To: \n"
|
||||||
"POT-Creation-Date: 2026-09-01 18:32+0200\n"
|
"POT-Creation-Date: 2026-09-01 18:25+0200\n"
|
||||||
"PO-Revision-Date: 2016-07-18\n"
|
"PO-Revision-Date: 2016-07-18\n"
|
||||||
"Last-Translator: Maréchal <thomas.girod@utbm.fr\n"
|
"Last-Translator: Maréchal <thomas.girod@utbm.fr\n"
|
||||||
"Language-Team: AE info <ae.info@utbm.fr>\n"
|
"Language-Team: AE info <ae.info@utbm.fr>\n"
|
||||||
@@ -260,6 +260,18 @@ msgstr ""
|
|||||||
"Si ce rôle est inactif, il ne pourra pas être attribué aux gens qui "
|
"Si ce rôle est inactif, il ne pourra pas être attribué aux gens qui "
|
||||||
"rejoignent le club."
|
"rejoignent le club."
|
||||||
|
|
||||||
|
#: club/models.py
|
||||||
|
msgid "Linked groups"
|
||||||
|
msgstr "Groupes liés"
|
||||||
|
|
||||||
|
#: club/models.py
|
||||||
|
msgid ""
|
||||||
|
"Groups that are automatically given or removed to user receiving or losing "
|
||||||
|
"this club role"
|
||||||
|
msgstr ""
|
||||||
|
"Les groupes qui sont automatiquement donnés ou retirés quand l'utilisateur "
|
||||||
|
"reçoit ou perd ce rôle de club"
|
||||||
|
|
||||||
#: club/models.py election/models.py
|
#: club/models.py election/models.py
|
||||||
msgid "club role"
|
msgid "club role"
|
||||||
msgstr "rôle de club"
|
msgstr "rôle de club"
|
||||||
@@ -485,6 +497,14 @@ msgstr "Du"
|
|||||||
msgid "To"
|
msgid "To"
|
||||||
msgstr "Au"
|
msgstr "Au"
|
||||||
|
|
||||||
|
#: club/templates/club/club_roles.jinja
|
||||||
|
msgid "Linked groups : "
|
||||||
|
msgstr "Groupes liés : "
|
||||||
|
|
||||||
|
#: club/templates/club/club_roles.jinja
|
||||||
|
msgid "Users receiving this role will also be assigned to those groups"
|
||||||
|
msgstr "Les utilisateurs recevant ce rôle seront aussi assignés à ces groupes"
|
||||||
|
|
||||||
#: club/templates/club/club_roles.jinja
|
#: club/templates/club/club_roles.jinja
|
||||||
msgid ""
|
msgid ""
|
||||||
"Roles give rights on the club. Higher roles grant more rights, and the "
|
"Roles give rights on the club. Higher roles grant more rights, and the "
|
||||||
|
|||||||
@@ -1,43 +0,0 @@
|
|||||||
# Generated by Django 5.2.17 on 2026-09-01 08:44
|
|
||||||
|
|
||||||
import django.utils.timezone
|
|
||||||
from django.db import migrations, models
|
|
||||||
from django.db.migrations.state import StateApps
|
|
||||||
from django.db.models import F, Value
|
|
||||||
from django.db.models.functions import Least
|
|
||||||
from django.utils.timezone import now
|
|
||||||
|
|
||||||
|
|
||||||
def make_default_creation_date(apps: StateApps, schema_editor):
|
|
||||||
Subscription = apps.get_model("subscription", "Subscription")
|
|
||||||
Subscription.objects.update(
|
|
||||||
created_at=Least(
|
|
||||||
F("subscription_start"), Value(now()), output_field=models.DateTimeField()
|
|
||||||
)
|
|
||||||
)
|
|
||||||
|
|
||||||
|
|
||||||
class Migration(migrations.Migration):
|
|
||||||
dependencies = [("subscription", "0016_alter_subscription_subscription_type")]
|
|
||||||
|
|
||||||
operations = [
|
|
||||||
migrations.AddField(
|
|
||||||
model_name="subscription",
|
|
||||||
name="created_at",
|
|
||||||
field=models.DateTimeField(
|
|
||||||
auto_now_add=True,
|
|
||||||
default=django.utils.timezone.now,
|
|
||||||
help_text=(
|
|
||||||
"When this subscription was created. "
|
|
||||||
"This date may differ from the start of the subscription."
|
|
||||||
),
|
|
||||||
verbose_name="created_at",
|
|
||||||
),
|
|
||||||
preserve_default=False,
|
|
||||||
),
|
|
||||||
migrations.RunPython(
|
|
||||||
make_default_creation_date,
|
|
||||||
reverse_code=migrations.RunPython.noop,
|
|
||||||
elidable=True,
|
|
||||||
),
|
|
||||||
]
|
|
||||||
@@ -64,14 +64,6 @@ class Subscription(models.Model):
|
|||||||
max_length=20,
|
max_length=20,
|
||||||
verbose_name=_("location"),
|
verbose_name=_("location"),
|
||||||
)
|
)
|
||||||
created_at = models.DateTimeField(
|
|
||||||
_("created_at"),
|
|
||||||
help_text=_(
|
|
||||||
"When this subscription was created. "
|
|
||||||
"This date may differ from the start of the subscription."
|
|
||||||
),
|
|
||||||
auto_now_add=True,
|
|
||||||
)
|
|
||||||
|
|
||||||
class Meta:
|
class Meta:
|
||||||
ordering = ["subscription_start"]
|
ordering = ["subscription_start"]
|
||||||
|
|||||||
Reference in New Issue
Block a user