TableBuilder: Rename `#links_builder` to `#build_links`

The method name makes more sense to me as a verb.

Also add a little writespace making the code more vertical for
readability.

Refs #3479

1 files changed, 9 insertions, 3 deletions

diff --git a/app/helpers/table_builder_helper.rb b/app/helpers/table_builder_helper.rb
index 112a2f033..2d32207aa 100644
--- a/app/helpers/table_builder_helper.rb
+++ b/app/helpers/table_builder_helper.rb
@@ -107,7 +107,14 @@ module TableBuilderHelper
               bcont << content_tag(:td, value)
             end
           end
-          bcont << content_tag(:td, links_builder(item, links), class: 'actions') if links.any?
+
+          if links.any?
+            bcont << content_tag(
+              :td,
+              build_links(item, links),
+              class: 'actions'
+            )
+          end
 
           bcont.join.html_safe
         end
@@ -115,8 +122,7 @@ module TableBuilderHelper
     end
   end
 
-  # TODO: `def build_link[s]`
-  def links_builder(item, actions)
+  def build_links(item, actions)
     trigger = content_tag :div, class: 'btn dropdown-toggle', data: { toggle: 'dropdown' } do
       content_tag :span, '', class: 'fa fa-cog'
     end