Merge pull request #22 from af83/3479-refactor-table_builder-helper

3479 refactor table builder helper

29 files changed, 864 insertions, 91 deletions

diff --git a/app/controllers/calendars_controller.rb b/app/controllers/calendars_controller.rb
index 86d567882..5662316b8 100644
--- a/app/controllers/calendars_controller.rb
+++ b/app/controllers/calendars_controller.rb
@@ -5,6 +5,12 @@ class CalendarsController < BreadcrumbController
   respond_to :html
   respond_to :js, only: :index
 
+  def show
+    show! do
+      @calendar = @calendar.decorate
+    end
+  end
+
   private
   def calendar_params
     permitted_params = [:id, :name, :short_name, periods_attributes: [:id, :begin, :end, :_destroy], date_values_attributes: [:id, :value, :_destroy]]
diff --git a/app/controllers/lines_controller.rb b/app/controllers/lines_controller.rb
index 7eedaeb05..1e2056aad 100644
--- a/app/controllers/lines_controller.rb
+++ b/app/controllers/lines_controller.rb
@@ -25,6 +25,11 @@ class LinesController < BreadcrumbController
   def show
     @group_of_lines = resource.group_of_lines
     show! do
+      @line = @line.decorate(context: {
+        line_referential: @line_referential,
+        current_organisation: current_organisation
+      })
+
       build_breadcrumb :show
     end
   end
diff --git a/app/controllers/referentials_controller.rb b/app/controllers/referentials_controller.rb
index aa5b359da..1239d512f 100644
--- a/app/controllers/referentials_controller.rb
+++ b/app/controllers/referentials_controller.rb
@@ -24,19 +24,21 @@ class ReferentialsController < BreadcrumbController
   end
 
   def show
-     resource.switch
-     show! do |format|
-       format.json {
-         render :json => { :lines_count => resource.lines.count,
-                :networks_count => resource.networks.count,
-                :vehicle_journeys_count => resource.vehicle_journeys.count + resource.vehicle_journey_frequencies.count,
-                :time_tables_count => resource.time_tables.count,
-                :referential_id => resource.id}
-       }
-       format.html { build_breadcrumb :show}
-     end
-
-     @reflines = lines_collection.paginate(page: params[:page], per_page: 10)
+    resource.switch
+    show! do |format|
+      @referential = @referential.decorate
+
+      format.json {
+        render :json => { :lines_count => resource.lines.count,
+               :networks_count => resource.networks.count,
+               :vehicle_journeys_count => resource.vehicle_journeys.count + resource.vehicle_journey_frequencies.count,
+               :time_tables_count => resource.time_tables.count,
+               :referential_id => resource.id}
+      }
+      format.html { build_breadcrumb :show}
+    end
+
+    @reflines = lines_collection.paginate(page: params[:page], per_page: 10)
   end
 
   def edit
diff --git a/app/controllers/routes_controller.rb b/app/controllers/routes_controller.rb
index 73febc4b9..786bd57cc 100644
--- a/app/controllers/routes_controller.rb
+++ b/app/controllers/routes_controller.rb
@@ -42,6 +42,12 @@ class RoutesController < ChouetteController
     end
 
     show! do
+      @route = @route.decorate(context: {
+        referential: @referential,
+        line: @line,
+        route_sp: @route_sp
+      })
+
       build_breadcrumb :show
     end
   end
diff --git a/app/controllers/routing_constraint_zones_controller.rb b/app/controllers/routing_constraint_zones_controller.rb
index 7707427b0..9d2fd712c 100644
--- a/app/controllers/routing_constraint_zones_controller.rb
+++ b/app/controllers/routing_constraint_zones_controller.rb
@@ -16,6 +16,10 @@ class RoutingConstraintZonesController < ChouetteController
 
   def show
     @routing_constraint_zone = collection.find(params[:id])
+    @routing_constraint_zone = @routing_constraint_zone.decorate(context: {
+      referential: @referential,
+      line: @line
+    })
   end
 
   protected
diff --git a/app/controllers/time_tables_controller.rb b/app/controllers/time_tables_controller.rb
index 5c4552afb..3704f2885 100644
--- a/app/controllers/time_tables_controller.rb
+++ b/app/controllers/time_tables_controller.rb
@@ -14,6 +14,10 @@ class TimeTablesController < ChouetteController
     @year = params[:year] ? params[:year].to_i : Date.today.cwyear
     @time_table_combination = TimeTableCombination.new
     show! do
+      @time_table = @time_table.decorate(context: {
+        referential: @referential
+      })
+
       build_breadcrumb :show
     end
   end
diff --git a/app/controllers/workbenches_controller.rb b/app/controllers/workbenches_controller.rb
index ccd55965b..1447c27de 100644
--- a/app/controllers/workbenches_controller.rb
+++ b/app/controllers/workbenches_controller.rb
@@ -14,6 +14,10 @@ class WorkbenchesController < BreadcrumbController
     q_for_result =
       scope.ransack(params[:q].merge(archived_at_not_null: nil, archived_at_null: nil))
     @wbench_refs = sort_result(q_for_result.result).paginate(page: params[:page], per_page: 30)
+    @wbench_refs = ModelDecorator.decorate(
+      @wbench_refs,
+      with: ReferentialDecorator
+    )
 
     @q = scope.ransack(params[:q])
     show! do
diff --git a/app/decorators/calendar_decorator.rb b/app/decorators/calendar_decorator.rb
new file mode 100644
index 000000000..37e2cfe80
--- /dev/null
+++ b/app/decorators/calendar_decorator.rb
@@ -0,0 +1,18 @@
+class CalendarDecorator < Draper::Decorator
+  delegate_all
+
+  def action_links
+    links = []
+
+    if h.policy(object).destroy?
+      links << Link.new(
+        content: h.destroy_link_content,
+        href: h.calendar_path(object),
+        method: :delete,
+        data: { confirm: h.t('calendars.actions.destroy_confirm') }
+      )
+    end
+
+    links
+  end
+end
diff --git a/app/decorators/company_decorator.rb b/app/decorators/company_decorator.rb
index 3a0cc16ce..51c1f3c61 100644
--- a/app/decorators/company_decorator.rb
+++ b/app/decorators/company_decorator.rb
@@ -1,4 +1,6 @@
 class CompanyDecorator < Draper::Decorator
+  decorates Chouette::Company
+
   delegate_all
 
   def self.collection_decorator_class
@@ -9,4 +11,43 @@ class CompanyDecorator < Draper::Decorator
     object.lines.count
   end
 
+  # Requires:
+  #   context: {
+  #     line_referential:
+  #   }
+  def action_links
+    links = []
+
+    if h.policy(Chouette::Company).create?
+      links << Link.new(
+        content: h.t('companies.actions.new'),
+        href: h.new_line_referential_company_path(context[:line_referential])
+      )
+    end
+
+    if h.policy(object).update?
+      links << Link.new(
+        content: h.t('companies.actions.edit'),
+        href: h.edit_line_referential_company_path(
+          context[:line_referential],
+          object
+        )
+      )
+    end
+
+    if h.policy(object).destroy?
+      links << Link.new(
+        content: t('companies.actions.destroy'),
+        href: h.line_referential_company_path(
+          context[:line_referential],
+          object
+        ),
+        method: :delete,
+        data: { confirm: h.t('companies.actions.destroy_confirm') }
+      )
+    end
+
+    links
+  end
+
 end
