@@ -53,6 +53,11 class CustomField < ActiveRecord::Base | |||||
53 |
|
53 | |||
54 | CUSTOM_FIELDS_NAMES = CUSTOM_FIELDS_TABS.collect{|v| v[:name]} |
|
54 | CUSTOM_FIELDS_NAMES = CUSTOM_FIELDS_TABS.collect{|v| v[:name]} | |
55 |
|
55 | |||
|
56 | def field_format=(arg) | |||
|
57 | # cannot change format of a saved custom field | |||
|
58 | super if new_record? | |||
|
59 | end | |||
|
60 | ||||
56 | def set_searchable |
|
61 | def set_searchable | |
57 | # make sure these fields are not searchable |
|
62 | # make sure these fields are not searchable | |
58 | self.searchable = false if %w(int float date bool).include?(field_format) |
|
63 | self.searchable = false if %w(int float date bool).include?(field_format) |
@@ -57,6 +57,12 class CustomFieldTest < ActiveSupport::TestCase | |||||
57 | assert field.valid? |
|
57 | assert field.valid? | |
58 | end |
|
58 | end | |
59 |
|
59 | |||
|
60 | def test_should_not_change_field_format_of_existing_custom_field | |||
|
61 | field = CustomField.find(1) | |||
|
62 | field.field_format = 'int' | |||
|
63 | assert_equal 'list', field.field_format | |||
|
64 | end | |||
|
65 | ||||
60 | def test_possible_values_should_accept_an_array |
|
66 | def test_possible_values_should_accept_an_array | |
61 | field = CustomField.new |
|
67 | field = CustomField.new | |
62 | field.possible_values = ["One value", ""] |
|
68 | field.possible_values = ["One value", ""] |
General Comments 0
You need to be logged in to leave comments.
Login now