Add helper method to handle cases where stimulus controller JS doesn't fire

Author: jazairi

app/helpers/results_helper.rb

@@ -80,4 +80,20 @@ def article?(format)
 
     format.match?(/\barticle\b/i)
   end
+
+  # Determines if a result has any fulfillment links to render
+  #
+  # @param result [Hash] A normalized Primo result hash
+  # @return [Boolean] True if the result has links, availability, or ThirdIron/OpenAlex triggers
+  def result_get?(result)
+    return true if result[:links].present? || result[:availability].present?
+    return true if Feature.enabled?(:oa_always) && article?(result[:format])
+
+    if ThirdIron.enabled?
+      return true if result[:doi].present? || result[:pmid].present?
+      return true if result[:format]&.downcase == 'journal' && result[:issn].present?
+    end
+
+    false
+  end
 end

app/javascript/controllers/content_loader_controller.js

@@ -25,7 +25,10 @@ export default class extends Controller {
             }
           }
         } else {
+          // Remove empty loader
           this.element.remove()
+
+          // Remove parent empty container, confirming first that it's empty
           if (!parentElement.textContent.trim()) {
             parentElement.remove()
           }

app/views/search/_result_primo.html.erb

@@ -75,6 +75,7 @@
         <% end %>
       </div>
 
+      <% if result_get?(result) %>
       <div class="result-get">
         <% if result[:links].present? %>
           <% result[:links].each do |link| %>
@@ -120,6 +121,7 @@
           <%= render(partial: 'trigger_browzine', locals: { issn: result[:issn] }) %>
         <% end %>
       </div>
+      <% end %>
     <% end %>
   </div>
 </li>

test/helpers/results_helper_test.rb

@@ -111,4 +111,55 @@ class ResultsHelperTest < ActionView::TestCase
     assert_not article?(nil)
     assert_not article?('')
   end
+
+  test 'result_get? returns true when result has links' do
+    assert result_get?({ links: [{ 'kind' => 'PDF', 'url' => 'https://example.com' }] })
+  end
+
+  test 'result_get? returns true when result has availability' do
+    assert result_get?({ availability: 'Available' })
+  end
+
+  test 'result_get? returns true when ThirdIron enabled and result has doi' do
+    ClimateControl.modify(THIRDIRON_ID: '123', THIRDIRON_KEY: 'abc') do
+      assert result_get?({ doi: '10.1234/test' })
+    end
+  end
+
+  test 'result_get? returns true when ThirdIron enabled and result has pmid' do
+    ClimateControl.modify(THIRDIRON_ID: '123', THIRDIRON_KEY: 'abc') do
+      assert result_get?({ pmid: '12345678' })
+    end
+  end
+
+  test 'result_get? returns true when ThirdIron enabled and result is a journal with issn' do
+    ClimateControl.modify(THIRDIRON_ID: '123', THIRDIRON_KEY: 'abc') do
+      assert result_get?({ format: 'Journal', issn: '1234-5678' })
+    end
+  end
+
+  test 'result_get? returns true when oa_always enabled and result is an article' do
+    ClimateControl.modify(FEATURE_OA_ALWAYS: 'true') do
+      assert result_get?({ format: 'Article' })
+    end
+  end
+
+  test 'result_get? returns false when result has no fulfillment content' do
+    ClimateControl.modify(THIRDIRON_ID: nil, THIRDIRON_KEY: nil, FEATURE_OA_ALWAYS: nil) do
+      assert_not result_get?({})
+      assert_not result_get?({ format: 'Book' })
+    end
+  end
+
+  test 'result_get? returns false when ThirdIron disabled even with doi' do
+    ClimateControl.modify(THIRDIRON_ID: nil, THIRDIRON_KEY: nil) do
+      assert_not result_get?({ doi: '10.1234/test' })
+    end
+  end
+
+  test 'result_get? returns false when oa_always disabled and result has no links' do
+    ClimateControl.modify(FEATURE_OA_ALWAYS: nil, THIRDIRON_ID: nil, THIRDIRON_KEY: nil) do
+      assert_not result_get?({ format: 'Article' })
+    end
+  end
 end