Created
May 27, 2016 20:34
-
-
Save aya-soft/3ae0dd78ad8dfeb2a7c2eaf3eb9a2808 to your computer and use it in GitHub Desktop.
Подобные методы лучше писать по-другому
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
передавать параметром params не нужно, т.к. это метод и он может быть вызван из любого места в контроллере
Название permit надо заменить на что-то более конкретное, т.к. мы тут пытаемся выхватить из params конкретный кусок данных относящийся к комментарию, так что лучше permit(params) переименовать в params_service_station_comment.
Внутри метода лучше действовать так:
params.require(:service_station_comment).permit(:name, ....)
А вообще все это методы из специального gemа,встроенного в rails, называется strong_parameters.
Подробнее можно почитать тут: strong_parameters