Skip to content

Commit 5920a74

Browse files
committed
Update Primo result partial to use new FRBR property
Why these changes are being introduced: The last commit in USE-329 introduced a bug. It renamed the `frbrized` property of normalized Primo records without making the corresponding change in the Primo result partial. This made the conditional that triggers the FRBR callout unreachable in the code. The test suite did not catch it because we do not test view behavior. Relevant ticket(s): - [USE-329](https://mitlibraries.atlassian.net/browse/USE-329) - [USE-354](https://mitlibraries.atlassian.net/browse/USE-354) How this addresses that need: This updates the Primo result partial to use the renamed property (`dedup_record`) to check for FRBRization. Side effects of this change: None.
1 parent e82cbd9 commit 5920a74

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

app/views/search/_result_primo.html.erb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
</h3>
1414

1515
<%# FRBR indicator: show a small callout and link to the Primo record %>
16-
<% if result[:frbrized] %>
16+
<% if result[:dedup_record] %>
1717
<div class="frbr-callout">
1818
<span>Multiple versions found.</span>
1919
<% frbr_link = result[:links]&.find { |link| link['kind'].to_s.downcase.include?('full') && link['kind'].to_s.downcase.include?('record') }&.dig('url') %>

0 commit comments

Comments
 (0)