Merge branch 'fix/organiser-api' into 'master'

Handle update of present/payment fields separately

See merge request !1085

(cherry picked from commit e5cd7da4)

1ff181d6 Handle update of present/payment fields separately

website/events/api/viewsets.py

@@ -187,8 +187,14 @@ class RegistrationViewSet(GenericViewSet, RetrieveModelMixin,
         return super().get_serializer(*args, **kwargs)
 
     def perform_update(self, serializer):
-        super().perform_update(serializer)
         registration = serializer.instance
+
+        if services.is_organiser(self.request.member, registration.event):
+            services.update_registration_by_organiser(
+                registration,
+                self.request.member,
+                serializer.validated_data)
+
         services.update_registration(registration.member,
                                      registration.event,
                                      serializer.field_values())

website/events/services.py

@@ -232,3 +232,17 @@ def registration_fields(member, event):
     else:
         raise RegistrationError(
             _("You are not allowed to update this registration."))
+
+
+def update_registration_by_organiser(registration, member, data):
+    if not is_organiser(member, registration.event):
+        raise RegistrationError(
+            _("You are not allowed to update this registration."))
+
+    if 'payment' in data:
+        registration.payment = data['payment']
+
+    if 'present' in data:
+        registration.present = data['present']
+
+    registration.save()