diff --git a/app/decorators/line_decorator.rb b/app/decorators/line_decorator.rb
new file mode 100644
index 000000000..f351103b2
--- /dev/null
+++ b/app/decorators/line_decorator.rb
@@ -0,0 +1,45 @@
+class LineDecorator < Draper::Decorator
+  decorates Chouette::Line
+
+  delegate_all
+
+  # Requires:
+  #   context: {
+  #     line_referential: ,
+  #     current_organisation:
+  #   }
+  def action_links
+    links = []
+
+    links << Link.new(
+      content: h.t('lines.actions.show_network'),
+      href: [context[:line_referential], object.network]
+    )
+
+    links << Link.new(
+      content: h.t('lines.actions.show_company'),
+      href: [context[:line_referential], object.company]
+    )
+
+    if h.policy(Chouette::Line).create? &&
+        context[:line_referential].organisations.include?(
+          context[:current_organisation]
+        )
+      links << Link.new(
+        content: h.t('lines.actions.new'),
+        href: h.new_line_referential_line_path(context[:line_referential])
+      )
+    end
+
+    if h.policy(object).destroy?
+      links << Link.new(
+        content: h.destroy_link_content('lines.actions.destroy_confirm'),
+        href: h.line_referential_line_path(context[:line_referential], object),
+        method: :delete,
+        data: { confirm: h.t('lines.actions.destroy_confirm') }
+      )
+    end
+
+    links
+  end
+end
diff --git a/app/decorators/model_decorator.rb b/app/decorators/model_decorator.rb
new file mode 100644
index 000000000..dee014cc3
--- /dev/null
+++ b/app/decorators/model_decorator.rb
@@ -0,0 +1,3 @@
+class ModelDecorator < PaginatingDecorator
+  delegate :model
+end
diff --git a/app/decorators/referential_decorator.rb b/app/decorators/referential_decorator.rb
new file mode 100644
index 000000000..9107de0b3
--- /dev/null
+++ b/app/decorators/referential_decorator.rb
@@ -0,0 +1,56 @@
+class ReferentialDecorator < Draper::Decorator
+  delegate_all
+
+  def action_links
+    links = [
+      Link.new(
+        content: h.t('time_tables.index.title'),
+        href: h.referential_time_tables_path(object)
+      )
+    ]
+
+    if h.policy(object).clone?
+      links << Link.new(
+        content: h.t('actions.clone'),
+        href: h.new_referential_path(from: object.id)
+      )
+    end
+
+    if h.policy(object).edit?
+      links << HTMLElement.new(
+        :button,
+        'Purger',
+        type: 'button',
+        data: {
+          toggle: 'modal',
+          target: '#purgeModal'
+        }
+      )
+
+      if object.archived?
+        links << Link.new(
+          content: h.t('actions.unarchive'),
+          href: h.unarchive_referential_path(object.id),
+          method: :put
+        )
+      else
+        links << Link.new(
+          content: h.t('actions.archive'),
+          href: h.archive_referential_path(object.id),
+          method: :put
+        )
+      end
+    end
+
+    if h.policy(object).destroy?
+      links << Link.new(
+        content: h.destroy_link_content,
+        href: h.referential_path(object),
+        method: :delete,
+        data: { confirm: h.t('referentials.actions.destroy_confirm') }
+      )
+    end
+
+    links
+  end
+end
diff --git a/app/decorators/route_decorator.rb b/app/decorators/route_decorator.rb
new file mode 100644
index 000000000..99b174dff
--- /dev/null
+++ b/app/decorators/route_decorator.rb
@@ -0,0 +1,64 @@
+class RouteDecorator < Draper::Decorator
+  decorates Chouette::Route
+
+  delegate_all
+
+  # Requires:
+  #   context: {
+  #     referential: ,
+  #     line: ,
+  #     route_sp
+  #   }
+  def action_links
+    links = []
+
+    if context[:route_sp].any?
+      links << Link.new(
+        content: h.t('journey_patterns.index.title'),
+        href: [
+          context[:referential],
+          context[:line],
+          object,
+          :journey_patterns_collection
+        ]
+      )
+    end
+
+    if object.journey_patterns.present?
+      links << Link.new(
+        content: h.t('vehicle_journeys.actions.index'),
+        href: [
+          context[:referential],
+          context[:line],
+          object,
+          :vehicle_journeys
+        ]
+      )
+    end
+
+    links << Link.new(
+      content: h.t('vehicle_journey_exports.new.title'),
+      href: h.referential_line_route_vehicle_journey_exports_path(
+        context[:referential],
+        context[:line],
+        object,
+        format: :zip
+      )
+    )
+
+    if h.policy(object).destroy?
+      links << Link.new(
+        content: h.destroy_link_content,
+        href: h.referential_line_route_path(
+          context[:referential],
+          context[:line],
+          object
+        ),
+        method: :delete,
+        data: { confirm: h.t('routes.actions.destroy_confirm') }
+      )
+    end
+
+    links
+  end
+end
diff --git a/app/decorators/routing_constraint_zone_decorator.rb b/app/decorators/routing_constraint_zone_decorator.rb
new file mode 100644
index 000000000..0b438a554
--- /dev/null
+++ b/app/decorators/routing_constraint_zone_decorator.rb
@@ -0,0 +1,42 @@
+class RoutingConstraintZoneDecorator < Draper::Decorator
+  decorates Chouette::RoutingConstraintZone
+
+  delegate_all
+
+  # Requires:
+  #   context: {
+  #     referential: ,
+  #     line:
+  #   }
+  def action_links
+    links = []
+
+    if h.policy(object).update?
+      links << Link.new(
+        content: h.t('actions.edit'),
+        href: h.edit_referential_line_routing_constraint_zone_path(
+          context[:referential],
+          context[:line],
+          object
+        )
+      )
+    end
+
+    if h.policy(object).destroy?
+      links << Link.new(
+        content: h.destroy_link_content,
+        href: h.referential_line_routing_constraint_zone_path(
+          context[:referential],
+          context[:line],
+          object
+        ),
+        method: :delete,
+        data: {
+          confirm: h.t('routing_constraint_zones.actions.destroy_confirm')
+        }
+      )
+    end
+
+    links
+  end
+end
diff --git a/app/decorators/time_table_decorator.rb b/app/decorators/time_table_decorator.rb
new file mode 100644
index 000000000..526537310
--- /dev/null
+++ b/app/decorators/time_table_decorator.rb
@@ -0,0 +1,53 @@
+class TimeTableDecorator < Draper::Decorator
+  decorates Chouette::TimeTable
+
+  delegate_all
+
+  # Requires:
+  #   context: {
+  #     referential: ,
+  #   }
+  def action_links
+    links = []
+
+    if object.calendar
+      links << Link.new(
+        content: h.t('actions.actualize'),
+        href: h.actualize_referential_time_table_path(
+          context[:referential],
+          object
+        ),
+        method: :post
+      )
+    end
+
+    links << Link.new(
+      content: h.t('actions.combine'),
+      href: h.new_referential_time_table_time_table_combination_path(
+        context[:referential],
+        object
+      )
+    )
+
+    if h.policy(object).duplicate?
+      links << Link.new(
+        content: h.t('actions.duplicate'),
+        href: h.duplicate_referential_time_table_path(
+          context[:referential],
+          object
+        )
+      )
+    end
+
+    if h.policy(object).destroy?
+      links << Link.new(
+        content: h.destroy_link_content,
+        href: h.referential_time_table_path(context[:referential], object),
+        method: :delete,
+        data: { confirm: h.t('time_tables.actions.destroy_confirm') }
+      )
+    end
+
+    links
+  end
+end
diff --git a/app/helpers/links_helper.rb b/app/helpers/links_helper.rb
new file mode 100644
index 000000000..683b66a52
--- /dev/null
+++ b/app/helpers/links_helper.rb
@@ -0,0 +1,5 @@
+module LinksHelper
+  def destroy_link_content(translation_key = 'actions.destroy')
+    content_tag(:span, nil, class: 'fa fa-trash') + t(translation_key)
+  end
+end
diff --git a/app/helpers/multiple_selection_toolbox_helper.rb b/app/helpers/multiple_selection_toolbox_helper.rb
new file mode 100644
index 000000000..85294af6d
--- /dev/null
+++ b/app/helpers/multiple_selection_toolbox_helper.rb
@@ -0,0 +1,40 @@
+module MultipleSelectionToolboxHelper
+  # Box of links that floats at the bottom right of the page
+  def multiple_selection_toolbox(actions)
+    links = content_tag :ul do
+      delete_path = nil
+
+      if params[:controller] = 'workbenches'
+        delete_path = referentials_workbench_path
+      end
+
+      actions.map do |action|
+        if action == :delete
+          action_link = link_to(
+            '#',
+            method: :delete,
+            data: {
+              path: delete_path,
+              confirm: 'Etes-vous sûr(e) de vouloir effectuer cette action ?'
+            },
+            title: t("actions.#{action}")
+          ) do
+            content_tag :span, '', class: 'fa fa-trash'
+          end
+        end
+
+        content_tag :li, action_link, class: 'st_action'
+      end.join.html_safe
+    end
+
+    label = content_tag(
+      :span,
+      ("<span>0</span> élément(s) sélectionné(s)").html_safe,
+      class: 'info-msg'
+    )
+
+    content_tag :div, '', class: 'select_toolbox noselect' do
+      links + label
+    end
+  end
+end
diff --git a/app/helpers/table_builder_helper.rb b/app/helpers/table_builder_helper.rb
new file mode 100644
index 000000000..b93e9b22b
--- /dev/null
+++ b/app/helpers/table_builder_helper.rb
@@ -0,0 +1,234 @@
+require 'table_builder_helper/column'
+require 'table_builder_helper/custom_links'
+require 'table_builder_helper/url'
+
+# table_builder_2
+# A Rails helper that constructs an HTML table from a collection of objects. It
+# receives the collection and an array of columns that get transformed into
+# `<td>`s. A column of checkboxes can be added to the left side of the table
+# for multiple selection. Columns are sortable by default, but sorting can be
+# disabled either at the table level or at the column level. An optional
+# `links` argument takes a set of symbols corresponding to controller actions
+# that should be inserted in a gear menu next to each row in the table. That
+# menu will also be populated with links defined in `collection#action_links`,
+# a list of `Link` objects defined in a decorator for the given object.
+#
+# Depends on `params` and `current_referential`.
+#
+# Example:
+#   table_builder_2(
+#     @companies,
+#     [
+#       TableBuilderHelper::Column.new(
+#         name: 'ID Codif',
+#         attribute: Proc.new { |n| n.try(:objectid).try(:local_id) },
+#         sortable: false
+#       ),
+#       TableBuilderHelper::Column.new(
+#         key: :name,
+#         attribute: 'name'
+#       ),
+#       TableBuilderHelper::Column.new(
+#         key: :phone,
+#         attribute: 'phone'
+#       ),
+#       TableBuilderHelper::Column.new(
+#         key: :email,
+#         attribute: 'email'
+#       ),
+#       TableBuilderHelper::Column.new(
+#         key: :url,
+#         attribute: 'url'
+#       ),
+#     ],
+#     links: [:show, :edit],
+#     cls: 'table has-search'
+#   )
+module TableBuilderHelper
+  # TODO: rename this after migration from `table_builder`
+  def table_builder_2(
+    # An `ActiveRecord::Relation`, wrapped in a decorator to provide a list of
+    # `Link` objects via an `#action_links` method
+    collection,
+
+    # An array of `TableBuilderHelper::Column`s
+    columns,
+
+    # When false, no columns will be sortable
+    sortable: true,
+
+    # When true, adds a column of checkboxes to the left side of the table
+    selectable: false,
+
+    # A set of controller actions that will be added as links to the top of the
+    # gear menu
+    links: [],
+
+    # A CSS class to apply to the <table>
+    cls: ''
+  )
+    content_tag :table,
+      thead(collection, columns, sortable, selectable, links.any?) +
+        tbody(collection, columns, selectable, links),
+      class: cls
+  end
+
+  private
+
+  def thead(collection, columns, sortable, selectable, has_links)
+    content_tag :thead do
+      content_tag :tr do
+        hcont = []
+
+        if selectable
+          hcont << content_tag(:th, checkbox(id_name: '0', value: 'all'))
+        end
+
+        columns.each do |column|
+          hcont << content_tag(:th, build_column_header(
+            column,
+            sortable,
+            collection.model,
+            params,
+            params[:sort],
+            params[:direction]
+          ))
+        end
+
+        # Inserts a blank column for the gear menu
+        hcont << content_tag(:th, '') if has_links
+
+        hcont.join.html_safe
+      end
+    end
+  end
+
+  def tbody(collection, columns, selectable, links)
+    content_tag :tbody do
+      collection.map do |item|
+
+        content_tag :tr do
+          bcont = []
+
+          if selectable
+            bcont << content_tag(
+              :td,
+              checkbox(id_name: item.try(:id), value: item.try(:id))
+            )
+          end
+
+          columns.each do |column|
+            value = column.value(item)
+
+            if column_is_linkable?(column)
+              # Build a link to the `item`
+              polymorph_url = URL.polymorphic_url_parts(item)
+              bcont << content_tag(:td, link_to(value, polymorph_url), title: 'Voir')
+            else
+              bcont << content_tag(:td, value)
+            end
+          end
+
+          if links.any?
+            bcont << content_tag(
+              :td,
+              build_links(item, links),
+              class: 'actions'
+            )
+          end
+
+          bcont.join.html_safe
+        end
+      end.join.html_safe
+    end
+  end
+
+  def build_links(item, links)
+    trigger = content_tag(
+      :div,
+      class: 'btn dropdown-toggle',
+      data: { toggle: 'dropdown' }
+    ) do
+      content_tag :span, '', class: 'fa fa-cog'
+    end
+
+    menu = content_tag :ul, class: 'dropdown-menu' do
+      (
+        CustomLinks.new(item, pundit_user, links).links +
+        item.action_links.select { |link| link.is_a?(Link) }
+      ).map do |link|
+        gear_menu_link(link)
+      end.join.html_safe
+    end
+
+    content_tag :div, trigger + menu, class: 'btn-group'
+  end
+
+  def build_column_header(
+    column,
+    table_is_sortable,
+    collection_model,
+    params,
+    sort_on,
+    sort_direction
+  )
+    if !table_is_sortable
+      return column.header_label(collection_model)
+    end
+
+    return column.name if !column.sortable
+
+    direction =
+      if column.key.to_s == sort_on && sort_direction == 'desc'
+        'asc'
+      else
+        'desc'
+      end
+
+    link_to(params.merge({direction: direction, sort: column.key})) do
+      arrow_up = content_tag(
+        :span,
+        '',
+        class: "fa fa-sort-asc #{direction == 'desc' ? 'active' : ''}"
+      )
+      arrow_down = content_tag(
+        :span,
+        '',
+        class: "fa fa-sort-desc #{direction == 'asc' ? 'active' : ''}"
+      )
+
+      arrow_icons = content_tag :span, arrow_up + arrow_down, class: 'orderers'
+
+      (
+        column.header_label(collection_model) +
+        arrow_icons
+      ).html_safe
+    end
+  end
+
+  def checkbox(id_name:, value:)
+    content_tag :div, '', class: 'checkbox' do
+      check_box_tag(id_name, value).concat(
+        content_tag(:label, '', for: id_name)
+      )
+    end
+  end
+
+  def column_is_linkable?(column)
+    column.attribute == 'name' || column.attribute == 'comment'
+  end
+
+  def gear_menu_link(link)
+    content_tag(
+      :li,
+      link_to(
+        link.href,
+        method: link.method,
+        data: link.data
+      ) do
+        link.content
+      end,
+      class: ('delete-action' if link.method == :delete)
+    )
+  end
+end
diff --git a/app/helpers/table_builder_helper/column.rb b/app/helpers/table_builder_helper/column.rb
new file mode 100644
index 000000000..800a8282e
--- /dev/null
+++ b/app/helpers/table_builder_helper/column.rb
@@ -0,0 +1,36 @@
+module TableBuilderHelper
+  class Column
+    attr_reader :key, :name, :attribute, :sortable
+
+    def initialize(key: nil, name: '', attribute:, sortable: true)
+      if key.nil? && name.empty?
+        raise ColumnMustHaveKeyOrNameError
+      end
+
+      @key = key
+      @name = name
+      @attribute = attribute
+      @sortable = sortable
+    end
+
+    def value(obj)
+      if @attribute.is_a?(Proc)
+        @attribute.call(obj)
+      else
+        obj.try(@attribute)
+      end
+    end
+
+    def header_label(model = nil)
+      return @name unless @name.empty?
+
+      # Transform `Chouette::Line` into "line"
+      model_key = model.to_s.demodulize.underscore
+
+      I18n.t("activerecord.attributes.#{model_key}.#{@key}")
+    end
+  end
+
+
+  class ColumnMustHaveKeyOrNameError < StandardError; end
+end
diff --git a/app/helpers/table_builder_helper/custom_links.rb b/app/helpers/table_builder_helper/custom_links.rb
new file mode 100644
index 000000000..abb907678
--- /dev/null
+++ b/app/helpers/table_builder_helper/custom_links.rb
@@ -0,0 +1,77 @@
+require 'table_builder_helper/url'
+
+module TableBuilderHelper
+  class CustomLinks
+    ACTIONS_TO_HTTP_METHODS = {
+      delete: :delete,
+      archive: :put,
+      unarchive: :put
+    }
+
+    def initialize(obj, user_context, actions)
+      @obj = obj
+      @user_context = user_context
+      @actions = actions
+    end
+
+    def links
+      actions_after_policy_check.map do |action|
+        Link.new(
+          content: I18n.t("actions.#{action}"),
+          href: polymorphic_url(action),
+          method: method_for_action(action)
+        )
+      end
+    end
+
+    def polymorphic_url(action)
+      polymorph_url = []
+
+      unless [:show, :delete].include?(action)
+        polymorph_url << action
+      end
+
+      polymorph_url += URL.polymorphic_url_parts(@obj)
+    end
+
+    def method_for_action(action)
+      ACTIONS_TO_HTTP_METHODS[action]
+    end
+
+    def actions_after_policy_check
+      @actions.select do |action|
+        # Has policy and can destroy
+        (action == :delete &&
+            Pundit.policy(@user_context, @obj).present? &&
+            Pundit.policy(@user_context, @obj).destroy?) ||
+
+          # Doesn't have policy
+          (action == :delete &&
+            !Pundit.policy(@user_context, @obj).present?) ||
+
+          # Has policy and can update
+          (action == :edit &&
+            Pundit.policy(@user_context, @obj).present? &&
+            Pundit.policy(@user_context, @obj).update?) ||
+
+          # Doesn't have policy
+          (action == :edit &&
+            !Pundit.policy(@user_context, @obj).present?) ||
+
+          # Object isn't archived
+          (action == :archive && !@obj.archived?) ||
+
+          # Object is archived
+          (action == :unarchive && @obj.archived?) ||
+
+          action_is_allowed_regardless_of_policy(action)
+      end
+    end
+
+    private
+
+    def action_is_allowed_regardless_of_policy(action)
+      ![:delete, :edit, :archive, :unarchive].include?(action)
+    end
+  end
+end
diff --git a/app/helpers/table_builder_helper/url.rb b/app/helpers/table_builder_helper/url.rb
new file mode 100644
index 000000000..f60864ac1
--- /dev/null
+++ b/app/helpers/table_builder_helper/url.rb
@@ -0,0 +1,25 @@
+module TableBuilderHelper
+  # Depends on `current_referential`, defined in object controllers
+  class URL
+    def self.polymorphic_url_parts(item)
+      polymorph_url = []
+
+      unless item.is_a?(Calendar) || item.is_a?(Referential)
+        if current_referential
+          polymorph_url << current_referential
+          polymorph_url << item.line if item.respond_to? :line
+          polymorph_url << item.route.line if item.is_a?(Chouette::RoutingConstraintZone)
+          polymorph_url << item if item.respond_to? :line_referential
+          polymorph_url << item.stop_area if item.respond_to? :stop_area
+          polymorph_url << item if item.respond_to? :stop_points || item.is_a?(Chouette::TimeTable)
+        elsif item.respond_to? :referential
+          polymorph_url << item.referential
+        end
+      else
+        polymorph_url << item
+      end
+
+      polymorph_url
+    end
+  end
+end
diff --git a/app/views/calendars/show.html.slim b/app/views/calendars/show.html.slim
index 3886cefaa..26248cea8 100644
--- a/app/views/calendars/show.html.slim
+++ b/app/views/calendars/show.html.slim
@@ -8,10 +8,12 @@
   / Below is secondary actions & optional contents (filters, ...)
   .row.mb-sm
     .col-lg-12.text-right
-      - if policy(@calendar).destroy?
-        = link_to calendar_path(@calendar), method: :delete, data: { confirm: t('calendars.actions.destroy_confirm') }, class: 'btn btn-primary' do
-          span.fa.fa-trash
-          span = t('actions.destroy')
+      - @calendar.action_links.each do |link|
+        = link_to link.href,
+            method: link.method,
+            data: link.data,
+            class: 'btn btn-primary' do
+              = link.content
 
 / PageContent
 .page_content
diff --git a/app/views/lines/show.html.slim b/app/views/lines/show.html.slim
index dbc019e72..6f75432e1 100644
--- a/app/views/lines/show.html.slim
+++ b/app/views/lines/show.html.slim
@@ -7,17 +7,12 @@
   / Below is secundary actions & optional contents
   .row
     .col-lg-12.text-right.mb-sm
-      = link_to t('lines.actions.show_network'), [@line_referential, @line.network], class: 'btn btn-primary'
-      = link_to t('lines.actions.show_company'), [@line_referential, @line.company], class: 'btn btn-primary'
-
-      - if policy(Chouette::Line).create? && @line_referential.organisations.include?(current_organisation)
-        = link_to t('lines.actions.new'), new_line_referential_line_path(@line_referential), class: 'btn btn-primary'
-      - if false && policy(@line).update?
-        = link_to t('lines.actions.edit'), edit_line_referential_line_path(@line_referential, @line), class: 'btn btn-primary'
-      - if policy(@line).destroy?
-        = link_to line_referential_line_path(@line_referential, @line), method: :delete, data: {confirm:  t('lines.actions.destroy_confirm')}, class: 'btn btn-primary' do
-          span.fa.fa-trash
-          span = t('lines.actions.destroy')
+      - @line.action_links.each do |link|
+        = link_to link.href,
+            method: link.method,
+            data: link.data,
+            class: 'btn btn-primary' do
+              = link.content
 
 / PageContent
 .page_content
