ArgumentError: Неверное количество аргументов (задано 0, ожидается 1) при попытке передать параметры в application_controller

Question

В моем приложении я пытаюсь передать параметры из моего представления в мой application_controller. Метод в контроллере приложения позволяет только определенной части страницы быть видимой, если пользователь является менеджером. Я пробовал это довольно много способов, это то, что я имею сейчас после некоторой итерации.

application_controller:

def is_manager(manager_param)
  manager_param = params[:manager_param]
  return true if manager_param == "Manager"
end

и на мой взгляд:

<% if signed_in? && is_manager(current_user.user_type) %>

По сути, я хочу выполнить проверку типа пользователя, который может быть «Пользователь / Менеджер / и т. Д.», И, если он менеджер, показать определенную часть страницы. В настоящее время у меня есть такая функция для администратора, например:

application_controller:

def is_admin(userparam)
  return true if userparam == true
end

и в представлении:

<% if signed_in? && is_admin(current_user.admin) %>
# Stuff only displayed for admins here
<% end %>

Однако проверка администратора выполняется для логического значения, поэтому оно работает. Диспетчер 1 - это строка в базе данных, а не логическое значение.

Я уверен, что это что-то глупое и простое, поэтому любая помощь будет признательна! Спасибо

EDIT: полное сообщение об ошибке + трассировка

ArgumentError in StaticPagesController#dashboard
wrong number of arguments (given 0, expected 1)

    def is_manager(manager_param)
      manager_param = params[:manager_param]
      return true if manager_param == "Manager"
    end

Full Trace:

