Skip to content

Commit

Permalink
issue #104 restrict access to person create/update
Browse files Browse the repository at this point in the history
  • Loading branch information
2ynn committed Jun 16, 2021
1 parent 8b926b0 commit 5311819
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions saskatoon/member/api.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@

from django.contrib.messages.views import SuccessMessageMixin
from django.contrib.auth.mixins import LoginRequiredMixin
from django.contrib.auth.mixins import LoginRequiredMixin, PermissionRequiredMixin
from django.views.generic import CreateView, UpdateView
from django.urls import reverse_lazy
from django.shortcuts import render
Expand All @@ -10,7 +10,8 @@
from harvest.models import Property
from .forms import PersonCreateForm, PersonUpdateForm

class PersonCreateView(LoginRequiredMixin, SuccessMessageMixin, CreateView):
class PersonCreateView(PermissionRequiredMixin, SuccessMessageMixin, CreateView):
permission_required = 'member.add_person'
model = Person
form_class = PersonCreateForm
template_name = 'app/generic/model_form.html'
Expand Down Expand Up @@ -50,7 +51,8 @@ def get_success_url(self):
except KeyError:
return reverse_lazy('home')

class PersonUpdateView(LoginRequiredMixin, SuccessMessageMixin, UpdateView):
class PersonUpdateView(PermissionRequiredMixin, SuccessMessageMixin, UpdateView):
permission_required = 'member.change_person'
model = Person
form_class = PersonUpdateForm
template_name = 'app/generic/model_form.html'
Expand Down

0 comments on commit 5311819

Please sign in to comment.