Add prijs change permission condition

src/open_producten/producttypen/admin/prijs.py

@@ -1,6 +1,7 @@
 from django.contrib import admin
 from django.core.exceptions import ValidationError
 from django.forms import BaseInlineFormSet
+from django.utils.datetime_safe import datetime
 from django.utils.translation import gettext_lazy as _
 
 from ..models import Prijs, PrijsOptie
@@ -36,3 +37,8 @@ class PrijsAdmin(admin.ModelAdmin):
 
     def get_queryset(self, request):
         return super().get_queryset(request).select_related("product_type")
+
+    def has_change_permission(self, request, obj=None):
+        if obj and obj.actief_vanaf < datetime.today().date():
+            return False
+        return super().has_change_permission(request, obj)

src/open_producten/producttypen/tests/test_prijs.py

@@ -1,9 +1,16 @@
 from datetime import date, timedelta
 from decimal import Decimal
 
+from django.contrib.admin.sites import AdminSite
 from django.core.exceptions import ValidationError
 from django.test import TestCase
 
+from freezegun import freeze_time
+
+from open_producten.producttypen.admin import PrijsAdmin
+from open_producten.producttypen.models import Prijs
+
+from ...accounts.tests.factories import UserFactory
 from .factories import PrijsFactory, PrijsOptieFactory, ProductTypeFactory
 
 
@@ -26,6 +33,33 @@ def test_min_date_validation(self):
             prijs = PrijsFactory.build(actief_vanaf=date(2020, 1, 1))
             prijs.full_clean()
 
+    @freeze_time("2024-01-02")
+    def test_change_permission_on_old_price(self):
+        instance = PrijsFactory.create(actief_vanaf=date(2024, 1, 1))
+        admin = PrijsAdmin(Prijs, AdminSite())
+        request = self.client.request()
+        request.user = UserFactory(superuser=True)
+
+        self.assertFalse(admin.has_change_permission(request, instance))
+
+    @freeze_time("2024-01-02")
+    def test_change_permission_on_future_price(self):
+        instance = PrijsFactory.create(actief_vanaf=date(2024, 3, 1))
+        admin = PrijsAdmin(Prijs, AdminSite())
+        request = self.client.request()
+        request.user = UserFactory(superuser=True)
+
+        self.assertTrue(admin.has_change_permission(request, instance))
+
+    @freeze_time("2024-01-02")
+    def test_change_permission_on_current_price(self):
+        instance = PrijsFactory.create(actief_vanaf=date(2024, 1, 2))
+        admin = PrijsAdmin(Prijs, AdminSite())
+        request = self.client.request()
+        request.user = UserFactory(superuser=True)
+
+        self.assertTrue(admin.has_change_permission(request, instance))
+
 
 class TestPrijsOptie(TestCase):
     def setUp(self):