app/controllers/application_controller.rb:118:in `is_manager'
activesupport (5.1.4) lib/active_support/callbacks.rb:413:in `block in make_lambda'
activesupport (5.1.4) lib/active_support/callbacks.rb:197:in `block (2 levels) in halting'
actionpack (5.1.4) lib/abstract_controller/callbacks.rb:12:in `block (2 levels) in <module:Callbacks>'
activesupport (5.1.4) lib/active_support/callbacks.rb:198:in `block in halting'
activesupport (5.1.4) lib/active_support/callbacks.rb:507:in `block in invoke_before'
activesupport (5.1.4) lib/active_support/callbacks.rb:507:in `each'
activesupport (5.1.4) lib/active_support/callbacks.rb:507:in `invoke_before'
activesupport (5.1.4) lib/active_support/callbacks.rb:130:in `run_callbacks'
actionpack (5.1.4) lib/abstract_controller/callbacks.rb:19:in `process_action'
actionpack (5.1.4) lib/action_controller/metal/rescue.rb:20:in `process_action'
actionpack (5.1.4) lib/action_controller/metal/instrumentation.rb:32:in `block in process_action'
activesupport (5.1.4) lib/active_support/notifications.rb:166:in `block in instrument'
activesupport (5.1.4) lib/active_support/notifications/instrumenter.rb:21:in `instrument'
activesupport (5.1.4) lib/active_support/notifications.rb:166:in `instrument'
actionpack (5.1.4) lib/action_controller/metal/instrumentation.rb:30:in `process_action'
actionpack (5.1.4) lib/action_controller/metal/params_wrapper.rb:252:in `process_action'
activerecord (5.1.4) lib/active_record/railties/controller_runtime.rb:22:in `process_action'
actionpack (5.1.4) lib/abstract_controller/base.rb:124:in `process'
actionview (5.1.4) lib/action_view/rendering.rb:30:in `process'
actionpack (5.1.4) lib/action_controller/metal.rb:189:in `dispatch'
actionpack (5.1.4) lib/action_controller/metal.rb:253:in `dispatch'
actionpack (5.1.4) lib/action_dispatch/routing/route_set.rb:49:in `dispatch'
actionpack (5.1.4) lib/action_dispatch/routing/route_set.rb:31:in `serve'
actionpack (5.1.4) lib/action_dispatch/routing/mapper.rb:16:in `block in <class:Constraints>'
actionpack (5.1.4) lib/action_dispatch/routing/mapper.rb:46:in `serve'
actionpack (5.1.4) lib/action_dispatch/journey/router.rb:50:in `block in serve'
actionpack (5.1.4) lib/action_dispatch/journey/router.rb:33:in `each'
actionpack (5.1.4) lib/action_dispatch/journey/router.rb:33:in `serve'
actionpack (5.1.4) lib/action_dispatch/routing/route_set.rb:834:in `call'
warden (1.2.7) lib/warden/manager.rb:36:in `block in call'
warden (1.2.7) lib/warden/manager.rb:35:in `catch'
warden (1.2.7) lib/warden/manager.rb:35:in `call'
rack (2.0.3) lib/rack/etag.rb:25:in `call'
rack (2.0.3) lib/rack/conditional_get.rb:25:in `call'
rack (2.0.3) lib/rack/head.rb:12:in `call'
rack (2.0.3) lib/rack/session/abstract/id.rb:232:in `context'
rack (2.0.3) lib/rack/session/abstract/id.rb:226:in `call'
actionpack (5.1.4) lib/action_dispatch/middleware/cookies.rb:613:in `call'
activerecord (5.1.4) lib/active_record/migration.rb:556:in `call'
actionpack (5.1.4) lib/action_dispatch/middleware/callbacks.rb:26:in `block in call'
activesupport (5.1.4) lib/active_support/callbacks.rb:97:in `run_callbacks'
actionpack (5.1.4) lib/action_dispatch/middleware/callbacks.rb:24:in `call'
actionpack (5.1.4) lib/action_dispatch/middleware/executor.rb:12:in `call'
actionpack (5.1.4) lib/action_dispatch/middleware/debug_exceptions.rb:59:in `call'
web-console (3.5.1) lib/web_console/middleware.rb:135:in `call_app'
web-console (3.5.1) lib/web_console/middleware.rb:20:in `block in call'
web-console (3.5.1) lib/web_console/middleware.rb:18:in `catch'
web-console (3.5.1) lib/web_console/middleware.rb:18:in `call'
actionpack (5.1.4) lib/action_dispatch/middleware/show_exceptions.rb:31:in `call'
railties (5.1.4) lib/rails/rack/logger.rb:36:in `call_app'
railties (5.1.4) lib/rails/rack/logger.rb:24:in `block in call'
activesupport (5.1.4) lib/active_support/tagged_logging.rb:69:in `block in tagged'
activesupport (5.1.4) lib/active_support/tagged_logging.rb:26:in `tagged'
activesupport (5.1.4) lib/active_support/tagged_logging.rb:69:in `tagged'
railties (5.1.4) lib/rails/rack/logger.rb:24:in `call'
sprockets-rails (3.2.1) lib/sprockets/rails/quiet_assets.rb:13:in `call'
actionpack (5.1.4) lib/action_dispatch/middleware/remote_ip.rb:79:in `call'
actionpack (5.1.4) lib/action_dispatch/middleware/request_id.rb:25:in `call'
rack (2.0.3) lib/rack/method_override.rb:22:in `call'
rack (2.0.3) lib/rack/runtime.rb:22:in `call'
activesupport (5.1.4) lib/active_support/cache/strategy/local_cache_middleware.rb:27:in `call'
actionpack (5.1.4) lib/action_dispatch/middleware/executor.rb:12:in `call'
actionpack (5.1.4) lib/action_dispatch/middleware/static.rb:125:in `call'
rack (2.0.3) lib/rack/sendfile.rb:111:in `call'
railties (5.1.4) lib/rails/engine.rb:522:in `call'
puma (3.10.0) lib/puma/configuration.rb:225:in `call'
puma (3.10.0) lib/puma/server.rb:605:in `handle_request'
puma (3.10.0) lib/puma/server.rb:437:in `process_client'
puma (3.10.0) lib/puma/server.rb:301:in `block in run'
puma (3.10.0) lib/puma/thread_pool.rb:120:in `block in spawn_thread'

Answer 1

Welp, я понял это, я немного смутился. Я был прав во всем, что я делал, но я помещал def в before_action, а не как helper_method.

Финальный код:

Контроллер приложения:

helper_method :is_manager

def is_manager(manager_param)
  return true if manager_param == "Manager"
end

Вид:

<% if signed_in? && is_manager(current_user.user_type) %>

Answer 2

Попробуйте это

def is_manager(manager_param)
  "Manager" == manager_param
end

Что неправильно в вашем коде, так это то, что вы передаете current_user.user_type как manager_param, а затем делаете manager_param = params[:manager_param], который пытается присвоить значение типу пользователя, что, я думаю, вы не хотите делать.

Кроме того, вы можете игнорировать «вернуть истину, если», так как проверяемое условие действительно возвращает истину, если истина, и ложь, если ложь.

Кроме того, вы должны переместить эту логику в пользовательскую модель:

def is_manager?
  'Manager' == user_type
end

и тогда ваш помощник может быть

def is_manager?
  current_user.is_manager?
end

и вам не нужно передавать текущего пользователя, поскольку он уже доступен в этом контексте.