From 4ab7478d1191e9ca01b6fcefe64240f80211fd3f Mon Sep 17 00:00:00 2001
From: Dylan Hillerbrand <dhillerbrand@gmail.com>
Date: Tue, 15 Oct 2024 09:32:55 -0400
Subject: [PATCH 1/9] fix(sources): modify source_completeness and
 complete_inventory display

Adjust options for Source.source_completeness field to "Full source", "Fragment", and "Reconstruction".

Adjust display for Source.complete_inventory boolean to `True` = "Full inventory" and `False` = "Partial inventory".
---
 django/cantusdb_project/main_app/forms.py           | 13 +++++--------
 django/cantusdb_project/main_app/models/source.py   |  6 +++---
 .../main_app/templates/source_create.html           |  2 +-
 .../main_app/templates/source_detail.html           |  6 +++---
 .../main_app/templates/source_edit.html             |  2 +-
 .../main_app/templates/source_list.html             |  2 +-
 6 files changed, 14 insertions(+), 17 deletions(-)

diff --git a/django/cantusdb_project/main_app/forms.py b/django/cantusdb_project/main_app/forms.py
index bea4213e8..076bb9dec 100644
--- a/django/cantusdb_project/main_app/forms.py
+++ b/django/cantusdb_project/main_app/forms.py
@@ -73,7 +73,6 @@ def label_from_instance(self, obj):
     widget = CheckboxSelectMultiple()
 
 
-
 class CantusDBLatinField(forms.CharField):
     """
     A custom CharField for chant text fields. Validates that the text
@@ -107,6 +106,7 @@ def validate(self, value):
             except ValueError as exc:
                 raise forms.ValidationError("Invalid characters in text.") from exc
 
+
 class StyledChoiceField(forms.ChoiceField):
     """
     A custom ChoiceField that uses the custom SelectWidget defined in widgets.py
@@ -116,7 +116,6 @@ class StyledChoiceField(forms.ChoiceField):
     widget = SelectWidget()
 
 
-
 class ChantCreateForm(forms.ModelForm):
     class Meta:
         model = Chant
@@ -314,7 +313,7 @@ class Meta:
         required=False,
     )
 
-    TRUE_FALSE_CHOICES_INVEN = ((True, "Complete"), (False, "Incomplete"))
+    TRUE_FALSE_CHOICES_INVEN = ((True, "Full inventory"), (False, "Partial inventory"))
 
     complete_inventory = StyledChoiceField(
         choices=TRUE_FALSE_CHOICES_INVEN, required=False
@@ -510,8 +509,8 @@ class Meta:
     )
 
     CHOICES_COMPLETE_INV = (
-        (True, "complete inventory"),
-        (False, "partial inventory"),
+        (True, "Full inventory"),
+        (False, "Partial inventory"),
     )
     complete_inventory = StyledChoiceField(choices=CHOICES_COMPLETE_INV, required=False)
 
@@ -763,7 +762,6 @@ class Meta:
     #     help_text="RISM-style siglum + Shelf-mark (e.g. GB-Ob 202).",
     # )
 
