Simplify method names in admin menu component

We no longer need to use the `menu_` prefix, since now these methods
aren't helper methods anymore, and so their names won't collide with
other helpers.
This commit is contained in:
Javi Martín
2020-06-20 14:37:23 +02:00
parent d501915954
commit f9e110aa9b
2 changed files with 23 additions and 23 deletions

View File

@@ -1,7 +1,7 @@
class Admin::MenuComponent < ApplicationComponent
private
def menu_moderated_content?
def moderated_content?
moderated_sections.include?(controller_name) && controller.class.parent != Admin::Legislation
end
@@ -10,50 +10,50 @@ class Admin::MenuComponent < ApplicationComponent
"hidden_budget_investments", "hidden_proposal_notifications"]
end
def menu_budgets?
def budgets?
controller_name.starts_with?("budget")
end
def menu_polls?
def polls?
controller.class.parent == Admin::Poll::Questions::Answers ||
%w[polls active_polls recounts results questions answers].include?(controller_name) &&
action_name != "booth_assignments"
end
def menu_booths?
def booths?
%w[officers booths shifts booth_assignments officer_assignments].include?(controller_name) ||
controller_name == "polls" && action_name == "booth_assignments"
end
def menu_profiles?
def profiles?
%w[administrators organizations officials moderators valuators managers users].include?(controller_name)
end
def menu_settings?
def settings?
controllers_names = ["settings", "tags", "geozones", "images", "content_blocks",
"local_census_records", "imports"]
controllers_names.include?(controller_name) &&
controller.class.parent != Admin::Poll::Questions::Answers
end
def menu_customization?
def customization?
["pages", "banners", "information_texts", "documents"].include?(controller_name) ||
menu_homepage? || menu_pages?
homepage? || pages?
end
def menu_homepage?
def homepage?
["homepage", "cards"].include?(controller_name) && params[:page_id].nil?
end
def menu_pages?
def pages?
["pages", "cards"].include?(controller_name) && params[:page_id].present?
end
def menu_dashboard?
def dashboard?
["actions", "administrator_tasks"].include?(controller_name)
end
def submenu_local_census_records?
def local_census_records?
controller_name == "local_census_records" ||
(controller_name == "imports" && controller.class.parent == Admin::LocalCensusRecords)
end