.

atlas/admin.py

@@ -1,5 +1,13 @@
 from django.contrib import admin
-from .models import Case, Series, SeriesImage, Condition, Subspecialty, Finding, SeriesFindings
+from .models import (
+    Case,
+    Series,
+    SeriesImage,
+    Condition,
+    Subspecialty,
+    Finding,
+    SeriesFinding,
+)
 
 import tagulous.admin
 
@@ -20,7 +28,7 @@ admin.site.register(SeriesImage)
 admin.site.register(Condition)
 admin.site.register(Subspecialty)
 admin.site.register(Finding)
-admin.site.register(SeriesFindings)
+admin.site.register(SeriesFinding)
 
 
 class AtlasAdminForm(ModelForm):

atlas/forms.py

@@ -8,12 +8,7 @@ from django.forms import (
 )
 from django.forms import inlineformset_factory
 
-from atlas.models import (
+from atlas.models import Case, Series, SeriesImage, SeriesFinding
-    Case,
-    Series,
-    SeriesImage,
-    SeriesFindings
-)
 
 from anatomy.models import Modality
 
@@ -33,9 +28,9 @@ class ExaminationForm(ModelForm):
         fields = ["examination"]
 
 
-class SeriesFindingsForm(ModelForm):
+class SeriesFindingForm(ModelForm):
     class Meta:
-        model = SeriesFindings
+        model = SeriesFinding
         exclude = ["series", "annotation_json"]
 
     def __init__(self, *args, **kwargs):
@@ -47,10 +42,11 @@ class SeriesFindingsForm(ModelForm):
             # a list of primary key for the selected data.
             initial["series"] = kwargs.pop("series_id")
 
-        super(SeriesFindingsForm, self).__init__(*args, **kwargs)
+        super(SeriesFindingForm, self).__init__(*args, **kwargs)
 
         ModelForm.__init__(self, *args, **kwargs)
 
+
 class SeriesForm(ModelForm):
     class Media:
         # Django also includes a few javascript files necessary

atlas/migrations/0007_auto_20211128_1622.py

@@ -8,42 +8,80 @@ import django.db.models.deletion
 class Migration(migrations.Migration):
 
     dependencies = [
-        ('atlas', '0006_auto_20211128_1448'),
+        ("atlas", "0006_auto_20211128_1448"),
     ]
 
     operations = [
         migrations.CreateModel(
-            name='Finding',
+            name="Finding",
             fields=[
-                ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                (
-                ('name', models.CharField(max_length=255)),
+                    "id",
-                ('primary_name', models.BooleanField(default='True')),
+                    models.BigAutoField(
-                ('synonym', models.ManyToManyField(blank=True, related_name='_atlas_finding_synonym_+', to='atlas.Finding')),
+                        auto_created=True,
+                        primary_key=True,
+                        serialize=False,
+                        verbose_name="ID",
+                    ),
+                ),
+                ("name", models.CharField(max_length=255)),
+                ("primary_name", models.BooleanField(default="True")),
+                (
+                    "synonym",
+                    models.ManyToManyField(
+                        blank=True,
+                        related_name="_atlas_finding_synonym_+",
+                        to="atlas.Finding",
+                    ),
+                ),
             ],
             bases=(atlas.models.SynMixin, models.Model),
         ),
         migrations.RemoveField(
-            model_name='case',
+            model_name="case",
-            name='findings',
+            name="findings",
         ),
         migrations.AlterField(
-            model_name='case',
+            model_name="case",
-            name='description',
+            name="description",
-            field=models.TextField(blank=True, help_text='Description of the case'),
+            field=models.TextField(blank=True, help_text="Description of the case"),
         ),
         migrations.AlterField(
-            model_name='series',
+            model_name="series",
-            name='description',
+            name="description",
-            field=models.TextField(blank=True, help_text='Description of stack'),
+            field=models.TextField(blank=True, help_text="Description of stack"),
         ),
         migrations.CreateModel(
-            name='SeriesFindings',
+            name="SeriesFinding",
             fields=[
-                ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                (
-                ('description', models.TextField(blank=True, help_text='Findings on the series / stack', null=True)),
+                    "id",
-                ('annotation_json', models.TextField(blank=True, null=True)),
+                    models.BigAutoField(
-                ('findings', models.ManyToManyField(blank=True, to='atlas.Finding')),
+                        auto_created=True,
-                ('series', models.ForeignKey(null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='findings', to='atlas.series')),
+                        primary_key=True,
+                        serialize=False,
+                        verbose_name="ID",
+                    ),
+                ),
+                (
+                    "description",
+                    models.TextField(
+                        blank=True,
+                        help_text="Findings on the series / stack",
+                        null=True,
+                    ),
+                ),
+                ("annotation_json", models.TextField(blank=True, null=True)),
+                ("findings", models.ManyToManyField(blank=True, to="atlas.Finding")),
+                (
+                    "series",
+                    models.ForeignKey(
+                        null=True,
+                        on_delete=django.db.models.deletion.SET_NULL,
+                        related_name="findings",
+                        to="atlas.series",
+                    ),
+                ),
             ],
         ),
     ]

