From 642d0799f2fb336fd4e28f511ae70ff6219a0214 Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Mon, 3 Nov 2025 08:30:13 -0500
Subject: [PATCH 01/17] Update Primo normalization to use symbols

---
 app/models/normalize_primo_record.rb        |  54 +++++-----
 app/views/search/_result_primo.html.erb     |  18 ++--
 test/controllers/search_controller_test.rb  |  10 +-
 test/models/normalize_primo_record_test.rb  | 110 ++++++++++----------
 test/models/normalize_primo_results_test.rb |   4 +-
 5 files changed, 98 insertions(+), 98 deletions(-)

diff --git a/app/models/normalize_primo_record.rb b/app/models/normalize_primo_record.rb
index d967eb33..cac88518 100644
--- a/app/models/normalize_primo_record.rb
+++ b/app/models/normalize_primo_record.rb
@@ -8,25 +8,25 @@ def initialize(record, query)
   def normalize
     {
       # Core fields
-      'title' => title,
-      'creators' => creators,
-      'source' => source,
-      'year' => year,
-      'format' => format,
-      'links' => links,
-      'citation' => citation,
-      'identifier' => record_id,
-      'summary' => summary,
-      'publisher' => publisher,
-      'location' => best_location,
-      'subjects' => subjects,
+      title:,
+      creators:,
+      source:,
+      year:,
+      format:,
+      links:,
+      citation:,
+      identifier:,
+      summary:,
+      publisher:,
+      location:,
+      subjects:,
       # Primo-specific fields
-      'container' => container_title,
-      'numbering' => numbering,
-      'chapter_numbering' => chapter_numbering,
-      'thumbnail' => thumbnail,
-      'availability' => best_availability,
-      'other_availability' => other_availability?
+      container:,
+      numbering:,
+      chapter_numbering:,
+      thumbnail:,
+      availability:,
+      other_availability:
     }
   end
 
@@ -115,7 +115,7 @@ def citation
     end
   end
 
-  def container_title
+  def container
     return unless @record['pnx']['addata']
 
     if @record['pnx']['addata']['jtitle'].present?
@@ -125,7 +125,7 @@ def container_title
     end
   end
 
-  def record_id
+  def identifier
     return unless @record['pnx']['control']['recordid']
 
     @record['pnx']['control']['recordid'].join
@@ -224,7 +224,7 @@ def openurl
     if openurl_server == record_openurl_server
       construct_primo_openurl
     else
-      Rails.logger.warn "Alma openurl server mismatch. Expected #{openurl_server}, but received #{record_openurl_server}. (record ID: #{record_id})"
+      Rails.logger.warn "Alma openurl server mismatch. Expected #{openurl_server}, but received #{record_openurl_server}. (record ID: #{identifier})"
       @record['delivery']['almaOpenurl']
     end
   end
@@ -265,7 +265,7 @@ def publisher
     @record['pnx']['addata']['pub'].first
   end
 
-  def best_location
+  def location
     return unless @record['delivery']
     return unless @record['delivery']['bestlocation']
 
@@ -279,13 +279,13 @@ def subjects
     @record['pnx']['display']['subject']
   end
 
-  def best_availability
-    return unless best_location
+  def availability
+    return unless location
 
     @record['delivery']['bestlocation']['availabilityStatus']
   end
 
-  def other_availability?
+  def other_availability
     return unless @record['delivery']['bestlocation']
     return unless @record['delivery']['holding']
 
@@ -302,9 +302,9 @@ def frbrized?
   end
 
   def alma_record?
-    return false unless record_id
+    return false unless identifier
 
-    record_id.start_with?('alma')
+    identifier.start_with?('alma')
   end
 
   def dedup_url
diff --git a/app/views/search/_result_primo.html.erb b/app/views/search/_result_primo.html.erb
index bd4c5bba..89a9cbb7 100644
--- a/app/views/search/_result_primo.html.erb
+++ b/app/views/search/_result_primo.html.erb
@@ -2,22 +2,22 @@
   <div class="result-content">
     <h3 class="record-title">
       <span class="sr">Title: </span>
-      <% if result['links']&.find { |link| link['kind'] == 'full record' } %>
-        <%= link_to(result['title'], result['links'].find { |link| link['kind'] == 'full record' }['url']) %>
+      <% if result[:links]&.find { |link| link['kind'] == 'full record' } %>
+        <%= link_to(result[:title], result[:links].find { |link| link['kind'] == 'full record' }['url']) %>
       <% else %>
-        <%= result['title'] %>
+        <%= result[:title] %>
       <% end %>
     </h3>
 
     <p class="pub-info">
-      <span><%= result['format'] %></span>
-      <span><%= result['year'] %></span>
+      <span><%= result[:format] %></span>
+      <span><%= result[:year] %></span>
     </p>
 
-    <% if result['creators'].present? %>
+    <% if result[:creators].present? %>
       <span class="sr">Contributors: </span>
       <ul class="list-inline truncate-list contributors">
-        <% result['creators'].each do |creator| %>
+        <% result[:creators].each do |creator| %>
           <li>
             <% if creator[:link] %>
               <%= link_to creator[:value], creator[:link]  %>
@@ -30,8 +30,8 @@
     <% end %>
 
     <div class="result-get">
-      <% if result['links'].present? %>
-        <% result['links'].each do |link| %>
+      <% if result[:links].present? %>
+        <% result[:links].each do |link| %>
           <%= link_to link['kind'].titleize, link['url'], class: 'link-button' %>
         <% end %>
       <% end %>
diff --git a/test/controllers/search_controller_test.rb b/test/controllers/search_controller_test.rb
index 4f5d0615..1f4d3bf3 100644
--- a/test/controllers/search_controller_test.rb
+++ b/test/controllers/search_controller_test.rb
@@ -4,14 +4,14 @@ class SearchControllerTest < ActionDispatch::IntegrationTest
   def mock_primo_search_success
     # Mock the Primo search components to avoid external API calls
     sample_doc = {
-      'title' => 'Sample Primo Document Title',
-      'format' => 'Article',
-      'year' => '2025',
-      'creators' => [
+      title: 'Sample Primo Document Title',
+      format: 'Article',
+      year: '2025',
+      creators: [
         { value: 'Foo Barston', link: nil },
         { value: 'Baz Quxley', link: nil }
       ],
-      'links' => [{ 'kind' => 'full record', 'url' => 'https://example.com/record' }]
+      links: [{ 'kind' => 'full record', 'url' => 'https://example.com/record' }]
     }
 
     mock_primo = mock('primo_search')
diff --git a/test/models/normalize_primo_record_test.rb b/test/models/normalize_primo_record_test.rb
index f7992551..b8e1740d 100644
--- a/test/models/normalize_primo_record_test.rb
+++ b/test/models/normalize_primo_record_test.rb
@@ -22,128 +22,128 @@ def cdi_record
 
   test 'normalizes title' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    assert_equal 'Testing the Limits of Knowledge', normalized['title']
+    assert_equal 'Testing the Limits of Knowledge', normalized[:title]
   end
 
   test 'handles missing title' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_equal 'unknown title', normalized['title']
+    assert_equal 'unknown title', normalized[:title]
   end
 
   test 'normalizes creators' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
     expected_creators = ['Smith, John A.', 'Jones, Mary B.', 'Brown, Robert C.']
-    assert_equal expected_creators.sort, normalized['creators'].map { |c| c[:value] }.sort
+    assert_equal expected_creators.sort, normalized[:creators].map { |c| c[:value] }.sort
   end
 
   test 'handles missing creators' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_empty normalized['creators']
+    assert_empty normalized[:creators]
   end
 
   test 'normalizes source' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    assert_equal 'Primo', normalized['source']
+    assert_equal 'Primo', normalized[:source]
   end
 
   test 'normalizes year' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    assert_equal '2023', normalized['year']
+    assert_equal '2023', normalized[:year]
   end
 
   test 'handles missing year' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['year']
+    assert_nil normalized[:year]
   end
 
   test 'normalizes format' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    assert_equal 'Book', normalized['format']
+    assert_equal 'Book', normalized[:format]
   end
 
   test 'handles missing format' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['format']
+    assert_nil normalized[:format]
   end
 
   test 'normalizes links' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
 
     # First link should be the record link
-    assert_equal 'full record', normalized['links'].first['kind']
+    assert_equal 'full record', normalized[:links].first['kind']
 
     # Second link should be the Alma openurl
-    assert_equal 'openurl', normalized['links'].second['kind']
+    assert_equal 'openurl', normalized[:links].second['kind']
   end
 
   test 'handles missing links' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_empty normalized['links']
+    assert_empty normalized[:links]
   end
 
   test 'normalizes citation' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    assert_equal 'volume 2 issue 3', normalized['citation']
+    assert_equal 'volume 2 issue 3', normalized[:citation]
   end
 
   test 'handles missing citation' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['citation']
+    assert_nil normalized[:citation]
   end
 
   test 'normalizes container title' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    assert_equal 'Journal of Testing', normalized['container']
+    assert_equal 'Journal of Testing', normalized[:container]
   end
 
   test 'handles missing container title' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['container']
+    assert_nil normalized[:container]
   end
 
   test 'normalizes identifier' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    assert_equal 'alma991000000001234567', normalized['identifier']
+    assert_equal 'alma991000000001234567', normalized[:identifier]
   end
 
   test 'normalizes summary' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    assert_equal 'A comprehensive study of testing methodologies', normalized['summary']
+    assert_equal 'A comprehensive study of testing methodologies', normalized[:summary]
   end
 
   test 'handles missing summary' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['summary']
+    assert_nil normalized[:summary]
   end
 
   test 'handles missing identifier' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['identifier']
+    assert_nil normalized[:identifier]
   end
 
   test 'normalizes numbering' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    assert_equal 'volume 2 issue 3', normalized['numbering']
+    assert_equal 'volume 2 issue 3', normalized[:numbering]
   end
 
   test 'handles missing numbering' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['numbering']
+    assert_nil normalized[:numbering]
   end
 
   test 'normalizes chapter numbering' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    assert_equal '2023, pp. 123-145', normalized['chapter_numbering']
+    assert_equal '2023, pp. 123-145', normalized[:chapter_numbering]
   end
 
   test 'handles missing chapter numbering' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['chapter_numbering']
+    assert_nil normalized[:chapter_numbering]
   end
 
   test 'includes FRBRized dedup record link in links when available' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    record_link = normalized['links'].find { |link| link['kind'] == 'full record' }
+    record_link = normalized[:links].find { |link| link['kind'] == 'full record' }
     assert_not_nil record_link
 
     # For FRBRized records, should use dedup URL format
@@ -154,69 +154,69 @@ def cdi_record
   test 'generates thumbnail from ISBN' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
     expected_url = 'https://syndetics.com/index.php?client=primo&isbn=9781234567890/sc.jpg'
-    assert_equal expected_url, normalized['thumbnail']
+    assert_equal expected_url, normalized[:thumbnail]
   end
 
   test 'handles missing ISBN for thumbnail' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['thumbnail']
+    assert_nil normalized[:thumbnail]
   end
 
   test 'extracts publisher information' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    assert_equal 'MIT Press', normalized['publisher']
+    assert_equal 'MIT Press', normalized[:publisher]
   end
 
   test 'handles missing publisher' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['publisher']
+    assert_nil normalized[:publisher]
   end
 
   test 'returns best location with call number' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
     expected_location = ['Hayden Library Stacks', 'QA76.73.R83 2023']
-    assert_equal expected_location, normalized['location']
+    assert_equal expected_location, normalized[:location]
   end
 
   test 'handles missing location' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['location']
+    assert_nil normalized[:location]
   end
 
   test 'extracts subjects' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
     expected_subjects = ['Computer Science', 'Software Testing']
-    assert_equal expected_subjects, normalized['subjects']
+    assert_equal expected_subjects, normalized[:subjects]
   end
 
   test 'handles missing subjects' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_empty normalized['subjects']
+    assert_empty normalized[:subjects]
   end
 
   test 'returns availability status' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    assert_equal 'available', normalized['availability']
+    assert_equal 'available', normalized[:availability]
   end
 
   test 'handles missing availability' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['availability']
+    assert_nil normalized[:availability]
   end
 
   test 'detects other availability when multiple holdings exist' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    assert normalized['other_availability']
+    assert normalized[:other_availability]
   end
 
   test 'handles missing other availability' do
     normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['other_availability']
+    assert_nil normalized[:other_availability]
   end
 
   test 'uses dedup URL as full record link for frbrized records' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    full_record_link = normalized['links'].find { |link| link['kind'] == 'full record' }
+    full_record_link = normalized[:links].find { |link| link['kind'] == 'full record' }
     assert_not_nil full_record_link
 
     expected_base = 'https://mit.primo.exlibrisgroup.com/discovery/search?'
@@ -230,18 +230,18 @@ def cdi_record
     record_without_frbr['pnx']['facets']['frbrtype'] = ['3']
 
     normalized = NormalizePrimoRecord.new(record_without_frbr, 'test').normalize
-    full_record_link = normalized['links'].find { |link| link['kind'] == 'full record' }
+    full_record_link = normalized[:links].find { |link| link['kind'] == 'full record' }
     assert_not_nil full_record_link
     assert_match %r{/discovery/fulldisplay\?}, full_record_link['url']
   end
 
   test 'includes expected link types when available' do
     normalized = NormalizePrimoRecord.new(full_record, 'test query').normalize
-    link_kinds = normalized['links'].map { |link| link['kind'] }
+    link_kinds = normalized[:links].map { |link| link['kind'] }
 
     assert_includes link_kinds, 'full record'
     assert_includes link_kinds, 'openurl'
-    assert_equal 2, normalized['links'].length # Only full record and openurl
+    assert_equal 2, normalized[:links].length # Only full record and openurl
   end
 
   # Additional coverage tests for existing methods
@@ -249,7 +249,7 @@ def cdi_record
     record = full_record.deep_dup
     record['pnx']['display']['creator'] = ['Smith, John A.; Doe, Jane B.']
     normalized = NormalizePrimoRecord.new(record, 'test').normalize
-    creators = normalized['creators'].map { |c| c[:value] }
+    creators = normalized[:creators].map { |c| c[:value] }
     assert_includes creators, 'Smith, John A.'
     assert_includes creators, 'Doe, Jane B.'
   end
@@ -258,12 +258,12 @@ def cdi_record
     record = full_record.deep_dup
     record['pnx']['display']['creator'] = ['Smith, John A.$$QAuthor']
     normalized = NormalizePrimoRecord.new(record, 'test').normalize
-    assert_equal 'Smith, John A.', normalized['creators'].first[:value]
+    assert_equal 'Smith, John A.', normalized[:creators].first[:value]
   end
 
   test 'constructs author search links' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    creator_link = normalized['creators'].first[:link]
+    creator_link = normalized[:creators].first[:link]
     assert_match 'discovery/search?query=creator,exact,', creator_link
     assert_match 'Smith%2C%20John%20A.', creator_link
   end
@@ -280,7 +280,7 @@ def cdi_record
       record = full_record.deep_dup
       record['pnx']['display']['type'] = [input]
       normalized = NormalizePrimoRecord.new(record, 'test').normalize
-      assert_equal expected, normalized['format']
+      assert_equal expected, normalized[:format]
     end
   end
 
@@ -289,7 +289,7 @@ def cdi_record
     record['pnx']['search'] = { 'creationdate' => ['2022'] }
 
     normalized = NormalizePrimoRecord.new(record, 'test').normalize
-    assert_equal '2022', normalized['year']
+    assert_equal '2022', normalized[:year]
   end
 
   test 'handles different citation formats' do
@@ -297,12 +297,12 @@ def cdi_record
     record = full_record.deep_dup
     record['pnx']['addata'] = { 'volume' => ['5'] }
     normalized = NormalizePrimoRecord.new(record, 'test').normalize
-    assert_equal 'volume 5', normalized['citation']
+    assert_equal 'volume 5', normalized[:citation]
 
     # Test with date and pages
     record['pnx']['addata'] = { 'date' => ['2023'], 'pages' => ['10-20'] }
     normalized = NormalizePrimoRecord.new(record, 'test').normalize
-    assert_equal '2023, pp. 10-20', normalized['citation']
+    assert_equal '2023, pp. 10-20', normalized[:citation]
   end
 
   test 'prefers jtitle over btitle for container' do
@@ -311,7 +311,7 @@ def cdi_record
     record['pnx']['addata']['btitle'] = ['Book Title']
 
     normalized = NormalizePrimoRecord.new(record, 'test').normalize
-    assert_equal 'Journal Title', normalized['container']
+    assert_equal 'Journal Title', normalized[:container]
   end
 
   test 'uses btitle when jtitle is not present for container' do
@@ -320,20 +320,20 @@ def cdi_record
     record['pnx']['addata']['btitle'] = ['Book Title Only']
 
     normalized = NormalizePrimoRecord.new(record, 'test').normalize
-    assert_equal 'Book Title Only', normalized['container']
+    assert_equal 'Book Title Only', normalized[:container]
   end
 
   test 'handles openurl server validation' do
     # Test with matching server
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
-    openurl_link = normalized['links'].find { |link| link['kind'] == 'openurl' }
+    openurl_link = normalized[:links].find { |link| link['kind'] == 'openurl' }
     assert_not_nil openurl_link
 
     # Test with mismatched server. Should log warning but still return URL
     record = full_record.deep_dup
     record['delivery']['almaOpenurl'] = 'https://different.server.com/openurl?param=value'
     normalized = NormalizePrimoRecord.new(record, 'test').normalize
-    openurl_link = normalized['links'].find { |link| link['kind'] == 'openurl' }
+    openurl_link = normalized[:links].find { |link| link['kind'] == 'openurl' }
     assert_not_nil openurl_link
   end
 
@@ -355,7 +355,7 @@ def cdi_record
 
   test 'generates FRBR dedup URL for Alma records when frbrized' do
     normalized = NormalizePrimoRecord.new(alma_record, 'test query').normalize
-    full_record_link = normalized['links'].find { |link| link['kind'] == 'full record' }
+    full_record_link = normalized[:links].find { |link| link['kind'] == 'full record' }
     assert_not_nil full_record_link
 
     # Should use dedup URL for Alma records
@@ -365,7 +365,7 @@ def cdi_record
 
   test 'does not generate FRBR dedup URL for CDI records even when frbrized' do
     normalized = NormalizePrimoRecord.new(cdi_record, 'test query').normalize
-    full_record_link = normalized['links'].find { |link| link['kind'] == 'full record' }
+    full_record_link = normalized[:links].find { |link| link['kind'] == 'full record' }
     assert_not_nil full_record_link
 
     # Should use regular record link for CDI records, not dedup URL
@@ -378,7 +378,7 @@ def cdi_record
     record['pnx']['facets']['frbrtype'] = ['3'] # Not frbrized
 
     normalized = NormalizePrimoRecord.new(record, 'test').normalize
-    full_record_link = normalized['links'].find { |link| link['kind'] == 'full record' }
+    full_record_link = normalized[:links].find { |link| link['kind'] == 'full record' }
     assert_not_nil full_record_link
 
     # Should use regular record link when not frbrized
diff --git a/test/models/normalize_primo_results_test.rb b/test/models/normalize_primo_results_test.rb
index 1fb9f649..f1d3ff16 100644
--- a/test/models/normalize_primo_results_test.rb
+++ b/test/models/normalize_primo_results_test.rb
@@ -27,8 +27,8 @@ def empty_results
     normalized = normalizer.normalize
 
     assert_equal 2, normalized.length
-    assert_equal 'Testing the Limits of Knowledge', normalized.first['title']
-    assert_equal 'unknown title', normalized.second['title']
+    assert_equal 'Testing the Limits of Knowledge', normalized.first[:title]
+    assert_equal 'unknown title', normalized.second[:title]
   end
 
   test 'handles empty results' do

From fd19cc0d0e3c7ab89c17ffe516729323b98cbbf3 Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Mon, 3 Nov 2025 10:24:51 -0500
Subject: [PATCH 02/17] TIMDEX title to sym

TIMDEX title to sym
---
 app/helpers/search_helper.rb                 |  4 ++--
 app/models/normalize_timdex_record.rb        |  2 +-
 app/views/search/_result_geo.html.erb        |  2 +-
 test/helpers/search_helper_test.rb           | 12 ++++++------
 test/models/normalize_timdex_record_test.rb  |  4 ++--
 test/models/normalize_timdex_results_test.rb |  4 ++--
 6 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/app/helpers/search_helper.rb b/app/helpers/search_helper.rb
index 742398df..fa1cbbc3 100644
--- a/app/helpers/search_helper.rb
+++ b/app/helpers/search_helper.rb
@@ -16,9 +16,9 @@ def format_highlight_label(field_name)
 
   def link_to_result(result)
     if result['source_link'].present?
-      link_to(result['title'], result['source_link'])
+      link_to(result[:title], result['source_link'])
     else
-      result['title']
+      result[:title]
     end
   end
 
diff --git a/app/models/normalize_timdex_record.rb b/app/models/normalize_timdex_record.rb
index cf17efd5..826b99a9 100644
--- a/app/models/normalize_timdex_record.rb
+++ b/app/models/normalize_timdex_record.rb
@@ -8,7 +8,7 @@ def initialize(record, query)
   def normalize
     {
       # Core fields
-      'title' => title,
+      title:,
       'creators' => creators,
       'source' => source,
       'year' => year,
diff --git a/app/views/search/_result_geo.html.erb b/app/views/search/_result_geo.html.erb
index 4f543366..e8426fb1 100644
--- a/app/views/search/_result_geo.html.erb
+++ b/app/views/search/_result_geo.html.erb
@@ -1,7 +1,7 @@
 <li class="result">
   <div class="result-content">
     <h3 class="record-title">
-      <span class="sr">Title: </span><%= link_to(result_geo['title'], record_path(result_geo['identifier'])) %>
+      <span class="sr">Title: </span><%= link_to(result_geo[:title], record_path(result_geo['identifier'])) %>
     </h3>
 
     <div class="data-info">
diff --git a/test/helpers/search_helper_test.rb b/test/helpers/search_helper_test.rb
index aac77742..ddb6def4 100644
--- a/test/helpers/search_helper_test.rb
+++ b/test/helpers/search_helper_test.rb
@@ -27,13 +27,13 @@ class SearchHelperTest < ActionView::TestCase
   end
 
   test 'renders view_online link if source_link is present' do
-    result = { 'title' => 'A record', 'source_link' => 'https://example.org' }
+    result = { title: 'A record', 'source_link' => 'https://example.org' }
     assert_equal '<a class="button button-primary" href="https://example.org">View online</a>',
                  view_online(result)
   end
 
   test 'does not render view_online link if source_link is absent' do
-    result = { 'title' => 'A record' }
+    result = { title: 'A record' }
     assert_nil view_online(result)
   end
 
@@ -169,7 +169,7 @@ class SearchHelperTest < ActionView::TestCase
 
   test 'link_to_result returns link when source_link is present' do
     result = {
-      'title' => 'Sample Document Title',
+      title: 'Sample Document Title',
       'source_link' => 'https://example.com/document'
     }
     expected_link = '<a href="https://example.com/document">Sample Document Title</a>'
@@ -178,7 +178,7 @@ class SearchHelperTest < ActionView::TestCase
 
   test 'link_to_result returns plain title when source_link is nil' do
     result = {
-      'title' => 'Sample Document Title',
+      title: 'Sample Document Title',
       'source_link' => nil
     }
     assert_equal 'Sample Document Title', link_to_result(result)
@@ -186,7 +186,7 @@ class SearchHelperTest < ActionView::TestCase
 
   test 'link_to_result returns plain title when source_link is empty string' do
     result = {
-      'title' => 'Sample Document Title',
+      title: 'Sample Document Title',
       'source_link' => ''
     }
     assert_equal 'Sample Document Title', link_to_result(result)
@@ -194,7 +194,7 @@ class SearchHelperTest < ActionView::TestCase
 
   test 'link_to_result returns plain title when source_link key is absent' do
     result = {
-      'title' => 'Sample Document Title'
+      title: 'Sample Document Title'
     }
     assert_equal 'Sample Document Title', link_to_result(result)
   end
diff --git a/test/models/normalize_timdex_record_test.rb b/test/models/normalize_timdex_record_test.rb
index 58addee7..70772ff3 100644
--- a/test/models/normalize_timdex_record_test.rb
+++ b/test/models/normalize_timdex_record_test.rb
@@ -11,14 +11,14 @@ def minimal_record
 
   test 'normalizes title' do
     normalized = NormalizeTimdexRecord.new(full_record, 'test').normalize
-    assert_equal 'Sample TIMDEX Record for Testing', normalized['title']
+    assert_equal 'Sample TIMDEX Record for Testing', normalized[:title]
   end
 
   test 'handles missing title' do
     record_without_title = minimal_record
     record_without_title.delete('title')
     normalized = NormalizeTimdexRecord.new(record_without_title, 'test').normalize
-    assert_equal 'Unknown title', normalized['title']
+    assert_equal 'Unknown title', normalized[:title]
   end
 
   test 'normalizes creators from contributors' do
diff --git a/test/models/normalize_timdex_results_test.rb b/test/models/normalize_timdex_results_test.rb
index 08c3fda9..58a44f95 100644
--- a/test/models/normalize_timdex_results_test.rb
+++ b/test/models/normalize_timdex_results_test.rb
@@ -20,7 +20,7 @@ def empty_timdex_response
 
     # Check first record (full record)
     first_result = results.first
-    assert_equal 'Sample TIMDEX Record for Testing', first_result['title']
+    assert_equal 'Sample TIMDEX Record for Testing', first_result[:title]
     assert_equal 'test-record-123', first_result['identifier']
     assert_equal 'Test Repository', first_result['source']
     assert_equal 'Dataset ; Geospatial data', first_result['format']
@@ -35,7 +35,7 @@ def empty_timdex_response
 
     # Check second record (minimal record)
     second_result = results.second
-    assert_equal 'Minimal Test Record', second_result['title']
+    assert_equal 'Minimal Test Record', second_result[:title]
     assert_equal 'minimal-record-456', second_result['identifier']
     assert_equal 'Test Repository', second_result['source']
   end

From 80fd10985b597265daa64d81ac449a7164253d80 Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Tue, 4 Nov 2025 09:18:37 -0500
Subject: [PATCH 03/17] TIMDEX contributors and creators to sym

---
 app/models/normalize_timdex_record.rb        | 4 ++--
 app/views/search/_result.html.erb            | 2 +-
 app/views/search/_result_geo.html.erb        | 4 ++--
 test/models/normalize_timdex_record_test.rb  | 6 +++---
 test/models/normalize_timdex_results_test.rb | 2 +-
 5 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/app/models/normalize_timdex_record.rb b/app/models/normalize_timdex_record.rb
index 826b99a9..5a8250bd 100644
--- a/app/models/normalize_timdex_record.rb
+++ b/app/models/normalize_timdex_record.rb
@@ -9,7 +9,7 @@ def normalize
     {
       # Core fields
       title:,
-      'creators' => creators,
+      creators:,
       'source' => source,
       'year' => year,
       'format' => format,
@@ -23,7 +23,7 @@ def normalize
       # TIMDEX-specific fields
       'content_type' => content_type,
       'dates' => dates,
-      'contributors' => contributors,
+      contributors:,
       'highlight' => highlight,
       'source_link' => source_link
     }
diff --git a/app/views/search/_result.html.erb b/app/views/search/_result.html.erb
index 42f20d0d..89122c76 100644
--- a/app/views/search/_result.html.erb
+++ b/app/views/search/_result.html.erb
@@ -16,7 +16,7 @@
 
     <span class="sr">Contributors: </span>
     <ul class="list-inline truncate-list contributors">
-      <%= render partial: 'shared/contributors', locals: { contributors: result['contributors'] } %>
+      <%= render partial: 'shared/contributors', locals: { contributors: result[:contributors] } %>
     </ul>
 
     <div class="result-highlights">
diff --git a/app/views/search/_result_geo.html.erb b/app/views/search/_result_geo.html.erb
index e8426fb1..98fb7546 100644
--- a/app/views/search/_result_geo.html.erb
+++ b/app/views/search/_result_geo.html.erb
@@ -8,11 +8,11 @@
       <%= render partial: 'shared/geo_data_info', locals: { metadata: result_geo } %>
     </div>
 
-    <% if result_geo['creators'].present? || result_geo['contributors'].present? %>
+    <% if result_geo[:creators].present? || result_geo[:contributors].present? %>
       <span class="sr">Contributors: </span>
       <ul class="list-inline truncate-list contributors">
         <!-- Use normalized creators if available, otherwise fall back to raw contributors -->
-        <% contributors = result_geo['creators'].present? ? result_geo['creators'] : result_geo['contributors'] %>
+        <% contributors = result_geo[:creators].present? ? result_geo[:creators] : result_geo[:contributors] %>
         <% if contributors %>
           <%= render partial: 'shared/contributors', locals: { contributors: contributors } %>
         <% end %>
diff --git a/test/models/normalize_timdex_record_test.rb b/test/models/normalize_timdex_record_test.rb
index 70772ff3..9ab2fd12 100644
--- a/test/models/normalize_timdex_record_test.rb
+++ b/test/models/normalize_timdex_record_test.rb
@@ -27,12 +27,12 @@ def minimal_record
       { 'value' => 'Smith, Jane', 'link' => nil },
       { 'value' => 'Doe, John', 'link' => nil }
     ]
-    assert_equal expected_creators, normalized['creators']
+    assert_equal expected_creators, normalized[:creators]
   end
 
   test 'handles missing creators' do
     normalized = NormalizeTimdexRecord.new(minimal_record, 'test').normalize
-    assert_empty normalized['creators']
+    assert_empty normalized[:creators]
   end
 
   test 'normalizes source' do
@@ -190,7 +190,7 @@ def minimal_record
       { 'kind' => 'Author', 'value' => 'Doe, John' },
       { 'kind' => 'Publisher', 'value' => 'MIT Libraries' }
     ]
-    assert_equal expected_contributors, normalized['contributors']
+    assert_equal expected_contributors, normalized[:contributors]
   end
 
   test 'includes TIMDEX-specific highlight field' do
diff --git a/test/models/normalize_timdex_results_test.rb b/test/models/normalize_timdex_results_test.rb
index 58a44f95..c253d471 100644
--- a/test/models/normalize_timdex_results_test.rb
+++ b/test/models/normalize_timdex_results_test.rb
@@ -29,7 +29,7 @@ def empty_timdex_response
     # Check TIMDEX-specific fields are preserved
     assert_includes first_result.keys, 'content_type'
     assert_includes first_result.keys, 'dates'
-    assert_includes first_result.keys, 'contributors'
+    assert_includes first_result.keys, :contributors
     assert_includes first_result.keys, 'highlight'
     assert_includes first_result.keys, 'source_link'
 

From 4e68d4bb0e6cf391103c3bec20f15d94d7dc20f1 Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Tue, 4 Nov 2025 09:24:00 -0500
Subject: [PATCH 04/17] TIMDEX source to sym

---
 app/models/normalize_timdex_record.rb        | 2 +-
 test/models/normalize_timdex_record_test.rb  | 4 ++--
 test/models/normalize_timdex_results_test.rb | 4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/app/models/normalize_timdex_record.rb b/app/models/normalize_timdex_record.rb
index 5a8250bd..1364014c 100644
--- a/app/models/normalize_timdex_record.rb
+++ b/app/models/normalize_timdex_record.rb
@@ -10,7 +10,7 @@ def normalize
       # Core fields
       title:,
       creators:,
-      'source' => source,
+      source:,
       'year' => year,
       'format' => format,
       'links' => links,
diff --git a/test/models/normalize_timdex_record_test.rb b/test/models/normalize_timdex_record_test.rb
index 9ab2fd12..5a22d021 100644
--- a/test/models/normalize_timdex_record_test.rb
+++ b/test/models/normalize_timdex_record_test.rb
@@ -37,14 +37,14 @@ def minimal_record
 
   test 'normalizes source' do
     normalized = NormalizeTimdexRecord.new(full_record, 'test').normalize
-    assert_equal 'Test Repository', normalized['source']
+    assert_equal 'Test Repository', normalized[:source]
   end
 
   test 'handles missing source' do
     record_without_source = minimal_record.dup
     record_without_source.delete('source')
     normalized = NormalizeTimdexRecord.new(record_without_source, 'test').normalize
-    assert_equal 'Unknown source', normalized['source']
+    assert_equal 'Unknown source', normalized[:source]
   end
 
   test 'extracts year from publication date' do
diff --git a/test/models/normalize_timdex_results_test.rb b/test/models/normalize_timdex_results_test.rb
index c253d471..35906497 100644
--- a/test/models/normalize_timdex_results_test.rb
+++ b/test/models/normalize_timdex_results_test.rb
@@ -22,7 +22,7 @@ def empty_timdex_response
     first_result = results.first
     assert_equal 'Sample TIMDEX Record for Testing', first_result[:title]
     assert_equal 'test-record-123', first_result['identifier']
-    assert_equal 'Test Repository', first_result['source']
+    assert_equal 'Test Repository', first_result[:source]
     assert_equal 'Dataset ; Geospatial data', first_result['format']
     assert_equal '2023', first_result['year']
 
@@ -37,7 +37,7 @@ def empty_timdex_response
     second_result = results.second
     assert_equal 'Minimal Test Record', second_result[:title]
     assert_equal 'minimal-record-456', second_result['identifier']
-    assert_equal 'Test Repository', second_result['source']
+    assert_equal 'Test Repository', second_result[:source]
   end
 
   test 'handles empty TIMDEX response' do

From 1137661b98b6246809daa028e4ea740a29b80820 Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Tue, 4 Nov 2025 09:27:07 -0500
Subject: [PATCH 05/17] TIMDEX year to sym

---
 app/models/normalize_timdex_record.rb        | 2 +-
 test/models/normalize_timdex_record_test.rb  | 6 +++---
 test/models/normalize_timdex_results_test.rb | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/app/models/normalize_timdex_record.rb b/app/models/normalize_timdex_record.rb
index 1364014c..1552e77f 100644
--- a/app/models/normalize_timdex_record.rb
+++ b/app/models/normalize_timdex_record.rb
@@ -11,7 +11,7 @@ def normalize
       title:,
       creators:,
       source:,
-      'year' => year,
+      year:,
       'format' => format,
       'links' => links,
       'citation' => citation,
diff --git a/test/models/normalize_timdex_record_test.rb b/test/models/normalize_timdex_record_test.rb
index 5a22d021..4abd929b 100644
--- a/test/models/normalize_timdex_record_test.rb
+++ b/test/models/normalize_timdex_record_test.rb
@@ -49,12 +49,12 @@ def minimal_record
 
   test 'extracts year from publication date' do
     normalized = NormalizeTimdexRecord.new(full_record, 'test').normalize
-    assert_equal '2023', normalized['year']
+    assert_equal '2023', normalized[:year]
   end
 
   test 'handles missing year' do
     normalized = NormalizeTimdexRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['year']
+    assert_nil normalized[:year]
   end
 
   test 'extracts year from fallback date when no publication date' do
@@ -64,7 +64,7 @@ def minimal_record
       { 'kind' => 'Creation', 'value' => 'Created in 1998' }
     ]
     normalized = NormalizeTimdexRecord.new(record_with_coverage_date, 'test').normalize
-    assert_equal '1995', normalized['year']
+    assert_equal '1995', normalized[:year]
   end
 
   test 'normalizes format from content type' do
diff --git a/test/models/normalize_timdex_results_test.rb b/test/models/normalize_timdex_results_test.rb
index 35906497..cc9bfa77 100644
--- a/test/models/normalize_timdex_results_test.rb
+++ b/test/models/normalize_timdex_results_test.rb
@@ -24,7 +24,7 @@ def empty_timdex_response
     assert_equal 'test-record-123', first_result['identifier']
     assert_equal 'Test Repository', first_result[:source]
     assert_equal 'Dataset ; Geospatial data', first_result['format']
-    assert_equal '2023', first_result['year']
+    assert_equal '2023', first_result[:year]
 
     # Check TIMDEX-specific fields are preserved
     assert_includes first_result.keys, 'content_type'

From 27385129051a46c79d77b29e2da27319fb012bdf Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Tue, 4 Nov 2025 09:28:30 -0500
Subject: [PATCH 06/17] TIMDEX format to sym

---
 app/models/normalize_timdex_record.rb        | 2 +-
 test/models/normalize_timdex_record_test.rb  | 4 ++--
 test/models/normalize_timdex_results_test.rb | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/models/normalize_timdex_record.rb b/app/models/normalize_timdex_record.rb
index 1552e77f..600f8439 100644
--- a/app/models/normalize_timdex_record.rb
+++ b/app/models/normalize_timdex_record.rb
@@ -12,7 +12,7 @@ def normalize
       creators:,
       source:,
       year:,
-      'format' => format,
+      format:,
       'links' => links,
       'citation' => citation,
       'identifier' => identifier,
diff --git a/test/models/normalize_timdex_record_test.rb b/test/models/normalize_timdex_record_test.rb
index 4abd929b..4dedcec1 100644
--- a/test/models/normalize_timdex_record_test.rb
+++ b/test/models/normalize_timdex_record_test.rb
@@ -69,14 +69,14 @@ def minimal_record
 
   test 'normalizes format from content type' do
     normalized = NormalizeTimdexRecord.new(full_record, 'test').normalize
-    assert_equal 'Dataset ; Geospatial data', normalized['format']
+    assert_equal 'Dataset ; Geospatial data', normalized[:format]
   end
 
   test 'handles missing format' do
     record_without_format = minimal_record.dup
     record_without_format.delete('contentType')
     normalized = NormalizeTimdexRecord.new(record_without_format, 'test').normalize
-    assert_empty normalized['format']
+    assert_empty normalized[:format]
   end
 
   test 'normalizes links from source link' do
diff --git a/test/models/normalize_timdex_results_test.rb b/test/models/normalize_timdex_results_test.rb
index cc9bfa77..4f0a1a55 100644
--- a/test/models/normalize_timdex_results_test.rb
+++ b/test/models/normalize_timdex_results_test.rb
@@ -23,7 +23,7 @@ def empty_timdex_response
     assert_equal 'Sample TIMDEX Record for Testing', first_result[:title]
     assert_equal 'test-record-123', first_result['identifier']
     assert_equal 'Test Repository', first_result[:source]
-    assert_equal 'Dataset ; Geospatial data', first_result['format']
+    assert_equal 'Dataset ; Geospatial data', first_result[:format]
     assert_equal '2023', first_result[:year]
 
     # Check TIMDEX-specific fields are preserved

From 491136291980ad60ac29a323e5bf42266534e6d8 Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Tue, 4 Nov 2025 09:29:49 -0500
Subject: [PATCH 07/17] TIMDEX links to sym

---
 app/models/normalize_timdex_record.rb       | 2 +-
 test/models/normalize_timdex_record_test.rb | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/models/normalize_timdex_record.rb b/app/models/normalize_timdex_record.rb
index 600f8439..ac1b8b4e 100644
--- a/app/models/normalize_timdex_record.rb
+++ b/app/models/normalize_timdex_record.rb
@@ -13,7 +13,7 @@ def normalize
       source:,
       year:,
       format:,
-      'links' => links,
+      links:,
       'citation' => citation,
       'identifier' => identifier,
       'summary' => summary,
diff --git a/test/models/normalize_timdex_record_test.rb b/test/models/normalize_timdex_record_test.rb
index 4dedcec1..57321e54 100644
--- a/test/models/normalize_timdex_record_test.rb
+++ b/test/models/normalize_timdex_record_test.rb
@@ -88,12 +88,12 @@ def minimal_record
         'text' => 'View full record'
       }
     ]
-    assert_equal expected_links, normalized['links']
+    assert_equal expected_links, normalized[:links]
   end
 
   test 'handles missing links' do
     normalized = NormalizeTimdexRecord.new(minimal_record, 'test').normalize
-    assert_empty normalized['links']
+    assert_empty normalized[:links]
   end
 
   test 'normalizes citation' do

From f4ebcce4d2549cf3b8bc3849c6660a12c4e6ad14 Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Tue, 4 Nov 2025 09:31:00 -0500
Subject: [PATCH 08/17] TIMDEX citation to sym

---
 app/models/normalize_timdex_record.rb       | 2 +-
 test/models/normalize_timdex_record_test.rb | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/models/normalize_timdex_record.rb b/app/models/normalize_timdex_record.rb
index ac1b8b4e..f2030ddf 100644
--- a/app/models/normalize_timdex_record.rb
+++ b/app/models/normalize_timdex_record.rb
@@ -14,7 +14,7 @@ def normalize
       year:,
       format:,
       links:,
-      'citation' => citation,
+      citation:,
       'identifier' => identifier,
       'summary' => summary,
       'publisher' => publisher,
diff --git a/test/models/normalize_timdex_record_test.rb b/test/models/normalize_timdex_record_test.rb
index 57321e54..fe37f02b 100644
--- a/test/models/normalize_timdex_record_test.rb
+++ b/test/models/normalize_timdex_record_test.rb
@@ -99,12 +99,12 @@ def minimal_record
   test 'normalizes citation' do
     normalized = NormalizeTimdexRecord.new(full_record, 'test').normalize
     assert_equal 'Smith, J. & Doe, J. (2023). Sample TIMDEX Record for Testing. Test Repository.',
-                 normalized['citation']
+                 normalized[:citation]
   end
 
   test 'handles missing citation' do
     normalized = NormalizeTimdexRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['citation']
+    assert_nil normalized[:citation]
   end
 
   test 'normalizes identifier' do

From e48f6f5f22eed6b3ced4054828d935b1c69d4558 Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Tue, 4 Nov 2025 10:04:25 -0500
Subject: [PATCH 09/17] TIMDEX identifier to sym

---
 app/models/normalize_timdex_record.rb        | 2 +-
 app/views/search/_result_geo.html.erb        | 4 ++--
 test/models/normalize_timdex_record_test.rb  | 2 +-
 test/models/normalize_timdex_results_test.rb | 4 ++--
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/app/models/normalize_timdex_record.rb b/app/models/normalize_timdex_record.rb
index f2030ddf..b5c970c9 100644
--- a/app/models/normalize_timdex_record.rb
+++ b/app/models/normalize_timdex_record.rb
@@ -15,7 +15,7 @@ def normalize
       format:,
       links:,
       citation:,
-      'identifier' => identifier,
+      identifier:,
       'summary' => summary,
       'publisher' => publisher,
       'location' => location,
diff --git a/app/views/search/_result_geo.html.erb b/app/views/search/_result_geo.html.erb
index 98fb7546..b9ac420d 100644
--- a/app/views/search/_result_geo.html.erb
+++ b/app/views/search/_result_geo.html.erb
@@ -1,7 +1,7 @@
 <li class="result">
   <div class="result-content">
     <h3 class="record-title">
-      <span class="sr">Title: </span><%= link_to(result_geo[:title], record_path(result_geo['identifier'])) %>
+      <span class="sr">Title: </span><%= link_to(result_geo[:title], record_path(result_geo[:identifier])) %>
     </h3>
 
     <div class="data-info">
@@ -32,7 +32,7 @@
     <% end %>
 
     <div class="result-record">
-      <%= view_record(result_geo['identifier']) %>
+      <%= view_record(result_geo[:identifier]) %>
     </div>
   </div>
 </li>
diff --git a/test/models/normalize_timdex_record_test.rb b/test/models/normalize_timdex_record_test.rb
index fe37f02b..081c1492 100644
--- a/test/models/normalize_timdex_record_test.rb
+++ b/test/models/normalize_timdex_record_test.rb
@@ -109,7 +109,7 @@ def minimal_record
 
   test 'normalizes identifier' do
     normalized = NormalizeTimdexRecord.new(full_record, 'test').normalize
-    assert_equal 'test-record-123', normalized['identifier']
+    assert_equal 'test-record-123', normalized[:identifier]
   end
 
   test 'normalizes summary' do
diff --git a/test/models/normalize_timdex_results_test.rb b/test/models/normalize_timdex_results_test.rb
index 4f0a1a55..ebc04dbc 100644
--- a/test/models/normalize_timdex_results_test.rb
+++ b/test/models/normalize_timdex_results_test.rb
@@ -21,7 +21,7 @@ def empty_timdex_response
     # Check first record (full record)
     first_result = results.first
     assert_equal 'Sample TIMDEX Record for Testing', first_result[:title]
-    assert_equal 'test-record-123', first_result['identifier']
+    assert_equal 'test-record-123', first_result[:identifier]
     assert_equal 'Test Repository', first_result[:source]
     assert_equal 'Dataset ; Geospatial data', first_result[:format]
     assert_equal '2023', first_result[:year]
@@ -36,7 +36,7 @@ def empty_timdex_response
     # Check second record (minimal record)
     second_result = results.second
     assert_equal 'Minimal Test Record', second_result[:title]
-    assert_equal 'minimal-record-456', second_result['identifier']
+    assert_equal 'minimal-record-456', second_result[:identifier]
     assert_equal 'Test Repository', second_result[:source]
   end
 

From a0a0965b444687048133589f1c7d303d228a10ed Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Tue, 4 Nov 2025 10:39:44 -0500
Subject: [PATCH 10/17] TIMDEX summary to sym

---
 app/models/normalize_timdex_record.rb       | 2 +-
 app/views/search/_result_geo.html.erb       | 4 ++--
 test/models/normalize_timdex_record_test.rb | 4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/app/models/normalize_timdex_record.rb b/app/models/normalize_timdex_record.rb
index b5c970c9..589cd774 100644
--- a/app/models/normalize_timdex_record.rb
+++ b/app/models/normalize_timdex_record.rb
@@ -16,7 +16,7 @@ def normalize
       links:,
       citation:,
       identifier:,
-      'summary' => summary,
+      summary:,
       'publisher' => publisher,
       'location' => location,
       'subjects' => subjects,
diff --git a/app/views/search/_result_geo.html.erb b/app/views/search/_result_geo.html.erb
index b9ac420d..54d76159 100644
--- a/app/views/search/_result_geo.html.erb
+++ b/app/views/search/_result_geo.html.erb
@@ -19,9 +19,9 @@
       </ul>
     <% end %>
 
-    <% if result_geo['summary'].present? %>
+    <% if result_geo[:summary].present? %>
       <p class="result-summary truncate-list">
-        <span class="sr">Summary: </span><%= result_geo['summary'] %>
+        <span class="sr">Summary: </span><%= result_geo[:summary] %>
       </p>
     <% end %>
 
diff --git a/test/models/normalize_timdex_record_test.rb b/test/models/normalize_timdex_record_test.rb
index 081c1492..e9a2c8c9 100644
--- a/test/models/normalize_timdex_record_test.rb
+++ b/test/models/normalize_timdex_record_test.rb
@@ -115,12 +115,12 @@ def minimal_record
   test 'normalizes summary' do
     normalized = NormalizeTimdexRecord.new(full_record, 'test').normalize
     assert_equal 'This is a comprehensive test record with all possible fields populated for testing normalization.',
-                 normalized['summary']
+                 normalized[:summary]
   end
 
   test 'handles missing summary' do
     normalized = NormalizeTimdexRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['summary']
+    assert_nil normalized[:summary]
   end
 
   test 'extracts publisher from contributors' do

From 9732cacd706f853d48830d22b189aaa4d01001b5 Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Tue, 4 Nov 2025 10:43:11 -0500
Subject: [PATCH 11/17] TIMDEX publisher and location to sym

---
 app/models/normalize_timdex_record.rb       |  4 ++--
 test/models/normalize_timdex_record_test.rb | 10 +++++-----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/app/models/normalize_timdex_record.rb b/app/models/normalize_timdex_record.rb
index 589cd774..d4b318d3 100644
--- a/app/models/normalize_timdex_record.rb
+++ b/app/models/normalize_timdex_record.rb
@@ -17,8 +17,8 @@ def normalize
       citation:,
       identifier:,
       summary:,
-      'publisher' => publisher,
-      'location' => location,
+      publisher:,
+      location:,
       'subjects' => subjects,
       # TIMDEX-specific fields
       'content_type' => content_type,
diff --git a/test/models/normalize_timdex_record_test.rb b/test/models/normalize_timdex_record_test.rb
index e9a2c8c9..75b93409 100644
--- a/test/models/normalize_timdex_record_test.rb
+++ b/test/models/normalize_timdex_record_test.rb
@@ -125,22 +125,22 @@ def minimal_record
 
   test 'extracts publisher from contributors' do
     normalized = NormalizeTimdexRecord.new(full_record, 'test').normalize
-    assert_equal 'MIT Libraries', normalized['publisher']
+    assert_equal 'MIT Libraries', normalized[:publisher]
   end
 
   test 'handles missing publisher' do
     normalized = NormalizeTimdexRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['publisher']
+    assert_nil normalized[:publisher]
   end
 
   test 'normalizes location' do
     normalized = NormalizeTimdexRecord.new(full_record, 'test').normalize
-    assert_equal 'Cambridge, MA', normalized['location']
+    assert_equal 'Cambridge, MA', normalized[:location]
   end
 
   test 'handles missing location' do
     normalized = NormalizeTimdexRecord.new(minimal_record, 'test').normalize
-    assert_nil normalized['location']
+    assert_nil normalized[:location]
   end
 
   test 'joins multiple locations with semicolon' do
@@ -151,7 +151,7 @@ def minimal_record
       { 'value' => 'New York, NY' }
     ]
     normalized = NormalizeTimdexRecord.new(record_with_multiple_locations, 'test').normalize
-    assert_equal 'Cambridge, MA; Boston, MA; New York, NY', normalized['location']
+    assert_equal 'Cambridge, MA; Boston, MA; New York, NY', normalized[:location]
   end
 
   test 'normalizes subjects' do

From f517f5f82a422f4775d22bb6244c12eccf56bda8 Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Tue, 4 Nov 2025 10:59:32 -0500
Subject: [PATCH 12/17] TIMDEX subjects to sym

---
 app/models/normalize_timdex_record.rb       | 2 +-
 test/models/normalize_timdex_record_test.rb | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/models/normalize_timdex_record.rb b/app/models/normalize_timdex_record.rb
index d4b318d3..3c7a4185 100644
--- a/app/models/normalize_timdex_record.rb
+++ b/app/models/normalize_timdex_record.rb
@@ -19,7 +19,7 @@ def normalize
       summary:,
       publisher:,
       location:,
-      'subjects' => subjects,
+      subjects:,
       # TIMDEX-specific fields
       'content_type' => content_type,
       'dates' => dates,
diff --git a/test/models/normalize_timdex_record_test.rb b/test/models/normalize_timdex_record_test.rb
index 75b93409..8ec913c1 100644
--- a/test/models/normalize_timdex_record_test.rb
+++ b/test/models/normalize_timdex_record_test.rb
@@ -156,12 +156,12 @@ def minimal_record
 
   test 'normalizes subjects' do
     normalized = NormalizeTimdexRecord.new(full_record, 'test').normalize
-    assert_equal ['Geographic Information Systems', 'Remote Sensing'], normalized['subjects']
+    assert_equal ['Geographic Information Systems', 'Remote Sensing'], normalized[:subjects]
   end
 
   test 'handles missing subjects' do
     normalized = NormalizeTimdexRecord.new(minimal_record, 'test').normalize
-    assert_empty normalized['subjects']
+    assert_empty normalized[:subjects]
   end
 
   # Test TIMDEX-specific fields

From 13d880e3cea24595cfbd9c7da83bc32198e2afe0 Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Tue, 4 Nov 2025 11:06:43 -0500
Subject: [PATCH 13/17] TIMDEX content_type to sym

---
 app/models/normalize_timdex_record.rb        | 2 +-
 app/views/search/_result.html.erb            | 2 +-
 test/models/normalize_timdex_record_test.rb  | 2 +-
 test/models/normalize_timdex_results_test.rb | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/models/normalize_timdex_record.rb b/app/models/normalize_timdex_record.rb
index 3c7a4185..bbf731ab 100644
--- a/app/models/normalize_timdex_record.rb
+++ b/app/models/normalize_timdex_record.rb
@@ -21,7 +21,7 @@ def normalize
       location:,
       subjects:,
       # TIMDEX-specific fields
-      'content_type' => content_type,
+      content_type:,
       'dates' => dates,
       contributors:,
       'highlight' => highlight,
diff --git a/app/views/search/_result.html.erb b/app/views/search/_result.html.erb
index 89122c76..7aecd043 100644
--- a/app/views/search/_result.html.erb
+++ b/app/views/search/_result.html.erb
@@ -6,7 +6,7 @@
     </h3>
 
     <p class="pub-info">
-      <span><%= result['content_type']&.each { |type| type['value'] }&.join(' ; ') %></span>
+      <span><%= result[:content_type]&.each { |type| type['value'] }&.join(' ; ') %></span>
       <span>
         <% result['dates']&.each do |date| %>
           <%= date['value'] if date['kind'] == 'Publication date' %>
diff --git a/test/models/normalize_timdex_record_test.rb b/test/models/normalize_timdex_record_test.rb
index 8ec913c1..8cfbf543 100644
--- a/test/models/normalize_timdex_record_test.rb
+++ b/test/models/normalize_timdex_record_test.rb
@@ -171,7 +171,7 @@ def minimal_record
       { 'value' => 'Dataset' },
       { 'value' => 'Geospatial data' }
     ]
-    assert_equal expected_content_type, normalized['content_type']
+    assert_equal expected_content_type, normalized[:content_type]
   end
 
   test 'includes TIMDEX-specific dates field' do
diff --git a/test/models/normalize_timdex_results_test.rb b/test/models/normalize_timdex_results_test.rb
index ebc04dbc..09683298 100644
--- a/test/models/normalize_timdex_results_test.rb
+++ b/test/models/normalize_timdex_results_test.rb
@@ -27,7 +27,7 @@ def empty_timdex_response
     assert_equal '2023', first_result[:year]
 
     # Check TIMDEX-specific fields are preserved
-    assert_includes first_result.keys, 'content_type'
+    assert_includes first_result.keys, :content_type
     assert_includes first_result.keys, 'dates'
     assert_includes first_result.keys, :contributors
     assert_includes first_result.keys, 'highlight'

From cf2532970637f9b860a0ca202ed8f9ec1bea54b4 Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Tue, 4 Nov 2025 12:05:30 -0500
Subject: [PATCH 14/17] TIMDEX dates to sym

---
 app/models/normalize_timdex_record.rb        | 2 +-
 app/views/search/_result.html.erb            | 2 +-
 test/models/normalize_timdex_record_test.rb  | 2 +-
 test/models/normalize_timdex_results_test.rb | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/models/normalize_timdex_record.rb b/app/models/normalize_timdex_record.rb
index bbf731ab..064668b4 100644
--- a/app/models/normalize_timdex_record.rb
+++ b/app/models/normalize_timdex_record.rb
@@ -22,7 +22,7 @@ def normalize
       subjects:,
       # TIMDEX-specific fields
       content_type:,
-      'dates' => dates,
+      dates:,
       contributors:,
       'highlight' => highlight,
       'source_link' => source_link
diff --git a/app/views/search/_result.html.erb b/app/views/search/_result.html.erb
index 7aecd043..ec8a59fd 100644
--- a/app/views/search/_result.html.erb
+++ b/app/views/search/_result.html.erb
@@ -8,7 +8,7 @@
     <p class="pub-info">
       <span><%= result[:content_type]&.each { |type| type['value'] }&.join(' ; ') %></span>
       <span>
-        <% result['dates']&.each do |date| %>
+        <% result[:dates]&.each do |date| %>
           <%= date['value'] if date['kind'] == 'Publication date' %>
         <% end %>
       </span>
diff --git a/test/models/normalize_timdex_record_test.rb b/test/models/normalize_timdex_record_test.rb
index 8cfbf543..b5d611c2 100644
--- a/test/models/normalize_timdex_record_test.rb
+++ b/test/models/normalize_timdex_record_test.rb
@@ -180,7 +180,7 @@ def minimal_record
       { 'kind' => 'Publication date', 'value' => '2023-01-15' },
       { 'kind' => 'Coverage', 'value' => '2020-2023' }
     ]
-    assert_equal expected_dates, normalized['dates']
+    assert_equal expected_dates, normalized[:dates]
   end
 
   test 'includes TIMDEX-specific contributors field' do
diff --git a/test/models/normalize_timdex_results_test.rb b/test/models/normalize_timdex_results_test.rb
index 09683298..c25cc62d 100644
--- a/test/models/normalize_timdex_results_test.rb
+++ b/test/models/normalize_timdex_results_test.rb
@@ -28,7 +28,7 @@ def empty_timdex_response
 
     # Check TIMDEX-specific fields are preserved
     assert_includes first_result.keys, :content_type
-    assert_includes first_result.keys, 'dates'
+    assert_includes first_result.keys, :dates
     assert_includes first_result.keys, :contributors
     assert_includes first_result.keys, 'highlight'
     assert_includes first_result.keys, 'source_link'

From c8138cbfd1f71f27ba8cb1d45687f965b72e7171 Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Tue, 4 Nov 2025 13:27:27 -0500
Subject: [PATCH 15/17] TIMDEX highlight to sym

---
 app/helpers/search_helper.rb                 |  4 ++--
 app/models/normalize_timdex_record.rb        |  2 +-
 app/views/search/_result_geo.html.erb        |  4 ++--
 test/helpers/search_helper_test.rb           | 20 ++++++++++----------
 test/models/normalize_timdex_record_test.rb  |  2 +-
 test/models/normalize_timdex_results_test.rb |  2 +-
 6 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/app/helpers/search_helper.rb b/app/helpers/search_helper.rb
index fa1cbbc3..cc4ff968 100644
--- a/app/helpers/search_helper.rb
+++ b/app/helpers/search_helper.rb
@@ -4,9 +4,9 @@ def displayed_fields
   end
 
   def trim_highlights(result)
-    return unless result['highlight']&.any?
+    return unless result[:highlight]&.any?
 
-    result['highlight'].reject { |h| displayed_fields.include? h['matchedField'] }
+    result[:highlight].reject { |h| displayed_fields.include? h['matchedField'] }
   end
 
   def format_highlight_label(field_name)
diff --git a/app/models/normalize_timdex_record.rb b/app/models/normalize_timdex_record.rb
index 064668b4..81fe495d 100644
--- a/app/models/normalize_timdex_record.rb
+++ b/app/models/normalize_timdex_record.rb
@@ -24,7 +24,7 @@ def normalize
       content_type:,
       dates:,
       contributors:,
-      'highlight' => highlight,
+      highlight:,
       'source_link' => source_link
     }
   end
diff --git a/app/views/search/_result_geo.html.erb b/app/views/search/_result_geo.html.erb
index 54d76159..558df01a 100644
--- a/app/views/search/_result_geo.html.erb
+++ b/app/views/search/_result_geo.html.erb
@@ -25,9 +25,9 @@
       </p>
     <% end %>
 
-    <% if result_geo['highlight'] %>
+    <% if result_geo[:highlight] %>
       <div class="result-highlights">
-        <%= render partial: 'search/highlights', locals: { result: { 'highlight' => result_geo['highlight'] } } %>
+        <%= render partial: 'search/highlights', locals: { result: { highlight: result_geo[:highlight] } } %>
       </div>
     <% end %>
 
diff --git a/test/helpers/search_helper_test.rb b/test/helpers/search_helper_test.rb
index ddb6def4..e3c29320 100644
--- a/test/helpers/search_helper_test.rb
+++ b/test/helpers/search_helper_test.rb
@@ -4,24 +4,24 @@ class SearchHelperTest < ActionView::TestCase
   include SearchHelper
 
   test 'removes displayed fields from highlights' do
-    result = { 'highlight' => [{ 'matchedField' => 'title', 'matchedPhrases' => 'Very important data' },
-                               { 'matchedField' => 'title.exact_value', 'matchedPhrases' => 'Very important data' },
-                               { 'matchedField' => 'content_type', 'matchedPhrases' => 'Dataset' },
-                               { 'matchedField' => 'dates.value', 'matchedPhrases' => '2022' },
-                               { 'matchedField' => 'contributors.value', 'matchedPhrases' => 'Jane Datascientist' }] }
+    result = { highlight: [{ 'matchedField' => 'title', 'matchedPhrases' => 'Very important data' },
+                           { 'matchedField' => 'title.exact_value', 'matchedPhrases' => 'Very important data' },
+                           { 'matchedField' => 'content_type', 'matchedPhrases' => 'Dataset' },
+                           { 'matchedField' => 'dates.value', 'matchedPhrases' => '2022' },
+                           { 'matchedField' => 'contributors.value', 'matchedPhrases' => 'Jane Datascientist' }] }
     assert_empty trim_highlights(result)
   end
 
   test 'does not remove undisplayed fields from highlights' do
-    result = { 'highlight' => [{ 'matchedField' => 'summary', 'matchedPhrases' => 'Have some data' }] }
+    result = { highlight: [{ 'matchedField' => 'summary', 'matchedPhrases' => 'Have some data' }] }
     assert_equal [{ 'matchedField' => 'summary', 'matchedPhrases' => 'Have some data' }], trim_highlights(result)
   end
 
   test 'returns correct set of highlights when result includes displayed and undisplayed fields' do
-    result = { 'highlight' => [{ 'matchedField' => 'title', 'matchedPhrases' => 'Very important data' },
-                               { 'matchedField' => 'content_type', 'matchedPhrases' => 'Dataset' },
-                               { 'matchedField' => 'summary', 'matchedPhrases' => '2022' },
-                               { 'matchedField' => 'citation', 'matchedPhrases' => 'Datascientist, Jane' }] }
+    result = { highlight: [{ 'matchedField' => 'title', 'matchedPhrases' => 'Very important data' },
+                           { 'matchedField' => 'content_type', 'matchedPhrases' => 'Dataset' },
+                           { 'matchedField' => 'summary', 'matchedPhrases' => '2022' },
+                           { 'matchedField' => 'citation', 'matchedPhrases' => 'Datascientist, Jane' }] }
     assert_equal [{ 'matchedField' => 'summary', 'matchedPhrases' => '2022' },
                   { 'matchedField' => 'citation', 'matchedPhrases' => 'Datascientist, Jane' }], trim_highlights(result)
   end
diff --git a/test/models/normalize_timdex_record_test.rb b/test/models/normalize_timdex_record_test.rb
index b5d611c2..df5e0555 100644
--- a/test/models/normalize_timdex_record_test.rb
+++ b/test/models/normalize_timdex_record_test.rb
@@ -199,7 +199,7 @@ def minimal_record
       'title' => ['Sample <em>TIMDEX</em> Record'],
       'summary' => ['comprehensive <em>test</em> record']
     }
-    assert_equal expected_highlight, normalized['highlight']
+    assert_equal expected_highlight, normalized[:highlight]
   end
 
   test 'includes TIMDEX-specific source_link field' do
diff --git a/test/models/normalize_timdex_results_test.rb b/test/models/normalize_timdex_results_test.rb
index c25cc62d..07be8a7b 100644
--- a/test/models/normalize_timdex_results_test.rb
+++ b/test/models/normalize_timdex_results_test.rb
@@ -30,7 +30,7 @@ def empty_timdex_response
     assert_includes first_result.keys, :content_type
     assert_includes first_result.keys, :dates
     assert_includes first_result.keys, :contributors
-    assert_includes first_result.keys, 'highlight'
+    assert_includes first_result.keys, :highlight
     assert_includes first_result.keys, 'source_link'
 
     # Check second record (minimal record)

From fc333d144d3cf84df904a06f007340a86ee9f7b0 Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Tue, 4 Nov 2025 13:32:09 -0500
Subject: [PATCH 16/17] TIMDEX source_link to sym

---
 app/helpers/search_helper.rb                 | 8 ++++----
 app/models/normalize_timdex_record.rb        | 2 +-
 test/helpers/search_helper_test.rb           | 8 ++++----
 test/models/normalize_timdex_record_test.rb  | 2 +-
 test/models/normalize_timdex_results_test.rb | 2 +-
 5 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/app/helpers/search_helper.rb b/app/helpers/search_helper.rb
index cc4ff968..d638c769 100644
--- a/app/helpers/search_helper.rb
+++ b/app/helpers/search_helper.rb
@@ -15,17 +15,17 @@ def format_highlight_label(field_name)
   end
 
   def link_to_result(result)
-    if result['source_link'].present?
-      link_to(result[:title], result['source_link'])
+    if result[:source_link].present?
+      link_to(result[:title], result[:source_link])
     else
       result[:title]
     end
   end
 
   def view_online(result)
-    return unless result['source_link'].present?
+    return unless result[:source_link].present?
 
-    link_to 'View online', result['source_link'], class: 'button button-primary'
+    link_to 'View online', result[:source_link], class: 'button button-primary'
   end
 
   def view_record(record_id)
diff --git a/app/models/normalize_timdex_record.rb b/app/models/normalize_timdex_record.rb
index 81fe495d..9851e9e0 100644
--- a/app/models/normalize_timdex_record.rb
+++ b/app/models/normalize_timdex_record.rb
@@ -25,7 +25,7 @@ def normalize
       dates:,
       contributors:,
       highlight:,
-      'source_link' => source_link
+      source_link:
     }
   end
 
diff --git a/test/helpers/search_helper_test.rb b/test/helpers/search_helper_test.rb
index e3c29320..ccb83f6a 100644
--- a/test/helpers/search_helper_test.rb
+++ b/test/helpers/search_helper_test.rb
@@ -27,7 +27,7 @@ class SearchHelperTest < ActionView::TestCase
   end
 
   test 'renders view_online link if source_link is present' do
-    result = { title: 'A record', 'source_link' => 'https://example.org' }
+    result = { title: 'A record', source_link: 'https://example.org' }
     assert_equal '<a class="button button-primary" href="https://example.org">View online</a>',
                  view_online(result)
   end
@@ -170,7 +170,7 @@ class SearchHelperTest < ActionView::TestCase
   test 'link_to_result returns link when source_link is present' do
     result = {
       title: 'Sample Document Title',
-      'source_link' => 'https://example.com/document'
+      source_link: 'https://example.com/document'
     }
     expected_link = '<a href="https://example.com/document">Sample Document Title</a>'
     assert_equal expected_link, link_to_result(result)
@@ -179,7 +179,7 @@ class SearchHelperTest < ActionView::TestCase
   test 'link_to_result returns plain title when source_link is nil' do
     result = {
       title: 'Sample Document Title',
-      'source_link' => nil
+      source_link: nil
     }
     assert_equal 'Sample Document Title', link_to_result(result)
   end
@@ -187,7 +187,7 @@ class SearchHelperTest < ActionView::TestCase
   test 'link_to_result returns plain title when source_link is empty string' do
     result = {
       title: 'Sample Document Title',
-      'source_link' => ''
+      source_link: ''
     }
     assert_equal 'Sample Document Title', link_to_result(result)
   end
diff --git a/test/models/normalize_timdex_record_test.rb b/test/models/normalize_timdex_record_test.rb
index df5e0555..d3540603 100644
--- a/test/models/normalize_timdex_record_test.rb
+++ b/test/models/normalize_timdex_record_test.rb
@@ -204,7 +204,7 @@ def minimal_record
 
   test 'includes TIMDEX-specific source_link field' do
     normalized = NormalizeTimdexRecord.new(full_record, 'test').normalize
-    assert_equal 'https://example.com/source/record/123', normalized['source_link']
+    assert_equal 'https://example.com/source/record/123', normalized[:source_link]
   end
 
   # Test that Primo-only fields are not included
diff --git a/test/models/normalize_timdex_results_test.rb b/test/models/normalize_timdex_results_test.rb
index 07be8a7b..c7e71267 100644
--- a/test/models/normalize_timdex_results_test.rb
+++ b/test/models/normalize_timdex_results_test.rb
@@ -31,7 +31,7 @@ def empty_timdex_response
     assert_includes first_result.keys, :dates
     assert_includes first_result.keys, :contributors
     assert_includes first_result.keys, :highlight
-    assert_includes first_result.keys, 'source_link'
+    assert_includes first_result.keys, :source_link
 
     # Check second record (minimal record)
     second_result = results.second

From 6b1a1142861b8f1fd58be54ea448802073f6717b Mon Sep 17 00:00:00 2001
From: Jeremy Prevost <JPrevost@users.noreply.github.com>
Date: Tue, 4 Nov 2025 14:06:07 -0500
Subject: [PATCH 17/17] Fixups

---
 app/helpers/record_helper.rb             |  2 +-
 app/views/shared/_geo_data_info.html.erb | 13 ++++++++++---
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/app/helpers/record_helper.rb b/app/helpers/record_helper.rb
index d4b70033..020777e3 100644
--- a/app/helpers/record_helper.rb
+++ b/app/helpers/record_helper.rb
@@ -14,7 +14,7 @@ def date_range(range)
   end
 
   def publication_date(metadata)
-    metadata['dates'].select { |date| date['kind'] == 'Publication date' }.first['value']
+    metadata[:dates].select { |date| date['kind'] == 'Publication date' }.first['value']
   end
 
   # Display the machine-format key in human-readable text.
diff --git a/app/views/shared/_geo_data_info.html.erb b/app/views/shared/_geo_data_info.html.erb
index 19362195..74254731 100644
--- a/app/views/shared/_geo_data_info.html.erb
+++ b/app/views/shared/_geo_data_info.html.erb
@@ -1,6 +1,13 @@
 <ul class="list-inline">
-  <li><%= metadata['contentType']&.each { |type| type['value'] }&.join(' ; ') %></li>
-  <% if parse_geo_dates(metadata['dates']) %>
+  <% if metadata[:content_type] %>
+    <li><%= metadata[:content_type]&.each { |type| type['value'] }&.join(' ; ') %></li>
+  <% elsif metadata['contentType'] %>
+    <li><%= metadata['contentType']&.each { |type| type['value'] }&.join(' ; ') %></li>
+  <% end %>
+
+  <% if parse_geo_dates(metadata[:dates]) %>
+    <li><%= parse_geo_dates(metadata[:dates]) %></li>
+  <% elsif parse_geo_dates(metadata['dates']) %>
     <li><%= parse_geo_dates(metadata['dates']) %></li>
   <% end %>
   <% if access_type(metadata) == 'unknown: check with owning institution' %>
@@ -11,4 +18,4 @@
   <% elsif access_type(metadata) == 'MIT authentication required' %>
     <li><span class="access-restricted"><%= access_type(metadata) %></span></li>
   <% end %>
-</ul>
\ No newline at end of file
+</ul>