diff --git a/app/views/referentials/show.html.slim b/app/views/referentials/show.html.slim
index 3c1e36302..dfa264c0b 100644
--- a/app/views/referentials/show.html.slim
+++ b/app/views/referentials/show.html.slim
@@ -8,23 +8,15 @@
   / Below is secondary actions & optional contents (filters, ...)
   .row.mb-sm
     .col-lg-12.text-right
-      = link_to t('time_tables.index.title'), referential_time_tables_path(@referential), class: 'btn btn-primary'
-
-      - if policy(@referential).clone?
-        = link_to t('actions.clone'), new_referential_path(from: @referential.id), class: 'btn btn-primary'
-
-      - if policy(@referential).edit?
-        button.btn.btn-primary type='button' data-toggle='modal' data-target='#purgeModal' Purger
-
-        - if @referential.archived?
-          = link_to t('actions.unarchive'), unarchive_referential_path(@referential.id), method: :put, class: 'btn btn-primary'
+      - @referential.action_links.each do |link|
+        - if link.is_a?(HTMLElement)
+          = link.to_html(class: 'btn btn-primary')
         - else
-          = link_to t('actions.archive'), archive_referential_path(@referential.id), method: :put, class: 'btn btn-primary'
-
-      - if policy(@referential).destroy?
-        = link_to referential_path(@referential), method: :delete, data: {confirm: t('referentials.actions.destroy_confirm')}, class: 'btn btn-primary' do
-          span.fa.fa-trash
-          span = t('actions.destroy')
+          = link_to link.href,
+              method: link.method,
+              data: link.data,
+              class: 'btn btn-primary' do
+                = link.content
 
 / PageContent
 .page_content
