Commit def555bd authored by Eliot Berriot's avatar Eliot Berriot
Browse files

Merge branch '890-assign-admin' into 'develop'

See #890: assign report to moderator on resolution

See merge request !881
parents 955f88e9 daad6a5d
......@@ -490,6 +490,14 @@ class ManageReportViewSet(
required_scope = "instance:reports"
ordering_fields = ["id", "creation_date", "handled_date"]
def perform_update(self, serializer):
is_handled = serializer.instance.is_handled
if not is_handled and serializer.validated_data.get("is_handled") is True:
# report was resolved, we assign to the mod making the request
serializer.save(assigned_to=self.request.user.actor)
else:
serializer.save()
class ManageNoteViewSet(
mixins.ListModelMixin,
......
......@@ -499,3 +499,17 @@ def test_report_update(factories, superuser_api_client):
assert response.status_code == 200
report.refresh_from_db()
assert report.is_handled is True
def test_report_update_is_handled_true_assigns(factories, superuser_api_client):
actor = superuser_api_client.user.create_actor()
report = factories["moderation.Report"]()
url = reverse(
"api:v1:manage:moderation:reports-detail", kwargs={"uuid": report.uuid}
)
response = superuser_api_client.patch(url, {"is_handled": True})
assert response.status_code == 200
report.refresh_from_db()
assert report.is_handled is True
assert report.assigned_to == actor
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment