Error accepting transfer (3354)

[mbarton]

Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/django/core/handlers/exception.py", line 55, in inner
    response = get_response(request)
               ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/django/core/handlers/base.py", line 197, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/django/contrib/auth/decorators.py", line 59, in _view_wrapper
    return view_func(request, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/app/epilepsy12/decorator.py", line 402, in wrapper
    return view(request, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/app/epilepsy12/decorator.py", line 311, in wrapper
    return view(request, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/django/contrib/auth/decorators.py", line 59, in _view_wrapper
    return view_func(request, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/app/epilepsy12/views/case_views.py", line 498, in transfer_response
    site = Site.objects.get(
          
  File "/usr/local/lib/python3.11/site-packages/django/db/models/manager.py", line 87, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/django/db/models/query.py", line 635, in get
    raise self.model.DoesNotExist(
    ^

Exception Type: DoesNotExist at /organisation/136/case/3354/transfer-request/accept

[mbarton]

This appears to be crashing because there is no Site object for that patient to NORTH WEST BOROUGHS HEALTHCARE NHS FOUNDATION TRUST (id 136). I think in a transfer the new site object should be created when the transfer is initiated.

This patient has also been transferred previously.

[eatyourpeas]

This child does have an active site (which is the primary centre of epilepsy care and actively involved which is SOUTH TYNESIDE AND SUNDERLAND NHS FOUNDATION TRUST) which has the active_transfer set to False. Somehow the transfer_response endpoint has been called for this child (either to accept or reject), but that should not be possible if the active_transfer flag is negative. In the UI, it should only be cases where this flag is True that the accept/reject buttons should be shown.

[mbarton]

Looked at this again I was mistaken in my original analysis, I was looking up trust with the ID of 136 when it should have been the organisation.

Running it through again, it looks like the case was transferred away and then back to the same organisation:

organisation=136 site_is_actively_involved_in_epilepsy_care=False site_is_primary_centre_of_epilepsy_care=True
organisation=327 site_is_actively_involved_in_epilepsy_care=False site_is_primary_centre_of_epilepsy_care=True
organisation=136 site_is_actively_involved_in_epilepsy_care=True site_is_primary_centre_of_epilepsy_care=True

We need to investigate why they are still showing as an active transfer but we think it's likely they are in the right place so this issue doesn't block cohort 7 analysis.