diff --git a/app/views/routes/show.html.slim b/app/views/routes/show.html.slim
index 6d2d4e90d..92a5080ae 100644
--- a/app/views/routes/show.html.slim
+++ b/app/views/routes/show.html.slim
@@ -8,17 +8,12 @@
   / Below is secundary actions & optional contents (filters, ...)
   .row.mb-sm
     .col-lg-12.text-right
-      - if @route_sp.any?
-        = link_to t('journey_patterns.index.title'), [@referential, @line, @route, :journey_patterns_collection], class: 'btn btn-primary'
-      - if @route.journey_patterns.present?
-        = link_to t('vehicle_journeys.actions.index'), [@referential, @line, @route, :vehicle_journeys], class: 'btn btn-primary'
-
-      = link_to t('vehicle_journey_exports.new.title'), referential_line_route_vehicle_journey_exports_path(@referential, @line, @route, format: :zip), class: 'btn btn-primary'
-
-      - if policy(@route).destroy?
-        = link_to referential_line_route_path(@referential, @line, @route), method: :delete, data: {confirm: t('routes.actions.destroy_confirm')}, class: 'btn btn-primary' do
-          span.fa.fa-trash
-          span = t('actions.destroy')
+      - @route.action_links.each do |link|
+        = link_to link.href,
+            method: link.method,
+            data: link.data,
+            class: 'btn btn-primary' do
+              = link.content
 
 / PageContent
 .page_content
