# Deduction views

* refactors deduction views on interventions and eco accounts from function to class based
* introduces basic checks on shared access and permission on BaseView on dispatching --> checks shall be overwritten on inheriting classes

intervention/views/deduction.py

@@ -5,51 +5,22 @@ Contact: ksp-servicestelle@sgdnord.rlp.de
 Created on: 19.08.22
 
 """
-from django.contrib.auth.decorators import login_required
-from django.utils.decorators import method_decorator
+from django.contrib.auth.mixins import LoginRequiredMixin
 
 from intervention.models import Intervention
-from konova.decorators import default_group_required, shared_access_required
 from konova.views.deduction import AbstractNewDeductionView, AbstractEditDeductionView, AbstractRemoveDeductionView
 
 
-class NewInterventionDeductionView(AbstractNewDeductionView):
-    def _custom_check(self, obj):
-        pass
-
-    model = Intervention
-    redirect_url = "intervention:detail"
-
-    @method_decorator(login_required)
-    @method_decorator(default_group_required)
-    @method_decorator(shared_access_required(Intervention, "id"))
-    def dispatch(self, request, *args, **kwargs):
-        return super().dispatch(request, *args, **kwargs)
+class NewInterventionDeductionView(LoginRequiredMixin, AbstractNewDeductionView):
+    _MODEL = Intervention
+    _REDIRECT_URL = "intervention:detail"
 
 
-class EditInterventionDeductionView(AbstractEditDeductionView):
-    def _custom_check(self, obj):
-        pass
-
-    model = Intervention
-    redirect_url = "intervention:detail"
-
-    @method_decorator(login_required)
-    @method_decorator(default_group_required)
-    @method_decorator(shared_access_required(Intervention, "id"))
-    def dispatch(self, request, *args, **kwargs):
-        return super().dispatch(request, *args, **kwargs)
+class EditInterventionDeductionView(LoginRequiredMixin, AbstractEditDeductionView):
+    _MODEL = Intervention
+    _REDIRECT_URL = "intervention:detail"
 
 
-class RemoveInterventionDeductionView(AbstractRemoveDeductionView):
-    def _custom_check(self, obj):
-        pass
-
-    model = Intervention
-    redirect_url = "intervention:detail"
-
-    @method_decorator(login_required)
-    @method_decorator(default_group_required)
-    @method_decorator(shared_access_required(Intervention, "id"))
-    def dispatch(self, request, *args, **kwargs):
-        return super().dispatch(request, *args, **kwargs)
+class RemoveInterventionDeductionView(LoginRequiredMixin, AbstractRemoveDeductionView):
+    _MODEL = Intervention
+    _REDIRECT_URL = "intervention:detail"

intervention/views/intervention.py

@@ -102,7 +102,7 @@ def new_view(request: HttpRequest):
 
 class InterventionIdentifierGeneratorView(LoginRequiredMixin, BaseIdentifierGeneratorView):
     _MODEL_CLS = Intervention
-    _REDIRECT_URL_NAME = "intervention:index"
+    _REDIRECT_URL = "intervention:index"
 
 
 @login_required