atlas/models.py

@@ -73,6 +73,7 @@ def findMiddle(input_list):
         return input_list[int(middle)]
         return (input_list[int(middle)], input_list[int(middle - 1)])
 
+
 class SynMixin(object):
     # class Meta:
     #    abstract = True
@@ -94,6 +95,7 @@ class Finding(SynMixin, models.Model):
 
     primary_name = models.BooleanField(default="True")
 
+
 class Condition(SynMixin, models.Model):
     name = models.CharField(max_length=255)
 
@@ -103,12 +105,14 @@ class Condition(SynMixin, models.Model):
 
     subspecialty = models.ManyToManyField("subspecialty", blank=True)
 
+
 class Subspecialty(models.Model):
     name = models.CharField(max_length=255)
 
     def __str__(self) -> str:
         return self.name
 
+
 @reversion.register
 class Case(models.Model):
     # class SubspecialtyChoices(models.TextChoices):
@@ -205,18 +209,22 @@ class SeriesImage(models.Model):
             info = "File is not a dicom."
         return info
 
-class SeriesFindings(models.Model):
+
+class SeriesFinding(models.Model):
 
     series = models.ForeignKey(
         "Series", related_name="findings", on_delete=models.SET_NULL, null=True
     )
-    description  = models.TextField(null=True, blank=True, help_text="Findings on the series / stack")
+    description = models.TextField(
+        null=True, blank=True, help_text="Findings on the series / stack"
+    )
     findings = models.ManyToManyField(Finding, blank=True)
     annotation_json = models.TextField(null=True, blank=True)
 
     def __str__(self) -> str:
         return f"{self.series.id}/{self.description}"
 
+
 @reversion.register
 class Series(models.Model):
     modality = models.ForeignKey(

atlas/views.py

@@ -30,7 +30,7 @@ from .forms import (
     SeriesImageFormSet,
     SeriesFormSet,
     ExaminationForm,
-    SeriesFindingsForm
+    SeriesFindingForm,
 )
 from .models import (
     Case,
@@ -38,8 +38,8 @@ from .models import (
     Examination,
     Finding,
     Subspecialty,
-    SeriesFindings,
+    SeriesFinding,
-    SeriesImage
+    SeriesImage,
 )
 from .tables import AtlasTable, SeriesTable
 from .filters import AtlasFilter, SeriesFilter
@@ -117,11 +117,14 @@ def series_detail(request, pk):
 
     # if request.user not in atlas.author.all():
     #    raise PermissionDenied
-    series_finding_form = SeriesFindingsForm(series_id=series.id)
+    series_finding_form = SeriesFindingForm(series_id=series.id)
-
 
     # logging.debug(atlas.subspecialty.first().name.all())
-    return render(request, "atlas/series.html", {"series": series, "series_finding_form": series_finding_form})
+    return render(
+        request,
+        "atlas/series.html",
+        {"series": series, "series_finding_form": series_finding_form},
+    )
 
 
 @login_required
@@ -516,21 +519,21 @@ class SeriesImagesZipView(SuperuserRequiredMixin, BaseZipView):
 
         return [i.image.file for i in series.images.all()]
 
+
 def create_series_findings(request):
     # posts = Post.objects.all()
     response_data = {}
 
-    if request.POST.get('action') == 'post':
+    if request.POST.get("action") == "post":
-        series_id = request.POST.get('series')
+        series_id = request.POST.get("series")
-        findings_ids = json.loads(request.POST.get('findings'))
+        findings_ids = json.loads(request.POST.get("findings"))
-        description = request.POST.get('description')
+        description = request.POST.get("description")
-        annotation_json = json.loads(request.POST.get('annotation_json'))
+        annotation_json = json.loads(request.POST.get("annotation_json"))
 
         series = Series.objects.get(pk=series_id)
         findings = Finding.objects.filter(pk__in=findings_ids)
 
-
+        sf = SeriesFinding.objects.create(
-        sf = SeriesFindings.objects.create(
             series=series,
             description=description,
             annotation_json=annotation_json,
@@ -540,4 +543,4 @@ def create_series_findings(request):
         return JsonResponse({"success": True})
 
     return JsonResponse({"success": False})
-    return render(request, 'create_post.html', {'posts':posts})  
+    return render(request, "create_post.html", {"posts": posts})