-
     shelfmark = forms.CharField(
         required=True,
         widget=TextInputWidget,
@@ -771,7 +769,6 @@ class Meta:
 
     name = forms.CharField(required=False, widget=TextInputWidget)
 
-
     holding_institution = forms.ModelChoiceField(
         queryset=Institution.objects.all().order_by("city", "name"),
         required=False,
@@ -841,7 +838,7 @@ class Meta:
         widget=FilteredSelectMultiple(verbose_name="other editors", is_stacked=False),
     )
 
-    TRUE_FALSE_CHOICES_INVEN = ((True, "Complete"), (False, "Incomplete"))
+    TRUE_FALSE_CHOICES_INVEN = ((True, "Full inventory"), (False, "Partial inventory"))
 
     complete_inventory = forms.ChoiceField(
         choices=TRUE_FALSE_CHOICES_INVEN, required=False
diff --git a/django/cantusdb_project/main_app/models/source.py b/django/cantusdb_project/main_app/models/source.py
index 2887b95f6..05fe45a03 100644
--- a/django/cantusdb_project/main_app/models/source.py
+++ b/django/cantusdb_project/main_app/models/source.py
@@ -78,14 +78,14 @@ class Source(BaseModel):
     )
 
     class SourceCompletenessChoices(models.IntegerChoices):
-        FULL_SOURCE = 1, "Full source"
-        FRAGMENT = 2, "Fragment/Fragmented"
+        FULL_SOURCE = 1, "Complete source"
+        FRAGMENT = 2, "Fragment"
         RECONSTRUCTION = 3, "Reconstruction"
 
     source_completeness = models.IntegerField(
         choices=SourceCompletenessChoices.choices,
         default=SourceCompletenessChoices.FULL_SOURCE,
-        verbose_name="Full Source/Fragment",
+        verbose_name="Complete Source/Fragment",
     )
 
     full_source = models.BooleanField(blank=True, null=True)
diff --git a/django/cantusdb_project/main_app/templates/source_create.html b/django/cantusdb_project/main_app/templates/source_create.html
index 395a373fa..99d769cff 100644
--- a/django/cantusdb_project/main_app/templates/source_create.html
+++ b/django/cantusdb_project/main_app/templates/source_create.html
@@ -162,7 +162,7 @@ <h3>Create Source</h3>
                 <div class="form-row">
                     <div class="form-group m-1 col-lg-2">
                         <label for="{{ form.complete_inventory.id_for_label }}">
-                            <small>Complete / partial inventory:</small>
+                            <small>Full / partial inventory:</small>
                         </label>
                         {{ form.complete_inventory }}
                     </div>
diff --git a/django/cantusdb_project/main_app/templates/source_detail.html b/django/cantusdb_project/main_app/templates/source_detail.html
index 60e84a39d..996a1e50d 100644
--- a/django/cantusdb_project/main_app/templates/source_detail.html
+++ b/django/cantusdb_project/main_app/templates/source_detail.html
@@ -113,11 +113,11 @@ <h3>{{ source.heading }}</h3>
     {% endif %}
 
     {% if source.complete_inventory is not None %}
-        <dt>Complete/Partial Inventory</dt>
-        <dd>{{ source.complete_inventory|yesno:"Complete Inventory,Partial Inventory" }}</dd>
+        <dt>Full/Partial Inventory</dt>
+        <dd>{{ source.complete_inventory|yesno:"Full Inventory,Partial Inventory" }}</dd>
     {% endif %}
 
-    <dt>Full Source/Fragment</dt>
+    <dt>Complete Source/Fragment</dt>
     <dd>{{ source.get_source_completeness_display }}</dd>
 
     {% if user.is_authenticated %}
diff --git a/django/cantusdb_project/main_app/templates/source_edit.html b/django/cantusdb_project/main_app/templates/source_edit.html
index 385ebc1e9..b953367b5 100644
--- a/django/cantusdb_project/main_app/templates/source_edit.html
+++ b/django/cantusdb_project/main_app/templates/source_edit.html
@@ -170,7 +170,7 @@ <h3>
         <div class="form-row mb-3">
             <div class="form-group m-1 col-lg-5">
                 <label for="{{ form.complete_inventory.id_for_label }}">
-                    Complete / partial inventory:
+                    Full / partial inventory:
                 </label>
                 {{ form.complete_inventory }}
             </div>
diff --git a/django/cantusdb_project/main_app/templates/source_list.html b/django/cantusdb_project/main_app/templates/source_list.html
index 330f48e82..37d47722b 100644
--- a/django/cantusdb_project/main_app/templates/source_list.html
+++ b/django/cantusdb_project/main_app/templates/source_list.html
@@ -70,7 +70,7 @@ <h3>Browse Sources</h3>
                 </select>
             </div>
             <div class="form-group m-1 col-lg">
-                <label for="fullSourceFilter"><small>Full source/Fragment</small></label>
+                <label for="fullSourceFilter"><small>Complete Source/Fragment</small></label>
                 <select id="fullSourceFilter"  name="fullSource" class="form-control custom-select custom-select-sm">
                     <option value="">- Any -</option>
                     <option value="true">Complete source</option>

From 9d63616d56b4d396f02d53085f8c6779a1dbf274 Mon Sep 17 00:00:00 2001
From: Dylan Hillerbrand <dhillerbrand@gmail.com>
Date: Tue, 15 Oct 2024 09:35:03 -0400
Subject: [PATCH 2/9] fix(populate_source_completeness): map full_source = None
 to source_completeness = "Full Source"

Sources where `full_source` was `None` are now mapped by the `populate_source_completeness` command to `source_completeness` = "Full source".

Updates `make_fake_source` to use `source_completeness` field.

Updates test of `populate_source_completeness` field.
---
 .../commands/populate_source_completeness.py      |  2 +-
 .../cantusdb_project/main_app/tests/make_fakes.py |  4 ++--
 .../tests/test_populate_source_completeness.py    | 15 ++++++++++++---
 3 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/django/cantusdb_project/main_app/management/commands/populate_source_completeness.py b/django/cantusdb_project/main_app/management/commands/populate_source_completeness.py
index fe7d9e279..7a16fa6f5 100644
--- a/django/cantusdb_project/main_app/management/commands/populate_source_completeness.py
+++ b/django/cantusdb_project/main_app/management/commands/populate_source_completeness.py
@@ -12,7 +12,7 @@ class Command(BaseCommand):
     def handle(self, *args, **options):
         sources = Source.objects.all()
         for source in sources:
-            if source.full_source:
+            if source.full_source or source.full_source is None:
                 source.source_completeness = (
                     source.SourceCompletenessChoices.FULL_SOURCE
                 )
diff --git a/django/cantusdb_project/main_app/tests/make_fakes.py b/django/cantusdb_project/main_app/tests/make_fakes.py
index 5991df946..c47ae5f30 100644
--- a/django/cantusdb_project/main_app/tests/make_fakes.py
+++ b/django/cantusdb_project/main_app/tests/make_fakes.py
@@ -405,7 +405,7 @@ def make_fake_source(
     summary: Optional[str] = None,
     provenance: Optional[Provenance] = None,
     century: Optional[Century] = None,
-    full_source: Optional[bool] = True,
+    source_completeness: int = Source.SourceCompletenessChoices.FULL_SOURCE,
     indexing_notes: Optional[str] = None,
 ) -> Source:
     """Generates a fake Source object."""
@@ -447,7 +447,7 @@ def make_fake_source(
         summary=summary,
         provenance=provenance,
         # century: ManyToManyField, must be set below
-        full_source=full_source,
+        source_completeness=source_completeness,
         indexing_notes=indexing_notes,
         provenance_notes=faker.sentence(),
         date=faker.sentence(nb_words=3),
diff --git a/django/cantusdb_project/main_app/tests/test_populate_source_completeness.py b/django/cantusdb_project/main_app/tests/test_populate_source_completeness.py
index 065bbb2ea..205815d79 100644
--- a/django/cantusdb_project/main_app/tests/test_populate_source_completeness.py
+++ b/django/cantusdb_project/main_app/tests/test_populate_source_completeness.py
@@ -9,15 +9,24 @@ class TestPopulateSourceCompleteness(TestCase):
     def test_populate_source_completeness(self):
         # make a few "Full Source" sources
         for _ in range(5):
-            make_fake_source(full_source=True)
+            source = make_fake_source()
+            source.full_source = True
+            source.save()
         # make a few "Fragment" sources
         for _ in range(3):
-            make_fake_source(full_source=False)
+            source = make_fake_source()
+            source.full_source = False
+            source.save()
+        # make a few "Full Source" sources with full_source=None
+        for _ in range(2):
+            source = make_fake_source()
+            source.full_source = None
+            source.save()
         # run the command
         call_command("populate_source_completeness")
         sources = Source.objects.all()
         for source in sources:
-            if source.full_source:
+            if source.full_source or source.full_source is None:
                 self.assertEqual(
                     source.source_completeness,
                     source.SourceCompletenessChoices.FULL_SOURCE,

From 7561b0ebd209adfd0298cea640300233af827795 Mon Sep 17 00:00:00 2001
From: Dylan Hillerbrand <dhillerbrand@gmail.com>
Date: Tue, 15 Oct 2024 09:35:31 -0400
Subject: [PATCH 3/9] fix(source list view): use source_completeness field to
 filter source list

---
 .../main_app/tests/test_views/test_source.py     | 16 +++++++---------
 django/cantusdb_project/main_app/views/source.py |  9 ++++++---
 2 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/django/cantusdb_project/main_app/tests/test_views/test_source.py b/django/cantusdb_project/main_app/tests/test_views/test_source.py
index 8c9e529a5..e316495a9 100644
--- a/django/cantusdb_project/main_app/tests/test_views/test_source.py
+++ b/django/cantusdb_project/main_app/tests/test_views/test_source.py
@@ -846,17 +846,17 @@ def test_filter_by_century(self):
 
     def test_filter_by_full_source(self):
         full_source = make_fake_source(
-            full_source=True,
+            source_completeness=Source.SourceCompletenessChoices.FULL_SOURCE,
             published=True,
             shelfmark="full source",
         )
         fragment = make_fake_source(
-            full_source=False,
+            source_completeness=Source.SourceCompletenessChoices.FRAGMENT,
             published=True,
             shelfmark="fragment",
         )
-        unknown = make_fake_source(
-            full_source=None,
+        reconstruction = make_fake_source(
+            source_completeness=Source.SourceCompletenessChoices.RECONSTRUCTION,
             published=True,
             shelfmark="full_source field is empty",
         )
@@ -864,18 +864,16 @@ def test_filter_by_full_source(self):
         # display full sources
         response = self.client.get(reverse("source-list"), {"fullSource": "true"})
         sources = response.context["sources"]
-        # full_source and unknown_source should be in the list, fragment should not
         self.assertIn(full_source, sources)
         self.assertNotIn(fragment, sources)
-        self.assertIn(unknown, sources)
+        self.assertNotIn(reconstruction, sources)
 
         # display fragments
         response = self.client.get(reverse("source-list"), {"fullSource": "false"})
         sources = response.context["sources"]
-        # fragment should be in the list, full_source and unknown_source should not
         self.assertNotIn(full_source, sources)
         self.assertIn(fragment, sources)
-        self.assertNotIn(unknown, sources)
+        self.assertNotIn(reconstruction, sources)
 
         # display all sources
         response = self.client.get(reverse("source-list"))
@@ -883,7 +881,7 @@ def test_filter_by_full_source(self):
         # all three should be in the list
         self.assertIn(full_source, sources)
         self.assertIn(fragment, sources)
-        self.assertIn(unknown, sources)
+        self.assertIn(reconstruction, sources)
 
     def test_search_by_title(self):
         """The "general search" field searches in `title`, `shelfmark`, `description`, and `summary`"""
diff --git a/django/cantusdb_project/main_app/views/source.py b/django/cantusdb_project/main_app/views/source.py
index 9942fd760..bf81e10e4 100644
--- a/django/cantusdb_project/main_app/views/source.py
+++ b/django/cantusdb_project/main_app/views/source.py
@@ -256,10 +256,13 @@ def get_queryset(self) -> QuerySet[Source]:
             q_obj_filter &= Q(segment__id=int(segment_id))
         if (full_source_str := self.request.GET.get("fullSource")) in ["true", "false"]:
             if full_source_str == "true":
-                full_source_q = Q(full_source=True) | Q(full_source=None)
-                q_obj_filter &= full_source_q
+                q_obj_filter &= Q(
+                    source_completeness=Source.SourceCompletenessChoices.FULL_SOURCE
+                )
             else:
-                q_obj_filter &= Q(full_source=False)
+                q_obj_filter &= Q(
+                    source_completeness=Source.SourceCompletenessChoices.FRAGMENT
+                )
 
         if general_str := self.request.GET.get("general"):
             # Strip spaces at the beginning and end. Then make list of terms split on spaces

From b00327862383dea7260d4cd2ca714b569b3d07fa Mon Sep 17 00:00:00 2001
From: Dylan Hillerbrand <dhillerbrand@gmail.com>
Date: Tue, 15 Oct 2024 10:38:10 -0400
Subject: [PATCH 4/9] fix(migrations): add migrations for adjusted
 source_completeness options

---
 .../0032_alter_source_source_completeness.py  | 26 +++++++++++++++++++
 1 file changed, 26 insertions(+)
 create mode 100644 django/cantusdb_project/main_app/migrations/0032_alter_source_source_completeness.py

diff --git a/django/cantusdb_project/main_app/migrations/0032_alter_source_source_completeness.py b/django/cantusdb_project/main_app/migrations/0032_alter_source_source_completeness.py
new file mode 100644
index 000000000..9809b8d03
--- /dev/null
+++ b/django/cantusdb_project/main_app/migrations/0032_alter_source_source_completeness.py
@@ -0,0 +1,26 @@
+# Generated by Django 4.2.14 on 2024-10-15 14:37
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("main_app", "0031_source_name_source_production_method_and_more"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="source",
+            name="source_completeness",
+            field=models.IntegerField(
+                choices=[
+                    (1, "Complete source"),
+                    (2, "Fragment"),
+                    (3, "Reconstruction"),
+                ],
+                default=1,
+                verbose_name="Complete Source/Fragment",
+            ),
+        ),
+    ]

From 55162dd9494d76cdd26c11d6af20a56134b92eef Mon Sep 17 00:00:00 2001
From: Dylan Hillerbrand <dhillerbrand@gmail.com>
Date: Tue, 15 Oct 2024 10:48:25 -0400
Subject: [PATCH 5/9] fix(source admin): remove custom `full_source` field

---
 django/cantusdb_project/main_app/forms.py | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/django/cantusdb_project/main_app/forms.py b/django/cantusdb_project/main_app/forms.py
index 076bb9dec..1c153c6ca 100644
--- a/django/cantusdb_project/main_app/forms.py
+++ b/django/cantusdb_project/main_app/forms.py
@@ -778,12 +778,6 @@ class Meta:
         queryset=Provenance.objects.all().order_by("name"),
         required=False,
     )
-    TRUE_FALSE_CHOICES_SOURCE = (
-        (True, "Full source"),
-        (False, "Fragment or Fragmented"),
-    )
-
-    full_source = forms.ChoiceField(choices=TRUE_FALSE_CHOICES_SOURCE, required=False)
 
     century = forms.ModelMultipleChoiceField(
         queryset=Century.objects.all().order_by("name"),

From a167fc7c2c1a0b6a064e544a302ceb1ce418b222 Mon Sep 17 00:00:00 2001
From: Dylan Hillerbrand <dhillerbrand@gmail.com>
Date: Tue, 15 Oct 2024 10:49:01 -0400
Subject: [PATCH 6/9] refactor(forms): define complete_inventory choices once
 in forms.py

---
 django/cantusdb_project/main_app/forms.py | 21 +++++++++++----------
 1 file changed, 11 insertions(+), 10 deletions(-)

diff --git a/django/cantusdb_project/main_app/forms.py b/django/cantusdb_project/main_app/forms.py
index 1c153c6ca..8ec2597c7 100644
--- a/django/cantusdb_project/main_app/forms.py
+++ b/django/cantusdb_project/main_app/forms.py
@@ -35,6 +35,13 @@
 # ModelForm allows to build a form directly from a model
 # see https://docs.djangoproject.com/en/3.0/topics/forms/modelforms/
 
+# Define choices for the Source model's
+# complete_inventory BooleanField
+COMPLETE_INVENTORY_FORM_CHOICES = (
+    (True, "Full inventory"),
+    (False, "Partial inventory"),
+)
+
 
 class NameModelChoiceField(forms.ModelChoiceField):
     """
@@ -313,10 +320,8 @@ class Meta:
         required=False,
     )
 
-    TRUE_FALSE_CHOICES_INVEN = ((True, "Full inventory"), (False, "Partial inventory"))
-
     complete_inventory = StyledChoiceField(
-        choices=TRUE_FALSE_CHOICES_INVEN, required=False
+        choices=COMPLETE_INVENTORY_FORM_CHOICES, required=False
     )
 
 
@@ -508,11 +513,9 @@ class Meta:
         required=False,
     )
 
-    CHOICES_COMPLETE_INV = (
-        (True, "Full inventory"),
-        (False, "Partial inventory"),
+    complete_inventory = StyledChoiceField(
+        choices=COMPLETE_INVENTORY_FORM_CHOICES, required=False
     )
-    complete_inventory = StyledChoiceField(choices=CHOICES_COMPLETE_INV, required=False)
 
 
 class SequenceEditForm(forms.ModelForm):
@@ -832,10 +835,8 @@ class Meta:
         widget=FilteredSelectMultiple(verbose_name="other editors", is_stacked=False),
     )
 
-    TRUE_FALSE_CHOICES_INVEN = ((True, "Full inventory"), (False, "Partial inventory"))
-
     complete_inventory = forms.ChoiceField(
-        choices=TRUE_FALSE_CHOICES_INVEN, required=False
+        choices=COMPLETE_INVENTORY_FORM_CHOICES, required=False
     )
 
 

From a2f98754115c2e2024a413800966e5afe767742f Mon Sep 17 00:00:00 2001
From: Dylan Hillerbrand <dhillerbrand@gmail.com>
Date: Tue, 15 Oct 2024 13:31:10 -0400
Subject: [PATCH 7/9] fix(command): add Norway to migrate_records countries

---
 .../main_app/management/commands/migrate_records.py              | 1 +
 1 file changed, 1 insertion(+)

diff --git a/django/cantusdb_project/main_app/management/commands/migrate_records.py b/django/cantusdb_project/main_app/management/commands/migrate_records.py
index bc3f26325..fca35a564 100644
--- a/django/cantusdb_project/main_app/management/commands/migrate_records.py
+++ b/django/cantusdb_project/main_app/management/commands/migrate_records.py
@@ -71,6 +71,7 @@
     "HR": "Croatia",
     "I": "Italy",
     "IRL": "Ireland",
+    "N": "Norway",
     "NL": "Netherlands",
     "NZ": "New Zealand",
     "P": "Portugal",

From 81f3c0910719cb5e02ee1396769ecd900c472271 Mon Sep 17 00:00:00 2001
From: Dylan Hillerbrand <dhillerbrand@gmail.com>
Date: Tue, 15 Oct 2024 13:39:37 -0400
Subject: [PATCH 8/9] fix(institutions): Show `[No City]` if no institution
 city

---
 .../main_app/templates/institution_detail.html              | 2 +-
 .../main_app/templates/institution_list.html                | 6 +++++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/django/cantusdb_project/main_app/templates/institution_detail.html b/django/cantusdb_project/main_app/templates/institution_detail.html
index c862ae679..3e2baa111 100644
--- a/django/cantusdb_project/main_app/templates/institution_detail.html
+++ b/django/cantusdb_project/main_app/templates/institution_detail.html
@@ -10,7 +10,7 @@
             {% include "global_search_bar.html" %}
         </object>
         <h3>{{ institution.name }} {% if institution.siglum %}({{ institution.siglum }}){% endif %}</h3>
-        <h4>{{ institution.city }}, {{ institution.country }}</h4>
+        <h4>{% if institution.city %}{{ institution.city }}{% else %}[No City]{% endif %}, {{ institution.country }}</h4>
         {% if institution_authorities %}
         <hr />
         <div class="row">
diff --git a/django/cantusdb_project/main_app/templates/institution_list.html b/django/cantusdb_project/main_app/templates/institution_list.html
index 0bc32c6f3..b4fa0c1a7 100644
--- a/django/cantusdb_project/main_app/templates/institution_list.html
+++ b/django/cantusdb_project/main_app/templates/institution_list.html
@@ -27,7 +27,11 @@ <h3>Institutions</h3>
                             {{ institution.country }}
                         </td>
                         <td>
-                            {{ institution.city }}
+                            {% if institution.city %}
+                                {{ institution.city }}
+                            {% else %}
+                                [No City]
+                            {% endif %}
                         </td>
                         <td class="text-wrap">
                             <a href="{% url "institution-detail" institution.id %}">

From 147c0f4aa6aeb47cd8c9796954e66b319a88b6b0 Mon Sep 17 00:00:00 2001
From: Dylan Hillerbrand <dhillerbrand@gmail.com>
Date: Tue, 15 Oct 2024 15:20:04 -0400
Subject: [PATCH 9/9] fix(institution detail): show colloquial name if exists

Change hardcoded name for Bower segment sources. This will be modified as part of our segment overhaul (see #1668)
---
 .../main_app/templates/institution_detail.html         | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/django/cantusdb_project/main_app/templates/institution_detail.html b/django/cantusdb_project/main_app/templates/institution_detail.html
index 3e2baa111..b22ba9bbb 100644
--- a/django/cantusdb_project/main_app/templates/institution_detail.html
+++ b/django/cantusdb_project/main_app/templates/institution_detail.html
@@ -29,14 +29,14 @@ <h5>Cantus Database</h5>
                 <table class="table table-bordered table-sm small">
                     <thead>
                         <tr>
-                            <th>Shelfmark</th>
+                            <th>Sources</th>
                         </tr>
                     </thead>
                     <tbody>
                     {% for source in cantus_sources %}
                         <tr>
                             <td>
-                                <a href="{% url "source-detail" source.id %}"><b>{{ source.shelfmark }}</b></a>
+                                <a href="{% url "source-detail" source.id %}"><b>{{ source.shelfmark }}{% if source.name %} ("{{ source.name }}"){% endif %}</b></a>
                             </td>
                         </tr>
                     {% endfor %}
@@ -47,18 +47,18 @@ <h5>Cantus Database</h5>
         
             {% if num_bower_sources > 0 %}
                 <div class="col">
-                    <h5>Bower Sequence Database</h5>
+                    <h5>Clavis Sequentiarum (Sequence Database by Calvin Bower)</h5>
                     <table class="table table-bordered table-sm small">
                         <thead>
                         <tr>
-                            <th>Shelfmark</th>
+                            <th>Sources</th>
                         </tr>
                         </thead>
                         <tbody>
                         {% for source in bower_sources %}
                             <tr>
                                 <td>
-                                    <a href="{% url "source-detail" source.id %}"><b>{{ source.shelfmark }}</b></a>
+                                    <a href="{% url "source-detail" source.id %}"><b>{{ source.shelfmark }}{% if source.name %} ("{{ source.name }}"){% endif %}</b></a>
                                 </td>
                             </tr>
                         {% endfor %}