diff --git a/app/views/routing_constraint_zones/show.html.slim b/app/views/routing_constraint_zones/show.html.slim
index f0c244387..1dad4f561 100644
--- a/app/views/routing_constraint_zones/show.html.slim
+++ b/app/views/routing_constraint_zones/show.html.slim
@@ -7,13 +7,12 @@
   / Below is secundary actions & optional contents
   .row
     .col-lg-12.text-right.mb-sm
-      - if policy(@routing_constraint_zone).update?
-        = link_to t('actions.edit'), edit_referential_line_routing_constraint_zone_path(@referential, @line, @routing_constraint_zone), class: 'btn btn-primary'
-
-      - if policy(@routing_constraint_zone).destroy?
-        = link_to referential_line_routing_constraint_zone_path(@referential, @line, @routing_constraint_zone), method: :delete, data: {confirm: t('routing_constraint_zones.actions.destroy_confirm')}, class: 'btn btn-primary' do
-          span.fa.fa-trash
-          span = t('actions.destroy')
+      - @routing_constraint_zone.action_links.each do |link|
+        = link_to link.href,
+            method: link.method,
+            data: link.data,
+            class: 'btn btn-primary' do
+              = link.content
 
 / PageContent
 .page_content
diff --git a/app/views/time_tables/show.html.slim b/app/views/time_tables/show.html.slim
index 2e71ebb9e..f596fd480 100644
--- a/app/views/time_tables/show.html.slim
+++ b/app/views/time_tables/show.html.slim
@@ -10,21 +10,12 @@
   / Below is secundary actions & optional contents (filters, ...)
   .row.mb-sm
     .col-lg-12.text-right
-      / - if policy(@time_table).create? && @referential.organisation == current_organisation
-      /   = link_to t('time_tables.actions.new'), new_referential_time_table_path(@referential), class: 'btn btn-primary'
-      - if @time_table.calendar
-        = link_to t('actions.actualize'), actualize_referential_time_table_path(@referential, @time_table), method: :post, class: 'btn btn-primary'
-
-      /- if policy(@time_table).create? && @referential.organisation == current_organisation
-      = link_to t('actions.combine'), new_referential_time_table_time_table_combination_path(@referential, @time_table), class: 'btn btn-primary'
-
-      - if policy(@time_table).duplicate?
-        = link_to t('actions.duplicate'), duplicate_referential_time_table_path(@referential, @time_table), class: 'btn btn-primary'
-
-      - if policy(@time_table).destroy?
-        = link_to referential_time_table_path(@referential, @time_table), method: :delete, data: {confirm:  t('time_tables.actions.destroy_confirm')}, class: 'btn btn-primary' do
-          span.fa.fa-trash
-          span = t('actions.destroy')
+      - @time_table.action_links.each do |link|
+        = link_to link.href,
+            method: link.method,
+            data: link.data,
+            class: 'btn btn-primary' do
+              = link.content
 
 / PageContent
 .page_content
