-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Премахване на protected_attributes #133
Comments
Дай малко контекст. |
Ами protected_attributes се ползва с цел mass assignment защитата да остане в моделите, понеже в rails 4 беше преместена в контролерите. Бих се наел да рефакторирам и да описваме позволените атрибути в контролерите. Има ли конкретна причина това да не е направено? |
Няма, трябвада се рефакторира. |
Аз имам едно наум към тази промяна. От една страна не съм напълно убеден, че контролерите са адекватно място за това описание, а от друга – не знам до каква степен решението ми харесва въобще. Уви, нямам ясен спомен как бих решил този проблем. Съветвам те да пробваш с един контролер и да отвориш pull request. Най-важния пиниз в цялата работа ще е controller spec-а – ако там стане каша, значи трябва да си мисли в друга посока. |
Бих се ангажирал да премахна protected_attributes. Предполагам никой не е имал време затова? Давате ли зелена светлина?
The text was updated successfully, but these errors were encountered: