From 775bb35fe93b76de740d5709bf38fd82563f8cff Mon Sep 17 00:00:00 2001
From: jazairi <16103405+jazairi@users.noreply.github.com>
Date: Mon, 24 Aug 2026 11:50:26 -0700
Subject: [PATCH] List full metadata for FRBR preferred records

This is a speculative change for stakeholder review only
---
 app/views/search/_result_primo.html.erb | 107 ++++++++++++------------
 1 file changed, 53 insertions(+), 54 deletions(-)

diff --git a/app/views/search/_result_primo.html.erb b/app/views/search/_result_primo.html.erb
index 7b8d47ac..ff43c851 100644
--- a/app/views/search/_result_primo.html.erb
+++ b/app/views/search/_result_primo.html.erb
@@ -14,69 +14,68 @@
       <% end %>
     </div>
 
-    <%# FRBR indicator: show a small callout and link to the Primo record %>
-    <% if result[:dedup_record] %>
-      <div class="frbr-callout">
-        <span>Multiple versions found.</span>
-        <% frbr_link = result[:links]&.find { |link| link['kind'].to_s.downcase.include?('full') && link['kind'].to_s.downcase.include?('record') }&.dig('url') %>
-        <% if frbr_link %>
-          <%= link_to 'Expand to all editions/formats', frbr_link, class: 'button frbr-link', aria: { label: "View all editions for #{result[:title]}" } %>
-        <% end %>
-      </div>
-    <% else %>
+    <div class="result-metadata">
+      <p class="pub-info">
+        <span><%= result[:year] %></span>
+      </p>
 
-      <div class="result-metadata">
-        <p class="pub-info">
-          <span><%= result[:year] %></span>
-        </p>
+      <%# unclear why TIMDEX is using contributors and Primo is using creators. Should we normalize this? %>
+      <% if result[:creators].present? %>
+        <span class="sr">Contributors: </span>
+        <ul class="list-inline truncate-list contributors">
+          <% result[:creators].each do |creator| %>
+            <li>
+              <% if creator[:link] %>
+                <%= link_to creator[:value], creator[:link] %>
+              <% else %>
+                <%= creator[:value] %>
+              <% end %>
+            </li>
+          <% end %>
+        </ul>
+      <% end %>
 
-        <%# unclear why TIMDEX is using contributors and Primo is using creators. Should we normalize this? %>
-        <% if result[:creators].present? %>
-          <span class="sr">Contributors: </span>
-          <ul class="list-inline truncate-list contributors">
-            <% result[:creators].each do |creator| %>
-              <li>
-                <% if creator[:link] %>
-                  <%= link_to creator[:value], creator[:link] %>
-                <% else %>
-                  <%= creator[:value] %>
-                <% end %>
-              </li>
-            <% end %>
-          </ul>
+      <p class="result-source">Source:
+        <% if result[:source_url].present? %>
+          <%= link_to result[:source], result[:source_url] %>
+        <% else %>
+          <%= result[:source] %>
         <% end %>
+      </p>
 
-        <p class="result-source">Source:
-          <% if result[:source_url].present? %>
-            <%= link_to result[:source], result[:source_url] %>
-          <% else %>
-            <%= result[:source] %>
-          <% end %>
-        </p>
+      <% if result[:citation].present? %>
+        <div class="result-container">
+          <span>In: </span><%= result[:citation] %>
+        </div>
+      <% end %>
 
-        <% if result[:citation].present? %>
-          <div class="result-container">
-            <span>In: </span><%= result[:citation] %>
-          </div>
-        <% end %>
+      <% if result[:subjects].present? %>
+        <div class="result-subjects truncate-list">
+          <span class="sr">Subjects: </span>
+          <ul class="list-inline truncate-list subjects">
+            <% result[:subjects].each do |subject| %>
+              <li><%= subject %></li>
+            <% end %>
+          </ul>
+        </div>
+      <% end %>
 
-        <% if result[:subjects].present? %>
-          <div class="result-subjects truncate-list">
-            <span class="sr">Subjects: </span>
-            <ul class="list-inline truncate-list subjects">
-              <% result[:subjects].each do |subject| %>
-                <li><%= subject %></li>
-              <% end %>
-            </ul>
-          </div>
-        <% end %>
+      <% if result[:summary].present? %>
+        <div class="result-summary truncate-list">
+          <h4 class="sr">Summary: </h4><%= sanitize result[:summary] %>
+        </div>
+      <% end %>
+    </div>
 
-        <% if result[:summary].present? %>
-          <div class="result-summary truncate-list">
-            <h4 class="sr">Summary: </h4><%= sanitize result[:summary] %>
-          </div>
+    <% if result[:dedup_record] %>
+      <div class="frbr-callout">
+        <span>Multiple versions found.</span>
+        <% frbr_link = result[:links]&.find { |link| link['kind'].to_s.downcase.include?('full') && link['kind'].to_s.downcase.include?('record') }&.dig('url') %>
+        <% if frbr_link %>
+          <%= link_to 'Expand to all editions/formats', frbr_link, class: 'button frbr-link', aria: { label: "View all editions for #{result[:title]}" } %>
         <% end %>
       </div>
+    <% else %>
 
       <% if AlmaSru.enabled? && AlmaSru.valid_alma_id?(result[:identifier]) %>
         <%= render(partial: 'trigger_almasru', locals: { doc_id: result[:identifier] }) %>