TableBuilderHelper#thead: Change `links` argument to `has_links`

Since this method doesn't depend on the actual list of links, don't ask
for it as an argument. Instead, just pass the answer to "are there any
links?".

Refs #3479

1 files changed, 3 insertions, 3 deletions

diff --git a/app/helpers/table_builder_helper.rb b/app/helpers/table_builder_helper.rb
index ef728a12d..95c84d50e 100644
--- a/app/helpers/table_builder_helper.rb
+++ b/app/helpers/table_builder_helper.rb
@@ -18,14 +18,14 @@ module TableBuilderHelper
   )
 
     content_tag :table,
-      thead(collection, columns, selectable, links) +
+      thead(collection, columns, selectable, links.any?) +
         tbody(collection, columns, selectable, links),
       class: cls
   end
 
   private
 
-  def thead(collection, columns, selectable, links)
+  def thead(collection, columns, selectable, has_links)
     content_tag :thead do
       content_tag :tr do
         hcont = []
@@ -44,7 +44,7 @@ module TableBuilderHelper
           end
         end
         # Inserts a blank column for the gear menu
-        hcont << content_tag(:th, '') if links.any?
+        hcont << content_tag(:th, '') if has_links
 
         hcont.join.html_safe
       end