Skip to content

Instantly share code, notes, and snippets.

Show Gist options
  • Save aya-soft/3ae0dd78ad8dfeb2a7c2eaf3eb9a2808 to your computer and use it in GitHub Desktop.
Save aya-soft/3ae0dd78ad8dfeb2a7c2eaf3eb9a2808 to your computer and use it in GitHub Desktop.
Подобные методы лучше писать по-другому
def permit(params)
params.permit(:spare_id, :service_station_id, :installing_current_part_price, :time_of_installing_current_part, :labor_times_price,:blocked_at, :comment)
end
@aya-soft
Copy link
Author

  1. передавать параметром params не нужно, т.к. это метод и он может быть вызван из любого места в контроллере

  2. Название permit надо заменить на что-то более конкретное, т.к. мы тут пытаемся выхватить из params конкретный кусок данных относящийся к комментарию, так что лучше permit(params) переименовать в params_service_station_comment.

  3. Внутри метода лучше действовать так:
    params.require(:service_station_comment).permit(:name, ....)

  4. А вообще все это методы из специального gemа,встроенного в rails, называется strong_parameters.

Подробнее можно почитать тут: strong_parameters

@romhi
Copy link

romhi commented May 30, 2016

Сделал

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment