diff options
author | Yves Senn <yves.senn@gmail.com> | 2014-01-11 12:57:09 +0100 |
---|---|---|
committer | Yves Senn <yves.senn@gmail.com> | 2014-01-11 12:57:09 +0100 |
commit | 792c66f868b27cfd4b71c541202caae4bee1d172 (patch) | |
tree | a86bee9c33a7c6e750d35c31ee97e87db5905e3d /activesupport/lib/active_support/values | |
parent | caa981d88112f019ade868f75af6b5f399c244a4 (diff) | |
download | rails-792c66f868b27cfd4b71c541202caae4bee1d172.tar.gz rails-792c66f868b27cfd4b71c541202caae4bee1d172.tar.bz2 rails-792c66f868b27cfd4b71c541202caae4bee1d172.zip |
use enum labels as form values. Achieved by `_before_type_cast`.
Closes #13650, #13672
This is an alternate implementation to solve #13650. Currently form fields
contain the enum value (eg. "1"). This breaks because the setter `enum=`
expects the label (eg. "active").
ActiveRecord::Enum allows you to use labels in your application but store numbers.
We should make sure that all parts after AR are dealing with labels and not the
underlying mapping to a number.
This patch defines `_before_type_cast` on every enum column to return the label.
This method is later used to fetch the value to display in form fields.
I deliberately copied the implementation of the enum getter instead of delegating to it.
This allows you to overwrite the getter and for example return a `Value Object` but have it
still work for form fields.
Diffstat (limited to 'activesupport/lib/active_support/values')
0 files changed, 0 insertions, 0 deletions