Merge pull request #5560 from avalonmediasystem/collection_landing_we…

…bsite

Only return link html if website url present

app/presenters/collection_presenter.rb

@@ -52,7 +52,7 @@ def contact_email
   end
 
   def website_link
-    view_context.link_to document["website_label_ssi"], document["website_url_ssi"] unless document["website_url_ssi"].nil?
+    view_context.link_to document["website_label_ssi"], document["website_url_ssi"] if document["website_url_ssi"].present?
   end
 
   def as_json(_)

spec/presenters/collection_presenter_spec.rb

@@ -86,6 +86,14 @@
         expect(presenter.website_link).to be_nil
       end
     end
+
+    context 'with a blank website url' do
+      let(:website_url) { '' }
+
+      it 'returns null' do
+        expect(presenter.website_link).to be_nil
+      end
+    end
   end
 
   describe '#as_json' do