Skip to content

Commit

Permalink
Rename primary_biopsy -> concomitant_biopsy
Browse files Browse the repository at this point in the history
  • Loading branch information
danlamanna committed Feb 26, 2024
1 parent 2cd2a1d commit 4e2ac72
Showing 1 changed file with 9 additions and 13 deletions.
22 changes: 9 additions & 13 deletions isic/ingest/models/accession.py
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,14 @@ class Meta(CreationSortedTimeStampedModel.Meta):
& ~Q(blob_name="")
| ~Q(status=AccessionStatus.SUCCEEDED),
),
CheckConstraint(
name="accession_concomitant_biopsy_diagnosis_confirm_type",
check=Q(
concomitant_biopsy=True,
metadata__diagnosis_confirm_type="histopathology",
)
| ~Q(concomitant_biopsy=True),
),
# identical lesion_id implies identical patient_id
ExclusionConstraint(
name="accession_lesion_id_patient_id_exclusion",
Expand Down Expand Up @@ -197,7 +205,7 @@ class Meta(CreationSortedTimeStampedModel.Meta):
Patient, on_delete=models.SET_NULL, null=True, blank=True, related_name="accessions"
)

primary_biopsy = models.BooleanField(default=False, db_index=True)
concomitant_biopsy = models.BooleanField(default=False, db_index=True)

objects = AccessionQuerySet.as_manager()

Expand Down Expand Up @@ -501,15 +509,3 @@ def remove_unstructured_metadata(
self.save()

return modified

def full_clean(self, *args, **kwargs):
if (
self.lesion
and self.primary_biopsy
and self.cohort.accessions.filter(lesion=self.lesion, primary_biopsy=True).exclude(
pk=self.pk
)
):
raise ValidationError("Lesion already has a primary biopsy.")

return super().full_clean(*args, **kwargs)

0 comments on commit 4e2ac72

Please sign in to comment.