fix crash in LegalRepresentativeContactUpdateView

Bengfort requested to merge test-legal-rep-update-view into main

This fixes two regressions in LegalRepresentativeContactUpdateView. One was caused by !2072 (merged), the other by !2070 (merged).

I actually did something similar for SubjectMixin to what I have done for StudyMixin in !2070 (merged): In order to better control the order in which get_subject(), get_study() and has_permission() are executed, I hooked them all into a new check_auth_conditions() method.

Edited by Bengfort

Merge request reports