diff --git a/app/views/workbenches/_filters.html.slim b/app/views/workbenches/_filters.html.slim
index 7c5055963..0aedbdd62 100644
--- a/app/views/workbenches/_filters.html.slim
+++ b/app/views/workbenches/_filters.html.slim
@@ -12,7 +12,7 @@
       = f.input :associated_lines_id_eq, as: :select, collection: @workbench.lines.includes(:company).order(:name), input_html: { 'data-select2ed': 'true', 'data-select2ed-placeholder': 'Indiquez une ligne...' }, label: false, label_method: :display_name, wrapper_html: { class: 'select2ed'}
 
     .form-group.togglable
-      = f.label @wbench_refs.human_attribute_name(:status), required: false, class: 'control-label'
+      = f.label Referential.human_attribute_name(:status), required: false, class: 'control-label'
       .form-group.checkbox_list
         = f.input :archived_at_not_null, label: ("<span>Conservé<span class='fa fa-archive'></span></span>").html_safe, as: :boolean, wrapper_html: { class: 'checkbox-wrapper' }
         = f.input :archived_at_null, label: ("<span>En préparation<span class='sb sb-lg sb-preparing'></span></span>").html_safe, as: :boolean, wrapper_html: { class: 'checkbox-wrapper' }
@@ -22,7 +22,7 @@
       = f.input :organisation_name_eq_any, collection: Organisation.order('name').pluck(:name), as: :check_boxes, label: false, label_method: lambda{|w| ("<span>#{w}</span>").html_safe}, required: false, wrapper_html: { class: 'checkbox_list' }
 
     .form-group.togglable
-      = f.label @wbench_refs.human_attribute_name(:validity_period), required: false, class: 'control-label'
+      = f.label Referential.human_attribute_name(:validity_period), required: false, class: 'control-label'
       .filter_menu
         = f.simple_fields_for :validity_period do |p|
           = p.input :begin_gteq, as: :date, label: t('simple_form.from'), wrapper_html: { class: 'date filter_menu-item' }, default: @begin_range, include_blank: @begin_range ? false : true
diff --git a/app/views/workbenches/show.html.slim b/app/views/workbenches/show.html.slim
index 77e670923..37c396b46 100644
--- a/app/views/workbenches/show.html.slim
+++ b/app/views/workbenches/show.html.slim
@@ -22,18 +22,47 @@
     - if @wbench_refs.any?
       .row
         .col-lg-12
-          = table_builder @wbench_refs,
-            { :name => 'name',
-              :status => Proc.new {|w| w.archived? ? ("<div class='td-block'><span class='fa fa-archive'></span><span>Conservé</span></div>").html_safe : ("<div class='td-block'><span class='sb sb-lg sb-preparing'></span><span>En préparation</span></div>").html_safe},
-              :organisation => Proc.new {|w| w.organisation.name},
-              :validity_period => Proc.new {|w| w.validity_period.nil? ? '-' : t('validity_range', debut: l(w.try(:validity_period).try(:begin), format: :short), end: l(w.try(:validity_period).try(:end), format: :short))},
-              :lines => Proc.new {|w| w.lines.count},
-              :created_at => Proc.new {|w| l(w.created_at, format: :short)},
-              :updated_at => Proc.new {|w| l(w.updated_at, format: :short)},
-              :published_at => ''},
-            [:show, :edit, :archive, :unarchive, :delete],
-            [:delete],
-            'table has-filter has-search'
+          .select_table
+            = table_builder_2 @wbench_refs,
+              [ \
+                TableBuilderHelper::Column.new( \
+                  key: :name, \
+                  attribute: 'name' \
+                ), \
+                TableBuilderHelper::Column.new( \
+                  key: :status, \
+                  attribute: Proc.new {|w| w.archived? ? ("<div class='td-block'><span class='fa fa-archive'></span><span>Conservé</span></div>").html_safe : ("<div class='td-block'><span class='sb sb-lg sb-preparing'></span><span>En préparation</span></div>").html_safe} \
+                ), \
+                TableBuilderHelper::Column.new( \
+                  key: :organisation, \
+                  attribute: Proc.new {|w| w.organisation.name} \
+                ), \
+                TableBuilderHelper::Column.new( \
+                  key: :validity_period, \
+                  attribute: Proc.new {|w| w.validity_period.nil? ? '-' : t('validity_range', debut: l(w.try(:validity_period).try(:begin), format: :short), end: l(w.try(:validity_period).try(:end), format: :short))} \
+                ), \
+                TableBuilderHelper::Column.new( \
+                  key: :lines, \
+                  attribute: Proc.new {|w| w.lines.count} \
+                ), \
+                TableBuilderHelper::Column.new( \
+                  key: :created_at, \
+                  attribute: Proc.new {|w| l(w.created_at, format: :short)} \
+                ), \
+                TableBuilderHelper::Column.new( \
+                  key: :updated_at, \
+                  attribute: Proc.new {|w| l(w.updated_at, format: :short)} \
+                ), \
+                TableBuilderHelper::Column.new( \
+                  key: :published_at, \
+                  attribute: '' \
+                ) \
+              ],
+              selectable: true,
+              links: [:show, :edit],
+              cls: 'table has-filter has-search'
+
+            = multiple_selection_toolbox([:delete])
 
           = new_pagination @wbench_refs, 'pull-right'