diff --git a/apps/member/tables.py b/apps/member/tables.py index 8c979f08..bc40368c 100644 --- a/apps/member/tables.py +++ b/apps/member/tables.py @@ -112,7 +112,7 @@ class MembershipTable(tables.Table): fee=0, ) if PermissionBackend.check_perm(get_current_authenticated_user(), - "member:add_membership", empty_membership): # If the user has right + "member.add_membership", empty_membership): # If the user has right renew_url = reverse_lazy('member:club_renew_membership', kwargs={"pk": record.pk}) t = format_html( diff --git a/apps/member/views.py b/apps/member/views.py index 6c4673fe..5cce34f5 100644 --- a/apps/member/views.py +++ b/apps/member/views.py @@ -395,7 +395,8 @@ class ClubDetailView(ProtectQuerysetMixin, LoginRequiredMixin, DetailView): if PermissionBackend.check_perm(self.request.user, "member.change_club_membership_start", club): club.update_membership_dates() # managers list - managers = Membership.objects.filter(club=self.object, roles__name="Bureau de club")\ + managers = Membership.objects.filter(club=self.object, roles__name="Bureau de club", + date_start__lte=date.today(), date_end__gte=date.today())\ .order_by('user__last_name').all() context["managers"] = ClubManagerTable(data=managers, prefix="managers-") # transaction history @@ -641,8 +642,8 @@ class ClubAddMemberView(ProtectQuerysetMixin, ProtectedCreateView): if club.name != "Kfet" and club.parent_club and not Membership.objects.filter( user=form.instance.user, club=club.parent_club, - date_start__lte=club.parent_club.membership_start, - date_end__gte=club.parent_club.membership_end, + date_start__gte=club.parent_club.membership_start, + date_end__lte=club.parent_club.membership_end, ).exists(): form.add_error('user', _('User is not a member of the parent club') + ' ' + club.parent_club.name) error = True diff --git a/apps/permission/fixtures/initial.json b/apps/permission/fixtures/initial.json index 49d595ac..6769c0ae 100644 --- a/apps/permission/fixtures/initial.json +++ b/apps/permission/fixtures/initial.json @@ -2988,6 +2988,7 @@ 137, 138, 139, + 140, 143, 146, 147,