Fix (#839) Signs and Symptoms and Vital Signs forms unavailable with

Auditor/read-only role

Author: JonathanWillitts

CHANGES

@@ -3,9 +3,11 @@ Changes
 
 unreleased
 ------
-- fixes issues where addition of some unscheduled visits could result in
+- fix issues where addition of some unscheduled visits could result in
   a server error when trying to render subject's dashboard
   (#798, #807, #824, #827, #829, #830, #831)
+- fix Signs and Symptoms and Vital Signs forms unavailable with
+  Auditor/read-only role (#839)
 - update use of format_html for DJ6
 - remove all references in templates to `bootstrap3` folders
 - bump to edc-csf 1.0.0

effect_subject/forms/signs_and_symptoms_form.py

@@ -20,8 +20,10 @@ class SignsAndSymptomsForm(CrfModelFormMixin, ActionItemFormMixin, forms.ModelFo
 
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
-        self.fields["current_sx"].queryset = SiSx.objects.order_by("display_index")
-        self.fields["current_sx_gte_g3"].queryset = SiSx.objects.order_by("display_index")
+        if self.fields.get("current_sx"):
+            self.fields["current_sx"].queryset = SiSx.objects.order_by("display_index")
+        if self.fields.get("current_sx_gte_g3"):
+            self.fields["current_sx_gte_g3"].queryset = SiSx.objects.order_by("display_index")
 
     class Meta:
         model = SignsAndSymptoms

effect_subject/forms/vital_signs_form.py

@@ -12,8 +12,10 @@ class VitalSignsForm(CrfModelFormMixin, ActionItemCrfFormMixin, forms.ModelForm)
 
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
-        self.fields["reportable_as_ae"].choices = YES_NO
-        self.fields["patient_admitted"].choices = YES_NO
+        if self.fields.get("reportable_as_ae"):
+            self.fields["reportable_as_ae"].choices = YES_NO
+        if self.fields.get("patient_admitted"):
+            self.fields["patient_admitted"].choices = YES_NO
 
     class Meta(ActionItemCrfFormMixin.Meta):
         model = VitalSigns