Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Beheer permissions #116

Merged
merged 2 commits into from
Sep 25, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 18 additions & 0 deletions app/apps/authorisatie/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,11 @@ class TaakAnnulerenPermissie(BasisPermissie):
codenaam = "taak_annuleren"


class GebruikerLijstBekijkenPermissie(BasisPermissie):
naam = "Gebruiker lijst bekijken"
codenaam = "gebruiker_lijst_bekijken"


class GebruikerAanmakenPermissie(BasisPermissie):
naam = "Gebruiker aanmaken"
codenaam = "gebruiker_aanmaken"
Expand All @@ -51,6 +56,11 @@ class GebruikerBekijkenPermissie(BasisPermissie):
codenaam = "gebruiker_bekijken"


class GebruikerAanpassenPermissie(BasisPermissie):
naam = "Gebruiker aanpassen"
codenaam = "gebruiker_aanpassen"


class GebruikerVerwijderenPermissie(BasisPermissie):
naam = "Gebruiker verwijderen"
codenaam = "gebruiker_verwijderen"
Expand All @@ -76,6 +86,11 @@ class GebruikersgroepVerwijderenPermissie(BasisPermissie):
codenaam = "gebruikersgroep_verwijderen"


class BeheerBekijkenPermissie(BasisPermissie):
naam = "Beheer bekijken"
codenaam = "beheer_bekijken"


class MSBPermissie(BasisPermissie):
naam = "MSB toegang"
codenaam = "msb_toegang"
Expand All @@ -88,13 +103,16 @@ class MSBPermissie(BasisPermissie):
MeldingenLijstBekijkenPermissie,
TaakAanmakenPermissie,
TaakAfrondenPermissie,
GebruikerLijstBekijkenPermissie,
GebruikerAanmakenPermissie,
GebruikerBekijkenPermissie,
GebruikerAanpassenPermissie,
GebruikerVerwijderenPermissie,
GebruikersgroepToekennenPermissie,
GebruikersgroepAanmakenPermissie,
GebruikersgroepBekijkenPermissie,
GebruikersgroepVerwijderenPermissie,
BeheerBekijkenPermissie,
MSBPermissie,
)

Expand Down
18 changes: 13 additions & 5 deletions app/apps/beheer/views.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
from apps.authenticatie.forms import GebruikerAanmakenForm, GebruikerAanpassenForm
from django.contrib.auth import get_user_model
from django.contrib.auth.decorators import login_required, permission_required
from django.contrib.auth.decorators import permission_required
from django.shortcuts import render
from django.urls import reverse_lazy
from django.utils.decorators import method_decorator
Expand All @@ -20,12 +20,17 @@ def beheer(request):
)


@method_decorator(
permission_required("authorisatie.gebruiker_bekijken"), name="dispatch"
)
class GebruikerView(View):
model = Gebruiker
success_url = reverse_lazy("gebruiker_lijst")


@method_decorator(login_required, name="dispatch")
@method_decorator(
permission_required("authorisatie.gebruiker_lijst_bekijken"), name="dispatch"
)
class GebruikerLijstView(GebruikerView, ListView):
def get_context_data(self, **kwargs):
context = super().get_context_data(**kwargs)
Expand All @@ -37,7 +42,6 @@ def get_context_data(self, **kwargs):
return context


@method_decorator(login_required, name="dispatch")
class GebruikerAanmakenAanpassenView(GebruikerView):
def form_valid(self, form):
if not hasattr(form.instance, "profiel"):
Expand All @@ -54,7 +58,9 @@ def form_valid(self, form):
return super().form_valid(form)


@method_decorator(login_required, name="dispatch")
@method_decorator(
permission_required("authorisatie.gebruiker_aanpassen"), name="dispatch"
)
class GebruikerAanpassenView(GebruikerAanmakenAanpassenView, UpdateView):
form_class = GebruikerAanpassenForm
template_name = "authenticatie/gebruiker_aanpassen.html"
Expand All @@ -68,7 +74,9 @@ def get_initial(self):
return initial


@method_decorator(login_required, name="dispatch")
@method_decorator(
permission_required("authorisatie.gebruiker_aanmaken"), name="dispatch"
)
class GebruikerAanmakenView(GebruikerAanmakenAanpassenView, CreateView):
template_name = "authenticatie/gebruiker_aanmaken.html"
form_class = GebruikerAanmakenForm
2 changes: 2 additions & 0 deletions app/apps/regie/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,8 @@ def http_500(request):
def root(request):
if request.user.has_perms(["authorisatie.melding_lijst_bekijken"]):
return redirect(reverse("melding_lijst"))
if request.user.has_perms(["authorisatie.beheer_bekijken"]):
return redirect(reverse("beheer"))
return redirect(reverse("account"))


Expand Down
Loading